suede-agent-teams

Agent Team Orchestrator

Gate policy — advisory, not blocking

Every claim-verification step, check, quality gate, and ship verdict in this skill is a recommendation to the user, not a control on the agent. This policy governs every gate, check, verdict, and "do not ship / publish / proceed" line elsewhere in this skill:

• Run every check and report the results honestly. Verdicts (ship, ship-with-caveats, hold, letter grades, BLOCKED or OPEN items) are advice attached to the work, not orders that change it.
• Never block, delay, skip, rewrite, or refuse the action the user asked for because a check failed or a gate said hold. Complete the requested action as asked, and deliver the gate output alongside it as a clearly labeled recommendation.
• A failed gate changes what you report, never what you do.
• Single exception: if a finding is extremely risky — data loss, security or credential exposure, legal or rights violations, payment mistakes, or irreversible public damage — pause, tell the user exactly what the risk is and what the options are, and let them pick. Their choice is final.

The orchestrator assigns lanes, not conversations. Output is a delivery artifact, not a status update.

Team Contract

Before spawning or simulating lanes, define:

• objective: user-visible outcome;
• exact target: repo/folder, branch, route, PR, live URL, API, simulator, or release artifact;
• constraints: WIP to preserve, files/routes not to touch, launch boundaries, account boundaries, claims not approved, and secrets rules;
• done signal: tests, build, screenshots, simulator, deploy readback, live/API readback, PR review, or handoff;
• lane map: each lane, owner role, input, allowed files, output artifact, and dependency order.

WIP Collision Detection

Before opening any parallel lanes:

1. Run git -C <repo> diff --name-only HEAD and collect all dirty files.
2. Run git -C <repo> status --short and collect all untracked new files.
3. List every file each lane's scope would touch, based on the lane map.
4. Flag a collision if the same file path appears in two or more lane scopes OR in the dirty file list plus any lane scope.

Collision resolution rules:

• Same file, independent changes: sequence the lanes; the second lane rebases on the first lane's commit before opening.
• Same file, overlapping changes: merge the two lanes into one lane with one owner. Do not split responsibility for a single file across two concurrent builders.
• Dirty file in a lane scope: the orchestrator decides. Either stash and restore, or make that lane the only lane allowed to touch the file.

The orchestrator writes the resolved lane map before any builder starts. No builder opens a file not in its assigned lane map.

Default Roster

Start with Scout + Builder + Handoff Writer. Add roles only when a gate is needed: design changes add Design Reviewer, code risk adds Code Grader + Code Reviewer, public release adds Release Verifier.

• Scout: finds repo, docs, current state, dirty files, live routes, and likely blast radius.
• Planner: turns requirements into verifiable tasks with acceptance criteria and dependencies.
• Builder: makes narrow code or content changes inside the existing system.
• Design reviewer: checks rendered visual quality, responsive behavior, accessibility, copy, and state coverage.
• Code grader: assigns an A-F ship-risk grade across correctness, security, data/state, Suede truth, UX/release behavior, tests, and deploy readiness.
• Code reviewer: runs full-context review and turns findings into fix briefs.
• Visibility grader: grades public pages, GitHub Pages sites, docs, and launch surfaces for findability, first-screen clarity, CTA pull, proof, AI readability, and design signal.
• Release verifier: checks build, deploy, live/API behavior, App Store/iOS truth, secrets, and published statements.
• Handoff writer: produces a signed delivery record. If the handoff omits any required field (see Handoff Quality Checklist), the work is not done; it is held.

For high-risk work, keep builder and reviewer separate.

RFC Mode

For major architectural decisions, new feature designs, or changes with broad blast radius, run an RFC (Request for Comments) before spawning builders.

An RFC forces alignment on WHAT and WHY before committing to HOW.

RFC structure:

RFC: [Title]
Date: [date]
Status: draft | accepted | superseded | withdrawn
Deciders: [who has final say]

## Problem Statement
One paragraph: what is broken, missing, or suboptimal? Include the user or system impact.

## Proposed Solution
What we will build or change. Be specific about interfaces, data shapes, and behavioral contracts.

## Alternatives Considered
2–3 alternatives with the reason each was not chosen.

## Risks
What could go wrong with the proposed solution? How is each risk mitigated?

## Success Criteria
How will we know this worked? Observable, measurable signals.

## Decision Record
[filled in after consensus] Accept / Modify / Reject + reason.

