src: set default config as node.config.json #57171

marco-ippolito · 2025-02-21T15:28:20Z

This PR changes the current behavior:

node --experimental-default-config-file foo.js

Will automatically look for a node.config.json file (I dont have strong opinion on the default)

If a --experimental-config-file is provided it will use that:

node --experimental-config-file=foo.json foo.js

jasnell · 2025-02-21T16:18:18Z

Hmm, I'm a bit concerned about naming collisions with existing uses of node.json such as this https://github.com/kubeedge/kubeedge/blob/master/build/node.json.

marco-ippolito · 2025-02-21T16:20:46Z

Maybe noderc.json? That should not have collisions (I HOPE 😆)

targos · 2025-02-21T16:33:19Z

In any case, I don't think it should be unflagged from the first release.

marco-ippolito · 2025-02-21T16:34:35Z

In any case, I don't think it should be unflagged from the first release.

it's not unflagged, I'm not sure what are you referring to

targos · 2025-02-21T16:37:20Z

Okay then, I guess I didn't read the commit message correctly

codecov · 2025-02-21T16:45:28Z

Codecov Report

Attention: Patch coverage is 73.33333% with 4 lines in your changes missing coverage. Please review.

Project coverage is 90.26%. Comparing base (044d69a) to head (e38690d).
Report is 542 commits behind head on main.

Files with missing lines	Patch %	Lines
src/node_config_file.cc	63.63%	2 Missing and 2 partials ⚠️

Additional details and impacted files

@@ Coverage Diff @@ ## main #57171 +/- ## ========================================== - Coverage 90.35% 90.26% -0.09%  ========================================== Files 629 630 +1 Lines 184424 184643 +219 Branches 36034 36135 +101 ========================================== + Hits 166633 166673 +40  - Misses 10913 11024 +111  - Partials 6878 6946 +68

Files with missing lines	Coverage Δ
lib/internal/process/pre_execution.js	`93.03% <100.00%> (-0.28%)`	⬇️
src/node_options.cc	`85.34% <100.00%> (+0.01%)`	⬆️
src/node_options.h	`98.33% <100.00%> (+<0.01%)`	⬆️
src/node_config_file.cc	`73.18% <63.63%> (+1.21%)`	⬆️

... and 41 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.

jasnell · 2025-02-21T16:50:23Z

When searching I was only able to find examples of node.json in common use. Even nodejs.json might work (despite the ugly double js in there). Or .node.json (making it a dot prefixed file). The other possibility is that we look for a .nodejs directory in cwd and load .nodejs/config.json. Every option is likely equally unfortunate in their own special ways tho.

marco-ippolito · 2025-02-21T18:04:33Z

Best options are
.node.json or
noderc.json imho, I'll try a survey on social media so see what people think

anonrig

How does this play with file permissions?

marco-ippolito · 2025-02-22T09:54:17Z

How does this play with file permissions?

I added a test, also will need a rebase after #57170 lands.

There are 6k results for node.json on github:
https://github.com/search?q=path:**/node.json+language:JSON&type=code&l=JSON

I wish we could just take it 🥺

Otherwise:

node.config.json
nodejs.json

darcyrush · 2025-02-24T06:50:04Z

Modern versions of eslint and prettier both support a version of .config naming notion such as eslint.config.cjs, so node.config.json would align nicely, and having .config in the name makes it pretty obvious and explicit.

bricss · 2025-02-24T11:24:45Z

IMO, node.config.json to de moon 🚀🌔

marco-ippolito · 2025-02-24T12:05:00Z

node.config.json it is

doc/api/cli.md

PR-URL: #57171 Backport-PR-URL: #57958 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>

