RuboCop拡張の読み込み方法をプラグイン方式に移行した by tyrrell-IH · Pull Request #9370 · fjordllc/bootcamp

tyrrell-IH · 2025-12-17T03:39:33Z

Issue

rubocopのpluginシステムを最新に書き換える #9309

概要

RuboCopの拡張用Gemの読み込み方式をrequireからpluginsへ変更しました。

それに合わせてRuboCopも最新のバージョン（1.82.1参考）へアップデートし、
RuboCop拡張rubocop-minitest rubocop-capybara rubocop-performance rubocop-railsもアップデートしました。

変更確認方法

chore/migrate-rubocop-pluginsをローカルに取り込む
bin/lintを実行しRuboCopの警告・エラー、slim-lintの警告・エラーが発生しないことを確認する。

参考

以下にRuboCop導入手順等を載せておきます。

量が多いので必ず目を通してもらう必要はないですが、実装に疑問点があった際によかったら参照してください。

RuboCop導入手順

導入手順

1. parserのアップデート

RuboCopを最新の1.82系にアップデートしようとするとparserの互換性の警告が出ます。

bundle update rubocop
[DEPRECATED] Platform :mingw, :x64_mingw is deprecated. Please use platform :windows instead.
Fetching gem metadata from https://rubygems.org/........
Resolving dependencies...
Could not find compatible versions

Because rubocop >= 1.82.0 depends on rubocop-ast >= 1.48.0, < 2.0
  and rubocop-ast >= 1.41.0 depends on parser >= 3.3.7.2,
  rubocop >= 1.82.0 requires parser >= 3.3.7.2.
So, because Gemfile depends on parser = 3.3.0.2
  and Gemfile depends on rubocop ~> 1.82,
  version solving has failed.

なので最初にparserのバージョンを3.3.7.2に指定しました。

2. RuboCopのアップデート

GemfileにRuboCopのバージョンを'~> 1.82'と指定してアップデートしました。
バージョンを固定しなくても最新にはアップデートできると思いますが、意図せずバージョンが上がらないように今回は固定しました。

bundle update rubocop
Fetching gem metadata from https://rubygems.org/........
Resolving dependencies...
Resolving dependencies...
Fetching prism 1.7.0
Fetching unicode-emoji 4.2.0
Using parser 3.3.7.2 (was 3.2.2.4)
Installing unicode-emoji 4.2.0
Installing prism 1.7.0 with native extensions
Using unicode-display_width 3.2.0 (was 2.6.0)
Fetching rubocop-ast 1.49.0 (was 1.30.0)
Installing rubocop-ast 1.49.0 (was 1.30.0)
Fetching rubocop 1.82.1 (was 1.59.0)
Installing rubocop 1.82.1 (was 1.59.0)
Bundle updated!
9 installed gems you directly depend on are looking for funding.
  Run `bundle fund` for details

拡張Gemのアップデート

特にバージョン等を指定せずに
bundle update rubocop-minitest rubocop-capybara rubocop-performance rubocop-rails
を実行してアップデートしました。

RuboCop（及び拡張Gem）のアップデートに伴い新たに発生した警告について

