|
1 | 1 | # ADR 003: Host-Neutral Canonical Coordinator in the Workspace |
2 | 2 |
|
3 | | -**Status:** Proposed |
| 3 | +**Status:** Accepted (option C), 2026-11-13 |
4 | 4 |
|
5 | 5 | **Tracking:** [#2438](https://github.com/github/squad/issues/2438) (Phase 7) |
6 | 6 |
|
7 | 7 | **Affects:** `squad-client-harness-adapters/v0.2` §4–§5 (projection contract |
8 | 8 | and canonical paths) and §13 (upgrade and rollback) |
9 | 9 |
|
10 | | -This ADR records a decision to be made. It does not change any code. Merging it |
11 | | -with status Proposed does not approve it. |
| 10 | +The maintainers chose option C. Option B stays an opt-in experiment |
| 11 | +([#2488](https://github.com/github/squad/issues/2488)) and is not part of 2.0. |
12 | 12 |
|
13 | 13 | ## Context |
14 | 14 |
|
@@ -185,12 +185,28 @@ coordinator behaves correctly, so C must be shown with behavioral scenarios run |
185 | 185 | with the Copilot file absent. C is tracked in |
186 | 186 | [#2487](https://github.com/github/squad/issues/2487) and the B spike in |
187 | 187 | [#2488](https://github.com/github/squad/issues/2488). This records the |
188 | | -discussion; it does not accept this ADR. |
| 188 | +discussion; the decision is recorded below. |
189 | 189 |
|
190 | 190 | ## Decision |
191 | 191 |
|
192 | | -Not yet made. Implementation, if approved, is a separate PR with its own |
193 | | -projection version bump and CHANGELOG entry. |
| 192 | +**Option C**, agreed by the maintainers on 2026-11-13. |
| 193 | + |
| 194 | +- `.github/agents/squad.agent.md` stays the canonical coordinator, and it is |
| 195 | + defined only for Copilot. Copilot's output doesn't change. |
| 196 | +- Every other host's digest is complete on its own and never points at |
| 197 | + Copilot's file. The digest parity tests keep the hosts in step. This shipped |
| 198 | + in [#2489](https://github.com/github/squad/pull/2489). |
| 199 | +- Behavioral scenarios ran live on Claude Code with Copilot's file deleted, and |
| 200 | + passed: routing, human sign-off, reviewer rejection, restart continuity, |
| 201 | + customization precedence, monorepo subfolder, backend outage and the |
| 202 | + satellite repository. Azure DevOps is deferred. The evidence is in |
| 203 | + [#2487](https://github.com/github/squad/issues/2487) and the host feature |
| 204 | + matrix. |
| 205 | +- Option B (a host-neutral canonical file that Copilot's is generated from) is |
| 206 | + not adopted. It stays a bounded, opt-in spike in |
| 207 | + [#2488](https://github.com/github/squad/issues/2488). Revisit it if Copilot's |
| 208 | + file needs host-specific content, or if remote teams need the coordinator |
| 209 | + served through `squad_state`. |
194 | 210 |
|
195 | 211 | ## Related artifacts |
196 | 212 |
|
|
0 commit comments