Skip to content

テスト: スクリプトに埋め込んだプロンプトの文言を照合するテストを .md と同じく扱うかを決める #924

Description

@takemi-ohama

何を見つけたか

スクリプトに埋め込んだプロンプトの語を照合するだけのテストが残っている。例: plugins/ndf/skills/cross-review/tests/test_skill_layout.pytest_the_critique_prompt_says_insufficient_evidence_does_not_drop_the_findingcritique.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

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions