Skip to content

[codex] Add first-class Sora video provider#255

Merged
calesthio merged 2 commits into
mainfrom
codex/sora-dotenv-discovery
Jul 5, 2026
Merged

[codex] Add first-class Sora video provider#255
calesthio merged 2 commits into
mainfrom
codex/sora-dotenv-discovery

Conversation

@calesthio

Copy link
Copy Markdown
Owner

Summary

  • Add sora_video as a first-class OpenMontage video_generation provider for OpenAI Sora.
  • Route Sora availability through the normal BaseTool / registry dotenv path, so OPENAI_API_KEY from repo .env is discoverable during preflight.
  • Add regression coverage for registry discovery and repo .env loading from a clean process environment.

Root Cause

Sora was only available through the standalone Codex skill CLI. That CLI checks the already-running process environment and does not load OpenMontage's repo .env, while OpenMontage tools and registry do load .env via BaseTool / ToolRegistry.

Validation

  • python -m pytest tests\tools\test_sora_video.py -q
  • python -m pytest tests\tools\test_sora_video.py tests\contracts\test_phase1_contracts.py tests\contracts\test_phase2_contracts.py tests\tools\test_hyperframes_compose.py::test_provider_menu_summary_returns_expected_shape -q
  • git diff --check
  • python -m py_compile tools\video\sora_video.py tests\tools\test_sora_video.py

@calesthio calesthio marked this pull request as ready for review July 1, 2026 17:40
@calesthio calesthio force-pushed the codex/sora-dotenv-discovery branch from 35d9550 to b3ff734 Compare July 1, 2026 17:51
@calesthio calesthio merged commit 97c9ca1 into main Jul 5, 2026
1 check passed
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