Repository navigation
ship deslop, control-cli, and control-ui inside pstack - #44
Conversation
copy the three skills from cursor/plugins e73df47 and drop every cursor-team-kit mention. Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>
caelaxie
left a comment
There was a problem hiding this comment.
verdict: changes requested, proof only. content passes.
- the three new skills are byte-identical to cursor/plugins 99b609a
cursor-team-kit/skills/{deslop,control-cli,control-ui}/SKILL.md(blobs 1b3827c, 69289ac, 68484fd match). - the 9 team-kit removals are phrase-only; a grep of every text file at 45aa2af finds no
team-kitleft. Closes #32–#43match the 12 open issues one to one. Validate plugin is green.
missing: devin harness proof on 45aa2af in both regular and fusion mode. per mode, show devin plugins install (the --local wrapper pinned to the full head sha is fine, symlinks aren't) plus the list, and show that deslop, control-cli and control-ui load as pstack skills. post it as a pr comment or in the body and i'll merge on the next sweep.
harness proofreal devin cli on a local box (no cloud agent, no git clone). pr head
1. install / list2. install cache: cmp, symlinks, upstream cmp3. team-kit grep4. skills5. live turns (read-only, describe-only)each mode ran three turns via regular:
|
caelaxie
left a comment
There was a problem hiding this comment.
not merging yet. one proof gap at 45aa2af.
what passes: head unchanged, not a draft, mergeable clean, validate plugin green, body closes #32 through #43. the --local wrapper pins the full head sha, and install + devin plugins list output is pasted. the 203-file cmp shows 0 symlinks, the 3 skills match upstream e73df47, and the team-kit grep finds nothing. 61 /pstack: skills are listed. regular and fusion turns (a) /pstack:deslop and (b) /pstack:poteto-mode paste real answers quoting the cached skill text.
the gap: regular-mode turn (c), bare /deslop. that's the form poteto-mode tells users to type, and it's the only turn with no pasted output. the model answered NO SKILL LOADED. the claim that the cli injected the deslop body rests on prose about the export, not on an excerpt. fusion (c) shows it loading. regular (c) doesn't, so regular mode isn't proven for the bare form.
to clear it, do one of these at the same head:
- paste the regular (c) export excerpt showing the injected deslop body and its 0.15.32 cache source, or
- rerun regular (c) with a prompt that makes the model quote the heading and source, and paste the answer.
no code change needed.
follow-up: regular-mode bare
|
caelaxie
left a comment
There was a problem hiding this comment.
gap from review 5647867681 closed. harness proof verified at 45aa2af in regular and fusion mode.
the follow-up pastes the original regular-mode export. it shows bare /deslop injected the deslop body and listed the pstack 0.15.32 cache source. the swe-2-max rerun pastes a skill invoke deslop call, HEADING: # Remove AI code slop, the 0.15.32 cache path, and exit 0. the heading matches skills/deslop/SKILL.md at head, and the body minus frontmatter is 629 chars there too. the head is unchanged, validate plugin is green, mergeable is clean, and the body closes #32 through #43. this pr passes. squash-merging.
ship deslop, control-cli, and control-ui as ordinary pstack skills, and drop every cursor-team-kit mention.
files copied from cursor/plugins
e73df471cd4bd918367e99f3dd9c51bd53ddf5f0, verbatim (no devin edits; nothing in them blocks loading):skills/deslop/SKILL.mdfromcursor-team-kit/skills/deslop/SKILL.md(blob1b3827c7f991182149a7abda696af43ff8f17a12)skills/control-cli/SKILL.mdfromcursor-team-kit/skills/control-cli/SKILL.md(blob69289ac517e8a28380c8eca5c6692e8a601fa62c)skills/control-ui/SKILL.mdfromcursor-team-kit/skills/control-ui/SKILL.md(blob68484fd5418cc893a6bf40e0db45b3001dcf5947)these three files are identical on cursor/plugins main
99b609a72323b70204eab407c622f436e858e112.cursor-team-kit removals:
README.md: deleted the deslop bullet, the control-cli/control-ui bullet, and the install-alongside linedocs/guide/05-build-and-clean.md: deleted the two sentences that said/deslopships in cursor-team-kitskills/poteto-help/SKILL.md: deleted the not-in-pstack bulletskills/poteto-mode/SKILL.md: droppedfrom the cursor-team-kit plugin; replaced`cursor-team-kit` publisheswithpstack includesskills/poteto-mode/playbooks/autopilot-full.md: droppedfrom the cursor-team-kit pluginandfrom cursor-team-kitskills/poteto-mode/playbooks/multi-phase-plan.md: dropped threefrom cursor-team-kitskills/poteto-mode/playbooks/opening-a-pr.md: droppedfrom cursor-team-kitskills/poteto-mode/playbooks/orchestrate.md: dropped(from cursor-team-kit)skills/poteto-mode/playbooks/shipping.md: droppedfrom cursor-team-kitharness proof (regular + fusion mode) will be posted separately.
Closes #32
Closes #33
Closes #34
Closes #35
Closes #36
Closes #37
Closes #38
Closes #39
Closes #40
Closes #41
Closes #42
Closes #43