feat(pstack): map Cursor Task slugs for pstack roles - #37
Conversation
pstack skills read ~/.claude/pstack-models.md. Cursor Task rejects the Claude Code slugs those skills name by default. .cursor/pstack-models.md holds the Cursor mapping. scripts/link-agent-skills.sh copies it to the home path when CURSOR_AGENT=1 and appends the CLAUDE.md include once. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Delete the four-line comment above the home-sheet installer and rename the function so Cursor-Cloud-only copy is obvious without prose. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 34191a3974
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| When spawning a pstack subagent, use the role-to-model lines in | ||
| `.cursor/pstack-models.md`. `scripts/link-agent-skills.sh` copies that sheet to |
There was a problem hiding this comment.
Read the installed pstack override
When a user runs /setup-pstack to change a role, that skill rewrites ~/.claude/pstack-models.md, or ~/.codex/pstack-models.md on Codex, but this new rule tells subsequent spawns to keep reading the unchanged .cursor/pstack-models.md. The advertised customization is therefore ignored, and non-Cursor agents are directed to submit Cursor-only model slugs. Point agents to the installed runtime-specific override and reserve .cursor/pstack-models.md for Cursor bootstrap.
AGENTS.md reference: AGENTS.md:L14-L16
Useful? React with 👍 / 👎.
Why
/setup-pstackwrites per-role model overrides that pstack skills read from~/.claude/pstack-models.md. This workspace is Cursor, and the Task tool only accepts Cursor slugs. The Claude Code defaults in each pstack skill (claude-opus-4-8,claude-haiku-4-5, and the un-suffixed Opus/Fable/Sonnet 5 names) do not resolve here.Scope
Adds
.cursor/pstack-models.mdwith Cursor Task slugs for every pstack role.Opus 4.8 and the single-role default map to
claude-opus-5-thinking-high. Strongest judgment maps toclaude-fable-5-thinking-xhigh. Panel quads keep Opus, Fable, and Sonnet thinking-high, and usecomposer-2.5-fastin place of Haiku.scripts/link-agent-skills.shcopies that sheet to~/.claude/pstack-models.mdwhenCURSOR_AGENT=1, and appends@~/.claude/pstack-models.mdto~/.claude/CLAUDE.mdonce. A local Claude Code home is left alone.AGENTS.mdpoints Cursor agents at.cursor/pstack-models.md.test/link-agent-skills.test.jschecks every slug against the confirmed Task list and checks the installer copy, include, and skip paths.Tradeoffs
The mapping proceeds without a per-role confirmation prompt. Cloud Agent sessions cannot wait on
AskUserQuestion. Re-run/setup-pstackto change a role.composer-2.5-fastis not Haiku. Cursor has no Haiku slug, and the fourth panel seat needs a fast model from a different family.Blast Radius
Cursor Cloud agents that spawn pstack subagents pick up these slugs after boot. Claude Code users who run
scripts/link-agent-skills.shwithoutCURSOR_AGENT=1keep their own~/.claude/pstack-models.md. App runtime, the web UI, and skill pack contents are unchanged.Verification
npm testpassed. 5 suites, 59 tests, including the 4 new installer checks.npm run lintexited 0. 31 warnings, all pre-existing, none in the new test file or installer.CURSOR_AGENT=1 bash scripts/link-agent-skills.shran twice against this home.~/.claude/pstack-models.mdmatches.cursor/pstack-models.md.~/.claude/CLAUDE.mdcontains the include line once.A run with
CURSOR_AGENTunset against a tempHOMElinked skill packs and leftpstack-models.mduncreated.