Skip to content

Conversation

@vmoroz
Copy link
Member

@vmoroz vmoroz commented Jan 8, 2026

PR #59953 added the new napi_create_object_with_properties function to Node-API.
Unfortunately, we did not catch in the PR review that it uses the old function prefix.
This PR addresses the issue by renaming it to node_api_create_object_with_properties where we use the proper node_api_ prefix.

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/node-api
  • @nodejs/performance

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. node-api Issues and PRs related to the Node-API. labels Jan 8, 2026
@codecov
Copy link

codecov bot commented Jan 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.55%. Comparing base (842448b) to head (dfd1f3c).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #61319      +/-   ##
==========================================
+ Coverage   88.00%   88.55%   +0.54%     
==========================================
  Files         704      704              
  Lines      208739   208739              
  Branches    40196    40287      +91     
==========================================
+ Hits       183706   184841    +1135     
+ Misses      16992    15917    -1075     
+ Partials     8041     7981      -60     
Files with missing lines Coverage Δ
src/js_native_api_v8.cc 76.49% <100.00%> (ø)

... and 104 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@avivkeller
Copy link
Member

This is a breaking change, don't we need to go through the usual deprecation process, even if it's a typo fix (like we did with util.getCallSite())?

@avivkeller avivkeller added the semver-major PRs that contain breaking changes and should be released in the next major version. label Jan 9, 2026
@vmoroz
Copy link
Member Author

vmoroz commented Jan 9, 2026

This is a breaking change, don't we need to go through the usual deprecation process, even if it's a typo fix (like we did with util.getCallSite())?

Good point! I will add the original function back with the deprecated attribute.

@legendecas
Copy link
Member

legendecas commented Jan 9, 2026

This is an experimental API and does not require a full cycle of breaking change or semver-major. This was also landed not long ago, so I'd argue that we should fix it sooner.

@legendecas legendecas removed the semver-major PRs that contain breaking changes and should be released in the next major version. label Jan 9, 2026
@legendecas legendecas moved this from Need Triage to In Progress in Node-API Team Project Jan 9, 2026
@vmoroz vmoroz force-pushed the pr/fix-name-node_api_create_object_with_properties branch from 0a6852b to dfd1f3c Compare January 9, 2026 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. node-api Issues and PRs related to the Node-API.

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

4 participants