chore(cli-version): verify agy against 1.1.23 - #962
Merged
Conversation
Gate B: 7/7, 139.53s, against a manifest download whose sha512 was checked before it ran. The suite log reports only version=1.1.23. Gate C: CLEAR, read from the candidate binary's own `agy changelog`. Both sources were checked and they have swapped again -- the binary covers 1.1.23 while the web page still stops at 1.1.22. Six days ago it was the other way round, which is why both are now checked every run. One item lands on a frame we parse: "Reduced subagent streaming overhead by sending subagent trajectory metadata once per subtrajectory instead of with every step." translate.rs:264 reads `subagent_info` on the ACTIVE frame to name the tool card; `unwrap_or_default()` means a regression here would be quiet rather than loud. Gate B cannot resolve it -- no agy test dispatches a subagent -- so it was probed: the dispatch step still carries the list on both frames. idx=2 state=ACTIVE subagent_info=YES agents=1 idx=2 state=DONE subagent_info=YES agents=1 The change is to trajectory metadata inside the subagent's own stream, not to the field we consume. The probe binary was byte-identical before and after (mtime 07:55:50, size 180065392, ver 1.1.23), so the attribution holds. Separately worth recording: 1.1.23 fixes `agy models` hanging on an inherited, unclosed stdin -- the exact defect probe_models works around by dropping the stdin handle (models.rs:88-92, measured against 1.1.9). Our workaround is now belt-and-braces; it is harmless on older installs so nothing is changed here. Record: ~/aion/protocols/samples/antigravity-cli/1.1.23/
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.1.22 to 1.1.23.Gate B ran against a manifest download whose sha512 was checked before it executed; the log reports only
version=1.1.23. The machine's own agy stayed on 1.1.18.The two sources swapped again
agy changelogfrom the 1.1.23 binarySix days ago it was the other way round (the web page had 1.1.21/1.1.22 while the 1.1.22 binary's bundled copy stopped at 1.1.20). Neither is authoritative, so both are checked every run and the record says which was used.
The REVIEW item, resolved by probing rather than arguing
This lands on a frame we parse.
translate.rs:264readssubagent_infoon the ACTIVE frame to name the subagent tool card, andunwrap_or_default()means a regression would degrade to a generic card rather than fail loudly — quiet, which is the dangerous kind.Gate B cannot resolve it: no agy test in the suite dispatches a subagent. So it was probed directly, and the dispatch step still carries the list on both frames:
The change is to trajectory metadata inside the subagent's own stream, not to the field we consume. Capture kept as
probe-subagent-dispatch.ndjson.Attribution checked — the probe binary was byte-identical before and after (
mtime=07:55:50 size=180065392 ver=1.1.23, matching the values recorded at download). That check exists because an earlier run's agy A/B had to be retracted when a copy self-updated mid-probe.Upstream fixed a hang we already work around
That is the exact defect
probe_modelswas built around —models.rs:88-92drops the stdin handle, citing a measurement against 1.1.9 ("stdin left open = still running after 40 minutes; stdin closed = exits in ~3s"). Our workaround is now belt-and-braces. It is harmless and still correct on older installs, so nothing is changed here; recorded so it can be retired deliberately if the floor ever rises past 1.1.23.Not our surface
Permission-prompt titles (agy's own interactive UI — we pass
--dangerously-skip-permissionsand render from our hook bridge), subagentenable_mcp_toolsdefinitions, Gemini history reconstruction, and Windows BOM parsing of the MCP config file we do not write.Tests
Pinned assertions moved with the constant, including the literal verified-release case.
cargo test -p aionui-session --lib cli_version: 14/14. Clippy clean, fmt clean.Record:
~/aion/protocols/samples/antigravity-cli/1.1.23/Constants and record only — no source change.