Add Codex OAuth LLM provider#8
Closed
ethanj wants to merge 8 commits into
Closed
Conversation
Contributor
Author
|
Closed as superseded by the internal-first flow. The reviewed and green internal PR atomicstrata/atomicmemory-internal#3 was merged to internal main and then published to public main per the documented workflow. |
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
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
LLM_PROVIDER=codexto core provider config and provider construction.codex login(CODEX_AUTH_PATH,CODEX_HOME/auth.json, or~/.codex/auth.json) and posts to the Codex backend directly.@openai/codexCLI install from the core Docker image; Docker QA now mounts only a copied Codex auth home.Verification
pnpm --filter @atomicmemory/core test -- src/services/__tests__/codex-llm.test.ts src/services/__tests__/llm-providers.test.ts src/__tests__/config-env.test.ts --reporter verbosepnpm --filter @atomicmemory/core typecheckpnpm --filter @atomicmemory/core buildpnpm --filter @atomicmemory/core code-healthgit diff --checkbash ci/run-required.sh --tests core-extraction --allow-partial --host-tools none --llm-provider codexpassed with 1 passed row, 0 failed rows, 0 skipped rows.Notes
This mode is intended for local personal development. It consumes the logged-in Codex account limits and is not intended for hosted or team deployments. Claude Code was not changed; both Hindsight and AtomicMemory use the Claude Agent SDK path for Claude account auth.