@barefootjs/mojolicious 0.18.7 → 0.19.0

package/dist/adapter/index.js

@@ -1001,7 +1001,7 @@ function collectBooleanTypedProps(ir) {
   return new Set(ir.metadata.propsParams.filter((prop) => prop.type?.primitive === "boolean" || prop.type?.raw === "boolean").map((prop) => prop.name));
 }
 function collectNullableOptionalProps(ir) {
-  return new Set(ir.metadata.propsParams.filter((p) => p.defaultValue === undefined && !p.isRest && p.type?.kind !== "primitive").map((p) => p.name));
+  return new Set(ir.metadata.propsParams.filter((p) => p.defaultValue === undefined && !p.isRest && (p.type?.kind !== "primitive" || p.optional)).map((p) => p.name));
 }
 function collectStringValueNames(ir) {
   const names = new Set;

package/dist/adapter/props/prop-classes.d.ts

@@ -30,11 +30,10 @@ export declare function collectBooleanTypedProps(ir: ComponentIR): Set<string>;
  * resolved `interface{}` field type; here it is the absence of a default (the
  * analyzer reports `rows` — a `TextareaHTMLAttributes` member destructured
  * without a default — as no-default, `type.kind: 'unknown'`).
- * Excludes concrete-primitive types (`string`/`number`/`boolean`) to match
- * the Go adapter's scope, which guards only `interface{}` (nillable) fields
- * and leaves concrete fields unconditional. So a required, no-default
- * `string` prop still emits `attr=""` like Hono, and only nillable
- * (`unknown`/object/array) no-default props guard.
+ * A REQUIRED concrete-primitive prop (`string`/`number`/`boolean`) is
+ * excluded — the caller always supplies it, so it emits `attr=""` like Hono
+ * — but an OPTIONAL primitive is presence-uncertain and stays guarded
+ * (#2259).
  */
 export declare function collectNullableOptionalProps(ir: ComponentIR): Set<string>;
 /**

package/dist/adapter/props/prop-classes.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"prop-classes.d.ts","sourceRoot":"","sources":["../../../src/adapter/props/prop-classes.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAyB,KAAK,WAAW,EAAE,MAAM,iBAAiB,CAAA;AAGzE;;;;GAIG;AACH,wBAAgB,wBAAwB,CAAC,EAAE,EAAE,WAAW,GAAG,GAAG,CAAC,MAAM,CAAC,CAMrE;AAED;;;;;GAKG;AACH,wBAAgB,wBAAwB,CAAC,EAAE,EAAE,WAAW,GAAG,GAAG,CAAC,MAAM,CAAC,CAMrE;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,4BAA4B,CAAC,EAAE,EAAE,WAAW,GAAG,GAAG,CAAC,MAAM,CAAC,CAWzE;AAED;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AACH,wBAAgB,uBAAuB,CAAC,EAAE,EAAE,WAAW,GAAG,GAAG,CAAC,MAAM,CAAC,CAepE"}
+{"version":3,"file":"prop-classes.d.ts","sourceRoot":"","sources":["../../../src/adapter/props/prop-classes.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAyB,KAAK,WAAW,EAAE,MAAM,iBAAiB,CAAA;AAGzE;;;;GAIG;AACH,wBAAgB,wBAAwB,CAAC,EAAE,EAAE,WAAW,GAAG,GAAG,CAAC,MAAM,CAAC,CAMrE;AAED;;;;;GAKG;AACH,wBAAgB,wBAAwB,CAAC,EAAE,EAAE,WAAW,GAAG,GAAG,CAAC,MAAM,CAAC,CAMrE;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,4BAA4B,CAAC,EAAE,EAAE,WAAW,GAAG,GAAG,CAAC,MAAM,CAAC,CAWzE;AAED;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AACH,wBAAgB,uBAAuB,CAAC,EAAE,EAAE,WAAW,GAAG,GAAG,CAAC,MAAM,CAAC,CAepE"}

package/dist/build.js

@@ -1001,7 +1001,7 @@ function collectBooleanTypedProps(ir) {
   return new Set(ir.metadata.propsParams.filter((prop) => prop.type?.primitive === "boolean" || prop.type?.raw === "boolean").map((prop) => prop.name));
 }
 function collectNullableOptionalProps(ir) {
-  return new Set(ir.metadata.propsParams.filter((p) => p.defaultValue === undefined && !p.isRest && p.type?.kind !== "primitive").map((p) => p.name));
+  return new Set(ir.metadata.propsParams.filter((p) => p.defaultValue === undefined && !p.isRest && (p.type?.kind !== "primitive" || p.optional)).map((p) => p.name));
 }
 function collectStringValueNames(ir) {
   const names = new Set;

package/dist/index.js

@@ -1001,7 +1001,7 @@ function collectBooleanTypedProps(ir) {
   return new Set(ir.metadata.propsParams.filter((prop) => prop.type?.primitive === "boolean" || prop.type?.raw === "boolean").map((prop) => prop.name));
 }
 function collectNullableOptionalProps(ir) {
-  return new Set(ir.metadata.propsParams.filter((p) => p.defaultValue === undefined && !p.isRest && p.type?.kind !== "primitive").map((p) => p.name));
+  return new Set(ir.metadata.propsParams.filter((p) => p.defaultValue === undefined && !p.isRest && (p.type?.kind !== "primitive" || p.optional)).map((p) => p.name));
 }
 function collectStringValueNames(ir) {
   const names = new Set;

package/lib/BarefootJS/Backend/Mojo.pm

@@ -1,5 +1,5 @@
 package BarefootJS::Backend::Mojo;
-our $VERSION = "0.18.5";
+our $VERSION = "0.18.7";
 use Mojo::Base -base, -signatures;
 
 use Mojo::ByteStream qw(b);

package/lib/Mojolicious/Plugin/BarefootJS/DevReload.pm

@@ -1,5 +1,5 @@
 package Mojolicious::Plugin::BarefootJS::DevReload;
-our $VERSION = "0.18.5";
+our $VERSION = "0.18.7";
 use Mojo::Base 'Mojolicious::Plugin', -signatures;
 
 =head1 NAME

package/lib/Mojolicious/Plugin/BarefootJS.pm

@@ -1,5 +1,5 @@
 package Mojolicious::Plugin::BarefootJS;
-our $VERSION = "0.18.5";
+our $VERSION = "0.18.7";
 use Mojo::Base 'Mojolicious::Plugin', -signatures;
 
 use Mojo::File qw(path);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@barefootjs/mojolicious",
-  "version": "0.18.7",
+  "version": "0.19.0",
   "description": "Mojolicious EP template adapter for BarefootJS - generates .html.ep files from IR",
   "type": "module",
   "main": "./dist/index.js",
@@ -52,7 +52,7 @@
     "directory": "packages/adapter-mojolicious"
   },
   "dependencies": {
-    "@barefootjs/shared": "0.18.7"
+    "@barefootjs/shared": "0.19.0"
   },
   "peerDependencies": {
     "@barefootjs/jsx": ">=0.2.0",
@@ -60,6 +60,6 @@
   },
   "devDependencies": {
     "@barefootjs/adapter-tests": "0.1.0",
-    "@barefootjs/jsx": "0.18.7"
+    "@barefootjs/jsx": "0.19.0"
   }
 }

package/src/__tests__/mojo-adapter.test.ts

@@ -59,6 +59,28 @@ runAdapterConformanceTests({
     // #1467 Phase 2e: same `/* @client */` keyed-map elision (data-table).
     'data-table',
   ]),
+  skipDataPoints: new Set<string>([
+    // #2255 — Perl length() counts codepoints under utf8; JS counts
+    // UTF-16 code units, so a surrogate-pair character is 2 in JS, 1 here.
+    'string-length-text:astral',
+    // #2260 — controlled boolean props: the SSR seed evaluates only the
+    // static fallback of `props.X ?? internal()` chains.
+    'toggle:gen:pressed:true',
+    'switch:gen:checked:true',
+    'checkbox:gen:checked:true',
+    // #2261 — invalid dynamic CSS value kept (escaped) where the oracle
+    // drops the property.
+    'style-object-dynamic:gen:color:markup',
+    // #2262 — dynamic `.flat` depth 0/negative violates the documented
+    // shallow-copy contract.
+    'array-flat-dynamic-depth:gen:depth:zero',
+    'array-flat-dynamic-depth:gen:depth:negative',
+    // #2266 — asChild=true with plain-text children breaks the Slot
+    // rendering path (same class as the Go Slot define; observed on the
+    // real-Perl CI run).
+    'button:gen:asChild:true',
+    'kbd:gen:asChild:true',
+  ]),
   onRenderError: (err, id) => {
     if (err instanceof PerlNotAvailableError) {
       console.log(`Skipping [${id}]: ${err.message}`)

package/src/adapter/props/prop-classes.ts

@@ -47,11 +47,10 @@ export function collectBooleanTypedProps(ir: ComponentIR): Set<string> {
  * resolved `interface{}` field type; here it is the absence of a default (the
  * analyzer reports `rows` — a `TextareaHTMLAttributes` member destructured
  * without a default — as no-default, `type.kind: 'unknown'`).
- * Excludes concrete-primitive types (`string`/`number`/`boolean`) to match
- * the Go adapter's scope, which guards only `interface{}` (nillable) fields
- * and leaves concrete fields unconditional. So a required, no-default
- * `string` prop still emits `attr=""` like Hono, and only nillable
- * (`unknown`/object/array) no-default props guard.
+ * A REQUIRED concrete-primitive prop (`string`/`number`/`boolean`) is
+ * excluded — the caller always supplies it, so it emits `attr=""` like Hono
+ * — but an OPTIONAL primitive is presence-uncertain and stays guarded
+ * (#2259).
  */
 export function collectNullableOptionalProps(ir: ComponentIR): Set<string> {
   return new Set(
@@ -60,7 +59,7 @@ export function collectNullableOptionalProps(ir: ComponentIR): Set<string> {
         p =>
           p.defaultValue === undefined &&
           !p.isRest &&
-          p.type?.kind !== 'primitive',
+          (p.type?.kind !== 'primitive' || p.optional),
       )
       .map(p => p.name),
   )

package/src/test-render.ts

@@ -583,7 +583,15 @@ function buildPerlProps(
   if (props) {
     for (const [key, value] of Object.entries(props)) {
       if (routedKeys.has(key)) continue
-      if (typeof value === 'string') {
+      if (value === null) {
+        // An explicit-null prop must still DECLARE the template var —
+        // `typeof null === 'object'` fails every branch below, so the key
+        // emitted nothing and `Mojo::Template`'s `vars => 1` never ran
+        // `my $user`, tripping Perl's strict-mode "Global symbol requires
+        // explicit package name" before the `//` fallback could apply
+        // (data-point conformance, optional-chaining-prop:null-user).
+        entries.push(`${key} => undef`)
+      } else if (typeof value === 'string') {
         entries.push(`${key} => '${value.replace(/'/g, "\\'")}'`)
       } else if (typeof value === 'number') {
         entries.push(`${key} => ${value}`)