cross-refactoring: 名前が test_ で始まる本番のモジュール（lib/test_strategy.py）をテストとみなし、範囲テストのテンプレートの試走が終了コード 5 で止まる #2516

依頼（原文）

現象

開発ループ lroot3 の周回 10（スプリント lroot3-10、#2369、Pull Request #2515 ）の検査のプランで、cross-refactoring の init が範囲テストのテンプレートの試走で止まった（refactor.py init が終了コード 4）。

⛔ 範囲テストのテンプレートが {paths} に入るテストのファイルを受けません（suite: pytest）
   入れたパス: plugins/ndf/scripts/lib/test_strategy.py plugins/ndf/scripts/lib/test_triage.py
   試走: uv run --frozen --project . --all-extras pytest plugins/ndf/scripts/lib/test_strategy.py plugins/ndf/scripts/lib/test_triage.py -q -n 4 ... → 終了コード 5

記録: /tmp/ndf-worktrees/devbasex--ai-plugins/rf2515/work/.cross_refactoring/template-check.json、プランの報告 /work/ai-plugins/.ndf/loop/lroot3/lap-10/sprint/4-check-state/report.md（手元の記録）

原因

plugins/ndf/scripts/lib/test_strategy.py と test_triage.py は本番のモジュールで、テスト関数を持たない。テストかどうかを名前だけで決めているため、テストのファイルとして扱われる。

• refactor_lib/scope.py の test_files_under は名前の形（TEST_NAME_PATTERNS）だけで拾う。test_locations は「配下にテストの名前のファイルがある」ことで plugins/ndf/scripts/lib をテストの置き場所とみなす（決定 12 の本体と同じ場所に置く構成）
• template_check._probe_candidates がその 2 本を試走の候補にし、pytest がテストを 1 件も集められず終了コード 5 で終わる。着手前のテストは通っているので defect と判定され、テンプレートの不備として止まる
• テンプレート（.ndf/project.json の test.suites[0].scope_command）は正しい。直す箇所の案内が誤っている

plugins/ndf/scripts/lib を触る Pull Request なら、#2369 に限らず同じように止まる（refactor-scope.py が変更のディレクトリを --scope に入れるため）。pathkinds.is_test_path（test_ を名前に含めばテスト）も同じ基準で、危険度や本番コードの行数の数え方にも効いている。

再現の手順

1. plugins/ndf/scripts/lib/ の本番のファイルを 1 行変える Pull Request を作る
2. python3 plugins/ndf/scripts/refactor-scope.py --pr <番号> の出力を --scope に渡して drive.py <番号> --workflow-step を打つ
3. init が上の ⛔ で止まる

受け入れ条件

1. テスト関数を持たない test_*.py（本番のモジュール）が、範囲テストのテンプレートの試走の候補とテストの置き場所の判定に入らない。上の再現の手順で init が止まらない
2. 直す箇所の案内が、テンプレートが正しいのに scope_command を指さない
3. 全体テストが通る

補足

• 即時修正の 4 条件のうち「既存の方針の範囲に収まる」を満たすか判断がつかない（テストの見分け方を名前から中身へ広げると、他言語の構成と決定 12 の扱いが変わる）ため、その場で直さずに起票した
• この不具合が解けるまで、契約の棚卸しの残り: 呼び手の無い関数の片付けと子の課題 17 件の判定（#2336 の受け入れ条件 8・10） #2369 の Pull Request 2515 は検査のプランを通れない

（書き手: 起票者（開発ループ lroot3 の conductor）、日付: 2026-11-05。利用者の指示の原文は含まない）

#2516。人へ問わずに進め、決められない点は前提か未決として課題の本文へ書く。

（書き手: AI の起動指示、日付: 2026-11-05）

起票者の推定

課題の全文が起票者（AI）の記録で、利用者の指示の原文は無い。承認ゲート 1 で利用者が確かめるまで、次の条件を利用者の決定として扱わない。S4・S5 は要求を書いた AI が足した判断である。

目的

• 名前が test_ で始まるプロダクションコードのファイルを持つリポジトリでも、cross-refactoring の着手（init）がテンプレートの試走で誤って止まらないようにする
• 止まったときの案内が、正しいテンプレートを直す箇所として示さないようにする
• 契約の棚卸しの残り: 呼び手の無い関数の片付けと子の課題 17 件の判定（#2336 の受け入れ条件 8・10） #2369 の Pull Request スプリント lroot3-10 #2515 が検査のプランを通れるようにする

前提

