메뉴
Detect review findings that belong in CI/lint/formatter rather than human review.
기본적으로 소스를 먼저 확인하는 Prompt가 선택됩니다. 직접 명령으로 전환하거나 로컬 사본을 다운로드할 수도 있습니다.
설치 여부를 결정하기 전에 SKILL.md와 SkillsMP에 표시된 보조 파일을 읽어 보세요.
Codex 또는 Claude로 설치 이 Prompt를 복사해 Codex, Claude 또는 다른 어시스턴트에 붙여 넣으면 Skill 페이지를 검토하고 설치를 진행할 수 있습니다.
직접 명령은 검토 Prompt를 거치지 않습니다. 실행하기 전에 소스를 확인하세요.
npx skills add https://github.com/s977043/river-review --skill review-automation-boundary-guard명령은 한 줄로 유지됩니다. 복사하기 전에 가로로 스크롤해 전체 내용을 확인하세요.
로컬 사본을 원하시나요? SkillsMP에서 현재 제공할 수 있는 파일을 다운로드하세요.
SOC 직업 분류 기준
SKILL.md 표시 중
| id | review-automation-boundary |
| name | Review Automation Boundary Guard |
| description | Detect review findings that belong in CI/lint/formatter rather than human review. |
| version | 0.1.0 |
| category | midstream |
| phase | midstream |
| applyTo | ["**/*"] |
| tags | ["review-process","automation","ci","meta-review","midstream"] |
| severity | minor |
| inputContext | ["diff"] |
| outputKind | ["findings","actions","metrics"] |
| modelHint | balanced |
Primary pattern: Reviewer Secondary patterns: Inversion Why: 自動化可能なレビュー指摘を検出しCI/lint委譲を提案するが、コード変更を含まない差分では実行不要
外部実証: 「コードレビュー指摘300件を3ヶ月分類したら効いていたのは2種類だけだった」（井本 賢, 2026-07-04）は、実際に効果があった指摘は Bug/Spec 系のみだったと報告する。Style/Naming/Refactor/Arch 系は効果が低く自動化・前倒しに適するという結果は、本スキルの方針を裏付ける一次データとして参照できる。
外部実証（続報）: 「AIレビューのブロックを100PRで測ったら、70PRはno-verifyで通り抜けていた」（井本 賢, 2026-07-23）は、100 PR の実測で pre-commit hook のブロックのうち 70% が --no-verify で回避されていたと報告します。強制ゲートの守備範囲を「壊れて動かないコード」（構文エラー・smoke test 失敗）へ絞り、lint / 型 / スタイルの判定を AI レビュー側へ移したところ、ブロック後の修正率は 30% から 83% へ改善しました。この前後比較も、本スキルの方針を裏付ける一次データとして参照できます。
このスキルは以下の条件がすべて満たされない限りNO_REVIEWを返す。
.eslintrc, .prettierrc, tsconfig.json等）のみの変更ではないゲート不成立時の出力: NO_REVIEW: review-automation-boundary — 自動化境界レビューの対象となるソースコード変更が検出されない
console.log の残留、コメントアウトコード等）が含まれる場合、対応する lint ルールの追加を促す。naming-convention ルール等）の追加を促す。<file>:<line> で追える内容）。<file>:<line>: <message> 形式。コメントは日本語で返す。
自動化カテゴリごとに集約して出力する（同じパターンが複数箇所にある場合は件数を添える）。
例:
src/api.ts:23: console.log が 3 箇所残留。レビューでの指摘は属人的。Fix: ESLint no-console: error を追加し CI で強制prettier / gofmt / black 等のフォーマッタで自動修正可能なものconsole.log / console.error のまま残ったデバッグ出力（no-console ルール）no-unused-vars, @typescript-eslint/no-unused-vars）no-magic-numbers ルール）max-depth ルール）注: このレベルではanyや!のコード品質上の問題を指摘しない（それはtypescript-strict / typescript-nullcheckのスコープ）。ここでは「そのルールがCIで強制されていない」というプロセスの不備のみを指摘し、ツール設定の追加を提案する。
any の使用（@typescript-eslint/no-explicit-any でゼロトレランスにできる）! の使用（@typescript-eslint/no-non-null-assertion）import/order, simple-import-sort ルール）Bad（人間レビューで指摘）: 「インデントが 2 スペースではなく 4 スペースになっています」
Good（自動化）: prettier --write . を CI に組み込み、フォーマット差分は PR マージブロックにする。
Bad（人間レビューで指摘）: 「console.log が残っています」
Good（自動化）: ESLint の no-console ルールを error にし、CI で強制する。
Bad（人間レビューで指摘）: 「any を使っています」
Good（自動化）: @typescript-eslint/no-explicit-any: error と tsconfig.json の strict: true を CI で強制する。
.prettierrc / .editorconfig の導入と CI への format:check ステップ追加を提案する。tsconfig.json の strict オプション名と対応する ESLint ルール名を提示する。automation_debtメトリクスとして出力する。{ "automation_debt": <integer> }をメトリクスブロックのトップレベルキーとして返す。値は検出パターン数の合計。console.log を消してください」と言うだけで自動化の提案がない）、ツールが存在しない自動化の提案。