IT 엔지니어를 위한 일본어 코드 리뷰 커뮤니케이션 가이드: 리뷰 요청·피드백·제안 표현과 비즈니스 일본어

【기본 리뷰 요청 패턴】

1. 「レビューのほど、よろしくお願いいたします。」
   (리뷰 부탁드립니다.)
   → 가장 정중하고 범용적인 표현. 상사나 시니어에게 적합.

2. 「お手すきの際にレビューいただけますと幸いです。」
   (시간 되실 때 리뷰해 주시면 감사하겠습니다.)
   → 긴급하지 않을 때 사용. お手すき는 "한가할 때"라는 뜻.

3. 「お忙しいところ恐れ入りますが、レビューをお願いできますでしょうか。」
   (바쁘신 중에 죄송합니다만, 리뷰를 부탁드릴 수 있을까요?)
   → 상대가 바쁜 것을 알고 있을 때 배려를 표현.

4. 「ご確認いただけると助かります。」
   (확인해 주시면 도움이 됩니다.)
   → 동료나 비슷한 연차에게 사용하는 부드러운 표현.

5. 「軽くで構いませんので、目を通していただけないでしょうか。」
   (가볍게라도 괜찮으니, 한번 봐 주실 수 없을까요?)
   → 전체 리뷰가 아닌 간단한 확인을 요청할 때.

【마감 기한을 포함한 리뷰 요청】

「明日のリリースに間に合わせたいため、本日17時までにレビュー
 いただけますと大変助かります。」
(내일 릴리즈에 맞추고 싶어서, 오늘 17시까지 리뷰해 주시면 매우 도움이 됩니다.)

「今週金曜のデプロイに含めたいので、木曜日中のレビューを
 お願いできますでしょうか。」
(이번 주 금요일 배포에 포함하고 싶으므로, 목요일 중 리뷰를 부탁드릴 수 있을까요?)

「スプリントの締め切りが迫っておりますので、なるべく早めに
 ご確認いただけると幸いです。」
(스프린트 마감이 다가오고 있으므로, 가급적 빨리 확인해 주시면 감사하겠습니다.)

【Slack 리뷰 요청 전체 예시】

@tanaka-san
お疲れさまです。金です。
認証モジュールのリファクタリングPRを作成しましたので、
レビューのほどよろしくお願いいたします。

■ PR: #1234 認証ロジックのリファクタリング
■ 変更概要: JWTトークンの検証処理を共通化
■ 影響範囲: ログイン・ログアウト機能
■ 確認ポイント: エラーハンドリングのパターン

お手すきの際にご確認いただけると助かります。

(@tanaka 씨
수고하십니다. 김입니다.
인증 모듈의 리팩토링 PR을 작성했으므로,
리뷰 부탁드립니다.

■ PR: #1234 인증 로직 리팩토링
■ 변경 개요: JWT 토큰 검증 처리를 공통화
■ 영향 범위: 로그인·로그아웃 기능
■ 확인 포인트: 에러 핸들링 패턴

시간 되실 때 확인해 주시면 도움이 됩니다.)

【MUST - 반드시 수정 필요】
[MUST] SQLインジェクションの脆弱性があります。
プレースホルダを使用してください。
(SQL 인젝션 취약점이 있습니다. 플레이스홀더를 사용해 주세요.)

【nit - 사소한 지적】
[nit] 変数名ですが、dataよりもuserProfileDataの方が
意図が伝わりやすいかもしれません。
(변수명인데요, data보다 userProfileData 쪽이 의도가 전달되기 쉬울지도 모릅니다.)

【IMO - 개인 의견】
[IMO] 個人的には、この処理はカスタムフックに切り出した方が
再利用しやすいかなと思いました。あくまで個人的な意見ですので、
ご参考程度にどうぞ。
(개인적으로는 이 처리를 커스텀 훅으로 분리하는 편이 재사용하기 쉬울 것 같았습니다.
어디까지나 개인 의견이므로, 참고 정도로 봐 주세요.)

【suggestion - 제안】
[suggestion] ここでOptionalを使うと、nullチェックが
よりシンプルになるかと思います。
(여기서 Optional을 사용하면 null 체크가 더 심플해질 것 같습니다.)

