Repository navigation
Allow an optional decision round for vague plans - #9
Merged
cursor[bot] merged 2 commits intoOct 7, 2026
Merged
Conversation
When the assignment is missing a goal, boundary, interface, success check, or material trade-off, planning may ask only what blocks a sound plan and offer a recommendation. Clear assignments still need no interview. Co-authored-by: Dennis Geldmacher <geldmacher.dennis@outlook.de>
The vague-scope decision round put planSetupGap 19 tokens over 3500. Record that one scenario at the measured maximum plus 10%. Co-authored-by: Dennis Geldmacher <geldmacher.dennis@outlook.de>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Idea
When a planning assignment is vague or underspecified,
plan-workmay run a short decision round and offer a concrete recommendation the user can accept or steer. The round asks only what blocks a sound plan: a missing goal, boundaries, interfaces, success checks, or a material trade-off that would change the plan. A few focused questions, then stop.This is the small positive alternative Dennis asked for after workflow#8. That discussion already argued against importing Matt Pocock’s grill-me and grill-with-docs. The spark was Matt’s “specialize grill-me” note (tweet): a product-design bar, a security survey, a preview, and a relentless design-tree interview. Workflow takes only the useful fragment for vague scope, and leaves the rest where it already lives.
Clear assignments are unchanged. They still need no interview and no advance design of routine details.
Why this shape
The round stays inside existing
plan-work. It does not become a new skill, a slash command, or a gate on Plan → Implement or Auto-Work. A clear assignment still proceeds straight to a proportionate plan. Dark and Light still do not resolve consequential missing choices by assumption; the recommendation is something to accept or steer, not a silent default.Matt’s other three tips are already covered elsewhere, so this change does not pull them into Workflow:
What was rejected
What changed
skills/plan-work/SKILL.md— optional short decision round for a vague assignment, with the existing “clear assignments need no interview” rule left in place.docs/manual-workflow.mdanddocs/behavior-validation.md— the same rule beside the existing clarify guidance, including a walkthrough row that still expects no interview for a clear small change.docs/methodology-sources.md— why this is the lean alternative to grill-me, and what was not imported.scripts/context-limits.json—planSetupGapwas 3519 against a 3500 limit after the plan-work paragraph. That one scenario is now 3900 (measured maximum plus 10%, rounded up to 100). Other limits are unchanged.No new skill folder.
Checks
npm run link-check,npm run validate, andnpm run context-budgetpassed. Cursorplanis 2132 estimated tokens (limit 2900). CursorplanSetupGapis 3519 (limit 3900).npm test: 77 passed. The 24 GitHub release tests fail in this environment becauseoriginis rewritten to an authenticated URL (repository must identify one GitHub repository). That check is unchanged by this diff.