🤖 Merge PR DefinitelyTyped#71493 [@types/node] correct ResolveFnOutput.format (can be intermediary) by @JakobJingleheimer

Co-authored-by: René <[email protected]>

Author: JakobJingleheimer

types/node/module.d.ts

@@ -325,9 +325,9 @@ declare module "module" {
         }
         interface ResolveFnOutput {
             /**
-             * A hint to the load hook (it might be ignored)
+             * A hint to the load hook (it might be ignored); can be an intermediary value.
              */
-            format?: ModuleFormat | null | undefined;
+            format?: string | null | undefined;
             /**
              * The import attributes to use when caching the module (optional; if excluded the input will be used)
              */
@@ -365,9 +365,9 @@ declare module "module" {
              */
             conditions: string[];
             /**
-             * The format optionally supplied by the `resolve` hook chain
+             * The format optionally supplied by the `resolve` hook chain (can be an intermediary value).
              */
-            format: ModuleFormat | null | undefined;
+            format: string | null | undefined;
             /**
              *  An object whose key-value pairs represent the assertions for the module to import
              */

types/node/test/module.ts

@@ -191,7 +191,7 @@ Module.Module === Module;
     const load: Module.LoadHook = async (url, context, nextLoad) => {
         const { format } = context;
 
-        if (format) {
+        if (format === 'commonjs') { // Needs to match only sometimes
             return {
                 format,
                 shortCircuit: true,