Greptile Summary

This PR fixes a bug where the synthetic weekly token quota never reported an "exhausted" state, even when usage reached 100%, allowing requests to leak through to the upstream provider instead of being blocked.

• Core fix in synthetic_checker.go: Adds usageRatio >= 1.0 check to weeklyTokenLimitStatus and replaces the hardcoded Ready: true with IsReadyStatus(status), so the weekly limit now correctly blocks traffic when fully consumed.
• Consistency refactoring in claudecode_checker.go and nanogpt_checker.go: Replaces the inline status != \"exhausted\" expression with the shared IsReadyStatus helper, which is functionally equivalent for the reachable statuses in those code paths.
• Test in synthetic_checker_test.go: Corrects the exhausted-state assertions from \"warning\" / Ready: true to \"exhausted\" / Ready: false, directly covering the reported bug.

Confidence Score: 5/5

Safe to merge — the fix is minimal, targeted at a clear gap in exhausted-state detection, and the refactored IsReadyStatus calls are semantically identical to the inline expressions they replace.

The weekly quota exhausted detection bug (Ready: true hardcoded, no usageRatio >= 1.0 check) is correctly addressed. The IsReadyStatus refactoring in claudecode_checker.go and nanogpt_checker.go is functionally equivalent because the only reachable status values in those functions are "available", "warning", and "exhausted" — no edge case is introduced. The test directly exercises the fixed code path with percentRemaining: 0.0.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[weeklyTokenLimitStatus called] --> B{PercentRemaining nil?}
    B -- Yes --> C[status = available\nusageRatio = 0.0]
    B -- No --> D[usageRatio = 1.0 - percentRemaining/100]
    D --> E{usageRatio >= 1.0?}
    E -- Yes --> F[status = exhausted]
    E -- No --> G{usageRatio > WarningThresholdRatio?}
    G -- Yes --> H[status = warning]
    G -- No --> I[status = available]
    F --> J[IsReadyStatus: false]
    H --> K[IsReadyStatus: true]
    I --> L[IsReadyStatus: true]
    C --> L
    J --> M[QuotaLimitStatus returned\nReady: false — blocks upstream]
    K --> N[QuotaLimitStatus returned\nReady: true — allows upstream]
    L --> N

_{Reviews (2): Last reviewed commit: "refactor(quota): use IsReadyStatus helpe..." | Re-trigger Greptile}