Add JS rewriting rule to ignore import function rewriting

Author: benoit74

src/zimscraperlib/rewriting/js.py

@@ -186,6 +186,7 @@ def create_js_rules() -> list[TransformationRule]:
         # As the rule will match first, it will prevent next rule matching `import` to
         # be apply to `async import`.
         (re.compile(r"async\s+import\s*\("), m2str(lambda x: x)),
+        (re.compile(r"[^$.]\bimport\s*\([^)]*\)\s*\{"), m2str(lambda x: x)),
         # esm dynamic import, if found, mark as module
         (
             re.compile(r"[^$.]\bimport\s*\("),

tests/rewriting/test_js_rewriting.py

@@ -271,6 +271,12 @@ def wrap_import(text: str) -> str:
 import { Z } from "../../../path.js";
 
 B = await import(somefile);
+
+class X {
+  import(a, b, c) {
+    await import (somefile);
+  }
+}
 """,
             expected="""
 import * from "../../../example.com/file.js"
@@ -282,6 +288,12 @@ def wrap_import(text: str) -> str:
 import { Z } from "../../path.js";
 
 B = await ____wb_rewrite_import__(import.meta.url, somefile);
+
+class X {
+  import(a, b, c) {
+    await ____wb_rewrite_import__ (import.meta.url, somefile);
+  }
+}
 """,
         ),
         ImportTestContent(
@@ -341,6 +353,7 @@ def test_import_rewrite(rewrite_import_content: ImportTestContent):
         "a.window.x = 5",
         "  postMessage({'a': 'b'})",
         "simport(5);",
+        "import(e) {",
         "a.import(5);",
         "$import(5);",
         "async import(val) { ... }",