feat: module error js api (#14351)

Author: SyMind

crates/node_binding/napi-binding.d.ts

@@ -69,6 +69,7 @@ export interface NormalModule extends Module {
 	readonly loaders: JsLoaderItem[];
 	get matchResource(): string | undefined;
 	set matchResource(val: string | undefined);
+	get error(): RspackError | undefined;
 }
 
 export interface ConcatenatedModule extends Module {

crates/node_binding/scripts/banner.d.ts

@@ -69,6 +69,7 @@ export interface NormalModule extends Module {
 	readonly loaders: JsLoaderItem[];
 	get matchResource(): string | undefined;
 	set matchResource(val: string | undefined);
+	get error(): RspackError | undefined;
 }
 
 export interface ConcatenatedModule extends Module {

crates/rspack_binding_api/src/modules/normal_module.rs

@@ -3,8 +3,11 @@ use napi::{
   bindgen_prelude::{FromNapiMutRef, Object, ToNapiValue},
 };
 use rspack_core::{ResourceData, ResourceParsedData, parse_resource};
+use rspack_error::Diagnosable;
 
 use crate::{
+  diagnostic,
+  error::RspackError,
   impl_module_methods,
   module::{MODULE_PROPERTIES_BUFFER, Module},
   plugins::JsLoaderItem,
@@ -86,6 +89,22 @@ impl NormalModule {
       Ok(())
     }
 
+    #[js_function]
+    fn error_getter(ctx: CallContext<'_>) -> napi::Result<Either<RspackError, ()>> {
+      let this = ctx.this_unchecked::<JsObject>();
+      let env = ctx.env.raw();
+      let wrapped_value = unsafe { NormalModule::from_napi_mut_ref(env, this.raw())? };
+
+      let (compilation, module) = wrapped_value.as_ref()?;
+      Ok(match module.first_error() {
+        Some(diagnostic) => Either::A(RspackError::try_from_diagnostic(
+          compilation,
+          diagnostic.as_ref(),
+        )?),
+        None => Either::B(()),
+      })
+    }
+
     MODULE_PROPERTIES_BUFFER.with(|ref_cell| {
       let mut properties = ref_cell.borrow_mut();
       properties.clear();
@@ -126,6 +145,12 @@ impl NormalModule {
           .with_getter(match_resource_getter)
           .with_setter(match_resource_setter),
       );
+      // Info from Build
+      properties.push(
+        napi::Property::new()
+          .with_utf8_name("error")?
+          .with_getter(error_getter),
+      );
       Self::new_inherited(self, env, &mut properties)
     })
   }

tests/rspack-test/errorCases/warning-write.js

@@ -21,39 +21,40 @@ module.exports = {
 	},
 	async check() {
 		expect(warnings).toMatchInlineSnapshot(`
-		Array [
-		  Object {
-		    error: Object {
-		  "message": "Unsupported feature: require.main.require() is not supported by Rspack.",
-		  "name": "Error",
-		  "stack": undefined,
-		},
-		    index: 0,
-		    module: NormalModule {
-		      buildInfo: KnownBuildInfo {},
-		      buildMeta: Object {},
-		      context: <TEST_ROOT>/fixtures/errors,
-		      factoryMeta: Object {},
-		      layer: undefined,
-		      loaders: Array [],
-		      matchResource: undefined,
-		      rawRequest: ./require.main.require,
-		      request: <TEST_ROOT>/fixtures/errors/require.main.require.js,
-		      resource: <TEST_ROOT>/fixtures/errors/require.main.require.js,
-		      resourceResolveData: ReadonlyResourceData {
-		        fragment: ,
-		        path: <TEST_ROOT>/fixtures/errors/require.main.require.js,
-		        query: ,
-		        resource: <TEST_ROOT>/fixtures/errors/require.main.require.js,
-		      },
-		      type: javascript/auto,
-		      useSimpleSourceMap: false,
-		      useSourceMap: false,
-		      userRequest: <TEST_ROOT>/fixtures/errors/require.main.require.js,
-		    },
-		    name: ModuleParseWarning,
-		  },
-		]
-	`);
+			Array [
+			  Object {
+			    error: Object {
+			  "message": "Unsupported feature: require.main.require() is not supported by Rspack.",
+			  "name": "Error",
+			  "stack": undefined,
+			},
+			    index: 0,
+			    module: NormalModule {
+			      buildInfo: KnownBuildInfo {},
+			      buildMeta: Object {},
+			      context: <TEST_ROOT>/fixtures/errors,
+			      error: undefined,
+			      factoryMeta: Object {},
+			      layer: undefined,
+			      loaders: Array [],
+			      matchResource: undefined,
+			      rawRequest: ./require.main.require,
+			      request: <TEST_ROOT>/fixtures/errors/require.main.require.js,
+			      resource: <TEST_ROOT>/fixtures/errors/require.main.require.js,
+			      resourceResolveData: ReadonlyResourceData {
+			        fragment: ,
+			        path: <TEST_ROOT>/fixtures/errors/require.main.require.js,
+			        query: ,
+			        resource: <TEST_ROOT>/fixtures/errors/require.main.require.js,
+			      },
+			      type: javascript/auto,
+			      useSimpleSourceMap: false,
+			      useSourceMap: false,
+			      userRequest: <TEST_ROOT>/fixtures/errors/require.main.require.js,
+			    },
+			    name: ModuleParseWarning,
+			  },
+			]
+		`);
 	}
 };