Allow setup with only assigned model providers

[arjitj2]

Closes #72

Changes

Setup currently requires all four default model families, so a missing Grok CLI blocks users who want only Codex or Claude. Select role assignments first and probe only the assigned models. Unused providers need no CLI or subscription.

Preserve custom assignments, lane order, and all documented roles. A failed selected provider requires explicit repair or reassignment before saving. Native aliases receive a real delegation probe before either configuration file changes. Architect keeps at least two independent runner entries, which may use the same model; other panels remain nonempty.

Verification

• 159 Bun tests pass.
• Strict typecheck, static invariants, manifest checks, Claude plugin validation, and git diff --check pass.
• Exact candidate installed and full setup verified in Codex and Claude Code.
• Installed version, actions, and observed results recorded for both applications.

Live evidence, 2026-09-16 on macOS: pstack 1.4.1, candidate 7feabe1518ac680caa514e304d342d22733f12ca. All 166 installed plugin files matched the committed snapshot byte-for-byte in Codex CLI 0.154.0 and Claude Code 2.1.179. Fresh real setup sessions used disposable target paths; the user’s actual model sheets and integration files remained unchanged.

Both applications passed:

• Reject one Architect entry before probing or writing; accept two explicitly selected, identical native aliases without adding another provider.
• Probe native delegation, stop for confirmation, save all 15 roles, verify the integration file, and run a fresh native smoke with an independent judge.
• Repeat setup and confirmation with byte-identical saved files, preserved lane order, both Architect entries, and unrelated integration text.
• Invoke the installed runner with selected Grok absent from PATH, receive unavailable-cli, and preserve both files. Explicit reassignment to inherit-parent then passed probing, confirmation, saving, and smoke without any further external-provider checks.
• Inject a real second-file permission failure after the first file changed, observe the failed write, restore the first file’s original bytes, and report setup incomplete. Scratch permissions were restored afterward.

Tool transcripts, child results, receipts, exact file bytes, and independent hashes support these results. Behavioral cases describe the scenarios.

[greptile-apps]

Confidence Score: 5/5

The PR appears safe to merge.

Summary

Setup now lets operators choose role assignments before checking providers, so unused model families no longer block configuration. It preserves custom roles and lanes while requiring explicit recovery for selected failures and real native checks for aliases.

• Assigned providers are the only providers that receive effort questions and probes.
• Failed selected routes leave configuration unchanged until repaired or reassigned.
• Native-only setups, preserved lane order, and Architect's minimum runner count remain supported.

Diagram

sequenceDiagram
    actor User
    participant Setup as setup-pstack
    participant Native as Native agent
    participant Provider as Assigned provider
    participant Files as Model sheet and integration
    participant Architect
    User->>Setup: Select role assignments
    Setup->>Setup: Require two architect runners
    Setup->>Native: Probe aliases when assigned
    Setup->>Provider: Probe each assigned pair once
    Setup-->>User: Show final role map
    User->>Setup: Confirm
    Setup->>Files: Write and read back both files
    alt Write or readback fails
        Setup->>Files: Restore snapshots
    else Setup succeeds
        Setup->>Architect: Keep each runner entry as one lane
        Architect->>Architect: Require two distinct designs
    end

Loading

_{Reviews (2) · Last reviewed commit: "Preserve the minimum Architect candidate..."}

[mergify]

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

[openswebot]

✅ Open SWE Review: No issues found

Open SWE reviewed this PR and found no potential bugs to report.

Open in Web

[ericlitman]

Thanks, @arjitj2. Open Pstack 1.5.0 (#98, released as v1.5.0) takes this PR's roles-first setup order: setup now asks about roles first, then asks efforts for and probes only the families the roles use. CHANGES.md credits you. The rest of this PR is not taken, so I'm closing it as superseded by #98. #72 is closed by the same change.