miniread 1.10.0 → 1.12.0

package/dist/scripts/evaluate/create-output-slug.d.ts

@@ -0,0 +1,7 @@
+type CreateOutputSlugOptions = {
+    transforms: string[];
+    recommendedTransforms: readonly string[];
+    allTransformIds: readonly string[];
+};
+export declare const createOutputSlug: (options: CreateOutputSlugOptions) => string;
+export {};

package/dist/scripts/evaluate/create-output-slug.js

@@ -0,0 +1,38 @@
+import crypto from "node:crypto";
+import { sanitizeFileComponent } from "./sanitize.js";
+const isSameOrderedList = (a, b) => {
+    if (a.length !== b.length)
+        return false;
+    for (const [index, value] of a.entries()) {
+        const other = b[index];
+        if (other === undefined)
+            return false;
+        if (value !== other)
+            return false;
+    }
+    return true;
+};
+const createRawTransformSlug = (options) => {
+    const { transforms, recommendedTransforms, allTransformIds } = options;
+    if (transforms.length === 0)
+        return "none";
+    if (isSameOrderedList(transforms, recommendedTransforms))
+        return "recommended";
+    if (isSameOrderedList(transforms, allTransformIds))
+        return "all";
+    return transforms.join("-");
+};
+export const createOutputSlug = (options) => {
+    const sanitized = sanitizeFileComponent(createRawTransformSlug(options));
+    const maxLength = 120;
+    if (sanitized.length <= maxLength)
+        return sanitized;
+    const hash = crypto
+        .createHash("sha256")
+        .update(sanitized)
+        .digest("hex")
+        .slice(0, 10);
+    const prefixLength = maxLength - hash.length - 1;
+    const prefix = sanitized.slice(0, Math.max(0, prefixLength));
+    return `${prefix}-${hash}`;
+};

package/dist/scripts/evaluate/diff-utilities.d.ts

@@ -5,7 +5,6 @@ export type DiffStats = {
     removedLines: number;
     changedHunks: number;
 };
