feat: add Codex Local AI provider#54
Open
fedepalu2 wants to merge 1 commit into
Open
Conversation
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.
Description
Adds a
Codex LocalAI provider that lets OpenReel call a trusted loopback Codex bridge instead of hosted OpenAI/Anthropic APIs for text enhancement, and exposes the same provider choice for image editing/generation workflows.Related Issue
No linked issue.
Type of Change
Changes Made
Codex Localto the settings provider registry without showing it as an API-key service.pnpm codex:bridgedevelopment bridge backed bycodex execfor text enhancement.Testing
Test Plan:
pnpm test)pnpm typecheck)Commands run locally:
npx pnpm install --frozen-lockfilenpx pnpm --filter @openreel/web typechecknpx pnpm --filter @openreel/web test:run -- src/services/codex-local.test.tsnpx pnpm --filter @openreel/web lint(passes with existing repo warnings)npx pnpm --filter @openreel/web build(passes with existing Vite warnings)pnpm codex:bridgehealth check viaGET http://127.0.0.1:14567/healthhttp://127.0.0.1:4173Browsers Tested:
Screenshots/Videos
Preview smoke screenshot captured locally from the built app. Not attached because the first screen is unchanged by this provider integration.
Security Notes
127.0.0.1by default and rejects non-loopback host overrides.spawn()argument arrays instead of shell interpolation, andcodex exec --sandbox read-only --ask-for-approval never.Checklist
console.logstatements and debug codeAdditional Context
This is intentionally scoped as a provider/bridge integration rather than a broader timeline-control agent. The included bridge implements text enhancement; image-capable local bridges can implement the documented
/v1/images/editscontract and OpenReel will consume the result.