Notable changes: deps: * update timezone to 2025b (Node.js GitHub Bot) #57857 doc: * add dario-piotrowicz to collaborators (Dario Piotrowicz) #58102 * (SEMVER-MINOR) graduate multiple experimental apis (James M Snell) #57765 esm: * (SEMVER-MINOR) graduate import.meta properties (James M Snell) #58011 * (SEMVER-MINOR) support top-level Wasm without package type (Guy Bedford) #57610 sqlite: * (SEMVER-MINOR) add StatementSync.prototype.columns() (Colin Ihrig) #57490 src: * (SEMVER-MINOR) set default config as `node.config.json` (Marco Ippolito) #57171 * (SEMVER-MINOR) create `THROW_ERR_OPTIONS_BEFORE_BOOTSTRAPPING` (Marco Ippolito) #57016 * (SEMVER-MINOR) add config file support (Marco Ippolito) #57016 * (SEMVER-MINOR) add ExecutionAsyncId getter for any Context (Attila Szegedi) #57820 stream: * (SEMVER-MINOR) preserve AsyncLocalStorage context in finished() (Gürgün Dayıoğlu) #57865 util: * (SEMVER-MINOR) add `types.isFloat16Array()` (Livia Medeiros) #57879 worker: * (SEMVER-MINOR) add worker.getHeapStatistics() (Matteo Collina) #57888 PR-URL: #58388

PR-URL: #57171 Backport-PR-URL: #57958 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>

Notable changes: deps: * update timezone to 2025b (Node.js GitHub Bot) #57857 doc: * add dario-piotrowicz to collaborators (Dario Piotrowicz) #58102 * (SEMVER-MINOR) graduate multiple experimental apis (James M Snell) #57765 esm: * (SEMVER-MINOR) graduate import.meta properties (James M Snell) #58011 * (SEMVER-MINOR) support top-level Wasm without package type (Guy Bedford) #57610 sqlite: * (SEMVER-MINOR) add StatementSync.prototype.columns() (Colin Ihrig) #57490 src: * (SEMVER-MINOR) set default config as `node.config.json` (Marco Ippolito) #57171 * (SEMVER-MINOR) create `THROW_ERR_OPTIONS_BEFORE_BOOTSTRAPPING` (Marco Ippolito) #57016 * (SEMVER-MINOR) add config file support (Marco Ippolito) #57016 * (SEMVER-MINOR) add ExecutionAsyncId getter for any Context (Attila Szegedi) #57820 stream: * (SEMVER-MINOR) preserve AsyncLocalStorage context in finished() (Gürgün Dayıoğlu) #57865 util: * (SEMVER-MINOR) add `types.isFloat16Array()` (Livia Medeiros) #57879 worker: * (SEMVER-MINOR) add worker.getHeapStatistics() (Matteo Collina) #57888 PR-URL: TODO

Notable changes: deps: * update timezone to 2025b (Node.js GitHub Bot) #57857 doc: * add dario-piotrowicz to collaborators (Dario Piotrowicz) #58102 * (SEMVER-MINOR) graduate multiple experimental apis (James M Snell) #57765 esm: * (SEMVER-MINOR) graduate import.meta properties (James M Snell) #58011 * (SEMVER-MINOR) support top-level Wasm without package type (Guy Bedford) #57610 sqlite: * (SEMVER-MINOR) add StatementSync.prototype.columns() (Colin Ihrig) #57490 src: * (SEMVER-MINOR) set default config as `node.config.json` (Marco Ippolito) #57171 * (SEMVER-MINOR) create `THROW_ERR_OPTIONS_BEFORE_BOOTSTRAPPING` (Marco Ippolito) #57016 * (SEMVER-MINOR) add config file support (Marco Ippolito) #57016 * (SEMVER-MINOR) add ExecutionAsyncId getter for any Context (Attila Szegedi) #57820 stream: * (SEMVER-MINOR) preserve AsyncLocalStorage context in finished() (Gürgün Dayıoğlu) #57865 util: * (SEMVER-MINOR) add `types.isFloat16Array()` (Livia Medeiros) #57879 worker: * (SEMVER-MINOR) add worker.getHeapStatistics() (Matteo Collina) #57888 PR-URL: #58388

