Upgrade to Pro — share decks privately, control downloads, hide ads and more …

CodeRabbitと過ごした1ヶ月 ─ AIコードレビュー導入で実感したチーム開発の進化

CodeRabbitと過ごした1ヶ月 ─ AIコードレビュー導入で実感したチーム開発の進化

mitohato14

April 30, 2025
Tweet

More Decks by mitohato14

Other Decks in Technology

Transcript

  1. 会員数 MAU 機能 DAU で手に入れよう モバイルアプリ版 6 万 カジュアル ゲーム

    ポイント 管理 万 (会員IDベース(Web含む)) 10〜 13 万 5〜 6 通知 送客 広告 59 ※2025年02月25日時点
  2. © DMM コードレビュー 1. Pull Request作成 2. Pull Requestについて説明 3.

    少ない時間でレビュー 4. ミーティング 5. 落ち着いた頃にレビュー 6. 細かい指摘点 7. 再レビュー 8. … 9. マージ 16
  3. © DMM 細かい指摘 何度も細かいことを指摘するのは双方に負担大 • レビュアー • 重箱の隅をつつくような指摘ばかりになってしまう • 細かい部分が気になって大枠で見れない

    • 時間が取られてしまう • レビュイー • 見落としが多い • 細かいところで時間取らせてしまい申し訳ない • マージまでに時間がかかる 20
  4. © DMM CodeRabbit AIを活用したコードレビュー支援ツール • PR概要生成 • コードレビュー • 改善提案

    • シーケンス図の生成 • 類似リポジトリの列挙 • チャット機能 • 過去のやりとりも反映 • リポジトリ全体を考慮したレビュー 28
  5. © DMM 嬉しかったこと • 変更内容の要約 • どのようなファイルにどんな変更が入ったかわかりやすい • 実装ミスに気づきやすい •

    シーケンス図の生成 • レビュアーのコード理解促進 • チャット形式でCodeRabbitとやりとりできる • 指摘について質問したりプロダクト事情を伝えたり • レビューが早い • メンバーに見てもらう前に簡単な内容を修正できる • ポエム • ふふっと柔らかい気持ちになる
  6. © DMM 作成からマージまでの時間 指標 時間( h) 備考 平均 約51.6時間 約2.15日

    中央値 約24.2時間 約1.01日 最大 約596.7時間 約24.9日 最小 約0.44時間 約26分 指標 時間( h) 備考 平均 約67.6時間 約2.82日 中央値 約14.4時間 約0.60日 最大 約615.9時間 約25.66日 最小 約0.04時間 約2.6分(0.002日) 導入前 導入後
  7. © DMM 作成から初レビューまでの時間 指標 時間( h) 備考 平均 約0.0023時間 約8秒

    中央値 約0.0022時間 約8秒 最大 約0.0089時間 約32秒 最小 約0.0014時間 約5秒 指標 時間( h) 備考 平均 約205.7時間 約8.6日 中央値 約17.1時間 約17時間7分 最大 約502.9時間 約20.9日 最小 約0.45時間 約27分 導入前 導入後 ※CIでのLintコメントは除く
  8. © DMM メンバーの声 気になること • プッシュのたびに実行され、指摘内容の重複がある • 設定変更検討の余地 • 提案をそのまま受け入れるとエラーになる

    • suggest形式をそのまま受け入れない • 返信で後続対応しますといっても同じ指摘してくれる • TODOコメント書けば良い
  9. © DMM Kotlin/Android特有の指摘 • paddingの書き方指摘 • 複数のpadding設定→1つのpadding • 好みもあるがきちんとComposeを理解している •

    テストコードで非同期処理はrunTestを推奨 • AndroidViewのfactoryとupdateの挙動の違い • ダークモード対応の色指定をUpdateに
  10. © DMM 課題への結果 細かい指摘が多くなりがち → CodeRabbitが実施 テストのシナリオケースとテストコードにズレ → CodeRabbitが検知 細かい部分を見落としがち

    → 違和感をCodeRabbitが検知 レビュー時間が大きく取られる → 一部解決、ロジックや仕様などより重要な部分に時間をかけられる レビュー品質がレビュアーに大きく依存 → 一定のクオリティは担保(要ブラッシュアップ) 64