chore(cli-version): verify agy against 1.2.2 - #991
Merged
Merged
Conversation
Gate B: 7/7, 150.92s, against a manifest download whose sha512 was checked before it ran. The log reports only version=1.2.2, and the binary was byte-identical before and after (mtime 11:02:31, size 180089984). Gate C: CLEAR, read from the candidate binary's own `agy changelog`; the web page has neither 1.2.1 nor 1.2.2. Range covers both, since the updater went straight from 1.2.0 to 1.2.2. --help and models --help are byte-identical 1.2.0 -> 1.2.2, diffed against the captures stored with the previous record. All nine flags build_argv emits are present. Everything on our surface is a fix. Transient 502/503/504/429 and mid-stream interruptions now retry in-process while PRESERVING completed tool call outputs (a retry that discarded them would replay tool cards), with clear user-facing messages when unrecovered. MCP schema validation relaxes back to honouring open object schemas, undoing the over-strictness 1.1.27 introduced; the MCP test passed. view_file now rejects binary and >100 MB files with a clear error instead of overflowing the context window, which surfaces as a tool ERROR step our card already renders. One item renames tools -- plugin-bundled MCP servers are now namespaced as <plugin>_<server> -- but only for plugin-bundled servers, which we do not use. Ours are provisioned through our own path and the MCP test is unchanged. Correction to the previous record: `View release` anchors on the web page are NOT CLI-only. The Hub uses them too, and Hub versions like 1.21.9 / 1.22.2 match a `View release 1\.[0-9]+\.[0-9]+` pattern. The only reliable web check is an exact match on the candidate, closing quote included. Record: ~/aion/protocols/samples/antigravity-cli/1.2.2/
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.
Moves
VERIFIED_AGY_VERSIONfrom 1.2.0 to 1.2.2. The updater went straight from 1.2.0 to 1.2.2, so the triage covers both 1.2.1 and 1.2.2.Gate B ran against a manifest download whose sha512 was checked before it executed; the log reports only
version=1.2.2, and the binary was byte-identical before and after (mtime=11:02:31 size=180089984). The machine's own agy stayed on 1.1.18.Gate C was read from the candidate binary — the web changelog has neither 1.2.1 nor 1.2.2 (exact match, 0 hits each).
Flag surface: unchanged
--helpandmodels --helpare byte-identical 1.2.0 → 1.2.2, diffed against the captures stored with the previous record. All nine flagsbuild_argvemits are present.Everything on our surface is a fix
502/503/504/per-minute429and mid-stream interruptions now retry in-process with backoff while preserving completed tool call outputs, with clear user-facing messages when unrecovered (1.2.1). Directly our path — and the preserved outputs matter, because a retry that discarded them would replay tool cards.{"type":"object"},additionalProperties: true) instead of rejecting undeclared arguments (1.2.1). This undoes the over-strictness 1.1.27 introduced. Gate B drives an MCP tool call and it passed.view_filenow rejects non-UTF-8 binary files and files over 100 MB with a clear error instead of overflowing the context window (1.2.2). That surfaces as a toolERRORstep, which our card already renders.--new-project+--add-dir, so that one cannot reach us).One item renames tools — plugin-bundled MCP servers are now namespaced
<plugin>_<server>— but only for plugin-bundled servers, which we do not use. Ours are provisioned through our own path, and the MCP test is unchanged.Not our surface
excludeDefaultComponentsin Markdown agent frontmatter (we never pass--agent), the--continuefallback (we resume by explicit--conversation <id>), deprecatedunsandboxedpermission-rule warnings (we pass--dangerously-skip-permissions, not rules), inline-mode flashes, Remote Control companion UI,--sandboxstatus line, image zoom keys,/resumecache speed, a step-cache goroutine leak, deleted-conversation SQLite recreation, artifact-review path matching.A correction to the previous record
View releaseanchors on the web page are not CLI-only — the Hub uses them too, and Hub versions such as1.21.9/1.22.2match aView release 1\.[0-9]+\.[0-9]+pattern. The only reliable web check is an exact match on the candidate, closing quote included:Tests
Pinned assertions moved with the constant, including the literal verified-release case and the
1.1.29older-than-1.2.xboundary that guards component-wise comparison.cargo test -p aionui-session --lib cli_version: 14/14. Clippy clean, fmt clean.Record:
~/aion/protocols/samples/antigravity-cli/1.2.2/Constants and record only — no source change.