| id | fix-scope-integrity |
| name | Fix Scope Integrity 指摘対応ループのスコープ逸脱・前提破壊検出 |
| description | 指摘対応の反復(レビューコメント→修正→再レビュー)で、個別には正しい修正の蓄積が当初スコープを逸脱した(scope creep)、または成立済みの前提("動いていた"状態)を破壊した(premise break)連鎖を diff-time で検出する。技術的正しさとスコープ整合性を別軸で評価し、report-only で finding/question のみ出力する。1コメント単位のトリアージ・構造変更後の caller 残骸・plan 前提解消は隣接 skill へ委譲する |
| version | 0.1.0 |
| category | midstream |
| phase | midstream |
| applyTo | ["src/**/*.{ts,tsx,js,jsx,mjs}","runners/**/*.{ts,js,mjs}","scripts/**/*.mjs","app/**/*.{ts,tsx,js,jsx,php}"] |
| tags | ["fix-scope-integrity","scope-creep","premise-break","fix-scope-guard","review-response-loop","scope-consistency","midstream"] |
| severity | major |
| inputContext | ["diff","fullFile"] |
| outputKind | ["findings","questions"] |
| modelHint | high-accuracy |
| dependencies | ["code_search"] |
Origin / 由来
inspired by:
上記は観測された現象と対処法の紹介であり、本文の転載・著者による endorsement を含まない(nominative fair use)。命名は skills/README.md Naming Q0–Q5 に従い value を表す新規名として付与した。
Pattern declaration
Primary pattern: Reviewer
Secondary patterns: Inversion
Why: スコープ整合性・前提破壊は意味的判断が主だが、指摘対応ループの文脈(元スコープ/前提の discover)が成立しない差分では実行を止めるゲートが必要。
Goal / 目的
指摘対応の反復(レビューコメント→修正→再レビュー)という時間軸上で、個別には正しい修正の蓄積が次のいずれかを起こしていないかを diff-time で検出する。技術的正しさとは独立した軸で評価する。
- Scope creep(スコープ逸脱の蓄積): 当初スコープ・移植元・受入基準に無い制約/挙動を、指摘対応を通じて追加している。
- Premise break(成立済み前提の破壊): ある修正が、先に受理された変更 or 既存システムが依存していた前提("動いていた"状態)を無効化し、次の欠陥の起点になっている。
report-only(ADR-005)。finding/question のみを出力し、自動修正・自動マージはしない。
Non-goals / 扱わないこと(委譲表)
- 1コメント単位のトリアージ: コメントの重要度づけ・対応方針(自然言語)・返信案は
review-comment-triage が担う。本 skill は複数往復にまたがる蓄積のスコープ/前提整合のみを見る。
- 構造変更後の caller 残骸: 記号再採番・シグネチャ変更・ファイル分割/移動に伴う caller 側の旧参照残骸 defect は
cross-file-leakage が担う。本 skill は構造変更を伴わない挙動レベルの前提破壊を見る。
- plan 前提の解消トレース:
plan artifact 保有時の plan assumption / open question の解消証拠突合は assumption-resolution-trace が担う。本 skill は plan を伴わない指摘対応ループの前提を都度追跡する。
- 完了主張の監査: 「全部置換した」「-N%削減」等の完了主張 vs 残骸/試算の突合は
refactor-claim-audit が担う。本 skill は完了主張ではなくスコープ/前提の逸脱を見る。
- evidence-sufficiency の横断合成: 6観点全体の証拠充足合成は agent-skill
unknown-coverage-review が担う。本 skill は defect 寄り(scope/premise 破壊という意味的欠陥)に集中し、合成は委譲する。
- 技術的正しさそのものの検証: 指摘対応が技術的に正しいか(バグ有無・実測妥当性)は defect 系観点の責務。本 skill は正しくてもスコープ整合性が別軸である点だけを扱う。
Pre-execution Gate / 実行前ゲート
このスキルは以下の条件がすべて満たされない限り NO_REVIEW を返す。