fix: ignore dynamic import for hasModulesyntax

README.md

@@ -176,6 +176,15 @@ const [,,, hasModuleSyntax] = parse(`
 hasModuleSyntax === true;
 ```
 
+Dynamic imports are ignored since they can be used in Non-ESM files.
+
+```js
+const [,,, hasModuleSyntax] = parse(`
+  import('./foo.js')
+`);
+hasModuleSyntax === false;
+```
+
 ### Environment Support
 
 Node.js 10+, and [all browsers with Web Assembly support](https://caniuse.com/#feat=wasm).

src/lexer.h

@@ -123,7 +123,8 @@ void addImport (const char16_t* statement_start, const char16_t* start, const ch
   import->dynamic = dynamic;
   import->safe = dynamic == STANDARD_IMPORT;
   import->next = NULL;
-  hasModuleSyntax = true;
+  if (dynamic == IMPORT_META || dynamic == STANDARD_IMPORT)
+    hasModuleSyntax = true;
 }
 
 void addExport (const char16_t* start, const char16_t* end, const char16_t* local_start, const char16_t* local_end) {

test/_unit.cjs

@@ -1399,7 +1399,8 @@ function x() {
   })
   test('hasModuleSyntax import3', () => {
     const [,,, hasModuleSyntax] = parse('import("./foo")')
-    assert.strictEqual(hasModuleSyntax, true)
+    // dynamic imports can be used in non-ESM files as well
+    assert.strictEqual(hasModuleSyntax, false)
   })
   test('hasModuleSyntax import4', () => {
     const [,,, hasModuleSyntax] = parse('import.meta.url')