docs(witan): give the two colliding ADRs their own numbers

[blarghmatey]

Numbers 0004 and 0006 were each used by two different decisions:

0004 Keycloak JWT per-user actor mapping ← keeps 0004
0004 Optional task phase field → 0008
0006 Code-graph branch ownership + reaping ← keeps 0006
0006 Stateless MCP protocol era → 0009

ADRs are cross-referenced by number from code comments, other ADRs, and
changelog entries ("see ADR-0005", "Related: ADR-0002"), so a duplicated
number makes those references ambiguous — a reader following "ADR-0006" cannot
tell which document is meant.

Which of each pair kept its number was decided by reference count, so the
renumber moves the fewest citations: keycloak carries 10 by-filename
references (and every bare ADR-0004 in the tree is about identity — oidc.py,
identity.py, omnigraph.py, config.py) against 1 for the phase tag; branch
ownership carries 7 against 4 for the stateless era.

This lands before the documentation site rather than after it deliberately.
Once the site publishes, /explanation/decisions/0006-stateless-.../ is a URL
people link to and search engines index, and renumbering then would break
those. Doing it first also means the site ships correct numbers from its first
build instead of carrying a note explaining the collision.

Two references needed judgement beyond a path rewrite:

• witan/config.py had a bare ADR-0006 meaning the stateless era. Left
  alone it would have silently started pointing at branch ownership.
• The changelog entry for the stateless work named both the number and the
  path. A changelog is a historical record, so the entry now reads "ADR-0009
  (…, numbered 0006 at the time)" rather than being quietly rewritten to look
  as though it always said 0009.

Verified: no reference to either old filename remains anywhere in the tree, no
ADR number is used twice, and witan-council's 839 tests pass.

Co-Authored-By: Claude Opus 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_017oVnpzjRJQVWq9hJGE2Rs7

---

_{Stack created with GitHub Stacks CLI • Give Feedback 💬}

[Copilot]

Pull request overview

Renumbers duplicate Witan ADRs and updates related references.

Changes:

• Renumbers the phase ADR from 0004 to 0008.
• Renumbers the stateless protocol ADR from 0006 to 0009.
• Updates selected citations and preserves historical changelog context.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
mcp/servers/witan/witan/config.py	Updates the stateless-era citation.
mcp/servers/witan/docs/adr/0009-stateless-mcp-protocol-era.md	Renumbers the stateless protocol ADR.
mcp/servers/witan/docs/adr/0008-optional-task-phase-tag.md	Renumbers the optional phase ADR.
mcp/servers/witan/docs/adr/0005-secure-cli-path-into-deployed-witan.md	Updates stateless ADR links.
mcp/servers/witan/docs/adr/0004-keycloak-jwt-per-user-actor-mapping.md	Updates its stateless ADR link.
mcp/servers/witan/CHANGELOG.md	Records the new and historical ADR numbers.

Suppressed comments (1)

mcp/servers/witan/docs/adr/0009-stateless-mcp-protocol-era.md:1

• Using 0009 creates another ambiguous identifier: ol-infrastructure's ADR-0009 is already cited by bare number in this tree (for example, 0005-secure-cli-path-into-deployed-witan.md:34,100 and 0007-local-to-shared-store-migration-transport.md:33,35). Those references now coexist with this local ADR-0009 and can no longer be resolved by number alone—the ambiguity this PR is intended to eliminate. Please use an unused number or qualify every existing ADR-0009 citation.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.