Docs update: Create missing human-members.md template#315
Merged
bradygaster merged 1 commit intobradygaster:mainfrom Mar 10, 2026
Merged
Docs update: Create missing human-members.md template#315bradygaster merged 1 commit intobradygaster:mainfrom
bradygaster merged 1 commit intobradygaster:mainfrom
Conversation
Adds .squad/templates/human-members.md — the coordinator reference doc for human team member management. This file was referenced in squad.agent.md but did not exist. Covers: triggers, AI vs human comparison, adding/routing/reviewing humans, decision framework for when to add, and removal steps. Closes bradygaster#312 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
diberry
commented
Mar 9, 2026
Collaborator
Author
diberry
left a comment
diberry
left a comment
There was a problem hiding this comment.
REVIEW: LGTM ✅
The human-members.md template is architecturally sound and aligns with the coordinator's governance logic in squad.agent.md. All key patterns are captured correctly:
What's solid:
- ✅ Comparison table (AI Agent vs Human Member) is accurate — charter/history presence, spawning behavior, casting rules, review rights.
- ✅ Routing pause behavior matches coordinator logic: non-dependent work continues, stale reminders after >1 turn.
- ✅ Decision framework litmus test is sharp — "If agents should stop and wait, add them."
- ✅ Reviewer rejection lockout correctly applies to humans (coordinator rule consistency).
- ✅ Badge convention (👤 Human) and roster format match the coordinator's expectations.
Minor observation (not blocking):
The template doesn't explicitly mention that multiple humans are supported and tracked independently (the coordinator knows this from squad.agent.md). Not a blocker — the template is an onboarding guide, not a spec dump.
This is a clean reference that will make human-member onboarding seamless. Ready to merge.
— Procedures
diberry
added a commit
to diberry/squad
that referenced
this pull request
Mar 9, 2026
… recommendations Session: 2026-03-09T19-16-49Z-docs-consolidation Requested by: Copilot (Scribe role) Changes: - Merged 5 inbox decisions (user directives, McManus docs audit, Keaton config audit) - Updated decisions.md with progressive disclosure framework and docs work classification - Appended cross-agent team updates to PAO, FIDO, Procedures history files - Cleared decision inbox (5 files removed) Summary: - PAO approved all 4 docs PRs (bradygaster#318, bradygaster#317, bradygaster#305, bradygaster#303) with docs-test sync verified - FIDO validated test assertions for 4 new docs files (EXPECTED_CONCEPTS array correct, no gaps) - Procedures approved human-members.md template PR bradygaster#315 - 44-issue docs backlog now configured with routing, ceremonies, skills, and @copilot capability profile - Team aligned on progressive disclosure and documentation conventions Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Collaborator
Author
|
@bradygaster — This PR is ready for your review. CI is green. 🚀 |
Ab3y
pushed a commit
to Ab3y/squad
that referenced
this pull request
Mar 10, 2026
Replace unsafe \�s unknown as SquadSession\ cast with a proper runtime adapter class that maps CopilotSession methods to SquadSession interface: - sendMessage() → send() - on()/off() with unsubscribe tracking - close() → destroy() Fixes coordinatorSession.sendMessage is not a function in Codespaces. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Ab3y
pushed a commit
to Ab3y/squad
that referenced
this pull request
Mar 10, 2026
…aster#315) The @github/copilot-sdk CopilotSession exposes send(), not sendMessage(). The previous �s unknown as SquadSession cast was a type-only assertion that didn't add runtime methods, causing "sendMessage is not a function" in GitHub Codespaces. CopilotSessionAdapter now properly maps: - sendMessage() → send() - off() → unsubscribe function tracking - close() → destroy() 9 new adapter tests, all passing. Bug reported by @spboyer (Shayne Boyer) in Codespace environment. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.
Summary
Creates the missing
.squad/templates/human-members.mdcoordinator reference document.This file is referenced in
squad.agent.mdbut did not exist, leaving the coordinator without guidance for human team member management.What's included
Decision framework highlight
The template includes a litmus test: If you want agents to stop and wait for someone's input before proceeding, add them. If they review asynchronously via GitHub, don't bother.
Closes #312
cc @bradygaster - cannot request reviewers from fork, requesting review via mention.