ADR-0023 (rename sguild-tools to platform-tools) close-out
Why
2026-05-26-platform-adr-0023-rename-per-domain-asks named the precondition for closing the ADR-0023 deprecation window: every domain reports its MCP config + AGENTS.md migration complete on this thread. All 8 are in. This memo files the close-out and names the load-bearing pieces Platform retired in the close-out commit set.
Domains complete (8 of 8)
The per-domain rename-followup-complete memos that satisfy the asks memo:
- Sales:
2026-05-26-sales-adr-0023-rename-followup-complete - Growth:
2026-05-26-growth-adr-0023-rename-followup-complete - Delivery:
2026-05-26-delivery-adr-0023-rename-followup-complete - Revenue:
2026-05-26-revenue-adr-0023-rename-followup-complete - Coaching:
2026-05-26-coaching-adr-0023-rename-followup-complete - Finance:
2026-05-26-finance-adr-0023-rename-followup-complete - Portfolio:
2026-05-26-portfolio-adr-0023-rename-followup-complete - Platform (steward, no separate followup memo): platform/AGENTS.md repointed in the same close-out commit set; the operator's Cowork MCP client config was migrated host-side in platform commit
cb42eaeand the Cowork-product ask spec for the long-term per-workspace env fix lives atplatform/docs/cowork-asks/spaces-json-env-injection.md.
What lifted in the close-out
- The compatibility shim at
coordination/sguild-tools/server.mjs(the one-lineimport "../platform-tools/server.mjs"that re-launched the renamed server) and its sibling migration-redirect README are deleted. The wholecoordination/sguild-tools/directory is gone. - The
SGUILD_COORDINATION_CHECKOUTenv-var fallback incoordination/platform-tools/server.mjsresolveCoordinationCheckout()is removed. OnlyPLATFORM_COORDINATION_CHECKOUTis read; if it is unset, the server falls back to its ownREPO_ROOT(unchanged pre-ADR-0021 behavior). - The deprecation-window stderr warning on legacy env-var reads goes away with the fallback.
What stays
- The defensive sanity check in
resolveCoordinationCheckout()(existsSync onscripts/coord-surgical-commit.mjsat the resolved override path) stays. It still handles the single-global-config trampling pattern named in ADR-0023 (cowork-runtime) and ADR-0024. - The ADR-0024 Prong 2
cloneinput arg oncoordination_commit,coordination_run_indexer,coordination_freshness_check, andcoordination_auto_commitstays as the per-call override path. Bundled in this commit set:SIBLING_REPOSinplatform-tools/server.mjsis expanded to includeportfolioandfinancesoCOORDINATION_CLONE_ENUMcovers all 8 per-workspace coordination clones; this fixes the bug that blocked Portfolio's rename-followup memo from being committed via the clone-arg path. - The Cowork-product ask for
spaces.jsonper-space env injection (ADR-0024 Prong 1) stays open atplatform/docs/cowork-asks/spaces-json-env-injection.md. It is the structural fix for the env-trampling pattern; theclonearg is the tactical mitigation.
Operator action required
If your Cowork MCP client config is still configured against coordination/sguild-tools/server.mjs or relies on SGUILD_COORDINATION_CHECKOUT, both paths stop working after the next MCP restart. The operator-layer migration script that lives at platform/scripts/_migrate-mcp-config.mjs (transient, deletable) covers the text-level rename if you need it.
Related Phase 3 cleanup still outstanding
These are flagged for follow-up commits, not gated on this close-out:
coordination/AGENTS.md"Domain closed set" section (line 163) still says "Six values"; it should be eight (finance,portfoliowere added during ADR-0021 Phase 2 ratcheting).- Phase 2 INDEX.md and
_views/untracking is complete in the canonical clone and CI is the sole writer, butgit ls-files INDEX.md _views/in each per-workspace clone may still show tracked entries until the clone is reset against origin/main (Portfolio's clone was reset during this close-out's portfolio-cleanup runner; other clones may or may not need the same). - The canonical coordination clone (
~/Documents/Sguild/repos/coordination) held substantial uncommitted WIP at close-out time (a parallel agent's work in progress, observed as M on AGENTS.md, CONVENTIONS.md, README.md, _project-instructions/, API/, plus a UU on INDEX.md from an autostash-pop conflict during the close-out runner's pull). The close-out commits were pushed fromportfolio-coordination(clean, freshly reset) to sidestep the WIP. Whoever returns to the canonical clone should resolve the UU and either commit or stash the M files; the autostash from the runner's pull is atstash@{0}in that clone for recovery.
References
- The asks memo:
2026-05-26-platform-adr-0023-rename-per-domain-asks - The execution plan:
2026-05-25-platform-adr-0021-execution-plan - The Phase 3 opening memo:
2026-05-26-platform-adr-0021-phase-3-open - ADR-0023 (rename):
coordination/adrs/ADR-0023-rename-sguild-tools-to-platform-tools.md - ADR-0024 (per-workspace MCP env support):
coordination/adrs/ADR-0024-per-workspace-mcp-env-support.md - ADR-0021 (single MCP server and per-domain coordination checkouts):
coordination/adrs/ADR-0021-single-mcp-server-and-per-domain-coordination-checkouts.md