-export declare const createTransformSlug: (transforms: string[]) => string;
 type ComputeDirectoryDiffOptions = {
     diffPath: string;
     directoryA: string;

package/dist/scripts/evaluate/diff-utilities.js

@@ -4,9 +4,6 @@ import fs from "node:fs";
 import * as fsPromises from "node:fs/promises";
 import path from "node:path";
 import readline from "node:readline";
-export const createTransformSlug = (transforms) => {
-    return transforms.length > 0 ? transforms.join("-") : "none";
-};
 const isErrnoException = (error) => {
     return error instanceof Error;
 };

package/dist/scripts/evaluate/parse-evaluate-cli-options.js

@@ -1,8 +1,7 @@
 import path from "node:path";
 import { transformPresets } from "../../transforms/transform-presets.js";
 import { allTransformIds } from "../../transforms/transform-registry.js";
-import { createTransformSlug } from "./diff-utilities.js";
-import { sanitizeFileComponent } from "./sanitize.js";
+import { createOutputSlug } from "./create-output-slug.js";
 const parseTransformList = (value) => {
     if (!value || value === "none")
         return [];
@@ -66,8 +65,16 @@ export const parseEvaluateCliOptions = (options) => {
     const verbose = rawOptions.verbose ?? false;
     const overwrite = rawOptions.overwrite ?? false;
     const formatCode = rawOptions.formatCode;
-    const baselineSlug = sanitizeFileComponent(createTransformSlug(baselineTransforms));
-    const testSlug = sanitizeFileComponent(createTransformSlug(testTransforms));
+    const baselineSlug = createOutputSlug({
+        transforms: baselineTransforms,
+        recommendedTransforms: transformPresets.recommended,
+        allTransformIds,
+    });
+    const testSlug = createOutputSlug({
+        transforms: testTransforms,
+        recommendedTransforms: transformPresets.recommended,
+        allTransformIds,
+    });
     const sourcesDirectory = path.resolve(cwd, rawOptions.sources);
     const evaluationDirectory = path.resolve(cwd, rawOptions.evaluationDir);
     const manifestPath = path.resolve(cwd, rawOptions.manifestFile);

package/dist/transforms/expand-sequence-expressions-v5/expand-expression-statement-sequence.d.ts

@@ -0,0 +1,3 @@
+import type { NodePath } from "@babel/traverse";
+import type { ExpressionStatement } from "@babel/types";
+export declare const tryExpandExpressionStatementSequence: (path: NodePath<ExpressionStatement>) => boolean;

package/dist/transforms/expand-sequence-expressions-v5/expand-expression-statement-sequence.js

@@ -0,0 +1,92 @@
+import { createRequire } from "node:module";
+const require = createRequire(import.meta.url);
+// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+const t = require("@babel/types");
+const canReplaceWithMultiple = (path) => path.parentPath.isProgram() ||
+    path.parentPath.isBlockStatement() ||
+    path.parentPath.isStaticBlock() ||
+    path.parentPath.isSwitchCase();
+const canWrapInBlock = (path) => {
+    const parentPath = path.parentPath;
+    if (parentPath.isIfStatement() &&
+        (path.key === "consequent" || path.key === "alternate")) {
+        return true;
+    }
+    if ((parentPath.isForStatement() ||
+        parentPath.isForInStatement() ||
+        parentPath.isForOfStatement() ||
+        parentPath.isWhileStatement() ||
+        parentPath.isDoWhileStatement() ||
+        parentPath.isWithStatement() ||
+        parentPath.isLabeledStatement()) &&
+        path.key === "body") {
+        return true;
+    }
+    return false;
+};
+const isDirectiveProloguePosition = (path) => {
+    const parentPath = path.parentPath;
+    if (parentPath.isBlockStatement()) {
+        const grandParent = parentPath.parentPath;
+        if (parentPath.key !== "body")
+            return false;
+        if (!grandParent.isFunction())
+            return false;
+    }
+    else if (!parentPath.isProgram()) {
+        return false;
+    }
+    const siblings = parentPath.get("body");
+    if (!Array.isArray(siblings))
+        return false;
+    const index = siblings.findIndex((sibling) => sibling.node === path.node);
+    if (index === -1)
+        return false;
+    for (let bodyIndex = 0; bodyIndex < index; bodyIndex++) {
+        const sibling = siblings[bodyIndex];
+        if (!sibling)
+            return false;
+        if (!sibling.isExpressionStatement())
+            return false;
+        if (sibling.node.expression.type !== "StringLiteral")
+            return false;
+    }
+    return true;
+};
+export const tryExpandExpressionStatementSequence = (path) => {
+    if (!canReplaceWithMultiple(path) && !canWrapInBlock(path))
+        return false;
+    const expression = path.node.expression;
+    if (expression.type !== "SequenceExpression")
+        return false;
+    if (expression.expressions.length < 2)
+        return false;
+    const firstExpression = expression.expressions[0];
+    const shouldWrapFirstStringLiteralDirective = firstExpression?.type === "StringLiteral" &&
+        canReplaceWithMultiple(path) &&
+        isDirectiveProloguePosition(path);
+    const statements = expression.expressions.map((expr, index) => {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        let clonedExpression = t.cloneNode(expr, true);
+        if (shouldWrapFirstStringLiteralDirective &&
+            index === 0 &&
+            expr.type === "StringLiteral") {
+            // Prevent inserting a directive prologue statement like `"use strict";`.
+            // `("use strict");` is not treated as a directive.
+            // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+            clonedExpression = t.parenthesizedExpression(clonedExpression);
+        }
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        return t.expressionStatement(clonedExpression);
+    });
+    if (canReplaceWithMultiple(path)) {
+        path.replaceWithMultiple(statements);
+        return true;
+    }
+    if (canWrapInBlock(path)) {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        path.replaceWith(t.blockStatement(statements));
+        return true;
+    }
+    return false;
+};

package/dist/transforms/expand-sequence-expressions-v5/expand-if-statement-test-sequence.d.ts

@@ -0,0 +1,3 @@
+import type { NodePath } from "@babel/traverse";
+import type { IfStatement } from "@babel/types";
+export declare const tryExpandIfStatementTestSequence: (path: NodePath<IfStatement>) => boolean;

package/dist/transforms/expand-sequence-expressions-v5/expand-if-statement-test-sequence.js

@@ -0,0 +1,104 @@
+import { createRequire } from "node:module";
+const require = createRequire(import.meta.url);
+// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+const t = require("@babel/types");
+const canReplaceWithMultiple = (path) => {
+    return (path.parentPath.isProgram() ||
+        path.parentPath.isBlockStatement() ||
+        path.parentPath.isStaticBlock() ||
+        path.parentPath.isSwitchCase());
+};
+const canWrapInBlock = (path) => {
+    const parentPath = path.parentPath;
+    if (parentPath.isIfStatement() &&
+        (path.key === "consequent" || path.key === "alternate")) {
+        return true;
+    }
+    if ((parentPath.isForStatement() ||
+        parentPath.isForInStatement() ||
+        parentPath.isForOfStatement() ||
+        parentPath.isWhileStatement() ||
+        parentPath.isDoWhileStatement() ||
+        parentPath.isWithStatement() ||
+        parentPath.isLabeledStatement()) &&
+        path.key === "body") {
+        return true;
+    }
+    return false;
+};
+const isDirectiveProloguePosition = (path) => {
+    const parentPath = path.parentPath;
+    if (parentPath.isBlockStatement()) {
+        const grandParent = parentPath.parentPath;
+        if (parentPath.key !== "body")
+            return false;
+        if (!grandParent.isFunction())
+            return false;
+    }
+    else if (!parentPath.isProgram()) {
+        return false;
+    }
+    const siblings = parentPath.get("body");
+    if (!Array.isArray(siblings))
+        return false;
+    const index = siblings.findIndex((sibling) => sibling.node === path.node);
+    if (index === -1)
+        return false;
+    for (let bodyIndex = 0; bodyIndex < index; bodyIndex++) {
+        const sibling = siblings[bodyIndex];
+        if (!sibling)
+            return false;
+        if (!sibling.isExpressionStatement())
+            return false;
+        if (sibling.node.expression.type !== "StringLiteral")
+            return false;
+    }
+    return true;
+};
+const createExpressionStatement = (expression, options) => {
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    let clonedExpression = t.cloneNode(expression, true);
+    if (options.wrapStringLiteralDirective &&
+        expression.type === "StringLiteral") {
+        // Prevent inserting a directive prologue statement like `"use strict";`.
+        // `("use strict");` is not treated as a directive.
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        clonedExpression = t.parenthesizedExpression(clonedExpression);
+    }
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    return t.expressionStatement(clonedExpression);
+};
+export const tryExpandIfStatementTestSequence = (path) => {
+    if (!canReplaceWithMultiple(path) && !canWrapInBlock(path))
+        return false;
+    const test = path.node.test;
+    if (test.type !== "SequenceExpression")
+        return false;
+    if (test.expressions.length < 2)
+        return false;
+    const leadingExpressions = test.expressions.slice(0, -1);
+    const lastExpression = test.expressions.at(-1);
+    if (!lastExpression)
+        return false;
+    const firstExpression = leadingExpressions[0];
+    const shouldWrapFirstStringLiteralDirective = firstExpression?.type === "StringLiteral" &&
+        isDirectiveProloguePosition(path);
+    const statements = leadingExpressions.map((expression, index) => createExpressionStatement(expression, {
+        wrapStringLiteralDirective: shouldWrapFirstStringLiteralDirective && index === 0,
+    }));
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    const updatedIfStatement = t.cloneNode(path.node, true);
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    updatedIfStatement.test = t.cloneNode(lastExpression, true);
+    statements.push(updatedIfStatement);
+    if (canReplaceWithMultiple(path)) {
+        path.replaceWithMultiple(statements);
+        return true;
+    }
+    if (canWrapInBlock(path)) {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        path.replaceWith(t.blockStatement(statements));
+        return true;
+    }
+    return false;
+};

package/dist/transforms/expand-sequence-expressions-v5/expand-return-sequence.d.ts

@@ -0,0 +1,3 @@
+import type { NodePath } from "@babel/traverse";
+import type { ReturnStatement } from "@babel/types";
+export declare const tryExpandReturnSequence: (path: NodePath<ReturnStatement>) => boolean;

package/dist/transforms/expand-sequence-expressions-v5/expand-return-sequence.js

@@ -0,0 +1,100 @@
+import { createRequire } from "node:module";
+const require = createRequire(import.meta.url);
+// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+const t = require("@babel/types");
+const canReplaceWithMultiple = (path) => {
+    return path.parentPath.isBlockStatement() || path.parentPath.isSwitchCase();
+};
+const canWrapInBlock = (path) => {
+    const parentPath = path.parentPath;
+    if (parentPath.isIfStatement() &&
+        (path.key === "consequent" || path.key === "alternate")) {
+        return true;
+    }
+    if ((parentPath.isForStatement() ||
+        parentPath.isForInStatement() ||
+        parentPath.isForOfStatement() ||
+        parentPath.isWhileStatement() ||
+        parentPath.isDoWhileStatement() ||
+        parentPath.isWithStatement() ||
+        parentPath.isLabeledStatement()) &&
+        path.key === "body") {
+        return true;
+    }
+    return false;
+};
+const replaceReturnWithStatements = (path, statements) => {
+    if (canReplaceWithMultiple(path)) {
+        path.replaceWithMultiple(statements);
+        return true;
+    }
+    if (canWrapInBlock(path)) {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        path.replaceWith(t.blockStatement(statements));
+        return true;
+    }
+    return false;
+};
+const isDirectiveProloguePosition = (path) => {
+    const parentPath = path.parentPath;
+    if (parentPath.isBlockStatement()) {
+        if (parentPath.key !== "body")
+            return false;
+        if (!parentPath.parentPath.isFunction())
+            return false;
+    }
+    else if (!parentPath.isProgram()) {
+        return false;
+    }
+    const siblings = parentPath.get("body");
+    if (!Array.isArray(siblings))
+        return false;
+    const index = siblings.findIndex((sibling) => sibling.node === path.node);
+    if (index === -1)
+        return false;
+    for (let bodyIndex = 0; bodyIndex < index; bodyIndex++) {
+        const sibling = siblings[bodyIndex];
+        if (!sibling)
+            return false;
+        if (!sibling.isExpressionStatement())
+            return false;
+        if (sibling.node.expression.type !== "StringLiteral")
+            return false;
+    }
+    return true;
+};
+export const tryExpandReturnSequence = (path) => {
+    if (!canReplaceWithMultiple(path) && !canWrapInBlock(path))
+        return false;
+    const argument = path.node.argument;
+    if (!argument)
+        return false;
+    if (argument.type !== "SequenceExpression")
+        return false;
+    if (argument.expressions.length < 2)
+        return false;
+    const leadingExpressions = argument.expressions.slice(0, -1);
+    const lastExpression = argument.expressions.at(-1);
+    if (!lastExpression)
+        return false;
+    const firstExpression = leadingExpressions[0];
+    const shouldWrapFirstStringLiteralDirective = firstExpression?.type === "StringLiteral" &&
+        isDirectiveProloguePosition(path);
+    const statements = leadingExpressions.map((expression) => {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        let clonedExpression = t.cloneNode(expression, true);
+        if (shouldWrapFirstStringLiteralDirective &&
+            expression === firstExpression) {
+            // Prevent inserting a directive prologue statement like `"use strict";`.
+            // `("use strict");` is not treated as a directive.
+            // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+            clonedExpression = t.parenthesizedExpression(clonedExpression);
+        }
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        return t.expressionStatement(clonedExpression);
+    });
+    statements.push(
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    t.returnStatement(t.cloneNode(lastExpression, true)));
+    return replaceReturnWithStatements(path, statements);
+};

package/dist/transforms/expand-sequence-expressions-v5/expand-sequence-expressions-v5-transform.d.ts

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

package/dist/transforms/expand-sequence-expressions-v5/expand-sequence-expressions-v5-transform.js

@@ -0,0 +1,55 @@
+import { createRequire } from "node:module";
+import { tryExpandExpressionStatementSequence } from "./expand-expression-statement-sequence.js";
+import { tryExpandIfStatementTestSequence } from "./expand-if-statement-test-sequence.js";
+import { tryExpandReturnSequence } from "./expand-return-sequence.js";
+import { tryExpandThrowSequence } from "./expand-throw-sequence.js";
+import { tryExpandVariableDeclarationSequenceInitializers } from "./expand-variable-declaration-sequence.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 expandSequenceExpressionsV5Transform = {
+    id: "expand-sequence-expressions-v5",
+    description: "Expands comma operator sequences in returns, throws, statements (including control-flow bodies), variable initializers, and `if` tests",
+    scope: "file",
+    parallelizable: true,
+    transform(context) {
+        const { projectGraph } = context;
+        let nodesVisited = 0;
+        let transformationsApplied = 0;
+        for (const [, fileInfo] of projectGraph.files) {
+            traverse(fileInfo.ast, {
+                ReturnStatement(path) {
+                    nodesVisited++;
+                    if (!tryExpandReturnSequence(path))
+                        return;
+                    transformationsApplied += 1;
+                },
+                ThrowStatement(path) {
+                    nodesVisited++;
+                    if (!tryExpandThrowSequence(path))
+                        return;
+                    transformationsApplied += 1;
+                },
+                ExpressionStatement(path) {
+                    nodesVisited++;
+                    if (!tryExpandExpressionStatementSequence(path))
+                        return;
+                    transformationsApplied += 1;
+                },
+                VariableDeclaration(path) {
+                    nodesVisited++;
+                    if (!tryExpandVariableDeclarationSequenceInitializers(path))
+                        return;
+                    transformationsApplied += 1;
+                },
+                IfStatement(path) {
+                    nodesVisited++;
+                    if (!tryExpandIfStatementTestSequence(path))
+                        return;
+                    transformationsApplied += 1;
+                },
+            });
+        }
+        return Promise.resolve({ nodesVisited, transformationsApplied });
+    },
+};

package/dist/transforms/expand-sequence-expressions-v5/expand-throw-sequence.d.ts

@@ -0,0 +1,3 @@
+import type { NodePath } from "@babel/traverse";
+import type { ThrowStatement } from "@babel/types";
+export declare const tryExpandThrowSequence: (path: NodePath<ThrowStatement>) => boolean;

package/dist/transforms/expand-sequence-expressions-v5/expand-throw-sequence.js

@@ -0,0 +1,101 @@
+import { createRequire } from "node:module";
+const require = createRequire(import.meta.url);
+// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+const t = require("@babel/types");
+const canReplaceWithMultiple = (path) => {
+    return (path.parentPath.isProgram() ||
+        path.parentPath.isBlockStatement() ||
+        path.parentPath.isStaticBlock() ||
+        path.parentPath.isSwitchCase());
+};
+const canWrapInBlock = (path) => {
+    const parentPath = path.parentPath;
+    if (parentPath.isIfStatement() &&
+        (path.key === "consequent" || path.key === "alternate")) {
+        return true;
+    }
+    if ((parentPath.isForStatement() ||
+        parentPath.isForInStatement() ||
+        parentPath.isForOfStatement() ||
+        parentPath.isWhileStatement() ||
+        parentPath.isDoWhileStatement() ||
+        parentPath.isWithStatement() ||
+        parentPath.isLabeledStatement()) &&
+        path.key === "body") {
+        return true;
+    }
+    return false;
+};
+const replaceThrowWithStatements = (path, statements) => {
+    if (canReplaceWithMultiple(path)) {
+        path.replaceWithMultiple(statements);
+        return true;
+    }
+    if (canWrapInBlock(path)) {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        path.replaceWith(t.blockStatement(statements));
+        return true;
+    }
+    return false;
+};
+const isDirectiveProloguePosition = (path) => {
+    const parentPath = path.parentPath;
+    if (parentPath.isBlockStatement()) {
+        const grandParent = parentPath.parentPath;
+        if (parentPath.key !== "body")
+            return false;
+        if (!grandParent.isFunction())
+            return false;
+    }
+    else if (!parentPath.isProgram()) {
+        return false;
+    }
+    const siblings = parentPath.get("body");
+    if (!Array.isArray(siblings))
+        return false;
+    const index = siblings.findIndex((sibling) => sibling.node === path.node);
+    if (index === -1)
+        return false;
+    for (let bodyIndex = 0; bodyIndex < index; bodyIndex++) {
+        const sibling = siblings[bodyIndex];
+        if (!sibling)
+            return false;
+        if (!sibling.isExpressionStatement())
+            return false;
+        if (sibling.node.expression.type !== "StringLiteral")
+            return false;
+    }
+    return true;
+};
+export const tryExpandThrowSequence = (path) => {
+    if (!canReplaceWithMultiple(path) && !canWrapInBlock(path))
+        return false;
+    const argument = path.node.argument;
+    if (argument.type !== "SequenceExpression")
+        return false;
+    if (argument.expressions.length < 2)
+        return false;
+    const leadingExpressions = argument.expressions.slice(0, -1);
+    const lastExpression = argument.expressions.at(-1);
+    if (!lastExpression)
+        return false;
+    const firstExpression = leadingExpressions[0];
+    const shouldWrapFirstStringLiteral = firstExpression?.type === "StringLiteral" &&
+        isDirectiveProloguePosition(path);
+    const statements = leadingExpressions.map((expression) => {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        let clonedExpression = t.cloneNode(expression, true);
+        if (shouldWrapFirstStringLiteral && expression === firstExpression) {
+            // Prevent inserting a directive prologue statement like `"use strict";`.
+            // `("use strict");` is not treated as a directive.
+            // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+            clonedExpression = t.parenthesizedExpression(clonedExpression);
+        }
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        return t.expressionStatement(clonedExpression);
+    });
+    statements.push(
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    t.throwStatement(t.cloneNode(lastExpression, true)));
+    return replaceThrowWithStatements(path, statements);
+};

package/dist/transforms/expand-sequence-expressions-v5/expand-variable-declaration-sequence.d.ts

@@ -0,0 +1,3 @@
+import type { NodePath } from "@babel/traverse";
+import type { VariableDeclaration } from "@babel/types";
+export declare const tryExpandVariableDeclarationSequenceInitializers: (path: NodePath<VariableDeclaration>) => boolean;

package/dist/transforms/expand-sequence-expressions-v5/expand-variable-declaration-sequence.js

@@ -0,0 +1,103 @@
+import { createRequire } from "node:module";
+const require = createRequire(import.meta.url);
+// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+const t = require("@babel/types");
+const createExpressionStatement = (expression, options) => {
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    let clonedExpression = t.cloneNode(expression, true);
+    if (options.wrapStringLiteralDirective &&
+        expression.type === "StringLiteral") {
+        // Prevent inserting a directive prologue statement like `"use strict";`.
+        // `("use strict");` is not treated as a directive.
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+        clonedExpression = t.parenthesizedExpression(clonedExpression);
+    }
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    return t.expressionStatement(clonedExpression);
+};
+const createVariableDeclarationStatement = (kind, declarator) => {
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    return t.variableDeclaration(kind, [declarator]);
+};
+const createVariableDeclarator = (id, init) => {
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    return t.variableDeclarator(
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    t.cloneNode(id, true), init
+        ? // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+            t.cloneNode(init, true)
+        : undefined);
+};
+const isSupportedStatementContainer = (path) => {
+    const parentPath = path.parentPath;
+    if (!parentPath)
+        return false;
+    return (parentPath.isProgram() ||
+        parentPath.isBlockStatement() ||
+        parentPath.isStaticBlock() ||
+        parentPath.isSwitchCase());
+};
+const isDirectiveProloguePosition = (path) => {
+    const parentPath = path.parentPath;
+    if (!parentPath)
+        return false;
+    if (parentPath.isBlockStatement()) {
+        const grandParent = parentPath.parentPath;
+        if (parentPath.key !== "body")
+            return false;
+        if (!grandParent.isFunction())
+            return false;
+    }
+    else if (!parentPath.isProgram()) {
+        return false;
+    }
+    const siblings = parentPath.get("body");
+    if (!Array.isArray(siblings))
+        return false;
+    const index = siblings.findIndex((sibling) => sibling.node === path.node);
+    if (index === -1)
+        return false;
+    for (let bodyIndex = 0; bodyIndex < index; bodyIndex++) {
+        const sibling = siblings[bodyIndex];
+        if (!sibling)
+            return false;
+        if (!sibling.isExpressionStatement())
+            return false;
+        if (sibling.node.expression.type !== "StringLiteral")
+            return false;
+    }
+    return true;
+};
+export const tryExpandVariableDeclarationSequenceInitializers = (path) => {
+    if (!isSupportedStatementContainer(path))
+        return false;
+    const isDirectivePrologueAtInsertion = isDirectiveProloguePosition(path);
+    const hasAnySequence = path.node.declarations.some((declarator) => declarator.init?.type === "SequenceExpression" &&
+        declarator.init.expressions.length >= 2);
+    if (!hasAnySequence)
+        return false;
+    const kind = path.node.kind;
+    const statements = [];
+    for (const declarator of path.node.declarations) {
+        const init = declarator.init;
+        if (init?.type !== "SequenceExpression") {
+            statements.push(createVariableDeclarationStatement(kind, declarator));
+            continue;
+        }
+        const leadingExpressions = init.expressions.slice(0, -1);
+        const lastExpression = init.expressions.at(-1);
+        if (!lastExpression) {
+            statements.push(createVariableDeclarationStatement(kind, declarator));
+            continue;
+        }
+        for (const expression of leadingExpressions) {
+            statements.push(createExpressionStatement(expression, {
+                wrapStringLiteralDirective: isDirectivePrologueAtInsertion && statements.length === 0,
+            }));
+        }
+        const newDeclarator = createVariableDeclarator(declarator.id, lastExpression);
+        statements.push(createVariableDeclarationStatement(kind, newDeclarator));
+    }
+    path.replaceWithMultiple(statements);
+    return true;
+};

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

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

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

@@ -0,0 +1,121 @@
+import { createRequire } from "node:module";
+import { isArrayPattern, isAssignmentPattern, isClassDeclaration, isExportNamedDeclaration, isExportSpecifier, isFunctionDeclaration, isIdentifier, isObjectPattern, isObjectProperty, isRestElement, isVariableDeclaration, } from "@babel/types";
+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/...",
+    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, init } = path.node;
+                    if (id.type !== "Identifier")
+                        return;
+                    if (init?.type !== "ThisExpression")
+                        return;
+                    const currentName = id.name;
+                    if (isStableRenamed(currentName))
+                        return;
+                    const binding = path.scope.getBinding(currentName);
+                    if (!binding)
+                        return;
+                    if (!binding.constant)
+                        return;
+                    if (binding.scope.block.type === "Program" &&
+                        exportedNames.has(currentName)) {
+                        return;
+                    }
+                    group.add({
+                        scope: binding.scope,
+                        currentName,
+                        baseName: BASE_NAME,
+                    });
+                },
+            });
+            transformationsApplied += group.apply();
+        }
+        return Promise.resolve({
+            nodesVisited,
+            transformationsApplied,
+        });
+    },
+};

