Module hooks' context.importAttributes is undefined for require(), but documented as an <Object> #67209

Affected URL(s)

• https://nodejs.org/api/module.html#synchronous-resolvespecifier-context-nextresolve
• https://nodejs.org/api/module.html#synchronous-loadurl-context-nextload

Description of the problem

Both hooks document context.importAttributes as an <Object>, but for a module that require() loads it's undefined. That looks deliberate: the CommonJS loader passes /* importAttributes */ undefined to the hooks (resolve, load, load), and customization_hooks.js types it ImportAttributes|undefined.

So a hook written from the docs, e.g. context.importAttributes.type === "json", throws as soon as anything is require()d, which in practice means as soon as a CommonJS dependency loads. @types/node follows the docs too (importAttributes: ImportAttributes), so TypeScript doesn't catch it either.

On v26.13.0, logging it from registerHooks() for require() and import() of the same kinds of files:

load    main.cjs {}
resolve ./a.cjs undefined
load    a.cjs undefined
resolve ./b.mjs undefined
load    b.mjs undefined
resolve ./c.json undefined
load    c.json undefined
resolve ./b.mjs?x {}
load    b.mjs?x {}
resolve ./c.json?x {"type":"json"}
load    c.json?x {"type":"json"}

import { registerHooks } from "node:module";

registerHooks({
  resolve(specifier, context, nextResolve) {
    if (specifier.startsWith("./")) {
      console.log("resolve", specifier, JSON.stringify(context.importAttributes));
    }
    return nextResolve(specifier, context);
  },
  load(url, context, nextLoad) {
    if (url.startsWith("file:")) {
      console.log("load   ", url.split("/").pop(), JSON.stringify(context.importAttributes));
    }
    return nextLoad(url, context);
  },
});

require("./a.cjs");
require("./b.mjs");
require("./c.json");
import("./b.mjs?x").then(() => import("./c.json?x", { with: { type: "json" } }));

Either documenting it as <Object> | <undefined> (undefined for require()), or passing {} there, as import() without attributes already gets, would fix it. If it's meant to stay undefined, documenting that is enough, and @types/node can follow.