Repository navigation
port pstack changes after 0.15.25 - #3
Conversation
Copy the shared doc and skill edits from upstream main. Leave the version at 0.15.25. Co-authored-by: Kaitian Xie <[email protected]>
caelaxie
left a comment
There was a problem hiding this comment.
verdict: changes requested (posted as comment, same account authored the pr).
content looks right on a first pass. typescript-best-practices SKILL.md + references/patterns.md match upstream #700 verbatim, docs/harness/claude-code.md is a straight copy, and the #704 guide edits keep the droid-specific wording only where droid needs it. skips (claude-code tree, readme lines, check-pstack-docs.mjs) are justified. versions unchanged at pstack 0.15.25 / cursor-team-kit 1.2.0, which matches upstream. no open issue to link.
blocking: no harness proof yet. the body says it follows in a comment. before merge, post the real output of:
droid plugin marketplace addfor this branchdroid plugin install pstack@droid-pstackdroid plugin listshowing pstack loadedtypescript-best-practicesexercised in droid (the one changed skill)
also let Validate plugin finish green. i'll re-review when the proof comment or a new commit lands.
|
harness on cache is
branded-types row:
no ci |
caelaxie
left a comment
There was a problem hiding this comment.
approve (comment review, same account as author).
harness proof is in now: marketplace add, droid plugin install pstack@droid-pstack@…, and droid plugin list all on c1ee4e9, cache shows the type-guard brand line in typescript-best-practices and the #704 guide pages. validate-plugin, typecheck, and 68/68 tests pass; ci green on the same sha.
content and parity already passed last round. skips are justified. no issue to link. the live droid exec turn is blocked on FACTORY_API_KEY, noted, not blocking for doc-only skill text that's proven loaded from the installed cache.
squash-merging.
syncs this port with upstream cursor/plugins
pstack/on main after the0.15.25bump (6ce45c59b9). upstream is still0.15.25. cursor-team-kit is still1.2.0. no version bump.checked upstream main through
019a6a2059. nothing newer touchespstack/.ported
#7000317eb8b85brand via a type guard inskills/typescript-best-practices/SKILL.mdandreferences/patterns.md. both files matched upstream0.15.25byte for byte, so the new text is copied verbatim.#704019a6a2059guide wording indocs/guide/02throughdocs/guide/10.03-understand.md,04-design.md, and08-principles.mdnow match upstream. the other pages keep the existing droid wording (install,~/.factory/paths, background worktrees, bundled/deslop, no custom mode). the parallel-agents pitfall takes the new first sentence and keeps "own worktree" / background subagents.#637182f2ce414and#7051b80080058shared docdocs/harness/claude-code.md, copied verbatim from upstream head (the#705edits included).docs/guide/01-setup.mdhad one upstream hunk, the custom-mode docs anchor. this port already replaced that paragraph because droid has no custom modes, so the file is unchanged.already present
compared file content with upstream at
6ce45c59b9and at019a6a2059.#633e5136d8241the four principle skills,skills/correct/SKILL.md, thepoteto-modeprinciple index, and the attack-the-premise / test-behavior lines indocs/guide/08-principles.mdalready match upstream head.#63499743e9c15lead-judgment.md,principle-make-operations-idempotent,principle-migrate-callers-then-delete-legacy-apis,principle-type-system-discipline, and the three cuts inskills/swarm/SKILL.mdalready match. swarm still uses the droid seat wording and~/.factory/AGENTS.md.#702a968159326poteto-modeandmake-bot-uialready use their folder names. ci already requires every skillnameto match its directory.skipped
#637/#705generated treeharness/pstack/claude-code/,.claude-pluginmanifests, andscripts/generate-pstack-harness-trees.mjs. those install pstack into claude code. this repo is the droid plugin.#637/#704/#705pstack/README.mdlines. this readme is the droid install page. it does not carry the upstream skill tables, the custom-mode paragraph, or the claude code install notes. the principle restatements from those readme lines are already in the skill files and indocs/guide/08-principles.md.#704scripts/check-pstack-docs.mjs. a path-only adaptation (this repo as the tree,droids/instead ofagents/) still exits 1 with 83 problems, all "readme does not list" a skill, playbook, or droid. the guide checks themselves pass. making the script pass would mean rewriting the readme contract, not adapting paths. not wired into ci.cursor-team-kit
deslop,control-cli, andcontrol-uimatch cursor-team-kit1.2.0at upstream head. no commits undercursor-team-kit/since the0.15.25bump. left alone.harness
droid
0.244.0, isolatedHOME. marketplace add ofhttps://github.com/pstack-ports/droid-pstack#cursor/port-pstack-post-0.15.25-e093registered[email protected]. install landed atc1ee4e9. the cache has 57 skills, everynamematches its folder, andtypescript-best-practiceshas the type-guard brand line.node scripts/validate-plugin.mjsok.bun run typecheckexit 0.bun test68 pass, 0 fail.droid execexited 1: not logged in, andFACTORY_API_KEYis unset.ci
Validate pluginis green onc1ee4e9: run 39087509222. it printedValidated 57 skills, 3 droids, and 23 playbooks.command output is in a comment.