@barefootjs/test 0.31.2 → 0.31.4

package/dist/index.js

@@ -188846,6 +188846,24 @@ function templatePartsToJsExpr(parts, opts) {
   return result;
 }
 
+// ../jsx/src/identifier-pattern.ts
+function withUnicodeFlag(flags) {
+  return flags.includes("u") ? flags : `${flags}u`;
+}
+function escapeIdentifierForRegex(name) {
+  return name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+}
+var ID_BOUNDARY_BEFORE = "(?<![\\p{ID_Continue}$])";
+var ID_BOUNDARY_AFTER = "(?![\\p{ID_Continue}$])";
+function identifierPattern(name, flags = "") {
+  const esc = escapeIdentifierForRegex(name);
+  return new RegExp(`${ID_BOUNDARY_BEFORE}${esc}${ID_BOUNDARY_AFTER}`, withUnicodeFlag(flags));
+}
+function identifierCallPattern(name, flags = "") {
+  const esc = escapeIdentifierForRegex(name);
+  return new RegExp(`${ID_BOUNDARY_BEFORE}${esc}\\s*\\(`, withUnicodeFlag(flags));
+}
+
 // ../jsx/src/scanner/js-scanner.ts
 var import_typescript2 = __toESM(require_typescript(), 1);
 function* iterateJsTokens(text, start = 0, end = text.length) {
@@ -189229,6 +189247,83 @@ function extractFreeIdentifiersFromText(text) {
   return extractFreeIdentifiersFromNode(expr);
 }
 
+// ../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/ir-to-client-js/html-template.ts
 function splitTemplateInterpolations(inner) {
   const parts = [];
@@ -190429,7 +190524,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 +190538,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 +192109,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,
@@ -193631,7 +193728,7 @@ function findReachableNames(primaryRefs, declarations) {
   const reachable = new Set;
   const queue = [];
   for (const name of allNames) {
-    if (new RegExp(`\\b${name}\\b`).test(primaryRefs)) {
+    if (identifierPattern(name).test(primaryRefs)) {
       reachable.add(name);
       queue.push(name);
     }
@@ -193640,7 +193737,7 @@ function findReachableNames(primaryRefs, declarations) {
     const current = queue.shift();
     const body = bodyMap.get(current) || "";
     for (const name of allNames) {
-      if (!reachable.has(name) && new RegExp(`\\b${name}\\b`).test(body)) {
+      if (!reachable.has(name) && identifierPattern(name).test(body)) {
         reachable.add(name);
         queue.push(name);
       }
@@ -194472,8 +194569,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,22 +194647,22 @@ function createTransformContext(analyzer) {
     spreadIdCounter: 0,
     isRoot: true,
     insideComponentChildren: false,
-    loopParams: new Set,
+    scope: BindingScope.EMPTY,
     loopDepth: 0,
     patterns: {
       signals: analyzer.signals.map((s) => ({
         getter: s.getter,
-        pattern: new RegExp(`\\b${s.getter}\\s*\\(`)
+        pattern: identifierCallPattern(s.getter)
       })),
       memos: analyzer.memos.map((m) => ({
         name: m.name,
-        pattern: new RegExp(`\\b${m.name}\\s*\\(`)
+        pattern: identifierCallPattern(m.name)
       })),
-      props: analyzer.propsParams.filter((p) => p.name !== "children").map((p) => ({ name: p.name, pattern: new RegExp(`\\b${p.name}\\b`) })),
+      props: analyzer.propsParams.filter((p) => p.name !== "children").map((p) => ({ name: p.name, pattern: identifierPattern(p.name) })),
       constants: analyzer.localConstants.map((c) => ({
         name: c.name,
         value: c.value,
-        pattern: new RegExp(`\\b${c.name}\\b`)
+        pattern: identifierPattern(c.name)
       }))
     },
     getJS(node) {
@@ -194626,7 +194724,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 +194740,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 +195433,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) => identifierPattern(p).test(exprText));
   const callsReactive = exprCallsReactiveGetters(expr, ctx);
   const hasCalls = exprHasFunctionCalls(expr);
   const needsSlot = reactive || isClientOnly || refsLoopParam || callsReactive || hasCalls;
@@ -195369,7 +195469,7 @@ function transformJsxFunctionCall(callExpr, jsxFunc, ctx, _isClientOnly) {
   const substitutedGetJS = (node) => {
     let text = baseGetJS(node);
     for (const [paramName, argExpr] of substitutions) {
-      text = text.replace(new RegExp(`\\b${paramName}\\b`, "g"), argExpr);
+      text = text.replace(identifierPattern(paramName, "g"), () => argExpr);
     }
     return text;
   };
@@ -195411,7 +195511,7 @@ function transformMultiReturnJsxFunctionCall(callExpr, info, ctx) {
   const substitutedGetJS = (node) => {
     let text = baseGetJS(node);
     for (const [paramName, argExpr] of substitutions) {
-      text = text.replace(new RegExp(`\\b${paramName}\\b`, "g"), argExpr);
+      text = text.replace(identifierPattern(paramName, "g"), () => argExpr);
     }
     return text;
   };
@@ -196422,7 +196522,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 +196681,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 +196743,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 +196830,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 +196861,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 +197608,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 +197624,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 +197703,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,10 +198052,11 @@ 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) {
-    if (new RegExp(`\\b${p}\\b`).test(expr))
+  for (const p of boundNames) {
+    if (identifierPattern(p).test(expr))
       return true;
   }
   return false;
@@ -198026,9 +198135,10 @@ function hasReactiveAttributes(attrs, ctx) {
     if (isSignalOrMemoReference(valueToCheck, ctx) || isPropsReference(valueToCheck, ctx)) {
       return true;
     }
-    if (ctx.loopParams.size > 0) {
-      for (const p of ctx.loopParams) {
-        if (new RegExp(`\\b${p}\\b`).test(valueToCheck))
+    const scopeValueNames = ctx.scope.valueBoundNames();
+    if (scopeValueNames.size > 0) {
+      for (const p of scopeValueNames) {
+        if (identifierPattern(p).test(valueToCheck))
           return true;
       }
     }
@@ -198765,7 +198875,7 @@ class JsxAdapter extends BaseAdapter {
         lines.push(`  const ${signal.getter} = () => ${initialValue}`);
       }
       if (signal.setter) {
-        const setterUsed = new RegExp(`\\b${signal.setter}\\b`).test(setterRefText);
+        const setterUsed = identifierPattern(signal.setter).test(setterRefText);
         if (setterUsed) {
           lines.push(`  const ${signal.setter} = (..._args: any[]) => {}`);
         }
@@ -198785,7 +198895,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 +198905,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 +199013,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 +199023,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))
@@ -198960,6 +199073,7 @@ class JsxAdapter extends BaseAdapter {
 }
 
 // ../jsx/src/adapters/template-imports.ts
+var import_typescript25 = __toESM(require_typescript(), 1);
 var CLIENT_PACKAGE_SOURCES = new Set([
   "@barefootjs/client",
   "@barefootjs/client/runtime"
@@ -199349,9 +199463,9 @@ function registerBuiltinLoweringPlugins() {
     registerLoweringPlugin(plugin);
 }
 // ../jsx/src/combine-client-js.ts
-var import_typescript25 = __toESM(require_typescript(), 1);
-// ../jsx/src/debug.ts
 var import_typescript26 = __toESM(require_typescript(), 1);
+// ../jsx/src/debug.ts
+var import_typescript27 = __toESM(require_typescript(), 1);
 function escapeForIdBoundary(name) {
   return name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 }
@@ -199443,7 +199557,7 @@ function resolveSetters(handler, setterToSignal, fnSetters) {
   return refs;
 }
 // ../jsx/src/profiler.ts
-var import_typescript27 = __toESM(require_typescript(), 1);
+var import_typescript28 = __toESM(require_typescript(), 1);
 
 // ../jsx/src/index.ts
 registerBuiltinLoweringPlugins();

package/package.json

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