refactor(design-docs): dissolve product/ into agent/ server/ ui/ (RIG-2577 T8) - #825
Merged
Merged
Conversation
This was referenced Sep 1, 2026
Closed
rigel-mintaka
force-pushed
the
compass-repo/rig-2577-t8-collapse
branch
from
September 1, 2026 01:12
ec74314 to
97c81d0
Compare
…-2577 T8) Collapse of the T8 product/-dissolution work into one PR (the meta/ bucket landed separately as #820). Moves the remaining 69 product records into their ruled top-level buckets (RIG-2577 §Approach 2 roster + Matt's 2026-08-31 drift ratification), leaving product/ holding exactly the 8 T10-deferred native-family records. Buckets (69 records): - agent/ (+25): agent behavior/lifecycle/comms/tools/ask. Frozen roster's 19 agent-block records + 6 ratified drift (agent-forge-tools, agent-org-mgmt-tools, agent-peer-dm, ask-answer-recovery, manager-role-taxonomy, per-agent-overrides). - server/ (+18): server domain-model / write paths. Frozen roster's 11 + 7 ratified drift (forge-agent-notification, forge-app-credentials, forge-board-webhook-ingestion, linear-return-path-indirection-amendment, linear-webhook-mount-amendment, handle-addressing-cutover, linear-agent-responder). - ui/ (+26): the product's visible surfaces. Frozen roster's 24 (minus native family) + 2 ratified drift (coaching-tooltips, leader-chords). DEFERRED to T10 (RIG-3113; the native family stays under the transitional product/ governed root while RIG-2542's client-only direction settles): compass-native-app, -client-mode, -client-only, -packaging, tauri-shell.md, multi-window, stack-cross-process-teardown, gtk4-migration. After this PR product/ holds exactly these 8. Link + citation integrity (RIG-2577 Global Constraints / CONTRIBUTING rule 5): - DECISIONS.md Record cells re-pointed product/<r>/ -> <bucket>/<r>/ (284 rows unchanged; pure relocation, no decision changed). - Cross-bucket body links: agent->server (forge-tools, spawn-despawn), server->ui (zulip-threading-model). - External citation sweep: proto sources (agent, agent_gateway, forge) + their generated pb.go/pb.ts comments; Go forge/store/runner/linearagent/server comments; compass-agent TS comments; .github/workflows/ci.yml and apps/ui/.stylelintrc.cjs comments; gate test fixtures; inter-record prose. Fixes every reorg-caused dangling link a two-pass review surfaced across the whole tree (not just the per-bucket touched sets), including surfaces #820 moved targets out from under: - Bucket-less sibling arch-lineage links now that arch-lineage lives in meta/: agent/compass-agent-container-runtime.md and the deferred product/compass-tauri-shell.md -> ../meta/ (flat-file depth, one ../). - docs/concepts/ inbound links (outside the designs tree, missed by a product/-token sweep): comms-model.md -> server/zulip-threading-model, agent/agent-session-persistence, agent/ask-comms-roundtrip; self-host-and-managed.md -> server/linear-agent-responder. - docs/specs/product/compass.md arch-lineage citation -> designs/meta/. - docs/specs/product/README.md: product/ dir references rewritten to the domain-bucketed designs corpus indexed by DECISIONS.md (product/ is no longer the record home). - docs/designs/infra/ci/compass-dogfood-e2e/design.md forge-integration-testing link -> ../../../server/ (target moved product/->server/ here; the ../ depth was also one short on base — corrected in the same edit). Verified: design-ledger-gate OK (284 rows, 120 headers); eng-docs gather 73/73; gate suites 103/103; UI touched-file tests green; biome clean; every re-pointed link resolves; no dangling reorg-caused product/ refs remain (deferred natives + frozen reorg recon prose + synthetic test fixtures excluded); proto swept so buf regen introduces no drift. Out of scope (pre-existing, not reorg-caused, left for a separate cleanup): ~28 self-referential ledger-quote anchors of the form <own-record>/design.md#L that dangled on the merged base independent of any product/ move. Ledger-impact: records relocated within governance, no decision changed Co-authored-by: Matt Wilkinson <matt@rigel.build>
rigel-mintaka
force-pushed
the
compass-repo/rig-2577-t8-collapse
branch
from
September 1, 2026 01:15
97c81d0 to
16c4643
Compare
Contributor
Author
T9 — post-move verification sweep (evidence)This collapse PR is the final T8 move PR, so T9's straggler-fixing + zero-reference proof fold in here (per the frozen record, T9 "Produces: verification evidence in the final PR description; any stragglers fixed"). The T10 native-family sweep + Zero-reference proof (run on the collapse tree, head
|
| Surface | Was | Now |
|---|---|---|
agent/compass-agent-container-runtime.md:7 |
bare compass-architecture-lineage/… |
../meta/… |
product/compass-tauri-shell.md:7 |
../../meta/… (overshoot) |
../meta/… |
docs/concepts/comms-model.md:27 |
designs/product/compass-zulip-threading-model |
designs/server/… |
docs/concepts/comms-model.md:28 |
designs/product/compass-agent-session-persistence |
designs/agent/… |
docs/concepts/comms-model.md:68 |
designs/product/compass-ask-comms-roundtrip |
designs/agent/… |
docs/concepts/self-host-and-managed.md:53 |
designs/product/compass-linear-agent-responder |
designs/server/… |
docs/specs/product/compass.md:9 |
designs/product/compass-architecture-lineage |
designs/meta/… |
docs/specs/product/README.md:5,17 |
../../designs/product/ (dir prose) |
bucketed corpus + DECISIONS.md |
docs/designs/infra/ci/compass-dogfood-e2e/design.md:831 |
../../product/compass-forge-integration-testing |
../../../server/… (also a base ../-depth bug) |
.github/workflows/ci.yml, apps/ui/.stylelintrc.cjs |
product/… citation comments |
bucketed |
Every re-pointed target verified to exist in the final tree.
Out of scope (pre-existing, NOT reorg-caused)
~28 self-referential ledger-quote anchors of the form <own-record>/design.md#anchor dangle on the merged base independent of any product/ move — left for a separate corpus link-clean pass, not a T8/T9 straggler.
Gates
design-ledger-gate OK · eng-docs gather 73/73 · gate suites (design-ledger-gate + sea-ref-gate) 103/103 · UI touched-file tests green · biome clean.
mattwilkinsonn
approved these changes
Sep 1, 2026
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
2 participants
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.
Collapse of the T8 product/-dissolution work into one PR (the meta/ bucket landed
separately as #820). Moves the remaining 69 product records into their ruled
top-level buckets (RIG-2577 §Approach 2 roster + Matt's 2026-08-31 drift
ratification), leaving product/ holding exactly the 8 T10-deferred native-family
records.
Buckets (69 records):
agent-block records + 6 ratified drift (agent-forge-tools, agent-org-mgmt-tools,
agent-peer-dm, ask-answer-recovery, manager-role-taxonomy, per-agent-overrides).
7 ratified drift (forge-agent-notification, forge-app-credentials,
forge-board-webhook-ingestion, linear-return-path-indirection-amendment,
linear-webhook-mount-amendment, handle-addressing-cutover, linear-agent-responder).
family) + 2 ratified drift (coaching-tooltips, leader-chords).
DEFERRED to T10 (RIG-3113; the native family stays under the transitional product/
governed root while RIG-2542's client-only direction settles): compass-native-app,
-client-mode, -client-only, -packaging, tauri-shell.md, multi-window,
stack-cross-process-teardown, gtk4-migration. After this PR product/ holds exactly
these 8.
Same-PR link integrity (RIG-2577 Global Constraints):
unchanged; pure relocation, no decision changed).
server->ui (zulip-threading-model), and two bare-sibling arch-lineage links
(agent/container-runtime, deferred product/tauri-shell) re-pointed to their
target buckets (../meta/, ../../server/, ../../ui/ canonical forms).
generated pb.go/pb.ts comments; Go forge/store/runner/linearagent/server
comments; compass-agent TS comments; .github/workflows/ci.yml and
apps/ui/.stylelintrc.cjs comments; gate test fixtures; inter-record prose.
Also fixes the four link/citation stragglers a review of the pre-collapse stack
surfaced (two dangling arch-lineage bare-sibling links that #820 merged without,
the ci.yml + stylelintrc citations outside the per-bucket touched sets).
Verified: design-ledger-gate OK (284 rows, 120 headers); eng-docs gather 73/73;
gate suites 103/103; UI touched-file tests green; biome clean; no dangling product/
refs to moved records (deferred natives + frozen reorg recon prose excluded);
proto swept so buf regen introduces no drift.
Ledger-impact: records relocated within governance, no decision changed
Co-authored-by: Matt Wilkinson matt@rigel.build