Notable changes: deps: * update timezone to 2025b (Node.js GitHub Bot) nodejs#57857 doc: * add dario-piotrowicz to collaborators (Dario Piotrowicz) nodejs#58102 * (SEMVER-MINOR) graduate multiple experimental apis (James M Snell) nodejs#57765 esm: * (SEMVER-MINOR) graduate import.meta properties (James M Snell) nodejs#58011 * (SEMVER-MINOR) support top-level Wasm without package type (Guy Bedford) nodejs#57610 sqlite: * (SEMVER-MINOR) add StatementSync.prototype.columns() (Colin Ihrig) nodejs#57490 src: * (SEMVER-MINOR) set default config as `node.config.json` (Marco Ippolito) nodejs#57171 * (SEMVER-MINOR) create `THROW_ERR_OPTIONS_BEFORE_BOOTSTRAPPING` (Marco Ippolito) nodejs#57016 * (SEMVER-MINOR) add config file support (Marco Ippolito) nodejs#57016 * (SEMVER-MINOR) add ExecutionAsyncId getter for any Context (Attila Szegedi) nodejs#57820 stream: * (SEMVER-MINOR) preserve AsyncLocalStorage context in finished() (Gürgün Dayıoğlu) nodejs#57865 util: * (SEMVER-MINOR) add `types.isFloat16Array()` (Livia Medeiros) nodejs#57879 worker: * (SEMVER-MINOR) add worker.getHeapStatistics() (Matteo Collina) nodejs#57888 PR-URL: nodejs#58388

Notable changes: deps: * update timezone to 2025b (Node.js GitHub Bot) #57857 doc: * add dario-piotrowicz to collaborators (Dario Piotrowicz) #58102 * (SEMVER-MINOR) graduate multiple experimental apis (James M Snell) #57765 esm: * (SEMVER-MINOR) graduate import.meta properties (James M Snell) #58011 * (SEMVER-MINOR) support top-level Wasm without package type (Guy Bedford) #57610 sqlite: * (SEMVER-MINOR) add StatementSync.prototype.columns() (Colin Ihrig) #57490 src: * (SEMVER-MINOR) set default config as `node.config.json` (Marco Ippolito) #57171 * (SEMVER-MINOR) create `THROW_ERR_OPTIONS_BEFORE_BOOTSTRAPPING` (Marco Ippolito) #57016 * (SEMVER-MINOR) add config file support (Marco Ippolito) #57016 * (SEMVER-MINOR) add ExecutionAsyncId getter for any Context (Attila Szegedi) #57820 stream: * (SEMVER-MINOR) preserve AsyncLocalStorage context in finished() (Gürgün Dayıoğlu) #57865 util: * (SEMVER-MINOR) add `types.isFloat16Array()` (Livia Medeiros) #57879 worker: * (SEMVER-MINOR) add worker.getHeapStatistics() (Matteo Collina) #57888 PR-URL: #58388

nodejs/node#57171

* chore: bump node in DEPS to v22.16.0 * crypto: remove BoringSSL dh-primes addition nodejs/node#57023 * tools: enable linter in test/fixtures/test\-runner/output nodejs/node#57698 * src: improve thread safety of TaskQueue nodejs/node#57910 * buffer: define global v8::CFunction objects as const nodejs/node#57676 * src: disable abseil deadlock detection nodejs/node#57582 * zlib: fix pointer alignment nodejs/node#57727 * chore: fixup patch indices * src: set default config as node.config.json nodejs/node#57171 * src: update std::vector<v8::Local<T>> to use v8::LocalVector<T> nodejs/node#57578 * test: disable chmod tests failing in Docker nodejs/node#58326 --------- Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com> Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>

nodejs/node#57171

* chore: bump node in DEPS to v22.16.0 * crypto: remove BoringSSL dh-primes addition nodejs/node#57023 * tools: enable linter in test/fixtures/test\-runner/output nodejs/node#57698 * src: improve thread safety of TaskQueue nodejs/node#57910 * buffer: define global v8::CFunction objects as const nodejs/node#57676 * src: disable abseil deadlock detection nodejs/node#57582 * zlib: fix pointer alignment nodejs/node#57727 * chore: fixup patch indices * src: set default config as node.config.json nodejs/node#57171 * src: update std::vector<v8::Local<T>> to use v8::LocalVector<T> nodejs/node#57578 * test: disable chmod tests failing in Docker nodejs/node#58326 --------- Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com> Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>

