背景 / コンテキスト
Issue #212(anvil:impl-issue のブランチ作成を git worktree 運用に対応させる)の実装後、/forge:review code --diff で plugins/anvil/skills/impl-issue/SKILL.md 全文をレビューした際、今回の変更範囲(Phase 0-3 のブランチ作成手順)とは無関係な既存記述に、規範文書(REQ-005_issue_driven_flow.md / DES-025_issue_driven_flow_design.md)との不整合・矛盾が複数件発見された。いずれも Issue #212 の受け入れ基準とは無関係のため、別 Issue として記録する。
現象(実際の動作)
Codex reviewer + evaluator(forge:review パイプライン)による精査で、以下 7 件の問題を確認した(session: .claude/.temp/review-9518ba、target: plugins/anvil/skills/impl-issue/SKILL.md)。
- RuntimeKind を自動判定している(P1, critical): target 194-207。REQ-005 I-04/FNC-04「種別判定は実行時のユーザー選択に一元化し、Issue本文・ラベル等からの自動判定は行わない」に反する。
- Issue 書き戻しを update-issue に委譲していない(P1, critical): target 352-364。
gh issue edit --body-file による本文再構成のみで、DES-025 §3.2/§6.1-6.3/§8.2 が定義する anvil:update-issue(マーカー限定更新・三時点hash検証・ロック)を一切呼び出していない。
- SDD 工程を forge skill に委譲していない(P1, critical): target 221-441(Phase2-12)。DES-025 §2.1 のアーキテクチャ図(II→SR/SD/SP/SI)と現状実装が乖離しており、
/forge:start-requirements 等の呼び出しが存在しない。
- FIGMA_PAT をコマンド引数に露出している(P1, critical): target 209-219。
curl -s -H "X-Figma-Token: $FIGMA_PAT" が環境変数を argv へシェル展開しており、DES-025 の明文規定(Authorization系ヘッダを env= で渡し argv に埋め込まない)に反する。
- 別リポジトリ Issue の Closes 指定が矛盾している(P2, critical): target 75-84(Phase0-1)/ 451-480(Phase13)。Phase0-1・13-1 は別リポ時
Closes <owner>/<repo>#<N> 形式を使うと明記する一方、Phase13-2 の「PR本文に含める内容」は無条件に Closes #<issue-number> のみを列挙しており、別リポ時の切替が反映されない。
- 前提条件・失敗・監査ログの制御が欠落している(P1, major): target 53-219。DES-025 §3.2 が定義する
check_prereqs.py / audit_log.py / verify_pushed.py はリポジトリ内に一切存在しない(plugins/anvil/scripts ディレクトリ自体が無い)。
- 非 UI Issue の実装設計書要否が内部で矛盾している(P2, major): target 340-350。Phase8 の成果物表は非UI Issueにも実装設計書の出力先を定義している一方、直後の文で「実装設計書はUI Issueの場合のみ作成する」と明記しており、実ファイル上で内部矛盾がある。
なお、指摘のうち「PR 作成前の独立した承認がない」(DES-025 §9.1 TBD-005 を根拠とする指摘)は、TBD-005 が未決事項であり現行の自動継続方針(SKILL.md 冒頭の「フロー継続 [MANDATORY]」)と整合する正当な設計判断であるため false positive と判定済み(本 Issue には含めない)。
期待動作
上記 7 件について、規範文書(REQ-005 / DES-025)と実装(SKILL.md)の整合を取る。具体的な対応方針(規範側を実装に合わせて更新するか、実装を規範に合わせて拡張するか)は、triage/設計フェーズで個別に判断する。
再現手順
- 現行の
develop ブランチで /forge:review code --files plugins/anvil/skills/impl-issue/SKILL.md --interactive --codex を実行する
- reviewer(Codex)→ evaluator の判定結果として、上記 7 件が
recommendation: fix または needs_review で出力される
環境情報
- リポジトリ: BlueEventHorizon/bw-cc-plugins
- 対象: plugins/anvil/skills/impl-issue/SKILL.md
- レビューエンジン: Codex(forge:review --codex)
重大度
機能に支障あり
証跡
なし(レビューセッション .claude/.temp/review-9518ba はローカル一時ディレクトリのため参照不可。上記「現象」に指摘内容を転記済み)
原因
REQ-005 / DES-025 策定後、anvil:impl-issue SKILL.md の実装が規範文書の一部規定(RuntimeKind判定・update-issue委譲・forge SDD委譲・PAT取扱い・監査ログ)に追従しないまま先行して運用されてきたため。DES-025 冒頭の「既知の乖離」注記は起動経路(triage経由への一本化)のみを対象としており、上記7件はその対象外の別問題。
背景 / コンテキスト
Issue #212(anvil:impl-issue のブランチ作成を git worktree 運用に対応させる)の実装後、
/forge:review code --diffでplugins/anvil/skills/impl-issue/SKILL.md全文をレビューした際、今回の変更範囲(Phase 0-3 のブランチ作成手順)とは無関係な既存記述に、規範文書(REQ-005_issue_driven_flow.md / DES-025_issue_driven_flow_design.md)との不整合・矛盾が複数件発見された。いずれも Issue #212 の受け入れ基準とは無関係のため、別 Issue として記録する。現象(実際の動作)
Codex reviewer + evaluator(forge:review パイプライン)による精査で、以下 7 件の問題を確認した(session:
.claude/.temp/review-9518ba、target:plugins/anvil/skills/impl-issue/SKILL.md)。gh issue edit --body-fileによる本文再構成のみで、DES-025 §3.2/§6.1-6.3/§8.2 が定義するanvil:update-issue(マーカー限定更新・三時点hash検証・ロック)を一切呼び出していない。/forge:start-requirements等の呼び出しが存在しない。curl -s -H "X-Figma-Token: $FIGMA_PAT"が環境変数を argv へシェル展開しており、DES-025 の明文規定(Authorization系ヘッダを env= で渡し argv に埋め込まない)に反する。Closes <owner>/<repo>#<N>形式を使うと明記する一方、Phase13-2 の「PR本文に含める内容」は無条件にCloses #<issue-number>のみを列挙しており、別リポ時の切替が反映されない。check_prereqs.py/audit_log.py/verify_pushed.pyはリポジトリ内に一切存在しない(plugins/anvil/scriptsディレクトリ自体が無い)。なお、指摘のうち「PR 作成前の独立した承認がない」(DES-025 §9.1 TBD-005 を根拠とする指摘)は、TBD-005 が未決事項であり現行の自動継続方針(SKILL.md 冒頭の「フロー継続 [MANDATORY]」)と整合する正当な設計判断であるため false positive と判定済み(本 Issue には含めない)。
期待動作
上記 7 件について、規範文書(REQ-005 / DES-025)と実装(SKILL.md)の整合を取る。具体的な対応方針(規範側を実装に合わせて更新するか、実装を規範に合わせて拡張するか)は、triage/設計フェーズで個別に判断する。
再現手順
developブランチで/forge:review code --files plugins/anvil/skills/impl-issue/SKILL.md --interactive --codexを実行するrecommendation: fixまたはneeds_reviewで出力される環境情報
重大度
機能に支障あり
証跡
なし(レビューセッション
.claude/.temp/review-9518baはローカル一時ディレクトリのため参照不可。上記「現象」に指摘内容を転記済み)原因
REQ-005 / DES-025 策定後、
anvil:impl-issueSKILL.md の実装が規範文書の一部規定(RuntimeKind判定・update-issue委譲・forge SDD委譲・PAT取扱い・監査ログ)に追従しないまま先行して運用されてきたため。DES-025 冒頭の「既知の乖離」注記は起動経路(triage経由への一本化)のみを対象としており、上記7件はその対象外の別問題。