oxc-parser 0.110.0 → 0.112.0

package/src-js/generated/lazy/constructors.js

@@ -11274,13 +11274,11 @@ const DebugTSImportEqualsDeclaration = class TSImportEqualsDeclaration {};
 function constructTSModuleReference(pos, ast) {
   switch (ast.buffer[pos]) {
     case 0:
-      return constructBoxIdentifierReference(pos + 8, ast);
+      return constructBoxTSExternalModuleReference(pos + 8, ast);
     case 1:
-      return constructBoxTSQualifiedName(pos + 8, ast);
+      return constructBoxIdentifierReference(pos + 8, ast);
     case 2:
-      return constructBoxThisExpression(pos + 8, ast);
-    case 3:
-      return constructBoxTSExternalModuleReference(pos + 8, ast);
+      return constructBoxTSQualifiedName(pos + 8, ast);
     default:
       throw new Error(`Unexpected discriminant ${ast.buffer[pos]} for TSModuleReference`);
   }

package/src-js/generated/lazy/walk.js

@@ -4572,16 +4572,13 @@ function walkTSImportEqualsDeclaration(pos, ast, visitors) {
 function walkTSModuleReference(pos, ast, visitors) {
   switch (ast.buffer[pos]) {
     case 0:
-      walkBoxIdentifierReference(pos + 8, ast, visitors);
+      walkBoxTSExternalModuleReference(pos + 8, ast, visitors);
       return;
     case 1:
-      walkBoxTSQualifiedName(pos + 8, ast, visitors);
+      walkBoxIdentifierReference(pos + 8, ast, visitors);
       return;
     case 2:
-      walkBoxThisExpression(pos + 8, ast, visitors);
-      return;
-    case 3:
-      walkBoxTSExternalModuleReference(pos + 8, ast, visitors);
+      walkBoxTSQualifiedName(pos + 8, ast, visitors);
       return;
     default:
       throw new Error(`Unexpected discriminant ${ast.buffer[pos]} for TSModuleReference`);

package/src-js/generated/visit/walk.js

@@ -3,11 +3,9 @@
 
 export { walkProgram };
 
-const { isArray } = Array;
-
 function walkNode(node, visitors) {
   if (node == null) return;
-  if (isArray(node)) {
+  if (Array.isArray(node)) {
     let len = node.length;
     for (let i = 0; i < len; i++) walkNode(node[i], visitors);
   } else

package/src-js/index.d.ts

@@ -142,9 +142,17 @@ export declare const enum ImportNameKind {
 }
 
 /**
- * Parse asynchronously.
+ * Parse JS/TS source asynchronously on a separate thread.
  *
- * Note: This function can be slower than `parseSync` due to the overhead of spawning a thread.
+ * Note that not all of the workload can happen on a separate thread.
+ * Parsing on Rust side does happen in a separate thread, but deserialization of the AST to JS objects
+ * has to happen on current thread. This synchronous deserialization work typically outweighs
+ * the asynchronous parsing by a factor of between 3 and 20.
+ *
+ * i.e. the majority of the workload cannot be parallelized by using this method.
+ *
+ * Generally `parseSync` is preferable to use as it does not have the overhead of spawning a thread.
+ * If you need to parallelize parsing multiple files, it is recommended to use worker threads.
  */
 export declare function parse(filename: string, sourceText: string, options?: ParserOptions | undefined | null): Promise<ParseResult>
 
@@ -189,7 +197,16 @@ export interface ParserOptions {
   showSemanticErrors?: boolean
 }
 
-/** Parse synchronously. */
+/**
+ * Parse JS/TS source synchronously on current thread.
+ *
+ * This is generally preferable over `parse` (async) as it does not have the overhead
+ * of spawning a thread, and the majority of the workload cannot be parallelized anyway
+ * (see `parse` documentation for details).
+ *
+ * If you need to parallelize parsing multiple files, it is recommended to use worker threads
+ * with `parseSync` rather than using `parse`.
+ */
 export declare function parseSync(filename: string, sourceText: string, options?: ParserOptions | undefined | null): ParseResult
 
 /** Returns `true` if raw transfer is supported on this platform. */

package/src-js/visit/visitor.js

@@ -78,8 +78,6 @@ import {
   NODE_TYPES_COUNT,
 } from "../generated/visit/type_ids.js";
 
-const { isArray } = Array;
-
 // Compiled visitor used for visiting each file.
 // Same array is reused for each file.
 //
@@ -225,7 +223,7 @@ export function addVisitorToCompiled(visitor) {
       // Leaf node - store just 1 function, not enter+exit pair
       if (existing === null) {
         compiledVisitor[typeId] = visitFn;
-      } else if (isArray(existing)) {
+      } else if (Array.isArray(existing)) {
         if (isExit) {
           existing.push(visitFn);
         } else {
@@ -256,7 +254,7 @@ export function addVisitorToCompiled(visitor) {
         const { exit } = existing;
         if (exit === null) {
           existing.exit = visitFn;
-        } else if (isArray(exit)) {
+        } else if (Array.isArray(exit)) {
           exit.push(visitFn);
         } else {
           existing.exit = createVisitFnArray(exit, visitFn);
@@ -266,7 +264,7 @@ export function addVisitorToCompiled(visitor) {
         const { enter } = existing;
         if (enter === null) {
           existing.enter = visitFn;
-        } else if (isArray(enter)) {
+        } else if (Array.isArray(enter)) {
           enter.push(visitFn);
         } else {
           existing.enter = createVisitFnArray(enter, visitFn);