fix: should use specific runtime to get target url (#11372)

Author: JSerFeng

crates/rspack_core/src/artifacts/code_generation_results.rs

@@ -275,6 +275,26 @@ impl CodeGenerationResults {
     }
   }
 
+  /**
+   * This API should be used carefully, it will return one of the code generation result,
+   * make sure the module has the same code generation result for all runtimes.
+   */
+  pub fn get_one(
+    &self,
+    module_identifier: &ModuleIdentifier,
+  ) -> &BindingCell<CodeGenerationResult> {
+    self
+      .map
+      .get(module_identifier)
+      .and_then(|entry| {
+        entry
+          .values()
+          .next()
+          .and_then(|m| self.module_generation_result_map.get(m))
+      })
+      .unwrap_or_else(|| panic!("No code generation result for {module_identifier}"))
+  }
+
   pub fn add(
     &mut self,
     module_identifier: ModuleIdentifier,

crates/rspack_plugin_css/src/dependency/url.rs

@@ -35,11 +35,9 @@ impl CssUrlDependency {
     identifier: &ModuleIdentifier,
     compilation: &Compilation,
   ) -> Option<String> {
-    // Here we need the asset module's runtime to get the code generation result, which is not equal to
-    // the css module's runtime, but actually multiple runtime optimization doesn't affect asset module,
-    // in different runtime asset module will always have the same code generation result, so we use
-    // `runtime: None` to get the only one code generation result
-    let code_gen_result = compilation.code_generation_results.get(identifier, None);
+    // url points to asset modules, and asset modules should have same codegen results for all runtimes
+    let code_gen_result = compilation.code_generation_results.get_one(identifier);
+
     if let Some(url) = code_gen_result.data.get::<CodeGenerationDataUrl>() {
       Some(url.inner().to_string())
     } else if let Some(data) = code_gen_result.data.get::<CodeGenerationDataFilename>() {

packages/rspack-test-tools/tests/configCases/css/rspack-issue-11345/image.svg

@@ -0,0 +1,3 @@
+.test {
+    background: url('./image.svg');
+}

packages/rspack-test-tools/tests/configCases/css/rspack-issue-11345/main.css

@@ -0,0 +1,3 @@
+it('should compile without panic', () => {
+
+})

packages/rspack-test-tools/tests/configCases/css/rspack-issue-11345/main.js

@@ -0,0 +1,22 @@
+module.exports = {
+	entry: {
+		asset: "./image.svg",
+		style: "./main.css",
+		main: "./main.js"
+	},
+	target: "web",
+	module: {
+		rules: [
+			{
+				test: /\.svg$/,
+				type: "asset/resource"
+			}
+		]
+	},
+	output: {
+		filename: "[name].js"
+	},
+	experiments: {
+		css: true
+	}
+};