Repository navigation
Bring the DESIGN.md index back under its line budget - #3523
Merged
Merged
Conversation
… line budget Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Code Review
This pull request updates DESIGN.md to allow closely related design notes to share a single line, separated by a middle dot (·). It applies this formatting change to several existing design notes to consolidate them, helping to keep the index within its 250-line budget. There are no review comments, and I have no feedback to provide.
kriszyp
marked this pull request as ready for review
October 24, 2026 03:45
kriszyp
enabled auto-merge (squash)
October 24, 2026 03:45
dawsontoth
added a commit
that referenced
this pull request
Oct 24, 2026
Takes #3523's DESIGN.md index, which shares closely related entries on one line with a `·` separator, and adds this branch's two index changes in that form: the deployment-record entry beside the install-fingerprint one, and the entry-point warning in the dangling-symlink description. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
kriszyp
added a commit
that referenced
this pull request
Oct 24, 2026
Picks up #3523, which brings the root DESIGN.md index back under its 250-line budget (runLinter was red on the old base). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Dispatch-Task: pr-maint-453f9cb53dca9dee9f4a96c955ec94da
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.
📝 Summary
runLinterfails onmain: rootDESIGN.mdis 251 lines bycheck:design-docs's count (budget 250) since Start and stop one isolated application without restarting the shared workers added two index lines. This merges three pairs of index lines for closely related notes into one line each (restart and drop, packer, config watchers), which brings the index to 248. No note is removed; every section is still linked. The index rules (intro, where a note goes) now allow two closely related notes to share a line, separated by·.✅ Verification
npm run check:design-docs: "index complete, links resolve, budgets met" (fails onmainwith "DESIGN.md is 251 lines").npx prettier --check DESIGN.mdpasses.DESIGN.md; whichever lands second rebases onto the lower count.PR QA: skipped @ 5b0126f — docs index only
🤖 Generated by Claude Opus 5.5 with Claude Code; posted via @kriszyp.
Release target: main — fixes main's lint gate; v5.3's index is separate
Related PRs: #3339 overlaps (merged; added the lines that crossed the budget), #3483 overlaps (open; also edits DESIGN.md), #3502 overlaps (open; also edits DESIGN.md)
Complexity: trivial
Review-Coverage: authored=claude; ran=cursor-composer,gemini; declined=codex,cursor-grok,muse,domain; rounds=1; full=1 @ d6c0774
Review-Attention: skim ~2m (raised: degraded review) @ d6c0774