@barefootjs/erb 0.17.0 → 0.18.1

package/README.md

@@ -0,0 +1,32 @@
+# barefoot_js
+
+Ruby runtime for [BarefootJS](https://barefootjs.dev/) marked templates, targeting ERB.
+
+[BarefootJS](https://github.com/piconic-ai/barefootjs) is a fine-grained reactive TSX compiler: you write components in TSX, and the compiler emits templates for your backend's template engine plus the client-side JS that hydrates them. This gem is the server half for Ruby — it renders the `.erb` templates produced by the `@barefootjs/erb` adapter.
+
+## Installation
+
+```sh
+gem install barefoot_js
+```
+
+## Usage
+
+Every compiled `.erb` template receives a `BarefootJS::Context` as the `bf` local. The context is engine-agnostic; everything that depends on how a template is rendered is delegated to a pluggable backend (`BarefootJS::Backend::Erb` is the ERB reference implementation):
+
+```ruby
+require 'barefoot_js'
+
+bf = BarefootJS::Context.new(backend)
+```
+
+Values are JSON-shaped Ruby data with symbol hash keys throughout (props, env hashes, array-of-hash records).
+
+## Documentation
+
+- [barefootjs.dev](https://barefootjs.dev/) — core documentation
+- [GitHub: piconic-ai/barefootjs](https://github.com/piconic-ai/barefootjs) — monorepo (this gem lives at `packages/adapter-erb`)
+
+## License
+
+MIT

package/dist/adapter/index.js

@@ -18568,7 +18568,7 @@ https://github.com/browserify/crypto-browserify`);
 
 // ../../node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib/typescript.js
 var require_typescript = __commonJS((exports, module) => {
-  var __dirname = "/Users/kfly8/src/github.com/piconic-ai/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib", __filename = "/Users/kfly8/src/github.com/piconic-ai/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib/typescript.js";
+  var __dirname = "/home/runner/work/barefootjs/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib", __filename = "/home/runner/work/barefootjs/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib/typescript.js";
   /*! *****************************************************************************
   Copyright (c) Microsoft Corporation. All rights reserved.
   Licensed under the Apache License, Version 2.0 (the "License"); you may not use

package/dist/build.js

@@ -18568,7 +18568,7 @@ https://github.com/browserify/crypto-browserify`);
 
 // ../../node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib/typescript.js
 var require_typescript = __commonJS((exports, module) => {
-  var __dirname = "/Users/kfly8/src/github.com/piconic-ai/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib", __filename = "/Users/kfly8/src/github.com/piconic-ai/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib/typescript.js";
+  var __dirname = "/home/runner/work/barefootjs/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib", __filename = "/home/runner/work/barefootjs/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib/typescript.js";
   /*! *****************************************************************************
   Copyright (c) Microsoft Corporation. All rights reserved.
   Licensed under the Apache License, Version 2.0 (the "License"); you may not use

package/dist/index.js

@@ -18568,7 +18568,7 @@ https://github.com/browserify/crypto-browserify`);
 
 // ../../node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib/typescript.js
 var require_typescript = __commonJS((exports, module) => {
-  var __dirname = "/Users/kfly8/src/github.com/piconic-ai/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib", __filename = "/Users/kfly8/src/github.com/piconic-ai/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib/typescript.js";
+  var __dirname = "/home/runner/work/barefootjs/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib", __filename = "/home/runner/work/barefootjs/barefootjs/node_modules/.bun/typescript@5.9.3/node_modules/typescript/lib/typescript.js";
   /*! *****************************************************************************
   Copyright (c) Microsoft Corporation. All rights reserved.
   Licensed under the Apache License, Version 2.0 (the "License"); you may not use

package/lib/barefoot_js.rb

@@ -324,7 +324,15 @@ module BarefootJS
           extra[name] = props.key?(name) ? props[name] : d[:value]
           next
         end
-        prop_name = d[:propName]
+        # `propName` rides in from the JSON manifest as a String -- JSON has
+        # no symbol type, and the manifest's `symbolize_names: true` parse
+        # only symbolizes hash KEYS, never string values. Runtime prop
+        # hashes, meanwhile, are symbol-keyed because compiled ERB templates
+        # pass `{ children: ... }` literals, so `props.key?(prop_name)` with
+        # the String would always miss, silently falling back to the static
+        # default for every manifest-registered child (e.g. `children`
+        # rendering empty) (#2157).
+        prop_name = d[:propName]&.to_sym
         extra[name] =
           if !prop_name.nil? && props.key?(prop_name) && !props[prop_name].nil?
             props[prop_name]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@barefootjs/erb",
-  "version": "0.17.0",
+  "version": "0.18.1",
   "description": "ERB (Embedded Ruby) adapter for BarefootJS — compiles IR to .erb templates and ships the Ruby rendering backend; runs under any Rack app (Sinatra, Rails)",
   "type": "module",
   "main": "./dist/index.js",
@@ -54,14 +54,14 @@
     "directory": "packages/adapter-erb"
   },
   "dependencies": {
-    "@barefootjs/shared": "0.17.1"
+    "@barefootjs/shared": "0.18.1"
   },
   "peerDependencies": {
     "@barefootjs/jsx": ">=0.2.0"
   },
   "devDependencies": {
     "@barefootjs/adapter-tests": "0.1.0",
-    "@barefootjs/jsx": "0.17.1",
+    "@barefootjs/jsx": "0.18.1",
     "typescript": "^5.0.0"
   }
 }

package/src/test-render.ts

@@ -22,7 +22,18 @@
  * Child components are wired through the production
  * `BarefootJS::Context#register_child_renderer` so the child's `bf-s` scope
  * id derives from `<parentScope>_<slotId>` exactly as a real `bf build`
- * page would.
+ * page would. As of #2158, the child-vars seeding step also runs the exact
+ * production sequence — `BarefootJS::Context.derive_vars_from_defaults`,
+ * the same call `register_components_from_manifest` makes
+ * (`lib/barefoot_js.rb` ~lines 283-292) — instead of a harness-local
+ * `defaults.merge(child_props)` shortcut. That shortcut is why the #2157
+ * bug (a manifest `propName` riding in as a JSON String, looked up against
+ * symbol-keyed runtime prop Hashes, so `children` silently fell back to
+ * the static default and rendered empty) was invisible to this whole
+ * conformance corpus: the harness never exercised the code path the bug
+ * lived in. Now it does, so the same class of regression is caught by
+ * every fixture that renders a child component, not just a dedicated
+ * regression test.
  */
 
 import { compileJSX, extractSsrDefaults, importsSearchParams } from '@barefootjs/jsx'
@@ -255,12 +266,21 @@ export async function renderErbComponent(options: RenderOptions): Promise<string
     const rootScopeIdRaw = typeof props?.__instanceId === 'string' ? props.__instanceId : 'test'
     const rootScopeId = rubyStringLiteral(rootScopeIdRaw)
 
-    // Static ssrDefaults per child, simplified to bare values (the
-    // caller's props always win — see `buildChildRenderersRuby`) and
-    // written as one JSON file keyed by snake_case template name.
-    const childDefaults: Record<string, Record<string, unknown>> = {}
+    // Static ssrDefaults per child, written VERBATIM — `propName` /
+    // `isRestProps` / `value` intact, exactly the shape the production
+    // build manifest embeds — and written as one JSON file keyed by
+    // snake_case template name. `buildChildRenderersRuby` below feeds this
+    // straight into `BarefootJS::Context.derive_vars_from_defaults`, the
+    // same call `register_components_from_manifest` makes in the published
+    // gem (#2158). A prior version of this harness simplified each entry
+    // to its bare `value` and did `defaults.merge(child_props)` directly —
+    // that sidestepped `derive_vars_from_defaults`'s `propName` lookup
+    // entirely, which is exactly why the #2157 String-vs-symbol `propName`
+    // bug (children rendering empty) was invisible to this whole
+    // conformance corpus.
+    const childDefaults: Record<string, Record<string, SsrDefault>> = {}
     for (const [childName, { ir: childIR }] of childTemplates) {
-      childDefaults[toSnakeCase(childName)] = simplifySsrDefaults(extractSsrDefaults(childIR.metadata) ?? {})
+      childDefaults[toSnakeCase(childName)] = extractSsrDefaults(childIR.metadata) ?? {}
     }
     if (childTemplates.size > 0) {
       await Bun.write(resolve(tempDir, 'child_defaults.json'), JSON.stringify(childDefaults))
@@ -343,10 +363,19 @@ function collectImportedComponentNames(ir: ComponentIR): string[] {
  * derives the child scope id from `<parentScope>_<slotId>` (the parent's
  * `bf.render_child('<name>', { …, _bf_slot: '<slotId>' })` passes
  * `_bf_slot`), seeds signal / memo / prop defaults from the child IR's
- * `ssrDefaults` (loaded once from `child_defaults.json`, caller props win
- * via `Hash#merge`), shares the parent's script list, and renders the
- * child `.erb` through the same backend. Loop children (no `_bf_slot`)
- * fall back to `<ComponentName>_<rand>` like the Perl harnesses.
+ * `ssrDefaults` (loaded once from `child_defaults.json`) THROUGH THE SAME
+ * `BarefootJS::Context.derive_vars_from_defaults` call
+ * `register_components_from_manifest` makes (lib/barefoot_js.rb ~lines
+ * 283-292) — not a harness-local re-derivation — so the propName /
+ * isRestProps resolution the published gem actually runs is exactly what
+ * this harness exercises (#2158; this is the fix for the #2157 class of
+ * bug, where a harness-local `defaults.merge(child_props)` shortcut never
+ * touched that code path and so never caught the String-vs-symbol
+ * `propName` mismatch). Caller props still win over static defaults —
+ * that's `derive_vars_from_defaults`'s own contract — shares the parent's
+ * script list, and renders the child `.erb` through the same backend.
+ * Loop children (no `_bf_slot`) fall back to `<ComponentName>_<rand>`
+ * like the Perl harnesses.
  */
 function buildChildRenderersRuby(
   childTemplates: Map<string, { template: string; ir: ComponentIR }>,
@@ -411,8 +440,16 @@ function buildChildRenderersRuby(
     lines.push(`  child_bf._child_renderers(bf._child_renderers)`)
     lines.push(`  child_bf._scripts(bf._scripts)`)
     lines.push(`  child_bf._script_seen(bf._script_seen)`)
-    // Seed template vars: static ssrDefaults first, caller's props win.
-    lines.push(`  vars = defaults_${snakeName}.merge(child_props)`)
+    // Seed template vars through the production sequence — the exact
+    // call `register_components_from_manifest` makes in the published gem
+    // (lib/barefoot_js.rb ~lines 283-292) — instead of a harness-local
+    // `defaults.merge(child_props)` shortcut. `derive_vars_from_defaults`
+    // resolves each entry's `propName` (symbolized) against the live
+    // `child_props`, falling back to the static `value`; `isRestProps`
+    // entries pass the rest bag through. Caller props still win overall
+    // via the final `merge`.
+    lines.push(`  extra = BarefootJS::Context.derive_vars_from_defaults(defaults_${snakeName}, child_props)`)
+    lines.push(`  vars = child_props.merge(extra)`)
     lines.push(`  rendered = backend.render_named(${rubyStringLiteral(snakeName)}, child_bf, vars)`)
     lines.push(`  rendered.chomp`)
     lines.push(`end)`)
@@ -430,23 +467,6 @@ function rubySymbol(name: string): string {
   return /^[A-Za-z_][A-Za-z0-9_]*[?!]?$/.test(name) ? `:${name}` : `:"${name.replace(/"/g, '\\"')}"`
 }
 
-/**
- * Simplify an `extractSsrDefaults` map to bare values for the child
- * defaults JSON file. Each entry is either a bare value or
- * `{ value, propName?, isRestProps? }`; the child renderer always merges
- * the caller's live `child_props` OVER these defaults (`Hash#merge`), so
- * only the static fallback `value` is needed here — the `propName`-aware
- * resolution the production manifest path does is redundant with that
- * merge.
- */
-function simplifySsrDefaults(defaults: Record<string, SsrDefault>): Record<string, unknown> {
-  const out: Record<string, unknown> = {}
-  for (const [name, d] of Object.entries(defaults)) {
-    out[name] = d.value
-  }
-  return out
-}
-
 /**
  * Convert PascalCase to snake_case for template naming (matches the
  * adapter's `toTemplateName`).