Skip to content

docs(adr-003): accept option C - #2501

Merged
bradygaster merged 2 commits into
devfrom
squad/adr-003-option-c
Nov 14, 2026
Merged

bradygaster merged 2 commits into
devfrom
squad/adr-003-option-c

Conversation

@bradygaster

Copy link
Copy Markdown
Collaborator

Records the maintainers' decision on ADR-003: option C.

This closes the last #2487 done-when item that needed maintainer agreement. Docs only, with no code or projection change.

🤖 Generated with Claude Code

The maintainers agreed on option C: Copilot's coordinator file stays canonical,
and every other host's digest is self-sufficient (#2489). Option B stays an
opt-in spike (#2488).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI dynamic review requested due to automatic review settings November 14, 2026 04:27
@github-actions

Copy link
Copy Markdown
Contributor

🟢 Impact Analysis — PR #2501

Risk tier: 🟢 LOW

📊 Summary

Metric Count
Files changed 1
Files added 0
Files modified 1
Files deleted 0
Modules touched 1

🎯 Risk Factors

  • 1 files changed (≤5 → LOW)
  • 1 module(s) touched (≤1 → LOW)

📦 Modules Affected

docs (1 file)
  • docs/specification/adr-003-host-neutral-canonical-coordinator.md

This report is generated automatically for every PR. See #733 for details.

@github-actions

github-actions Bot commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

🛫 PR Readiness Check

ℹ️ This comment updates on each push. Last checked: commit ebec172

PR Scope: 🔧 Infrastructure

⚠️ 5 item(s) to address before review

Status Check Details
❌ Single commit 2 commits — consider squashing before review
✅ Not in draft Ready for review
❌ Branch up to date dev is 1 commit(s) ahead — rebase recommended
❌ Copilot review No Copilot review yet — it may still be processing
✅ Changeset present No source files changed — changeset not required
✅ Scope clean No .squad/ or docs/proposals/ files
✅ No merge conflicts Merge status unknown — GitHub is still computing
❌ Copilot threads resolved 1 unresolved Copilot thread(s) — fix and resolve before merging
❌ CI passing 2 check(s) still running

Files Changed (1 file, +22 −6)

File +/−
docs/specification/adr-003-host-neutral-canonical-coordinator.md +22 −6

Total: +22 −6


This check runs automatically on every push. Fix any ❌ items and push again.
See CONTRIBUTING.md and PR Requirements for details.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

An active documentation inconsistency remains: the host feature matrix does not reflect passing bridge/live scenarios and still shows the affected capabilities as unsupported because of known digest gaps.

1 open issue
What changed in this PR

This documentation-only PR records the maintainers’ acceptance of ADR-003 option C: Copilot retains its canonical coordinator file, while non-Copilot digests remain self-sufficient. Option B remains an opt-in experiment outside the 2.0 scope.

Changes:

  • Updates ADR-003 status from Proposed to Accepted.
  • Documents option C’s architectural decision and behavioral validation.
  • Records option B’s follow-up scope and related implementation references.
File Description
docs/​specification/​adr-003-host-neutral-canonical-coordinator.md Records the accepted option C decision, validation scope, implementation references, and option B follow-up status.

Review effort: Lite (dynamic)

Comment on lines +199 to +203
- Behavioral scenarios ran live on Claude Code with Copilot's file deleted, and
passed: routing, human sign-off, reviewer rejection, restart continuity,
customization precedence, monorepo subfolder, backend outage and the
satellite repository. Azure DevOps is deferred. The evidence is in
[#2487](https://github.com/github/squad/issues/2487) and the host feature
@bradygaster
bradygaster merged commit 6ce4e1b into dev Nov 14, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants