refactor(design-docs): dissolve product/ into ui/ — visible-surface records (RIG-2577 T8)

apps/ui/src/components/BadgeGlyph.test.tsx

@@ -11,7 +11,7 @@ import { BadgeGlyph } from "./BadgeGlyph";
 // a CSS-selector change could silently break.
 
 // The six frozen glyph grids, transcribed verbatim from the ASCII art in
-// docs/designs/product/compass-badge-clarity/design.md §"The six glyph grids"
+// docs/designs/ui/compass-badge-clarity/design.md §"The six glyph grids"
 // (`#` = lit, `.` = off; 9×9). This is an INDEPENDENT copy of the contract —
 // the component transcribes the same grids into [x,y] coordinate arrays, so
 // comparing the rendered rects against these grids catches a wrong grid even

docs/designs/meta/compass-design-ledger/design.md

@@ -48,7 +48,7 @@ citing `packages/compass-agent/src/mapping.ts:16-19`:
 > not opaque bytes, and explicitly **not ACP**."
 
 The sibling record even flags the comment as stale
-(`docs/designs/product/compass-message-surface-rendering/design.md:448-451`:
+(`docs/designs/ui/compass-message-surface-rendering/design.md:448-451`:
 "Compass-0.8 ruled the trace 'not opaque bytes, and explicitly **not ACP**'
 … the comment's vintage is stale") — but no surface an agent reads *first*
 carried the supersession. Supersession churn is real and concentrated:

docs/designs/server/compass-zulip-threading-model/design.md

@@ -14,8 +14,8 @@ Status: Active
 > (store + comms), compass-agent (tools), compass-ui, compass-runner (gateway).
 > **Supersedes by citation** (never rewrites the merged records):
 > `compass-0.8-threading-and-session-renderer` (retired, RIG-2453),
-> [compass-threading-ui](../../product/compass-threading-ui/design.md),
-> [compass-slack-thread-rendering](../../product/compass-slack-thread-rendering.md),
+> [compass-threading-ui](../../ui/compass-threading-ui/design.md),
+> [compass-slack-thread-rendering](../../ui/compass-slack-thread-rendering.md),
 > ledger rows **DL-040** and **DL-041**, and **RIG-1364 T3's conversation
 > write-through** (the streamed-turn → comms auto-post path — D7/T7);
 > **clarifies** DL-037, DL-029, DL-028, DL-072. The ledger delta ships in this

docs/designs/product/compass-dock-in-sidebar/design.md → docs/designs/ui/compass-dock-in-sidebar/design.md

@@ -573,7 +573,7 @@ living-spec check: `docs/specs/product/compass.md` contains no dock references
 
 - **The frozen record's reasoning is never rewritten.** This record supersedes
   ADE-shell D7, amends ADE-shell D5, and reverses ADE-shell resolved decision 7.
-  The only edit to `docs/designs/product/compass-ade-shell/design.md` is the
+  The only edit to `docs/designs/ui/compass-ade-shell/design.md` is the
   bidirectional supersede/amend **pointer** the house convention requires — a
   one-blockquote note atop D7 and D5 pointing forward to this record, so a
   reader of the frozen decision in isolation learns it was overridden (the

docs/designs/product/compass-keyboard-shortcuts-overlay/design.md → docs/designs/ui/compass-keyboard-shortcuts-overlay/design.md

@@ -544,7 +544,7 @@ independently of RIG-2529's schedule.
 ### Merge coordination with RIG-2483 (command palette)
 
 This record and the palette record
-(`docs/designs/product/compass-command-palette/design.md`, RIG-2483) rewrite
+(`docs/designs/ui/compass-command-palette/design.md`, RIG-2483) rewrite
 the SAME four regions: the `createKeyboardSpine` deps object + signature
 (`spine.ts:66-68`), the one-line spine creation call (`store.ts:1896`), the
 `App.tsx:4-8` import run plus a shell-root `<Show>`, and the

docs/designs/product/compass-keyboard-spine-app-root/design.md → docs/designs/ui/compass-keyboard-spine-app-root/design.md

@@ -4,7 +4,7 @@ Status: Draft
 
 Realization design for the slice the parent record froze and RIG-2130
 deferred: the parent record
-`docs/designs/product/compass-bridge-keyboard-nav/design.md` T1 (§352-416)
+`docs/designs/ui/compass-bridge-keyboard-nav/design.md` T1 (§352-416)
 froze the WHAT — install the keymap dispatcher **once at the App root**
 ("install at the App root (`App.tsx:35-42`, where the store's router seam
 already binds)", §403-405) and register `view.bridge → store.showBridge()` as

docs/designs/product/compass-sidebar-pins-unreachable-amendment/design.md → docs/designs/ui/compass-sidebar-pins-unreachable-amendment/design.md

@@ -14,7 +14,7 @@ ledger gate's touch-coupling leg is satisfied directly; no `Ledger-impact:`
 escape hatch is needed in the PR body.
 
 > **Amends `compass-sidebar-pins` (frozen).** This record is a sibling
-> amendment to `docs/designs/product/compass-sidebar-pins/design.md` (RIG-1632,
+> amendment to `docs/designs/ui/compass-sidebar-pins/design.md` (RIG-1632,
 > DL-096). The merged record is frozen; per sealed convention a later change
 > ADDS a record. This amendment supersedes the frozen record's §T2
 > unresolvable-pin filtering and §T3 fluctuation-coercion clauses per Matt's
@@ -113,7 +113,7 @@ the new contract.
 ## What this amendment supersedes
 
 Both superseded clauses live in the frozen
-`docs/designs/product/compass-sidebar-pins/design.md` (Status: Active,
+`docs/designs/ui/compass-sidebar-pins/design.md` (Status: Active,
 Tracker RIG-1632, ledger DL-096). Everything else in that record — the
 configurable pin layer, Supervisor/Warden removal, empty default set,
 per-workspace persistence, boot-on-first-resolvable-pin, the tree affordance

docs/designs/product/compass-sidebar-pins/design.md → docs/designs/ui/compass-sidebar-pins/design.md

@@ -277,7 +277,7 @@ common abstraction now is speculative generality with one real consumer.
 
 ### T1 — this record + ledger row (this PR, docs-only)
 
-Freeze this record at `docs/designs/product/compass-sidebar-pins/design.md`
+Freeze this record at `docs/designs/ui/compass-sidebar-pins/design.md`
 and append DL-096 to `docs/designs/product/DECISIONS.md` in the same diff
 (§Ledger delta). No code changes.
 

tools/design-ledger-gate/index.test.ts

@@ -840,13 +840,13 @@ describe("evaluate — record-level Superseded pointer", () => {
 				[],
 				[
 					header({
-						path: "docs/designs/product/compass-ade-shell/design.md",
+						path: "docs/designs/ui/compass-ade-shell/design.md",
 						statusLine:
 							"Status: Superseded by ../compass-dock-in-sidebar/design.md",
 					}),
 				],
 				noChange,
-				onlyExists("product/compass-dock-in-sidebar/design.md"),
+				onlyExists("ui/compass-dock-in-sidebar/design.md"),
 			),
 		).toEqual([]);
 	});
@@ -872,18 +872,18 @@ describe("evaluate — record-level Superseded pointer", () => {
 	test("a designs-root-relative form does NOT resolve from a nested record (base is locked)", () => {
 		// Writing the pointer bucket-qualified (`compass-dock-in-sidebar/design.md`)
 		// from inside a nested record is wrong: it resolves record-relative to
-		// `product/compass-ade-shell/compass-dock-in-sidebar/design.md`, which the
+		// `ui/compass-ade-shell/compass-dock-in-sidebar/design.md`, which the
 		// resolver rejects.
 		const vs = evaluate(
 			[],
 			[
 				header({
-					path: "docs/designs/product/compass-ade-shell/design.md",
+					path: "docs/designs/ui/compass-ade-shell/design.md",
 					statusLine: "Status: Superseded by compass-dock-in-sidebar/design.md",
 				}),
 			],
 			noChange,
-			onlyExists("product/compass-dock-in-sidebar/design.md"),
+			onlyExists("ui/compass-dock-in-sidebar/design.md"),
 		);
 		expect(vs.length).toBe(1);
 		expect(vs[0]?.message).toContain("does not resolve");

tools/sea-ref-gate/index.test.ts

@@ -56,20 +56,20 @@ describe("isCarveOut", () => {
 		expect(isCarveOut("bun.lock")).toBe(true);
 	});
 	test("a canonical docs/ record is NOT carved out", () => {
-		expect(
-			isCarveOut("docs/designs/product/compass-board-view/design.md"),
-		).toBe(false);
+		expect(isCarveOut("docs/designs/ui/compass-board-view/design.md")).toBe(
+			false,
+		);
 	});
 });
 
 describe("findViolations", () => {
 	test("flags a real reference in a scanned file", () => {
 		const v = findViolations([
-			"docs/designs/product/compass-board-view/design.md:12:tracked as SEA-1512 here",
+			"docs/designs/ui/compass-board-view/design.md:12:tracked as SEA-1512 here",
 		]);
 		expect(v).toHaveLength(1);
 		expect(v[0]).toEqual({
-			file: "docs/designs/product/compass-board-view/design.md",
+			file: "docs/designs/ui/compass-board-view/design.md",
 			line: 12,
 			text: "tracked as SEA-1512 here",
 		});