Repository navigation
port upstream pstack through 0.15.30 - #26
Conversation
sync pstack to 4ec7ffa: the statechart skill, guide, poteto-help, and version pins. Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>
caelaxie
left a comment
There was a problem hiding this comment.
changes requested (posting as comment, same account as the author).
reviewed at ed6a0b9 against upstream 4ec7ffa.
passes:
- content: statechart skill, scripts, guide page 6, poteto-help row, and readme row match upstream;
/pstack:prefix andtriggers: [user]are the usual devin mapping. - layout/target: skills under
skills/, no cursor packaging carried; skippingpstack/.cursor-plugin/plugin.json, the claude-code tree, and the monorepo workflow step is right. - parity: version
0.15.30inplugin.json,.devin-plugin/plugin.json, and the readme marker. - ci:
Validate plugingreen. - linkage: no open issue covers this sync, so none needed.
blocking:
- harness proof is missing. the body says it will follow as a comment. post it on this head (or a new one):
devin plugins installpinned to the full PR head sha (the--localwrapper that lets devin fetch the repo itself is fine; symlink-only proofs aren't), thendevin plugins listshowingpstackat0.15.30, plus the installed copy showingskills/statechart/SKILL.md. if login blocks a live turn, install + list + installed copy is enough for this text/script change.
once that lands i'll merge.
harness proofReal Devin CLI on a local box (no cloud agent, no git clone). PR head
1–3. before list, install, after list4–6. installed cache copy7. describe-only exercise8. 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. content/versions match upstream 4ec7ffa (0.15.30), layout is harness-correct, ci Validate plugin green on ed6a0b9, and harness proof (devin plugins install --local wrapper pinned to full head sha + list + installed statechart byte-identical) is on the pr. squash-merging.
synced to
4ec7ffa. compared88d4751..4ec7ffaforpstack/. later commits on upstreammainthrough9b93e81do not touchpstack/. version is0.15.30inplugin.json,.devin-plugin/plugin.json, and the readme marker.ported:
eb4db2b(#741) add the statechart skill. guide page 6, poteto-help, and the readme skill table match. slash commands keep the/pstack:prefix.disable-model-invocation: truestaystriggers: [user].7698375(#776) six fixes to the statechart skill. scripts copied verbatim.4ec7ffa(#779) delete statechart comments that repeat the code, and rename helpers. the version bumps to0.15.30. the same bump is in.devin-plugin/plugin.jsonand the readme marker, which is where this repo records the version.skipped:
harness/pstack/claude-code/copies in #741, #776, and #779. 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..github/workflows/validate-plugins.ymltest step. that workflow belongs to the cursor/plugins monorepo and also checks generated harness trees. this repo's ci is separate, so the step was not copied.node --test skills/statechart/scripts/test/*.test.mjspassed locally (67 tests, with tlc).harness proof (install,
devin plugins list, exercising/pstack:statechartand/pstack:poteto-mode) will follow as a comment. the devin cli can't log in from this cloud environment.