Fixes linting

Author: arcanis

doc/api/errors.md

@@ -2507,7 +2507,7 @@ added: REPLACEME
 A package attempted to import another package that exists in the [package map][]
 but is not listed in its `dependencies` array.
 
-```js
+```mjs
 // package-map.json declares "app" with dependencies: ["utils"]
 // but "app" tries to import "secret-lib" which exists in the map
 
@@ -4504,7 +4504,6 @@ An error occurred trying to allocate memory. This should never happen.
 [`new URL(input)`]: url.md#new-urlinput-base
 [`new URLPattern(input)`]: url.md#new-urlpatternstring-baseurl-options
 [`new URLSearchParams(iterable)`]: url.md#new-urlsearchparamsiterable
-[package map]: packages.md#package-maps
 [`package.json`]: packages.md#nodejs-packagejson-field-definitions
 [`postMessage()`]: worker_threads.md#portpostmessagevalue-transferlist
 [`postMessageToThread()`]: worker_threads.md#worker_threadspostmessagetothreadthreadid-value-transferlist-timeout
@@ -4535,6 +4534,7 @@ An error occurred trying to allocate memory. This should never happen.
 [domains]: domain.md
 [event emitter-based]: events.md#class-eventemitter
 [file descriptors]: https://en.wikipedia.org/wiki/File_descriptor
+[package map]: packages.md#package-maps
 [relative URL]: https://url.spec.whatwg.org/#relative-url-string
 [self-reference a package using its name]: packages.md#self-referencing-a-package-using-its-name
 [special scheme]: https://url.spec.whatwg.org/#special-scheme

doc/api/esm.md

@@ -1303,6 +1303,7 @@ resolution for ESM specifiers is [commonjs-extension-resolution-loader][].
 [Loading ECMAScript modules using `require()`]: modules.md#loading-ecmascript-modules-using-require
 [Module customization hooks]: module.md#customization-hooks
 [Node.js Module Resolution And Loading Algorithm]: #resolution-algorithm-specification
+[Package maps]: packages.md#package-maps
 [Source Phase Imports]: https://github.com/tc39/proposal-source-phase-imports
 [Terminology]: #terminology
 [URL]: https://url.spec.whatwg.org/
@@ -1331,7 +1332,6 @@ resolution for ESM specifiers is [commonjs-extension-resolution-loader][].
 [custom https loader]: module.md#import-from-https
 [import.meta.resolve]: #importmetaresolvespecifier
 [merve]: https://github.com/anonrig/merve/tree/v1.0.0
-[Package maps]: packages.md#package-maps
 [percent-encoded]: url.md#percent-encoding-in-urls
 [special scheme]: https://url.spec.whatwg.org/#special-scheme
 [status code]: process.md#exit-codes

doc/api/modules.md

@@ -1273,6 +1273,7 @@ This section was moved to
 [Determining module system]: packages.md#determining-module-system
 [ECMAScript Modules]: esm.md
 [GLOBAL_FOLDERS]: #loading-from-the-global-folders
+[Package maps]: packages.md#package-maps
 [`"main"`]: packages.md#main
 [`"type"`]: packages.md#type
 [`--experimental-package-map`]: cli.md#--experimental-package-mappath
@@ -1300,6 +1301,5 @@ This section was moved to
 [module namespace object]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/import#module_namespace_object
 [module resolution]: #all-together
 [native addons]: addons.md
-[Package maps]: packages.md#package-maps
 [subpath exports]: packages.md#subpath-exports
 [subpath imports]: packages.md#subpath-imports

lib/internal/modules/cjs/loader.js

@@ -655,7 +655,7 @@ function trySelf(parentPath, request, conditions) {
 /**
  * Try to resolve using package map (if enabled via --experimental-package-map).
  * @param {string} request - The bare specifier
- * @param {string} parentPath - File path of the parent module
+ * @param {Module} parent - The parent module
  * @param {Set<string>} conditions - Export conditions
  * @returns {string|undefined}
  */

lib/internal/modules/package_map.js

@@ -18,7 +18,6 @@ const getPackageMapPath = getLazy(() =>
   require('internal/options').getOptionValue('--experimental-package-map'),
 );
 const fs = require('fs');
-const { fileURLToPath } = require('internal/url');
 
 const {
   ERR_PACKAGE_MAP_ACCESS_DENIED,
@@ -88,7 +87,7 @@ class PackageMap {
       this.#packages.set(key, {
         path: absolutePath,
         dependencies: new SafeSet(entry.dependencies ?? []),
-        name: entry.name,  // undefined for nameless packages
+        name: entry.name,  // Undefined for nameless packages
       });
 
       // Index by name (only named packages)
@@ -114,7 +113,7 @@ class PackageMap {
    */
   #getKeyForPath(filePath) {
     const cached = this.#pathToKeyCache.get(filePath);
-    if (cached !== undefined) return cached;
+    if (cached !== undefined) { return cached; }
 
     // Walk up to find containing package
     let checkPath = filePath;
@@ -126,7 +125,7 @@ class PackageMap {
         return key;
       }
       const parent = dirname(checkPath);
-      if (parent === checkPath) break;
+      if (parent === checkPath) { break; }
       checkPath = parent;
     }
 
@@ -146,7 +145,7 @@ class PackageMap {
     const parentKey = this.#getKeyForPath(parentPath);
 
     // Parent not in map - fall back to standard resolution
-    if (parentKey === null) return undefined;
+    if (parentKey === null) { return undefined; }
 
     // Check cache
     const cacheKey = `${parentKey}\0${specifier}`;
@@ -281,7 +280,7 @@ function emitExperimentalWarning() {
  * @returns {PackageMap|null}
  */
 function getPackageMap() {
-  if (packageMap !== undefined) return packageMap;
+  if (packageMap !== undefined) { return packageMap; }
 
   packageMapPath = getPackageMapPath();
   if (!packageMapPath) {
@@ -306,7 +305,6 @@ function hasPackageMap() {
  * Resolve a package specifier using the package map.
  * Returns the package path and subpath, or undefined if resolution should
  * fall back to standard resolution.
- *
  * @param {string} specifier - The bare specifier (e.g., "lodash", "react/jsx-runtime")
  * @param {string} parentPath - File path of the importing module
  * @returns {{packagePath: string, subpath: string}|undefined}

test/es-module/test-esm-package-map.mjs

@@ -1,12 +1,10 @@
-import '../common/index.mjs';
 import { spawnPromisified } from '../common/index.mjs';
 import * as fixtures from '../common/fixtures.mjs';
 import assert from 'node:assert';
 import { execPath } from 'node:process';
 import { describe, it } from 'node:test';
 
 const packageMapPath = fixtures.path('package-map/package-map.json');
-const fixturesPath = fixtures.path('package-map');
 
 describe('ESM: --experimental-package-map', () => {