1. はじめに
長年エンジニアとして様々なチームで開発をしてきました。中にはとてもワクワクするプロジェクトもありましたが、なぜか納品のタイミングになると一気にモチベーションが下がってしまうことがありました。
当初は「自分が繊細すぎるのかもしれない」「もっと割り切るべきなのかも」と思っていました。違和感を抱えつつも「そういうものなのかもしれない」と受け入れながら進んでいたこともありましたが、今振り返ると、そこには見過ごしてはいけないサインがあったように思います。
でもある時ふと、
「なぜある程度の経験を積んでも、レビューに対して嫌だと感じてしまうことがあるのだろう?」
「逆に、厳しいレビューでも素直に受け入れられるケースとの違いは何だろう?」
そう自問したことがきっかけで、徐々にある共通点に気付き始めました。
それは、レビューで指摘される**「内容」ではなく、「伝え方」や「態度」**によって、自信や集中力が削がれていたということです。
本記事では、精神的な負担となりうるレビューのパターンとその背景にある文化的要因、そしてそれに対してどう向き合えばよいかを考えてみます。
2. トリガーとなった出来事
レビューの内容そのものよりも、それがどのように伝えられるかは受け手に大きな影響を与えます。例えば次のようなコメントを見たことはないでしょうか?
「要対応であることは認識されてますでしょうか 👀?」
「まだ直ってなさそうですー」
このような表現は一見丁寧に見えますが、実際には相手の理解力や注意力に疑問を投げかけているように感じられ、受け手の自己肯定感を下げてしまう場合があります。
また、次のような状況もあります:
- API変更が事前共有なく突然レビューで示される
- 「お願いします」と指示的な口調で対応を求められる
- バグの発見を「軽く」扱い、真剣に受け止めない
本来であれば、仕様の解釈や認識違いはPRコメントではなく、チャットや短いミーティングで確認するのが適切かもしれません。レビューは本来「一緒に品質を上げるための場」のはずです。しかし、指摘の仕方次第では信頼関係やモチベーションに悪影響を与えることがあります。
3. 観察された非建設的なパターン
🔸 パフォーマンス的な丁寧さ
「深夜作業お疲れ様です!無理はしないでくださいね🥹」
気遣っているようで、レビュー対象とは関係のない感情表現が過剰だと、逆に表面的な礼儀として受け取られることがあります。
🔸 暗にスキルを疑う表現
「この部分は対応が必要だと思うのですが、認識されていますか 👀?」
こうした表現は「あなたはこれに気づいていないのでは?」というニュアンスを含み、暗に能力不足をほのめかしていると感じさせる場合があります。
🔸 感情的なリアクション
「/api/module...? のリクエストが発生してそうです🥹🥹🥹🥹」
技術的な課題に対して感情的な絵文字を多用すると、バグの指摘が個人への非難のように感じられることがあります。
🔸 あいまいなコメント
「直ってなさそうですー」
どこがどう直っていないのか具体性がないまま、あいまいな疑念だけが残されると、受け手は混乱します。
🔸 合意なき変更の押し付け
「このAPIに新たに項目を追加し、名称も変更しました。今後はこちらを使うようお願いします」
このような表現は丁寧な依頼のようでありながら、事前の相談なく設計上の判断がなされている場合には、事実上の一方的な指示となってしまいます。
特に、すでに別の実装が進んでいる中でのこうした変更指示は、レビューの場で設計方針がひっくり返されるような印象を与えかねません。本来は、こうした設計レベルの調整はレビュー前に共有・合意形成されていることが望ましいでしょう。特に、本来の要件に基づいて機能を実装・完成させたタイミングで、PRコメントにて新たな仕様や設計変更が提示され、それに従うことを求められると、成果物の評価というより、納品を止めるための後出し仕様に見えてしまうこともあります。
4. 組織的な背景にあるバイアス
いくつかのチームでは、PRは「完成してから出すもの」であり、「修正コメントが多い=品質が低い」とされる文化があります。
このような環境では、以下のような誤った認識が広がりがちです:
- コメント数が多い人=熱心なレビュー担当者
- 指摘が多いPR=ダメな実装
- 丁寧な言葉遣い=良いコミュニケーション
しかし、実際には以下のような問題も発生しています:
- コメントの多くが軽微なNITやスタイルに偏っている
- 機能や設計の根拠には触れていない
- フィードバックの質が評価されていない
このような文化は、レビューに感情を持ち込んだり、「目立つためにレビューする」行動を助長してしまうリスクがあります。
5. 冷静に、建設的に対応する方法
ここでは、感情的な反応を避けつつ、プロフェッショナルに対応するための具体的なアプローチを紹介します:
✅ 冷静かつ事実ベースで返答する
「ご指摘ありがとうございます。APIの仕様に合わせて修正しました。」
✅ あいまいな指摘には明確化を求める
「ご指摘の内容について、具体的な期待値を教えていただけますか?」
✅ スコープ外の要求は切り分ける
「こちらは別チケットで対応した方が良さそうです。本PRでは当初の要件に集中します。」
✅ 過度な謝罪は不要
「ハードコーディングされていたため、APIの値を参照するよう修正しました。」
✅ コメントに対して時間内で対応する姿勢を示す
「こちらの対応は明日の作業時間内に実施いたします。」
✅ 指摘対応の実績を可視化する
- 「指摘50件中、24時間以内に47件対応」
- 「テストカバレッジ:663/663パス」
- 「仕様外の変更なし」
✅ チームとしてのレビュー方針を明確化する
「レビューの観点や基準を整理すると、対応もスムーズになると思います。」
6. おわりに
レビューは単なるバグの指摘ではなく、チームの信頼やコラボレーションを築く重要な場です。
もし、レビューで強い違和感や精神的な負担を感じたなら、それは単なる「自分の感受性」のせいではないかもしれません。
指摘内容を冷静に整理し、事実ベースで対応し、それでもなお繰り返し否定的・曖昧・過剰な指摘が続く場合、課題はあなた個人ではなく、組織や文化にある可能性を検討してもよいでしょう。
👉 高いパフォーマンスは、健全なメンタルと環境によって支えられます。
レビューが成長や品質向上のための建設的な時間になるよう、チーム全体で見直していくことが大切です。
🔽 付録:よくある非建設的なレビュー表現と改善案
| よくある表現 | 問題点 | より良い表現例 |
|---|---|---|
| 「認識されていますか?👀」 | 暗にスキルを疑う | 「こちらも対応が必要そうです」 |
| 「直ってなさそうですー」 | あいまいで不安を煽る | 「◯◯の挙動が期待と異なっているようです」 |
| 「🥹🥹🥹」 | 感情的・プレッシャー | 絵文字を使わず淡々と伝える |
| 「この変更をお願いします」 | 設計変更が事前共有なく、一方的に指示される印象を与える | 「この設計変更について、影響範囲など相談させてください」 |
| デザイン細部への強調 | スコープ外や責任転嫁の印象 | 「デザインに合わせるか確認中です」 |
他にも似たような体験がある方は、ぜひコメントで教えてください。
👉 もしよろしければ、どんなレビューのパターンに困ったことがあるか、それにどう対応されたかなども共有していただけると嬉しいです。さまざまな経験が集まることで、この記事を読んでいる方にとっても参考になるヒントが増えるかもしれません。
Comments
私自身も、日々たくさんのレビューを受ける中で対応に困ったり、精神的な負担を感じたりすることは多々あるので、記事に書かれているモヤモヤする気持ち自体はとてもよく分かります。ただ、私自身の場合は、そうした困りごとの多くが「自分自身の成果物の質の低さ」からきていると実感しています。そのため、どうすれば品質を向上できるかをレビュアーに相談し、フィードバックをいただきながら日々励むようにしています。
そうした経験を踏まえると、CTOを経験された方の視点としては、当事者意識に欠ける「イチ作業者」的なスタンスに偏ってしまっているのかな、とも感じました。
レビューで疑問を投げかけられてしまうのって、多くの場合、提出された実装の品質や設計そのものに何かしらの課題があるからですよね。そうした根本的な原因から目を背けて、コミュニケーションの「伝え方」ばかりを問題視したり、数字を使って防衛するようなハックに走ってしまうのは、プロフェッショナルとしての自分ごと化が少し足りていないように見えてしまいます。
指摘されたことだけを淡々とこなして、まるで外注の作業員さんのように境界線を引いてしまうカルチャーは、チームにとって健全ではないと思います。レビューを単なる「タスク消化の場」にしないためにも、まずは自分がつくる成果物の品質に、しっかり責任を持つスタンスが大前提にあるべきではないでしょうか。
Let's comment your feelings that are more than good