fix: label Codex weekly usage window as "Week" instead of "Day" by Sid-Qin · Pull Request #26267 · openclaw/openclaw

Sid-Qin · 2026-02-25T07:47:55Z

Summary

Problem: The secondary usage window label logic treated any window >= 24h as "Day". Codex plans with weekly quotas (604800s / 168h) were mislabeled as "Day" even though the reset timer showed "resets 2d 4h".
Why it matters: Users saw conflicting information — a "Day" label with a multi-day reset timer, causing confusion about their actual quota window.
What changed: Added a >= 168h check that labels weekly windows as "Week". The 24h→"Day" and shorter→hour-based labels remain unchanged.
What did NOT change: Primary window labeling, usage percentage calculation, and reset time formatting are unaffected.

Change Type (select all)

Bug fix

Scope (select all touched areas)

UI / DX

Linked Issue/PR

Closes [Bug]: CLI usage labels: Codex second window appears weekly but labeled “Day” #25812

User-visible / Behavior Changes

openclaw status --usage now shows Week: 90% left · resets 2d 4h instead of Day: 90% left · resets 2d 4h for Codex weekly quotas.

Security Impact (required)

New permissions/capabilities? No
Secrets/tokens handling changed? No
New/changed network calls? No
Command/tool execution surface changed? No
Data access scope changed? No

Repro + Verification

Environment

Provider: OpenAI Codex (Plus plan with weekly quota)

Steps

Run openclaw status --usage
Check the secondary window label for Codex

Expected

Week: 90% left · resets 2d 4h

Actual (before fix)

Day: 90% left · resets 2d 4h

Evidence

Failing test/log before + passing after
New test "labels weekly secondary window as Week" with 604800s window

Human Verification (required)

Verified scenarios: Weekly (604800s → "Week"), daily (86400s → "Day"), hourly (10800s → "3h")
Edge cases checked: Boundary at exactly 168h
What you did not verify: Live Codex API response format

Compatibility / Migration

Backward compatible? Yes
Config/env changes? No
Migration needed? No

Failure Recovery (if this breaks)

How to disable/revert this change quickly: Revert the label ternary to windowHours >= 24 ? "Day" : ...
Files/config to restore: src/infra/provider-usage.fetch.codex.ts
Known bad symptoms: Incorrect label text in usage display

Risks and Mitigations

None

Greptile Summary

Fixes a UI labeling bug where weekly Codex quota windows (604,800s / 168h) were incorrectly displayed as "Day" instead of "Week". The secondary window label logic now checks for >= 168h before checking for >= 24h, ensuring weekly windows get the correct label while maintaining backward compatibility with daily and hourly windows.

Updated ternary logic in src/infra/provider-usage.fetch.codex.ts:68 to prioritize weekly check
Added test coverage for the weekly window case (604,800 seconds)
No behavior changes to usage percentage calculation or reset time formatting

Confidence Score: 5/5

This PR is safe to merge with no risk
Simple, well-tested fix that only affects display labeling logic. The change adds a single condition to an existing ternary chain in the correct precedence order. New test validates the fix, and the logic is straightforward with no side effects or behavioral changes beyond the intended label correction.
No files require special attention

_{Last reviewed commit: 1d5e25b}

The secondary window label logic treated any window >= 24h as "Day", but Codex plans can have a weekly (604800s / 168h) quota window. The reset timer showed "resets 2d 4h" while the label said "Day", which was confusing. Now windows >= 168h are labeled "Week", >= 24h remain "Day", and shorter windows show the hour count. Closes openclaw#25812 Co-authored-by: Cursor <cursoragent@cursor.com>

