docs(product): Compass DS token full-cutover + canvas migration design - #234
Open
seal-agent wants to merge 2 commits into
Open
docs(product): Compass DS token full-cutover + canvas migration design#234seal-agent wants to merge 2 commits into
seal-agent wants to merge 2 commits into
Conversation
Full-cutover design record superseding the DL-157 incremental strangler per Matt's ruling (2026-08-07): a pre-dogfood codebase has no live users to protect, so the legacy :root token tier is deleted and every consumer flips to --cx-* in one atomically-merged migration — canvas to Night Owl, base.css un-shadowed, state dots recolored, D7 stylelint guard wired at error — proven by a Playwright visual-smoke harness Matt reviews. Ledger: flip DL-157 -> Superseded by DL-161; append DL-161. Ledger-impact: DL-161 added; DL-157 superseded. Four load-bearing Open Questions parked for Matt (overnight): the 4 --purple consumers, the --st-merged PR badge, --accent-dim fidelity + paired foreground, and PR-train topology. Record merges once ruled. Refs SEA-1876 Co-authored-by: Matt Wilkinson <matt@sealedsecurity.com>
Incomplete Open-Questions renumbering left 5 dangling refs after the scrollbar-hover OQ was demoted to a stated assumption and topology became OQ-4. Reconcile every OQ-n reference against the final 4-item list (OQ-1 purple, OQ-2 st-merged, OQ-3 accent-dim, OQ-4 topology): - L119-122 / T2: scrollbar-hover now points at the stated-assumption bullet, not a phantom OQ-4. - Global constraints: "see OQ-5 topology" / "is OQ-5" -> OQ-4. - tokens.css read-only note: drop OQ-4 from the compass-ux-coined list (topology coins no token); keep OQ-2. - T3: "4 --purple refs" -> "4 remaining --purple consumers (5 var() refs)" to match the ref/consumer vocabulary the record uses. Co-authored-by: Matt Wilkinson <matt@sealedsecurity.com>
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.
Full-cutover design record superseding the DL-157 incremental strangler
per Matt's ruling (2026-08-07): a pre-dogfood codebase has no live users
to protect, so the legacy :root token tier is deleted and every consumer
flips to --cx-* in one atomically-merged migration — canvas to Night Owl,
base.css un-shadowed, state dots recolored, D7 stylelint guard wired at
error — proven by a Playwright visual-smoke harness Matt reviews.
Ledger: flip DL-157 -> Superseded by DL-161; append DL-161.
Ledger-impact: DL-161 added; DL-157 superseded.
Four load-bearing Open Questions parked for Matt (overnight): the 4
--purple consumers, the --st-merged PR badge, --accent-dim fidelity +
paired foreground, and PR-train topology. Record merges once ruled.
Refs SEA-1876
Co-authored-by: Matt Wilkinson matt@sealedsecurity.com