Skip to content

fix(credentials): align priority tie-break with CPA - #389

Merged
Willxup merged 1 commit into
mainfrom
worktree/auth-file-priority-order
Jul 30, 2026
Merged

fix(credentials): align priority tie-break with CPA#389
Willxup merged 1 commit into
mainfrom
worktree/auth-file-priority-order

Conversation

@Willxup

@Willxup Willxup commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • Keep numeric credential priority as the primary descending sort.
  • Sort equal-priority Auth Files by the case-insensitive CPA file name so Keeper matches CPAMC ordering.
  • Add regression coverage for same-email Codex credentials with account-hash file names.

Validation

  • go test ./internal/repository/... -count=1
  • go test ./cmd/... ./internal/...
  • git diff --check

Fixes #356

Sort auth files with equal numeric priority by the CPA file name instead of the display name so Keeper matches CPAMC ordering.

Refs: #356
@Willxup
Willxup merged commit 7aa8595 into main Jul 30, 2026
4 checks passed
@Willxup
Willxup deleted the worktree/auth-file-priority-order branch July 30, 2026 11:24
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.

[Bug] Wrong Priority order

1 participant