Skip to content

[Bug] anvil:impl-issue SKILL.md が REQ-005/DES-025 の規範に違反する7件の問題(RuntimeKind自動判定、update-issue未委譲、SDD工程未委譲他) #218

Description

@BlueEventHorizon

背景 / コンテキスト

Issue #212(anvil:impl-issue のブランチ作成を git worktree 運用に対応させる)の実装後、/forge:review code --diffplugins/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)。

  1. RuntimeKind を自動判定している(P1, critical): target 194-207。REQ-005 I-04/FNC-04「種別判定は実行時のユーザー選択に一元化し、Issue本文・ラベル等からの自動判定は行わない」に反する。
  2. 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検証・ロック)を一切呼び出していない。
  3. SDD 工程を forge skill に委譲していない(P1, critical): target 221-441(Phase2-12)。DES-025 §2.1 のアーキテクチャ図(II→SR/SD/SP/SI)と現状実装が乖離しており、/forge:start-requirements 等の呼び出しが存在しない。
  4. FIGMA_PAT をコマンド引数に露出している(P1, critical): target 209-219。curl -s -H "X-Figma-Token: $FIGMA_PAT" が環境変数を argv へシェル展開しており、DES-025 の明文規定(Authorization系ヘッダを env= で渡し argv に埋め込まない)に反する。
  5. 別リポジトリ 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> のみを列挙しており、別リポ時の切替が反映されない。
  6. 前提条件・失敗・監査ログの制御が欠落している(P1, major): target 53-219。DES-025 §3.2 が定義する check_prereqs.py / audit_log.py / verify_pushed.py はリポジトリ内に一切存在しない(plugins/anvil/scripts ディレクトリ自体が無い)。
  7. 非 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/設計フェーズで個別に判断する。

再現手順

  1. 現行の develop ブランチで /forge:review code --files plugins/anvil/skills/impl-issue/SKILL.md --interactive --codex を実行する
  2. 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件はその対象外の別問題。

Metadata

Metadata

Labels

BUGSomething isn't working

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions