@barefootjs/go-template 0.9.1 → 0.9.2

package/dist/test-render.d.ts

@@ -17,6 +17,13 @@ export interface RenderOptions {
     props?: Record<string, unknown>;
     /** Additional component files (filename → source) */
     components?: Record<string, string>;
+    /**
+     * Explicit component to render when `source` declares multiple
+     * exports (e.g. `ReactiveProps.tsx` → `PropsReactivityComparison`).
+     * Mirrors the Hono reference's `componentName`; omitted for
+     * single-export fixtures, which fall back to the default/first export.
+     */
+    componentName?: string;
 }
 export declare function renderGoTemplateComponent(options: RenderOptions): Promise<string>;
 //# sourceMappingURL=test-render.d.ts.map

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;AAOnE,qBAAa,mBAAoB,SAAQ,KAAK;IAC5C,YAAY,OAAO,EAAE,MAAM,EAG1B;CACF;AAgCD,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;CACpC;AAED,wBAAsB,yBAAyB,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,MAAM,CAAC,CAsRvF"}
+{"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;AAOnE,qBAAa,mBAAoB,SAAQ,KAAK;IAC5C,YAAY,OAAO,EAAE,MAAM,EAG1B;CACF;AAgCD,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;;;;;OAKG;IACH,aAAa,CAAC,EAAE,MAAM,CAAA;CACvB;AAED,wBAAsB,yBAAyB,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,MAAM,CAAC,CA4RvF"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@barefootjs/go-template",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "Go html/template adapter for BarefootJS - generates Go template files from IR",
   "type": "module",
   "main": "./dist/index.js",
@@ -54,6 +54,6 @@
   },
   "devDependencies": {
     "@barefootjs/adapter-tests": "0.1.0",
-    "@barefootjs/jsx": "0.9.1"
+    "@barefootjs/jsx": "0.9.2"
   }
 }

package/src/__tests__/go-template-adapter.test.ts

@@ -37,34 +37,38 @@ runAdapterConformanceTests({
   skipJsx: [
     // #1244 stress catalog (#1326): `children={<span/>}` — the IR
     // hoists the span with `needsScope: true` so the Hono reference
-    // emits `bf-s` on the inner `<span>`. The Go adapter renders the
-    // span up front as a compile-time HTML fragment containing the
-    // `{{bfScopeAttr .}}` action, then passes it via `template.HTML`
-    // through the parent's `{{.Children}}` interpolation — but
-    // `template.HTML` is marked-as-safe-output, not recursively
-    // parsed, so the action survives as literal text in the rendered
-    // HTML. Fixing this requires either (a) re-emitting the inner
-    // span as its own named template definition the outer template
-    // can pass its struct to, or (b) embedding the resolved scope ID
-    // at compile time. Neither lands in this PR; the Mojo sibling
-    // case is handled by routing the hoisted JSX through the same
-    // `begin %>…<% end` capture as nested children (see #1326 fix).
+    // emits `bf-s` on the inner `<span>`. The Go adapter has no path to
+    // render a hoisted-JSX child handed in through the `children`
+    // attribute, so the child is DROPPED: the parent renders
+    // `<div bf-s="test_s0"></div>` with an empty body (verified — no
+    // compile error, no literal `{{bfScopeAttr}}` text either; that
+    // older literal-survives-in-output failure mode no longer occurs).
+    // Closing the gap needs the inner span re-emitted as its own named
+    // template definition the outer template can pass its struct to, or
+    // the resolved scope id embedded at compile time. Neither lands
+    // here; the Mojo / Xslate siblings already pass by routing the
+    // hoisted JSX through the same nested-children capture.
     'children-jsx-expression',
-    // #1335: fragment-wrapped form of the same shape. Now that the IR
-    // unwraps `<><span/></>` into the bare-element form, the Go adapter
-    // hits the identical `template.HTML` interpolation gap as
-    // `children-jsx-expression` above.
+    // #1335: fragment-wrapped form of the same shape. Once the IR
+    // unwraps `<><span/></>` into the bare-element form the Go adapter
+    // hits the identical drop as `children-jsx-expression` above.
     'fragment-wrapped-children-jsx-expression',
-    // Shared-component multi-component fixtures (#1466). Boolean
-    // attribute divergence is now collapsed by `normalizeHTML`, so
-    // single-root variants (`conditional-return-*`, `form`, `portal`,
-    // `reactive-props`) participate again. These two still diverge
-    // because the harness's child renderer pins child `bf-s` to a
-    // `test_<sN>` literal rather than `<ChildName>_<id>_<sN>`. Same
-    // class of test-harness scope-id plumbing the `componentName`
-    // option fixed on the Hono side. Separate follow-up.
+    // `toggle-shared`: the parent maps a `ToggleItemProps[]` prop into
+    // sibling `ToggleItem` children inside a keyed `.map`. The Go
+    // child-slice init types the loop input as `[]any` rather than
+    // `[]ToggleItemInput`, so the generated `NewToggleProps` struct
+    // literal fails to compile (`cannot use []any{…} as
+    // []ToggleItemInput value in struct literal`). Separate follow-up —
+    // needs the loop-child init to emit the typed element slice, plus
+    // per-item `defaultOn` seeding and the `data-key` attribute shape.
+    //
+    // `props-reactivity-comparison` graduated: the Go test-render now
+    // honours the fixture's `componentName` (`PropsReactivityComparison`,
+    // the second export of `ReactiveProps.tsx`) instead of always
+    // rendering the first export, and the generated child constructors
+    // compute the `displayValue = props.value * 10` memo from the
+    // passed prop — so SSR matches Hono byte-for-byte.
     'toggle-shared',
-    'props-reactivity-comparison',
   ],
   // Per-fixture build-time contracts for shapes the Go template
   // adapter intentionally refuses to lower. Lives here (not on the

package/src/test-render.ts

@@ -59,10 +59,17 @@ export interface RenderOptions {
   props?: Record<string, unknown>
   /** Additional component files (filename → source) */
   components?: Record<string, string>
+  /**
+   * Explicit component to render when `source` declares multiple
+   * exports (e.g. `ReactiveProps.tsx` → `PropsReactivityComparison`).
+   * Mirrors the Hono reference's `componentName`; omitted for
+   * single-export fixtures, which fall back to the default/first export.
+   */
+  componentName?: string
 }
 
 export async function renderGoTemplateComponent(options: RenderOptions): Promise<string> {
-  const { source, adapter, props, components } = options
+  const { source, adapter, props, components, componentName: requestedName } = options
 
   if (!adapter.generateTypes) {
     throw new Error('Go Template adapter must implement generateTypes()')
@@ -139,7 +146,13 @@ export async function renderGoTemplateComponent(options: RenderOptions): Promise
   const irFiles = result.files.filter(f => f.type === 'ir')
   if (irFiles.length === 0) throw new Error('No IR output (set outputIR: true)')
   const irs = irFiles.map(f => JSON.parse(f.content) as ComponentIR)
+  // Explicit `componentName` wins (multi-export sources pin which export
+  // is the render target); otherwise default-export, then first inline-
+  // exported, then first IR. Mirrors the Hono reference's selection so
+  // multi-component fixtures (e.g. `ReactiveProps.tsx`) render the same
+  // export across adapters.
   const ir =
+    (requestedName ? irs.find(i => i.metadata.componentName === requestedName) : undefined) ??
     irs.find(i => i.metadata.hasDefaultExport) ??
     irs.find(i => i.metadata.isExported) ??
     irs[0]