Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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 templates/commands/clarify.md
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ Execution steps:

For each category with Partial or Missing status, add a candidate question opportunity unless:
- Clarification would not materially change implementation or validation strategy
- Information is better deferred to planning phase (note internally)
- The item is specifically about implementation method, tech-stack comparison, or task breakdown (note internally)

4. Generate (internally) a prioritized queue of candidate clarification questions (maximum 5). Do NOT output them all at once. Apply these constraints:
- Maximum of 5 total questions across the whole session.
Expand Down
19 changes: 19 additions & 0 deletions tests/test_clarify_stage_gate.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
"""The planning deferral in ``templates/commands/clarify.md`` must stay bounded (#1717).

The old catch-all ("Information is better deferred to planning phase") let
agents skip NFRs, acceptance criteria, and edge cases. Defer only
implementation method, tech-stack comparison, or task breakdown.
"""

from pathlib import Path

CLARIFY = Path(__file__).parent.parent / "templates" / "commands" / "clarify.md"


def test_clarify_planning_deferral_is_bounded() -> None:
text = CLARIFY.read_text(encoding="utf-8")
assert "- Information is better deferred to planning phase (note internally)" not in text
assert (
"implementation method, tech-stack comparison, or task breakdown"
in text
)