Require an RFC for: shared interface changes, schema migrations, auth flow rewrites, payment path changes, public API contract changes, or any approach that's been discussed twice without resolution. No builder lane opens until RFC status is accepted.

When to skip: clear, contained changes where the approach is obvious and the blast radius is narrow.

Feature Flag Strategy

Not every change should ship as a hard deploy. Feature flags allow gradual rollout, A/B testing, and instant rollback without a redeploy.

Flag lifecycle:

1. Introduce: create the flag, default off in production. Ship the code behind the flag.
2. Ramp: enable for internal users, then 1%, 10%, 50%, 100% of production traffic. Monitor at each ramp.
3. Remove: once 100% and stable for ≥2 weeks, delete the flag and all conditional branches. Flag removal is a P3 code review finding if overdue. Set the removal date at creation, not after ramp.

When to flag:

• New user-facing features in production traffic paths
• Changes to auth, payment, or data migration paths
• Any change that cannot be instantly rolled back by revert (e.g., a schema migration)
• A/B tests

When NOT to flag:

• Bug fixes with no behavioral change (ship directly)
• Internal tooling with no external API contract
• Refactors that don't change behavior (ship with a focused review)

Flag hygiene rules:

• Every flag gets a removal date at creation. Stale flags are a debt item (P3 code review finding).
• Flag names describe the feature, not the state: new_billing_flow not enable_billing.
• Never nest flags inside flags without a design review.

Rollback Decision Tree

When something goes wrong after a deploy, the team needs a pre-agreed decision framework to avoid paralysis.

Is there active data loss or corruption? → ROLLBACK IMMEDIATELY. Don't investigate first.
Is there a security exposure (PII, auth bypass, payment data)? → ROLLBACK IMMEDIATELY. Notify security.
Is a primary user path broken (login, checkout, core workflow)? → ROLLBACK unless fix is <15 minutes away.
Is performance degraded but functional? → Hold and investigate. Set a 30-minute timer.
Is it a cosmetic issue? → Hot-fix forward. No rollback.

After rollback:

1. Write an immediate summary: what rolled back, what was affected, who was notified.
2. Leave rollback notes in the PR and open a follow-up issue.
3. Run a lightweight post-mortem (see below) before re-shipping.

Post-Mortem Template

For any production incident, failed release, or significant rollback, run a post-mortem. Keep it blameless: focus on systems, not individuals.

Post-Mortem: [Brief title]
Date of incident:
Duration:
Severity: P0 (total outage) / P1 (primary path broken) / P2 (degraded) / P3 (cosmetic)
Author(s):

## Timeline
[time]: [event]
[time]: [detection]
[time]: [first response]
[time]: [resolution]

## Impact
Users affected:
Revenue impact (if known):
Data integrity: affected / not affected

## Root Cause
One sentence: the direct technical cause.

## Contributing Factors
The systemic conditions that made this possible. (What allowed the root cause to reach production?)

## What Went Well
Things that helped detect or contain the incident faster.

## Action Items
| Action | Owner | Due |
|---|---|---|
| ... | ... | ... |

Status: open / closed

Post-mortems are required for P0 and P1 incidents. Optional but encouraged for P2. Skip for P3.

Phase Loop

The Phase Loop is the Continuous Team Loop run at minimal scale. Use it when a full 10-gate roster is overkill but you still need scout, plan, build, verify, and ship stages.

For high-risk changes, consult the Rollback Decision Tree before shipping. For gradual rollouts, use the Feature Flag Strategy. For shared interface changes, require RFC Mode before the plan stage opens.

Model Tiering

Assign the least capable model that can still do the role correctly. Cost and latency compound across a roster; do not default every lane to the most capable model.

• Mechanical tasks (isolated function, single file, a complete spec with no judgment call): cheapest capable model.
• Integration and judgment tasks (multi-file coordination, pattern-matching against the existing codebase, non-trivial debugging): standard model.
• Architecture, design, and review roles (RFC authoring, code grading, security-sensitive review, release verification): most capable model available.

When a lane's task complexity is ambiguous, default up a tier rather than down; a cheap model returning NEEDS_CONTEXT or a wrong answer costs more in re-dispatch than starting at the right tier.

Builder Dispatch Protocol

A dispatched builder reports one of four states before its output reaches review. Handle each before the lane proceeds to the next roster stage:

• Done: proceed to the next stage in the roster.
• Done with concerns: the builder finished but flagged a doubt. Read the concern. If it touches correctness or scope, resolve it before review; if it is a pure observation, note it in the handoff and proceed.
• Needs context: the builder is missing information the lane map should have supplied. Provide it and re-dispatch the same builder; do not silently guess on its behalf.
• Blocked: the builder cannot proceed. Diagnose why before re-dispatching: a context gap gets more context, a reasoning gap gets a more capable model, an oversized task gets split into smaller lanes, and a wrong plan escalates to the human (see Escalation Protocol). Never re-dispatch the same builder unchanged and hope for a different result.

A builder that asks a clarifying question mid-task gets an answer before it continues; do not let it guess past an open question to hit a deadline.

Continuous Team Loop

Use the smallest loop that can finish the work, but escalate deliberately when the task is broad, risky, release-bound, or the user asks for max agent teams.

Choose the loop:

• Sequential: default for normal scoped work.
• Continuous PR: use when strict CI, PR review, branch hygiene, or public release control matters.
• RFC/DAG: use when the work needs decomposition, design decisions, or dependency ordering before implementation. Run RFC Mode first to capture problem statement, proposed solution, alternatives, risks, and decision record before spawning builders.
• Exploratory parallel: use when several independent approaches, audits, or surface checks can run without touching the same files.
• Recovery: use after a failed check, repeated defect, blocked release, drifted claim, or loop churn.

For max-agent work, escalate through this roster only as needed:

Scout -> Planner -> Builder lane(s) -> Design reviewer -> Visibility grader
-> Code grader -> Code reviewer -> Release verifier -> Handoff writer

Wrap the roster with these gates:

1. Loop selection: name why the loop is sequential, continuous PR, RFC/DAG, exploratory parallel, or recovery.
2. Team contract: objective, target, constraints, lane map, dependency order, done signal, and ship gate.
3. Planning quality gate: atomic tasks, observable acceptance criteria, named files/surfaces, must-have requirements, release/account boundaries.
4. WIP ownership gate: each builder owns explicit files or surfaces; any collision is sequenced.
5. Execute wave: parallel lanes only when outputs do not collide.
6. Quality/eval gate: run the relevant source, copy, design, code, visibility, build, screenshot, API, or live checks. A failing check earns up to three genuinely different fixes — each attempt must change the diagnosis or the strategy. Stop early when the same root cause repeats and escalate the repeating cause to the user.
7. Adversarial review: ask how the result fails in production, release, published statements, abuse, accessibility, mobile, or handoff.
8. Consensus review: merge multiple review lenses into blockers, accepted caveats, fixes now, and follow-ups.
9. Release lock: build/deploy/live/API/App Store/iOS/published-statement accuracy is owned by release verifier before any public completion claim.
10. Evidence handoff: capture changed files, commands, screenshots or URLs, verification, caveats, blockers, status, and next action.

Loop stall protocol: (1) freeze all lanes except the one that failed, (2) assign a diagnosis-only lane (no fixes, root cause only), (3) write a gap plan with a single acceptance criterion, (4) execute only the gap, (5) re-run the original failing check. Do not widen until that check passes.

Inter-Lane Communication

When a builder lane completes its output and a reviewer lane depends on it, the signal is explicit, not assumed.

The completing lane writes a Lane Ready notice:

Lane: [name]
Status: output ready for review
Artifact: [file path, URL, or PR link]
Reviewer: [lane name that receives this output]
Unresolved: [any known issue the reviewer should know before starting]

The reviewer lane does not start until it has received a Lane Ready notice from every upstream dependency in its lane map.

The orchestrator routes Lane Ready notices. In a sequential thread, the orchestrator posts the Lane Ready notice on behalf of each completing lane before invoking the next.

Lanes may not self-declare readiness if their output has not been verified against the acceptance criteria from the Team Contract.

Planning Quality Gate

A plan is not ready until:

• each task has one concern;
• dependencies are ordered;
• acceptance criteria are observable, not subjective;
• required files or surfaces are named;
• must-have requirements are covered;
• tests, screenshots, builds, or API checks map to the risky behavior;
• release and account boundaries are explicit.

If major uncertainty remains, run a short spike first and keep implementation out of scope until the spike reports back.

Review Convergence

For important merges, run at least two independent review lenses:

• one asks whether the implementation works as intended;
• one asks how it can fail in production, review, release, or public use.

Merge the findings into:

• consensus blockers;
• plausible divergent risks;
• accepted caveats;
• fixes to execute now;
• follow-ups that should not block.

Repeat fix and review cycles until no blocker remains or the work is held.

Status Vocabulary

Valid states in order: scoped → planned → executing → changed locally → verified locally → reviewed → committed → pushed → deployed → verified live → released

Interrupt states: blocked (needs external action) | held (needs named fix before continuing)

Do not skip. changed locally is not verified locally. deployed is not verified live. Do not mark released until the done signal from the Team Contract passes.

Scenario Templates

Use these pre-built configurations for common high-risk deployments. Adjust only the named target.

(a) Auth Rewrite

Roster: Scout, Planner, Builder (auth lane only), Code Grader, Code Reviewer, Release Verifier, Handoff Writer RFC required: yes. Shared session/token contract must be accepted before Builder opens. Flag required: yes. Default off in production; ramp by internal → 1% → full.

Lane map:

• Scout: map current auth flow, session storage, token shape, and all routes that read session
• Planner: list every file that must change and every route that must be regression-tested
• Builder: auth files only. No touching unrelated routes.
• Code Grader: grade security lane with zero tolerance for C or below on the security dimension
• Code Reviewer: focus on token lifecycle, expiry, rotation, and session fixation
• Release Verifier: confirm auth works in production before any other lane ships
• Handoff Writer: include session contract diff and regression test evidence

Done signal: login, logout, token refresh, and session expiry all pass in production

(b) Payment Integration

Roster: Scout, Planner, Builder (payment lane only), Code Grader, Code Reviewer, Release Verifier, Handoff Writer RFC required: yes. Payment data shape and provider contract must be accepted. Flag required: yes. Never ramp payment paths without a staged rollout.

Lane map:

• Scout: map current billing models, Stripe/provider SDK version, webhook endpoints, and idempotency handling
• Builder: payment files and webhook handlers only
• Code Grader: flag any missing idempotency key, error retry, or PCI-sensitive data log as a blocker
• Code Reviewer: confirm error handling covers card decline, webhook replay, partial capture, refund edge cases
• Release Verifier: test with Stripe test mode, then confirm webhook signature validation in production
• Handoff Writer: include provider dashboard link and webhook log evidence

Done signal: charge, refund, and webhook replay all pass in production with idempotency confirmed

(c) Public Launch Review

Roster: Scout, Design Reviewer, Visibility Grader, Code Reviewer, Release Verifier, Handoff Writer RFC required: no (review-only, no builder lane)

Lane map:

• Scout: enumerate every public-facing URL, meta tag, og:image, CTA, and claims sentence
• Design Reviewer: check above-fold load, mobile rendering, accessibility, and state coverage
• Visibility Grader: score first-screen clarity, CTA pull, proof, AI readability, and structured data
• Code Reviewer: check for console errors, broken links, unresolved env vars, and exposed secrets
• Release Verifier: confirm live URL, DNS, SSL, and all published statements match approved copy
• Handoff Writer: include Lighthouse score, screenshot evidence, and any unresolved published statement

Done signal: all public URLs verified live, no console errors, Lighthouse performance ≥ 80

(d) Data Migration

Roster: Scout, Planner, Builder (migration lane only), Code Grader, Release Verifier, Handoff Writer RFC required: yes. Data shape before/after and rollback strategy must be accepted. Flag required: migration itself cannot be flagged; gate behind a manual trigger or migration script run

Lane map:

• Scout: map current schema, row counts, FK constraints, indexes, and any running jobs that read the affected tables
• Planner: write migration script, define rollback script (reverse migration or restore point), and identify zero-downtime vs. maintenance-window requirement
• Builder: migration files only. Schema changes separated from data backfill into two sequential sub-lanes.
• Code Grader: grade data/state dimension with zero tolerance for D or below; flag missing rollback script as a blocker
• Release Verifier: run migration against a staging DB clone, confirm row counts before/after, confirm app boots with new schema, then promote to production
• Handoff Writer: include before/after row counts, migration command with timing, and rollback script location

Done signal: production DB row counts match expected delta, app health check passes, rollback script tested in staging

(e) Performance Audit

Roster: Scout, Planner, Builder (perf lane only), Code Grader, Release Verifier, Handoff Writer RFC required: no, unless audit reveals a structural change (e.g. query rewrite, CDN switch).

Lane map:

