@barefootjs/mojolicious 0.19.1 → 0.20.0

package/dist/adapter/expr/array-method.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"array-method.d.ts","sourceRoot":"","sources":["../../../src/adapter/expr/array-method.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAMH,OAAO,KAAK,EACV,UAAU,EACV,WAAW,EACX,cAAc,EACd,SAAS,EACV,MAAM,iBAAiB,CAAA;AAExB;;;;;;;;GAQG;AACH,wBAAgB,iBAAiB,CAC/B,MAAM,EAAE,WAAW,EACnB,MAAM,EAAE,UAAU,EAClB,IAAI,EAAE,UAAU,EAAE,EAClB,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,CAgOR;AA6BD;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAC5B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,UAAU,EAChB,MAAM,EAAE,MAAM,EAAE,EAChB,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,GAAG,IAAI,CAOf;AAED;;;;;;;GAOG;AACH,wBAAgB,gBAAgB,CAC9B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,UAAU,EAChB,MAAM,EAAE,MAAM,EAAE,EAChB,IAAI,EAAE,UAAU,EAChB,SAAS,EAAE,MAAM,GAAG,OAAO,EAC3B,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,GAAG,IAAI,CAmBf;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,mBAAmB,CACjC,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,EACZ,SAAS,EAAE,UAAU,EACrB,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,EAC/B,OAAO,CAAC,EAAE,OAAO,GAChB,MAAM,GAAG,IAAI,CAMf;AAED;;;;;;;GAOG;AACH,wBAAgB,iBAAiB,CAC/B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,UAAU,EAChB,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,GAAG,IAAI,CAKf;AAED;;;;;;;;GAQG;AACH,wBAAgB,aAAa,CAC3B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,UAAU,EAChB,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,GAAG,IAAI,CAKf;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,CAAC,EAAE,cAAc,GAAG,MAAM,CAaxE;AAKD,wBAAgB,gBAAgB,CAC9B,IAAI,EAAE,MAAM,EACZ,KAAK,EAAE,SAAS,GAAG;IAAE,IAAI,EAAE,UAAU,CAAA;CAAE,EACvC,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,CAoBR"}
+{"version":3,"file":"array-method.d.ts","sourceRoot":"","sources":["../../../src/adapter/expr/array-method.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAMH,OAAO,KAAK,EACV,UAAU,EACV,WAAW,EACX,cAAc,EACd,SAAS,EACV,MAAM,iBAAiB,CAAA;AAExB;;;;;;;;GAQG;AACH,wBAAgB,iBAAiB,CAC/B,MAAM,EAAE,WAAW,EACnB,MAAM,EAAE,UAAU,EAClB,IAAI,EAAE,UAAU,EAAE,EAClB,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,CAsOR;AA6BD;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAC5B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,UAAU,EAChB,MAAM,EAAE,MAAM,EAAE,EAChB,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,GAAG,IAAI,CAOf;AAED;;;;;;;GAOG;AACH,wBAAgB,gBAAgB,CAC9B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,UAAU,EAChB,MAAM,EAAE,MAAM,EAAE,EAChB,IAAI,EAAE,UAAU,EAChB,SAAS,EAAE,MAAM,GAAG,OAAO,EAC3B,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,GAAG,IAAI,CAmBf;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,mBAAmB,CACjC,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,EACZ,SAAS,EAAE,UAAU,EACrB,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,EAC/B,OAAO,CAAC,EAAE,OAAO,GAChB,MAAM,GAAG,IAAI,CAMf;AAED;;;;;;;GAOG;AACH,wBAAgB,iBAAiB,CAC/B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,UAAU,EAChB,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,GAAG,IAAI,CAKf;AAED;;;;;;;;GAQG;AACH,wBAAgB,aAAa,CAC3B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,UAAU,EAChB,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,GAAG,IAAI,CAKf;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,CAAC,EAAE,cAAc,GAAG,MAAM,CAaxE;AAKD,wBAAgB,gBAAgB,CAC9B,IAAI,EAAE,MAAM,EACZ,KAAK,EAAE,SAAS,GAAG;IAAE,IAAI,EAAE,UAAU,CAAA;CAAE,EACvC,IAAI,EAAE,CAAC,CAAC,EAAE,UAAU,KAAK,MAAM,GAC9B,MAAM,CAoBR"}

package/dist/adapter/index.js

@@ -149,7 +149,7 @@ function renderArrayMethod(method, object, args, emit) {
     case "join": {
       const obj = emit(object);
       const sep = args.length >= 1 ? emit(args[0]) : `','`;
-      return `join(${sep}, @{${obj}})`;
+      return `bf->join(${obj}, ${sep})`;
     }
     case "includes": {
       const obj = emit(object);

package/dist/build.js

@@ -149,7 +149,7 @@ function renderArrayMethod(method, object, args, emit) {
     case "join": {
       const obj = emit(object);
       const sep = args.length >= 1 ? emit(args[0]) : `','`;
-      return `join(${sep}, @{${obj}})`;
+      return `bf->join(${obj}, ${sep})`;
     }
     case "includes": {
       const obj = emit(object);

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;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AAEtD,eAAO,MAAM,eAAe,EAAE,eAmI7B,CAAA"}
+{"version":3,"file":"conformance-pins.d.ts","sourceRoot":"","sources":["../src/conformance-pins.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AAEtD,eAAO,MAAM,eAAe,EAAE,eAyI7B,CAAA"}

package/dist/index.js

@@ -149,7 +149,7 @@ function renderArrayMethod(method, object, args, emit) {
     case "join": {
       const obj = emit(object);
       const sep = args.length >= 1 ? emit(args[0]) : `','`;
-      return `join(${sep}, @{${obj}})`;
+      return `bf->join(${obj}, ${sep})`;
     }
     case "includes": {
       const obj = emit(object);
@@ -1912,7 +1912,8 @@ var conformancePins = {
   "filter-nested-find-predicate": [
     { code: "BF101", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2038" }
   ],
-  "dangerous-inner-html-dynamic": [{ code: "BF101", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2215" }]
+  "dangerous-inner-html-dynamic": [{ code: "BF101", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2215" }],
+  "date-method-uncatalogued": [{ code: "BF021", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2274" }]
 };
 // src/render-divergences.ts
 var renderDivergences = {};

package/lib/BarefootJS/Backend/Mojo.pm

@@ -1,5 +1,5 @@
 package BarefootJS::Backend::Mojo;
-our $VERSION = "0.19.0";
+our $VERSION = "0.19.1";
 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.19.0";
+our $VERSION = "0.19.1";
 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.19.0";
+our $VERSION = "0.19.1";
 use Mojo::Base 'Mojolicious::Plugin', -signatures;
 
 use Mojo::File qw(path);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@barefootjs/mojolicious",
-  "version": "0.19.1",
+  "version": "0.20.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.19.1"
+    "@barefootjs/shared": "0.20.0"
   },
   "peerDependencies": {
     "@barefootjs/jsx": ">=0.2.0",
@@ -60,6 +60,6 @@
   },
   "devDependencies": {
     "@barefootjs/adapter-tests": "0.1.0",
-    "@barefootjs/jsx": "0.19.1"
+    "@barefootjs/jsx": "0.20.0"
   }
 }

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

@@ -59,17 +59,7 @@ runAdapterConformanceTests({
     // #1467 Phase 2e: same `/* @client */` keyed-map elision (data-table).
     'data-table',
   ]),
-  skipDataPoints: new Set<string>([
-    // #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',
-    // #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',
-  ]),
+  skipDataPoints: new Set<string>(),
   onRenderError: (err, id) => {
     if (err instanceof PerlNotAvailableError) {
       console.log(`Skipping [${id}]: ${err.message}`)
@@ -915,7 +905,7 @@ export function C() {
     // filter, `.join`) hit a separate refusal gate and the chain
     // emitted BF101 — making the scaffold `<Button>` / `<Card>`
     // unusable on Mojo. The fix lowers all three to Embedded Perl
-    // (`join(' ', @{[grep { $_ } @{[...]}]})`), unblocking the
+    // (`bf->join([grep { $_ } @{[...]}], ' ')`), unblocking the
     // registry surface. The #1421 recursion guard stays in place
     // as defence in depth for other unsupported shapes, but this
     // specific chain no longer reaches the loop because the parser
@@ -938,7 +928,7 @@ export { Slot }
     )
     expect(result.errors?.filter(e => e.code === 'BF101') ?? []).toEqual([])
     const template = result.files.find(f => f.path.endsWith('.html.ep'))?.content ?? ''
-    expect(template).toContain(`join(' ', @{[grep { $_ } @{[$className]}]})`)
+    expect(template).toContain(`bf->join([grep { $_ } @{[$className]}], ' ')`)
   })
 
   test('lowers .includes(x) on an array prop via bf->includes(...) (#1448 Tier A)', () => {
@@ -1168,7 +1158,7 @@ export { A }`, 'A.tsx', { adapter })
 export { A }`, 'A.tsx', { adapter })
     expect(result.errors?.filter(e => e.code === 'BF101') ?? []).toEqual([])
     const template = result.files.find(f => f.path.endsWith('.html.ep'))?.content ?? ''
-    expect(template).toContain("join(' ', @{bf->reverse($items)})")
+    expect(template).toContain("bf->join(bf->reverse($items), ' ')")
     expect(template).not.toContain('$bf->reverse')
   })
 
@@ -1180,7 +1170,7 @@ export { A }`, 'A.tsx', { adapter })
 export { A }`, 'A.tsx', { adapter })
     expect(result.errors?.filter(e => e.code === 'BF101') ?? []).toEqual([])
     const template = result.files.find(f => f.path.endsWith('.html.ep'))?.content ?? ''
-    expect(template).toContain("join(' ', @{bf->reverse($items)})")
+    expect(template).toContain("bf->join(bf->reverse($items), ' ')")
   })
 
   test('lowers .slice(start, end).join(\' \') via bf->slice + join (#1448 Tier A)', () => {
@@ -1192,7 +1182,7 @@ export { A }`, 'A.tsx', { adapter })
 export { A }`, 'A.tsx', { adapter })
     expect(result.errors?.filter(e => e.code === 'BF101') ?? []).toEqual([])
     const template = result.files.find(f => f.path.endsWith('.html.ep'))?.content ?? ''
-    expect(template).toContain("join(' ', @{bf->slice($items, 1, 3)})")
+    expect(template).toContain("bf->join(bf->slice($items, 1, 3), ' ')")
     expect(template).not.toContain('$bf->slice')
   })
 
@@ -1206,7 +1196,7 @@ export { A }`, 'A.tsx', { adapter })
 export { A }`, 'A.tsx', { adapter })
     expect(result.errors?.filter(e => e.code === 'BF101') ?? []).toEqual([])
     const template = result.files.find(f => f.path.endsWith('.html.ep'))?.content ?? ''
-    expect(template).toContain("join(' ', @{bf->slice($items, 2, undef)})")
+    expect(template).toContain("bf->join(bf->slice($items, 2, undef), ' ')")
   })
 
   test('lowers .concat(other).join(\' \') via bf->concat + join (#1448 Tier A)', () => {
@@ -1222,7 +1212,7 @@ export { A }`, 'A.tsx', { adapter })
 export { A }`, 'A.tsx', { adapter })
     expect(result.errors?.filter(e => e.code === 'BF101') ?? []).toEqual([])
     const template = result.files.find(f => f.path.endsWith('.html.ep'))?.content ?? ''
-    expect(template).toContain("join(' ', @{bf->concat($left, $right)})")
+    expect(template).toContain("bf->join(bf->concat($left, $right), ' ')")
     expect(template).not.toContain('$bf->concat')
   })
 
@@ -1484,10 +1474,10 @@ describe('MojoAdapter - #1448 Tier A/B fixture-driven lowering pins', () => {
     { fixture: arraySliceFixture,       expect: 'bf->slice($items, 1, 3)' },
     // #1448 full-arity — zero-arg defaults.
     { fixture: arraySliceCopyFixture,   expect: 'bf->slice($items, 0, undef)' },
-    { fixture: arrayJoinDefaultFixture, expect: `join(',', @{$items})` },
+    { fixture: arrayJoinDefaultFixture, expect: `bf->join($items, ',')` },
     // `.at()` → index 0; `.concat()` → the receiver (shallow copy).
     { fixture: arrayAtDefaultFixture,   expect: 'bf->at($items, 0)' },
-    { fixture: arrayConcatCopyFixture,  expect: `join('|', @{$items})` },
+    { fixture: arrayConcatCopyFixture,  expect: `bf->join($items, '|')` },
     { fixture: arrayReverseFixture,     expect: 'bf->reverse($items)' },
     // .toReversed shares the helper with .reverse — pinning both
     // routings catches a future divergence between them.
@@ -1497,7 +1487,7 @@ describe('MojoAdapter - #1448 Tier A/B fixture-driven lowering pins', () => {
     { fixture: stringTrimFixture,       expect: 'bf->trim($value)' },
     // #1448 Tier B — string → array. `.split(',')` lowers to
     // `bf->split`, here chained into `.join('|')` so the array ref is
-    // observable (`join('|', @{bf->split($value, ',')})`).
+    // observable (`bf->join(bf->split($value, ','), '|')`).
     { fixture: stringSplitFixture,      expect: `bf->split($value, ',')` },
     { fixture: stringSplitLimitFixture, expect: `bf->split($value, ',', 2)` },
     // #1448 Tier B — string → boolean at condition position (`% if`).
@@ -1687,7 +1677,7 @@ export { C }
   // projects each element (no flatten) and composes through `.join`.
   test('.map(t => `#${t}`).join(" ") emits bf->map_eval composed into join', () => {
     const t = emitMap("tags.map(t => `#${t}`).join(' ')")
-    expect(t).toContain(`join(' ', @{bf->map_eval($tags,`)
+    expect(t).toContain(`bf->join(bf->map_eval($tags,`)
     expect(t).toContain(`"kind":"template-literal"`)
   })
 

package/src/adapter/expr/array-method.ts

@@ -35,14 +35,20 @@ export function renderArrayMethod(
 ): string {
   switch (method) {
     case 'join': {
-      // arr.join(sep) → join(sep, @{arr}). The default `${obj}->{join}`
-      // hash-lookup fallback would emit invalid Perl, which is why the
-      // IR carves out a dedicated method node instead of routing
-      // through the generic call dispatcher. `.join()` defaults the
-      // separator to `,` (JS) and ignores any extra argument.
+      // arr.join(sep) → bf->join(arr, sep), NOT Perl's native `join`
+      // builtin directly on `@{arr}` — a nested-array element (e.g.
+      // `.flat(0)`'s shallow copy) would stringify to its Perl memory
+      // address ("ARRAY(0x...)") under native `join`'s scalar coercion
+      // instead of JS's recursive comma-join (#2262/#2272). `bf->join`
+      // (BarefootJS.pm) routes each element through `string()` first.
+      // The default `${obj}->{join}` hash-lookup fallback would emit
+      // invalid Perl, which is why the IR carves out a dedicated method
+      // node instead of routing through the generic call dispatcher.
+      // `.join()` defaults the separator to `,` (JS) and ignores any
+      // extra argument.
       const obj = emit(object)
       const sep = args.length >= 1 ? emit(args[0]) : `','`
-      return `join(${sep}, @{${obj}})`
+      return `bf->join(${obj}, ${sep})`
     }
     case 'includes': {
       // Both `arr.includes(x)` and `str.includes(sub)` route here —

package/src/conformance-pins.ts

@@ -88,7 +88,7 @@ export const conformancePins: ConformancePins = {
   // below for the shared `/* @client */` keyed-map slot-id elision
   // contract (same as `todo-app`), not a render or BF101 gap.
   // #1443: `[a, b].filter(Boolean).join(' ')` (the registry Slot's
-  // shape) now lowers to `join(' ', @{[grep { $_ } @{[$a, $b]}]})`.
+  // shape) now lowers to `bf->join([grep { $_ } @{[$a, $b]}], ' ')`.
   // No BF101 expected — pinned positively via the
   // `branch-local-filter-join` template-output test below.
   //
@@ -139,4 +139,10 @@ export const conformancePins: ConformancePins = {
   // A dynamic/signal-derived value still refuses with BF101 — see the
   // `dangerous-inner-html-dynamic` fixture/pin below (tracked: #2215).
   'dangerous-inner-html-dynamic': [{ code: 'BF101', severity: 'error', issue: 'https://github.com/piconic-ai/barefootjs/issues/2215' }],
+  // #2273: a method call on a prop typed as a built-in host rich type
+  // (Date, Map, …) has no catalogued lowering in any adapter — this is a
+  // compiler-level refusal (`checkRichTypeMethodCalls`, wired ahead of
+  // `adapter.generate()`), not an adapter-specific gap, so it is pinned
+  // identically across every adapter package including Hono.
+  'date-method-uncatalogued': [{ code: 'BF021', severity: 'error', issue: 'https://github.com/piconic-ai/barefootjs/issues/2274' }],
 }