nodejs/node#57171

* chore: bump node in DEPS to v22.16.0 * crypto: remove BoringSSL dh-primes addition nodejs/node#57023 * tools: enable linter in test/fixtures/test\-runner/output nodejs/node#57698 * src: improve thread safety of TaskQueue nodejs/node#57910 * buffer: define global v8::CFunction objects as const nodejs/node#57676 * zlib: fix pointer alignment nodejs/node#57727 * chore: fixup patch indices * src: set default config as node.config.json nodejs/node#57171 * src: update std::vector<v8::Local<T>> to use v8::LocalVector<T> nodejs/node#57578 * test: disable chmod tests failing in Docker nodejs/node#58326 * chore: fix out of date patch --------- Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com> Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>

* chore: bump node in DEPS to v22.16.0 * crypto: remove BoringSSL dh-primes addition nodejs/node#57023 * tools: enable linter in test/fixtures/test\-runner/output nodejs/node#57698 * src: improve thread safety of TaskQueue nodejs/node#57910 * buffer: define global v8::CFunction objects as const nodejs/node#57676 * src: disable abseil deadlock detection nodejs/node#57582 * zlib: fix pointer alignment nodejs/node#57727 * chore: fixup patch indices * src: set default config as node.config.json nodejs/node#57171 * src: update std::vector<v8::Local<T>> to use v8::LocalVector<T> nodejs/node#57578 * test: disable chmod tests failing in Docker nodejs/node#58326 --------- Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com> Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>

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. labels Feb 21, 2025

marco-ippolito added the semver-minor PRs that contain new features and should be released in the next minor version. label Feb 21, 2025

marco-ippolito mentioned this pull request Feb 21, 2025

src: namespace config file flags #57170

Closed

anonrig reviewed Feb 22, 2025

View reviewed changes

marco-ippolito marked this pull request as draft February 24, 2025 11:40

marco-ippolito force-pushed the default-value-node-json branch from f1b6d67 to 70b3289 Compare February 24, 2025 12:02

marco-ippolito changed the title ~~src: set default config as node.json~~ src: set default config as node.config.json Feb 24, 2025

marco-ippolito force-pushed the default-value-node-json branch from 70b3289 to e00ade9 Compare February 24, 2025 12:03

marco-ippolito marked this pull request as ready for review February 24, 2025 12:05

marco-ippolito force-pushed the default-value-node-json branch 3 times, most recently from c0373f2 to 6456729 Compare February 24, 2025 12:16

marco-ippolito requested review from anonrig and jasnell February 24, 2025 13:16

GeoffreyBooth reviewed Feb 24, 2025

View reviewed changes

doc/api/cli.md Outdated Show resolved Hide resolved

marco-ippolito force-pushed the default-value-node-json branch 2 times, most recently from 434360e to 54b272c Compare February 25, 2025 06:03

aduh95 pushed a commit that referenced this pull request May 17, 2025

src: set default config as node.config.json

6a7087e

PR-URL: #57171 Backport-PR-URL: #57958 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>

aduh95 pushed a commit that referenced this pull request May 17, 2025

src: set default config as node.config.json

188ad24

PR-URL: #57171 Backport-PR-URL: #57958 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>

aduh95 pushed a commit that referenced this pull request May 17, 2025

src: set default config as node.config.json

a7e7b7e

PR-URL: #57171 Backport-PR-URL: #57958 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>

aduh95 pushed a commit that referenced this pull request May 18, 2025

src: set default config as node.config.json

e85580d

PR-URL: #57171 Backport-PR-URL: #57958 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>

github-actions bot mentioned this pull request May 19, 2025

2025-05-21, Version 22.16.0 'Jod' (LTS) #58388

