Skip to content

ci(rust): synchronize napi dependencies#80

Merged
gautamsirdeshmukh merged 1 commit into
strands-agents:mainfrom
gautamsirdeshmukh:chore/sync-napi-dependencies
Jul 22, 2026
Merged

ci(rust): synchronize napi dependencies#80
gautamsirdeshmukh merged 1 commit into
strands-agents:mainfrom
gautamsirdeshmukh:chore/sync-napi-dependencies

Conversation

@gautamsirdeshmukh

@gautamsirdeshmukh gautamsirdeshmukh commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Description

Dependabot split the tightly coupled napi-rs runtime and derive updates across #76 and #77. Either update alone produces an incompatible runtime/macro combination and breaks the Node.js build.

This change applies both updates together, keeping napi, napi-sys, napi-derive, and napi-derive-backend compatible.

Related Issues

Supersedes and closes #76 and #77.

Documentation PR

N/A

Type of Change

Other: dependency update

Testing

  • npm run build:debug
  • npm test (41 tests passed)
  • npm run typecheck
  • cargo test --workspace --all-targets --locked
  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets --locked

Checklist

  • I have read the CONTRIBUTING document
  • I have reviewed and understand every line of code in this PR, including any generated by AI tools, and I can explain why it works
  • My change is focused and reasonably small; I have split unrelated work into separate PRs
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@gautamsirdeshmukh
gautamsirdeshmukh merged commit e740704 into strands-agents:main Jul 22, 2026
31 checks 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.

3 participants