何を見つけたか
スクリプトに埋め込んだプロンプトの語を照合するだけのテストが残っている。例: plugins/ndf/skills/cross-review/tests/test_skill_layout.py の test_the_critique_prompt_says_insufficient_evidence_does_not_drop_the_finding。critique.sh の本文に "数から落ち" という語があるかを assert ... in read_text() で見るだけで、スクリプトもプロンプトも実行しない。
どこで見つけたか
PR #918 のクロスレビュー(kiro の指摘、test_skill_layout.py:10)。
なぜこの変更の範囲外なのか
#885 の方針(tdd-cycle/references/test-quality.md の例 8)が対象にするのは、リポジトリで追跡している .md の文言を照合するテストである。スクリプト(.sh・.py)に埋め込んだプロンプトの文言は対象に入っていない。設計(issues/issue-880-885-design.md の「分類の規則」)も .md について決めている。
直さないと何が起きるか
.md では「文言を固定しても振る舞いを保証しない」として削ったのと同じ性質の照合が、置き場所がスクリプトだという理由だけで残る。プロンプトの言い回しを直すたびに、このテストも直すことになる。方針を .sh / .py に埋め込んだ文言にも広げるか(広げるなら何を残すか。例: スクリプトが解析するプレースホルダは残す)を決める必要がある。
由来
PR #918(issue #885)
何を見つけたか
スクリプトに埋め込んだプロンプトの語を照合するだけのテストが残っている。例:
plugins/ndf/skills/cross-review/tests/test_skill_layout.pyのtest_the_critique_prompt_says_insufficient_evidence_does_not_drop_the_finding。critique.shの本文に"数から落ち"という語があるかをassert ... in read_text()で見るだけで、スクリプトもプロンプトも実行しない。どこで見つけたか
PR #918 のクロスレビュー(kiro の指摘、
test_skill_layout.py:10)。なぜこの変更の範囲外なのか
#885 の方針(
tdd-cycle/references/test-quality.mdの例 8)が対象にするのは、リポジトリで追跡している.mdの文言を照合するテストである。スクリプト(.sh・.py)に埋め込んだプロンプトの文言は対象に入っていない。設計(issues/issue-880-885-design.mdの「分類の規則」)も.mdについて決めている。直さないと何が起きるか
.mdでは「文言を固定しても振る舞いを保証しない」として削ったのと同じ性質の照合が、置き場所がスクリプトだという理由だけで残る。プロンプトの言い回しを直すたびに、このテストも直すことになる。方針を.sh/.pyに埋め込んだ文言にも広げるか(広げるなら何を残すか。例: スクリプトが解析するプレースホルダは残す)を決める必要がある。由来
PR #918(issue #885)