Design docs: concise decision records (overlay #2000)#19
Merged
Conversation
… decisions only Office-hours design docs were written to ~/.gstack/projects/<opaque-id>/ and followed a 15-section template that re-argued approaches the user had already ruled out in conversation (upstream garrytan#2000, garrytan#1422; observed on the Examripper fixmatch doc). The overlay makes the repo docs/designs/ copy canonical, keeps the state-dir copy for downstream discovery, requires the absolute path whenever the doc is mentioned, and caps the doc at one page with ruled-out approaches collapsed to a single line each. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…one-page cap Per user feedback: no fixed page limit; bullets over paragraphs, every word earns its place, ruled-out approaches still one line each. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…rytan#886 as concision-only Main's garrytan#703 overlay already owns repo-local design-doc placement, so garrytan#2000 narrows to the concision judgment (bullets over paragraphs, one line per ruled-out approach, settled sections omitted). Parity inventory: 28 overlays, 4,914 checks. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
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.
What
Office-hours design docs were session transcripts: full sections re-arguing approaches the user had already ruled out in chat. This adds judgment overlay garrytan#2000 (GSTACK2_FIX_2000_DESIGN_DOC_CONCISION, targets office-hours): bullets over paragraphs, one bullet per decision with its why, ruled-out approaches collapsed to one line each, settled/empty template sections omitted. No fixed page cap; extra length must come from genuinely open questions.
Placement was originally part of this branch but main's garrytan#703 overlay (planfix merge) already owns repo-local
docs/designs/placement, so this narrows to concision only.Evidence
~/.gstack/projects/project_cdb.../sinabina-fixmatch-design-*.md(Examripper support-system doc), majority of content was chat-rejected approaches. Upstream issue Allow artifacts to be created in the project repo intended by the author garrytan/gstack#2000.bun run scripts/gstack2/run-parity.ts: 4,914 checks, 28 regression ports, green.bun run test:gstack2: exit 0.bun test: exit 0.bun run eval:select: 0/178 E2E, 0/28 LLM-judge tests affected by this diff.Ships as v1.61.1.0.
🤖 Generated with Claude Code
Summary by cubic
Updates
office-hoursdesign docs to concise decision records so they’re quick to read and act on. Removes transcript-style padding and keeps only decisions, reasons, and genuinely open questions.evals/parity/regressions/pr-2000.json; parity now 4,914 checks across 28 overlays.Written for commit 953bb0c. Summary will update on new commits.