Repository navigation
ship deslop, control-cli, and control-ui - #3
Conversation
copy the three skill directories in unchanged. drop the lines that said they are not bundled, and list them with the other skills. Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>
|
validation proof 1. npm test — pass2. harness — bundle and skill catalog pass; model reply blocked on DEEPSEEK_API_KEYinstalled the newest published runtime, profile bundles after that command: trimmed: plugins page text included:
skill catalog, session cwd
both turns then ended before a model reply: the web composer stays on "Add API Key / Sign in", so the visual |
caelaxie
left a comment
There was a problem hiding this comment.
changes requested (comment review, same-account author).
passes: deslop/control-cli/control-ui are byte-identical to cursor-team-kit at both ab69380 and the pinned 99b609a, they live in .agents/skills/, no cursor-team-kit mention is left, and the proof comment shows dsh plugin --profile web add + the bundle mounted + skills/list returning all three (plus /deslop injecting). model turn blocked only on DEEPSEEK_API_KEY, which is the accepted precedent for text-only skill changes. Validate DSH bundle is green. no open issue to link.
one divergence left, see the inline comment. fix it and this merges.
| ## Clean before you commit | ||
|
|
||
| The [Opening a PR playbook](../../.agents/skills/poteto-mode/playbooks/opening-a-pr.md) runs `/deslop` on the diff before each commit and applies [`/unslop`](../../.agents/skills/unslop/SKILL.md) to the PR description and commit bodies. `/deslop` is a separate skill, not bundled with pstack. If you don't have it, ask for the same outcome in plain words: remove narrating comments, unsupported guards, dead compatibility paths, and unrelated edits. | ||
| The [Opening a PR playbook](../../.agents/skills/poteto-mode/playbooks/opening-a-pr.md) runs [`/deslop`](../../.agents/skills/deslop/SKILL.md) on the diff before each commit and applies [`/unslop`](../../.agents/skills/unslop/SKILL.md) to the PR description and commit bodies. If you don't have it, ask for the same outcome in plain words: remove narrating comments, unsupported guards, dead compatibility paths, and unrelated edits. |
There was a problem hiding this comment.
minimum changes: upstream (99b609a pstack/docs/guide/05-build-and-clean.md:59) has plain `/deslop` here, not a link. the only edit the harness rule needs is dropping the cursor-team-kit sentence. keep runs `/deslop` on the diff unlinked:
| The [Opening a PR playbook](../../.agents/skills/poteto-mode/playbooks/opening-a-pr.md) runs [`/deslop`](../../.agents/skills/deslop/SKILL.md) on the diff before each commit and applies [`/unslop`](../../.agents/skills/unslop/SKILL.md) to the PR description and commit bodies. If you don't have it, ask for the same outcome in plain words: remove narrating comments, unsupported guards, dead compatibility paths, and unrelated edits. | |
| The [Opening a PR playbook](../../.agents/skills/poteto-mode/playbooks/opening-a-pr.md) runs `/deslop` on the diff before each commit and applies [`/unslop`](../../.agents/skills/unslop/SKILL.md) to the PR description and commit bodies. If you don't have it, ask for the same outcome in plain words: remove narrating comments, unsupported guards, dead compatibility paths, and unrelated edits. |
There was a problem hiding this comment.
applied. /deslop stays unlinked. only the not-bundled sentence is gone.
drop only the sentence that said it was not bundled. Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>
caelaxie
left a comment
There was a problem hiding this comment.
verdict: pass (comment review, same-account author).
0c83f73 applies the suggestion exactly: /deslop is unlinked again, matching upstream 99b609a pstack/docs/guide/05-build-and-clean.md, and only the cursor-team-kit sentence is gone. docs-only delta since the dsh add + skills/list proof on 5d13593, so that proof still holds. Validate DSH bundle green on this head. squash-merging.
copied the skill directories
deslop,control-cli, andcontrol-uifrom https://github.com/cursor/plugins atab69380dd50ee47d426b35cb8369cffdddafc218into.agents/skills/. each directory is the upstreamSKILL.md, byte for byte.adaptations: none inside the skill files. they do not name cursor tools, so nothing was mapped. frontmatter stays
nameanddescription. dsh loads that as user-invocable and model-invocable.readme: removed the not-shipped lines for those three, and the sentence that said to install them alongside pstack. added the three rows to the existing skills table. the install note now lists them with
/setup-pstackas model-invocable, because that is how the copied frontmatter loads.also dropped the same "not bundled" lines in
poteto-helpanddocs/guide/05-build-and-clean.md, and named the three model-invocable skills there.the upstream pin in
AGENTS.mdstays at pstack 0.15.32. these skills do not need a bump. the copy isab69380dd50ee47d426b35cb8369cffdddafc218.