Skip to content

[Fix] ToC生成前にdprint fmtを実行しchecksum不一致を解消#205

Merged
BlueEventHorizon merged 1 commit into
developfrom
fix/202-dprint-fmt-toc-checksum-mismatch
Jul 15, 2026
Merged

[Fix] ToC生成前にdprint fmtを実行しchecksum不一致を解消#205
BlueEventHorizon merged 1 commit into
developfrom
fix/202-dprint-fmt-toc-checksum-mismatch

Conversation

@BlueEventHorizon

Copy link
Copy Markdown
Owner

概要

  • forge の完了処理フロー(レビュー→ToC更新→commit)で、ToC生成後にcommit側のdprint fmtがファイル内容を書き換え、ToCのchecksumと実際のcommit内容が不一致になる問題を修正
  • update-db-specs/update-db-rules にToC生成前のdprint fmt実行ステップを追加

Closes #202

背景

/forge:review requirement --files <file> --auto/forge:update-db-specs(ToC生成)→ /anvil:commit の順で実行すると、anvil:commit Phase 0 の dprint fmt が対象ファイルを書き換え、ToC生成時点のchecksumと実際にcommitされる内容が不一致になっていた(SHA256比較で実証済み)。

やったこと

  • update-db-specs/update-db-rules にStep 1.5(dprint fmt 実行)を追加し、ToC生成前に最終的な内容へ整形する
  • dprint fmt実行ロジックを共有スクリプト化(forge: plugins/forge/scripts/doc_structure/run_dprint_fmt.sh、anvil: plugins/anvil/skills/commit/scripts/run_dprint_fmt.sh)し、SKILL.mdへのインライン重複を解消
  • anvil:commit のPhase 0も同じ共有スクリプトを呼ぶよう変更
  • レビューで発見したDES-001の記述陳腐化(実行フロー・前提・スキル一覧・テストパス)を現行実装に合わせて更新

やらないこと(このプルリクエストのスコープ外とすること)

  • start-implement のToC更新ステップ欠落(非対称)は別Issueで扱う
  • REQ-001 FNC-006の完了処理順序自体の変更(update-db-specs/rules内部で解決するため不要)

レビュー観点

  • /forge:review code --auto によるAIレビュー(6件のfinding、全件fix済み)を実施済み
  • 単体テスト1427件(skip 1件)は全てパス

レビューレベル

  • Lv0: まったく見ないでAcceptする
  • Lv1: ぱっとみて違和感がないかチェックしてAcceptする
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証してAcceptする
  • Lv3: 実際に環境で動作確認したうえでAcceptする

update-db-specs/update-db-rulesにdprint fmt実行ステップ(Step 1.5)を追加し、
ToC生成時点のファイル内容と実際にcommitされる内容の不一致を解消する。
dprint fmt実行ロジックはforge/anvil双方に共有スクリプトとして切り出し、
SKILL.mdへのインライン重複を避ける。DES-001の記述も現行実装(dirs/exclude方式、
user-invocable: true)に合わせて更新。

Closes #202
@BlueEventHorizon BlueEventHorizon self-assigned this Jul 15, 2026
@BlueEventHorizon
BlueEventHorizon marked this pull request as ready for review July 15, 2026 09:07
@BlueEventHorizon
BlueEventHorizon merged commit 857a8d3 into develop Jul 15, 2026
1 check passed
@BlueEventHorizon
BlueEventHorizon deleted the fix/202-dprint-fmt-toc-checksum-mismatch branch July 15, 2026 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant