Skip to content

fix(status-bar): keep Antigravity usage visible - #7878

Closed
bbingz wants to merge 1 commit into
stablyai:mainfrom
bbingz:fix/antigravity-usage-status
Closed

fix(status-bar): keep Antigravity usage visible#7878
bbingz wants to merge 1 commit into
stablyai:mainfrom
bbingz:fix/antigravity-usage-status

Conversation

@bbingz

@bbingz bbingz commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Treat checked Antigravity Usage as the durable visibility signal when the Antigravity agent is available, so the status-bar segment stays visible while usage data is pending.
  • Add antigravity to rate-limit state shape, status-bar defaults/migrations, runtime UI RPC schema, provider icons/display copy, and Appearance status-bar toggles.
  • Split usage status-bar toggle catalog to stay under max-lines and added regression coverage for the checked-toggle vs visible-segment contract.

Fixes #7809

Test Plan

  • pnpm exec vitest run --config config/vitest.config.ts src/renderer/src/components/status-bar/status-bar-provider-visibility.test.ts src/renderer/src/store/slices/rate-limits.test.ts src/shared/rate-limit-types.test.ts src/renderer/src/components/status-bar/status-bar-agent-gating.test.ts src/renderer/src/store/slices/ui.test.ts src/renderer/src/components/settings/appearance-status-bar-search.test.ts src/main/runtime/rpc/methods/client-ui.test.ts src/renderer/src/components/status-bar/usage-error-copy.test.ts src/renderer/src/components/status-bar/tooltip.test.ts src/renderer/src/components/settings/AppearancePane.test.tsx
  • pnpm run typecheck
  • pnpm run lint
  • read-only review: PASS / APPROVED, no concrete findings

Note: local commands emit the existing engine warning because this shell uses Node v26.4.0 while package.json wants Node 24.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This PR adds "antigravity" as a new provider across the codebase. Shared types (ProviderRateLimits, RateLimitState, StatusBarItem, PersistedUIState) and default status bar items were extended. The main-process rate-limit service and RPC UI schema now track antigravity state. Renderer changes include StatusBar rendering, provider icon/tooltip support, visibility/gating logic, usage-empty-state checks, a new usage-toggle search catalog module, and store/UI hydration adding a one-shot default-on migration flag. Localization strings for antigravity were added across en, es, ja, ko, and zh locale files, alongside corresponding test updates.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers summary and testing, but it omits required Screenshots, AI Review Report, Security Audit, and Notes sections. Add the missing template sections and include screenshots/no-visual-change, AI review details, security audit notes, and any platform-specific follow-up.
Docstring Coverage ⚠️ Warning Docstring coverage is 7.69% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the PR’s main change: keeping Antigravity usage visible in the status bar.
Linked Issues check ✅ Passed The PR implements the requested visibility, empty-state, initial rate-limit state, and regression test changes for #7809.
Out of Scope Changes check ✅ Passed The added changes stay aligned with the Antigravity visibility fix and its supporting UI, schema, localization, and test updates.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
src/renderer/src/components/status-bar/StatusBar.tsx (1)

1848-1857: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Duplicate computation of the Antigravity checked+detected condition.

antigravityUsageConfigured (Lines 1848-1850) and showAntigravity (Lines 1880-1883) both independently compute statusBarItems.includes('antigravity') && isStatusBarItemAvailable('antigravity', detectedAgentIds). Since showAntigravity already requires visibleAntigravity !== null, which in turn depends on antigravityUsageConfigured via getVisibleUsageProvider, the second occurrence is redundant and could just reuse the variable.

♻️ Proposed simplification
   const showAntigravity =
     visibleAntigravity !== null &&
-    statusBarItems.includes('antigravity') &&
-    isStatusBarItemAvailable('antigravity', detectedAgentIds)
+    antigravityUsageConfigured

Also applies to: 1880-1883


ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a046aebf-441d-4d7d-ab52-42ed2bf20c27

📥 Commits

Reviewing files that changed from the base of the PR and between 7e1a777 and 444c365.

📒 Files selected for processing (31)
  • src/main/rate-limits/service.ts
  • src/main/runtime/rpc/methods/client-ui.test.ts
  • src/main/runtime/rpc/methods/client-ui.ts
  • src/renderer/src/components/settings/AppearancePane.test.tsx
  • src/renderer/src/components/settings/AppearanceWindowSidebarSection.tsx
  • src/renderer/src/components/settings/appearance-status-bar-search.test.ts
  • src/renderer/src/components/settings/appearance-status-bar-search.ts
  • src/renderer/src/components/settings/appearance-status-bar-usage-toggles.ts
  • src/renderer/src/components/status-bar/StatusBar.tsx
  • src/renderer/src/components/status-bar/status-bar-agent-gating.test.ts
  • src/renderer/src/components/status-bar/status-bar-agent-gating.ts
  • src/renderer/src/components/status-bar/status-bar-provider-visibility.test.ts
  • src/renderer/src/components/status-bar/status-bar-provider-visibility.ts
  • src/renderer/src/components/status-bar/tooltip.test.ts
  • src/renderer/src/components/status-bar/tooltip.tsx
  • src/renderer/src/components/status-bar/usage-error-copy.test.ts
  • src/renderer/src/components/status-bar/usage-error-copy.ts
  • src/renderer/src/i18n/locales/en.json
  • src/renderer/src/i18n/locales/es.json
  • src/renderer/src/i18n/locales/ja.json
  • src/renderer/src/i18n/locales/ko.json
  • src/renderer/src/i18n/locales/zh.json
  • src/renderer/src/store/slices/rate-limits.test.ts
  • src/renderer/src/store/slices/rate-limits.ts
  • src/renderer/src/store/slices/ui.test.ts
  • src/renderer/src/store/slices/ui.ts
  • src/renderer/src/web/web-preload-api.ts
  • src/shared/rate-limit-types.test.ts
  • src/shared/rate-limit-types.ts
  • src/shared/status-bar-defaults.ts
  • src/shared/types.ts

@bbingz

bbingz commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by replacement PR #7996 which combines Antigravity + Grok usage status (with #7901). Closing in favor of that stack.

@bbingz

bbingz commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Closed in favor of #7996 (combined Antigravity + Grok usage status).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Antigravity Usage — status-bar segment disappears while toggle is checked (W28)

2 participants