Repository navigation
2.0.0-rc.6: Lauren's 99743e9, marginal sections cut from four skills - #25
Merged
Merged
Conversation
All three components move from e5136d8 to 99743e9. Upstream changed only pstack: four skills lose marginal sections (pstack #634). interrogate's lead-judgment reference drops its preamble and Verdict Calibration, type-system-discipline keeps three of its eight rules and its description drops "refuse to lie to the compiler", migrate-callers drops its intro, three of its rules, and its closing line, and swarm drops its intro line, Phase A step 3, and one Phase C sentence. Two rules rewrote swarm lines upstream deleted, "Fan out N parallel cloud workers." and "not the cloud concurrency limit.", so both now match nothing and are removed. Removing them moves the derived merge base, which makes the three-way merge conflict on swarm, so pstack ran through bun tools/sync.mjs --bootstrap: it derived 120 files, kept the 13 declared forks, and changed only these four. cursor-team-kit and thermos synced cleanly with no file changes. The skill-file diff is upstream's line for line except the T3 form of those two deleted lines. poteto-mode's principle index entry never quoted the dropped clause and upstream left it alone, so it does not change.
VERSION, the manifests, and package.json move to 2.0.0-rc.6. CHANGES.md gets the entry with its behavior-delta table. NOTICE.md, NOTICE-skills.md, and the README point at upstream 99743e9, and the README's pinned installer command names the new tag.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
t3-pstack moves to Lauren Tan's pstack at cursor/plugins
99743e9for all three components. Only pstack changed upstream, in one commit, and its version stays 0.15.25:interrogate'sreferences/lead-judgment.mddrops its preamble and Verdict Calibration.principle-type-system-disciplinekeeps three of its eight rules, and its description drops "refuse to lie to the compiler".principle-migrate-callers-then-delete-legacy-apisdrops its intro, three of its four rules, and its closing line.swarmdrops its intro line, Phase A step 3, and one Phase C sentence. Its README change falls under a pathtools/upstream.jsonexcludes.This is Lauren's text plus mechanical version lines, with two substitutions removed because upstream deleted their targets. No other rule or fork changed. The port's diff to the four skill files is upstream's line for line, apart from the T3 form of the two deleted swarm lines those rules wrote.
Removed substitutions
Both rules had no
filesscope, and both rewrote a swarm line that #634 deletes, so at99743e9neither matches anything in any component:"Fan out N parallel cloud workers."to"Fan out N parallel workers."(swarm's intro line)."not the cloud concurrency limit."to"not the number that run at once."(swarm's Phase A step 3).Commits
sync.mjsderives the old side (e5136d8) without them too, so swarm's merge base no longer matched the port and the three-way merge conflicted on an undeclared path. pstack therefore ran throughbun tools/sync.mjs pstack 99743e9 --bootstrap, which derived 120 files, kept the 13 declared forks, and changed only the four files upstream touched. cursor-team-kit and thermos synced normally with no file changes. All three pins now read99743e9.VERSIONand the manifests move to 2.0.0-rc.6. CHANGES gets the entry with its behavior-delta table. NOTICE, NOTICE-skills, and the README point at99743e9.On T3
swarmkeeps every T3 dispatch anchor. Phase A step 4 still reads theswarm workersline in the model sheet, falls back to the default in its Models section, omitstargetforautoorinherit-parent, and falls back within a family throughorchestrator_capabilities. Upstream kept that step numbered 4 after deleting step 3, so Phase B's "delegate_taskcalls withmode: "async"and the step 4target" still names it. The per-worker worktree rule and the Models stamp are untouched. With step 3 gone, nothing defines N beyond the description, which is upstream's choice and carries no T3 text.interrogate's T3 dispatch lives inSKILL.md, which #634 leaves alone: Step 3'sdelegate_taskperinterrogate reviewersentry withrole: "review", the reviewer table, the family fallback, and the Models stamp. No rule touched the deletedlead-judgment.mdtext. Step 5 still casts the lead as a pragmatic engineer, not a neutral aggregator, and points atlead-judgment.md, which now opens at its filtering principles. The two-family panel lives inmodels.json's panel tier, enforced bygenerate.mjsandt3-sheet.mjs, andthermosreads the first two families of theinterrogate reviewersline. None of that moved.poteto-mode's index entry for Type System Discipline never quoted "refuse to lie to the compiler". It matches upstream's entry at99743e9, which #634 did not touch, so it stays. No port-only file quotes any deleted text.Checks
bun tools/sync.mjs <component> 99743e9 --dry-runfor pstack, cursor-team-kit, and thermos: exit 0, no undeclared fork, no denylist hit, no unmatched substitution. At the pin, pstack reads 120 unchanged files and the same 13 declared forks with 469 changed lines as before.bun tools/generate.mjs --check: 17 generated files current, and the hooks freeze check passes. Neither hooks file differs fromorigin/main.GIT_CONFIG_GLOBAL=/dev/null bun test tests/ --timeout 30000: 537 pass, 46 skip, 0 fail.bun run typecheck,bun test orch watch-pr check-plan.test.ts(119 pass), and the Prettier check all pass.t3-sheet.mjsagainst this machine's sheet and catalog: every role line parses and dispatches.Note
Low Risk
Upstream skill text deletions and version/sync metadata only; T3 dispatch rules in swarm and interrogate SKILL.md are untouched.
Overview
Release 2.0.0-rc.6 syncs t3-pstack to Lauren Tan's upstream pstack at
cursor/plugins99743e9(pstack #634): four skills are trimmed by deleting redundant sections only—no new behavior in those files.interrogatelead-judgment.mdloses its opening framing and Verdict Calibration.principle-type-system-disciplinekeeps three rules (brand primitives, parse at boundaries, derive from schemas) and drops five others plus "refuse to lie to the compiler" from the description.principle-migrate-callers-then-delete-legacy-apiskeeps the test-update rule and applicability bullets; intro, three rules, and the dual-path closing are removed.swarmdrops its intro, Phase A step 3 (setting N), and one Phase C aggregation line; T3delegate_task/ worktree wording is unchanged.Port tooling:
tools/upstream.jsonpins all three components to99743e9. Two entries intools/substitutions.jsonare removed because upstream deleted the "cloud workers" lines they rewrote.CHANGES.md,NOTICE*, andREADMErecord the bump and new pin; version manifests move to 2.0.0-rc.6.Reviewed by Cursor Bugbot for commit 0dcf87d. Bugbot is set up for automated code reviews on this repo. Configure here.