feat: add agy (Antigravity CLI) as a supported --cli provider by yetmike · Pull Request #231 · steipete/summarize

yetmike · 2026-05-31T15:00:46Z

Adds agy alongside claude, gemini, codex, agent, openclaw, opencode, and copilot. Uses a plain-text path (--print + stdin) since agy does not support --output-format json. Default model is null - agy uses its own session default (Gemini 3.5 Flash) when no model is specified. Binary resolved via AGY_PATH env var or agy on PATH.

clawsweeper · 2026-05-31T15:01:35Z

Codex review: needs maintainer review before merge. Reviewed May 31, 2026, 12:28 PM ET / 16:28 UTC.

Summary
Adds explicit Antigravity agy CLI provider support across CLI/model parsing, config, daemon env/model discovery, extension settings, README docs, and tests.

Reproducibility: not applicable. as a feature PR rather than a bug report. The contributor did provide terminal proof of the new summarize --cli agy path producing a real summary.

Review metrics: 2 noteworthy metrics.

Changed surface: 26 files changed, +324/-22. Provider registration crosses CLI execution, config parsing, daemon discovery, extension settings, docs, and tests.
Provider test coverage: 1 added test file, 4 updated test files. The diff adds focused agy provider tests and updates config/daemon tests for the new provider path.

Merge readiness
Overall: 🐚 platinum hermit
Proof: 🦞 diamond lobster
Patch quality: 🐚 platinum hermit
Result: ready for maintainer review.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

none.

Risk before merge

[P1] This adds a new authenticated provider path plus daemon snapshot handling for ANTIGRAVITY_API_KEY, so maintainers should explicitly accept that provider/auth surface before merge.
[P1] I did not run pnpm -s check because this review is read-only; contributor-reported targeted tests and the passing secret scan are useful, but the full repository gate remains the final integration check.

Maintainer options:

Accept explicit Antigravity support
Maintainers can accept the new AGY_PATH and ANTIGRAVITY_API_KEY surface because the PR keeps agy opt-in and includes focused provider/config/daemon coverage.
Require full gate proof first
Before merge, ask for or run pnpm -s check on the latest head to cover the cross-package provider, daemon, and extension surface.
Pause provider adoption
If Antigravity is not a desired core provider, pause or close this PR rather than carrying a new authenticated CLI surface in core.

Next step before merge

No automated repair is needed; the remaining action is maintainer acceptance of the new provider/auth surface and normal gate validation.

Security
Cleared: No concrete supply-chain issue was found; the new credential/env handling follows existing daemon snapshot patterns and the dangerous tool flag is gated by the existing allowTools path.

Review details

Best possible solution:

Merge the explicit-only agy provider if maintainers want Antigravity support in core, keeping it opt-in and covered by provider, config, daemon, extension, and docs tests.

Do we have a high-confidence way to reproduce the issue?

Not applicable as a feature PR rather than a bug report. The contributor did provide terminal proof of the new summarize --cli agy path producing a real summary.

Is this the best way to solve the issue?

Yes, if maintainers want this provider in core: the PR follows the existing CLI-provider pattern, keeps agy out of default auto fallback, and covers the daemon/config paths. The remaining decision is provider/auth acceptance, not a code repair.

AGENTS.md: found and applied where relevant.

Codex review notes: model gpt-5.5, reasoning high; reviewed against 6d43d3427296.

Label changes

Label justifications:

P2: This is a scoped provider feature with daemon/auth plumbing and limited blast radius, not an urgent production regression.
merge-risk: 🚨 auth-provider: The diff adds a new authenticated CLI provider plus AGY_PATH and ANTIGRAVITY_API_KEY handling that normal CI cannot fully validate without a real installed/authenticated agy setup.
rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🦞 diamond lobster and patch quality is 🐚 platinum hermit.
status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (terminal): The PR discussion includes terminal output from a real summarize --cli agy run that produced an after-change summary.
proof: sufficient: Contributor real behavior proof is sufficient. The PR discussion includes terminal output from a real summarize --cli agy run that produced an after-change summary.

Evidence reviewed

