feat(provider): complete the custom API provider feature on dev (#68–#73) - #75
Merged
Conversation
Completes the custom API provider feature. #67 (profiles store + CLI) and #72 (Cloud Agent environment) are already on dev; this adds the remaining pieces as a single clean commit on top of dev so the repo's rebase-and-merge can apply it without conflicts: - launch <provider>: codex -c overrides + env-injected API key (never argv, never ~/.codex/auth.json) - TUI Accounts/Providers tabs + add/remove provider wizard - per-provider Codex overrides: --set KEY=VALUE, --reasoning, --no-web-search, and matching TUI wizard steps (values passed to Codex verbatim) - provider documentation (docs/wiki/Providers.md, etc.) The original per-PR history is preserved in the already-merged #68-#73. Full suite: 1019 passed. Co-authored-by: xJoker <xjoker@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Lands the remaining parts of the custom API provider feature onto
dev.devalready contains #67 (provider data model + storage + CLI) and #72 (Cloud Agent environment); this PR adds #68–#73, which were previously merged only into their in-stack parent branches and never reacheddev.Why this replaces #74
The repository merges via Rebase and merge. The earlier PR #74 carried two merge commits plus #67's original commit (already on
devunder a different SHA), so it could not be rebased ("cannot be rebased due to conflicts").This PR is a single clean commit on top of the real
dev(devis its ancestor, fully linear), so rebase-and-merge applies without conflicts. It supersedes #74 — #74 can be closed once this merges.What it adds (#68–#73)
launch <provider>: applies the provider viacodex -coverrides and injects the API key through the child-process env — never on argv, never touching~/.codex/auth.json.docs/wiki/Providers.md, etc.).--set KEY=VALUE, convenience--reasoning/--no-web-search, and two extra TUI wizard steps. Values pass through to Codex verbatim — Codex, not codex-switch, decides which keys/values are valid.The original per-PR commit history is preserved in the already-merged #68–#73; it is squashed here only so the change can be rebase-merged cleanly.
Validation
cargo fmt --checkandcargo clippy --all-targets --all-features -- -D warningsare green.cargo test --all: 1019 passed / 0 failed, no regressions.--reasoning, the--no-web-search+--reasoningcombo, and a tool operation that actually created a file — all exited 0.