Repository navigation
fix(orchestrate): restore the model-family verifier and the path for runtimes without nested agents (#265) - #266
Conversation
…runtimes without nested agents Fixes #265 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Merge Protections🟢 All 2 merge protections satisfied — ready to merge. Show 2 satisfied protections🟢 Queue ready pull requestsAutomatically queue non-draft pull requests targeting main after verify and live-gate pass on the head and every review thread is resolved.
🟢 🚦 Auto-queueWhen all merge protections are satisfied and these conditions match, this pull request will be queued automatically.
|
Live evidence: Open Pstack
|
|
[Low impact] The changes appear safe to merge once the required live gate passes. SummaryThe PR restores Orchestrate’s different-model-family verifier rule and lets the coordinator launch workers when a sub-coordinator cannot.
Reviews (1) · Last reviewed commit: "fix(orchestrate): restore the model-fami..." · Reviewed by Greptile |
✅ Merged into
|
✅ No blocking findingsLight review ran with 1 elected reviewer. WalkthroughThis fixes two wording errors that the last upstream sync
Details
First review of this pull request. @unfret review · @unfret status · @unfret help |
Why
The 0.15.36 sync (#264) left two errors in the Orchestrate playbook: the verifier rule could not be met on a native-only sheet, and a sub-coordinator had no way to start workers on a runtime whose subagents cannot start agents. Fixes #265.
Summary
Evidence
orchestrate.md:13says "on a different provider";cursor/main:pstack/skills/poteto-mode/playbooks/orchestrate.md:13says "on a different model family". A Claude-only sheet (Opus and Sonnet) has no second provider. After: matches upstream.b4c5e3d) had the sub-coordinator spawn workers only "where the runtime lets a subagent spawn one"; sync: Cursor pstack 0.15.36 as Open Pstack 1.17.0 (#262) #264's rewrite dropped that.plugins/pstack/agents/pstack-*.mddisallowAgentandTask. After: the coordinator starts those workers.skill-invocation:poteto-modein Claude Code and Codex.Merge Danger
Door: two-way
Blast Radius: prose
Wording in one playbook and one mapping line.
Acceptance criteria
orchestrate.md:13.codex-tools.md." Done,orchestrate.md:11andcodex-tools.md#subagent-policy.verify-open-pstack." Gates below; live gate pending.Gate results
On
a2474ff:bun run test0,bun run typecheck0,bun run static0,claude plugin validate plugins/pstack0.Not verified
spawn_agent; the new bullet makes no Codex claim.Introduced by: #264
🤖 Generated with Claude Code