@barefootjs/test 0.31.2 → 0.31.3

package/dist/index.js

@@ -190429,7 +190429,8 @@ function visit(node, ctx, targetComponentName, namedExports) {
   if (!ctx.componentNode) {
     collectAmbientGlobals(node, ctx);
   }
-  if (import_typescript9.default.isVariableStatement(node) && !ctx.componentNode) {
+  const isDeclareStatement = import_typescript9.default.isVariableStatement(node) && (node.modifiers?.some((m) => m.kind === import_typescript9.default.SyntaxKind.DeclareKeyword) ?? false);
+  if (import_typescript9.default.isVariableStatement(node) && !ctx.componentNode && !isDeclareStatement) {
     const isExported = node.modifiers?.some((m) => m.kind === import_typescript9.default.SyntaxKind.ExportKeyword) ?? false;
     const isLet = (node.declarationList.flags & import_typescript9.default.NodeFlags.Let) !== 0;
     const isModuleClientDirective = hasLeadingClientDirectiveOnStatement(node, ctx.sourceFile);
@@ -190442,7 +190443,7 @@ function visit(node, ctx, targetComponentName, namedExports) {
         }
         continue;
       }
-      if (import_typescript9.default.isIdentifier(decl.name) && decl.initializer && !isArrowComponentFunction(decl)) {
+      if (import_typescript9.default.isIdentifier(decl.name) && (decl.initializer || isLet) && !isArrowComponentFunction(decl)) {
         collectConstant(decl, ctx, true, isLet ? "let" : "const", isExported);
       }
     }
@@ -192013,6 +192014,7 @@ function collectConstant(node, ctx, _isModule, declarationKind = "const", isExpo
     value,
     parsed,
     typedValue: typedValue !== value ? typedValue : undefined,
+    typeAnnotation: node.type ? node.type.getText(ctx.sourceFile) : undefined,
     valueBranches,
     declarationKind,
     isExported,
@@ -194292,6 +194294,83 @@ var toLocaleDatePlugin = {
   }
 };
 
+// ../jsx/src/scope/binding-scope.ts
+class BindingScope {
+  frames;
+  static EMPTY = new BindingScope([]);
+  constructor(frames) {
+    this.frames = frames;
+  }
+  enterLoopRow(loop) {
+    const bindings = new Map;
+    if (loop.paramBindings && loop.paramBindings.length > 0) {
+      for (const b of loop.paramBindings)
+        bindings.set(b.name, { source: "destructure" });
+    } else {
+      bindings.set(loop.param, { source: "item" });
+    }
+    if (loop.index != null)
+      bindings.set(loop.index, { source: "index" });
+    for (const name of loop.preamble?.declaredNames ?? [])
+      bindings.set(name, { source: "preamble" });
+    const frame = { kind: "loop-row", bindings };
+    return new BindingScope([frame, ...this.frames]);
+  }
+  enterCallback(params) {
+    const bindings = new Map;
+    for (const name of params)
+      bindings.set(name, { source: "param" });
+    const frame = { kind: "callback", bindings };
+    return new BindingScope([frame, ...this.frames]);
+  }
+  isBound(name) {
+    for (const frame of this.frames) {
+      if (frame.bindings.has(name))
+        return true;
+    }
+    return false;
+  }
+  lookup(name) {
+    for (let depth = 0;depth < this.frames.length; depth++) {
+      const frame = this.frames[depth];
+      const binding = frame.bindings.get(name);
+      if (binding)
+        return { depth, frame, binding };
+    }
+    return null;
+  }
+  boundNames() {
+    if (this.boundNamesCache)
+      return this.boundNamesCache;
+    const names = new Set;
+    for (const frame of this.frames) {
+      for (const name of frame.bindings.keys())
+        names.add(name);
+    }
+    this.boundNamesCache = names;
+    return names;
+  }
+  boundNamesCache;
+  valueBoundNamesCache;
+  valueBoundNames() {
+    if (this.valueBoundNamesCache)
+      return this.valueBoundNamesCache;
+    const names = new Set;
+    for (const frame of this.frames) {
+      for (const [name, binding] of frame.bindings) {
+        if (binding.source === "item" || binding.source === "index" || binding.source === "destructure") {
+          names.add(name);
+        }
+      }
+    }
+    this.valueBoundNamesCache = names;
+    return names;
+  }
+  asShadowPredicate() {
+    return (name) => this.isBound(name);
+  }
+}
+
 // ../jsx/src/jsx-to-ir.ts
 var CLIENT_DIRECTIVE_INTERIOR_RE2 = /^\s*@client\s*$/;
 var BLOCK_COMMENT_RE2 = /\/\*([\s\S]*?)\*\//g;
@@ -194472,8 +194551,9 @@ function rewriteBarePropRefs2(text, expr, ctx) {
   let propNames = getDestructuredPropNames(ctx);
   if (!propNames)
     return dateLowered === text ? undefined : dateLowered;
-  if (ctx.loopParams.size > 0) {
-    const filtered = new Set([...propNames].filter((n) => !ctx.loopParams.has(n)));
+  const shadowingNames = ctx.scope.boundNames();
+  if (shadowingNames.size > 0) {
+    const filtered = new Set([...propNames].filter((n) => !shadowingNames.has(n)));
     if (filtered.size === 0)
       return dateLowered === text ? undefined : dateLowered;
     propNames = filtered;
@@ -194549,7 +194629,7 @@ function createTransformContext(analyzer) {
     spreadIdCounter: 0,
     isRoot: true,
     insideComponentChildren: false,
-    loopParams: new Set,
+    scope: BindingScope.EMPTY,
     loopDepth: 0,
     patterns: {
       signals: analyzer.signals.map((s) => ({
@@ -194626,7 +194706,8 @@ function generateSpreadSlotId(ctx) {
   return `Spread_${ctx.spreadIdCounter++}`;
 }
 function makeBindingEnv(ctx) {
-  const loopKey = ctx.loopParams.size === 0 ? "" : Array.from(ctx.loopParams).sort().join("\x00");
+  const boundNames = ctx.scope.valueBoundNames();
+  const loopKey = boundNames.size === 0 ? "" : Array.from(boundNames).sort().join("\x00");
   if (ctx._bindingEnv && ctx._bindingEnvLoopKey === loopKey) {
     return ctx._bindingEnv;
   }
@@ -194641,7 +194722,7 @@ function makeBindingEnv(ctx) {
     localFunctions: a.localFunctions,
     imports: a.imports,
     ambientGlobals: a.ambientGlobals,
-    loopParams: new Set(ctx.loopParams),
+    loopParams: boundNames,
     checker: a.checker
   };
   ctx._bindingEnv = env;
@@ -195334,7 +195415,8 @@ function transformExpressionInner(expr, ctx, node, isClientOnly) {
     freeRefs
   };
   const reactive = isReactiveExpression(exprText, ctx, expr) || isReactiveOrigin(origin);
-  const refsLoopParam = ctx.loopParams.size > 0 && Array.from(ctx.loopParams).some((p) => new RegExp(`\\b${p}\\b`).test(exprText));
+  const scopeValueNames = ctx.scope.valueBoundNames();
+  const refsLoopParam = scopeValueNames.size > 0 && Array.from(scopeValueNames).some((p) => new RegExp(`\\b${p}\\b`).test(exprText));
   const callsReactive = exprCallsReactiveGetters(expr, ctx);
   const hasCalls = exprHasFunctionCalls(expr);
   const needsSlot = reactive || isClientOnly || refsLoopParam || callsReactive || hasCalls;
@@ -196422,7 +196504,7 @@ function extractItemConditionalKey(cond) {
   return a ?? b;
 }
 function transformMapCall(node, ctx, isClientOnly = false, method = "map") {
-  const isNested = ctx.loopParams.size > 0;
+  const isNested = ctx.scope.valueBoundNames().size > 0;
   const diagCountAtEntry = ctx.analyzer.errors.length;
   const depth = ctx.loopDepth;
   const propAccess = node.expression;
@@ -196581,14 +196663,8 @@ function transformMapCall(node, ctx, isClientOnly = false, method = "map") {
         indexType = secondParam.type.getText(ctx.sourceFile);
       }
     }
-    if (paramBindings) {
-      for (const b of paramBindings)
-        ctx.loopParams.add(b.name);
-    } else {
-      ctx.loopParams.add(param);
-    }
-    if (index)
-      ctx.loopParams.add(index);
+    const savedScope = ctx.scope;
+    ctx.scope = ctx.scope.enterLoopRow({ param, index, paramBindings });
     ctx.loopDepth++;
     const tryTransformRenderableBody = (expr) => {
       if (!import_typescript12.default.isBinaryExpression(expr))
@@ -196649,6 +196725,24 @@ function transformMapCall(node, ctx, isClientOnly = false, method = "map") {
         }
       }
       const returnStmt = children.length === 0 ? body.statements.find((s) => import_typescript12.default.isReturnStatement(s) && s.expression != null) : undefined;
+      let rowScopeBeforePreamble = null;
+      if (returnStmt) {
+        const preambleNames = new Set;
+        for (const stmt of body.statements) {
+          if (stmt === returnStmt)
+            break;
+          collectPreambleDeclaredNames(stmt, preambleNames);
+        }
+        if (preambleNames.size > 0) {
+          rowScopeBeforePreamble = ctx.scope;
+          ctx.scope = savedScope.enterLoopRow({
+            param,
+            index,
+            paramBindings,
+            preamble: { declaredNames: [...preambleNames] }
+          });
+        }
+      }
       if (returnStmt && returnStmt.expression) {
         let returnExpr = returnStmt.expression;
         while (import_typescript12.default.isParenthesizedExpression(returnExpr)) {
@@ -196718,6 +196812,9 @@ function transformMapCall(node, ctx, isClientOnly = false, method = "map") {
           }
         }
       }
+      if (rowScopeBeforePreamble) {
+        ctx.scope = rowScopeBeforePreamble;
+      }
       if (method === "flatMap" && children.length === 0 && !flatMapProjectionCall(body)) {
         flatMapCallback = buildFlatMapCallback(callback, body, ctx);
       }
@@ -196746,14 +196843,7 @@ function transformMapCall(node, ctx, isClientOnly = false, method = "map") {
         }
       }));
     }
-    if (paramBindings) {
-      for (const b of paramBindings)
-        ctx.loopParams.delete(b.name);
-    } else {
-      ctx.loopParams.delete(param);
-    }
-    if (index)
-      ctx.loopParams.delete(index);
+    ctx.scope = savedScope;
     ctx.loopDepth--;
   }
   if (children.length === 0 && !flatMapCallback) {
@@ -197500,7 +197590,7 @@ function parseTemplateLiteral(expr, ctx) {
 }
 function tryResolveTemplateSpanFromConst(expr, ctx) {
   if (import_typescript12.default.isIdentifier(expr)) {
-    if (ctx.loopParams.has(expr.text))
+    if (ctx.scope.isBound(expr.text))
       return null;
     const constInfo = findLocalConst(expr.text, ctx.analyzer);
     if (!constInfo)
@@ -197516,7 +197606,7 @@ function tryResolveTemplateSpanFromConst(expr, ctx) {
   if (import_typescript12.default.isElementAccessExpression(expr)) {
     if (!import_typescript12.default.isIdentifier(expr.expression))
       return null;
-    if (ctx.loopParams.has(expr.expression.text))
+    if (ctx.scope.isBound(expr.expression.text))
       return null;
     const constInfo = findLocalConst(expr.expression.text, ctx.analyzer);
     if (!constInfo)
@@ -197595,7 +197685,7 @@ function hasDynamicTagBinding(name, sourceFile) {
   return found;
 }
 function tryResolveIdentifierAsTemplateLiteral(ident, ctx) {
-  if (ctx.loopParams.has(ident.text))
+  if (ctx.scope.isBound(ident.text))
     return null;
   const constInfo = findLocalConst(ident.text, ctx.analyzer);
   if (!constInfo)
@@ -197944,9 +198034,10 @@ function isSignalOrMemoArray(array, ctx) {
   return false;
 }
 function referencesLoopParam(expr, ctx) {
-  if (ctx.loopParams.size === 0)
+  const boundNames = ctx.scope.valueBoundNames();
+  if (boundNames.size === 0)
     return false;
-  for (const p of ctx.loopParams) {
+  for (const p of boundNames) {
     if (new RegExp(`\\b${p}\\b`).test(expr))
       return true;
   }
@@ -198026,8 +198117,9 @@ function hasReactiveAttributes(attrs, ctx) {
     if (isSignalOrMemoReference(valueToCheck, ctx) || isPropsReference(valueToCheck, ctx)) {
       return true;
     }
-    if (ctx.loopParams.size > 0) {
-      for (const p of ctx.loopParams) {
+    const scopeValueNames = ctx.scope.valueBoundNames();
+    if (scopeValueNames.size > 0) {
+      for (const p of scopeValueNames) {
         if (new RegExp(`\\b${p}\\b`).test(valueToCheck))
           return true;
       }
@@ -198785,7 +198877,7 @@ class JsxAdapter extends BaseAdapter {
         continue;
       const keyword = constant.declarationKind ?? "const";
       if (!constant.value) {
-        const typeAnnotation = preserveTypes && constant.type ? `: ${constant.type.raw}` : "";
+        const typeAnnotation = preserveTypes && (constant.typeAnnotation ?? constant.type) ? `: ${constant.typeAnnotation ?? constant.type?.raw}` : "";
         lines.push(`  ${keyword} ${constant.name}${typeAnnotation}`);
         continue;
       }
@@ -198795,7 +198887,8 @@ class JsxAdapter extends BaseAdapter {
       if (!reachable.has(constant.name))
         continue;
       const constValue = preserveTypes ? constant.typedValue ?? constant.value : constant.value;
-      lines.push(`  ${keyword} ${constant.name} = ${constValue}`);
+      const letTypeAnnotation = preserveTypes && keyword === "let" && constant.typeAnnotation ? `: ${constant.typeAnnotation}` : "";
+      lines.push(`  ${keyword} ${constant.name}${letTypeAnnotation} = ${constValue}`);
     }
     for (const func of localFunctions) {
       if (moduleScopeNames.has(func.name))
@@ -198902,7 +198995,8 @@ class JsxAdapter extends BaseAdapter {
       const keyword = c.declarationKind ?? "const";
       const exportKw = c.isExported ? "export " : "";
       if (!c.value) {
-        entries.push({ line: c.loc.start.line, text: `${exportKw}${keyword} ${c.name}` });
+        const typeAnnotation = preserveTypes && (c.typeAnnotation ?? c.type) ? `: ${c.typeAnnotation ?? c.type?.raw}` : "";
+        entries.push({ line: c.loc.start.line, text: `${exportKw}${keyword} ${c.name}${typeAnnotation}` });
         continue;
       }
       const trimmed = c.value.trim();
@@ -198911,7 +199005,8 @@ class JsxAdapter extends BaseAdapter {
       if (c.isExported && /^createContext\b/.test(trimmed))
         continue;
       const value = preserveTypes ? c.typedValue ?? c.value : c.value;
-      entries.push({ line: c.loc.start.line, text: `${exportKw}${keyword} ${c.name} = ${value}` });
+      const letTypeAnnotation = preserveTypes && keyword === "let" && c.typeAnnotation ? `: ${c.typeAnnotation}` : "";
+      entries.push({ line: c.loc.start.line, text: `${exportKw}${keyword} ${c.name}${letTypeAnnotation} = ${value}` });
     }
     for (const f of ir.metadata.localFunctions) {
       if (!f.isModule || !moduleNames.has(f.name))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@barefootjs/test",
-  "version": "0.31.2",
+  "version": "0.31.3",
   "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.31.2"
+    "@barefootjs/jsx": "0.31.3"
   },
   "devDependencies": {
     "typescript": "^5.0.0"