miniread 1.21.0 → 1.23.0

package/dist/transforms/_generated/manifest.js

@@ -75,6 +75,12 @@ const manifestData = {
         evaluatedAt: "2026-01-22T12:49:10.952Z",
         notes: "Auto-added by evaluation script. Measured with baseline none: 0.09%.",
     },
+    "rename-error-variables": {
+        diffReductionImpact: 0.00030157383846951547,
+        recommended: true,
+        evaluatedAt: "2026-01-24T10:58:07.496Z",
+        notes: "Auto-added by evaluation script.",
+    },
     "rename-event-parameters": {
         diffReductionImpact: 0,
         recommended: true,
@@ -101,9 +107,16 @@ const manifestData = {
         notes: "Unifies rename-loop-index-variables and rename-loop-index-variables-v2 without modifying older transforms. Measured with baseline none: 0.00%.",
     },
     "rename-parameters-to-match-properties": {
-        diffReductionImpact: 0.00003774938185385768,
+        diffReductionImpact: 0,
+        recommended: false,
+        notes: "Superseded by rename-parameters-to-match-properties-v2.",
+        supersededBy: "rename-parameters-to-match-properties-v2",
+    },
+    "rename-parameters-to-match-properties-v2": {
+        diffReductionImpact: 0.00009424182452166807,
         recommended: true,
-        notes: "Added manually based on high-confidence heuristic.",
+        evaluatedAt: "2026-01-24T09:58:42.363Z",
+        notes: "Extends v1 to also handle this.property = param assignments.",
     },
     "rename-promise-executor-parameters": {
         diffReductionImpact: 0,

package/dist/transforms/_generated/registry.js

@@ -12,11 +12,13 @@ import { renameCharcodeVariablesTransform } from "../rename-charcode-variables/r
 import { renameCharcodeVariablesV2Transform } from "../rename-charcode-variables-v2/rename-charcode-variables-v2-transform.js";
 import { renameComparisonFlagsTransform } from "../rename-comparison-flags/rename-comparison-flags-transform.js";
 import { renameDestructuredAliasesTransform } from "../rename-destructured-aliases/rename-destructured-aliases-transform.js";
+import { renameErrorVariablesTransform } from "../rename-error-variables/rename-error-variables-transform.js";
 import { renameEventParametersTransform } from "../rename-event-parameters/rename-event-parameters-transform.js";
 import { renameLoopIndexVariablesTransform } from "../rename-loop-index-variables/rename-loop-index-variables-transform.js";
 import { renameLoopIndexVariablesV2Transform } from "../rename-loop-index-variables-v2/rename-loop-index-variables-v2-transform.js";
 import { renameLoopIndexVariablesV3Transform } from "../rename-loop-index-variables-v3/rename-loop-index-variables-v3-transform.js";
 import { renameParametersToMatchPropertiesTransform } from "../rename-parameters-to-match-properties/rename-parameters-to-match-properties-transform.js";
+import { renameParametersToMatchPropertiesV2Transform } from "../rename-parameters-to-match-properties-v2/rename-parameters-to-match-properties-v2-transform.js";
 import { renamePromiseExecutorParametersTransform } from "../rename-promise-executor-parameters/rename-promise-executor-parameters-transform.js";
 import { renameReplaceChildParametersTransform } from "../rename-replace-child-parameters/rename-replace-child-parameters-transform.js";
 import { renameThisAliasesTransform } from "../rename-this-aliases/rename-this-aliases-transform.js";
@@ -39,11 +41,13 @@ export const transformRegistry = {
     [renameCharcodeVariablesV2Transform.id]: renameCharcodeVariablesV2Transform,
     [renameComparisonFlagsTransform.id]: renameComparisonFlagsTransform,
     [renameDestructuredAliasesTransform.id]: renameDestructuredAliasesTransform,
+    [renameErrorVariablesTransform.id]: renameErrorVariablesTransform,
     [renameEventParametersTransform.id]: renameEventParametersTransform,
     [renameLoopIndexVariablesTransform.id]: renameLoopIndexVariablesTransform,
     [renameLoopIndexVariablesV2Transform.id]: renameLoopIndexVariablesV2Transform,
     [renameLoopIndexVariablesV3Transform.id]: renameLoopIndexVariablesV3Transform,
     [renameParametersToMatchPropertiesTransform.id]: renameParametersToMatchPropertiesTransform,
+    [renameParametersToMatchPropertiesV2Transform.id]: renameParametersToMatchPropertiesV2Transform,
     [renamePromiseExecutorParametersTransform.id]: renamePromiseExecutorParametersTransform,
     [renameReplaceChildParametersTransform.id]: renameReplaceChildParametersTransform,
     [renameThisAliasesTransform.id]: renameThisAliasesTransform,

package/dist/transforms/rename-error-variables/manifest.json

@@ -0,0 +1,6 @@
+{
+    "diffReductionImpact": 0.00030157383846951547,
+    "recommended": true,
+    "notes": "Auto-added by evaluation script.",
+    "evaluatedAt": "2026-01-24T10:58:07.496Z"
+}

package/dist/transforms/rename-error-variables/rename-error-variables-transform.d.ts

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

package/dist/transforms/rename-error-variables/rename-error-variables-transform.js

@@ -0,0 +1,81 @@
+import { createRequire } from "node:module";
+import { isStableRenamed, RenameGroup } 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 = "error";
+const ERROR_CONSTRUCTORS = [
+    "Error",
+    "TypeError",
+    "RangeError",
+    "ReferenceError",
+    "SyntaxError",
+    "URIError",
+    "EvalError",
+    "AggregateError",
+];
+/**
+ * Checks if a node is an Error call expression and returns the constructor name.
+ * Matches: Error("message"), new Error("message"), TypeError("..."), etc.
+ */
+const getErrorConstructor = (node) => {
+    if (!node)
+        return undefined;
+    let calleeName = undefined;
+    if (node.type === "CallExpression" || node.type === "NewExpression") {
+        const callee = node.callee;
+        if (callee.type === "Identifier") {
+            calleeName = callee.name;
+        }
+    }
+    if (calleeName &&
+        ERROR_CONSTRUCTORS.includes(calleeName)) {
+        return calleeName;
+    }
+    return undefined;
+};
+export const renameErrorVariablesTransform = {
+    id: "rename-error-variables",
+    description: "Renames variables assigned error constructor calls (Error, TypeError, etc.) to $error/$error2/...",
+    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, {
+                VariableDeclarator(path) {
+                    nodesVisited++;
+                    const id = path.node.id;
+                    const init = path.node.init;
+                    // Only handle identifier bindings (not destructuring patterns)
+                    if (id.type !== "Identifier")
+                        return;
+                    // Check if the init is an Error constructor call
+                    const errorConstructor = getErrorConstructor(init);
+                    if (!errorConstructor)
+                        return;
+                    // Skip if the error constructor is locally shadowed
+                    if (path.scope.hasBinding(errorConstructor, true))
+                        return;
+                    const currentName = id.name;
+                    // Skip already-stable names
+                    if (isStableRenamed(currentName))
+                        return;
+                    group.add({
+                        scope: path.scope,
+                        currentName,
+                        baseName: BASE_NAME,
+                    });
+                },
+            });
+            transformationsApplied += group.apply();
+        }
+        return Promise.resolve({
+            nodesVisited,
+            transformationsApplied,
+        });
+    },
+};