Merged

aduh95 pushed a commit that referenced this pull request May 19, 2025

src: set default config as node.config.json

5d1230b

PR-URL: #57171 Backport-PR-URL: #57958 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>

codebytere added a commit to electron/electron that referenced this pull request May 23, 2025

src: set default config as node.config.json

781f3f9

nodejs/node#57171

codebytere added a commit to electron/electron that referenced this pull request May 26, 2025

src: set default config as node.config.json

603d0cd

nodejs/node#57171

codebytere mentioned this pull request May 26, 2025

chore: bump node to v22.16.0 (main) electron/electron#47217

Merged

codebytere added a commit to electron/electron that referenced this pull request May 29, 2025

src: set default config as node.config.json

9c9fa0e

nodejs/node#57171

codebytere added a commit to electron/electron that referenced this pull request Jun 2, 2025

src: set default config as node.config.json

8b75146

nodejs/node#57171

codebytere added a commit to electron/electron that referenced this pull request Jun 3, 2025

src: set default config as node.config.json

790f220

nodejs/node#57171

codebytere added a commit to electron/electron that referenced this pull request Jun 3, 2025

src: set default config as node.config.json

bfecc07

nodejs/node#57171

codebytere added a commit to electron/electron that referenced this pull request Jun 3, 2025

src: set default config as node.config.json

984bc0d

nodejs/node#57171

codebytere added a commit to electron/electron that referenced this pull request Jun 3, 2025

src: set default config as node.config.json

3124297

nodejs/node#57171

codebytere added a commit to electron/electron that referenced this pull request Jun 3, 2025

src: set default config as node.config.json

2b24838

nodejs/node#57171

codebytere added a commit to electron/electron that referenced this pull request Jun 3, 2025

src: set default config as node.config.json

23daf0c

nodejs/node#57171

codebytere added a commit to electron/electron that referenced this pull request Jun 6, 2025

src: set default config as node.config.json

b99d281

nodejs/node#57171

ghost mentioned this pull request Jun 8, 2025

### Notable Changes autofix-ci/action#28

Closed

codebytere added a commit to electron/electron that referenced this pull request Jun 18, 2025

src: set default config as node.config.json

0656902

nodejs/node#57171

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

src: set default config as node.config.json #57171

src: set default config as node.config.json #57171

Uh oh!

marco-ippolito commented Feb 21, 2025 •

edited

Loading

jasnell commented Feb 21, 2025

marco-ippolito commented Feb 21, 2025

targos commented Feb 21, 2025

marco-ippolito commented Feb 21, 2025

targos commented Feb 21, 2025

codecov bot commented Feb 21, 2025 •

edited

Loading

jasnell commented Feb 21, 2025

marco-ippolito commented Feb 21, 2025

anonrig left a comment

marco-ippolito commented Feb 22, 2025

darcyrush commented Feb 24, 2025 •

edited

Loading

bricss commented Feb 24, 2025

marco-ippolito commented Feb 24, 2025

Uh oh!

Labels

11 participants

Uh oh!

src: set default config as node.config.json #57171

src: set default config as node.config.json #57171

Uh oh!

Conversation

marco-ippolito commented Feb 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

jasnell commented Feb 21, 2025

marco-ippolito commented Feb 21, 2025

targos commented Feb 21, 2025

marco-ippolito commented Feb 21, 2025

targos commented Feb 21, 2025

codecov bot commented Feb 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

jasnell commented Feb 21, 2025

marco-ippolito commented Feb 21, 2025

anonrig left a comment

Choose a reason for hiding this comment

marco-ippolito commented Feb 22, 2025

darcyrush commented Feb 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

bricss commented Feb 24, 2025

marco-ippolito commented Feb 24, 2025

Uh oh!

Labels

11 participants

marco-ippolito commented Feb 21, 2025 •

edited

Loading

codecov bot commented Feb 21, 2025 •

edited

Loading

darcyrush commented Feb 24, 2025 •

edited

Loading