From b4daa00b4e0c9f163b63bbe35b9360f986e9c629 Mon Sep 17 00:00:00 2001 From: mlugo-apx <37851545+mlugo-apx@users.noreply.github.com> Date: Thu, 12 Feb 2026 00:31:18 -0500 Subject: [PATCH] fix(prompt): resolve contradiction in response.ts blocking without test output MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove "No test output captured" from example block reasons — it contradicts the decision value definition where null means "insufficient information to determine." Add matching approval reason for missing test output. Without this fix, projects using custom/browser-based test frameworks (or any framework without a supported reporter) get blocked on every implementation edit because test.json is never created. Co-Authored-By: Claude Opus 4.6 --- src/validation/prompts/response.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/validation/prompts/response.ts b/src/validation/prompts/response.ts index d13838fa..eafdb40e 100644 --- a/src/validation/prompts/response.ts +++ b/src/validation/prompts/response.ts @@ -25,12 +25,12 @@ When blocking, your reason must: - "Over-implementation violation. Test fails with 'Calculator is not defined' but implementation adds both class AND method. Create only an empty class first, then run test again." - "Refactoring without passing tests. Test output shows failures. Fix failing tests first, ensure all pass, then refactor." - "Premature implementation - implementing without a failing test. Write the test first, run it to see the specific failure, then implement only what's needed to address that failure." -- "No test output captured. Cannot validate TDD compliance without test results. Run tests using standard commands (npm test, pytest) without output filtering or redirection that may prevent the test reporter from capturing results." #### Example Approval Reasons: - "Adding single test to test file - follows TDD red phase" - "Minimal implementation addressing specific test failure" - "Refactoring with evidence of passing tests" +- "No test output available - insufficient information to determine TDD violation, approving changes" ### Focus Remember: You are ONLY evaluating TDD compliance, not: