Repository navigation
fix(pstack): map cursor-team-kit skills in the Claude Code adapter - #1153
Open
frankcalise wants to merge 1 commit into
Open
frankcalise wants to merge 1 commit into
frankcalise wants to merge 1 commit into
Conversation
pstack calls /deslop, control-ui, and control-cli from cursor-team-kit, and Cursor's built-in create-skill. The Claude Code adapter mapped none of them, so Claude Code agents skipped those steps. Two playbooks also named /no-comments in the same clause as /deslop, and agents skipped it as not installed, although it ships with pstack. The adapter gains a table for the four skills and says /no-comments ships with pstack. opening-a-pr.md and multi-phase-plan.md split the instruction so each skill names where it comes from.
frankcalise
marked this pull request as ready for review
November 2, 2026 01:30
flexdinesh
added a commit
to flexdinesh/diffx
that referenced
this pull request
Nov 2, 2026
cursor/plugins#1153 failed in staging: the cursor org's IP allow list refuses the app's installation tokens on REST pull request reads (HTTP 403), and previews tried only the repository's and the public installation. - github: Credential (installation, user access, anonymous); ErrRestricted for non-rate-limit 403s; ErrCredentialUnavailable for unmintable tokens. - githubsync: ordered public read sources (repository, public, viewer, anonymous) composed as app policy. Refused credentials fall back; 404 ends the chain (except a stale repository installation) and never reaches the viewer. Lease fetch credential follows the serving source; the fetcher never holds a user token. - Exhaustion answers failure `restricted` with remedy sign_in or connect_github; rate limits stay 429. - remoteserver: only browser sessions lend their GitHub connection. - API: `restricted` code and `remedy` field. - web: "GitHub restricts this repository" state with Sign in / Open Settings recovery. - docs: system, architecture, DESIGN, user docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
pstack calls
/deslop,control-ui, andcontrol-clifromcursor-team-kit, and Cursor's built-increate-skill. The Claude Code adapter maps none of them, and the Claude Code marketplace does not listcursor-team-kit, so Claude Code agents skip those steps. Two playbooks also name/no-commentsin the same clause as/deslopfromcursor-team-kit. Agents then skipped/no-commentsas "not installed", although it ships with pstack.What changed
/no-commentsships with pstack.opening-a-pr.mdandmulti-phase-plan.mdsplit the instruction into two sentences that say where each skill comes from.Scope
cursor-team-kitin.claude-plugin/marketplace.json.Tradeoffs
Listing
cursor-team-kitin the Claude Code marketplace would let/deslopresolve with no copy step. Nobody has checked its 18 skills, 2 agents, and.mdcrules under Claude Code, so that call belongs to the plugin's owners.Blast Radius
Claude Code users of pstack read one new adapter section. The table only adds mappings, so a session that already skipped these steps now runs them or skips them with the adapter's existing line.
Verification
cmpshows each edited harness file is identical to its source. The generator copies these files unchanged, because they are notSKILL.mdfiles and the new text has no links, sogenerate-pstack-harness-trees.mjs --checkin CI should pass. I did not run the generator locally.deslopcopied into~/.claude/skills/deslop/, a fresh headless Claude Code session listsdeslop, and/no-commentsresolves throughpstack:comment-sicko.