package/dist/transforms/transform-registry.js

@@ -1,6 +1,7 @@
 import { expandBooleanLiteralsTransform } from "./expand-boolean-literals/expand-boolean-literals-transform.js";
 import { expandSpecialNumberLiteralsTransform } from "./expand-special-number-literals/expand-special-number-literals-transform.js";
 import { expandSequenceExpressionsV4Transform } from "./expand-sequence-expressions-v4/expand-sequence-expressions-v4-transform.js";
+import { expandSequenceExpressionsV5Transform } from "./expand-sequence-expressions-v5/expand-sequence-expressions-v5-transform.js";
 import { expandUndefinedLiteralsTransform } from "./expand-undefined-literals/expand-undefined-literals-transform.js";
 import { renameCatchParametersTransform } from "./rename-catch-parameters/rename-catch-parameters-transform.js";
 import { renameDestructuredAliasesTransform } from "./rename-destructured-aliases/rename-destructured-aliases-transform.js";
@@ -10,6 +11,7 @@ import { renameLoopIndexVariablesV2Transform } from "./rename-loop-index-variabl
 import { renameLoopIndexVariablesV3Transform } from "./rename-loop-index-variables-v3/rename-loop-index-variables-v3-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";
 import { renameTimeoutIdsTransform } from "./rename-timeout-ids/rename-timeout-ids-transform.js";
 import { renameUseReferenceGuardsTransform } from "./rename-use-reference-guards/rename-use-reference-guards-transform.js";
 import { renameUseReferenceGuardsV2Transform } from "./rename-use-reference-guards-v2/rename-use-reference-guards-v2-transform.js";
