Skip to content

[Fix] test_changed_lines_policy の対象スコープを design/ に narrow#173

Merged
BlueEventHorizon merged 1 commit into
developfrom
fix/169-test-scope-standalone-subagent-check
Jul 6, 2026
Merged

[Fix] test_changed_lines_policy の対象スコープを design/ に narrow#173
BlueEventHorizon merged 1 commit into
developfrom
fix/169-test-scope-standalone-subagent-check

Conversation

@BlueEventHorizon

Copy link
Copy Markdown
Owner

概要

  • test_changed_lines_do_not_add_standalone_subagent の対象パス (CHECKED_PATH_PREFIXES) が要件定義書まで含んでおり、実装経路を指図しない一般用語 (subagent) の使用で誤検知していた問題を修正

背景

forge-session-dss 計画書の AI レビュー実行中に、docs/specs/forge/requirements/REQ-007_dss_v2.md:47 の「subagent (子)」で test_changed_lines_do_not_add_standalone_subagent が失敗することを発見した (Issue #169)。

docs/rules/skill_launch_paths_definitions.md §2 の用語統一ルールは対象を明示的に「SKILL.md / 設計書 / ガイド文書」と限定しており要件定義書は含まれない。これは docs/readme/guide_sdd_ja.md の What/How 境界(要件定義書=What・技術中立、設計書=How・技術依存)とも整合する。docs/specs/forge/design/ 配下は既に subagent 単独使用が 0 件であり、要件定義書側のみが誤検知源だった。

やったこと

  • tests/forge/subagent/test_changed_lines_policy.pyCHECKED_PATH_PREFIXESdocs/specs/forge/ から docs/specs/forge/design/ に narrow し、要件定義書を検査対象から除外

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

  • 他プラグイン(anvil / common)の requirements/design 分離対応(元々 CHECKED_PATH_PREFIXES の対象外)
  • REQ-007_dss_v2.md:47 自体の文言修正(要件定義書は用語統一の対象外という結論のため不要)

レビュー観点

  • CHECKED_PATH_PREFIXES を narrow したことで design/ 配下の既存検査が意図せず弱まっていないか
  • /forge:review code (evaluator) は本変更を false_positive の reviewer 指摘(DES-030 は本変更のスコープを管轄しない設計書)として却下済み

レビューレベル

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

備考

Closes #169

@BlueEventHorizon BlueEventHorizon self-assigned this Jul 6, 2026
@BlueEventHorizon
BlueEventHorizon marked this pull request as ready for review July 6, 2026 11:57
@BlueEventHorizon
BlueEventHorizon merged commit 649cb3c into develop Jul 6, 2026
1 check passed
@BlueEventHorizon
BlueEventHorizon deleted the fix/169-test-scope-standalone-subagent-check branch July 6, 2026 11:58
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