@barefootjs/hono 0.33.4 → 0.34.0

package/dist/conformance-pins.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"conformance-pins.d.ts","sourceRoot":"","sources":["../src/conformance-pins.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AAEtD,eAAO,MAAM,eAAe,EAAE,eAU7B,CAAA"}
+{"version":3,"file":"conformance-pins.d.ts","sourceRoot":"","sources":["../src/conformance-pins.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AAEtD,eAAO,MAAM,eAAe,EAAE,eAiB7B,CAAA"}

package/dist/index.js

@@ -667,7 +667,8 @@ var conformancePins = {
   "date-method-uncatalogued": [{ code: "BF021", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2356" }],
   "rich-prop-client-read": [{ code: "BF049", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2648" }],
   "jsx-element-prop-ternary": [{ code: "BF021", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2667" }],
-  "jsx-element-prop-array": [{ code: "BF021", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2667" }]
+  "jsx-element-prop-array": [{ code: "BF021", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2667" }],
+  "namespace-import-primitive": [{ code: "BF013", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2771" }]
 };
 export {
   honoAdapter,

package/dist/test-render.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"test-render.d.ts","sourceRoot":"","sources":["../src/test-render.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AAStD,MAAM,WAAW,aAAa;IAC5B,sBAAsB;IACtB,MAAM,EAAE,MAAM,CAAA;IACd,8BAA8B;IAC9B,OAAO,EAAE,eAAe,CAAA;IACxB,iCAAiC;IACjC,KAAK,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;IAC/B,qDAAqD;IACrD,UAAU,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;IACnC;;;;;;;;OAQG;IACH,gBAAgB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;IACzC;;;;;;;OAOG;IACH,aAAa,CAAC,EAAE,MAAM,CAAA;CACvB;AA2BD,wBAAsB,mBAAmB,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,MAAM,CAAC,CAsLjF"}
+{"version":3,"file":"test-render.d.ts","sourceRoot":"","sources":["../src/test-render.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AAStD,MAAM,WAAW,aAAa;IAC5B,sBAAsB;IACtB,MAAM,EAAE,MAAM,CAAA;IACd,8BAA8B;IAC9B,OAAO,EAAE,eAAe,CAAA;IACxB,iCAAiC;IACjC,KAAK,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;IAC/B,qDAAqD;IACrD,UAAU,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;IACnC;;;;;;;;OAQG;IACH,gBAAgB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;IACzC;;;;;;;OAOG;IACH,aAAa,CAAC,EAAE,MAAM,CAAA;CACvB;AAkDD,wBAAsB,mBAAmB,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,MAAM,CAAC,CAqMjF"}

package/dist/vite.js

@@ -8,10 +8,22 @@ import { devModuleUrl, loadManifest, resolveDevOrigin, resolveScriptAssets, toPo
 import ts25 from "typescript";
 
 // ../jsx/src/analyzer.ts
-import ts9 from "typescript";
+import ts10 from "typescript";
 
 // ../jsx/src/expression-parser.ts
 import ts from "typescript";
+
+// ../jsx/src/lowering-registry.ts
+var plugins = [];
+function registerLoweringPlugin(plugin) {
+  const existing = plugins.findIndex((p) => p.name === plugin.name);
+  if (existing >= 0)
+    plugins[existing] = plugin;
+  else
+    plugins.push(plugin);
+}
+
+// ../jsx/src/expression-parser.ts
 var UNSUPPORTED_METHODS = new Set([
   "filter",
   "map",
@@ -84,7 +96,7 @@ var EVAL_MATH_METHODS = new Set([
 ]);
 
 // ../jsx/src/prop-rewrite.ts
-import ts5 from "typescript";
+import ts7 from "typescript";
 
 // ../jsx/src/ir-to-client-js/utils.ts
 import ts3 from "typescript";
@@ -242,18 +254,45 @@ function escapeHtml(text) {
   return text.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;");
 }
 // ../jsx/src/ir-to-client-js/csr-substitute.ts
+import ts5 from "typescript";
+
+// ../jsx/src/props-binding.ts
 import ts4 from "typescript";
+function isIdentifierName(key) {
+  if (key.length === 0)
+    return false;
+  for (let i = 0;i < key.length; ) {
+    const cp = key.codePointAt(i);
+    const ok = i === 0 ? ts4.isIdentifierStart(cp, ts4.ScriptTarget.Latest) : ts4.isIdentifierPart(cp, ts4.ScriptTarget.Latest);
+    if (!ok)
+      return false;
+    i += cp > 65535 ? 2 : 1;
+  }
+  return true;
+}
+function propsDestructureBinding(p) {
+  const callerKey = p.sourceName ?? p.name;
+  const localName = p.name;
+  const binding = callerKey === localName ? localName : `${isIdentifierName(callerKey) ? callerKey : JSON.stringify(callerKey)}: ${localName}`;
+  return p.defaultValue ? `${binding} = ${p.defaultValue}` : binding;
+}
+var EMPTY_SET = new Set;
+
+// ../jsx/src/ir-to-client-js/csr-substitute.ts
 function extractFreeIdentifiersFromText(text) {
   if (!text || text.trim().length === 0)
     return new Set;
-  const sf = ts4.createSourceFile("__free_ids__.ts", `(${text});`, ts4.ScriptTarget.Latest, true, ts4.ScriptKind.TS);
+  const sf = ts5.createSourceFile("__free_ids__.ts", `(${text});`, ts5.ScriptTarget.Latest, true, ts5.ScriptKind.TS);
   const stmt = sf.statements[0];
-  if (!stmt || !ts4.isExpressionStatement(stmt))
+  if (!stmt || !ts5.isExpressionStatement(stmt))
     return new Set;
-  const expr = ts4.isParenthesizedExpression(stmt.expression) ? stmt.expression.expression : stmt.expression;
+  const expr = ts5.isParenthesizedExpression(stmt.expression) ? stmt.expression.expression : stmt.expression;
   return extractFreeIdentifiersFromNode(expr);
 }
 
+// ../jsx/src/ir-to-client-js/rewrite-props-object.ts
+import ts6 from "typescript";
+
 // ../jsx/src/scope/binding-scope.ts
 class BindingScope {
   frames;
@@ -375,27 +414,6 @@ var SKELETON_PATH_FORCE_CLOSE_GROUPS = [
   new Set(["li"])
 ];
 
-// ../jsx/src/props-binding.ts
-import ts6 from "typescript";
-function isIdentifierName(key) {
-  if (key.length === 0)
-    return false;
-  for (let i = 0;i < key.length; ) {
-    const cp = key.codePointAt(i);
-    const ok = i === 0 ? ts6.isIdentifierStart(cp, ts6.ScriptTarget.Latest) : ts6.isIdentifierPart(cp, ts6.ScriptTarget.Latest);
-    if (!ok)
-      return false;
-    i += cp > 65535 ? 2 : 1;
-  }
-  return true;
-}
-function propsDestructureBinding(p) {
-  const callerKey = p.sourceName ?? p.name;
-  const localName = p.name;
-  const binding = callerKey === localName ? localName : `${isIdentifierName(callerKey) ? callerKey : JSON.stringify(callerKey)}: ${localName}`;
-  return p.defaultValue ? `${binding} = ${p.defaultValue}` : binding;
-}
-
 // ../jsx/src/instrumentation.ts
 var _counters = freshCounters();
 function freshCounters() {
@@ -409,20 +427,21 @@ function freshCounters() {
 }
 
 // ../jsx/src/analyzer-context.ts
-import ts8 from "typescript";
+import ts9 from "typescript";
 
 // ../jsx/src/strip-types.ts
-import ts7 from "typescript";
+import ts8 from "typescript";
 
 // ../jsx/src/analyzer-context.ts
-var _typePrinter = ts8.createPrinter({ removeComments: true, omitTrailingSemicolon: true });
-var _blankTypeSourceFile = ts8.createSourceFile("__bf_types__.ts", "", ts8.ScriptTarget.Latest);
+var _typePrinter = ts9.createPrinter({ removeComments: true, omitTrailingSemicolon: true });
+var _blankTypeSourceFile = ts9.createSourceFile("__bf_types__.ts", "", ts9.ScriptTarget.Latest);
 
 // ../jsx/src/errors.ts
 var ErrorCodes = {
   MISSING_USE_CLIENT: "BF001",
   CLIENT_IMPORTING_SERVER: "BF003",
   SIGNAL_OUTSIDE_COMPONENT: "BF011",
+  PRIMITIVE_VIA_NAMESPACE_IMPORT: "BF013",
   UNSUPPORTED_JSX_PATTERN: "BF021",
   MISSING_KEY_IN_LIST: "BF023",
   MISSING_KEY_IN_NESTED_LIST: "BF024",
@@ -455,6 +474,7 @@ var errorMessages = {
   [ErrorCodes.MISSING_USE_CLIENT]: "'use client' directive required for components with createSignal or event handlers",
   [ErrorCodes.CLIENT_IMPORTING_SERVER]: "Client component cannot import server component",
   [ErrorCodes.SIGNAL_OUTSIDE_COMPONENT]: "Module-level reactive declaration (createSignal / createMemo) is not allowed. " + "The downstream codegen drops the declaration silently and every reference becomes a ReferenceError at SSR and at hydrate. " + "Move the declaration inside a component function so each mount gets its own state.",
+  [ErrorCodes.PRIMITIVE_VIA_NAMESPACE_IMPORT]: "Reactive primitive called through a namespace import of '@barefootjs/client' (import * as ns) is not recognized; " + "the declaration is dropped and its references throw ReferenceError at hydrate. Import the primitive by name instead.",
   [ErrorCodes.UNSUPPORTED_JSX_PATTERN]: "Unsupported JSX pattern",
   [ErrorCodes.MISSING_KEY_IN_LIST]: "Missing key attribute in list rendering. Add a key prop for efficient updates",
   [ErrorCodes.MISSING_KEY_IN_NESTED_LIST]: "Nested .map() loop requires key attribute for event delegation. Add a key prop to elements in the inner loop",
@@ -646,46 +666,46 @@ function extractFreeIdentifiersFromNode(node) {
   const ids = new Set;
   const boundNames = new Set;
   function addBindingNames(name, out) {
-    if (ts9.isIdentifier(name))
+    if (ts10.isIdentifier(name))
       out.push(name.text);
-    else if (ts9.isObjectBindingPattern(name))
+    else if (ts10.isObjectBindingPattern(name))
       name.elements.forEach((e) => addBindingNames(e.name, out));
-    else if (ts9.isArrayBindingPattern(name))
+    else if (ts10.isArrayBindingPattern(name))
       name.elements.forEach((e) => {
-        if (!ts9.isOmittedExpression(e))
+        if (!ts10.isOmittedExpression(e))
           addBindingNames(e.name, out);
       });
   }
   function visit(n) {
-    if (ts9.isTypeNode(n))
+    if (ts10.isTypeNode(n))
       return;
-    if (ts9.isIdentifier(n)) {
+    if (ts10.isIdentifier(n)) {
       const parent = n.parent;
-      if (parent && ts9.isPropertyAccessExpression(parent) && parent.name === n)
+      if (parent && ts10.isPropertyAccessExpression(parent) && parent.name === n)
         return;
-      if (parent && ts9.isPropertyAssignment(parent) && parent.name === n)
+      if (parent && ts10.isPropertyAssignment(parent) && parent.name === n)
         return;
-      if (parent && ts9.isParameter(parent) && parent.name === n)
+      if (parent && ts10.isParameter(parent) && parent.name === n)
         return;
-      if (parent && ts9.isVariableDeclaration(parent) && parent.name === n)
+      if (parent && ts10.isVariableDeclaration(parent) && parent.name === n)
         return;
       if (boundNames.has(n.text))
         return;
       ids.add(n.text);
       return;
     }
-    if (ts9.isArrowFunction(n)) {
+    if (ts10.isArrowFunction(n)) {
       const params = [];
       for (const p of n.parameters)
         addBindingNames(p.name, params);
       for (const name of params)
         boundNames.add(name);
-      ts9.forEachChild(n, visit);
+      ts10.forEachChild(n, visit);
       for (const name of params)
         boundNames.delete(name);
       return;
     }
-    ts9.forEachChild(n, visit);
+    ts10.forEachChild(n, visit);
   }
   visit(node);
   return ids;
@@ -708,7 +728,7 @@ var REACTIVE_PRIMITIVES = new Set([
 ]);
 
 // ../jsx/src/jsx-to-ir.ts
-import ts13 from "typescript";
+import ts14 from "typescript";
 
 // ../jsx/src/types.ts
 var SCOPE_FORBIDDEN = {
@@ -744,7 +764,7 @@ var REACTIVE_BINDING_KINDS = new Set([
 ]);
 
 // ../jsx/src/module-exports.ts
-import ts10 from "typescript";
+import ts11 from "typescript";
 function formatParamWithType(p) {
   const rest = p.isRest ? "..." : "";
   const optional = p.optional ? "?" : "";
@@ -779,21 +799,21 @@ function findAssignedNames(bodyText, candidates) {
   const assigned = new Set;
   if (candidates.size === 0)
     return assigned;
-  const sf = ts10.createSourceFile("bf-assignment-scan.tsx", bodyText, ts10.ScriptTarget.Latest, false, ts10.ScriptKind.TSX);
+  const sf = ts11.createSourceFile("bf-assignment-scan.tsx", bodyText, ts11.ScriptTarget.Latest, false, ts11.ScriptKind.TSX);
   const record = (target) => {
-    if (ts10.isIdentifier(target) && candidates.has(target.text)) {
+    if (ts11.isIdentifier(target) && candidates.has(target.text)) {
       assigned.add(target.text);
     }
   };
   const visit = (node) => {
-    if (ts10.isBinaryExpression(node) && isAssignmentOperator(node.operatorToken.kind)) {
+    if (ts11.isBinaryExpression(node) && isAssignmentOperator(node.operatorToken.kind)) {
       record(node.left);
-    } else if ((ts10.isPrefixUnaryExpression(node) || ts10.isPostfixUnaryExpression(node)) && (node.operator === ts10.SyntaxKind.PlusPlusToken || node.operator === ts10.SyntaxKind.MinusMinusToken)) {
+    } else if ((ts11.isPrefixUnaryExpression(node) || ts11.isPostfixUnaryExpression(node)) && (node.operator === ts11.SyntaxKind.PlusPlusToken || node.operator === ts11.SyntaxKind.MinusMinusToken)) {
       record(node.operand);
     }
-    ts10.forEachChild(node, visit);
+    ts11.forEachChild(node, visit);
   };
-  ts10.forEachChild(sf, visit);
+  ts11.forEachChild(sf, visit);
   return assigned;
 }
 function closeOverWritersOfMutableBindings(primaryRefs, declarations, mutableNames) {
@@ -816,14 +836,14 @@ function closeOverWritersOfMutableBindings(primaryRefs, declarations, mutableNam
   return reachable;
 }
 function isAssignmentOperator(kind) {
-  return kind >= ts10.SyntaxKind.FirstAssignment && kind <= ts10.SyntaxKind.LastAssignment;
+  return kind >= ts11.SyntaxKind.FirstAssignment && kind <= ts11.SyntaxKind.LastAssignment;
 }
 
 // ../jsx/src/reactivity-checker.ts
-import ts11 from "typescript";
+import ts12 from "typescript";
 
 // ../jsx/src/free-refs.ts
-import ts12 from "typescript";
+import ts13 from "typescript";
 var _bindingMapCache = new WeakMap;
 
 // ../jsx/src/to-locale-date-lowering.ts
@@ -1271,28 +1291,16 @@ var KEYWORDS_AND_GLOBALS = new Set([
 ]);
 
 // ../jsx/src/ir-to-client-js/walk-prop-accesses.ts
-import ts14 from "typescript";
-
-// ../jsx/src/ir-to-client-js/imports.ts
-import ts16 from "typescript";
-
-// ../jsx/src/value-references.ts
 import ts15 from "typescript";
 
-// ../jsx/src/relocate.ts
+// ../jsx/src/ir-to-client-js/imports.ts
 import ts17 from "typescript";
 
-// ../jsx/src/lowering-registry.ts
-var plugins = [];
-function registerLoweringPlugin(plugin) {
-  const existing = plugins.findIndex((p) => p.name === plugin.name);
-  if (existing >= 0)
-    plugins[existing] = plugin;
-  else
-    plugins.push(plugin);
-}
+// ../jsx/src/value-references.ts
+import ts16 from "typescript";
 
 // ../jsx/src/relocate.ts
+import ts18 from "typescript";
 var REGISTRY_SAFE_BINDING_KINDS = new Set([
   "global",
   "module-import",
@@ -1459,10 +1467,10 @@ function formatDateLocalNames(metadata) {
 }
 
 // ../jsx/src/ir-to-client-js/prune-unused-prop-extractions.ts
-import ts18 from "typescript";
+import ts19 from "typescript";
 
 // ../jsx/src/ir-to-client-js/control-flow/plan/lazy-preamble.ts
-import ts19 from "typescript";
+import ts20 from "typescript";
 var NO_PREAMBLE = {
   lazySafe: true,
   facts: { declaredNames: new Set, freeNames: new Set }
@@ -1512,7 +1520,7 @@ var INERT_BINDING_GLOBALS = new Set([
 ]);
 
 // ../jsx/src/ir-to-client-js/emit-reactive.ts
-import ts20 from "typescript";
+import ts21 from "typescript";
 
 // ../jsx/src/ir-to-client-js/control-flow/stringify/event-delegation.ts
 var NON_BUBBLING_EVENTS = new Set([
@@ -1526,9 +1534,6 @@ var NON_BUBBLING_EVENTS = new Set([
   "pointerleave"
 ]);
 
-// ../jsx/src/ir-to-client-js/rewrite-props-object.ts
-import ts21 from "typescript";
-
 // ../jsx/src/ir-to-client-js/source-map.ts
 var BASE64_CHARS = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/";
 function encodeVLQ(value) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@barefootjs/hono",
-  "version": "0.33.4",
+  "version": "0.34.0",
   "description": "Hono integration for BarefootJS",
   "type": "module",
   "main": "./dist/index.js",
@@ -122,7 +122,7 @@
   },
   "devDependencies": {
     "@barefootjs/adapter-tests": "0.1.0",
-    "@barefootjs/vite": "0.33.4",
+    "@barefootjs/vite": "0.34.0",
     "@types/jsdom": "^27.0.0",
     "hono": "^4.6.0",
     "jsdom": "^27.3.0",

package/src/conformance-pins.ts

@@ -21,4 +21,11 @@ export const conformancePins: ConformancePins = {
   // `rich-prop-client-read` above.
   'jsx-element-prop-ternary': [{ code: 'BF021', severity: 'error', issue: 'https://github.com/piconic-ai/barefootjs/issues/2667' }],
   'jsx-element-prop-array': [{ code: 'BF021', severity: 'error', issue: 'https://github.com/piconic-ai/barefootjs/issues/2667' }],
+  // #2771: a reactive primitive invoked through a namespace import
+  // (`import * as bf from '@barefootjs/client'`, `bf.createSignal(...)`)
+  // that the analyzer's checker-less fast path cannot recognize refuses
+  // loudly (BF013) instead of silently dropping the declaration — fired
+  // in the shared analyzer pass ahead of any adapter's `adapter.generate()`,
+  // so all nine adapters (including Hono) pin this identically.
+  'namespace-import-primitive': [{ code: 'BF013', severity: 'error', issue: 'https://github.com/piconic-ai/barefootjs/issues/2771' }],
 }

package/src/test-render.ts

@@ -70,6 +70,29 @@ function stripModuleExports(code: string): string {
     .replace(/\bexport\s+(default\s+)?/g, '')
 }
 
+/**
+ * A single named-import line for a child that's being INLINED (its
+ * declarations are spliced in verbatim, see `stripModuleExports` above) —
+ * returns `const <local> = <declared>` for every aliased specifier
+ * (`{ Label as AliasedLabel }`), or `null` when every specifier is a bare,
+ * non-aliased name (nothing left to bind; dropping the whole line is
+ * correct). A default-import specifier (`import Label from …`) has no
+ * declared name to alias against and is left unhandled — dropped like
+ * before, matching what a bare inlined child needs.
+ */
+function aliasDeclarationsFor(importLine: string): string | null {
+  const namedMatch = importLine.match(/\{([^}]*)\}/)
+  if (!namedMatch) return null
+  const aliases = namedMatch[1]
+    .split(',')
+    .map(spec => spec.trim())
+    .filter(Boolean)
+    .map(spec => spec.match(/^(\w+)\s+as\s+(\w+)$/))
+    .filter((m): m is RegExpMatchArray => m !== null)
+    .map(([, declared, local]) => `const ${local} = ${declared};`)
+  return aliases.length > 0 ? aliases.join(' ') : null
+}
+
 export async function renderHonoComponent(options: RenderOptions): Promise<string> {
   const { source, adapter, props, components, componentModules, componentName: requestedName } = options
 
@@ -171,7 +194,22 @@ export async function renderHonoComponent(options: RenderOptions): Promise<strin
         const [, prefix, importPath, suffix] = importMatch
         const moduleKey = matchKey(importPath, moduleTempPaths.keys())
         if (moduleKey) return `${prefix}${moduleTempPaths.get(moduleKey)}${suffix}`
-        if (matchKey(importPath, componentKeys)) return null
+        if (matchKey(importPath, componentKeys)) {
+          // The inlined child keeps its OWN declared name (`stripModuleExports`
+          // only removes `export `, it doesn't rename), but the parent's
+          // JSX still references whatever LOCAL name it imported the child
+          // under (#2777 — Hono correctly keeps the caller's local binding
+          // rather than the declared name). Dropping the import line
+          // unconditionally is only safe for a non-aliased specifier
+          // (`import { Label } from './label'`, where local === declared);
+          // an aliased one (`import { Label as AliasedLabel } from
+          // './label'`) needs a `const AliasedLabel = Label` left behind, or
+          // the parent's JSX throws `ReferenceError: AliasedLabel is not
+          // defined` — this harness inlines the child by literal source
+          // splice, not through a real module boundary, so there's no
+          // `import … as …` left to do the binding for us.
+          return aliasDeclarationsFor(line)
+        }
         return line
       })
       .filter((line): line is string => line !== null)