@@ -18,6 +20,7 @@ export const transformRegistry = {
     [expandBooleanLiteralsTransform.id]: expandBooleanLiteralsTransform,
     [expandSpecialNumberLiteralsTransform.id]: expandSpecialNumberLiteralsTransform,
     [expandSequenceExpressionsV4Transform.id]: expandSequenceExpressionsV4Transform,
+    [expandSequenceExpressionsV5Transform.id]: expandSequenceExpressionsV5Transform,
     [expandUndefinedLiteralsTransform.id]: expandUndefinedLiteralsTransform,
     [renameCatchParametersTransform.id]: renameCatchParametersTransform,
     [renameDestructuredAliasesTransform.id]: renameDestructuredAliasesTransform,
@@ -27,6 +30,7 @@ export const transformRegistry = {
     [renameLoopIndexVariablesV3Transform.id]: renameLoopIndexVariablesV3Transform,
     [renamePromiseExecutorParametersTransform.id]: renamePromiseExecutorParametersTransform,
     [renameReplaceChildParametersTransform.id]: renameReplaceChildParametersTransform,
+    [renameThisAliasesTransform.id]: renameThisAliasesTransform,
     [renameTimeoutIdsTransform.id]: renameTimeoutIdsTransform,
     [renameUseReferenceGuardsTransform.id]: renameUseReferenceGuardsTransform,
     [renameUseReferenceGuardsV2Transform.id]: renameUseReferenceGuardsV2Transform,

package/package.json

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

package/transform-manifest.json

@@ -81,15 +81,35 @@
       "evaluatedAt": "2026-01-22T12:49:10.952Z",
       "notes": "Auto-added by evaluation script. Measured with baseline none: 0.09%."
     },
+    {
+      "id": "rename-this-aliases",
+      "description": "Renames `var x = this` aliases to $thisRef (stable when unique) or thisRef/thisRef2/...",
+      "scope": "file",
+      "parallelizable": true,
+      "diffReductionImpact": 0.00003774938185385768,
+      "recommended": true,
+      "evaluatedAt": "2026-01-23T17:57:26.908Z",
+      "notes": "Measured with baseline none: 0.00%. Improves readability by stabilizing `this` aliases used for closures."
+    },
     {
       "id": "expand-sequence-expressions-v4",
       "description": "Expands comma operator sequences in returns, throws, statements (including control-flow bodies), and variable initializers",
       "scope": "file",
       "parallelizable": true,
       "diffReductionImpact": -0.01730809158000035,
-      "recommended": true,
+      "recommended": false,
       "evaluatedAt": "2026-01-23T10:57:45.082Z",
-      "notes": "Measured with baseline none: -1.73%. Supersedes all prior sequence expression transforms."
+      "notes": "Measured with baseline none: -1.73%. Supersedes all prior sequence expression transforms. Superseded by expand-sequence-expressions-v5.",
+      "supersededBy": "expand-sequence-expressions-v5"
+    },
+    {
+      "id": "expand-sequence-expressions-v5",
+      "description": "Expands comma operator sequences in returns, throws, statements (including control-flow bodies), variable initializers, and `if` tests",
+      "scope": "file",
+      "parallelizable": true,
+      "diffReductionImpact": -0.020243106019139034,
+      "recommended": true,
+      "notes": "Extends expand-sequence-expressions-v4 by also expanding `if ((a, b)) ...` style tests."
     },
     {
       "id": "rename-use-reference-guards",
@@ -164,8 +184,8 @@
   ],
   "presetStats": {
     "recommended": {
-      "diffReductionImpact": 0.0027273928389422997,
-      "notes": "Measured with baseline none: 0.27%."
+      "diffReductionImpact": 0.003690002076215948,
+      "notes": "Measured with baseline none: 0.37%."
     }
   }
 }