• 前提 1: 中身でテストかどうかを見分ける対象は Python のファイル（拡張子 .py）に限る。Python の場合、テストの関数とは pytest が既定で集める形（名前が test で始まる関数、名前が Test で始まるクラスの中の test で始まるメソッド）を指す。ほかの言語のファイルは今の名前の形の判定のまま扱う
• 前提 2: 行数と危険度の判定（is_test_path）は今はパスの文字列だけで決めている。見分け方をそろえるために作業ディレクトリの中身を読む形へ変えてよい。読めないファイル（削除済み・リポジトリの外）は今の名前の判定で扱う
• 前提 3: ディレクトリの名前でテストの置き場所とみなす判定（tests/ spec/ など）は変えない。tests/ の配下にあるテスト関数を持たない補助のファイル（tests/test_helpers.py など）も、試走の候補からは外す（受け入れ条件 1 の対象に含む）
• 前提 4: プロダクションコードのファイルの名前を変える（test_strategy.py を別名へ移す）ことはこの課題の直し方にしない。ほかのリポジトリにも同じ構成がありうるため、見分け方の側で直す（Value 6）

対象範囲

含む:

• cross-refactoring のテンプレートの試走の候補選び
• --scope のテストの置き場所の判定（本体と同じ場所にテストを置く構成の判定）
• プロダクションコードの行数と危険度の判定でのテストのファイルの見分け方（S4）
• 上の 3 つを確かめるテスト

含まない:

• Python 以外の言語のテストの見分け方の変更（S5）
• plugins/ndf/scripts/lib/test_strategy.py・test_triage.py の名前の変更（前提 4）
• 試走の判定の表（template_verdict）の作り直し。終了コード 5 の扱いは「未決」に置く
• cross-refactoring 以外の Skill のテストの見分け方

ドメインイベント

E2 と E4 がこの課題の対象である。E2 でテスト関数を持たないファイルを根拠に置き場所とみなすと、E4 でそのファイルが候補になり、E5 がテストを 1 件も集めずに落ちる。

用語

受け入れ条件

• 1. Python のファイルで、名前が test_ で始まり、テストの関数（前提 1）を 1 つも持たないものは、試走の候補に入らない（--scope の配下・リポジトリ全体のどちらから選ぶ場合も）
• 2. --scope の 1 件の配下に、テストの関数を持たない test_*.py しか無いとき、その 1 件はテストの置き場所とみなされない（ディレクトリの名前で当たる場合を除く。前提 3）
• 3. 本文の「再現の手順」と同じ条件（plugins/ndf/scripts/lib/ のプロダクションコードのファイルだけを変え、refactor-scope.py の出力を --scope に渡す）で、init がテンプレートの試走で止まらず、scope_command を直す箇所として示す案内も出ない
• 4. テストの関数を持つ test_*.py（tests/ の配下・本体と同じ場所のどちらでも）は、今までどおり試走の候補とテストの置き場所の判定に入る
• 5. プロダクションコードの行数と危険度の判定で、テストの関数を持たない test_*.py はプロダクションコードとして数えられる（受け入れ条件 1・2 と同じ見分け方になる）
• 6. Python 以外の言語の名前の形（*.spec.ts・*_test.go など）とディレクトリの名前による判定の結果は変わらない（既存のテストが変更なしで通る）
• 7. 全体テストが通る

非機能の条件

影響

検証手段

前提とする取り決め

境界

未決

進行

モード: — / 進め方: auto

• 要求と受け入れ条件 — 2026-11-05 11:36
• 作業場所の用意 — 2026-11-05 20:01
• 設計 — 2026-11-05 11:45
• 素材の収集と出典の確定
• ドキュメント再構成 — 2026-11-05 11:48
• ドキュメントレビュー
• 計画 — 2026-11-05 20:01
• 実装 — 2026-11-05 20:01
• 構造改善 — 2026-11-05 21:15
• 実装レビュー — 2026-11-05 21:45
• 完了判定 — 2026-11-05 20:27
• Pull Request — 2026-11-05 19:16
• 確定仕様化 — 2026-11-05 19:14
• 後片付け — 2026-11-05 19:52
• 配布 — 2026-11-05 22:19
• 体裁レビュー
• リリース後テスト — 2026-11-05 19:47
• 振り返り — 2026-11-05 19:48
• 棚卸し — 2026-11-05 19:52

人の判断待ち（開発ループ lroot3）

• 判定: unknown（承認ゲート 1）
• 理由: 判定を読めない
• 根拠の項目: —
• 設計 Pull Request: cross-refactoring: テストの関数を持たない test_*.py をプロダクションコードとして扱い、init の試走が誤って止まらないようにする（#2516） #2558
• 答え: approved
• 答え方: python3 /persistent/ai/.claude/plugins/cache/ai-plugins/ndf/10.27.21/scripts/sprint-state.py loop answer lroot3 2516 --approved|--rejected|--revise --note <指摘ファイル>

振り返り: #2587 (comment)

振り返り: #2631 (comment)