package/dist/transforms/rename-parameters-to-match-properties/manifest.json

@@ -1,5 +1,6 @@
 {
-    "diffReductionImpact": 0.00003774938185385768,
-    "recommended": true,
-    "notes": "Added manually based on high-confidence heuristic."
+    "diffReductionImpact": 0,
+    "recommended": false,
+    "notes": "Superseded by rename-parameters-to-match-properties-v2.",
+    "supersededBy": "rename-parameters-to-match-properties-v2"
 }

package/dist/transforms/rename-parameters-to-match-properties-v2/manifest.json

@@ -0,0 +1,6 @@
+{
+    "diffReductionImpact": 0.00009424182452166807,
+    "recommended": true,
+    "notes": "Extends v1 to also handle this.property = param assignments.",
+    "evaluatedAt": "2026-01-24T09:58:42.363Z"
+}

package/dist/transforms/rename-parameters-to-match-properties-v2/rename-parameters-to-match-properties-v2-transform.d.ts

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

package/dist/transforms/rename-parameters-to-match-properties-v2/rename-parameters-to-match-properties-v2-transform.js

@@ -0,0 +1,121 @@
+import { createRequire } from "node:module";
+import { isIdentifierName, isKeyword, isStrictBindReservedWord, } from "@babel/helper-validator-identifier";
+import { getFilesToProcess, } from "../../core/types.js";
+import { isStableRenamed } from "../../core/stable-naming.js";
+const require = createRequire(import.meta.url);
+// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+const traverse = require("@babel/traverse").default;
+export const renameParametersToMatchPropertiesV2Transform = {
+    id: "rename-parameters-to-match-properties-v2",
+    description: "Renames parameters assigned to object properties or this.property",
+    scope: "file",
+    parallelizable: true,
+    transform(context) {
+        let nodesVisited = 0;
+        let transformationsApplied = 0;
+        for (const fileInfo of getFilesToProcess(context)) {
+            traverse(fileInfo.ast, {
+                Function(path) {
+                    nodesVisited++;
+                    const parameters = path.get("params");
+                    for (const parameterPath of parameters) {
+                        if (!parameterPath.isIdentifier())
+                            continue;
+                        const currentName = parameterPath.node.name;
+                        if (isStableRenamed(currentName))
+                            continue;
+                        const binding = path.scope.getBinding(currentName);
+                        if (!binding)
+                            continue;
+                        let candidateName;
+                        let isConsistent = true;
+                        // Analyze all references to this parameter
+                        for (const referencePath of binding.referencePaths) {
+                            const parent = referencePath.parentPath;
+                            // Pattern 1: Value in object property { propName: param }
+                            if (parent?.isObjectProperty()) {
+                                // Ensure the param is the value, not the key
+                                if (parent.node.value !== referencePath.node)
+                                    continue;
+                                // Get the key name
+                                const key = parent.node.key;
+                                if (key.type === "Identifier" && !parent.node.computed) {
+                                    const propertyName = key.name;
+                                    if (candidateName === undefined) {
+                                        candidateName = propertyName;
+                                    }
+                                    else if (candidateName !== propertyName) {
+                                        // Conflicting property names (e.g. { x: a, y: a })
+                                        isConsistent = false;
+                                        break;
+                                    }
+                                }
+                            }
+                            // Pattern 2: Assignment to this.property - this.propName = param
+                            // Only match simple assignment (=), not compound (+=, -=, etc.)
+                            if (parent?.isAssignmentExpression() &&
+                                parent.node.operator === "=") {
+                                // Ensure the param is the right-hand side
+                                if (parent.node.right !== referencePath.node)
+                                    continue;
+                                const left = parent.node.left;
+                                // Check if left side is this.propName (non-computed)
+                                if (left.type === "MemberExpression" &&
+                                    !left.computed &&
+                                    left.object.type === "ThisExpression" &&
+                                    left.property.type === "Identifier") {
+                                    const propertyName = left.property.name;
+                                    if (candidateName === undefined) {
+                                        candidateName = propertyName;
+                                    }
+                                    else if (candidateName !== propertyName) {
+                                        // Conflicting property names (e.g. this.x = a; this.y = a)
+                                        isConsistent = false;
+                                        break;
+                                    }
+                                }
+                            }
+                        }
+                        if (candidateName) {
+                            // Check global references (conservative: if file uses 'console', don't use 'console')
+                            const programScope = path.scope.getProgramParent();
+                            if (Object.hasOwn(programScope.globals, candidateName)) {
+                                isConsistent = false;
+                            }
+                            // Check for nested scope shadowing
+                            // If we rename `a` -> `b`, and a nested scope has `b` bound,
+                            // and we use `a` in that nested scope, it will become `b` (referring to inner `b`).
+                            const wouldBeShadowed = binding.referencePaths.some((referencePath) => referencePath.scope !== path.scope &&
+                                referencePath.scope.hasBinding(candidateName));
+                            if (wouldBeShadowed) {
+                                isConsistent = false;
+                            }
+                        }
+                        // Constraints:
+                        // 1. New name must be valid identifier
+                        // 2. New name must not be a keyword/reserved word
+                        // 3. New name must be longer than current, OR current is very short (<= 2 chars)
+                        // 4. New name must be different from current
+                        // 5. New name must not already exist in the scope
+                        if (isConsistent &&
+                            candidateName &&
+                            candidateName !== currentName &&
+                            (candidateName.length > currentName.length ||
+                                currentName.length <= 2) &&
+                            isIdentifierName(candidateName) &&
+                            !isKeyword(candidateName) &&
+                            !isStrictBindReservedWord(candidateName, true) &&
+                            !path.scope.hasBinding(candidateName)) {
+                            path.scope.rename(currentName, candidateName);
+                            transformationsApplied++;
+                        }
+                    }
+                },
+            });
+        }
+        return Promise.resolve({
+            nodesVisited,
+            transformationsApplied,
+        });
+    },
+};

package/package.json

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