fix(ci): repair dev after the 2.65.0 round merges - #5768
Conversation
Move the account-rotation split out of request-log.ts for the 2,000-line limit, wire the claude-cli provider icon, and align the DeepSeek terminal-repair test with destination-based policy.
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configurationConfiguration used: Repository: lidge-jun/opencodex/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (4)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Deterministic PR hygiene checks passed. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Summary
Fixes the three failures on the
devCross-platform CI run 36028905040 (tip9cf3a06889) left by this round's merges:test 2/4,windows 5/9,macos 2/2): fix(usage): split the attempt row on account rotation, for every pool #5752 grewsrc/server/request-log.tsfrom 1,996 to 2,025 lines, over the 2,000-line limit for files without a baseline row (NEW_OVERSIZED). The account-rotation logic from fix(usage): split the attempt row on account rotation, for every pool #5752 (attemptAccountChangedandsealRequestAttemptIdentity) now lives insrc/server/request-log-account-rotation.tswith its full explanation;request-log.tsre-exportssealRequestAttemptIdentity, so callers are unchanged.request-log.tsis 1,983 lines. Behavior is identical.gates): feat(provider): add a Claude Code CLI subscription provider #5712'sclaude-cliprovider had a committedclaude-color.svgbut noPROVIDER_ICON_ALIASESrow; added.windows 4/9):tests/providers/deepseek-inbound-wire.test.tsstill asserted that a custom-named provider at the official DeepSeek endpoint gets no terminal repair. fix(providers): fallback to transport destination for registry model wire defaults #5697 intentionally made an alias at an exact canonical destination take that registry row's policy (the same contract pinned for OpenCode Go), so the assertion now expects{ graceMs: 5000 }.Verification
bun run typecheck: pass.bun run structure:check: pass.bun test tests/providers/deepseek-inbound-wire.test.ts tests/ci-workflows/file-size-ratchet.test.ts tests/usage/key-attribution.test.ts tests/usage/request-log-attempt-identity.test.ts: 63 pass, 0 fail.cd gui && bun test tests/provider-icons.test.ts: 6 pass, 0 fail.devCross-platform CI run dispatched after merge.Checklist