RuboCop及び拡張Gemのアップデートに伴いbin/lint(bundle exec rubocopとbundle exec slim-lint）実行時、RuboCop関連の警告25件、slim-lint関連の警告3件、合計28件の追加の警告が発生しました。

追加の警告

bundle exec rubocop

bundle exec rubocop
Inspecting 1032 files

省略

Offenses:

app/components/users/micro_reports/micro_report_component.rb:10:3: C: [Correctable] Rails/Delegate: Use delegate to define delegations.
  def comment_user
  ^^^
app/controllers/api/reactions_controller.rb:32:15: C: [Correctable] Style/RedundantParentheses: Don't use parentheses around a method call.
      users = (grouped_reactions[kind]&.map { |reaction| user_payload(reaction.user) }) || []
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/customer.rb:4:3: C: [Correctable] Rails/Delegate: Use delegate to define delegations.
  def retrieve(id)
  ^^^
app/models/subscription.rb:11:3: C: [Correctable] Rails/Delegate: Use delegate to define delegations.
  def retrieve(id)
  ^^^
lib/bootcamp/dumper.rb:8:11: C: [Correctable] Rails/Output: Do not write to stdout. Use Rails's logger if you want to log.
          puts <<~COURSE
          ^^^^
lib/bootcamp/dumper.rb:22:11: C: [Correctable] Rails/Output: Do not write to stdout. Use Rails's logger if you want to log.
          puts <<~LESSON
          ^^^^
lib/bootcamp/dumper.rb:37:11: C: [Correctable] Rails/Output: Do not write to stdout. Use Rails's logger if you want to log.
          puts <<~USER
          ^^^^
test/integration/api/users_test.rb:149:21: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
    assert_response 401
                    ^^^
test/integration/api/users_test.rb:160:21: C: [Correctable] Rails/HttpStatus: Prefer :bad_request over 400 to define HTTP status code.
    assert_response 400
                    ^^^
test/integration/scheduler/daily/auto_retire_test.rb:12:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/auto_retire_test.rb:17:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/auto_retire_test.rb:21:25: C: [Correctable] Rails/HttpStatus: Prefer :ok over 200 to define HTTP status code.
        assert_response 200
                        ^^^
test/integration/scheduler/daily/fetch_external_entry_test.rb:12:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/fetch_external_entry_test.rb:17:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/fetch_external_entry_test.rb:21:25: C: [Correctable] Rails/HttpStatus: Prefer :ok over 200 to define HTTP status code.
        assert_response 200
                        ^^^
test/integration/scheduler/daily/notify_certain_period_passed_after_last_answer_test.rb:12:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/notify_certain_period_passed_after_last_answer_test.rb:17:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/notify_certain_period_passed_after_last_answer_test.rb:21:25: C: [Correctable] Rails/HttpStatus: Prefer :ok over 200 to define HTTP status code.
        assert_response 200
                        ^^^
test/integration/scheduler/daily/notify_coming_soon_regular_events_test.rb:12:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/notify_coming_soon_regular_events_test.rb:17:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/notify_coming_soon_regular_events_test.rb:21:25: C: [Correctable] Rails/HttpStatus: Prefer :ok over 200 to define HTTP status code.
        assert_response 200
                        ^^^
test/models/concerns/mentioner_test.rb:14:19: C: [Correctable] Layout/TrailingWhitespace: Trailing whitespace detected.
      `@kananashi`
                  ^^^^
test/system/notification/assigned_as_checker_test.rb:15:3: C: [Correctable] Rails/Delegate: Use delegate to define delegations.
  def deliveries
  ^^^
test/system/notification/regular_events_test.rb:33:7: C: [Correctable] Layout/IndentationWidth: Use 2 (not -13) spaces for indentation.
      body = JSON.parse(request.body)
      ^^^^^^^^^^^^^
test/system/notification/reports_test.rb:131:1: C: [Correctable] Layout/EmptyLinesAroundMethodBody: Extra empty line detected at method body beginning.

1032 files inspected, 25 offenses detected, 25 offenses autocorrectable

bundle exec slim-lint

app/views/application/_global_nav.slim:49 [W] RuboCop: Style/RedundantParentheses: Don't use parentheses around a logical expression.

app/views/comments/_new_comment.html.slim:31 [W] RuboCop: Style/MultipleComparison: Avoid comparing a variable with multiple items in a conditional, use `Array#include?` instead.

app/views/practices/questions/index.html.slim:17 [W] RuboCop: Style/MultipleComparison: Avoid comparing a variable with multiple items in a conditional, use `Array#include?` instead.

リファレンスに従い修正し、以下の通りに動作確認しました。
動作確認を省略したものもあります

1. Rails/HttpStatus: 14件

警告

test/integration/api/users_test.rb:149:21: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
    assert_response 401
                    ^^^
test/integration/api/users_test.rb:160:21: C: [Correctable] Rails/HttpStatus: Prefer :bad_request over 400 to define HTTP status code.
    assert_response 400
                    ^^^
test/integration/scheduler/daily/auto_retire_test.rb:12:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/auto_retire_test.rb:17:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/auto_retire_test.rb:21:25: C: [Correctable] Rails/HttpStatus: Prefer :ok over 200 to define HTTP status code.
        assert_response 200
                        ^^^
test/integration/scheduler/daily/fetch_external_entry_test.rb:12:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/fetch_external_entry_test.rb:17:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/fetch_external_entry_test.rb:21:25: C: [Correctable] Rails/HttpStatus: Prefer :ok over 200 to define HTTP status code.
        assert_response 200
                        ^^^
test/integration/scheduler/daily/notify_certain_period_passed_after_last_answer_test.rb:12:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/notify_certain_period_passed_after_last_answer_test.rb:17:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/notify_certain_period_passed_after_last_answer_test.rb:21:25: C: [Correctable] Rails/HttpStatus: Prefer :ok over 200 to define HTTP status code.
        assert_response 200
                        ^^^
test/integration/scheduler/daily/notify_coming_soon_regular_events_test.rb:12:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/notify_coming_soon_regular_events_test.rb:17:23: C: [Correctable] Rails/HttpStatus: Prefer :unauthorized over 401 to define HTTP status code.
      assert_response 401
                      ^^^
test/integration/scheduler/daily/notify_coming_soon_regular_events_test.rb:21:25: C: [Correctable] Rails/HttpStatus: Prefer :ok over 200 to define HTTP status code.
        assert_response 200
                        ^^^

commit

27e1335

リファレンス

https://www.rubydoc.info/gems/rubocop/0.61.1/RuboCop/Cop/Rails/HttpStatus

動作確認

変更箇所は全てアサーションメソッドのexpectedの値だったので、

bin/rails test test/integration/api/users_test.rb
bin/rails test test/integration/scheduler/daily/auto_retire_test.rb
bin/rails test test/integration/scheduler/daily/fetch_external_entry_test.rb
bin/rails test test/integration/scheduler/daily/notify_certain_period_passed_after_last_answer_test.rb
bin/rails test test/integration/scheduler/daily/notify_coming_soon_regular_events_test.rb

上記テストが問題なく通ることを確認しました。

2.Rails/Delegate: 4件

commit

8553b9e

リファレンス

https://www.rubydoc.info/gems/rubocop/0.41.2/RuboCop/Cop/Rails/Delegate

動作確認

2-1. `app/components/users/micro_reports/micro_report_component.rb:10:3`

警告↓

app/components/users/micro_reports/micro_report_component.rb:10:3: C: [Correctable] Rails/Delegate: Use delegate to define delegations.
  def comment_user
  ^^^

MicroReportComponent#comment_userに対する単体のテストなし。

bootcamp/app/components/users/micro_reports/micro_report_component.html.slim

Line 8 in 90b5034

    
           = link_to user_path(comment_user), class: 'micro-report__title-link a-text-link' do

にてcomment_userを使用したリンクが正しく作成されているので、委譲処理が正しく動作していると判断しました。

2-2. `app/models/customer.rb:4:3`

警告↓

app/models/customer.rb:4:3: C: [Correctable] Rails/Delegate: Use delegate to define delegations.
  def retrieve(id)
  ^^^

Customer#retrieveに対する単体のテストなし。
Customer#retrieveは

bootcamp/app/models/user.rb

Lines 627 to 631 in e70c100

    
           def customer 
        
             return unless customer_id? 
        
             Customer.new.retrieve(customer_id) 
        
           end

User#customer内にて使用されている。

コンソール上で

rails c

user = User.find_by(login_name: "kyuukai")
user.customer.class 
=> Stripe::Customer

user.customer を呼び出すとStripe::Customer オブジェクトが返ってくることを確認。
delegateによってCustomer#retrieveからStripe::Customer.retrieve への委譲処理が正しく動作していると判断しました。

2-3. `app/models/subscription.rb:11:3`

警告↓

app/models/subscription.rb:11:3: C: [Correctable] Rails/Delegate: Use delegate to define delegations.
  def retrieve(id)
  ^^^

単体テストあり。

bootcamp/test/models/subscription_test.rb

Lines 6 to 9 in dedfce1

    
           test '#retrieve' do 
        
             VCR.use_cassette 'subscription/retrieve' do 
        
               subscription = Subscription.new.retrieve('sub_12345678') 
        
               assert_equal 'sub_12345678', subscription['id']

bin/rails test test/models/subscription_test.rb

が問題なく通ることを確認しました。

2-4. `test/system/notification/assigned_as_checker_test.rb:15:3`

警告↓

test/system/notification/assigned_as_checker_test.rb:15:3: C: [Correctable] Rails/Delegate: Use delegate to define delegations.
  def deliveries
  ^^^

テスト内で定義されている箇所の修正なので

bin/rails test test/system/notification/assigned_as_checker_test.rb

が問題なく通ることを確認しました。

3. test/system/notification/assigned_as_checker_test.rb: 2件

警告

rubocop

app/controllers/api/reactions_controller.rb:32:15: C: [Correctable] Style/RedundantParentheses: Don't use parentheses around a method call.
      users = (grouped_reactions[kind]&.map { |reaction| user_payload(reaction.user) }) || []
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

slim-lint

app/views/application/_global_nav.slim:49 [W] RuboCop: Style/RedundantParentheses: Don't use parentheses around a logical expression.

commit

09b38ac
f303952

リファレンス

https://www.rubydoc.info/gems/rubocop/0.41.2/RuboCop/Cop/Style/RedundantParentheses

動作確認

評価順が変わらないことはコードを見て判断できたため、動作確認は省略しました。

4. Layout/EmptyLinesAroundMethodBody: 1件

警告

test/system/notification/reports_test.rb:131:1: C: [Correctable] Layout/EmptyLinesAroundMethodBody: Extra empty line detected at method body beginning.

commit

06a2900

リファレンス

https://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Layout/EmptyLinesAroundMethodBody

動作確認

不要な空行を削除しただけなので、こちらも動作確認は省略しました。

5. Layout/IndentationWidth: 1件

警告

test/system/notification/regular_events_test.rb:33:7: C: [Correctable] Layout/IndentationWidth: Use 2 (not -13) spaces for indentation.
      body = JSON.parse(request.body)
      ^^^^^^^^^^^^^

commit

8f3df07

リファレンス

https://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Layout/IndentationWidth

動作確認

インデントの調整のみなので、こちらも動作確認は省略しました。

6. Layout/TrailingWhitespace: 1件

警告

test/models/concerns/mentioner_test.rb:14:19: C: [Correctable] Layout/TrailingWhitespace: Trailing whitespace detected.
      `@kananashi`
                  ^^^^

commit

40304ef

リファレンス

https://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Layout/TrailingWhitespace

動作確認

行末の不要な空白を削除しただけなので、動作確認は省略しました。

7. Rails/Output: 3件

警告

lib/bootcamp/dumper.rb:8:11: C: [Correctable] Rails/Output: Do not write to stdout. Use Rails's logger if you want to log.
          puts <<~COURSE
          ^^^^
lib/bootcamp/dumper.rb:22:11: C: [Correctable] Rails/Output: Do not write to stdout. Use Rails's logger if you want to log.
          puts <<~LESSON
          ^^^^
lib/bootcamp/dumper.rb:37:11: C: [Correctable] Rails/Output: Do not write to stdout. Use Rails's logger if you want to log.
          puts <<~USER
          ^^^^

commit

5b0bedc

リファレンス

https://www.rubydoc.info/gems/rubocop-rails/RuboCop/Cop/Rails/Output

対応

2025/1/14 チーム開発ミーティング夜の部にて、駒形さんより「ファイル毎削除して良い」との回答を得たのでファイルを削除しました。

8. Style/MultipleComparison: 2件

commit

671706a

リファレンス

https://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Style/MultipleComparison

動作確認

8-1. `app/views/comments/_new_comment.html.slim:31`

警告↓

app/views/comments/_new_comment.html.slim:31 [W] RuboCop: Style/MultipleComparison: Avoid comparing a variable with multiple items in a conditional, use `Array#include?` instead.

bootcamp/app/views/comments/_new_comment.html.slim

Lines 31 to 38 in 671706a

    
           - if mentor_login? && %w[Report Product].include?(commentable_type) && !commentable.try(:checked?) 
        
             .card-main-actions__item.is-only-mentor 
        
               button.a-button.is-sm.is-danger.is-block.js-comment-check-button(disabled=true) 
        
                 i.fa-solid.fa-check 
        
                 - if commentable_type == 'Report' 
        
                   | 確認OKにする 
        
                 - else 
        
                   | 合格にする

commentable_typeがReportもしくはProductの時に.card-main-actions__item.is-only-mentorクラスのボタンが表示されていることを確認しました。

8-2. `app/views/practices/questions/index.html.slim:17`

警告↓

app/views/practices/questions/index.html.slim:17 [W] RuboCop: Style/MultipleComparison: Avoid comparing a variable with multiple items in a conditional, use `Array#include?` instead.

bootcamp/app/views/practices/questions/index.html.slim

Lines 12 to 23 in 671706a

    
           nav.tab-nav 
        
             .container 
        
               ul.tab-nav__items 
        
                 li.tab-nav__item 
        
                   = link_to '全ての質問', practice_questions_path(@practice), 
        
                     class: "tab-nav__item-link #{%w[not_solved solved].include?(params[:target]) ? '' : 'is-active'}" 
        
                 li.tab-nav__item 
        
                   = link_to '解決済み', practice_questions_path(@practice, target: 'solved'), 
        
                     class: "tab-nav__item-link #{params[:target] == 'solved' ? 'is-active' : ''}" 
        
                 li.tab-nav__item 
        
                   = link_to '未解決', practice_questions_path(@practice, target: 'not_solved'), 
        
                     class: "tab-nav__item-link #{params[:target] == 'not_solved' ? 'is-active' : ''}"

クエリにtarget=not_solvedもしくはtarget=solvedがついていた場合、「全ての質問ボタン」が非active状態になっていることを確認しました。

_development__OS_X_Mountain_Lionをクリーンインストールするに関するQ_A___FBC

Screenshot

内部的な修正のためスクリーンショットなし。

Summary by CodeRabbit

リリースノート

リファクタリング
- 内部コードの委譲や条件表現の簡素化で可読性と保守性を向上
依存関係
- パーサーと静的解析ツールのバージョンを更新し設定方式を調整
テスト
- テスト内のHTTPステータス表記を標準的な記法に統一
その他
- 不要な内部ユーティリティを削除しプロジェクト整理を実施

_{✏️ Tip: You can customize this high-level summary in your review settings.}

coderabbitai · 2025-12-17T03:39:38Z

📝 Walkthrough

Walkthrough

RuboCop設定をrequireからpluginsへ移行し、Gemのバージョン更新を行いました。複数箇所で明示的メソッドをRailsのdelegateに置き換え、テストのHTTPステータスアサーションを数値から記号へ変更し、未使用のダンプユーティリティを削除しました。

Changes

Cohort / File(s)	Summary
RuboCop設定と依存更新 `\.rubocop.yml`, `Gemfile`	`.rubocop.yml`で`require:`→`plugins:`へ移行しプラグインを列挙。`parser`と`rubocop`のバージョン制約を更新。
delegateによるリファクタリング `app/components/users/micro_reports/micro_report_component.rb`, `app/models/customer.rb`, `app/models/subscription.rb`, `test/system/notification/assigned_as_checker_test.rb`	明示的メソッドを`delegate`へ置換（例: comment_user, retrieve, deliveries）。外部挙動は同等。
テストのステータス表記統一 `test/integration/api/users_test.rb`, `test/integration/scheduler/daily/...` (複数ファイル)	HTTPレスポンスアサーションを数値 (401,200) からシンボル (`:unauthorized`, `:ok`) に変更。
ビューの条件式簡潔化 `app/views/application/_global_nav.slim`, `app/views/comments/_new_comment.html.slim`, `app/views/practices/questions/index.html.slim`	条件判定を`include?`等で簡潔化し、文字列補間内の論理式を整理。表示ロジックに変更なし。
コントローラの小さなスタイル調整 `app/controllers/api/reactions_controller.rb`	不要な括弧を削除するなどの軽微なスタイル修正。
不要ユーティリティ削除 `lib/bootcamp/dumper.rb`	Bootcamp::Dumperファイルを完全削除（dump_categories/dump_practices/dump_usersの除去）。
テストのホワイトスペース調整 `test/models/concerns/mentioner_test.rb`, `test/system/notification/regular_events_test.rb`, `test/system/notification/reports_test.rb`	インデント・空行・末尾スペースの微修正のみ。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested reviewers

ryufuta
komagata

Poem

🐰 プラグインで跳ねて設定変わり、
委譲でぴょんとメソッド消え、
テストは記号で整いし、
古いダンプは穴へぽとん、
ふわふわコード、今日も健やかに 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	RuboCop拡張の読み込み方法をプラグイン方式に移行したという変更の主要内容を簡潔に説明しており、changeset全体の中核を捉えている。
Description check	✅ Passed	PR説明がテンプレートの主要セクション（Issue、概要、変更確認方法）を完備し、詳細な背景、動作確認、参考資料も含まれている。

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch chore/migrate-rubocop-plugins

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

tyrrell-IH · 2026-01-26T02:31:44Z

@ryufuta
リュウさんにレビュー依頼するのは2回目で申し訳ないんですが、よかったらレビューしていただけないでしょうか？

ryufuta · 2026-01-26T12:25:33Z

@tyrrell-IH
承知しました。今週中を目処にレビューします👍

ryufuta

@tyrrell-IH
コミットと説明が丁寧でわかりやすかったです。
Approveします。

tyrrell-IH · 2026-01-29T23:47:41Z

@ryufuta
ありがとうございました！

@komagata
チームメンバーのレビューが終わりましたのでレビューお願いします🙏

komagata

確認させていただきました。OKです〜🙆‍♂️

tyrrell-IH force-pushed the chore/migrate-rubocop-plugins branch from 744d400 to 3a028c4 Compare December 17, 2025 03:43

tyrrell-IH force-pushed the chore/migrate-rubocop-plugins branch 2 times, most recently from 6b091ff to 2d85cf2 Compare January 13, 2026 11:09

tyrrell-IH force-pushed the chore/migrate-rubocop-plugins branch from 2d85cf2 to 24b7378 Compare January 20, 2026 02:03

tyrrell-IH marked this pull request as ready for review January 20, 2026 06:46

github-actions Bot requested a review from komagata January 20, 2026 06:46

tyrrell-IH force-pushed the chore/migrate-rubocop-plugins branch from f96289f to 671706a Compare January 23, 2026 10:26

tyrrell-IH requested a review from ryufuta January 26, 2026 02:32

ryufuta assigned tyrrell-IH Jan 26, 2026

ryufuta approved these changes Jan 29, 2026

View reviewed changes

komagata force-pushed the main branch from f3c8a2f to 5b7af68 Compare January 31, 2026 04:03

tyrrell-IH added 12 commits January 31, 2026 20:30

rubocopのバージョンを1.82系にアップデート

7df4f70

Rubocop拡張Gemも併せてアップデート

a278ed6

RuboCop拡張の読み込み方法をrequireからplugins方式に移行した

badd514

Rubocop指摘事項: ステータスコードの指定を数字ではなくシンボルに変更

00db633

Rubocop指摘事項: メソッド呼び出しを委譲しているだけの箇所はdelegateを使って実装する

9d7ed30

Rubocop指摘事項: 不要な括弧を削除

46a0760

Rubocop指摘事項: 不要な空行を削除

669146e

Rubocop指摘事項: インデント幅を2スペースに修正

cf3dcd9

Rubocop指摘事項: 不要な行末の空白を削除

dd9c267

Rubocop指摘事項: Rails.loggerを使うべきとの指摘のファイルについて、削除可能だったので削除

8a2df42

slim-lint指摘事項: 不要な括弧を削除

0516ef0

slim-lint指摘事項: 条件式の中で変数を複数の値と比較するのを避け、include?を使用した

977fb03

tyrrell-IH force-pushed the chore/migrate-rubocop-plugins branch from 671706a to 977fb03 Compare January 31, 2026 11:31

komagata approved these changes Feb 9, 2026

View reviewed changes

komagata merged commit 9b725d1 into main Feb 9, 2026
6 checks passed

komagata deleted the chore/migrate-rubocop-plugins branch February 9, 2026 00:28

github-actions Bot mentioned this pull request Feb 9, 2026

Release 2026-02-12 13:50:56 +0000 #9623

Merged

11 tasks

coderabbitai Bot mentioned this pull request Apr 27, 2026

Flaky testを安定化する #9950

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

RuboCop拡張の読み込み方法をプラグイン方式に移行した#9370

RuboCop拡張の読み込み方法をプラグイン方式に移行した#9370
komagata merged 12 commits intomainfrom
chore/migrate-rubocop-plugins

tyrrell-IH commented Dec 17, 2025 •

edited

Loading

Uh oh!

coderabbitai Bot commented Dec 17, 2025 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Suggested reviewers

Poem

Uh oh!

tyrrell-IH commented Jan 26, 2026 •

edited

Loading

Uh oh!

ryufuta commented Jan 26, 2026

Uh oh!

ryufuta left a comment

Uh oh!

tyrrell-IH commented Jan 29, 2026

Uh oh!

komagata left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	def customer
	return unless customer_id?

	Customer.new.retrieve(customer_id)
	end

	test '#retrieve' do
	VCR.use_cassette 'subscription/retrieve' do
	subscription = Subscription.new.retrieve('sub_12345678')
	assert_equal 'sub_12345678', subscription['id']

	- if mentor_login? && %w[Report Product].include?(commentable_type) && !commentable.try(:checked?)
	.card-main-actions__item.is-only-mentor
	button.a-button.is-sm.is-danger.is-block.js-comment-check-button(disabled=true)
	i.fa-solid.fa-check
	- if commentable_type == 'Report'
	\| 確認OKにする
	- else
	\| 合格にする

	nav.tab-nav
	.container
	ul.tab-nav__items
	li.tab-nav__item
	= link_to '全ての質問', practice_questions_path(@practice),
	class: "tab-nav__item-link #{%w[not_solved solved].include?(params[:target]) ? '' : 'is-active'}"
	li.tab-nav__item
	= link_to '解決済み', practice_questions_path(@practice, target: 'solved'),
	class: "tab-nav__item-link #{params[:target] == 'solved' ? 'is-active' : ''}"
	li.tab-nav__item
	= link_to '未解決', practice_questions_path(@practice, target: 'not_solved'),
	class: "tab-nav__item-link #{params[:target] == 'not_solved' ? 'is-active' : ''}"

Conversation

tyrrell-IH commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue

概要

変更確認方法

参考

RuboCop導入手順

1. parserのアップデート

2. RuboCopのアップデート

拡張Gemのアップデート

RuboCop（及び拡張Gem）のアップデートに伴い新たに発生した警告について

警告

commit

リファレンス

動作確認

commit

リファレンス

動作確認

2-1. app/components/users/micro_reports/micro_report_component.rb:10:3

2-2. app/models/customer.rb:4:3

2-3. app/models/subscription.rb:11:3

2-4. test/system/notification/assigned_as_checker_test.rb:15:3

警告

commit

リファレンス

動作確認

警告

commit

リファレンス

動作確認

警告

commit

リファレンス

動作確認

警告

commit

リファレンス

動作確認

警告

commit

リファレンス

対応

commit

リファレンス

動作確認

8-1. app/views/comments/_new_comment.html.slim:31

8-2. app/views/practices/questions/index.html.slim:17

Screenshot

Summary by CodeRabbit

リリースノート

Uh oh!

coderabbitai Bot commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Suggested reviewers

Poem

Uh oh!

tyrrell-IH commented Jan 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ryufuta commented Jan 26, 2026

Uh oh!

ryufuta left a comment

Choose a reason for hiding this comment

Uh oh!

tyrrell-IH commented Jan 29, 2026

Uh oh!

komagata left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

tyrrell-IH commented Dec 17, 2025 •

edited

Loading

2-1. `app/components/users/micro_reports/micro_report_component.rb:10:3`

2-2. `app/models/customer.rb:4:3`

2-3. `app/models/subscription.rb:11:3`

2-4. `test/system/notification/assigned_as_checker_test.rb:15:3`

8-1. `app/views/comments/_new_comment.html.slim:31`

8-2. `app/views/practices/questions/index.html.slim:17`

coderabbitai Bot commented Dec 17, 2025 •

edited

Loading

tyrrell-IH commented Jan 26, 2026 •

edited

Loading