Acceptance criteria:

[P1] pnpm -s check.
[P1] pnpm -s test -- tests/llm.cli.agy.test.ts tests/config.test.ts tests/daemon.agent.test.ts tests/daemon.chat.test.ts tests/daemon.config.test.ts.

What I checked:

Repository policy read: AGENTS.md was read in full; its read-only, monorepo, and pnpm -s check guidance shaped this review. (AGENTS.md:1, 6d43d3427296)
PR head diff reviewed: The PR patch for head 172a3d2 changes 26 files with 324 additions and 22 deletions, spanning CLI provider registration, daemon paths, extension settings, docs, and tests. (172a3d2e532a)
Provider execution path: The PR adds an agy execution branch that invokes the resolved binary with --print, optional --model, optional tool-permission flag, stdin prompt input, and plain-text stdout handling. (src/llm/cli.ts:374, 172a3d2e532a)
Explicit-only default behavior: The PR adds agy to parsing/default-model support while documenting that it remains excluded from the default auto-fallback order unless explicitly requested or configured. (src/llm/provider-profile.ts:117, 172a3d2e532a)
Config and daemon auth surface: The PR adds cli.agy, AGY_PATH, and ANTIGRAVITY_API_KEY daemon env snapshot support, which is the main maintainer-facing provider/auth decision before merge. (src/daemon/env-snapshot.ts:41, 172a3d2e532a)
Earlier diagnostic gap fixed: The latest head maps CLI_AGY to agy, maps CLI_OPENCODE explicitly, falls back to unknown, and adds a regression test for unavailable cli/agy diagnostics. (src/daemon/agent-model.ts:206, 172a3d2e532a)

Likely related people:

steipete: Current-main blame and log history for the CLI provider registry, provider defaults, daemon model paths, extension settings, and README provider docs trace to Peter Steinberger's v0.16.3 release/import commit. (role: current-main area owner; confidence: medium; commits: fcf8c8e5e98d; files: src/llm/cli.ts, src/llm/provider-profile.ts, src/run/env.ts)

What the crustacean ranks mean

🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
🌊 off-meta tidepool: rating does not apply to this item.

Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

How this review workflow works

ClawSweeper keeps one durable marker-backed review comment per issue or PR.
Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
Maintainers can also comment @clawsweeper review to request a fresh review only.
Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

yetmike · 2026-05-31T15:17:54Z

Proof

Tested on Linux x64 with agy 1.0.3 and summarize 0.11.3 (fork build):

$ summarize --cli agy --length short "https://youtu.be/uWdIgftpvBI"

When starting a business, the most effective strategy is to avoid the
middle ground: either sell extremely expensive stuff to a select few or
sell something super cheap to everyone. The middle is where people die.

Cached · 18m 25s YouTube · 4.2k words · cli/agy/gemini-2.5-flash

Also fixed the three correctness gaps flagged by ClawSweeper:

src/run/run-models.ts — added agy config model lookup
src/model-auto-cli.ts — added agy to auto-fallback model chain
src/config/parse-helpers.ts — added agy to config provider parser

Commit updated: 3d55f5c

@clawsweeper re-review

clawsweeper · 2026-05-31T15:17:57Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

State: Complete
Detail: The targeted re-review finished, the durable review comment was updated, and the synced verdict was routed.
Run: https://github.com/openclaw/clawsweeper/actions/runs/26716448919
Updated: 2026-05-31T15:23:48.826Z

yetmike · 2026-05-31T15:33:11Z

Addressed all P2 findings:

src/config/sections.ts — parseCliConfig now parses and returns the cli.agy block (binary, model, extraArgs). Config like cli: { agy: { binary: "/usr/local/bin/agy", model: "Gemini 3.5 Flash (Medium)" } } is fully round-tripped. Test added in tests/config.test.ts.

src/daemon/env-snapshot.ts — AGY_PATH added to ENV_KEYS so daemon env snapshots preserve custom binary paths across restarts. Test updated in tests/daemon.config.test.ts.

