Repository navigation
port upstream pstack through 0.15.27 - #10
Conversation
Drop the rewritten-prompt ban from sharpen (#712), bump to 0.15.27 (#713), and offer formal verification in setup with the matching help and guide sync (#715). Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>
|
harness proof. devin 3000.11.92 (7f4e64980e22). not logged in, so install, list, and the skill run are blocked. validator passed first:
|
caelaxie
left a comment
There was a problem hiding this comment.
content and parity look good for 0.15.27 (#712/#713/#715). setup ### 9 formal offer is the right numbering. skips justified. versions + ci ok. no issue.
changes requested: harness proof is incomplete. comment shows devin auth not logged in, so install/list/skill run all failed. need a real devin plugins install + devin plugins list plus changed skills exercised (sharpen / setup-pstack / poteto-help). offline validator alone is not enough.
re-review on a passing proof comment or body edit.
harness proofReal Devin CLI on a local box (no cloud agent, no git clone). PR head
1. installThe CLI reads 2. list3. sharpenIn the exported transcript, the user turn is the sharpen skill body followed by the prompt. This is the post-#712 text, without the rewritten-prompt ban: 4. setup-pstack (describe-only, nothing written)The first run used no allowlist and the default For the rerun I added a read-only allowlist to the scratch project. Writes are denied, and I did not use After the run, 5. poteto-help6. installed sharpen SKILL.mdNotes: the box also has an unrelated always-on rule, |
caelaxie
left a comment
There was a problem hiding this comment.
approved at 4f3ab8e (posting as COMMENT since author and reviewer are the same account).
- minimum changes: unchanged since my last pass at this head. synced to upstream
f9be7aa, which is still the newestcursor/pluginscommit touchingpstack/. claude code tree,.cursor-plugin, and claude code install text skipped. no cursor-team-kit. - parity:
0.15.27inplugin.json,.devin-plugin/plugin.json, and the readme marker, matching upstream #713. - harness proof (#10 (comment)): accepted.
devin auth statusis logged in.devin plugins install --localinstalled a wrapper whoserequiredPluginspinspstack-ports/devin-pstackat the full head sha4f3ab8ef…. devin fetched pstack from github into its own plugin cache (github.com_pstack-ports_devin-pstack-…/0.15.27/). that is the whole repo at that sha, not a symlink into a working tree and not a hand-picked subset.devin plugins listshowspstack v0.15.27, a version that exists only on this branch./pstack:sharpenran with the post-#712 body./pstack:setup-pstack(describe-only) loaded from the cache path and listed the new #715 formal-verification offer./pstack:poteto-helpanswered from the installed README. - no issue linked, which fits a plain upstream sync.
CI Validate plugin is green on 4f3ab8e, and the head has not moved past the proof sha. squash-merging pinned to 4f3ab8ef851991a546b056b3fb52d584def338a4.
synced to
f9be7aa. upstreammainhas no later commit that touchespstack/. version is0.15.27inplugin.json,.devin-plugin/plugin.json, and the readme marker.ported:
75fabace(#712) drop the rewritten-prompt ban fromskills/sharpen/SKILL.md. kepttriggers: [user]and the/pstack:sharpendescription.0148b78c(#713) bumpplugin.jsonto0.15.27. the same bump is in.devin-plugin/plugin.jsonand the readme marker, which is where this repo records the version.f9be7aa(#715) offer formal verification in setup, and syncpoteto-help, its recipes, and the guide. slash commands keep the/pstack:prefix..cursor/skillsstays.devin/skills. the new setup step is### 9because this port already uses### 8for the verification offer, after the devin profile steps.skipped:
harness/pstack/claude-code/copies in #712, #713, and #715. 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.docs/guide/01-setup.md(/plugin marketplace add cursor/plugins,~/.claude/rules/pstack-models.md, link todocs/harness/claude-code.md). this port's install section is the devin cli, and that adapter file is not in the repo.poteto-helprewrite that adds the claude code install and restates the cursor-only paragraph. this port already replaced that paragraph with the devin wording, and the new text points at the claude code adapter this repo does not carry.