Repository navigation
chore(deps): upgrade native napi build and derive dependencies - #1302
mldangelo-oai merged 4 commits into
Conversation
Bumps [napi-derive](https://github.com/napi-rs/napi-rs) from 3.6.3 to 3.6.9. - [Release notes](https://github.com/napi-rs/napi-rs/releases) - [Commits](napi-rs/napi-rs@napi-derive-v3.6.3...napi-derive-v3.6.9) --- updated-dependencies: - dependency-name: napi-derive dependency-version: 3.6.9 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [napi-build](https://github.com/napi-rs/napi-rs) from 2.4.1 to 2.5.0. - [Release notes](https://github.com/napi-rs/napi-rs/releases) - [Commits](napi-rs/napi-rs@napi-build-v2.4.1...napi-build-v2.5.0) --- updated-dependencies: - dependency-name: napi-build dependency-version: 2.5.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
0c55672 to
43da2e6
Compare
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
mldangelo-oai
left a comment
There was a problem hiding this comment.
Reviewed both native dependency upgrades together, including the compatible derive/backend lock and all version-specific license updates. The combined branch passes locked native compilation, linting, notice generation, behavioral proofs, and portable source checks. Both reviewed changes have clean independent review and verification results. No blocking findings at 43da2e6; merge after required CI passes.
Summary
Upgrade the native bindings'
napi-builddependency from 2.4.1 to 2.5.0 andnapi-derivefrom 3.6.3 to 3.6.9, with compatible macro dependencies and complete license packaging.Changes
convert_case0.12.0.napi-derive-backendto 6.1.4 so it uses the samesynmajor version asnapi-derive3.6.9. Version 6.1.5 caused compiler type mismatches betweensyn2 and 3.Testing
plugins/codex-security.pnpm --dir sdk/typescript run build:cipassed.cargo fmt --checkandcargo clippy --locked -- -D warningspassed.PATH.Risk and rollout
No public CLI or runtime settings change. The lockfile preserves the compatible macro/backend pairing, and license overrides remain restricted to reviewed package versions. Supersedes #1301 by including its complete dependency update and fixes. Merge after the cross-platform native and package CI checks pass; close #1301 after this combined update lands.
Public disclosure review
Merge validation
Local validation and reviews passed. Administrator merge is used while GitHub Actions is degraded; hosted checks have not completed.