Skip to content

fix: align remove usage refresh with switch#61

Merged
Loongphy merged 1 commit intomainfrom
codex/align-remove-switch-usage
Apr 17, 2026
Merged

fix: align remove usage refresh with switch#61
Loongphy merged 1 commit intomainfrom
codex/align-remove-switch-usage

Conversation

@Loongphy
Copy link
Copy Markdown
Owner

@Loongphy Loongphy commented Apr 17, 2026

Summary

  • align the interactive remove flow with switch so the no-query picker refreshes latest API usage before rendering account choices
  • pass per-account usage overrides through the remove selectors so HTTP status and MissingAuth overlays render in both usage columns just like list and switch
  • add q quit handling to the remove TUI, and update docs/tests to describe and cover the new behavior

Validation

  • zig test src/main.zig -lc
  • zig build run -- list
  • zig test src/main.zig -lc --test-filter "Given remove query with one match when running remove then it deletes immediately and prints a summary"
  • zig test src/main.zig -lc --test-filter "Given non-tty stdin when running interactive remove then it falls back to the numbered selector"

Risks

  • the main behavior change is that remove without a query now does the same foreground Node/API preflight as the interactive switch picker when api.usage = true
  • query-driven remove and remove --all intentionally stay local-only before deletion, so reviewers should verify that scope split matches expectations
  • the remove picker now shares more of the switch-row rendering path, so extra attention on selector UI behavior and status overlays is useful

PR Loop Status

  • latest push intent: send the remove/switch alignment changes for CI and review
  • latest CI status: green across CI and preview-package checks
  • review comments addressed in the latest push: none yet
  • open comments intentionally left unresolved and why: none

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 17, 2026

Open in StackBlitz

@loongphy/codex-auth-darwin-arm64

npx https://pkg.pr.new/@loongphy/codex-auth-darwin-arm64@61

@loongphy/codex-auth-darwin-x64

npx https://pkg.pr.new/@loongphy/codex-auth-darwin-x64@61

@loongphy/codex-auth-linux-arm64

npx https://pkg.pr.new/@loongphy/codex-auth-linux-arm64@61

@loongphy/codex-auth-linux-x64

npx https://pkg.pr.new/@loongphy/codex-auth-linux-x64@61

@loongphy/codex-auth-win32-arm64

npx https://pkg.pr.new/@loongphy/codex-auth-win32-arm64@61

@loongphy/codex-auth-win32-x64

npx https://pkg.pr.new/@loongphy/codex-auth-win32-x64@61

@loongphy/codex-auth

npx https://pkg.pr.new/@loongphy/codex-auth@61

commit: 3ae8816

@Loongphy Loongphy marked this pull request as ready for review April 17, 2026 12:48
@Loongphy Loongphy merged commit def5bac into main Apr 17, 2026
11 checks passed
@Loongphy Loongphy deleted the codex/align-remove-switch-usage branch April 17, 2026 12:48
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