Repository navigation
port upstream pstack through 0.15.32 - #27
Conversation
sync pstack/ from cursor/plugins 4ec7ffa..e73df47. keep the devin path, slash-command, and worktree-audit adaptations already in this repo. Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>
caelaxie
left a comment
There was a problem hiding this comment.
changes requested (posted as comment, same account authors and reviews).
blocking: no harness proof yet. the body says devin proof "will follow as a comment", and nothing is on the pr. this pr touches skill content and scripts (statechart rewrite, new ketchup-video, watch-pr, orch, check-plan, worktree-audit), so green ci alone isn't enough.
need on head 1c35735:
devin plugins installpinned to the full head sha (the--localwrapper that lets devin fetch the repo itself is fine; symlink-only proofs aren't)devin plugins listshowing pstack 0.15.32- installed copy showing
skills/ketchup-video/SKILL.mdand the newskills/statechart/scripts/*.mjs, andOutputFolder.*gone - a live
/pstack:statechartor/pstack:ketchup-videoturn if devin auth allows; if auth blocks it, say so and the install+list+installed-copy proof is accepted for these changes
non-blocking: the skipped list (claude code tree, #806 readme note, .cursor-plugin/plugin.json) reads right. no issue linkage needed unless this fixes an open issue; if it does, add Closes #N.
ping me when the proof comment lands and i'll re-review.
harness proofReal Devin CLI on a local box (no cloud agent, no git clone). PR head
1–3. before list, install, after list3–6. installed cache copyDevin installed every blob in the head tree and added nothing, so I didn't have to exclude any paths. 7. skills from the 0.15.32 cache + spot checksThe script never calls 8. live turn (describe-only)The 9. re-check and cleanupNo code was changed, and no allowlist entries were added. No secrets appear in this comment. |
caelaxie
left a comment
There was a problem hiding this comment.
pass.
- the file list matches upstream
4ec7ffa..e73df47underpstack/. the only file left out is.cursor-plugin/plugin.json. all 36 scripts, tests, and.mmdfiles are byte-identical to upstream. the threeOutputFolder.*files andmodel/are gone. - every
SKILL.md, readme, and guide hunk matches its upstream hunk. the only differences are the/pstack:prefix,triggers: [user], devin skill paths in reflect, the swarm step 3 renumber on top of the existing devin wording, andrg -ain worktree-audit, which was already on main. - skipping #806 is fine because the readme has no claude code install section. no cursor files are added.
plugin.json,.devin-plugin/plugin.json, and the readme marker all say 0.15.32.- the proof comment installs with
devin plugins install --localthrough a wrapper pinned to1c35735b, anddevin plugins listshowspstack v0.15.32. - the head is still
1c35735b.validate pluginpassed on it. the pr is not a draft, and it fixes no open issue.
synced to
e73df47. compared4ec7ffa..e73df47forpstack/.origin/mainis that commit. version is0.15.32inplugin.json,.devin-plugin/plugin.json, and the readme marker.ported:
076f4b2(#791) statechart skill update and the pele renderer. scripts copied verbatim, including the removal of the outputfolder model.SKILL.mdkeeps/pstack:statechartandtriggers: [user].642289e(#792) bump to 0.15.31, folded into the 0.15.32 bump inplugin.jsonand.devin-plugin/plugin.json.00d5dec(#794) generated verify skills read each model-check result with/pstack:statechart.3c0fdb4(#797) add the ketchup-video skill. poteto-help, the readme skill table, and guide page 10 match. slash commands keep the/pstack:prefix.disable-model-invocation: truestaystriggers: [user].b1a1298(#798) bump to 0.15.32, inplugin.json,.devin-plugin/plugin.json, and the readme marker.7054003(#800) reflect edits leaveplugins/cache/(devin's checkout is~/.local/share/devin/cli/plugins/cache). the destination is.devin/skills/<skill>/or~/.config/devin/skills/<skill>/. no pr and no push.96dec8e(#801) watch-pr distinguishes fork heads. the pstack scripts were copied verbatim.fe096ef(#802) watch-pr caps polling sleeps at the deadline. scripts copied verbatim.86c2448(#803) check-plan recognizes matching inline code delimiters. scripts copied verbatim.3c596e7(#804) orch prefers the active pr. the changed files are underpstack/skills/poteto-mode/scripts/orch/and were copied verbatim.f83fbe6(#805) worktree-audit reads transcript mtimes with gnu stat and date. keptrg -aand the devinsessions.dbdirectory.e73df47(#807) swarm phase a is numbered 1 to 4, phase b points at step 3, and phase d asks for a result table.skipped:
a0b0ad9(#806) claude code auto-update note. it sits in the claude code install section, and this readme has no such section.harness/pstack/claude-code/copies in every commit above. this port does not carry the generated claude code tree.pstack/.cursor-plugin/plugin.json. cursor packaging. the version bump lands in.devin-plugin/plugin.jsoninstead.harness proof (
devin plugins install,devin plugins list,/pstack:statechart,/pstack:ketchup-video,/pstack:poteto-mode) will follow as a comment. the devin cli can't log in from this cloud environment.