DEFAULT_AUTO_CLI_ORDER — agy included with justification comment: it uses the same plain-text --print interface as copilot and is gated by binary availability (AGY_PATH / PATH check), so it fails gracefully when not installed.

19 files, 232 additions total. All tests pass.

@clawsweeper re-review

clawsweeper · 2026-05-31T15:33:15Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

State: Superseded
Detail: A newer re-review for this item started before this run finished, so GitHub cancelled this older run. Check the latest ClawSweeper run for the current result.
Run: https://github.com/openclaw/clawsweeper/actions/runs/26716801502
Updated: 2026-05-31T15:35:16.422Z

yetmike · 2026-05-31T15:35:03Z

Added README documentation following the existing pattern for other CLI providers:

Section heading updated to include Antigravity
Entry added: agy (Antigravity CLI) → --cli agy / --model cli/agy/<model>
AGY_PATH added to the PATH env var list
ANTIGRAVITY_API_KEY added to the auth requirements
Smoke test line added
Explicit allowlist and autoFallback order examples updated

@clawsweeper re-review

clawsweeper · 2026-05-31T15:35:05Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

State: Complete
Detail: The targeted re-review finished, the durable review comment was updated, and the synced verdict was routed.
Run: https://github.com/openclaw/clawsweeper/actions/runs/26716844373
Updated: 2026-05-31T15:41:47.086Z

yetmike · 2026-05-31T15:46:08Z

Completed all remaining provider registry updates:

src/run/run-config.ts:64 — added agy to the bare --cli default enable list

apps/chrome-extension/src/lib/settings.ts:147 — added agy to extension auto CLI order normalizer and default autoCliOrder string

src/run/cli-fallback-state.ts:13 — added agy to last-successful CLI provider state parser so remembered agy successes are read back for prioritization

Policy: agy is in DEFAULT_AUTO_CLI_ORDER (default-auto). Rationale in provider-profile.ts comment: same plain-text --print interface as copilot, fails gracefully when not installed via binary availability check.

23 files, 246 additions. All tests pass.

@clawsweeper re-review

clawsweeper · 2026-05-31T15:46:10Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

State: Complete
Detail: The targeted re-review finished, the durable review comment was updated, and the synced verdict was routed.
Run: https://github.com/openclaw/clawsweeper/actions/runs/26717095323
Updated: 2026-05-31T15:52:38.669Z

yetmike · 2026-05-31T15:55:48Z

Switched to explicit-only for implicit auto fallback:

Removed agy from DEFAULT_AUTO_CLI_ORDER in src/llm/provider-profile.ts — bare --cli and auto-fallback will not try agy unless explicitly configured
Removed from bare --cli default enable list in src/run/run-config.ts
Removed from extension default autoCliOrder in apps/chrome-extension/src/lib/settings.ts
README autoFallback order example no longer includes agy

Full explicit support remains intact: --cli agy, --model cli/agy/<model>, AGY_PATH, cli.agy config block, fallback state, daemon picker, and extension normalizer all work when opted in. A comment in provider-profile.ts documents the exclusion.

@clawsweeper re-review

clawsweeper · 2026-05-31T15:55:51Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

State: Complete
Detail: The targeted re-review finished, the durable review comment was updated, and the synced verdict was routed.
Run: https://github.com/openclaw/clawsweeper/actions/runs/26717325491
Updated: 2026-05-31T16:01:26.152Z

yetmike · 2026-05-31T16:03:30Z

Fixed both P1 daemon gaps:

src/daemon/env-snapshot.ts — added ANTIGRAVITY_API_KEY to ENV_KEYS so the credential survives daemon install/restart. Test updated in tests/daemon.config.test.ts.

src/daemon/chat.ts — resolveConfiguredCliModel — the fallthrough chain now explicitly handles opencode, agy, and copilot as separate branches instead of falling through to cli.opencode.model. cli/agy now correctly reads cli.agy.model from config. Test added in tests/daemon.chat.test.ts mirroring the existing opencode pattern.

All 27 tests across daemon.chat, daemon.config, and llm.cli.agy pass.

@clawsweeper re-review

clawsweeper · 2026-05-31T16:03:32Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

