chore(claude): add unknowns-discovery phases to /plan and /lfg - #58
Merged
Conversation
/plan gains a "Surface the unknowns" phase: a blindspot pass plus a one-question-at-a-time AskUserQuestion interview before design, with answers recorded as "Settled in interview" constraints in the plan. /lfg gains a deviation log (implementation-notes.md, folded into a mandatory "Deviations & judgment calls" PR-body section) and a comprehension close-out (key decisions + three merge-gate questions). Rationale: catch unknowns before execution in discovery loops, instead of after execution in review-fix loops.
The "Surface the unknowns" phase landed after Design; the interview must settle open decisions BEFORE candidate approaches are developed. Now Phase 2 (unknowns) precedes Phase 3 (Design), matching the other repos.
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
1 participant
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.
Summary
/plangains a new "Surface the unknowns" phase (Phase 3, renumbering the old Phase 3 → 4 and Phase 4 → 5): a blindspot pass over open decisions/edge-cases/no-precedent territory, followed by a one-question-at-a-timeAskUserQuestioninterview prioritized by blast radius. Answers are recorded in the plan's Decision section asSettled in interview:bullets the executor must not re-litigate.AskUserQuestionwas added to the command'sallowed-tools./lfggains a deviation log:implementation-notes.mdat the repo root, written the moment the plan and reality diverge (deviations / discoveries / judgment calls), folded into a mandatory## Deviations & judgment callsPR-body section and then deleted./lfggains a comprehension close-out (new Phase 8, after Commit & PR): the final message must surface the 3–5 non-obvious decisions (leading with the deviation log) and three merge-gate questions the user should be able to answer before merging.Why
Catch unknowns before execution (discovery) instead of after (review-fix loops); prose-only change to
.claude/commands/.Test plan
.claude/commands/plan.mdand.claude/commands/lfg.md