Skip to content

fix(usage): batch 2 — security, correctness, and UX fixes - #6

Merged
shidevil merged 1 commit into
mainfrom
feat/usage-command
May 3, 2026
Merged

fix(usage): batch 2 — security, correctness, and UX fixes#6
shidevil merged 1 commit into
mainfrom
feat/usage-command

Conversation

@shidevil

@shidevil shidevil commented May 3, 2026

Copy link
Copy Markdown
Owner
  • Move atomic_write_secret to shared helpers with mode(0o600) at open time (P1)
  • Add temp file cleanup on rename failure
  • Check Amp ok field instead of silently returning empty metrics
  • Persist Codex access token even when no new refresh token returned
  • Fix TUI empty state to distinguish never-fetched from fetched-empty
  • Fix truncate() off-by-one with ellipsis
  • Fix light-mode card width consistency across header/metrics/border
  • Deduplicate copilot hosts.yml parsing into shared helper

- Move atomic_write_secret to shared helpers with mode(0o600) at open time (P1)
- Add temp file cleanup on rename failure
- Check Amp ok field instead of silently returning empty metrics
- Persist Codex access token even when no new refresh token returned
- Fix TUI empty state to distinguish never-fetched from fetched-empty
- Fix truncate() off-by-one with ellipsis
- Fix light-mode card width consistency across header/metrics/border
- Deduplicate copilot hosts.yml parsing into shared helper

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@shidevil
shidevil merged commit 5758e58 into main May 3, 2026
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