菜单
実装済みの差分、PR、コミット、指定ファイルをレビューし、実装者の前提や正常系中心の説明を鵜呑みにせず、見落とされた失敗条件を疑う基本姿勢で敵対的検証を行い、仕様違反、回帰、公開契約破壊、セキュリティ、データ整合性、並行性、検証不足などのリスクを重要度順に指摘するときに使う。実装方針の相談は code-refactor、コード変更の実装は code-general を優先する。
默认使用会先检查来源的 Prompt；你也可以切换为直接命令，或下载本地副本。
决定是否安装前，请先阅读 SKILL.md，以及 SkillsMP 当前展示的配套文件。
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
直接命令不会经过审查 Prompt；运行前请先检查来源。
npx skills add https://github.com/version-1/agents-docs --skill code-review命令会保持在同一行。复制前请横向滚动并检查完整内容。
想先保存到本地？可下载 SkillsMP 当前能够提供的文件。
正在显示 SKILL.md
Git リポジトリで新しい作業を始める前に、最新のデフォルトブランチへ追従し、タスク内容に合う作業ブランチを作成して、不要なローカルブランチ整理を非同期に依頼するときに使う。ユーザーが「ブランチ切って」「作業を始める準備して」「新しいタスク用ブランチを作って」「最新 main から始めたい」などを依頼した場合はこの skill を使い、ブランチ名を報告してからタスク詳細の入力を待つ。
Issue Finder agent がコードベースを変更せず、指定された finder 分野の課題を根拠付きで調査し、優先順位付けした Markdown を指定ディレクトリへ出力するときの共通手順、安全制約、品質基準、出力 contract を定義する。
コード、設定、依存関係、権限境界にある悪用可能な脆弱性を安全に調査し、コードを変更せず課題レポートへ整理するため issue-finder agent に委譲するときに使う。一般的なコードレビュー、侵入や攻撃の実行、脆弱性修正には使わない。
基于 SOC 职业分类
| name | code-review |
| description | 実装済みの差分、PR、コミット、指定ファイルをレビューし、実装者の前提や正常系中心の説明を鵜呑みにせず、見落とされた失敗条件を疑う基本姿勢で敵対的検証を行い、仕様違反、回帰、公開契約破壊、セキュリティ、データ整合性、並行性、検証不足などのリスクを重要度順に指摘するときに使う。実装方針の相談は code-refactor、コード変更の実装は code-general を優先する。 |
このドキュメントは、コードレビューを行うときの思考の枠組み・判断基準・進め方 を定義します。
これは実装方法のガイドではありません。 「どう直すか」ではなく 「何を疑い、何を優先して指摘するか」 を示します。
findings first の出力形式レビューは次の順で行う。
軽微な命名やスタイルより、仕様違反・回帰・高リスク変更・未検証の方を優先する。
指摘を出すときは、最低限次を含める。
指摘に含めなくてよいもの:
次のいずれかに当てはまるもの。
次のいずれかに当てはまるもの。
次のいずれかに当てはまるもの。
補足:
references/points.md を参照。
出力は次の順で行う。
必要な指摘がない場合:
重大な指摘はなし と明示する原則:
悪い点 のような感想セクションは作らない