Skip to content

feat: add codex responses provider - #19

Merged
MarsDoge merged 1 commit into
mainfrom
feat/codex-responses-provider
May 14, 2026
Merged

MarsDoge merged 1 commit into
mainfrom
feat/codex-responses-provider

Conversation

@MarsDoge

Copy link
Copy Markdown
Owner

Summary

  • add codex-responses provider using the Responses API with streaming SSE parsing
  • document provider usage and expose it in CLI provider help/listing
  • add static provider test covering payload, endpoint, headers, streaming parser, and provider discovery

Tests

  • bash -n bin/llm-translate lib/stream.sh lib/providers/codex-responses.sh tests/codex-responses-static.sh
  • shellcheck bin/llm-translate lib/stream.sh lib/providers/codex-responses.sh tests/codex-responses-static.sh
  • ./tests/codex-responses-static.sh
  • ./bin/llm-translate --list-providers | grep -qx codex-responses

Secret review

  • Reviewed staged diff for token patterns; only placeholder/test values and environment variable references were present.

@MarsDoge
MarsDoge merged commit 5ec1f62 into main May 14, 2026
1 check passed
@MarsDoge
MarsDoge deleted the feat/codex-responses-provider branch May 14, 2026 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant