Skip to content

CI: roles module not available through workspace - squad-cli depends on published npm version #455

Closed

Description

@spboyer

Problem

The CLI package depends on a fixed version of the SDK (0.8.25) from npm instead of using a workspace reference. This causes npm ci to install the published npm package, which doesn't include the roles module added in PR #369.

Impact

Any PR targeting main that builds the CLI will fail with:

error TS2305: Module '@bradygaster/squad-sdk' has no exported member 'listRoles'

The roles source files exist on main, but the published npm v0.8.25 doesn't include them. CI doesn't run on main pushes (only PRs), so this wasn't caught when #369 was merged.

Root Cause

In packages/squad-cli/package.json, the dependency is pinned to the npm registry version:

"@bradygaster/squad-sdk": "0.8.25"

Instead of a workspace link. When npm ci runs, it installs the published package (which predates the roles module) rather than linking to the local workspace source.

Fix

Either:

  1. Change the CLI dependency to use workspace protocol so npm links to local source
  2. Publish a new version of @bradygaster/squad-sdk that includes the roles module

Blocked PRs

Activity

  1. spboyer commented on Mar 19, 2026

    @spboyer
    ContributorAuthor

    cc @bradygaster — this is blocking all PRs targeting main. The roles module from #369 was merged but the published npm package doesn't include it, so the CLI build fails. Either a new SDK publish or a workspace link fix would unblock CI.

  2. github-actions commented on Mar 20, 2026

    @github-actions
    Contributor

    📋 Assigned to GNC (Node.js Runtime)

    Issue: #455 — CI: roles module not available through workspace - squad-cli depends on published npm version

    GNC 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:
    GNC, work on issue #455

  3. github-actions commented on Mar 20, 2026

    @github-actions
    Contributor

    🏗️ Squad Triage — Flight (Lead)

    Issue: #455 — CI: roles module not available through workspace - squad-cli depends on published npm version
    Assigned to: FIDO (Quality Owner)
    Reason: Issue relates to testing/quality work

    @copilot evaluation: No strong capability match — routed to squad member.

    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.

  4. github-actions commented on Mar 20, 2026

    @github-actions
    Contributor

    📋 Assigned to Booster (CI/CD Engineer)

    Issue: #455 — CI: roles module not available through workspace - squad-cli depends on published npm version

    Booster 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:
    Booster, work on issue #455

  5. added a commit that references this issue on Mar 20, 2026
    6cbabb5
  6. bradygaster commented on Mar 20, 2026

    @bradygaster
    Collaborator

    Fixed. CI now auto-removes stale nested workspace entries from package-lock.json before install. Added ci-rerun.yml for manual retrigger on fork PRs. Default branch changed to dev.

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