Update return type of ScriptOptions.importModuleDynamically (DefinitelyTyped#71912)

rhumbertgz · Renegade334 · web-flow · commit 28959991e497 · 2025-02-13T14:27:43.000-08:00
Co-authored-by: René &lt;contact.9a5d6388@renegade334.me.uk&gt;
diff --git a/types/node/vm.d.ts b/types/node/vm.d.ts
@@ -69,7 +69,7 @@ declare module "vm" {
          * [Support of dynamic `import()` in compilation APIs](https://nodejs.org/docs/latest-v22.x/api/vm.html#support-of-dynamic-import-in-compilation-apis).
          */
         importModuleDynamically?:
-            | ((specifier: string, script: Script, importAttributes: ImportAttributes) => Module)
+            | ((specifier: string, script: Script, importAttributes: ImportAttributes) => Module | Promise<Module>)
             | typeof constants.USE_MAIN_CONTEXT_DEFAULT_LOADER
             | undefined;
     }

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ declare module "vm" {`
`69`	`69`	* [Support of dynamic `import()` in compilation APIs](https://nodejs.org/docs/latest-v22.x/api/vm.html#support-of-dynamic-import-in-compilation-apis).
`70`	`70`	`*/`
`71`	`71`	`importModuleDynamically?:`
`72`		`- \| ((specifier: string, script: Script, importAttributes: ImportAttributes) => Module)`
	`72`	`+ \| ((specifier: string, script: Script, importAttributes: ImportAttributes) => Module \| Promise<Module>)`
`73`	`73`	`\| typeof constants.USE_MAIN_CONTEXT_DEFAULT_LOADER`
`74`	`74`	`\| undefined;`
`75`	`75`	`}`