fix: for commonjs, set Program.sourceType to script

Author: ota-meshi

acorn-loose/src/statement.js

@@ -10,7 +10,7 @@ lp.parseTopLevel = function() {
   while (this.tok.type !== tt.eof) node.body.push(this.parseStatement())
   this.toks.adaptDirectivePrologue(node.body)
   this.last = this.tok
-  node.sourceType = this.options.sourceType
+  node.sourceType = this.options.sourceType === "commonjs" ? "script" : this.options.sourceType
   return this.finishNode(node, "Program")
 }
 

acorn/src/acorn.d.ts

@@ -38,7 +38,7 @@ export interface Literal extends Node {
 export interface Program extends Node {
   type: "Program"
   body: Array<Statement | ModuleDeclaration>
-  sourceType: "script" | "module" | "commonjs"
+  sourceType: "script" | "module"
 }
 
 export interface Function extends Node {

acorn/src/statement.js

@@ -27,7 +27,7 @@ pp.parseTopLevel = function(node) {
       this.raiseRecoverable(this.undefinedExports[name].start, `Export '${name}' is not defined`)
   this.adaptDirectivePrologue(node.body)
   this.next()
-  node.sourceType = this.options.sourceType
+  node.sourceType = this.options.sourceType === "commonjs" ? "script" : this.options.sourceType
   return this.finishNode(node, "Program")
 }
 

test/tests-commonjs.js

@@ -21,7 +21,7 @@ test("using x = resource;", {
     }],
     kind: "using"
   }],
-  sourceType: "commonjs"
+  sourceType: "script"
 }, {ecmaVersion: 17, sourceType: "commonjs"});
 
 // Top-level new.target with commonjs
@@ -35,7 +35,7 @@ test("new.target", {
       property: {type: "Identifier", name: "target"}
     }
   }],
-  sourceType: "commonjs"
+  sourceType: "script"
 }, {ecmaVersion: 6, sourceType: "commonjs"});
 
 // new.target at top-level scope with commonjs
@@ -61,7 +61,7 @@ test("let y = () => new.target", {
     }],
     kind: "let"
   }],
-  sourceType: "commonjs"
+  sourceType: "script"
 }, {ecmaVersion: 6, sourceType: "commonjs"});
 
 // Top-level return statement with commonjs
@@ -82,7 +82,7 @@ test("return {} / 2", {
       operator: "/"
     }
   }],
-  sourceType: "commonjs"
+  sourceType: "script"
 }, {sourceType: "commonjs"});
 
 // Illegal return statement with commonjs