コードレビューをなくして見えたもの

https://zenn.dev/henteko/articles/b76b15922cdf72

Transcript

1. ゆるAI勉強会 #2 コードレビューを なくして見えたもの henteko
2. 自己紹介 今井 健太 henteko 株式会社デプロイゲート 共同創業者 / 取締役CPO ミソラボ 代表 Podcastを通じた技術広報の支援 @henteko07 ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 2
3. Podcastもやってます 聞いてください くわラジ 八百万のOSS ぼくらの戦略論 スーパーエンジニアに質問して 深掘るゆるい技術雑談 毎回OSSを取り上げて 技術を深掘り 投資家と編集者が語る 戦略的な考え方 ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 3
4. 02 コードレビューを なくした ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 4
5. 人間のコードレビューを やめた ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 5
6. コードの正しさは AIが見る AIのコードレビューは、すでに人間を上回っている ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 6
7. 人が見るのは AIへの指示が 正しいか ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 7
8. 人が関わる場所を 3つ用意した ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 8
9. 01 書く 実装前に、人がコンセプトを書く ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 9
10. 02 見る 大きな変更は、実装前にチームで確認する ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 10
11. 03 共有する 実装後のズレを、開発定例で共有する ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 11
12. ① 書く ② 見る ③ 共有する 実装前に人がコンセプトを書く 実装前にチームで確認する 実装後のズレを定例で共有する この3つがあれば もっと高頻度に 改善を回せる と、考えた ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 12
13. 社内ツール Ito 4〜5人のチーム ／ マージ ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの ≒ リリース ／ 半年運用 13
14. 03 なくした結果 ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 14
15. 人のApproveが付いたPR 97% → 数% ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 15
16. 遅いPRがマージされるまで 2.8日 → 23時間 ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 16
17. 同じ4人で、週のマージ数 4.2本 → 5.5本 ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 17
18. hotfix・revertは 増えていない ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 18
19. 高頻度に 回せた ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 19
20. 04 課題点 ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 20
21. 人の関与が 消えていった ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 21
22. ① 書く ／ コンセプト作成 → 実装開始（中央値） 2分 人が考えて書くには、短すぎる ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 22
23. 実際に書いていたのは AI コンセプト本文の64%が、AIの事前調査 ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 23
24. ② 見る LGTM だけ ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 24
25. 事前確認した40件のうち 差し戻し 0件 ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 25
26. ③ 共有する 確認されなかった ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 26
27. 共有するはずの245件のうち 確認待ちのまま 59% ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 27
28. ① 書く ② 見る ③ 共有する ✗ AIが書いた ✗ LGTMだけ ✗ 確認されなかった 人の関与が ゼロに近づいた ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 28
29. AIが埋められる場所から 人はいなくなる ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 29
30. 唯一効いたのは コードから見えない 外部事情 ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 30
31. 人にしか出せない情報を 求める場所が なかった ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 31
32. 仕組みとして 人の判断を 明確に含める 必要がある ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 32
33. 05 今後は こうしてみる ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 33
34. 人は 判断に集中する それ以外は AI に任せる ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 34
35. ① 書く AIが下書きし 人には判断を聞く ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 35
36. 判断の欄は AIが埋めない ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 36
37. ② 見る 判断だけを見る この判断でいいか ／ コードから見えない外部事情が抜けていないか ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 37
38. ③ 共有する 全てを必ず 共有する 開発定例で共有するまで、完了にしない ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 38
39. 見るべきは コードでも AIの文章でもなく 人の判断 ゆるAI勉強会 #2 ／ コードレビューをなくして見えたもの 39
40. 詳細はZennに 書いています コードレビューを なくして見えたもの zenn.dev/henteko/ articles/b76b15922cdf72 AIが起点の改善ループでの考え方も書いています 40