[WIP] chore: client extraction

[jif-oai]

No description provided.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[jif-oai]

Ideally I would like to drop this in the future

[jif-oai]

todo jif

[jif-oai]

@codex fix ALL the tests and give a PR where all the tests succeeds

[chatgpt-codex-connector]

Summary

• Reworked ResponsesApiClient to loop until it can send a request with fresh auth, emit the initial rate‑limit snapshot, and translate HTTP failures (usage/quota, 401s, etc.) into structured error variants, complete with helper logic and regression tests. These new variants are exposed via codex-core so the session loop can react immediately. codex-rs/api-client/src/responses.rsL48-L218 codex-rs/api-client/src/error.rsL5-L49 codex-rs/core/src/client.rsL309-L344
• Rate-limit parsing now keeps windows even when usage is 0% and adds coverage to prevent regressions, ensuring the CLI still surfaces quota telemetry from header-only responses. codex-rs/api-client/src/client/rate_limits.rsL5-L79
• The Responses payload builder still patches Azure inputs with stored item IDs and now has a unit test to guard the behavior when store is enabled. codex-rs/core/src/wire_payload.rsL67-L145

Testing

• ✅ just fmt
• ✅ just fix -p codex-api-client
• ✅ just fix -p codex-core
• ✅ cargo test -p codex-api-client
• ⚠️ cargo test -p codex-core (fails in exec::tests::kill_child_process_group_kills_grandchildren_on_timeout because the sandbox cannot observe the grandchild exit)

View task →