Repository navigation
fix(antigravity): show CLI-only Argon quota without duplicate effort rows - #7245
yuanyuanlove wants to merge 2 commits into
Conversation
Supplement usable family quota with bounded, display-only Argon measurements using the CLI User-Agent. Collapse identical effort-alias readings without combining differing windows or influencing reset-first ranking.
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
⏳ DRAFT
What to do
Review readiness checklist
0/4 boxes ticked. This PR stays in draft until every box above is ticked. |
|
Heads-up from #7283 (fixes #7272), which touches
Whichever lands second should reconcile those points and test the combined tree, including that an untagged |
Reconcile the canonical Argon summary pool with optional CLI diagnostics, preserve effort-window grouping, and cover model-tag-only supplement suppression. Validation: 134 focused tests passed with Bun 1.4.3; typecheck, privacy scan, and structure checks passed. Full backend, test:changed, docs build and visual validation were not run. The PR remains draft.
|
Re-triage at |
Summary
gemini-4-argon. Group only matching normalized usage, reset and remaining window metadata; retain different readings and repeated same-wire windows. Never add or average values, infer backend bucket identity, or merge Argon into the existing Gem summary.modelIddiagnostic discriminator and 256-model-row ceiling. Keep routing/headroom, exhaustion, reset notifications and family counters unchanged; close the reset-first consumer's missing diagnostic-row exclusion.Verification
Bun 1.4.3, repository
scripts/test.tsisolation, synthetic credentials and injected fixed/pinned transports only:bun scripts/test.ts tests/providers/provider-antigravity-model-quota.test.ts tests/providers/provider-antigravity-quota-retry.test.ts ./gui/tests/provider-quota-model-id.test.ts— 50 passed, 0 failed, 175 assertions. Covers summary/fallback supplementation, exact Argon matching, alias grouping, differing/reset-less/invalid resets, nested metadata, repeated-window multiplicity, 256-row bounds, optional failures, primary auth denial, ranking/reset-first/exhaustion/observation isolation and existing GUI projection.bun scripts/test.ts tests/providers/provider-account-quota.test.ts tests/providers/provider-quota.test.ts tests/ci-workflows/file-size-ratchet.test.ts --test-name-pattern 'google-antigravity per-account quota|google-antigravity Fake-IP TUN quota probes without HTTP proxy|paused generic OAuth accounts are never quota-probed|Google Antigravity|file-size ratchet'— 76 passed, 0 failed, 169 filtered, 472 assertions. Covers account separation/currentness/paused rows, fixed destinations, redirect/DNS/Fake-IP policy and size bounds.bun run typecheck,bun run privacy:scan,bun run structure:check,git diff --check— passed on the final patch. The three script checks preceded a clean fast-forward over the GUI-only fix(gui): restore keep-native-v1 qualifier placement #7237 base commit; none of this PR's runtime, test or documentation files changed in that fast-forward.Scope exception / outstanding coverage: an earlier broad four-file backend invocation was stopped when an unrelated Kimi fixture attempted inherited SOCKS egress; it is not a completed or passing result. The full backend suite and import-graph-wide
test:changedwere not rerun; the reviewed fixed-transport allowlist above avoids that path. Full/cross-platform coverage remains for maintainer-started CI. No GUI build was needed because GUI files are unchanged; documentation-site build was not run because its dependencies are absent. This draft does not claim full-suite, docs-build or visual acceptance.Base: upstream
devat050bb444a0002372c08c19e6fcf5961004110868. No merge or deployment is requested by this PR.Checklist
Review readiness checklist
This PR stays in draft until every box below is ticked.