From e312d60e3f81dfa26d5922f3c23a371b2b2ce343 Mon Sep 17 00:00:00 2001 From: Jacob Smith <3012099+JakobJingleheimer@users.noreply.github.com> Date: Fri, 25 Oct 2024 22:40:54 +0200 Subject: [PATCH] module: add `findPackageJSON` util PR-URL: https://github.com/nodejs/node/pull/55412 Reviewed-By: Matteo Collina Reviewed-By: Antoine du Hamel --- doc/api/module.md | 82 ++++++ lib/internal/modules/cjs/loader.js | 22 +- lib/internal/modules/esm/resolve.js | 103 ++----- lib/internal/modules/package_json_reader.js | 265 ++++++++++++++---- lib/module.js | 7 +- .../packages/cjs-main-no-index/main.js | 1 + .../packages/cjs-main-no-index/other.js | 3 + .../packages/cjs-main-no-index/package.json | 5 + test/fixtures/packages/nested/package.json | 1 + .../packages/nested/sub-pkg-cjs/index.js | 3 + .../packages/nested/sub-pkg-cjs/package.json | 1 + .../packages/nested/sub-pkg-esm/index.js | 3 + .../packages/nested/sub-pkg-esm/package.json | 1 + .../packages/root-types-field/index.js | 0 .../packages/root-types-field/package.json | 5 + test/parallel/test-find-package-json.js | 152 ++++++++++ typings/internalBinding/modules.d.ts | 13 +- 17 files changed, 516 insertions(+), 151 deletions(-) create mode 100644 test/fixtures/packages/cjs-main-no-index/main.js create mode 100644 test/fixtures/packages/cjs-main-no-index/other.js create mode 100644 test/fixtures/packages/cjs-main-no-index/package.json create mode 100644 test/fixtures/packages/nested/package.json create mode 100644 test/fixtures/packages/nested/sub-pkg-cjs/index.js create mode 100644 test/fixtures/packages/nested/sub-pkg-cjs/package.json create mode 100644 test/fixtures/packages/nested/sub-pkg-esm/index.js create mode 100644 test/fixtures/packages/nested/sub-pkg-esm/package.json create mode 100644 test/fixtures/packages/root-types-field/index.js create mode 100644 test/fixtures/packages/root-types-field/package.json create mode 100644 test/parallel/test-find-package-json.js diff --git a/doc/api/module.md b/doc/api/module.md index 8662cc218898d5..5ead40d6da3fe0 100644 --- a/doc/api/module.md +++ b/doc/api/module.md @@ -217,6 +217,88 @@ added: v22.8.0 * Returns: {string|undefined} Path to the [module compile cache][] directory if it is enabled, or `undefined` otherwise. +### `module.findPackageJSON(specifier[, base])` + + + +> Stability: 1.1 - Active Development + +* `specifier` {string|URL} The specifier for the module whose `package.json` to + retrieve. When passing a _bare specifier_, the `package.json` at the root of + the package is returned. When passing a _relative specifier_ or an _absolute specifier_, + the closest parent `package.json` is returned. +* `base` {string|URL} The absolute location (`file:` URL string or FS path) of the + containing module. For CJS, use `__filename` (not `__dirname`!); for ESM, use + `import.meta.url`. You do not need to pass it if `specifier` is an `absolute specifier`. +* Returns: {string|undefined} A path if the `package.json` is found. When `startLocation` + is a package, the package's root `package.json`; when a relative or unresolved, the closest + `package.json` to the `startLocation`. + +> **Caveat**: Do not use this to try to determine module format. There are many things effecting +> that determination; the `type` field of package.json is the _least_ definitive (ex file extension +> superceeds it, and a loader hook superceeds that). + +```text +/path/to/project + ├ packages/ + ├ bar/ + ├ bar.js + └ package.json // name = '@foo/bar' + └ qux/ + ├ node_modules/ + └ some-package/ + └ package.json // name = 'some-package' + ├ qux.js + └ package.json // name = '@foo/qux' + ├ main.js + └ package.json // name = '@foo' +``` + +```mjs +// /path/to/project/packages/bar/bar.js +import { findPackageJSON } from 'node:module'; + +findPackageJSON('..', import.meta.url); +// '/path/to/project/package.json' +// Same result when passing an absolute specifier instead: +findPackageJSON(new URL('../', import.meta.url)); +findPackageJSON(import.meta.resolve('../')); + +findPackageJSON('some-package', import.meta.url); +// '/path/to/project/packages/bar/node_modules/some-package/package.json' +// When passing an absolute specifier, you might get a different result if the +// resolved module is inside a subfolder that has nested `package.json`. +findPackageJSON(import.meta.resolve('some-package')); +// '/path/to/project/packages/bar/node_modules/some-package/some-subfolder/package.json' + +findPackageJSON('@foo/qux', import.meta.url); +// '/path/to/project/packages/qux/package.json' +``` + +```cjs +// /path/to/project/packages/bar/bar.js +const { findPackageJSON } = require('node:module'); +const { pathToFileURL } = require('node:url'); +const path = require('node:path'); + +findPackageJSON('..', __filename); +// '/path/to/project/package.json' +// Same result when passing an absolute specifier instead: +findPackageJSON(pathToFileURL(path.join(__dirname, '..'))); + +findPackageJSON('some-package', __filename); +// '/path/to/project/packages/bar/node_modules/some-package/package.json' +// When passing an absolute specifier, you might get a different result if the +// resolved module is inside a subfolder that has nested `package.json`. +findPackageJSON(pathToFileURL(require.resolve('some-package'))); +// '/path/to/project/packages/bar/node_modules/some-package/some-subfolder/package.json' + +findPackageJSON('@foo/qux', __filename); +// '/path/to/project/packages/qux/package.json' +``` + ### `module.isBuiltin(moduleName)`