* Changelog: add LanceDB custom baseUrl + dimensions entry (#17874) * Changelog: add Ollama autodiscovery hardening entry (#29201) * Changelog: add Ollama context-window unification entry (#29205) * Changelog: add compaction audit injection removal entry (#28507) * Changelog: add browser url alias entry (#29260) * Changelog: add codex weekly usage label entry (#26267)

…claw#26267) The secondary window label logic treated any window >= 24h as "Day", but Codex plans can have a weekly (604800s / 168h) quota window. The reset timer showed "resets 2d 4h" while the label said "Day", which was confusing. Now windows >= 168h are labeled "Week", >= 24h remain "Day", and shorter windows show the hour count. Closes openclaw#25812 Co-authored-by: Cursor <cursoragent@cursor.com>

* Changelog: add LanceDB custom baseUrl + dimensions entry (openclaw#17874) * Changelog: add Ollama autodiscovery hardening entry (openclaw#29201) * Changelog: add Ollama context-window unification entry (openclaw#29205) * Changelog: add compaction audit injection removal entry (openclaw#28507) * Changelog: add browser url alias entry (openclaw#29260) * Changelog: add codex weekly usage label entry (openclaw#26267)

…claw#26267) The secondary window label logic treated any window >= 24h as "Day", but Codex plans can have a weekly (604800s / 168h) quota window. The reset timer showed "resets 2d 4h" while the label said "Day", which was confusing. Now windows >= 168h are labeled "Week", >= 24h remain "Day", and shorter windows show the hour count. Closes openclaw#25812 Co-authored-by: Cursor <cursoragent@cursor.com>

* Changelog: add LanceDB custom baseUrl + dimensions entry (openclaw#17874) * Changelog: add Ollama autodiscovery hardening entry (openclaw#29201) * Changelog: add Ollama context-window unification entry (openclaw#29205) * Changelog: add compaction audit injection removal entry (openclaw#28507) * Changelog: add browser url alias entry (openclaw#29260) * Changelog: add codex weekly usage label entry (openclaw#26267)

…claw#26267) The secondary window label logic treated any window >= 24h as "Day", but Codex plans can have a weekly (604800s / 168h) quota window. The reset timer showed "resets 2d 4h" while the label said "Day", which was confusing. Now windows >= 168h are labeled "Week", >= 24h remain "Day", and shorter windows show the hour count. Closes openclaw#25812 Co-authored-by: Cursor <cursoragent@cursor.com>

* Changelog: add LanceDB custom baseUrl + dimensions entry (openclaw#17874) * Changelog: add Ollama autodiscovery hardening entry (openclaw#29201) * Changelog: add Ollama context-window unification entry (openclaw#29205) * Changelog: add compaction audit injection removal entry (openclaw#28507) * Changelog: add browser url alias entry (openclaw#29260) * Changelog: add codex weekly usage label entry (openclaw#26267)

…claw#26267) The secondary window label logic treated any window >= 24h as "Day", but Codex plans can have a weekly (604800s / 168h) quota window. The reset timer showed "resets 2d 4h" while the label said "Day", which was confusing. Now windows >= 168h are labeled "Week", >= 24h remain "Day", and shorter windows show the hour count. Closes openclaw#25812 Co-authored-by: Cursor <cursoragent@cursor.com>

* Changelog: add LanceDB custom baseUrl + dimensions entry (openclaw#17874) * Changelog: add Ollama autodiscovery hardening entry (openclaw#29201) * Changelog: add Ollama context-window unification entry (openclaw#29205) * Changelog: add compaction audit injection removal entry (openclaw#28507) * Changelog: add browser url alias entry (openclaw#29260) * Changelog: add codex weekly usage label entry (openclaw#26267)

…claw#26267) The secondary window label logic treated any window >= 24h as "Day", but Codex plans can have a weekly (604800s / 168h) quota window. The reset timer showed "resets 2d 4h" while the label said "Day", which was confusing. Now windows >= 168h are labeled "Week", >= 24h remain "Day", and shorter windows show the hour count. Closes openclaw#25812 Co-authored-by: Cursor <cursoragent@cursor.com> (cherry picked from commit 37d32d6)

* Changelog: add LanceDB custom baseUrl + dimensions entry (openclaw#17874) * Changelog: add Ollama autodiscovery hardening entry (openclaw#29201) * Changelog: add Ollama context-window unification entry (openclaw#29205) * Changelog: add compaction audit injection removal entry (openclaw#28507) * Changelog: add browser url alias entry (openclaw#29260) * Changelog: add codex weekly usage label entry (openclaw#26267) (cherry picked from commit 20263d0)

…claw#26267) The secondary window label logic treated any window >= 24h as "Day", but Codex plans can have a weekly (604800s / 168h) quota window. The reset timer showed "resets 2d 4h" while the label said "Day", which was confusing. Now windows >= 168h are labeled "Week", >= 24h remain "Day", and shorter windows show the hour count. Closes openclaw#25812 Co-authored-by: Cursor <cursoragent@cursor.com> (cherry picked from commit 37d32d6)

* Changelog: add LanceDB custom baseUrl + dimensions entry (openclaw#17874) * Changelog: add Ollama autodiscovery hardening entry (openclaw#29201) * Changelog: add Ollama context-window unification entry (openclaw#29205) * Changelog: add compaction audit injection removal entry (openclaw#28507) * Changelog: add browser url alias entry (openclaw#29260) * Changelog: add codex weekly usage label entry (openclaw#26267) (cherry picked from commit 20263d0)

…claw#26267) The secondary window label logic treated any window >= 24h as "Day", but Codex plans can have a weekly (604800s / 168h) quota window. The reset timer showed "resets 2d 4h" while the label said "Day", which was confusing. Now windows >= 168h are labeled "Week", >= 24h remain "Day", and shorter windows show the hour count. Closes openclaw#25812 Co-authored-by: Cursor <cursoragent@cursor.com> (cherry picked from commit 37d32d6)

* Changelog: add LanceDB custom baseUrl + dimensions entry (openclaw#17874) * Changelog: add Ollama autodiscovery hardening entry (openclaw#29201) * Changelog: add Ollama context-window unification entry (openclaw#29205) * Changelog: add compaction audit injection removal entry (openclaw#28507) * Changelog: add browser url alias entry (openclaw#29260) * Changelog: add codex weekly usage label entry (openclaw#26267) (cherry picked from commit 20263d0)

…claw#26267) The secondary window label logic treated any window >= 24h as "Day", but Codex plans can have a weekly (604800s / 168h) quota window. The reset timer showed "resets 2d 4h" while the label said "Day", which was confusing. Now windows >= 168h are labeled "Week", >= 24h remain "Day", and shorter windows show the hour count. Closes openclaw#25812 Co-authored-by: Cursor <cursoragent@cursor.com>

* Changelog: add LanceDB custom baseUrl + dimensions entry (openclaw#17874) * Changelog: add Ollama autodiscovery hardening entry (openclaw#29201) * Changelog: add Ollama context-window unification entry (openclaw#29205) * Changelog: add compaction audit injection removal entry (openclaw#28507) * Changelog: add browser url alias entry (openclaw#29260) * Changelog: add codex weekly usage label entry (openclaw#26267)

…claw#26267) The secondary window label logic treated any window >= 24h as "Day", but Codex plans can have a weekly (604800s / 168h) quota window. The reset timer showed "resets 2d 4h" while the label said "Day", which was confusing. Now windows >= 168h are labeled "Week", >= 24h remain "Day", and shorter windows show the hour count. Closes openclaw#25812 Co-authored-by: Cursor <cursoragent@cursor.com> (cherry picked from commit e16d051) # Conflicts: # src/infra/provider-usage.fetch.codex.test.ts