chore: v3.0.0 release prep (exports fix + release.sh hardening)#3
Conversation
…ersion bump plugin-utils/view-builders exports gain a 'default' condition (fixes ERR_PACKAGE_PATH_NOT_EXPORTED under CJS/tsx consumers); release.sh npm-version gains --allow-same-version so the already-bumped plugin-types@3.0.0 doesn't abort the release.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d01166a047
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- Declare engines floor (^20.19.0 || ^22.12.0 || >=23.0.0) on plugin-utils + view-builders instead of shipping a dual CJS build; README documents the ESM-only position
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2d04ae90a5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- Remove engines gate from both published packages (install-time field can't express a CJS-path-only floor); README documents the require(esm) limitation as docs-only
Pre-publish fixes for the SDK 3.0.0 npm release: adds the
defaultexport condition to plugin-utils + view-builders (fixesERR_PACKAGE_PATH_NOT_EXPORTEDfor CJS/tsx consumers — the bug breaking 4/7 ytdlp test files), and makes release.sh's version bump idempotent (--allow-same-version) since plugin-types is already at 3.0.0. Build + tests green; pack dry-runs verified. After merge:npm loginthen./release.sh 3.0.0publishes all three packages.