【ask - 질문】
[ask] この部分の仕様について確認させてください。
ユーザーが未認証の場合、403と401のどちらを返す想定でしょうか？
(이 부분의 사양에 대해 확인시켜 주세요.
사용자가 미인증인 경우, 403과 401 중 어느 쪽을 반환하는 상정인가요?)

【FYI - 참고 정보】
[FYI] 参考までに、この処理に関連する公式ドキュメントのリンクを
共有いたします。
(참고로, 이 처리에 관련된 공식 문서 링크를 공유합니다.)

【チーム運用ルール例】

・プレフィックスなしのコメントは「質問」として扱う
・MUSTは「セキュリティ・バグ・仕様違反」に限定して使用する
・nitは「対応しなくてもApproveする」という意思表示
・IMOは「自分ならこうする」という参考意見
・コメント数が多い場合は、最後にサマリーを付ける

(프리픽스 없는 코멘트는 "질문"으로 취급한다
MUST는 "보안·버그·사양 위반"에 한정하여 사용한다
nit는 "대응하지 않아도 승인한다"라는 의사 표시
IMO는 "자기라면 이렇게 한다"라는 참고 의견
코멘트 수가 많을 경우, 마지막에 서머리를 붙인다)

【이유 + 대안을 포함한 정중한 제안】

「この処理ですが、N+1クエリが発生する可能性があるかと思います。
 パフォーマンスの観点から、EagerLoadingを使う方法もあるかと
 存じますが、いかがでしょうか。」
(이 처리인데요, N+1 쿼리가 발생할 가능성이 있을 것 같습니다.
 퍼포먼스 관점에서 Eager Loading을 사용하는 방법도 있을 것 같은데, 어떨까요?)

「エラーハンドリングについてなのですが、現在の実装ですと
 例外がそのまま上位に伝播してしまうケースがありそうです。
 カスタム例外クラスでラップする形はいかがでしょうか。」
(에러 핸들링에 대해서인데요, 현재 구현이면
 예외가 그대로 상위로 전파되는 케이스가 있을 것 같습니다.
 커스텀 예외 클래스로 래핑하는 형태는 어떨까요?)

「細かい点で恐縮ですが、この変数のスコープをもう少し
 狭くできるかもしれません。可読性の向上につながるかと思います。」
(사소한 점이라 죄송합니다만, 이 변수의 스코프를 좀 더
 좁힐 수 있을지도 모릅니다. 가독성 향상으로 이어질 것 같습니다.)

【개인 의견 명시 표현 모음】

・「個人的な意見ですが～」 (개인적인 의견입니다만~)
・「好みの問題かもしれませんが～」 (취향 문제일 수도 있지만~)
・「自分だったらこうするかなと思いました」 (저라면 이렇게 할 것 같다고 생각했습니다)
・「あくまで参考程度ですが～」 (어디까지나 참고 정도입니다만~)
・「別案として検討いただければと思います」 (대안으로 검토해 주시면 좋겠습니다)
・「強い意見ではないのですが～」 (강한 의견은 아닙니다만~)
・「対応は任せます」 (대응은 맡기겠습니다)

【감사 표현 패턴】

1. 「ご指摘ありがとうございます。」
   (지적 감사합니다.)
   → 가장 기본적이고 범용적인 감사 표현.

2. 「レビューありがとうございます。大変勉強になりました。」
   (리뷰 감사합니다. 매우 공부가 됐습니다.)
   → 새로운 것을 배웠을 때 사용.

3. 「細かいところまで見ていただき、ありがとうございます。」
   (세세한 부분까지 봐 주셔서 감사합니다.)
   → 꼼꼼한 리뷰에 대한 감사.

4. 「確かにおっしゃる通りですね。気づきませんでした。」
   (확실히 말씀하신 대로네요. 알아차리지 못했습니다.)
   → 리뷰어의 지적에 동의할 때.

5. 「鋭いご指摘、ありがとうございます。」
   (날카로운 지적 감사합니다.)
   → 중요한 문제를 발견해 줬을 때.

【수정 완료 보고 패턴】

1. 「ご指摘の箇所、修正いたしました。ご確認をお願いいたします。」
   (지적해 주신 부분, 수정했습니다. 확인 부탁드립니다.)

2. 「ご指摘いただいた内容を反映しました。再度レビューいただけますと幸いです。」
   (지적해 주신 내용을 반영했습니다. 다시 리뷰해 주시면 감사하겠습니다.)

3. 「修正対応が完了しましたので、お手すきの際にご確認ください。」
   (수정 대응이 완료됐으므로, 시간 되실 때 확인해 주세요.)

4. 「全てのご指摘に対応いたしました。変更内容はコミットメッセージに
   記載しております。」
   (모든 지적에 대응했습니다. 변경 내용은 커밋 메시지에 기재하고 있습니다.)

【정중한 반론 패턴】

1. 「ご指摘ありがとうございます。一点ご相談させていただきたいのですが、
   この実装にした背景として～という事情がございまして…」
   (지적 감사합니다. 한 가지 상담드리고 싶은데요,
   이 구현으로 한 배경으로서 ~라는 사정이 있어서요...)

2. 「おっしゃる通りの方法も検討したのですが、今回は～の理由で
   現在の実装を選択しました。もしよろしければ、ご意見を
   いただけないでしょうか。」
   (말씀하신 방법도 검토했습니다만, 이번에는 ~의 이유로
   현재 구현을 선택했습니다. 괜찮으시다면 의견을 주실 수 있을까요?)

3. 「大変参考になるご意見をありがとうございます。
   こちらの意図としては～なのですが、認識に齟齬がないか
   すり合わせさせていただけますか？」
   (매우 참고가 되는 의견 감사합니다.
   이쪽의 의도로서는 ~인데요, 인식에 차이가 없는지
   맞춰볼 수 있을까요?)

4. 「なるほど、その観点は考慮できておりませんでした。
   ただ、現状の要件ですと～という制約があるため、
   別の方法で対応してもよろしいでしょうか？」
   (그렇군요, 그 관점은 고려하지 못했습니다.
   다만 현재 요건이면 ~라는 제약이 있어서,
   다른 방법으로 대응해도 괜찮을까요?)

【LGTM 코멘트 패턴】

1. 「LGTM です！きれいにリファクタリングされていて、
   とても読みやすくなりました。」
   (LGTM입니다! 깔끔하게 리팩토링되어 있어서 매우 읽기 쉬워졌습니다.)

2. 「LGTM。テストケースも網羅的で、安心してマージできます。
   お疲れさまでした。」
   (LGTM. 테스트 케이스도 망라적이어서 안심하고 머지할 수 있습니다.
   수고하셨습니다.)

3. 「LGTM です。一点 nit がありますが、対応は任せます。」
   (LGTM입니다. 한 가지 nit가 있지만 대응은 맡기겠습니다.)

【조건부 승인 패턴】

1. 「この一点だけ修正いただければ、LGTM です。」
   (이 한 가지만 수정해 주시면 LGTM입니다.)

2. 「セキュリティ関連のコメントだけ対応いただいた上で、
   マージをお願いいたします。nitは対応不要です。」
   (보안 관련 코멘트만 대응해 주신 뒤에,
   머지를 부탁드립니다. nit는 대응 불필요입니다.)

3. 「ほぼ LGTM ですが、テストの追加だけお願いしたいです。
   それ以外は問題ありません。」
   (거의 LGTM이지만, 테스트 추가만 부탁하고 싶습니다.
   그 이외에는 문제 없습니다.)

【머지 관련 대화 패턴】

「Approveしましたので、マージはお任せします。」
(승인했으므로 머지는 맡기겠습니다.)

「全員のApproveが揃いましたので、マージしてもよろしいでしょうか？」
(전원의 승인이 갖춰졌으므로 머지해도 괜찮을까요?)

「CIが通ったことを確認してからマージお願いします。」
(CI가 통과한 것을 확인한 뒤 머지 부탁드립니다.)

「マージ後の動作確認は私の方で行います。」
(머지 후 동작 확인은 제 쪽에서 하겠습니다.)

「developブランチへのマージ、完了しました。お疲れさまでした。」
(develop 브랜치로의 머지, 완료했습니다. 수고하셨습니다.)

【Slack メッセージ】

金（新人）→ 田中さん（シニア）

金：田中さん、お疲れさまです。
  ユーザー登録APIの実装PRを作成しました。
  初めてのAPI実装なので、設計面も含めてレビューいただけると
  大変勉強になります。
  特にバリデーション周りが不安なので、重点的に見ていただけると
  助かります。
  PR: #567
  お手すきの際にお願いいたします。

田中：金さん、お疲れさまです。
  PR見ました。初めてのAPI実装にしてはとてもよく書けていますね。
  いくつかコメントを入れましたので、ご確認ください。
  [ask] バリデーションの順序について一点質問があります。
  [suggestion] エラーレスポンスの形式について提案があります。
  [nit] 変数名について細かい点が1つあります。
  分からないことがあれば、いつでも聞いてくださいね。

金：田中さん、丁寧なレビューありがとうございます！
  大変勉強になりました。
  バリデーションの順序について、ご質問の件ですが、
  仕様書の記載に従ったのですが、確かにご指摘の順序の方が
  効率的ですね。修正いたします。
  エラーレスポンスの提案もとても参考になりました。
  全て対応しましたので、再度ご確認いただけますと幸いです。

田中：修正確認しました。LGTM です！
  きれいに対応されていますね。お疲れさまでした。
  マージはお任せします。

【한국어 해설】

김(신입) → 다나카 씨(시니어)

김: 다나카 씨, 수고하십니다.
  사용자 등록 API 구현 PR을 작성했습니다.
  처음 하는 API 구현이라 설계 면도 포함해서 리뷰해 주시면
  매우 공부가 됩니다.
  특히 밸리데이션 쪽이 불안해서 중점적으로 봐 주시면 도움이 됩니다.
  PR: #567
  시간 되실 때 부탁드립니다.

다나카: 김 씨, 수고하십니다.
  PR 봤습니다. 처음 하는 API 구현치고 매우 잘 쓰여 있네요.
  몇 가지 코멘트를 넣었으니 확인해 주세요.
  [ask] 밸리데이션 순서에 대해 한 가지 질문이 있습니다.
  [suggestion] 에러 리스폰스 형식에 대해 제안이 있습니다.
  [nit] 변수명에 대해 사소한 점이 1개 있습니다.
  모르는 게 있으면 언제든지 물어보세요.

김: 다나카 씨, 정중한 리뷰 감사합니다!
  매우 공부가 됐습니다.
  밸리데이션 순서에 대해 질문하신 건인데요,
  사양서 기재에 따랐는데, 확실히 지적해 주신 순서가
  더 효율적이네요. 수정하겠습니다.
  에러 리스폰스 제안도 매우 참고가 됐습니다.
  전부 대응했으므로 다시 확인해 주시면 감사하겠습니다.

다나카: 수정 확인했습니다. LGTM입니다!
  깔끔하게 대응되어 있네요. 수고하셨습니다.
  머지는 맡기겠습니다.

【GitHub PR コメント】

佐藤（レビュアー）→ 金（レビュイー）

佐藤：[suggestion] お疲れさまです。一点ご提案なのですが、
  現在の実装では認証チェックが各エンドポイントに
  散在している状態かと思います。
  ミドルウェアパターンで共通化すると、今後エンドポイントが
  増えた際にも漏れなく認証チェックが適用できるかと思いますが、
  いかがでしょうか。
  もちろん、現在のスコープでは対応が難しい場合は、
  次のスプリントのタスクとして切り出すのもありかと思います。

金：佐藤さん、ご提案ありがとうございます。
  おっしゃる通り、ミドルウェアパターンの方が
  保守性が高いと思います。
  ただ、今回のリリースまでの期間を考えると、
  現在のスコープでは対応が厳しい状況です。
  次スプリントのタスクとして起票してもよろしいでしょうか？
  その際、今回のPRにTODOコメントを残しておきます。

佐藤：承知しました。それで問題ありません。
  TODOコメントを残していただければ、次スプリントで
  対応する際にも追跡しやすいので、ぜひお願いします。
  その方針で LGTM です。

