Skip to content

Conversation

@pakrym-oai
Copy link
Collaborator

@pakrym-oai pakrym-oai commented Feb 11, 2026

Do not bundle all binaries inside the SDK as it makes the package huge. Instead depend on openai/codex

Update build_npm_package to unconditionally set CODEX_NPM_NAME dependency
to the staged version and simplify stage_sources interface. Clarify README
to note vendor/ is hydrated for each package.
Copy link
Collaborator

@bolinfest bolinfest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are there any changes required to the GitHub workflows? For example, I assume we need to be extra sure that @openai/codex is published before @openai/codex-sdk?

Is there any code that can be deleted from the release workflow?

try {
const codexPackageJsonPath = moduleRequire.resolve(`${CODEX_NPM_NAME}/package.json`);
const codexRequire = createRequire(codexPackageJsonPath);
const platformPackageJsonPath = codexRequire.resolve(`${platformPackage}/package.json`);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

neat?

@pakrym-oai pakrym-oai merged commit 703fb38 into main Feb 12, 2026
32 checks passed
@pakrym-oai pakrym-oai deleted the pakrym/make-codex-sdk-depend-on-openai-codex branch February 12, 2026 00:20
@github-actions github-actions bot locked and limited conversation to collaborators Feb 12, 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.

2 participants