@barefootjs/test 0.35.4 → 0.35.5

package/dist/index.js

@@ -191466,6 +191466,7 @@ function collectInitStatement(node, ctx) {
     loc: getSourceLocation(node, ctx.sourceFile, ctx.filePath),
     freeIdentifiers: extractFreeIdentifiersFromNode(node),
     assignedIdentifiers: extractAssignedIdentifiersFromNode(node),
+    mutatedIdentifiers: extractMutatedIdentifiersFromNode(node),
     origin: { phase: "hydrate", scope: "init", effect: "pure" },
     needsLeadingSemi: leadsWithAsiHazard(body)
   });
@@ -191525,11 +191526,8 @@ function extractAssignedIdentifiersFromNode(node) {
     if (import_typescript11.default.isArrowFunction(n) || import_typescript11.default.isFunctionExpression(n) || import_typescript11.default.isFunctionDeclaration(n) || import_typescript11.default.isMethodDeclaration(n) || import_typescript11.default.isGetAccessorDeclaration(n) || import_typescript11.default.isSetAccessorDeclaration(n) || import_typescript11.default.isConstructorDeclaration(n)) {
       return;
     }
-    if (import_typescript11.default.isBinaryExpression(n)) {
-      const op = n.operatorToken.kind;
-      if (op === import_typescript11.default.SyntaxKind.EqualsToken || op === import_typescript11.default.SyntaxKind.PlusEqualsToken || op === import_typescript11.default.SyntaxKind.MinusEqualsToken || op === import_typescript11.default.SyntaxKind.AsteriskEqualsToken || op === import_typescript11.default.SyntaxKind.SlashEqualsToken || op === import_typescript11.default.SyntaxKind.PercentEqualsToken || op === import_typescript11.default.SyntaxKind.AsteriskAsteriskEqualsToken || op === import_typescript11.default.SyntaxKind.AmpersandEqualsToken || op === import_typescript11.default.SyntaxKind.BarEqualsToken || op === import_typescript11.default.SyntaxKind.CaretEqualsToken || op === import_typescript11.default.SyntaxKind.LessThanLessThanEqualsToken || op === import_typescript11.default.SyntaxKind.GreaterThanGreaterThanEqualsToken || op === import_typescript11.default.SyntaxKind.GreaterThanGreaterThanGreaterThanEqualsToken || op === import_typescript11.default.SyntaxKind.AmpersandAmpersandEqualsToken || op === import_typescript11.default.SyntaxKind.BarBarEqualsToken || op === import_typescript11.default.SyntaxKind.QuestionQuestionEqualsToken) {
-        addFromTarget(n.left);
-      }
+    if (import_typescript11.default.isBinaryExpression(n) && isAssignmentOperator(n.operatorToken.kind)) {
+      addFromTarget(n.left);
     }
     if (import_typescript11.default.isPrefixUnaryExpression(n) || import_typescript11.default.isPostfixUnaryExpression(n)) {
       if (n.operator === import_typescript11.default.SyntaxKind.PlusPlusToken || n.operator === import_typescript11.default.SyntaxKind.MinusMinusToken) {
@@ -191544,6 +191542,87 @@ function extractAssignedIdentifiersFromNode(node) {
     ids.delete(name);
   return ids;
 }
+var MUTATING_METHOD_NAMES = new Set([
+  "push",
+  "pop",
+  "shift",
+  "unshift",
+  "splice",
+  "sort",
+  "reverse",
+  "fill",
+  "copyWithin",
+  "set",
+  "add",
+  "delete",
+  "clear"
+]);
+function extractMutatedIdentifiersFromNode(node) {
+  const ids = new Set;
+  function rootIdentifierOf(expr) {
+    let current = expr;
+    while (true) {
+      if (import_typescript11.default.isIdentifier(current))
+        return current.text;
+      if (import_typescript11.default.isParenthesizedExpression(current)) {
+        current = current.expression;
+        continue;
+      }
+      if (import_typescript11.default.isNonNullExpression(current)) {
+        current = current.expression;
+        continue;
+      }
+      if (import_typescript11.default.isPropertyAccessExpression(current)) {
+        current = current.expression;
+        continue;
+      }
+      if (import_typescript11.default.isElementAccessExpression(current)) {
+        current = current.expression;
+        continue;
+      }
+      return null;
+    }
+  }
+  function visit2(n) {
+    if (import_typescript11.default.isArrowFunction(n) || import_typescript11.default.isFunctionExpression(n) || import_typescript11.default.isFunctionDeclaration(n) || import_typescript11.default.isMethodDeclaration(n) || import_typescript11.default.isGetAccessorDeclaration(n) || import_typescript11.default.isSetAccessorDeclaration(n) || import_typescript11.default.isConstructorDeclaration(n)) {
+      return;
+    }
+    if (import_typescript11.default.isBinaryExpression(n) && isAssignmentOperator(n.operatorToken.kind)) {
+      if (import_typescript11.default.isPropertyAccessExpression(n.left) || import_typescript11.default.isElementAccessExpression(n.left)) {
+        const root = rootIdentifierOf(n.left);
+        if (root)
+          ids.add(root);
+      }
+    } else if (import_typescript11.default.isPrefixUnaryExpression(n) || import_typescript11.default.isPostfixUnaryExpression(n)) {
+      const isIncDec = n.operator === import_typescript11.default.SyntaxKind.PlusPlusToken || n.operator === import_typescript11.default.SyntaxKind.MinusMinusToken;
+      if (isIncDec && (import_typescript11.default.isPropertyAccessExpression(n.operand) || import_typescript11.default.isElementAccessExpression(n.operand))) {
+        const root = rootIdentifierOf(n.operand);
+        if (root)
+          ids.add(root);
+      }
+    } else if (import_typescript11.default.isCallExpression(n)) {
+      const callee = n.expression;
+      if (import_typescript11.default.isPropertyAccessExpression(callee)) {
+        const methodName = callee.name.text;
+        if (MUTATING_METHOD_NAMES.has(methodName)) {
+          const root = rootIdentifierOf(callee.expression);
+          if (root)
+            ids.add(root);
+        } else if (methodName === "assign" && import_typescript11.default.isIdentifier(callee.expression) && callee.expression.text === "Object" && n.arguments.length > 0) {
+          const root = rootIdentifierOf(n.arguments[0]);
+          if (root)
+            ids.add(root);
+        }
+      }
+    }
+    import_typescript11.default.forEachChild(n, visit2);
+  }
+  visit2(node);
+  return ids;
+}
+function isAssignmentOperator(kind) {
+  return kind === import_typescript11.default.SyntaxKind.EqualsToken || kind === import_typescript11.default.SyntaxKind.PlusEqualsToken || kind === import_typescript11.default.SyntaxKind.MinusEqualsToken || kind === import_typescript11.default.SyntaxKind.AsteriskEqualsToken || kind === import_typescript11.default.SyntaxKind.SlashEqualsToken || kind === import_typescript11.default.SyntaxKind.PercentEqualsToken || kind === import_typescript11.default.SyntaxKind.AsteriskAsteriskEqualsToken || kind === import_typescript11.default.SyntaxKind.AmpersandEqualsToken || kind === import_typescript11.default.SyntaxKind.BarEqualsToken || kind === import_typescript11.default.SyntaxKind.CaretEqualsToken || kind === import_typescript11.default.SyntaxKind.LessThanLessThanEqualsToken || kind === import_typescript11.default.SyntaxKind.GreaterThanGreaterThanEqualsToken || kind === import_typescript11.default.SyntaxKind.GreaterThanGreaterThanGreaterThanEqualsToken || kind === import_typescript11.default.SyntaxKind.AmpersandAmpersandEqualsToken || kind === import_typescript11.default.SyntaxKind.BarBarEqualsToken || kind === import_typescript11.default.SyntaxKind.QuestionQuestionEqualsToken;
+}
 function collectLocalDeclarations(root) {
   const names = new Set;
   function addBindingName(name) {
@@ -192701,6 +192780,7 @@ function validateContext(ctx) {
       }));
     }
   }
+  markMutatedConstants(ctx);
   validateInitStatementReferences(ctx);
   validateReactiveFactoryCalls(ctx);
   validateNamespaceQualifiedPrimitives(ctx);
@@ -192864,6 +192944,25 @@ function fileHasUseClientDirective(filePath) {
   visit2(sf);
   return found;
 }
+function markMutatedConstants(ctx) {
+  if (ctx.initStatements.length === 0 || ctx.localConstants.length === 0)
+    return;
+  const mutatedNames = new Set;
+  for (const stmt of ctx.initStatements) {
+    if (stmt.mutatedIdentifiers)
+      for (const name of stmt.mutatedIdentifiers)
+        mutatedNames.add(name);
+    if (stmt.assignedIdentifiers)
+      for (const name of stmt.assignedIdentifiers)
+        mutatedNames.add(name);
+  }
+  if (mutatedNames.size === 0)
+    return;
+  for (const constant of ctx.localConstants) {
+    if (mutatedNames.has(constant.name))
+      constant.mutatedAfterDeclaration = true;
+  }
+}
 function validateInitStatementReferences(ctx) {
   if (ctx.initStatements.length === 0)
     return;
@@ -194113,7 +194212,7 @@ function findAssignedNames(bodyText, candidates) {
     }
   };
   const visit2 = (node) => {
-    if (import_typescript12.default.isBinaryExpression(node) && isAssignmentOperator(node.operatorToken.kind)) {
+    if (import_typescript12.default.isBinaryExpression(node) && isAssignmentOperator2(node.operatorToken.kind)) {
       record(node.left);
     } else if ((import_typescript12.default.isPrefixUnaryExpression(node) || import_typescript12.default.isPostfixUnaryExpression(node)) && (node.operator === import_typescript12.default.SyntaxKind.PlusPlusToken || node.operator === import_typescript12.default.SyntaxKind.MinusMinusToken)) {
       record(node.operand);
@@ -194142,7 +194241,7 @@ function closeOverWritersOfMutableBindings(primaryRefs, declarations, mutableNam
   }
   return reachable;
 }
-function isAssignmentOperator(kind) {
+function isAssignmentOperator2(kind) {
   return kind >= import_typescript12.default.SyntaxKind.FirstAssignment && kind <= import_typescript12.default.SyntaxKind.LastAssignment;
 }
 
@@ -198150,6 +198249,8 @@ function tryResolveTemplateSpanFromConst(expr, ctx) {
     const constInfo = findLocalConst(expr.text, ctx.analyzer);
     if (!constInfo)
       return null;
+    if (constInfo.mutatedAfterDeclaration)
+      return null;
     const ast = parseConstInitializer(constInfo);
     if (!ast)
       return null;
@@ -198166,6 +198267,8 @@ function tryResolveTemplateSpanFromConst(expr, ctx) {
     const constInfo = findLocalConst(expr.expression.text, ctx.analyzer);
     if (!constInfo)
       return null;
+    if (constInfo.mutatedAfterDeclaration)
+      return null;
     const ast = parseConstInitializer(constInfo);
     if (!ast || !import_typescript14.default.isObjectLiteralExpression(ast))
       return null;
@@ -198245,6 +198348,8 @@ function tryResolveIdentifierAsTemplateLiteral(ident, ctx) {
   const constInfo = findLocalConst(ident.text, ctx.analyzer);
   if (!constInfo)
     return null;
+  if (constInfo.mutatedAfterDeclaration)
+    return null;
   const ast = parseConstInitializer(constInfo);
   if (!ast)
     return null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@barefootjs/test",
-  "version": "0.35.4",
+  "version": "0.35.5",
   "description": "Test utilities for BarefootJS - IR-based component testing without a browser",
   "type": "module",
   "main": "./dist/index.js",
@@ -39,7 +39,7 @@
     "directory": "packages/test"
   },
   "dependencies": {
-    "@barefootjs/jsx": "0.35.4"
+    "@barefootjs/jsx": "0.35.5"
   },
   "devDependencies": {
     "typescript": "^5.0.0"