Skip to content

fix(prompt): resolve contradiction blocking without test output #97

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mlugo-apx wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(prompt): resolve contradiction blocking without test output #97

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/validation/prompts/response.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down