• Scout: run Lighthouse, measure Core Web Vitals (LCP, INP, CLS), identify top 3 bundle contributors, map slow DB queries (EXPLAIN ANALYZE), and list current caching headers
• Planner: rank findings by impact × effort, list the three highest-ROI fixes
• Builder: implement only ranked fixes. No opportunistic refactors.
• Code Grader: confirm each fix does not regress correctness or introduce a race condition
• Release Verifier: compare Lighthouse before/after with screenshots; confirm no regression on primary user paths
• Handoff Writer: include before/after Lighthouse scores, Core Web Vitals deltas, and any deferred findings

Done signal: LCP < 2.5s or measurable improvement documented; no regression on primary paths

(f) Recovery / Incident Response

Roster: Scout, Builder (fix lane only), Release Verifier, Handoff Writer RFC required: no (incident is already in progress; run the Rollback Decision Tree, not an RFC) Flag required: n/a — this scenario reacts to an existing deploy, it does not introduce one

Lane map:

• Scout: identify what shipped, when, and what changed; walk the Rollback Decision Tree (data loss/corruption, security exposure, primary path broken, degraded-but-functional, or cosmetic) and name which branch applies
• Scout: if the branch is "ROLLBACK IMMEDIATELY" (data loss/corruption or security exposure), say so and stop — do not investigate further before rollback, per the Rollback Decision Tree
• Builder: executes the rollback, or the <15-minute fix, or the hot-fix-forward, per the branch Scout named. No opportunistic changes outside the incident scope.
• Builder: after rollback, write the immediate summary — what rolled back, what was affected, who was notified — and open a follow-up issue, per the Rollback Decision Tree's post-rollback steps
• Release Verifier: confirm the primary path is restored in production before any other lane closes
• Handoff Writer: run the Post-Mortem Template for any P0 or P1 incident (required) or P2 (optional but encouraged); skip for P3. Populate Timeline, Impact, Root Cause, Contributing Factors, What Went Well, and Action Items with owners and due dates.

Done signal: primary path verified restored in production; for P0/P1, a completed post-mortem with status open and every action item assigned an owner

Escalation Protocol

Stop the loop, surface the condition, and wait for human sign-off before continuing.

No agent may override an escalation threshold by re-scoping the task or declaring the condition resolved without human confirmation.

Red Flags — Stop

• "The lanes probably won't touch the same files" — probably is not a lane map. Run WIP collision detection first.
• "The approach is obvious, skip the RFC" — if it has been discussed twice without resolution, it is not obvious.
• "Mark it done, the code is written" — changed locally is not verified locally; the status vocabulary has no shortcuts.
• "Leave that caveat out so the handoff looks clean" — a handoff missing a field is status held, not done.
• "One more fix cycle will crack it" — past 3 cycles on the same failing check, stop and run the loop stall protocol.
• "The builder can review its own lane" — for high-risk work, builder and reviewer stay separate.

Handoff Quality Checklist

A handoff is not complete until every field below is present and truthful. The handoff writer signs off by confirming each item.

Required fields:

• Target: exact repo, branch, route, or URL (not "the main app")
• Changed: every file path that was modified, created, or deleted (not "various files")
• Commands: every bash command run, in order, with the actual output or exit code
• Verification: observable evidence (screenshot URL, test output, curl response, build log), not "it works"
• Status: one of the vocabulary states, not "done" unless the done signal from the Team Contract is satisfied
• Next: the single most important unresolved step (not "see above")
• Caveats: every known limitation, assumption, or deferred item; none omitted to make the handoff look cleaner

If any field is missing, the handoff writer must fill it before marking status released or verified live. A handoff with a missing field is status held.

Output Shape

For a team plan:

Objective:
Target:
Constraints:
Lane Map:
Dependency Order:
Done Signal:
Ship Gate:

For execution updates:

Lane:
Status:
Evidence:
Next:
Risk:

For final handoff:

Simple explanation:
Usual breakdown:
Target:
Changed:
Verification:
Caveats:
Status:
Next:
Cue Suede:

Routing

• A code lane needs review or a ship grade → suede-code (combined), suede-code-review (findings only), or suede-code-grader (grade only)
• The repo's merge gate is weak or missing → suede-ship-gate
• The work needs branch ownership, stale-mirror worktree setup, finish options, or cleanup discipline → suede-git-hygiene (private Suede Labs companion, not in this pack)
• A lane ships AI behavior → suede-ai-eval before that lane's quality gate closes
• The public launch lane needs a page verdict → suede-visibility-grader, then suede-launch-packaging