es-check 9.6.4 → 9.7.2

package/lib/helpers/files.js

@@ -7,18 +7,19 @@ function processBatchedFiles(files, processor, batchSize = 0) {
     return files.map(processor);
   }
 
-  const results = [];
+  let results = [];
   for (let i = 0; i < files.length; i += batchSize) {
     const batch = files.slice(i, i + batchSize);
     const batchResults = batch.map(processor);
-    results.push(...batchResults);
+    results = results.concat(batchResults);
   }
 
   return results;
 }
 
 function readFile(file, fs, useCache = false) {
-  if (useCache && fileCache.has(file)) {
+  const shouldReadFromCache = useCache && fileCache.has(file);
+  if (shouldReadFromCache) {
     cacheHits++;
     return fileCache.get(file);
   }
@@ -62,13 +63,16 @@ function clearFileCache() {
 
 function getFileCacheStats() {
   const total = cacheHits + cacheMisses;
+  const hasNoReads = total === 0;
+  const hitPercent = ((cacheHits / total) * 100).toFixed(2);
+  const hitRate = hasNoReads ? "0.00%" : hitPercent + "%";
+
   return {
     size: fileCache.size,
     keys: Array.from(fileCache.keys()),
     hits: cacheHits,
     misses: cacheMisses,
-    hitRate:
-      total === 0 ? "0.00%" : ((cacheHits / total) * 100).toFixed(2) + "%",
+    hitRate,
   };
 }
 

package/lib/helpers/index.js

@@ -8,22 +8,11 @@ const sourcemap = require("./sourcemap");
 
 const detectFeaturesModule = require("../detectFeatures");
 
-module.exports = Object.assign(
-  {},
-  logger,
-  parsers,
-  ast,
-  astDetector,
-  files,
-  cli,
-  sourcemap,
-  {
-    detectFeatures: detectFeaturesModule,
-    polyfillDetector: {
-      detectPolyfills: detectFeaturesModule.detectPolyfills,
-      filterPolyfilled: detectFeaturesModule.filterPolyfilled,
-      filterPolyfillableFeatures:
-        detectFeaturesModule.filterPolyfillableFeatures,
-    },
+module.exports = Object.assign({}, logger, parsers, ast, astDetector, files, cli, sourcemap, {
+  detectFeatures: detectFeaturesModule,
+  polyfillDetector: {
+    detectPolyfills: detectFeaturesModule.detectPolyfills,
+    filterPolyfilled: detectFeaturesModule.filterPolyfilled,
+    filterPolyfillableFeatures: detectFeaturesModule.filterPolyfillableFeatures,
   },
-);
+});

package/lib/helpers/logger.js

@@ -22,8 +22,7 @@ function supportsColor(stream = process.stdout) {
   }
 
   const isTTY = stream.isTTY;
-  const colorsDisabled =
-    process.env.NO_COLOR || process.env.NODE_DISABLE_COLORS;
+  const colorsDisabled = process.env.NO_COLOR || process.env.NODE_DISABLE_COLORS;
   const colorsForced = process.env.FORCE_COLOR;
 
   if (!isTTY) return false;
@@ -52,9 +51,7 @@ function createLogger(options = {}) {
     if (!shouldLog) return;
 
     const message = args
-      .map((arg) =>
-        typeof arg === "string" ? arg : util.inspect(arg, { depth: null }),
-      )
+      .map((arg) => (typeof arg === "string" ? arg : util.inspect(arg, { depth: null })))
       .join(" ");
 
     const isErrorLevel = logLevel === "error" || logLevel === "warn";
@@ -83,8 +80,7 @@ function createLogger(options = {}) {
 
 function determineLogLevel(logger) {
   const hasLogger = logger !== null && logger !== undefined;
-  const hasIsLevelEnabled =
-    hasLogger && typeof logger.isLevelEnabled === "function";
+  const hasIsLevelEnabled = hasLogger && typeof logger.isLevelEnabled === "function";
 
   if (!hasIsLevelEnabled) return null;
 

package/lib/helpers/parsers.js

@@ -8,14 +8,11 @@ function detectRuntime() {
   return "node";
 }
 
-function stripTypesInNode(code) {
-  const moduleAPI = require("module");
+function stripTypesInNode(code, moduleAPI = require("module")) {
   if (typeof moduleAPI.stripTypeScriptTypes === "function") {
     return moduleAPI.stripTypeScriptTypes(code, { mode: "strip" });
   }
-  throw new Error(
-    "es-check execution on TypeScript files requires Node.js v22.13.0+",
-  );
+  throw new Error("es-check execution on TypeScript files requires Node.js v22.13.0+");
 }
 
 function buildLineOffsets(original, stripped) {
@@ -23,13 +20,11 @@ function buildLineOffsets(original, stripped) {
   const strippedLines = stripped.split("\n");
   const lineCount = Math.max(originalLines.length, strippedLines.length);
 
-  const offsets = [];
-  for (let i = 0; i < lineCount; i++) {
+  return Array.from({ length: lineCount }, (_item, i) => {
     const origLen = (originalLines[i] || "").length;
     const stripLen = (strippedLines[i] || "").length;
-    offsets.push(origLen - stripLen);
-  }
-  return offsets;
+    return origLen - stripLen;
+  });
 }
 
 function mapPosition(line, column, lineOffsets) {
@@ -48,15 +43,14 @@ function mapPosition(line, column, lineOffsets) {
 }
 
 function stripTypesInBun(code) {
-  if (typeof Bun !== "undefined" && typeof Bun.Transpiler !== "undefined") {
+  const hasBunTranspiler = typeof Bun !== "undefined" && typeof Bun.Transpiler !== "undefined";
+  if (hasBunTranspiler) {
     const transpiler = new Bun.Transpiler({ loader: "ts" });
     const stripped = transpiler.transformSync(code);
     const lineOffsets = buildLineOffsets(code, stripped);
     return { code: stripped, lineOffsets };
   }
-  throw new Error(
-    "es-check execution on TypeScript files requires Bun v1.0.0+",
-  );
+  throw new Error("es-check execution on TypeScript files requires Bun v1.0.0+");
 }
 
 function stripTypesInDeno() {
@@ -88,9 +82,7 @@ function stripTypeScript(code) {
     return stripTypesInDeno();
   }
 
-  throw new Error(
-    "es-check execution on TypeScript files is not supported in this runtime",
-  );
+  throw new Error("es-check execution on TypeScript files is not supported in this runtime");
 }
 
 function parseCode(code, acornOpts, acorn, file, options = {}) {
@@ -104,9 +96,22 @@ function parseCode(code, acornOpts, acorn, file, options = {}) {
   const shouldProcessTypeScript = hasTypeScriptOption && isTypeScriptFile;
 
   if (shouldProcessTypeScript) {
-    const result = stripTypeScript(code);
-    processedCode = result.code;
-    lineOffsets = result.lineOffsets;
+    try {
+      const result = stripTypeScript(code);
+      processedCode = result.code;
+      lineOffsets = result.lineOffsets;
+    } catch (err) {
+      return {
+        ast: null,
+        error: {
+          err,
+          stack: err.stack,
+          file,
+          line: null,
+          column: null,
+        },
+      };
+    }
   }
 
   try {

package/lib/helpers/sourcemap.js

@@ -1,7 +1,6 @@
 const fs = require("fs");
 
-const BASE64_CHARS =
-  "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/";
+const BASE64_CHARS = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/";
 const VLQ_BASE_SHIFT = 5;
 const VLQ_BASE = 1 << VLQ_BASE_SHIFT;
 const VLQ_BASE_MASK = VLQ_BASE - 1;
@@ -21,7 +20,10 @@ function decodeVLQValue(str, startIndex) {
     const reachedEnd = index >= str.length;
     if (reachedEnd) break;
 
-    const digit = base64Decode(str[index++]);
+    const char = str[index];
+    index += 1;
+
+    const digit = base64Decode(char);
     const invalidDigit = digit === -1;
     if (invalidDigit) break;
 
@@ -39,12 +41,12 @@ function decodeVLQValue(str, startIndex) {
 }
 
 function decodeVLQ(str) {
-  const values = [];
+  let values = [];
   let index = 0;
 
   while (index < str.length) {
     const decoded = decodeVLQValue(str, index);
-    values.push(decoded.value);
+    values = values.concat(decoded.value);
     index = decoded.nextIndex;
   }
 
@@ -62,20 +64,14 @@ function parseSegment(segment, state) {
   const generatedColumn = state.previousGeneratedColumn + values[0];
 
   const hasSourceIndex = values.length > 1;
-  const sourceIndex = hasSourceIndex
-    ? state.previousSourceIndex + values[1]
-    : null;
+  const sourceIndex = hasSourceIndex ? state.previousSourceIndex + values[1] : null;
   const source = hasSourceIndex ? state.sources[sourceIndex] : null;
 
   const hasOriginalLine = values.length > 2;
-  const originalLine = hasOriginalLine
-    ? state.previousSourceLine + values[2] + 1
-    : null;
+  const originalLine = hasOriginalLine ? state.previousSourceLine + values[2] + 1 : null;
 
   const hasOriginalColumn = values.length > 3;
-  const originalColumn = hasOriginalColumn
-    ? state.previousSourceColumn + values[3]
-    : null;
+  const originalColumn = hasOriginalColumn ? state.previousSourceColumn + values[3] : null;
 
   const hasNameIndex = values.length > 4;
   const nameIndex = hasNameIndex ? state.previousNameIndex + values[4] : null;
@@ -92,16 +88,20 @@ function parseSegment(segment, state) {
     name,
   };
 
-  const nextState = {
-    ...state,
+  const previousSourceIndex = sourceIndex ?? state.previousSourceIndex;
+  const previousSourceLine = hasOriginalLine
+    ? state.previousSourceLine + values[2]
+    : state.previousSourceLine;
+  const previousSourceColumn = originalColumn ?? state.previousSourceColumn;
+  const previousNameIndex = nameIndex ?? state.previousNameIndex;
+
+  const nextState = Object.assign({}, state, {
     previousGeneratedColumn: generatedColumn,
-    previousSourceIndex: sourceIndex ?? state.previousSourceIndex,
-    previousSourceLine: hasOriginalLine
-      ? state.previousSourceLine + values[2]
-      : state.previousSourceLine,
-    previousSourceColumn: originalColumn ?? state.previousSourceColumn,
-    previousNameIndex: nameIndex ?? state.previousNameIndex,
-  };
+    previousSourceIndex,
+    previousSourceLine,
+    previousSourceColumn,
+    previousNameIndex,
+  });
 
   return { mapping, nextState };
 }
@@ -111,16 +111,17 @@ function parseLine(line, state) {
   if (hasNoLine) {
     return {
       mappings: [],
-      nextState: {
-        ...state,
+      nextState: Object.assign({}, state, {
         generatedLine: state.generatedLine + 1,
         previousGeneratedColumn: 0,
-      },
+      }),
     };
   }
 
   const segments = line.split(",");
-  const initialState = { ...state, previousGeneratedColumn: 0 };
+  const initialState = Object.assign({}, state, {
+    previousGeneratedColumn: 0,
+  });
 
   const result = segments.reduce(
     (acc, segment) => {
@@ -129,7 +130,7 @@ function parseLine(line, state) {
 
       return hasMapping
         ? {
-            mappings: [...acc.mappings, parsed.mapping],
+            mappings: acc.mappings.concat(parsed.mapping),
             state: parsed.nextState,
           }
         : acc;
@@ -139,10 +140,9 @@ function parseLine(line, state) {
 
   return {
     mappings: result.mappings,
-    nextState: {
-      ...result.state,
+    nextState: Object.assign({}, result.state, {
       generatedLine: state.generatedLine + 1,
-    },
+    }),
   };
 }
 
@@ -164,7 +164,7 @@ function parseMappings(mappingsString, sources, names) {
     (acc, line) => {
       const parsed = parseLine(line, acc.state);
       return {
-        mappings: [...acc.mappings, ...parsed.mappings],
+        mappings: acc.mappings.concat(parsed.mappings),
         state: parsed.nextState,
       };
     },
@@ -185,8 +185,13 @@ function findClosestMapping(mappings, targetLine, targetColumn) {
     distance: Math.abs(mapping.generatedColumn - targetColumn),
   }));
 
-  const sorted = withDistance.sort((a, b) => a.distance - b.distance);
-  return sorted[0].mapping;
+  const closest = withDistance.reduce((best, current) => {
+    if (!best) return current;
+    const isCloser = current.distance < best.distance;
+    return isCloser ? current : best;
+  }, null);
+
+  return closest.mapping;
 }
 
 function buildSourcePath(sourceRoot, source) {
@@ -203,11 +208,7 @@ class SourceMapConsumer {
     this.mappings = sourceMapData.mappings || "";
     this.file = sourceMapData.file;
     this.sourceRoot = sourceMapData.sourceRoot || "";
-    this._parsedMappings = parseMappings(
-      this.mappings,
-      this.sources,
-      this.names,
-    );
+    this._parsedMappings = parseMappings(this.mappings, this.sources, this.names);
   }
 
   originalPositionFor({ line, column }) {