Skip to content

Fix duplicate Claude all-model quota row#2189

Merged
steipete merged 1 commit into
mainfrom
codex/fix-claude-all-models-scope
Jul 15, 2026
Merged

Fix duplicate Claude all-model quota row#2189
steipete merged 1 commit into
mainfrom
codex/fix-claude-all-models-scope

Conversation

@steipete

Copy link
Copy Markdown
Owner

Summary

  • suppress generic Anthropic All models scoped entries from the model-only quota rows
  • keep real model-scoped windows such as Fable only
  • document the mapping and add a regression fixture for both all-model representations

Fixes the duplicate All models only row reported in https://x.com/cryptodev9/status/2077157896341618799.

Proof

  • swift test --filter ClaudeScopedWeeklyLimitMapperTests — 4 passed
  • make check — SwiftFormat clean; SwiftLint zero violations
  • make test — 643 selections; 54/54 groups passed; zero retries/failures/timeouts
  • Codex autoreview — clean; no accepted/actionable findings

Caveat

No live Claude account probe. Repository policy requires explicit approval for real provider/account tests; regression proof uses parser fixtures.

@steipete
steipete merged commit 222d62c into main Jul 15, 2026
8 checks passed
@steipete
steipete deleted the codex/fix-claude-all-models-scope branch July 15, 2026 09:06
@steipete

Copy link
Copy Markdown
Owner Author

Landed as 222d62c77a00191e59690e59834f402772d9bd4d.

Proof:

  • swift test --filter ClaudeScopedWeeklyLimitMapperTests — 4 passed
  • make check — SwiftFormat clean; SwiftLint zero violations
  • make test — 643 selections; 54/54 groups passed; zero retries, failures, or timeouts
  • Codex autoreview (--mode local) — clean; no accepted/actionable findings
  • Exact-head CI for 87d0f57af115d6aa6dac47e391946891b10c545d — all checks green: lint, both Linux builds, both macOS test shards, aggregate gate, GitGuardian

Caveat: no live Claude account probe. Repository policy requires explicit approval for real provider/account tests; regression proof used parser fixtures.

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