State: Complete
Detail: The targeted re-review finished, the durable review comment was updated, and the synced verdict was routed.
Run: https://github.com/openclaw/clawsweeper/actions/runs/26717513092
Updated: 2026-05-31T16:09:31.698Z

Adds agy alongside claude, gemini, codex, agent, openclaw, opencode, and copilot. Uses a plain-text path (--print + stdin) since agy does not support --output-format json. Default model is null — agy uses its own session default (Gemini 3.5 Flash) when no model is specified. Binary resolved via AGY_PATH env var or agy on PATH. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

yetmike · 2026-05-31T16:23:26Z

Fixed the P2 diagnostic gap:

src/daemon/agent-model.ts — CLI_AGY now maps to "agy" in the unavailable-provider diagnostic. Also fixed a pre-existing latent bug: CLI_OPENCODE now explicitly maps to "opencode" instead of falling through, and the final fallback returns "unknown" rather than silently misidentifying unknown providers as "opencode".

tests/daemon.agent.test.ts — regression test added: asserts that when CLI_AGY is in the attempt list and agy is not on PATH, the error message contains CLI unavailable: agy.

26 files, 324 additions total. All tests pass.

@clawsweeper re-review

clawsweeper · 2026-05-31T16:23:29Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

State: Complete
Detail: The targeted re-review finished, the durable review comment was updated, and the synced verdict was routed.
Run: https://github.com/openclaw/clawsweeper/actions/runs/26717973862
Updated: 2026-05-31T16:28:50.226Z

yetmike force-pushed the main branch from 782225e to 3d55f5c Compare May 31, 2026 15:17

yetmike force-pushed the main branch 2 times, most recently from 422e6dd to 3bce124 Compare May 31, 2026 15:33

yetmike force-pushed the main branch from 3bce124 to c76e720 Compare May 31, 2026 15:34

yetmike force-pushed the main branch from c76e720 to 1433c52 Compare May 31, 2026 15:46

yetmike force-pushed the main branch from 1433c52 to fb1eec7 Compare May 31, 2026 15:55

yetmike force-pushed the main branch from fb1eec7 to 5b814ac Compare May 31, 2026 16:03

yetmike force-pushed the main branch from 5b814ac to 172a3d2 Compare May 31, 2026 16:23

steipete added 8 commits June 1, 2026 03:27

fix: align agy provider with print mode

08382b9

chore: document agy stdin transport

d03de50

fix: propagate agy print timeout

37ec127

fix: avoid auto-approving agy tools

8013fd7

fix: sandbox agy text summaries

2867a6b

fix: keep agy prompt on stdin

4f7a56a

fix: pass agy stdin prompt safely

bec9c6e

fix: use agy stdin print mode

bb9c485

steipete merged commit 821e766 into steipete:main Jun 1, 2026
1 check passed

Conversation

yetmike commented May 31, 2026

Uh oh!

clawsweeper Bot commented May 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yetmike commented May 31, 2026

Proof

Uh oh!

clawsweeper Bot commented May 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yetmike commented May 31, 2026

Uh oh!

clawsweeper Bot commented May 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yetmike commented May 31, 2026

Uh oh!

clawsweeper Bot commented May 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yetmike commented May 31, 2026

Uh oh!

clawsweeper Bot commented May 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yetmike commented May 31, 2026

Uh oh!

clawsweeper Bot commented May 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yetmike commented May 31, 2026

Uh oh!

clawsweeper Bot commented May 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yetmike commented May 31, 2026

Uh oh!

clawsweeper Bot commented May 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

clawsweeper Bot commented May 31, 2026 •

edited

Loading

clawsweeper Bot commented May 31, 2026 •

edited

Loading

clawsweeper Bot commented May 31, 2026 •

edited

Loading

clawsweeper Bot commented May 31, 2026 •

edited

Loading

clawsweeper Bot commented May 31, 2026 •

edited

Loading

clawsweeper Bot commented May 31, 2026 •

edited

Loading

clawsweeper Bot commented May 31, 2026 •

edited

Loading

clawsweeper Bot commented May 31, 2026 •

edited

Loading