@barefootjs/go-template 0.31.4 → 0.31.6

package/src/conformance-pins.ts

@@ -15,6 +15,9 @@ export const conformancePins: ConformancePins = {
   // JS-runtime target runs it, a DSL adapter surfaces BF021 + `/* @client */`.
   // See spec/callback-fidelity.md.
   'filter-typeof-predicate': [{ code: 'BF021', severity: 'error' }],
+  // Array-builder `.map()` body (imperative `push`-into-array preamble):
+  // BF021, with a verified `/* @client */` escape — `map-array-builder-
+  // body-client` (#2613). See that fixture's docstring.
   'map-array-builder-body': [{ code: 'BF021', severity: 'error' }],
   'map-array-builder-escaping': [{ code: 'BF021', severity: 'error' }],
   // `.fill(value)` mutates the receiver in place — no template lowering
@@ -80,14 +83,29 @@ export const conformancePins: ConformancePins = {
   // array bound to such a const. See the `renderLoop` comment at the check
   // site; Jinja / ERB apply the same narrow check for the same reason.
   'static-array-from-props': [
-    { code: 'BF101', severity: 'error', issue: 'https://github.com/piconic-ai/barefootjs/issues/2321' },
+    {
+      code: 'BF101',
+      severity: 'error',
+      issue: 'https://github.com/piconic-ai/barefootjs/issues/2321',
+    },
   ],
   // Same computed-const array as above — the destructure param itself no
   // longer contributes a diagnostic, and BF103 (sibling-imported child
   // component in the loop body) no longer fires either now that the
   // conformance harness passes `siblingTemplatesRegistered: true` (#2205).
+  // #2627 graduated: the `/* @client */` twin
+  // (`static-array-from-props-with-component-client`) now renders clean on
+  // real Go (its clientOnly nested `<Tag>` loop is excluded from the
+  // Input/Props/NewProps codegen instead of colliding with the `tags` prop's
+  // own field — see `GoTemplateAdapter.isOrphanedClientOnlyNested`), so this
+  // adapter has a verified escape for this fixture like every other one —
+  // no more `unescapable` line.
   'static-array-from-props-with-component': [
-    { code: 'BF101', severity: 'error', issue: 'https://github.com/piconic-ai/barefootjs/issues/2321' },
+    {
+      code: 'BF101',
+      severity: 'error',
+      issue: 'https://github.com/piconic-ai/barefootjs/issues/2321',
+    },
   ],
   // (`style-3-signals` graduated alongside `style-object-dynamic` — see note
   // above; the `style={{ … }}` object now lowers to a CSS string.)
@@ -106,9 +124,7 @@ export const conformancePins: ConformancePins = {
   'filter-nested-callback-predicate': [
     { code: 'BF101', severity: 'error', issue: 'https://github.com/piconic-ai/barefootjs/issues/2320' },
   ],
-  'filter-nested-find-predicate': [
-    { code: 'BF101', severity: 'error', issue: 'https://github.com/piconic-ai/barefootjs/issues/2320' },
-  ],
+  'filter-nested-find-predicate': [{ code: 'BF101', severity: 'error', issue: 'https://github.com/piconic-ai/barefootjs/issues/2320' }],
   // #1310 / #2087: rest destructure in .map() callback. `isLowerableLoopDestructure`
   // now admits every shape this fixture family exercises — each fixed/rest
   // binding resolves via `buildSegmentAccessor`/`buildDestructureBindingMap`

package/src/render-divergences.ts

@@ -13,9 +13,15 @@
  * the docs compatibility-matrix page. Graduating an entry means fixing
  * the adapter (or the shared compiler layer) and deleting the line.
  *
- * Empty — the file's last live entry (`aliased-destructured-prop`, #2525)
- * graduated: the Input struct field is now keyed by `sourceName ?? name`
- * (caller-facing), so the caller-side composite literal `go run`s clean.
+ * Empty — the file's last live entry (`static-array-from-props-with-
+ * component-client`, #2627) graduated: a clientOnly child-component loop
+ * whose array is neither a signal/memo nor a direct prop reference (e.g.
+ * `Object.entries(props.tags).filter(...)` feeding a `<Tag>` loop) is now
+ * excluded entirely from the Input/Props/NewProps codegen instead of
+ * colliding with the driving prop's own field — see
+ * `GoTemplateAdapter.isOrphanedClientOnlyNested`. The caller-side composite
+ * literal for the `tags` prop now type-checks against its own `interface{}`
+ * field and `go run`s clean.
  * Keep the file (and this header) when the set is empty — the next
  * divergence lands here, not in a re-created file.
  */
@@ -23,6 +29,26 @@
 import type { RenderDivergences } from '@barefootjs/jsx'
 
 export const renderDivergences: RenderDivergences = {
+  // #2630: a prop-backed loop whose BODY is a child component renders the
+  // loop host empty — compiles clean, `go run`s clean, emits no children.
+  // Its sibling `static-array-from-props-precomputed` (same fixture,
+  // inline element body instead of a `<Tag>` child) renders correctly on
+  // every adapter, so this is specific to child-component bodies over a
+  // prop-backed array — plausibly the same missing server-side population
+  // path the `todo-app-ssr` note below describes, where
+  // `buildDynamicChildLoopSeeding` covers only the SIGNAL-backed dynamic
+  // case and a prop-backed static child loop has no analogue.
+  //
+  // This one costs a user something concrete: the BF101 diagnostic for
+  // #2321 lists pass-as-prop FIRST because it is the escape that keeps
+  // full server output, so a go-template user following that advice for a
+  // child-component loop gets an empty container at SSR with nothing
+  // reporting it. The fixture asserts the CORRECT (reference) output, so
+  // deleting this entry once the gap is fixed turns it into the
+  // regression test.
+  'static-array-from-props-with-component-precomputed':
+    'prop-backed child-component loop renders the loop host empty at SSR (https://github.com/piconic-ai/barefootjs/issues/2630)',
+
   // `todo-app-ssr` no longer diverges (#2209). Two parts: (1) `.Todos`
   // (the loop's DATUM slice) is already seeded straight from the caller's
   // Input — the constructor derives it from `initialTodos`, and `[]Todo`