bestax-migrate 2.23.0 → 2.23.1

package/dist/sources/rbx/mapping.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"mapping.d.ts","sourceRoot":"","sources":["../../../src/sources/rbx/mapping.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAEH,OAAO,KAAK,EAAE,gBAAgB,EAAE,UAAU,EAAE,MAAM,gBAAgB,CAAC;AA8BnE;;;;;;;;;GASG;AACH,eAAO,MAAM,eAAe,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CA6CtD,CAAC;AAEF,uDAAuD;AACvD,eAAO,MAAM,WAAW,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAOrD,CAAC;AAEF,2DAA2D;AAC3D,eAAO,MAAM,aAAa,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAQvD,CAAC;AAEF;;;;GAIG;AACH,eAAO,MAAM,sBAAsB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAShE,CAAC;AAEF,eAAO,MAAM,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,gBAAgB,CAktBpD,CAAC;AAEF;;;;;;;GAOG;AACH,eAAO,MAAM,WAAW,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,EAAE,CA2FhD,CAAC;AAEF,sDAAsD;AACtD,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,gBAAgB,GAAG,SAAS,CAM3E"}
+{"version":3,"file":"mapping.d.ts","sourceRoot":"","sources":["../../../src/sources/rbx/mapping.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAEH,OAAO,KAAK,EAAE,gBAAgB,EAAE,UAAU,EAAE,MAAM,gBAAgB,CAAC;AA8BnE;;;;;;;;;GASG;AACH,eAAO,MAAM,eAAe,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CA4CtD,CAAC;AAEF,uDAAuD;AACvD,eAAO,MAAM,WAAW,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAOrD,CAAC;AAEF,2DAA2D;AAC3D,eAAO,MAAM,aAAa,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAQvD,CAAC;AAEF;;;;GAIG;AACH,eAAO,MAAM,sBAAsB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAShE,CAAC;AAEF,eAAO,MAAM,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,gBAAgB,CA0tBpD,CAAC;AAEF;;;;;;;GAOG;AACH,eAAO,MAAM,WAAW,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,EAAE,CA2FhD,CAAC;AAEF,sDAAsD;AACtD,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,gBAAgB,GAAG,SAAS,CAM3E"}

package/dist/sources/rbx/mapping.js

@@ -79,8 +79,7 @@ export const UNIVERSAL_PROPS = {
     // Per-component maps override this with AS_OK where bestax has an `as`.
     as: AS_TODO,
     // Same shape: the per-component maps override this with `rename: 'ref'` on
-    // the targets that forward one, and the `navbar-item` special handles the
-    // two it chooses between. Whatever is left reaches here, so no `innerRef`
+    // the targets that forward one. Whatever is left reaches here, so no `innerRef`
     // migrates in silence onto a target that cannot take it — the RBC sibling
     // `domRef` has been flagged universally all along.
     innerRef: {
@@ -697,7 +696,15 @@ export const MAPPING = {
                 target: 'Navbar.Item',
                 status: 'mapped',
                 special: 'navbar-item',
-                props: { active: {}, onClick: {}, as: AS_OK },
+                // Both targets the special picks forward a ref to the element rbx's
+                // `innerRef` reached: the item itself, or the `has-dropdown`
+                // container `Navbar.Dropdown` renders.
+                props: {
+                    active: {},
+                    onClick: {},
+                    as: AS_OK,
+                    innerRef: { rename: 'ref' },
+                },
                 subs: { Container: { status: 'mapped', special: 'navbar-container' } },
             },
             Dropdown: { status: 'mapped', special: 'navbar-dropdown' },

package/dist/sources/rbx/specials.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"specials.d.ts","sourceRoot":"","sources":["../../../src/sources/rbx/specials.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAO3C,OAAO,EAYL,KAAK,gBAAgB,EACtB,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAUL,KAAK,aAAa,EACnB,MAAM,8BAA8B,CAAC;AAEtC,YAAY,EAAE,aAAa,EAAE,CAAC;AAq3B9B,wBAAgB,UAAU,CACxB,IAAI,EAAE,MAAM,EACZ,GAAG,EAAE,gBAAgB,EACrB,IAAI,EAAE,OAAO,CAAC,GAAG,CAAC,EAClB,OAAO,EAAE,GAAG,GACX,aAAa,CAIf"}
+{"version":3,"file":"specials.d.ts","sourceRoot":"","sources":["../../../src/sources/rbx/specials.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAO3C,OAAO,EAYL,KAAK,gBAAgB,EACtB,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAUL,KAAK,aAAa,EACnB,MAAM,8BAA8B,CAAC;AAEtC,YAAY,EAAE,aAAa,EAAE,CAAC;AAu1B9B,wBAAgB,UAAU,CACxB,IAAI,EAAE,MAAM,EACZ,GAAG,EAAE,gBAAgB,EACrB,IAAI,EAAE,OAAO,CAAC,GAAG,CAAC,EAClB,OAAO,EAAE,GAAG,GACX,aAAa,CAIf"}

package/dist/sources/rbx/specials.js

@@ -510,31 +510,6 @@ const SPECIALS = {
             addTodo(ctx, path, `prop:${prop}`, `bestax \`Navbar.Item\` has no \`${prop}\` prop; restructure or add a class by hand`);
             ctx.dirty = true;
         }
-        // bestax's `Navbar.Dropdown` forwards a ref, but it is the one such target
-        // the `innerRef: { rename: 'ref' }` entries in mapping.ts cannot reach:
-        // that table is keyed on the rbx name (`Navbar.Item`), and only this
-        // handler knows which of the two targets was picked. The condition is
-        // about that, not about ref support: a plain `Navbar.Item` forwards one
-        // too since #661, so leaving `innerRef` alone there is a rename this
-        // handler declines rather than one the target cannot take. Performing it
-        // changes emitted output on code that migrates today, so it stays #734 —
-        // but declining silently is not an option either. `innerRef` is in no
-        // rbx prop table, so nothing downstream flags it and it used to reach the
-        // DOM as an unknown attribute with the report saying nothing, which is
-        // the silent skip bestax-migrate/CLAUDE.md rules out. It is flagged here
-        // instead, and the rename stays the user's to make.
-        const handledInnerRef = [];
-        const innerRefAttr = findAttr(element, 'innerRef');
-        if (innerRefAttr) {
-            handledInnerRef.push('innerRef');
-            if (target === 'Navbar.Dropdown') {
-                innerRefAttr.name = ctx.j.jsxIdentifier('ref');
-                ctx.dirty = true;
-            }
-            else {
-                addTodo(ctx, path, 'prop:innerRef', 'bestax `Navbar.Item` forwards a ref; rename `innerRef` to `ref` by hand');
-            }
-        }
         restrictAsToTargets(ctx, path, element, target ?? 'Navbar.Item', [
             'Navbar.Item',
         ]);
@@ -548,7 +523,6 @@ const SPECIALS = {
                 'expanded',
                 'hoverable',
                 'managed',
-                ...handledInnerRef,
             ],
         };
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bestax-migrate",
-  "version": "2.23.0",
+  "version": "2.23.1",
   "description": "Codemods for migrating existing React apps from other Bulma libraries to @allxsmith/bestax-bulma",
   "type": "module",
   "bin": {