【한국어 해설】

사토(리뷰어) → 김(리뷰이)

사토: [suggestion] 수고하십니다. 한 가지 제안인데요,
  현재 구현에서는 인증 체크가 각 엔드포인트에
  흩어져 있는 상태인 것 같습니다.
  미들웨어 패턴으로 공통화하면, 향후 엔드포인트가
  늘어날 때에도 빠짐없이 인증 체크가 적용될 것 같은데,
  어떨까요?
  물론 현재 스코프에서는 대응이 어려운 경우,
  다음 스프린트 태스크로 분리하는 것도 괜찮을 것 같습니다.

김: 사토 씨, 제안 감사합니다.
  말씀하신 대로 미들웨어 패턴이
  유지보수성이 높다고 생각합니다.
  다만 이번 릴리즈까지의 기간을 생각하면,
  현재 스코프에서는 대응이 어려운 상황입니다.
  다음 스프린트의 태스크로 기표해도 괜찮을까요?
  그때 이번 PR에 TODO 코멘트를 남겨 두겠습니다.

사토: 알겠습니다. 그걸로 문제 없습니다.
  TODO 코멘트를 남겨 주시면 다음 스프린트에서
  대응할 때에도 추적하기 쉬우므로 꼭 부탁드립니다.
  그 방침으로 LGTM입니다.

【GitHub PR コメント - 意見の相違】

鈴木：[MUST] この箇所ですが、パフォーマンスの観点から
  データベースへの問い合わせをキャッシュすべきだと考えます。

金：鈴木さん、ご指摘ありがとうございます。
  キャッシュの導入は検討したのですが、このデータは
  リアルタイム性が求められる金額情報のため、
  キャッシュの整合性担保が難しいと判断しました。
  具体的には、キャッシュの無効化タイミングの制御が
  複雑になるリスクがあると考えております。

鈴木：なるほど、リアルタイム性の要件は理解しました。
  それでしたら、キャッシュのTTLを短く設定する
  （例えば30秒程度）方法はいかがでしょうか。
  完全なリアルタイムではないですが、
  DBの負荷は大幅に軽減できるかと思います。

金：ありがとうございます。TTL 30秒であれば、
  ビジネス要件的にも許容範囲内かと思います。
  念のため、プロダクトオーナーにも確認を取った上で
  対応させていただいてもよろしいでしょうか？

鈴木：はい、その進め方で問題ありません。
  確認が取れ次第、対応をお願いします。

金：PO に確認が取れました。TTL 30秒で問題ないとのことです。
  キャッシュ処理を追加しましたので、再度ご確認をお願いいたします。

鈴木：確認しました。LGTM です。
  丁寧に対応いただきありがとうございました。

【한국어 해설】

스즈키: [MUST] 이 부분인데요, 퍼포먼스 관점에서
  데이터베이스 조회를 캐시해야 한다고 생각합니다.

김: 스즈키 씨, 지적 감사합니다.
  캐시 도입은 검토했습니다만, 이 데이터는
  리얼타임성이 요구되는 금액 정보이므로,
  캐시의 정합성 담보가 어렵다고 판단했습니다.
  구체적으로 캐시 무효화 타이밍 제어가
  복잡해지는 리스크가 있다고 생각하고 있습니다.

스즈키: 그렇군요, 리얼타임성 요건은 이해했습니다.
  그렇다면 캐시의 TTL을 짧게 설정하는
  (예를 들어 30초 정도) 방법은 어떨까요?
  완전한 리얼타임은 아니지만,
  DB 부하는 대폭 경감될 것 같습니다.

김: 감사합니다. TTL 30초라면
  비즈니스 요건적으로도 허용 범위 내일 것 같습니다.
  혹시 몰라 프로덕트 오너에게도 확인을 받은 뒤에
  대응해도 괜찮을까요?

스즈키: 네, 그 진행 방식으로 문제 없습니다.
  확인이 되는 대로 대응 부탁드립니다.

김: PO에게 확인을 받았습니다. TTL 30초로 문제없다고 합니다.
  캐시 처리를 추가했으므로 다시 확인 부탁드립니다.

