Skip to content

[0.64.1] Add preferred display currency and TRY support - #609

Draft
Finesssee wants to merge 4 commits into
mainfrom
codex/port-0.64.1-preferred-currency
Draft

Finesssee wants to merge 4 commits into
mainfrom
codex/port-0.64.1-preferred-currency

Conversation

@Finesssee

@Finesssee Finesssee commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

Port the v0.64.1 preferred display currency behavior, including TRY. A persisted preference controls presentation in Usage & Spend, tray labels, FloatBar, provider cost views, Claude Swap, and share PNGs. Conversion uses cached live rates with bundled fallback rates; copied and saved JSON exports retain source values.

The CI follow-up corrects Rust formatting and the Tauri async command return type. It also preserves non-currency unit labels such as Credits when conversion is unavailable, as required by an existing regression test.

Validation:

  • cargo fmt --all --check and git diff --check: passed.
  • cargo clippy --workspace --all-targets -- -D warnings: passed on Windows.
  • cargo test --workspace currency -- --test-threads=4: 14 passed across backend and Tauri.
  • Focused Vitest checks for currency, usageSpendSharing, GeneralTab, and ClaudeSwapAccountsSection: 40 passed.
  • pnpm --dir apps/desktop-tauri build: locale check, TypeScript, and Vite build passed.
  • Independent thermo review of the two corrective changes: approved.
  • Merged the shared CI test fixes from Stabilize tray layout and Claude login tests in CI #607 into this branch; the full hosted check is running for the updated head.

This remains a draft pending a fresh Windows desktop build and native UI proof of the complete feature.

@coderabbitai

coderabbitai Bot commented Sep 23, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

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.

❤️ Share

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

@Finesssee

Copy link
Copy Markdown
Collaborator Author

CI and review follow-up

Fixed the rustfmt failure, then the next hosted failure in the Tauri command macro: async commands borrowing State must return Result. The command now returns Ok(CurrencyRatesSnapshot), preserving the successful invoke payload.

The focused frontend run also caught an existing assertion failure (4.25 Credits was rendered as 4.25 CREDITS). The formatter now preserves non-currency labels while normalizing currency codes.

Verification: workspace clippy with warnings denied passed; 14 Rust currency tests and 40 focused frontend tests passed; the frontend production build and formatting checks passed. The two corrective changes received an independent thermo review with no blocker. These checks do not substitute for the complete feature's pending native Windows UI proof, so the PR remains draft.

The branch also contains the merged #607 CI test corrections. The full CircleCI gate must pass on the updated head.

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.

1 participant