feat: Extract cold-path sections from squad.agent.md (#98)#116
Closed
bradygaster wants to merge 3 commits intodevfrom
Closed
feat: Extract cold-path sections from squad.agent.md (#98)#116bradygaster wants to merge 3 commits intodevfrom
bradygaster wants to merge 3 commits intodevfrom
Conversation
Session: 2026-02-20-v050-pr-merges Requested by: Scribe (autonomous) Changes: - Created orchestration log: .ai-team/orchestration-log/20260220T000249-kobayashi.md (Kobayashi spawn summary) - Created session log: .ai-team/log/2026-02-20-v050-pr-merges.md (brief summary of v0.5.0 PR merge work) - Merged 7 inbox files into decisions.md (baer-108-email-scrub, fenster-104-templates-merge, kobayashi-103-workflow-dual-path, kobayashi-v050-pr-merges, kujan-network-error-diagnosis, mcmanus-105-docs-update, verbal-102-path-migration) - Deleted inbox files after merge (0 duplicates found) - Appended cross-agent update to keaton and fenster agent histories Decisions.md new size: 76.1 KB (up from 43.3 KB after merge, no deduplication needed, no archival triggered) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Extract 4 cold-path sections to satellite files to reduce coordinator prompt size: - Multi-Agent Artifact Format → templates/multi-agent-format.md - Constraint Budget Tracking → templates/constraint-tracking.md - Plugin Marketplace → templates/plugin-marketplace.md - MCP Integration config/samples → templates/mcp-config.md Retained core rules in squad.agent.md for each section. Full detail/schemas/samples now in on-demand reference files. Before: 70,629 characters After: 70,036 characters Reduction: 593 characters Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
joniba
pushed a commit
to joniba/squad
that referenced
this pull request
Mar 26, 2026
…radygaster#125) - Dead letter queue for persistent retry - Retry scheduler with escalating backoff (5m/15m/1h) - Health check (webhook reachability, queue depth, last success) - Auto-purge of aged failures (30 days) - Integration with existing notify.ps1 - Full Pester test coverage (34 tests) Co-authored-by: Elrond <elrond@squad.local> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
joniba
pushed a commit
to joniba/squad
that referenced
this pull request
Mar 26, 2026
….md and failure-recovery
R1: Rename proactive-notifications.md → human-attention-notifications.md
to match what issue bradygaster#112 specifies as the deliverable filename.
R6: Update SKILL.md to document BOTH notification systems:
- Old system (send-teams-notification.ps1): active, 2 callers
- New system (notify.ps1): built, 0 callers yet
Note: old system stays active until MVP is validated.
R7: Update failure-recovery.md notification section to specify:
- Current: use send-teams-notification.ps1 (old system)
- Target: switch to notify-blocked.ps1 after E2E validation (bradygaster#135)
- Explicit migration path instead of ambiguous "post to webhook"
GitHub issue actions (not in this commit):
- Closed bradygaster#115 (library code done in PR bradygaster#122)
- Updated bradygaster#116 body (clarify integration work remains)
- Labeled bradygaster#117 P3-low + scope decision comments (post-MVP)
- Created MVP issues bradygaster#132-bradygaster#135 (caller scripts + wiring + validation)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
joniba
pushed a commit
to joniba/squad
that referenced
this pull request
Mar 26, 2026
- Fixed 4 pre-existing test failures in notify-blocked.Tests.ps1: - Reason is no longer mandatory (multi-issue mode uses -Issues) - Card format uses FactSet JSON, not inline text patterns - Added Multi-Issue Mode tests (per-issue rows, -Reason ignored) - Added Dedup tests (unique eventIds, -Force bypass, notify.ps1 suppression) - Added Escalation Scenario tests: - Elrond research exhaustion → notification with context - Aragorn livesite block → immediate urgent notification - Gandalf double-rejection → multi-issue escalation - All 24 tests pass, 17 dispatcher tests pass Closes bradygaster#116. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This was referenced Mar 29, 2026
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
1 participant
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.
Closes #98
Summary
Extracted 4 cold-path sections from squad.agent.md to satellite template files to reduce coordinator prompt size and move toward GHE compatibility (30,000 character limit).
Sections Extracted
Multi-Agent Artifact Format → emplates/multi-agent-format.md
Constraint Budget Tracking → emplates/constraint-tracking.md
Plugin Marketplace → emplates/plugin-marketplace.md
MCP Integration → emplates/mcp-config.md
Impact
All tests pass (53/53). Each section follows the existing extraction pattern with **On-demand reference:**\ stubs in squad.agent.md pointing to the full content in templates/.
Next Candidates
Additional sections that could be extracted in future work: