miniread 1.35.0 → 1.36.0

package/dist/core/collect-exported-names.d.ts

@@ -0,0 +1,2 @@
+import type { Program } from "@babel/types";
+export declare const collectExportedNames: (program: Program) => Set<string>;

package/dist/core/collect-exported-names.js

@@ -0,0 +1,68 @@
+import { isArrayPattern, isAssignmentPattern, isClassDeclaration, isExportNamedDeclaration, isExportSpecifier, isFunctionDeclaration, isIdentifier, isObjectPattern, isObjectProperty, isRestElement, isVariableDeclaration, } from "@babel/types";
+const addBindingNamesFromNode = (node, out) => {
+    const babelNode = node;
+    if (!babelNode)
+        return;
+    if (babelNode.type === "VoidPattern")
+        return;
+    if (isIdentifier(babelNode)) {
+        out.add(babelNode.name);
+        return;
+    }
+    if (isObjectPattern(babelNode)) {
+        for (const property of babelNode.properties) {
+            if (isRestElement(property)) {
+                addBindingNamesFromNode(property.argument, out);
+                continue;
+            }
+            if (!isObjectProperty(property))
+                continue;
+            addBindingNamesFromNode(property.value, out);
+        }
+        return;
+    }
+    if (isArrayPattern(babelNode)) {
+        for (const element of babelNode.elements) {
+            addBindingNamesFromNode(element, out);
+        }
+        return;
+    }
+    if (isRestElement(babelNode)) {
+        addBindingNamesFromNode(babelNode.argument, out);
+        return;
+    }
+    if (isAssignmentPattern(babelNode)) {
+        addBindingNamesFromNode(babelNode.left, out);
+    }
+};
+export const collectExportedNames = (program) => {
+    const exportedNames = new Set();
+    for (const statement of program.body) {
+        if (!isExportNamedDeclaration(statement))
+            continue;
+        const declaration = statement.declaration;
+        if (declaration) {
+            if (isVariableDeclaration(declaration)) {
+                for (const declarator of declaration.declarations) {
+                    addBindingNamesFromNode(declarator.id, exportedNames);
+                }
+            }
+            else if (isFunctionDeclaration(declaration) ||
+                isClassDeclaration(declaration)) {
+                const id = declaration.id;
+                if (!id)
+                    continue;
+                exportedNames.add(id.name);
+            }
+        }
+        for (const specifier of statement.specifiers) {
+            if (!isExportSpecifier(specifier))
+                continue;
+            const local = specifier.local;
+            if (!isIdentifier(local))
+                continue;
+            exportedNames.add(local.name);
+        }
+    }
+    return exportedNames;
+};

package/dist/core/stable-naming.js

@@ -38,7 +38,7 @@ const hasShadowingRisk = (binding, bindingScope, targetName) => binding.referenc
 /**
  * Finds the next available name slot in a scope.
  */
-const findAvailableName = (scope, binding, baseName, startIndex, canBeStable) => {
+const findAvailableName = (scope, binding, baseName, startIndex, canBeStable, currentName) => {
     let index = startIndex;
     // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
     while (true) {
@@ -47,8 +47,12 @@ const findAvailableName = (scope, binding, baseName, startIndex, canBeStable) =>
         const candidateStable = makeStableName(candidateBase);
         const candidateReadable = candidateBase;
         // Check if either form is already taken in this scope
-        const stableTaken = scope.hasBinding(candidateStable);
-        const readableTaken = scope.hasBinding(candidateReadable);
+        const stableTaken = candidateStable === currentName
+            ? false
+            : scope.hasBinding(candidateStable);
+        const readableTaken = candidateReadable === currentName
+            ? false
+            : scope.hasBinding(candidateReadable);
         if (!stableTaken && !readableTaken) {
             // This slot is available
             const name = canBeStable ? candidateStable : candidateReadable;
@@ -124,7 +128,7 @@ export class RenameGroup {
                 let index = 1;
                 for (const entry of group) {
                     // Find next available name slot
-                    const targetName = findAvailableName(scope, scope.getBinding(entry.currentName), baseName, index, canBeStable);
+                    const targetName = findAvailableName(scope, scope.getBinding(entry.currentName), baseName, index, canBeStable, entry.currentName);
                     index = targetName.nextIndex;
                     if (entry.currentName !== targetName.name) {
                         scope.rename(entry.currentName, targetName.name);

package/dist/transforms/_generated/manifest.js

@@ -81,6 +81,12 @@ const manifestData = {
         evaluatedAt: "2026-01-23T18:10:00.000Z",
         notes: "Derives names from charCodeAt argument for better stability (e.g., $charCodeAtIndex, $charCodeAtIndexPlus1).",
     },
+    "rename-client-aliases": {
+        diffReductionImpact: 0,
+        recommended: true,
+        evaluatedAt: "2026-01-24T15:58:39.356Z",
+        notes: "Measured with baseline none: 0.00%. Added to recommended for readability.",
+    },
     "rename-comparison-flags": {
         diffReductionImpact: 0,
         recommended: true,

package/dist/transforms/_generated/registry.js

@@ -13,6 +13,7 @@ import { renameCatchParametersTransform } from "../rename-catch-parameters/renam
 import { renameCharCodeAtTransform } from "../rename-char-code-at/rename-char-code-at-transform.js";
 import { renameCharcodeVariablesTransform } from "../rename-charcode-variables/rename-charcode-variables-transform.js";
 import { renameCharcodeVariablesV2Transform } from "../rename-charcode-variables-v2/rename-charcode-variables-v2-transform.js";
+import { renameClientAliasesTransform } from "../rename-client-aliases/rename-client-aliases-transform.js";
 import { renameComparisonFlagsTransform } from "../rename-comparison-flags/rename-comparison-flags-transform.js";
 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";
@@ -55,6 +56,7 @@ export const transformRegistry = {
     [renameCharCodeAtTransform.id]: renameCharCodeAtTransform,
     [renameCharcodeVariablesTransform.id]: renameCharcodeVariablesTransform,
     [renameCharcodeVariablesV2Transform.id]: renameCharcodeVariablesV2Transform,
+    [renameClientAliasesTransform.id]: renameClientAliasesTransform,
     [renameComparisonFlagsTransform.id]: renameComparisonFlagsTransform,
     [renameDateNowStartTimesTransform.id]: renameDateNowStartTimesTransform,
     [renameDefaultOptionsParametersTransform.id]: renameDefaultOptionsParametersTransform,

package/dist/transforms/rename-client-aliases/manifest.json

@@ -0,0 +1,6 @@
+{
+    "diffReductionImpact": 0,
+    "recommended": true,
+    "evaluatedAt": "2026-01-24T15:58:39.356Z",
+    "notes": "Measured with baseline none: 0.00%. Added to recommended for readability."
+}

package/dist/transforms/rename-client-aliases/rename-client-aliases-transform.d.ts

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

package/dist/transforms/rename-client-aliases/rename-client-aliases-transform.js

@@ -0,0 +1,65 @@
+import { createRequire } from "node:module";
+import { collectExportedNames } from "../../core/collect-exported-names.js";
+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 = "client";
+const isClientMemberExpression = (node) => {
+    if (node.computed) {
+        return ((node.property.type === "StringLiteral" &&
+            node.property.value === "client") ||
+            (node.property.type === "TemplateLiteral" &&
+                node.property.expressions.length === 0 &&
+                node.property.quasis[0]?.value.cooked === "client"));
+    }
+    return node.property.type === "Identifier" && node.property.name === "client";
+};
+export const renameClientAliasesTransform = {
+    id: "rename-client-aliases",
+    description: "Renames aliases of *.client to $client/$client2/...",
+    scope: "file",
+    parallelizable: true,
+    transform(context) {
+        let nodesVisited = 0;
+        let transformationsApplied = 0;
+        for (const fileInfo of getFilesToProcess(context)) {
+            const group = new RenameGroup();
+            const exportedNames = collectExportedNames(fileInfo.ast.program);
+            traverse(fileInfo.ast, {
+                VariableDeclarator(path) {
+                    nodesVisited++;
+                    const id = path.node.id;
+                    if (id.type !== "Identifier")
+                        return;
+                    if (isStableRenamed(id.name))
+                        return;
+                    const init = path.node.init;
+                    if (init?.type !== "MemberExpression")
+                        return;
+                    if (!isClientMemberExpression(init))
+                        return;
+                    const binding = path.scope.getBinding(id.name);
+                    if (!binding)
+                        return;
+                    if (!binding.constant)
+                        return;
+                    if (binding.referencePaths.length === 0)
+                        return;
+                    if (binding.scope.block.type === "Program" &&
+                        exportedNames.has(id.name)) {
+                        return;
+                    }
+                    group.add({
+                        scope: binding.scope,
+                        currentName: id.name,
+                        baseName: BASE_NAME,
+                    });
+                },
+            });
+            transformationsApplied += group.apply();
+        }
+        return Promise.resolve({ nodesVisited, transformationsApplied });
+    },
+};

package/dist/transforms/rename-this-aliases/rename-this-aliases-transform.js

@@ -1,78 +1,11 @@
 import { createRequire } from "node:module";
-import { isArrayPattern, isAssignmentPattern, isClassDeclaration, isExportNamedDeclaration, isExportSpecifier, isFunctionDeclaration, isIdentifier, isObjectPattern, isObjectProperty, isRestElement, isVariableDeclaration, } from "@babel/types";
+import { collectExportedNames } from "../../core/collect-exported-names.js";
 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 = "thisRef";
-const addBindingNamesFromNode = (node, out) => {
-    const babelNode = node;
-    if (!babelNode)
-        return;
-    if (babelNode.type === "VoidPattern")
-        return;
-    if (isIdentifier(babelNode)) {
-        out.add(babelNode.name);
-        return;
-    }
-    if (isObjectPattern(babelNode)) {
-        for (const property of babelNode.properties) {
-            if (isRestElement(property)) {
-                addBindingNamesFromNode(property.argument, out);
-                continue;
-            }
-            if (!isObjectProperty(property))
-                continue;
-            addBindingNamesFromNode(property.value, out);
-        }
-        return;
-    }
-    if (isArrayPattern(babelNode)) {
-        for (const element of babelNode.elements) {
-            addBindingNamesFromNode(element, out);
-        }
-        return;
-    }
-    if (isRestElement(babelNode)) {
-        addBindingNamesFromNode(babelNode.argument, out);
-        return;
-    }
-    if (isAssignmentPattern(babelNode)) {
-        addBindingNamesFromNode(babelNode.left, out);
-    }
-};
-const collectExportedNames = (program) => {
-    const exportedNames = new Set();
-    for (const statement of program.body) {
-        if (!isExportNamedDeclaration(statement))
-            continue;
-        const declaration = statement.declaration;
-        if (declaration) {
-            if (isVariableDeclaration(declaration)) {
-                for (const declarator of declaration.declarations) {
-                    addBindingNamesFromNode(declarator.id, exportedNames);
-                }
-            }
-            else if (isFunctionDeclaration(declaration) ||
-                isClassDeclaration(declaration)) {
-                const id = declaration.id;
-                if (!id)
-                    continue;
-                exportedNames.add(id.name);
-            }
-        }
-        for (const specifier of statement.specifiers) {
-            if (!isExportSpecifier(specifier))
-                continue;
-            const local = specifier.local;
-            if (!isIdentifier(local))
-                continue;
-            exportedNames.add(local.name);
-        }
-    }
-    return exportedNames;
-};
 export const renameThisAliasesTransform = {
     id: "rename-this-aliases",
     description: "Renames `var x = this` aliases to $thisRef (stable when unique) or thisRef/thisRef2/...",

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

@@ -1,78 +1,11 @@
 import { createRequire } from "node:module";
-import { isArrayPattern, isAssignmentPattern, isClassDeclaration, isExportNamedDeclaration, isExportSpecifier, isFunctionDeclaration, isIdentifier, isObjectPattern, isObjectProperty, isRestElement, isVariableDeclaration, } from "@babel/types";
+import { collectExportedNames } from "../../core/collect-exported-names.js";
 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 = "type";
-const addBindingNamesFromNode = (node, out) => {
-    const babelNode = node;
-    if (!babelNode)
-        return;
-    if (babelNode.type === "VoidPattern")
-        return;
-    if (isIdentifier(babelNode)) {
-        out.add(babelNode.name);
-        return;
-    }
-    if (isObjectPattern(babelNode)) {
-        for (const property of babelNode.properties) {
-            if (isRestElement(property)) {
-                addBindingNamesFromNode(property.argument, out);
-                continue;
-            }
-            if (!isObjectProperty(property))
-                continue;
-            addBindingNamesFromNode(property.value, out);
-        }
-        return;
-    }
-    if (isArrayPattern(babelNode)) {
-        for (const element of babelNode.elements) {
-            addBindingNamesFromNode(element, out);
-        }
-        return;
-    }
-    if (isRestElement(babelNode)) {
-        addBindingNamesFromNode(babelNode.argument, out);
-        return;
-    }
-    if (isAssignmentPattern(babelNode)) {
-        addBindingNamesFromNode(babelNode.left, out);
-    }
-};
-const collectExportedNames = (program) => {
-    const exportedNames = new Set();
-    for (const statement of program.body) {
-        if (!isExportNamedDeclaration(statement))
-            continue;
-        const declaration = statement.declaration;
-        if (declaration) {
-            if (isVariableDeclaration(declaration)) {
-                for (const declarator of declaration.declarations) {
-                    addBindingNamesFromNode(declarator.id, exportedNames);
-                }
-            }
-            else if (isFunctionDeclaration(declaration) ||
-                isClassDeclaration(declaration)) {
-                const id = declaration.id;
-                if (!id)
-                    continue;
-                exportedNames.add(id.name);
-            }
-        }
-        for (const specifier of statement.specifiers) {
-            if (!isExportSpecifier(specifier))
-                continue;
-            const local = specifier.local;
-            if (!isIdentifier(local))
-                continue;
-            exportedNames.add(local.name);
-        }
-    }
-    return exportedNames;
-};
 export const renameTypeofVariablesTransform = {
     id: "rename-typeof-variables",
     description: "Renames variables assigned from typeof expressions to $type (stable when unique) or type/type2/...",

package/package.json

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