Skip to content

[codex] Split Python SDK helper logic#22939

Merged
aibrahim-oai merged 3 commits into
mainfrom
dev/sdk-python-helper-split
May 16, 2026
Merged

[codex] Split Python SDK helper logic#22939
aibrahim-oai merged 3 commits into
mainfrom
dev/sdk-python-helper-split

Conversation

@aibrahim-oai
Copy link
Copy Markdown
Collaborator

Summary

  • Move approval-mode mapping into sdk/python/src/openai_codex/_approval_mode.py.
  • Move initialize metadata parsing and normalization into sdk/python/src/openai_codex/_initialize_metadata.py.
  • Keep the public ApprovalMode export stable and retarget direct metadata helper coverage.

Integration coverage

  • Add an app-server harness smoke that exercises sync and async SDK initialization plus thread creation.

Validation

  • Local tests were not run per repo guidance. CI should validate this branch once the PR is online.

@aibrahim-oai aibrahim-oai marked this pull request as ready for review May 16, 2026 02:55
@aibrahim-oai
Copy link
Copy Markdown
Collaborator Author

@codex review

@aibrahim-oai aibrahim-oai merged commit a280248 into main May 16, 2026
45 of 47 checks passed
@aibrahim-oai aibrahim-oai deleted the dev/sdk-python-helper-split branch May 16, 2026 16:47
@github-actions github-actions Bot locked and limited conversation to collaborators May 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant