fix(module-tools): catch rebaseUrl error which break the build (#4802)

web-infra-dev · Oct 17, 2023 · 0488090 · 0488090
1 parent 822c75e
commit 0488090
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 14 deletions.
diff --git a/.changeset/giant-penguins-poke.md b/.changeset/giant-penguins-poke.md
@@ -0,0 +1,6 @@
+---
+'@modern-js/module-tools': patch
+---
+
+fix(module-tools): catch rebaseUrl error which break the build
+fix(module-tools): 捕获会破坏整个构建的 rebaseUrl 错误
diff --git a/packages/solutions/module-tools/src/builder/feature/style/utils.ts b/packages/solutions/module-tools/src/builder/feature/style/utils.ts
@@ -66,20 +66,25 @@ export async function rebaseUrls(
   if (!cssUrlRE.test(content)) {
     return { file };
   }
-  const rebased = await rewriteCssUrls(
-    content,
-    path.extname(file).slice(1),
-    url => {
-      if (url.startsWith('/')) {
-        return url;
-      }
-      return resolver(url, fileDir);
-    },
-  );
-  return {
-    file,
-    contents: rebased,
-  };
+  try {
+    // FIXME: use ast match instead of reg match
+    const rebased = await rewriteCssUrls(
+      content,
+      path.extname(file).slice(1),
+      url => {
+        if (url.startsWith('/')) {
+          return url;
+        }
+        return resolver(url, fileDir);
+      },
+    );
+    return {
+      file,
+      contents: rebased,
+    };
+  } catch (e) {
+    return { file };
+  }
 }
 
 export function rewriteCssUrls(