fix(web): only show Codex Fast badge for fast tier#1007
Conversation
There was a problem hiding this comment.
Findings
No issues found in the added or modified lines.
Summary
Review mode: initial
Reviewed the full PR diff for the StatusBar Fast badge change and the replacement TSX test coverage. The change now gates the badge on the explicit stored Codex service tier, consistent with the existing fast/standard schema and the composer service-tier flow. Residual risk: I did not execute tests in this automation review.
Testing
Not run (automation)
HAPI Bot
# Conflicts: # web/src/components/AssistantChat/StatusBar.test.tsx
There was a problem hiding this comment.
Findings
No issues found in the added or modified lines.
Summary
Review mode: follow-up after new commits
Reviewed the full current PR diff for the StatusBar Fast badge change and the TSX test replacement. The badge now follows the same explicit serviceTier === 'fast' semantics used by SessionHeader, and the added tests cover unset tier, mini-model heuristic removal, explicit fast, and explicit standard. Residual risk: I did not execute tests in this automation review.
Testing
Not run (automation)
HAPI Bot
Closes #1004
Summary
Why
Codex
/fastis a distinct service tier. Low reasoning and mini model variants must not be presented as Fast mode.Validation
upstream/main: 2 expected failures (low reasoning and mini model were mislabeled Fast)bun run --cwd web test -- src/components/AssistantChat/StatusBar.test.tsx— 6 passedtestpassed on the PR headgit diff --check