「おまえは今までレビューしたプルリクの数をおぼえているのか？」

こんにちは、kanaです。社内ではpixivというサービスでPHPとTypeScriptとVim scriptを書く仕事をしています。今日はpixivの開発におけるコードレビューの話をします。

問題

pixivは昨年でサービス開始から10周年を迎えました。サービス開始当初と比較すると山のように新しい機能や画面が増えています。なのでpixivのコードベースは巨大です。PHPファイルだけでも5000個以上あります。

kana@pixiv ~/pixiv (2) [master]
^-^)/> git ls-files '*.php' | wc -l
5555

これだけの数のファイルを全て把握するのは無理です。なので、各々の開発者が自由にコードを書いて混沌にならないよう、設計方針を始めとして各種コーディング規約が整備されており、秩序を保っています。

とはいえ誰もが常に完璧なコードを書けるわけではありません。なので何か変更をmasterにマージする為には、少なくとも2人のコードレビューを受けてLGTMを貰うルールになっています。これは、

レビューでの指摘漏れを減らして問題を事前に食い止め易くする
自分がレビューで気付けなかったところを把握して、次回からより良いレビューができるようにする
各自に少しづつpixiv全体のコードを把握してもらって、いざ障害が起きた時に誰かが直ぐに対応できる状態にする

という理由があります。

ところで、pixivの開発に長く携わっているとpixivの様々な部分の実装やインフラ構成や障害事例への理解が進んでいきます。そうすると、

「この分野はkanaさんが詳しいからkanaさんにレビューをお願いしよう」
「この機能はkanaさんが作ったからkanaさんに確認してもらおう」
「このデータ構成どうなってるのか謎だ……昔から居るkanaさんに相談しよう」

と周囲の開発者から頼られるようになります。毎年新卒エンジニアがどんどん増えているので、それに比例して頼られる回数も増えます。

頼られるのは嬉しい事です。ただ、あまりに多くのレビュー依頼が飛んでくるようになると、相対的に自分の作業時間が減って困ります。

自分が忙しい時は他の適切なエンジニアにレビューを代わってもらいます。長期的には頼れるエンジニアを増やして自分へのレビュー依頼を減らしたいので、各自の興味分野を勘案してレビュー依頼を再割り当てするようにもしています。

とはいえ、短期的にも何か効果を上げられる施策は無いものでしょうか。

設計

あれこれ頭を捻った結果、

本当に自分へのレビュー依頼が多数飛んできてるのか可視化する
可視化により「自分は忙しいから他の人にレビューを依頼してね」オーラを出してみる

という事ができれば良さそうです。

ピクシブでは社内のコミュニケーションツールにSlackを利用しています。なのでSlackの自分のステータスに自分へのレビュー依頼数を出せば忙しさオーラを出せます。具体的には以下のようなイメージです:

Slack標準のemojiだと 0 (:zero:) 1 (:one:) …… 9 (:nine:) 10 (:keycap_ten:) まで数えられます。 11以上になった場合は「とにかく沢山レビューしてるからもう勘弁して」という気持ちを込めて ∞ (:infinity:)にしておくと良さそうです(∞ は標準のemojiには無かったので追加しました)。

ただ、これだけだとカウントアップし続けて直ぐに ∞ になってしまうので、日付を跨いだらカウントをリセットする必要があります。これを判定する為には「最後にステータスを変更した日付」を何処かに記録しておく必要があります。ちゃんと記録するのは煩雑なので、ここはステータスの中に日付を表す文字列を埋め込んで済ます事にしましょう。