[Docs] 通常運用スコープの比例性原則を独立文書化#183
Merged
Merged
Conversation
- plugins/forge/docs/scope_proportionality_spec.md を新規作成 (カタストロフィックな異常事態 [OSクラッシュ・ファイルシステム破損等] への 防御はスコープ外と明示し、fix判定の3点チェックを一般化して集約) - review_priorities_spec.md §3.4 を全文複製から参照形式に変更 (重複していた §3.4 番号衝突も §3.5〜§3.7 に修正) - review/SKILL.md Phase 3 の ssot_refs[] 固定注入(P3)に追加し、 レビュー種別を問わず reviewer/evaluator が必ず読む構造的な参照にする (受動的な文書内参照だけでは読まれる保証がないため) - impl-issue/SKILL.md Phase 8 に、実装スコープ判断時の動的検索の 参照ポインタを追加(パス直書きせず /forge:query-forge-rules 経由) - forge ToC を再生成・同梱ファイルへ反映 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- scope_proportionality_spec.md: 3点チェックの適用対象を3類型 (カタストロフィックな外部要因防御/投機的防御実装/根拠なき品質改善の 即時修正化) に明示的に限定し、3類型に該当しない指摘(要件・設計・ ルール違反、具体的な実行時リスク、データ損失・セキュリティ等)は 対象外と明記 - 3点チェック2番目の「通常運用フローで顕在化するか」が 「レビューで見つかった=顕在化済みでskip可」と誤読される 自己論駁的な文言だったため、「レビュー対象コードとは独立した 既存の安全網」であることを明示し、「レビューで見つかったこと」 自体が根拠にならないと明記 - review_priorities_spec.md §3.4 も同様に、全fix判定ではなく 3類型該当時のみ適用されると明確化 - 判定に迷う場合は「該当しない(=通常通りfix対象)」とし、 false positive(過剰skip)を避ける方針を追加 Closes #182 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
BlueEventHorizon
marked this pull request as ready for review
July 7, 2026 03:38
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
背景
以前のレビューで、evaluator が「プロセスクラッシュを前提にした1箇所のみのアトミック化提案」のような、カタストロフィックな異常事態への過剰な防御実装を
fix推奨してしまう問題が見つかった。この判断基準(3点チェック)はreview_priorities_spec.md§3.4 に追記されていたが、以下の構造的欠陥があった:review_priorities_spec.md自体が reviewer/evaluator の必須読み込みリスト(ssot_refs[])に含まれておらず、実際に読まれる保証がなかったやったこと
1コミット目: 原則の独立文書化と構造的な必須参照化
plugins/forge/docs/scope_proportionality_spec.mdを新規作成し、比例性の原則を集約review_priorities_spec.md§3.4 を全文複製から参照形式に変更(重複していた §3.4 番号衝突も §3.5〜§3.7 に修正)review/SKILL.mdPhase 3 のssot_refs[]固定注入(P3)に追加し、レビュー種別を問わず reviewer/evaluator が必ず読む構造にした(受動的な文書内参照だけでは読まれる保証がないため)impl-issue/SKILL.mdPhase 8 に、実装スコープ判断時の動的検索の参照ポインタを追加(パス直書きせず/forge:query-forge-rules経由)2コミット目: Issue #182 対応(3点チェックの誤読防止)
skipの根拠にできないことを明記やらないこと(このプルリクエストのスコープ外とすること)
docs/rules/(プロジェクト自身の開発ルール)への配置は行わない。本原則は forge/anvil を使う全下流プロジェクトに適用されるべきものであり、配布物であるplugins/forge/docs/に置く必要があるレビュー観点
scope_proportionality_spec.md§2 の3類型定義が、Issue [Improve] scope_proportionality_spec の 3点チェックが通常のレビュー修正まで skip に倒しうる #182 が懸念した「通常のレビュー修正の過剰skip」を実際に防げる文言になっているかreview/SKILL.mdのssot_refs[]固定注入により、実際に全レビューで本文書が読まれる経路になっているかレビューレベル
Lv0: まったく見ないでAcceptするLv1: ぱっとみて違和感がないかチェックしてAcceptするLv3: 実際に環境で動作確認したうえでAcceptする備考
Closes #182
全1292テスト green(ToC鮮度契約テスト含む)。