스즈키: 확인했습니다. LGTM입니다.
  정중하게 대응해 주셔서 감사했습니다.

【리뷰어에게 (존경어 사용)】

・「ご確認いただけますでしょうか」 (확인해 주실 수 있을까요)
・「ご指摘いただいた点」 (지적해 주신 점)
・「おっしゃる通り」 (말씀하신 대로)
・「ご意見をいただけると」 (의견을 주시면)
・「ご覧いただけますと」 (봐 주시면)

【자기 행동 (겸양어 사용)】

・「修正いたしました」 (수정했습니다)
・「確認いたします」 (확인하겠습니다)
・「対応させていただきます」 (대응하겠습니다)
・「ご説明申し上げます」 (설명드리겠습니다)
・「拝見しました」 (봤습니다 - 겸양)

【よくある敬語の間違い - 자주 틀리는 경어】

✗ 「確認してください」 → ✓ 「ご確認ください」「ご確認いただけますか」
  (확인해 주세요 → 확인해 주세요/확인해 주실 수 있나요)
  → "して" 대신 "ご~ください" 패턴이 더 정중

✗ 「見ました」 → ✓ 「拝見しました」「確認いたしました」
  (봤습니다 → 배견했습니다/확인했습니다)
  → 겸양어를 사용하여 자신을 낮추는 표현

✗ 「分かりました」 → ✓ 「承知しました」「かしこまりました」
  (알겠습니다 → 알겠습니다(정중)/알겠습니다(최고 정중))
  → 비즈니스에서는 承知しました가 표준

✗ 「すみません」 → ✓ 「恐れ入りますが」「申し訳ございませんが」
  (죄송합니다 → 송구합니다만/죄송합니다만)
  → 비즈니스 문서에서는 より 격식체 사용

【이모지 활용 예시】

「ここ、少し気になりました 🤔」
(여기, 조금 신경 쓰였습니다 🤔)
→ 생각하는 이모지로 부드러운 의문 표시

「LGTM です！素晴らしい実装ですね 👏」
(LGTM입니다! 훌륭한 구현이네요 👏)
→ 박수 이모지로 칭찬 강화

「修正対応しました 🙇」
(수정 대응했습니다 🙇)
→ 절하는 이모지로 겸손함 표현

「なるほど、勉強になります 📝」
(그렇군요, 공부가 됩니다 📝)
→ 메모 이모지로 학습 의지 표현

【선배에게 리뷰할 때 - 정중한 표현】

「大変恐縮ですが、一点だけ確認させていただいてもよろしいでしょうか。
 こちらの処理なのですが、～という理解で合っておりますでしょうか。」
(매우 송구합니다만, 한 가지만 확인시켜 주셔도 괜찮을까요.
 이쪽 처리인데요, ~라는 이해가 맞는 걸까요?)

【후배에게 리뷰할 때 - 격려를 포함한 표현】

「全体的にとてもよく書けていますね。
 一点だけ、ここの部分はこういう書き方もあるので、
 参考にしてみてください。」
(전체적으로 매우 잘 쓰여 있네요.
 한 가지만, 여기 부분은 이런 작성 방법도 있으니
 참고해 보세요.)

【Slack에서 자주 쓰는 표현】

・「レビューお願いします 🙏」 (리뷰 부탁드립니다)
・「コメント返しました」 (코멘트 답변했습니다)
・「修正push しました」 (수정 push 했습니다)
・「Approve お願いします」 (승인 부탁드립니다)
・「マージしていいですか？」 (머지해도 될까요?)
・「コンフリクト解消しました」 (컨플릭트 해소했습니다)
・「対応完了です」 (대응 완료입니다)

【스레드에서의 논의 전환 표현】

・「PRのコメントにも書きましたが～」
  (PR 코멘트에도 썼습니다만~)
・「口頭で補足させてください」
  (구두로 보충 설명시켜 주세요)
・「ここはテキストだと伝わりにくいので、
  画面共有しながらお話しできますか？」
  (여기는 텍스트로는 전달이 어려우므로,
  화면 공유하면서 이야기할 수 있을까요?)