Add agy as an external pstack-runner provider - #31
Draft
realtime3392 wants to merge 2 commits into
Draft
Conversation
pstack-runner can now launch one Agy print-mode child with the same preflight, receipt, and no-fallback contract as Grok. Agy stays provider-only. xhigh and max pin to high. JSON receipts use pinned-argv because the CLI does not report a served model id.
Record Agy on the parent route table and in External lanes without adding a fifth first-run matrix family. Setup and the four-model panel stay Fable, Sol, Grok, and Opus.
Greptile SummaryThe PR adds Agy as an external runner provider while keeping Claude Code and Codex as the only parent harnesses.
Confidence Score: 5/5The reviewed implementation appears safe to merge once the draft PR’s stated real-surface verification requirement is completed. The Agy lane is consistently wired through provider validation, preflight, invocation, parsing, failure handling, receipts, tests, and documentation, with no concrete changed-code defect remaining.
|
| Filename | Overview |
|---|---|
| plugins/pstack/skills/poteto-mode/scripts/runner/commands.ts | Adds Agy preflight and invocation arguments, including access-mode and effort mappings. |
| plugins/pstack/skills/poteto-mode/scripts/runner/run.ts | Integrates Agy model-list validation, failure classification, and pinned-argv model evidence. |
| plugins/pstack/skills/poteto-mode/scripts/runner/parse-output.ts | Normalizes Agy’s JSON response, session identifier, and token usage into the runner receipt format. |
| plugins/pstack/skills/poteto-mode/scripts/runner/types.ts | Extends the external provider type and CLI validation universe with Agy. |
| plugins/pstack/skills/poteto-mode/scripts/runner/run.test.ts | Covers Agy execution, receipts, authentication failure, unavailable models, and environment handling. |
| plugins/pstack/skills/poteto-mode/references/provider-dispatch.md | Documents Agy routing, sandbox mappings, preflight behavior, effort normalization, and receipt requirements. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
P[Claude Code or Codex parent] --> R[pstack-runner]
R --> F[agy models preflight]
F -->|authenticated and model listed| M{Access mode}
F -->|sign-in required| U[Unauthenticated receipt]
F -->|model absent| A[Unavailable-model receipt]
M -->|read-only| RO[agy --mode plan --sandbox]
M -->|isolated-write| IW[agy --mode accept-edits --sandbox --disable-slash-commands]
RO --> J[Parse Agy JSON result]
IW --> J
J --> C[Complete receipt with pinned-argv evidence]
Reviews (1): Last reviewed commit: "docs(pstack): document the agy external ..." | Re-trigger Greptile
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.
What changed
pstack-runneraccepts--provider agyfrom a Claude Code or Codexparent. The lane preflights
agy models, invokes print mode once,maps read-only to
--mode plan --sandbox, maps isolated-write to--mode accept-edits --sandbox --disable-slash-commands, and neverpasses
--dangerously-skip-permissions. Agy--effortislow|medium|high;xhighandmaxpin tohigh. Receipts useCodex-style
pinned-argvbecause Agy JSON does not report a servedmodel id.
Agy is not a parent harness and is not a fifth first-run matrix
family. The four-model panel stays Fable, Sol, Grok, and Opus. The
portable slug is
agy:gemini-3.1-pro-high@high.Commits:
Verification
validation pass.
Live evidence:
feat/agy-external-providerin a localclone of
open-pstack(f9e5811), runner atplugins/pstack/skills/poteto-mode/scripts/runner/pstack-runner.Child CLI:
agy1.1.19 at/usr/local/bin/agy.pstack-runnerfrom the shell, parentclaude,provider
agy. Not yet installed into Claude Code or Codex pluginsurfaces.
pong, receiptstatus: complete,modelEvidence: pinned-argv, preflight passed, argv used--mode plan --sandboxand--print-timeout 8760h. No--dangerously-skip-permissions.Also: 162 Bun tests, strict typecheck,
PSTACK_STATIC_ONLY=1tests/skill-collision-repro.sh, andgrok plugin validate plugins/pstack.This stays a draft until the candidate is installed and the lane is
run from Claude Code and from Codex.
A pull request without live evidence remains a draft. Do not merge,
tag, release, or roll it out.