Repository navigation
Pin read-only seats and restore judgment spawn rules - #26
Conversation
how, interrogate, and the arena judge spawn droids with tools: read-only. poteto-judgment keeps the spawn, resume, and substitution rules, and setup pins the read-only twins with the seats. Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>
caelaxie
left a comment
There was a problem hiding this comment.
verdict: changes requested (comment review, same account as author).
content matches #17 and #18's smallest fixes: read-only twins with tools: read-only, how/interrogate/arena pointed at them, the "say so in the brief" sentences gone, explorer only for auto/inherit-parent, and poteto-judgment's spawn/resume/worker sentences restored. the setup-pstack twin-pinning line is a needed follow-on, fine.
blocking:
- ci is red on a339165.
Validate pluginfails ~13s in (log line 228, exit 1). every other open droid pr is green at the same base 3a60c35, so this diff broke it. my guess (unverified) is the install/loaded-droids check or droid rejecting the newtools: read-onlyfrontmatter. fix and get ci green at the final head. - harness proof. this adds droids and a new frontmatter key, so ci alone isn't enough. post in the pr:
droid plugin marketplace add pstack-ports/droid-pstack#<branch>+droid plugin install pstack@<name>+droid plugin listat the final head sha, the installeddroids/listing showing both-readonlytwins, and proof droid honorstools: read-only(e.g. a spawn ofpoteto-agent-readonlywhose tool list has no edit/execute tools, or droid's own droid listing showing its tools). if a live model turn is blocked only by cli auth, say so and show the installed copy.
non-blocking nit inline.
| @@ -0,0 +1,11 @@ | |||
| --- | |||
| name: poteto-agent-readonly | |||
| description: Read-only delegate subagent for code-writing and helper tasks spawned during a `/poteto-mode` run. Use for delegates, not as the runner for `/poteto-mode` itself. Spawn a fresh `poteto-agent-readonly` for each new task, and resume one only in the strict cases that poteto-mode's Subagents section names. Reads the `poteto-mode` skill's `SKILL.md` in full before any work, including its inline Principles index. Substituting `worker` skips that read and drifts. | |||
There was a problem hiding this comment.
nit: "Read-only delegate subagent for code-writing and helper tasks" contradicts itself; the parent reads this to pick a seat. consider "Read-only delegate subagent for exploration, review, and helper tasks" or just "Read-only delegate subagent for helper tasks". not blocking.
Filename order puts poteto-agent-readonly.md ahead of poteto-agent.md, while the id order does the reverse. Compare both sides as sorted ids so the read-only twins pass Validate plugin. Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>
|
Harness proof for d09b04a. Command output is unedited. |
caelaxie
left a comment
There was a problem hiding this comment.
verdict: pass at d09b04a, merging.
- ci is green now. the ci.yml change is one
.sort()so droid ids compare as ids; needed for the new-readonlyfilenames, nothing else touched. - matches #17 and #18's smallest fixes. the twins differ from their seats only in
name, aRead-onlyprefix, andtools: read-only;poteto-judgmentdiffers frompoteto-agentonly in name, model, seat role, and "on the judgment seat". - droid facts check out against custom droids:
tools: read-onlycategory, built-inexploreris read-only, task has noreadonlyfield. - no cursor files.
Closes #17/Closes #18, not a draft.
non-blocking: the twins' descriptions still say "code-writing" and "hardest changes" while the droid can't write. every spawn site names them explicitly, so it's fine for now.
Closes #17
Closes #18
Are two more droids too much?
No.
tools: read-onlyis what actually enforces the upstreamreadonly: trueguarantee. A sentence in the brief does not.explorercannot pin a seat model, so it is only theauto/inherit-parentpath. The two twins keep the model panel:poteto-agent-readonly(grok-4.7) andpoteto-judgment-readonly(claude-opus-5-5)./how,/interrogate, and the arena judge spawn those droids. Arena candidates still use the full-tool seats, because they write./setup-pstackwrites or deletes the read-only twin whenever it writes or deletes the seat, so a personal model pin applies to both.poteto-judgmentis the code-seat droid's description again, with the judgment-seat role, the judgment model, and "on the judgment seat" in the body. The spawn, resume, andworkersubstitution sentences are back.Validate plugin compared droid filenames (still ending in
.md) with ids (the.mdstripped).-sorts before., sopoteto-agent-readonly.mdcame out ahead ofpoteto-agent.md, while the idpoteto-agentcomes ahead ofpoteto-agent-readonly. The checker now sorts both sides as ids. That is the only workflow change.