miniread 1.85.0 → 1.86.0

package/dist/transforms/_generated/manifest.js

@@ -112,6 +112,10 @@ const manifestData = {
         recommended: true,
         evaluations: { "claude-code-2.1.10:claude-code-2.1.11": { "diffSizePercent": 100, "evaluatedAt": "2026-02-06T12:59:48.597Z", "changedLines": 774, "durationSeconds": 235.023578515, "stableNames": 1358 } },
     },
+    "rename-default-options-parameters-v3": {
+        recommended: true,
+        evaluations: { "claude-code-2.1.10:claude-code-2.1.11": { "diffSizePercent": 99.98303647158609, "evaluatedAt": "2026-02-06T21:30:46.281Z", "changedLines": 2568, "durationSeconds": 339.62647627599995, "stableNames": 1359 } },
+    },
     "rename-deferred-resolve-parameters": {
         recommended: true,
         notes: "Measured with baseline none: 0.00%. Renames single-parameter promise executors to $resolve when usage only stores or calls the resolver.",
@@ -482,6 +486,7 @@ export const recommendedTransformIds = [
     "rename-platform-win32-flags",
     "rename-date-now-start-times",
     "rename-default-options-parameters-v2",
+    "rename-default-options-parameters-v3",
     "rename-deferred-resolve-parameters",
     "rename-deferred-resolve-parameters-v2",
     "rename-destructured-aliases",

package/dist/transforms/_generated/registry.js

@@ -22,6 +22,7 @@ import { renameComparisonFlagsTransform } from "../rename-comparison-flags/renam
 import { renameDateNowStartTimesTransform } from "../rename-date-now-start-times/rename-date-now-start-times-transform.js";
 import { renameDefaultOptionsParametersTransform } from "../rename-default-options-parameters/rename-default-options-parameters-transform.js";
 import { renameDefaultOptionsParametersV2Transform } from "../rename-default-options-parameters-v2/rename-default-options-parameters-v2-transform.js";
+import { renameDefaultOptionsParametersV3Transform } from "../rename-default-options-parameters-v3/rename-default-options-parameters-v3-transform.js";
 import { renameDeferredResolveParametersTransform } from "../rename-deferred-resolve-parameters/rename-deferred-resolve-parameters-transform.js";
 import { renameDeferredResolveParametersV2Transform } from "../rename-deferred-resolve-parameters-v2/rename-deferred-resolve-parameters-v2-transform.js";
 import { renameDestructuredAliasesTransform } from "../rename-destructured-aliases/rename-destructured-aliases-transform.js";
@@ -114,6 +115,7 @@ export const transformRegistry = {
     [renameDateNowStartTimesTransform.id]: renameDateNowStartTimesTransform,
     [renameDefaultOptionsParametersTransform.id]: renameDefaultOptionsParametersTransform,
     [renameDefaultOptionsParametersV2Transform.id]: renameDefaultOptionsParametersV2Transform,
+    [renameDefaultOptionsParametersV3Transform.id]: renameDefaultOptionsParametersV3Transform,
     [renameDeferredResolveParametersTransform.id]: renameDeferredResolveParametersTransform,
     [renameDeferredResolveParametersV2Transform.id]: renameDeferredResolveParametersV2Transform,
     [renameDestructuredAliasesTransform.id]: renameDestructuredAliasesTransform,

package/dist/transforms/preset-stats.json

@@ -1,6 +1,6 @@
 {
     "recommended": {
-        "diffSizePercent": 30.450475921213833,
-        "notes": "Measured with baseline none: 30.45% of original diff."
+        "diffSizePercent": 30.431627556309493,
+        "notes": "Measured with baseline none: 30.43% of original diff."
     }
 }

package/dist/transforms/recommended-transform-order.d.ts

@@ -4,4 +4,4 @@
  * This lets us tune transform interactions intentionally instead of relying on
  * alphabetical ID sorting.
  */
-export declare const recommendedTransformOrder: readonly ["stabilize-top-level-bindings", "stabilize-nested-bindings", "stabilize-deferred-top-level-bindings", "stabilize-deferred-stable-rhs", "expand-boolean-literals", "expand-sequence-expressions-v5", "expand-special-number-literals", "expand-typeof-undefined-comparisons", "expand-undefined-literals", "remove-redundant-else", "rename-arguments-length-flags", "rename-asap-wrappers", "rename-awaiter-parameters", "rename-awaiter-helper-functions", "rename-buffer-variables", "rename-to-buffer-results", "rename-catch-parameters", "rename-promise-catch-parameters", "rename-char-code-at", "rename-charcode-variables-v2", "rename-client-aliases", "rename-comparison-flags", "rename-platform-win32-flags", "rename-date-now-start-times", "rename-default-options-parameters-v2", "rename-deferred-resolve-parameters", "rename-deferred-resolve-parameters-v2", "rename-destructured-aliases", "rename-rest-parameters", "rename-rest-pop-callbacks", "rename-execfile-arguments", "rename-file-reader-variables", "rename-error-first-callback-parameters", "rename-error-first-callback-parameters-v2", "rename-error-variables", "rename-event-parameters", "rename-add-event-listener-parameters", "rename-http-server-parameters", "rename-fs-sync-variables", "rename-http-method-parameters", "rename-interval-ids", "rename-indexeddb-request-variables", "rename-loop-index-variables-v3", "rename-loop-length-variables", "rename-document-fragment-variables", "rename-object-keys-variables", "rename-object-keys-iterator-variables", "rename-object-keys-reducer-parameters", "rename-object-entries-parameters", "rename-parameters-to-match-properties-v2", "rename-invalid-parameter-arguments", "rename-promise-executor-parameters-v2", "rename-range-parameters", "rename-read-file-lines", "rename-regex-builders", "rename-regex-source-parameters", "rename-search-parameters-variables", "rename-setstate-updater-parameters", "rename-file-extension-variables", "rename-string-split-variables", "rename-this-aliases", "rename-timeout-promises", "rename-timeout-ids", "rename-typeof-variables", "rename-typescript-helper-aliases", "rename-uint8array-concat-variables", "rename-url-parameters", "rename-url-variables", "rename-use-reference-guards-v2", "rename-use-reference-sets", "rename-worker-handles", "rename-zod-check-parameters", "simplify-boolean-negations", "simplify-string-trim", "split-variable-declarations", "use-optional-chaining", "use-object-property-shorthand", "use-object-shorthand", "replace-dynamic-require-eval"];
+export declare const recommendedTransformOrder: readonly ["stabilize-top-level-bindings", "stabilize-nested-bindings", "stabilize-deferred-top-level-bindings", "stabilize-deferred-stable-rhs", "expand-boolean-literals", "expand-sequence-expressions-v5", "expand-special-number-literals", "expand-typeof-undefined-comparisons", "expand-undefined-literals", "remove-redundant-else", "rename-arguments-length-flags", "rename-asap-wrappers", "rename-awaiter-parameters", "rename-awaiter-helper-functions", "rename-buffer-variables", "rename-to-buffer-results", "rename-catch-parameters", "rename-promise-catch-parameters", "rename-char-code-at", "rename-charcode-variables-v2", "rename-client-aliases", "rename-comparison-flags", "rename-platform-win32-flags", "rename-date-now-start-times", "rename-default-options-parameters-v2", "rename-default-options-parameters-v3", "rename-deferred-resolve-parameters", "rename-deferred-resolve-parameters-v2", "rename-destructured-aliases", "rename-rest-parameters", "rename-rest-pop-callbacks", "rename-execfile-arguments", "rename-file-reader-variables", "rename-error-first-callback-parameters", "rename-error-first-callback-parameters-v2", "rename-error-variables", "rename-event-parameters", "rename-add-event-listener-parameters", "rename-http-server-parameters", "rename-fs-sync-variables", "rename-http-method-parameters", "rename-interval-ids", "rename-indexeddb-request-variables", "rename-loop-index-variables-v3", "rename-loop-length-variables", "rename-document-fragment-variables", "rename-object-keys-variables", "rename-object-keys-iterator-variables", "rename-object-keys-reducer-parameters", "rename-object-entries-parameters", "rename-parameters-to-match-properties-v2", "rename-invalid-parameter-arguments", "rename-promise-executor-parameters-v2", "rename-range-parameters", "rename-read-file-lines", "rename-regex-builders", "rename-regex-source-parameters", "rename-search-parameters-variables", "rename-setstate-updater-parameters", "rename-file-extension-variables", "rename-string-split-variables", "rename-this-aliases", "rename-timeout-promises", "rename-timeout-ids", "rename-typeof-variables", "rename-typescript-helper-aliases", "rename-uint8array-concat-variables", "rename-url-parameters", "rename-url-variables", "rename-use-reference-guards-v2", "rename-use-reference-sets", "rename-worker-handles", "rename-zod-check-parameters", "simplify-boolean-negations", "simplify-string-trim", "split-variable-declarations", "use-optional-chaining", "use-object-property-shorthand", "use-object-shorthand", "replace-dynamic-require-eval"];

package/dist/transforms/recommended-transform-order.js

@@ -39,6 +39,7 @@ export const recommendedTransformOrder = [
     "rename-platform-win32-flags",
     "rename-date-now-start-times",
     "rename-default-options-parameters-v2",
+    "rename-default-options-parameters-v3",
     "rename-deferred-resolve-parameters",
     "rename-deferred-resolve-parameters-v2",
     "rename-destructured-aliases",

package/dist/transforms/rename-default-options-parameters-v3/manifest.json

@@ -0,0 +1,12 @@
+{
+    "recommended": true,
+    "evaluations": {
+        "claude-code-2.1.10:claude-code-2.1.11": {
+            "diffSizePercent": 99.98303647158609,
+            "evaluatedAt": "2026-02-06T21:30:46.281Z",
+            "changedLines": 2568,
+            "durationSeconds": 339.62647627599995,
+            "stableNames": 1359
+        }
+    }
+}

package/dist/transforms/rename-default-options-parameters-v3/rename-default-options-parameters-v3-transform.d.ts

@@ -0,0 +1,2 @@
+import { type Transform } from "../../core/types.js";
+export declare const renameDefaultOptionsParametersV3Transform: Transform;

package/dist/transforms/rename-default-options-parameters-v3/rename-default-options-parameters-v3-transform.js

@@ -0,0 +1,58 @@
+import { createRequire } from "node:module";
+import * as t from "@babel/types";
+import { RenameGroup, isStableRenamed } from "../../core/stable-naming.js";
+import { getFilesToProcess, } from "../../core/types.js";
+const require = createRequire(import.meta.url);
+// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+const traverse = require("@babel/traverse").default;
+const BASE_NAME = "options";
+const collectDefaultOptionsParameterNames = (path) => {
+    const parameterNames = [];
+    for (const parameter of path.node.params) {
+        if (!t.isAssignmentPattern(parameter))
+            continue;
+        if (!t.isIdentifier(parameter.left))
+            continue;
+        if (!t.isObjectExpression(parameter.right))
+            continue;
+        if (parameter.right.properties.length > 0)
+            continue;
+        parameterNames.push(parameter.left.name);
+    }
+    return parameterNames;
+};
+export const renameDefaultOptionsParametersV3Transform = {
+    id: "rename-default-options-parameters-v3",
+    description: "Renames parameters defaulted to {} into stable options names",
+    scope: "file",
+    parallelizable: true,
+    transform(context) {
+        let nodesVisited = 0;
+        let transformationsApplied = 0;
+        for (const fileInfo of getFilesToProcess(context)) {
+            const group = new RenameGroup();
+            traverse(fileInfo.ast, {
+                Function(path) {
+                    nodesVisited++;
+                    const parameterNames = collectDefaultOptionsParameterNames(path);
+                    for (const parameterName of parameterNames) {
+                        if (parameterName === BASE_NAME)
+                            continue;
+                        if (isStableRenamed(parameterName))
+                            continue;
+                        group.add({
+                            scope: path.scope,
+                            currentName: parameterName,
+                            baseName: BASE_NAME,
+                        });
+                    }
+                },
+            });
+            transformationsApplied += group.apply();
+        }
+        return Promise.resolve({
+            nodesVisited,
+            transformationsApplied,
+        });
+    },
+};

package/dist/transforms/rename-deferred-resolve-parameters/rename-binding-if-safe.d.ts

@@ -1,2 +1,2 @@
-import type { NodePath } from "@babel/traverse";
-export declare const renameBindingIfSafe: (bindingScope: NodePath["scope"], fromName: string, toName: string) => boolean;
+import type { Scope } from "@babel/traverse";
+export declare const renameBindingIfSafe: (bindingScope: Scope, fromName: string, toName: string) => boolean;

package/dist/transforms/rename-deferred-resolve-parameters/rename-binding-if-safe.js

@@ -1,22 +1,4 @@
-import { hasShadowingRisk } from "../../core/has-shadowing-risk.js";
-import { isValidBindingIdentifier } from "./is-valid-binding-identifier.js";
-const canRenameBindingSafely = (bindingScope, fromName, toName) => {
-    if (fromName === toName)
-        return false;
-    if (!isValidBindingIdentifier(toName))
-        return false;
-    if (bindingScope.hasOwnBinding(toName))
-        return false;
-    const programScope = bindingScope.getProgramParent();
-    if (Object.hasOwn(programScope.globals, toName))
-        return false;
-    const binding = bindingScope.getBinding(fromName);
-    if (!binding)
-        return false;
-    if (hasShadowingRisk(binding, bindingScope, toName))
-        return false;
-    return true;
-};
+import { canRenameBindingSafely } from "../shared/can-rename-binding-safely.js";
 export const renameBindingIfSafe = (bindingScope, fromName, toName) => {
     if (!canRenameBindingSafely(bindingScope, fromName, toName))
         return false;

package/dist/transforms/rename-deferred-resolve-parameters/single-executor-heuristics.d.ts

@@ -3,4 +3,4 @@ import type { ArrowFunctionExpression, FunctionExpression, NewExpression } from
 export declare const getSingleExecutorFunctionIfEligible: (path: NodePath<NewExpression>) => NodePath<FunctionExpression | ArrowFunctionExpression> | undefined;
 export declare const isDirectCallOfBinding: (referencePath: NodePath) => boolean;
 export declare const isAssignedToIdentifier: (referencePath: NodePath, bindingName: string) => boolean;
-export declare const wouldShadowReferencedOuterBinding: (executorPath: NodePath<FunctionExpression | ArrowFunctionExpression>, targetName: string) => boolean;
+export { wouldShadowReferencedOuterBinding } from "../shared/would-shadow-referenced-outer-binding.js";

package/dist/transforms/rename-deferred-resolve-parameters/single-executor-heuristics.js

@@ -1,6 +1,6 @@
-import { isValidBindingIdentifier } from "./is-valid-binding-identifier.js";
-const isExecutorParameterEligible = (parameter) => {
-    if (!isValidBindingIdentifier(parameter.name))
+import { isValidBindingIdentifier } from "../shared/is-valid-binding-identifier.js";
+const isExecutorParameterEligible = (parameterName) => {
+    if (!isValidBindingIdentifier(parameterName))
         return false;
     return true;
 };
@@ -20,7 +20,7 @@ export const getSingleExecutorFunctionIfEligible = (path) => {
         return;
     if (parameter.type !== "Identifier")
         return;
-    if (!isExecutorParameterEligible(parameter))
+    if (!isExecutorParameterEligible(parameter.name))
         return;
     return executor;
 };
@@ -48,24 +48,4 @@ export const isAssignedToIdentifier = (referencePath, bindingName) => {
     }
     return false;
 };
-export const wouldShadowReferencedOuterBinding = (executorPath, targetName) => {
-    const parentBinding = executorPath.scope.getBinding(targetName);
-    if (!parentBinding)
-        return false;
-    let foundReference = false;
-    executorPath.traverse({
-        Identifier(identifierPath) {
-            if (foundReference)
-                return;
-            if (identifierPath.node.name !== targetName)
-                return;
-            if (!identifierPath.isReferencedIdentifier())
-                return;
-            if (identifierPath.scope.getBinding(targetName) !== parentBinding)
-                return;
-            foundReference = true;
-            identifierPath.stop();
-        },
-    });
-    return foundReference;
-};
+export { wouldShadowReferencedOuterBinding } from "../shared/would-shadow-referenced-outer-binding.js";

package/dist/transforms/rename-event-parameters/process-event-handler-function.js

@@ -1,8 +1,8 @@
 import { isStableRenamed } from "../../core/stable-naming.js";
+import { canRenameBindingSafely } from "../shared/can-rename-binding-safely.js";
+import { wouldShadowReferencedOuterBinding } from "../shared/would-shadow-referenced-outer-binding.js";
 import { getEventParameterUsage, isHighConfidenceEventParameter, } from "./event-parameter-usage.js";
 import { getTargetEventName } from "./get-target-event-name.js";
-import { canRenameBindingSafely } from "./can-rename-binding-safely.js";
-import { wouldShadowReferencedOuterBinding } from "./would-shadow-referenced-outer-binding.js";
 const wouldShadowReferencedImplicitGlobal = (functionPath, targetName) => {
     const programScope = functionPath.scope.getProgramParent();
     if (!Object.hasOwn(programScope.globals, targetName))
@@ -53,8 +53,11 @@ export const processEventHandlerFunction = (path, group) => {
         return;
     if (wouldShadowReferencedOuterBinding(path, targetName))
         return;
-    if (!canRenameBindingSafely(bindingScope, currentName, targetName))
+    if (!canRenameBindingSafely(bindingScope, currentName, targetName, {
+        checkGlobals: false,
+    })) {
         return;
+    }
     group.add({
         scope: bindingScope,
         currentName,

package/dist/transforms/rename-promise-executor-parameters/promise-executor-heuristics.d.ts

@@ -3,4 +3,3 @@ import type { ArrowFunctionExpression, FunctionExpression, NewExpression } from
 export declare const getExecutorFunctionIfEligible: (path: NodePath<NewExpression>) => NodePath<FunctionExpression | ArrowFunctionExpression> | undefined;
 export declare const isDirectCallOfBinding: (referencePath: NodePath) => boolean;
 export declare const isAssignedToOnHandlerProperty: (referencePath: NodePath) => boolean;
-export declare const wouldShadowReferencedOuterBinding: (executorPath: NodePath<FunctionExpression | ArrowFunctionExpression>, targetName: string) => boolean;

package/dist/transforms/rename-promise-executor-parameters/promise-executor-heuristics.js

@@ -1,11 +1,11 @@
-import { isValidBindingIdentifier } from "./is-valid-binding-identifier.js";
+import { isValidBindingIdentifier } from "../shared/is-valid-binding-identifier.js";
 const allowedOnHandlerPropertyNames = new Set([
     "onabort",
     "onerror",
     "ontimeout",
 ]);
-const isExecutorParameterEligible = (parameter) => {
-    if (!isValidBindingIdentifier(parameter.name))
+const isExecutorParameterEligible = (parameterName) => {
+    if (!isValidBindingIdentifier(parameterName))
         return false;
     return true;
 };
@@ -30,9 +30,9 @@ export const getExecutorFunctionIfEligible = (path) => {
         return;
     if (secondParameter.type !== "Identifier")
         return;
-    if (!isExecutorParameterEligible(firstParameter))
+    if (!isExecutorParameterEligible(firstParameter.name))
         return;
-    if (!isExecutorParameterEligible(secondParameter))
+    if (!isExecutorParameterEligible(secondParameter.name))
         return;
     return executor;
 };
@@ -61,24 +61,3 @@ export const isAssignedToOnHandlerProperty = (referencePath) => {
         return false;
     return true;
 };
-export const wouldShadowReferencedOuterBinding = (executorPath, targetName) => {
-    const parentBinding = executorPath.scope.getBinding(targetName);
-    if (!parentBinding)
-        return false;
-    let foundReference = false;
-    executorPath.traverse({
-        Identifier(identifierPath) {
-            if (foundReference)
-                return;
-            if (identifierPath.node.name !== targetName)
-                return;
-            if (!identifierPath.isReferencedIdentifier())
-                return;
-            if (identifierPath.scope.getBinding(targetName) !== parentBinding)
-                return;
-            foundReference = true;
-            identifierPath.stop();
-        },
-    });
-    return foundReference;
-};

package/dist/transforms/rename-promise-executor-parameters/rename-binding-if-safe.d.ts

@@ -1,3 +1,3 @@
-import type { NodePath } from "@babel/traverse";
-export declare const canRenameBindingSafely: (bindingScope: NodePath["scope"], fromName: string, toName: string) => boolean;
-export declare const renameBindingIfSafe: (bindingScope: NodePath["scope"], fromName: string, toName: string) => boolean;
+import type { Scope } from "@babel/traverse";
+export declare const canRenamePromiseBindingSafely: (bindingScope: Scope, fromName: string, toName: string) => boolean;
+export declare const renameBindingIfSafe: (bindingScope: Scope, fromName: string, toName: string) => boolean;

package/dist/transforms/rename-promise-executor-parameters/rename-binding-if-safe.js

@@ -1,27 +1,11 @@
-import { hasShadowingRisk } from "../../core/has-shadowing-risk.js";
-import { isValidBindingIdentifier } from "./is-valid-binding-identifier.js";
-export const canRenameBindingSafely = (bindingScope, fromName, toName) => {
-    if (fromName === toName)
-        return false;
-    if (!isValidBindingIdentifier(toName))
-        return false;
-    if (bindingScope.hasOwnBinding(toName))
-        return false;
-    const programScope = bindingScope.getProgramParent();
-    // `globals` contains only referenced (unbound) globals for this file, so this does not block
-    // in files where the identifier is never used.
-    if (Object.hasOwn(programScope.globals, toName))
-        return false;
-    const binding = bindingScope.getBinding(fromName);
-    if (!binding)
-        return false;
-    if (hasShadowingRisk(binding, bindingScope, toName))
-        return false;
-    return true;
+import { canRenameBindingSafely } from "../shared/can-rename-binding-safely.js";
+export const canRenamePromiseBindingSafely = (bindingScope, fromName, toName) => {
+    return canRenameBindingSafely(bindingScope, fromName, toName);
 };
 export const renameBindingIfSafe = (bindingScope, fromName, toName) => {
-    if (!canRenameBindingSafely(bindingScope, fromName, toName))
+    if (!canRenamePromiseBindingSafely(bindingScope, fromName, toName)) {
         return false;
+    }
     bindingScope.rename(fromName, toName);
     return true;
 };

package/dist/transforms/rename-promise-executor-parameters/rename-promise-executor-parameters-transform.js

@@ -1,8 +1,9 @@
 import { createRequire } from "node:module";
 import { isStableRenamed } from "../../core/stable-naming.js";
 import { getFilesToProcess, } from "../../core/types.js";
-import { getExecutorFunctionIfEligible, isAssignedToOnHandlerProperty, isDirectCallOfBinding, wouldShadowReferencedOuterBinding, } from "./promise-executor-heuristics.js";
-import { canRenameBindingSafely, renameBindingIfSafe, } from "./rename-binding-if-safe.js";
+import { getExecutorFunctionIfEligible, isAssignedToOnHandlerProperty, isDirectCallOfBinding, } from "./promise-executor-heuristics.js";
+import { canRenamePromiseBindingSafely, renameBindingIfSafe, } from "./rename-binding-if-safe.js";
+import { wouldShadowReferencedOuterBinding } from "../shared/would-shadow-referenced-outer-binding.js";
 const STABLE_PREFIX = "$";
 const require = createRequire(import.meta.url);
 // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
@@ -56,10 +57,10 @@ export const renamePromiseExecutorParametersTransform = {
                     const wantsRejectRename = rejectParameter.name !== targetRejectName;
                     const canRenameResolve = !wantsResolveRename ||
                         (!resolveWouldShadow &&
-                            canRenameBindingSafely(bindingScope, resolveParameter.name, targetResolveName));
+                            canRenamePromiseBindingSafely(bindingScope, resolveParameter.name, targetResolveName));
                     const canRenameReject = !wantsRejectRename ||
                         (!rejectWouldShadow &&
-                            canRenameBindingSafely(bindingScope, rejectParameter.name, targetRejectName));
+                            canRenamePromiseBindingSafely(bindingScope, rejectParameter.name, targetRejectName));
                     // Avoid ending up with inconsistent executor signatures like `(WA, reject)` —
                     // only rename when both parameters are already correct or can be renamed safely.
                     if (!canRenameResolve || !canRenameReject)

package/dist/transforms/rename-promise-executor-parameters-v2/promise-executor-heuristics.d.ts

@@ -5,4 +5,3 @@ export declare const isDirectCallOfBinding: (referencePath: NodePath) => boolean
 export declare const isAssignedToOnHandlerProperty: (referencePath: NodePath) => boolean;
 export declare const isErrorEventHandlerArgument: (referencePath: NodePath) => boolean;
 export declare const isErrorEventHandlerRemovalArgument: (referencePath: NodePath) => boolean;
-export declare const wouldShadowReferencedOuterBinding: (executorPath: NodePath<FunctionExpression | ArrowFunctionExpression>, targetName: string) => boolean;

package/dist/transforms/rename-promise-executor-parameters-v2/promise-executor-heuristics.js

@@ -122,24 +122,3 @@ export const isErrorEventHandlerRemovalArgument = (referencePath) => {
     }
     return true;
 };
-export const wouldShadowReferencedOuterBinding = (executorPath, targetName) => {
-    const parentBinding = executorPath.scope.getBinding(targetName);
-    if (!parentBinding)
-        return false;
-    let foundReference = false;
-    executorPath.traverse({
-        Identifier(identifierPath) {
-            if (foundReference)
-                return;
-            if (identifierPath.node.name !== targetName)
-                return;
-            if (!identifierPath.isReferencedIdentifier())
-                return;
-            if (identifierPath.scope.getBinding(targetName) !== parentBinding)
-                return;
-            foundReference = true;
-            identifierPath.stop();
-        },
-    });
-    return foundReference;
-};

package/dist/transforms/rename-promise-executor-parameters-v2/rename-promise-executor-parameters-v2-transform.js

@@ -1,8 +1,9 @@
 import { createRequire } from "node:module";
 import { isStableRenamed } from "../../core/stable-naming.js";
 import { getFilesToProcess, } from "../../core/types.js";
-import { getExecutorFunctionIfEligible, isAssignedToOnHandlerProperty, isDirectCallOfBinding, isErrorEventHandlerArgument, isErrorEventHandlerRemovalArgument, wouldShadowReferencedOuterBinding, } from "./promise-executor-heuristics.js";
-import { canRenameBindingSafely } from "./rename-binding-if-safe.js";
+import { getExecutorFunctionIfEligible, isAssignedToOnHandlerProperty, isDirectCallOfBinding, isErrorEventHandlerArgument, isErrorEventHandlerRemovalArgument, } from "./promise-executor-heuristics.js";
+import { canRenameBindingSafely } from "../shared/can-rename-binding-safely.js";
+import { wouldShadowReferencedOuterBinding } from "../shared/would-shadow-referenced-outer-binding.js";
 const STABLE_PREFIX = "$";
 const require = createRequire(import.meta.url);
 // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
@@ -92,10 +93,10 @@ export const renamePromiseExecutorParametersV2Transform = {
                     const fromRejectName = rejectParameter.name;
                     const canRenameResolve = !wantsResolveRename ||
                         (!resolveWouldShadow &&
-                            canRenameBindingSafely(bindingScope, fromResolveName, targetResolveName));
+                            canRenameBindingSafely(bindingScope, fromResolveName, targetResolveName, { strictMode: false }));
                     const canRenameReject = !wantsRejectRename ||
                         (!rejectWouldShadow &&
-                            canRenameBindingSafely(bindingScope, fromRejectName, targetRejectName));
+                            canRenameBindingSafely(bindingScope, fromRejectName, targetRejectName, { strictMode: false }));
                     // Avoid ending up with inconsistent executor signatures like `(WA, reject)` —
                     // only rename when both parameters are already correct or can be renamed safely.
                     if (!canRenameResolve || !canRenameReject)

package/dist/transforms/shared/can-rename-binding-safely.d.ts

@@ -0,0 +1,8 @@
+import type { Scope } from "@babel/traverse";
+type CanRenameBindingSafelyOptions = {
+    checkGlobals?: boolean;
+    strictMode?: boolean;
+    validateIdentifier?: boolean;
+};
+export declare const canRenameBindingSafely: (bindingScope: Scope, fromName: string, toName: string, options?: CanRenameBindingSafelyOptions) => boolean;
+export {};

package/dist/transforms/{rename-event-parameters → shared}/can-rename-binding-safely.js

@@ -1,12 +1,21 @@
 import { hasShadowingRisk } from "../../core/has-shadowing-risk.js";
 import { isValidBindingIdentifier } from "./is-valid-binding-identifier.js";
-export const canRenameBindingSafely = (bindingScope, fromName, toName) => {
+export const canRenameBindingSafely = (bindingScope, fromName, toName, options = {}) => {
+    const checkGlobals = options.checkGlobals ?? true;
+    const strictMode = options.strictMode ?? true;
+    const validateIdentifier = options.validateIdentifier ?? true;
     if (fromName === toName)
         return false;
-    if (!isValidBindingIdentifier(toName))
+    if (validateIdentifier && !isValidBindingIdentifier(toName, strictMode)) {
         return false;
+    }
     if (bindingScope.hasOwnBinding(toName))
         return false;
+    if (checkGlobals) {
+        const programScope = bindingScope.getProgramParent();
+        if (Object.hasOwn(programScope.globals, toName))
+            return false;
+    }
     const binding = bindingScope.getBinding(fromName);
     if (!binding)
         return false;

package/dist/transforms/shared/is-valid-binding-identifier.d.ts

@@ -0,0 +1 @@
+export declare const isValidBindingIdentifier: (name: string, strictMode?: boolean) => boolean;

package/dist/transforms/{rename-event-parameters → shared}/is-valid-binding-identifier.js

@@ -1,10 +1,10 @@
 import { isIdentifierName, isKeyword, isStrictBindReservedWord, } from "@babel/helper-validator-identifier";
-export const isValidBindingIdentifier = (name) => {
+export const isValidBindingIdentifier = (name, strictMode = true) => {
     if (!isIdentifierName(name))
         return false;
     if (isKeyword(name))
         return false;
-    if (isStrictBindReservedWord(name, true))
+    if (isStrictBindReservedWord(name, strictMode))
         return false;
     return true;
 };

package/dist/transforms/stabilize-deferred-stable-rhs/stabilize-deferred-stable-rhs-transform.js

@@ -5,7 +5,8 @@ import * as t from "@babel/types";
 import { getFilesToProcess, } from "../../core/types.js";
 import { generateCode } from "../../cli/generate-code.js";
 import { isHashBasedStableName } from "../../core/stable-naming.js";
-import { canRenameBindingSafely, findEnclosingBindingName, renameBindingInPlace, } from "../stabilize-deferred-top-level-bindings/rename-helpers.js";
+import { findEnclosingBindingName, renameBindingInPlace, } from "../stabilize-deferred-top-level-bindings/rename-helpers.js";
+import { canRenameBindingSafely } from "../shared/can-rename-binding-safely.js";
 import { hasAllStableReferences } from "./check-rhs-stability.js";
 const require = createRequire(import.meta.url);
 // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access

package/dist/transforms/stabilize-deferred-top-level-bindings/rename-helpers.d.ts

@@ -12,10 +12,6 @@ export declare const extractHashFromStableName: (name: string) => string | undef
  * afterwards to refresh scope state.
  */
 export declare const renameBindingInPlace: (bindingScope: Scope, fromName: string, toName: string) => boolean;
-/**
- * Check if we can safely rename a binding.
- */
-export declare const canRenameBindingSafely: (bindingScope: Scope, fromName: string, toName: string) => boolean;
 /**
  * Walk up the AST to find the binding name of the enclosing function.
  * Handles `var $h_xxx = () => { ... }`, `$h_xxx = fn(() => { ... })`,

package/dist/transforms/stabilize-deferred-top-level-bindings/rename-helpers.js

@@ -1,5 +1,4 @@
 import * as t from "@babel/types";
-import { hasShadowingRisk } from "../../core/has-shadowing-risk.js";
 /**
  * Extract the hash suffix from a stable function name (`$h_` or `$f_` prefix).
  * Does not match `$v_` names — those are deferred variables, not functions.
@@ -48,24 +47,6 @@ export const renameBindingInPlace = (bindingScope, fromName, toName) => {
     }
     return true;
 };
-/**
- * Check if we can safely rename a binding.
- */
-export const canRenameBindingSafely = (bindingScope, fromName, toName) => {
-    if (fromName === toName)
-        return false;
-    if (bindingScope.hasOwnBinding(toName))
-        return false;
-    const programScope = bindingScope.getProgramParent();
-    if (Object.hasOwn(programScope.globals, toName))
-        return false;
-    const binding = bindingScope.getBinding(fromName);
-    if (!binding)
-        return false;
-    if (hasShadowingRisk(binding, bindingScope, toName))
-        return false;
-    return true;
-};
 /**
  * Walk up the AST to find the binding name of the enclosing function.
  * Handles `var $h_xxx = () => { ... }`, `$h_xxx = fn(() => { ... })`,

package/dist/transforms/stabilize-deferred-top-level-bindings/stabilize-deferred-top-level-bindings-transform.js

@@ -3,7 +3,8 @@ import { parse } from "@babel/parser";
 import { getFilesToProcess, } from "../../core/types.js";
 import { generateCode } from "../../cli/generate-code.js";
 import { collectDeferredVariables, filterToSingleVariableFunctions, } from "./collect-deferred-variables.js";
-import { canRenameBindingSafely, extractHashFromStableName, renameBindingInPlace, } from "./rename-helpers.js";
+import { extractHashFromStableName, renameBindingInPlace, } from "./rename-helpers.js";
+import { canRenameBindingSafely } from "../shared/can-rename-binding-safely.js";
 const require = createRequire(import.meta.url);
 // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
 const traverse = require("@babel/traverse").default;
@@ -45,8 +46,11 @@ export const stabilizeDeferredTopLevelBindingsTransform = {
                 const newName = `$v_${hash}`;
                 if (usedNewNames.has(newName))
                     continue;
-                if (!canRenameBindingSafely(scope, variableName, newName))
+                if (!canRenameBindingSafely(scope, variableName, newName, {
+                    validateIdentifier: false,
+                })) {
                     continue;
+                }
                 if (renameBindingInPlace(scope, variableName, newName)) {
                     usedNewNames.add(newName);
                     fileTransformations++;

package/dist/transforms/stabilize-top-level-bindings/stabilize-top-level-bindings-transform.js

@@ -2,7 +2,7 @@ import { parse } from "@babel/parser";
 import { collectExportedNames } from "../../core/collect-exported-names.js";
 import { getFilesToProcess, } from "../../core/types.js";
 import { generateCode } from "../../cli/generate-code.js";
-import { canRenameBindingSafely } from "./can-rename-binding-safely.js";
+import { canRenameBindingSafely } from "../shared/can-rename-binding-safely.js";
 import { collectRenameCandidates } from "./collect-rename-candidates.js";
 import { detectDynamicNameLookup } from "../dynamic-name-lookup/detect-dynamic-name-lookup.js";
 import { renameBindingInPlace } from "./rename-binding-in-place.js";

package/package.json

@@ -2,7 +2,7 @@
   "name": "miniread",
   "author": "Łukasz Jerciński",
   "license": "MIT",
-  "version": "1.85.0",
+  "version": "1.86.0",
   "description": "Transform minified JavaScript/TypeScript into a more readable form using deterministic AST-based transforms.",
   "repository": {
     "type": "git",

package/dist/transforms/rename-deferred-resolve-parameters/is-valid-binding-identifier.d.ts

@@ -1 +0,0 @@
-export declare const isValidBindingIdentifier: (name: string) => boolean;

package/dist/transforms/rename-deferred-resolve-parameters/is-valid-binding-identifier.js

@@ -1,10 +0,0 @@
-import { isIdentifierName, isKeyword, isStrictBindReservedWord, } from "@babel/helper-validator-identifier";
-export const isValidBindingIdentifier = (name) => {
-    if (!isIdentifierName(name))
-        return false;
-    if (isKeyword(name))
-        return false;
-    if (isStrictBindReservedWord(name, true))
-        return false;
-    return true;
-};

package/dist/transforms/rename-event-parameters/can-rename-binding-safely.d.ts

@@ -1,2 +0,0 @@
-import type { Scope } from "@babel/traverse";
-export declare const canRenameBindingSafely: (bindingScope: Scope, fromName: string, toName: string) => boolean;

package/dist/transforms/rename-event-parameters/is-valid-binding-identifier.d.ts

@@ -1 +0,0 @@
-export declare const isValidBindingIdentifier: (name: string) => boolean;

package/dist/transforms/rename-promise-executor-parameters/is-valid-binding-identifier.d.ts

@@ -1 +0,0 @@
-export declare const isValidBindingIdentifier: (name: string) => boolean;

package/dist/transforms/rename-promise-executor-parameters/is-valid-binding-identifier.js

@@ -1,10 +0,0 @@
-import { isIdentifierName, isKeyword, isStrictBindReservedWord, } from "@babel/helper-validator-identifier";
-export const isValidBindingIdentifier = (name) => {
-    if (!isIdentifierName(name))
-        return false;
-    if (isKeyword(name))
-        return false;
-    if (isStrictBindReservedWord(name, true))
-        return false;
-    return true;
-};

package/dist/transforms/rename-promise-executor-parameters-v2/is-valid-binding-identifier.d.ts

@@ -1 +0,0 @@
-export declare const isValidBindingIdentifier: (name: string) => boolean;

package/dist/transforms/rename-promise-executor-parameters-v2/is-valid-binding-identifier.js

@@ -1,13 +0,0 @@
-import { isIdentifierName, isKeyword, isStrictBindReservedWord, } from "@babel/helper-validator-identifier";
-export const isValidBindingIdentifier = (name) => {
-    if (!isIdentifierName(name))
-        return false;
-    if (isKeyword(name))
-        return false;
-    // Parse uses `sourceType: "unambiguous"` which often yields `script`; in that mode,
-    // some module-only reserved words (e.g. `await`) can be valid identifiers.
-    // When code is parsed as a module, invalid bindings are rejected by the parser anyway.
-    if (isStrictBindReservedWord(name, false))
-        return false;
-    return true;
-};

package/dist/transforms/rename-promise-executor-parameters-v2/rename-binding-if-safe.d.ts

@@ -1,2 +0,0 @@
-import type { NodePath } from "@babel/traverse";
-export declare const canRenameBindingSafely: (bindingScope: NodePath["scope"], fromName: string, toName: string) => boolean;

package/dist/transforms/rename-promise-executor-parameters-v2/rename-binding-if-safe.js

@@ -1,21 +0,0 @@
-import { hasShadowingRisk } from "../../core/has-shadowing-risk.js";
-import { isValidBindingIdentifier } from "./is-valid-binding-identifier.js";
-export const canRenameBindingSafely = (bindingScope, fromName, toName) => {
-    if (fromName === toName)
-        return false;
-    if (!isValidBindingIdentifier(toName))
-        return false;
-    if (bindingScope.hasOwnBinding(toName))
-        return false;
-    const programScope = bindingScope.getProgramParent();
-    // `globals` contains only referenced (unbound) globals for this file, so this does not block
-    // in files where the identifier is never used.
-    if (Object.hasOwn(programScope.globals, toName))
-        return false;
-    const binding = bindingScope.getBinding(fromName);
-    if (!binding)
-        return false;
-    if (hasShadowingRisk(binding, bindingScope, toName))
-        return false;
-    return true;
-};

package/dist/transforms/stabilize-top-level-bindings/can-rename-binding-safely.d.ts

@@ -1,2 +0,0 @@
-import type { Scope } from "@babel/traverse";
-export declare const canRenameBindingSafely: (bindingScope: Scope, fromName: string, toName: string) => boolean;

package/dist/transforms/stabilize-top-level-bindings/can-rename-binding-safely.js

@@ -1,24 +0,0 @@
-import { hasShadowingRisk } from "../../core/has-shadowing-risk.js";
-import { isValidBindingIdentifier } from "./is-valid-binding-identifier.js";
-export const canRenameBindingSafely = (bindingScope, fromName, toName) => {
-    if (fromName === toName)
-        return false; // no rename needed
-    if (!isValidBindingIdentifier(toName))
-        return false;
-    if (bindingScope.hasOwnBinding(toName))
-        return false;
-    // This transform renames only program-scope bindings; nested-scope collisions are
-    // prevented via `hasShadowingRisk` below.
-    const programScope = bindingScope.getProgramParent();
-    // NOTE: `toName` always uses the `$h_` prefix, so it can't collide with common
-    // builtin globals like `window`/`document` even if they aren't referenced. This
-    // check is about avoiding accidental capture of *referenced* globals.
-    if (Object.hasOwn(programScope.globals, toName))
-        return false;
-    const binding = bindingScope.getBinding(fromName);
-    if (!binding)
-        return false;
-    if (hasShadowingRisk(binding, bindingScope, toName))
-        return false;
-    return true;
-};

package/dist/transforms/stabilize-top-level-bindings/is-valid-binding-identifier.d.ts

@@ -1 +0,0 @@
-export declare const isValidBindingIdentifier: (name: string) => boolean;

package/dist/transforms/stabilize-top-level-bindings/is-valid-binding-identifier.js

@@ -1,10 +0,0 @@
-import { isIdentifierName, isKeyword, isStrictBindReservedWord, } from "@babel/helper-validator-identifier";
-export const isValidBindingIdentifier = (name) => {
-    if (!isIdentifierName(name))
-        return false;
-    if (isKeyword(name))
-        return false;
-    if (isStrictBindReservedWord(name, true))
-        return false;
-    return true;
-};