Plan of process.binding() deprecation
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 693
- Forks
- 142
- PR merge metrics
- No merged PRs in 30d
Description
This issue is not new but motivated by https://github.com/nodejs/node/pull/50687, which proposed to
- Ship the runtime deprecation in Node.js v22 --deprecated-process-binding on by default.
- --no-deprecated-process-binding becomes the default in Node.js v23
- process.binding(...) is removed entirely in Node.js v24, with the CLI option becoming a no-op
For background in 2021 the plan suggested in https://github.com/nodejs/node/pull/37485#pullrequestreview-600060802 and followed since https://github.com/nodejs/node/pull/37819 was:
- Do comprehensive research into what exact features ecosystem modules get through process.binding() (this isn't actually going to be a huge API surface!)
- Patch process.binding() so that it returns objects that cover those use cases, implemented in terms of public APIs
- Optionally, runtime-deprecate process.binding() only for the cases that users aren't commonly encountering
- Eventually, remove support for those cases specifically
- Leave it at that, don't runtime-deprecate process.binding() as a whole and leave the shim in place forever
I am opening this issue trying to slice the issue into different questions instead of bundling all together into just two different plans, in the hope of figuring out a better compromise.
There are several questions to the deprecation of process.binding():
- What's our end-goal when the deprecation reaches EOL?
a) Deleteprocess.bindingcompletely so that when trying to invoke it, users get an error.
b) A function that does no more than emitting a warning and returning empty objects. Users almost always get an error when they try to access an unsupported property, but if the code path is unused, e.g. only loading aprocess.binding()from the top-level but never actually hit the code path using anything from it, it would not fail.
c) Leave a low-maintenance and harmless shim in place forever. Users get an error only when they try to use a specific unsupported property. - How should the list of bindings (currently 20) be deprecated?
a) Individually, based on how problematic they are/how difficult it is to provide alternatives
b) Together at once - Should the runtime deprecation start before or after alternatives/shims are completed
a) We investigate the impact (e.g. through CITGM), develop and release alternatives on the way, monitor the changes in the impact, before deciding when it's ready to runtime-deprecate it (individually or all together).
b) Runtime-deprecate it directly and mitigate the impact afterwards based on the bug reports/feature requests we receive. - How should the warnings be rolled out when we start to emit warnings (there are a few more combination possibilities with 2)
a) Emit runtime warnings unconditionally
c) Non-node_module warnings first, node_module warnings in the next cycle
I think we as @nodejs/tsc need to take a deeper look into these questions and make some decisions before finishing a plan of the deprecation.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the referenced Node.js pull requests and the questions about end-state behavior, binding rollout, alternatives, and warning timing. Done means the TSC has resolved these questions and documented an agreed deprecation plan; no implementation files or tests are named.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, node.js
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100