Skip to content

Built-in base roles must be opt-in, not opt-out (follow-up from #369) #379

Closed

Description

@bradygaster

Context

PR #369 (built-in base roles) was merged but Brady's review comment required two conditions:

  1. Base roles must be opt-in, not opt-out. Casting by repo/prompt should remain the default behavior. Users love the casting feature — base roles should be an alternative users explicitly choose, not something that overrides casting.
  2. Support the --sdk switch (tracked separately).

See: https://github.com/bradygaster/squad/pull/369#issuecomment-4055689766

Acceptance Criteria

  • \squad init\ continues to cast from a fictional universe by default
  • Base roles are only used when explicitly requested (e.g., \squad init --roles\ or config flag)
  • Existing casting behavior is not affected

Activity

  1. github-actions commented on Mar 13, 2026

    @github-actions
    Contributor

    📋 Assigned to EECOM (Core Dev)

    Issue: #379 — Built-in base roles must be opt-in, not opt-out (follow-up from #369)

    EECOM will pick this up in the next Copilot session.

    For Copilot coding agent: If enabled, this issue will be worked automatically.
    Otherwise, start a Copilot session and say:
    EECOM, work on issue #379

  2. github-actions commented on Mar 13, 2026

    @github-actions
    Contributor

    🏗️ Squad Triage — Flight (Lead)

    Issue: #379 — Built-in base roles must be opt-in, not opt-out (follow-up from #369)
    Assigned to: DSKY (TUI Engineer)
    Reason: Issue relates to frontend/UI work

    @copilot evaluation: 🔴 Not suitable — issue involves work outside the coding agent's capability profile.

    Team roster:

    • Flight (Lead) → label: squad:flight
    • Procedures (Prompt Engineer) → label: squad:procedures
    • EECOM (Core Dev) → label: squad:eecom
    • FIDO (Quality Owner) → label: squad:fido
    • PAO (DevRel) → label: squad:pao
    • CAPCOM (SDK Expert) → label: squad:capcom
    • CONTROL (TypeScript Engineer) → label: squad:control
    • Surgeon (Release Manager) → label: squad:surgeon
    • Booster (CI/CD Engineer) → label: squad:booster
    • GNC (Node.js Runtime) → label: squad:gnc
    • Network (Distribution) → label: squad:network
    • RETRO (Security) → label: squad:retro
    • INCO (CLI UX & Visual Design) → label: squad:inco
    • GUIDO (VS Code Extension) → label: squad:guido
    • Telemetry (Aspire & Observability) → label: squad:telemetry
    • VOX (REPL & Interactive Shell) → label: squad:vox
    • DSKY (TUI Engineer) → label: squad:dsky
    • Sims (E2E Test Engineer) → label: squad:sims
    • Handbook (SDK Usability) → label: squad:handbook
    • Ralph (Work Monitor) → label: squad:ralph
    • @copilot (Coding Agent) → label: squad:copilot

    To reassign, remove the current squad:* label and add the correct one.

  3. bradygaster commented on Mar 15, 2026

    @bradygaster
    CollaboratorAuthor

    Fixed and merged to dev in the irritating bugs sprint.

  4. bradygaster commented on Mar 15, 2026

    @bradygaster
    CollaboratorAuthor

    Hey @spboyer — the base roles opt-in feature from your contribution is now merged to dev (PR #412). Users can pass --roles to squad init or /init --roles in the REPL to use the built-in role catalog during team casting. Thanks for the contribution! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    go:needs-researchNeeds investigationpriority:p1This sprintsquadSquad triage inbox — Lead will assign to a membertype:bugSomething broken

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions