bestax-migrate 2.1.3 → 2.1.4

package/dist/sources/react-bulma-components/jsx-utils.d.ts

@@ -55,6 +55,17 @@ export type LiteralValue = {
 };
 /** Extract the literal value of a JSX attribute, if it has one. */
 export declare function literalValueOf(attr: any): LiteralValue;
+export type Booleanish = 'truthy' | 'falsy' | 'expression';
+/**
+ * Classify a JSX attribute by the truthiness RBC evaluates it with at
+ * runtime (`!!value`): every statically-known literal (boolean, string, or
+ * number) resolves to `'truthy'`/`'falsy'`; only a genuine expression is
+ * `'expression'`. This is the single place the six call sites that branch on
+ * a "booleanish" prop (Button `remove`, Heading `heading`/`subtitle`,
+ * `booleanToProp`, the `active` ladders, Field `multiline`) agree on what
+ * counts as resolvable, so they can't drift out of sync with each other.
+ */
+export declare function resolveBooleanish(attr: any): Booleanish;
 /** Create `name` (bare boolean) or `name="value"` attribute. */
 export declare function makeAttr(j: JSCodeshift, name: string, value?: string): any;
 /**

package/dist/sources/react-bulma-components/jsx-utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"jsx-utils.d.ts","sourceRoot":"","sources":["../../../src/sources/react-bulma-components/jsx-utils.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAC/C,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAC3C,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAC;AAMpD,MAAM,WAAW,gBAAgB;IAC/B,CAAC,EAAE,WAAW,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,aAAa,GAAG,SAAS,CAAC;IACrC,uEAAuE;IACvE,QAAQ,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IACtB,8EAA8E;IAC9E,MAAM,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC5B;;;;OAIG;IACH,OAAO,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,MAAM,CAAC;IAClC;;;;OAIG;IACH,SAAS,EAAE,OAAO,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACnC,6EAA6E;IAC7E,OAAO,CAAC,EAAE,CAAC,IAAI,EAAE,GAAG,KAAK,MAAM,EAAE,GAAG,IAAI,CAAC;IACzC,KAAK,EAAE,OAAO,CAAC;CAChB;AAED,2EAA2E;AAC3E,wBAAgB,YAAY,CAAC,IAAI,EAAE,GAAG,GAAG,MAAM,EAAE,GAAG,IAAI,CASvD;AAED,4EAA4E;AAC5E,wBAAgB,YAAY,CAAC,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,MAAM,GAAG,GAAG,CAOhE;AAED,kEAAkE;AAClE,wBAAgB,aAAa,CAC3B,CAAC,EAAE,WAAW,EACd,OAAO,EAAE,GAAG,EACZ,MAAM,EAAE,MAAM,GACb,IAAI,CAKN;AAED,2CAA2C;AAC3C,wBAAgB,YAAY,CAAC,OAAO,EAAE,GAAG,GAAG,GAAG,EAAE,CAIhD;AAED,wBAAgB,QAAQ,CAAC,OAAO,EAAE,GAAG,EAAE,IAAI,EAAE,MAAM,GAAG,GAAG,GAAG,SAAS,CAEpE;AAED,wBAAgB,UAAU,CAAC,OAAO,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,GAAG,IAAI,CAIxD;AAED,wBAAgB,OAAO,CAAC,OAAO,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,GAAG,IAAI,CAGrD;AAED,MAAM,MAAM,YAAY,GACpB;IAAE,IAAI,EAAE,SAAS,CAAC;IAAC,KAAK,EAAE,OAAO,CAAA;CAAE,GACnC;IAAE,IAAI,EAAE,QAAQ,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,GACjC;IAAE,IAAI,EAAE,QAAQ,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,GACjC;IAAE,IAAI,EAAE,YAAY,CAAA;CAAE,CAAC;AAE3B,mEAAmE;AACnE,wBAAgB,cAAc,CAAC,IAAI,EAAE,GAAG,GAAG,YAAY,CActD;AAED,gEAAgE;AAChE,wBAAgB,QAAQ,CAAC,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,MAAM,GAAG,GAAG,CAK1E;AAED;;;;;GAKG;AACH,wBAAgB,OAAO,CACrB,GAAG,EAAE,gBAAgB,EACrB,IAAI,EAAE,OAAO,CAAC,GAAG,CAAC,EAClB,IAAI,EAAE,MAAM,EACZ,OAAO,EAAE,MAAM,GACd,IAAI,CAuCN;AAED,uEAAuE;AACvE,wBAAgB,YAAY,CAC1B,CAAC,EAAE,WAAW,EACd,GAAG,EAAE,MAAM,EACX,SAAS,EAAE,MAAM,GAAG,SAAS,EAC7B,KAAK,EAAE,GAAG,EAAE,EACZ,QAAQ,EAAE,GAAG,EAAE,EACf,WAAW,UAAQ,GAClB,GAAG,CAcL"}
+{"version":3,"file":"jsx-utils.d.ts","sourceRoot":"","sources":["../../../src/sources/react-bulma-components/jsx-utils.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAC/C,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAC3C,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAC;AAMpD,MAAM,WAAW,gBAAgB;IAC/B,CAAC,EAAE,WAAW,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,aAAa,GAAG,SAAS,CAAC;IACrC,uEAAuE;IACvE,QAAQ,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IACtB,8EAA8E;IAC9E,MAAM,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC5B;;;;OAIG;IACH,OAAO,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,MAAM,CAAC;IAClC;;;;OAIG;IACH,SAAS,EAAE,OAAO,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACnC,6EAA6E;IAC7E,OAAO,CAAC,EAAE,CAAC,IAAI,EAAE,GAAG,KAAK,MAAM,EAAE,GAAG,IAAI,CAAC;IACzC,KAAK,EAAE,OAAO,CAAC;CAChB;AAED,2EAA2E;AAC3E,wBAAgB,YAAY,CAAC,IAAI,EAAE,GAAG,GAAG,MAAM,EAAE,GAAG,IAAI,CASvD;AAED,4EAA4E;AAC5E,wBAAgB,YAAY,CAAC,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,MAAM,GAAG,GAAG,CAOhE;AAED,kEAAkE;AAClE,wBAAgB,aAAa,CAC3B,CAAC,EAAE,WAAW,EACd,OAAO,EAAE,GAAG,EACZ,MAAM,EAAE,MAAM,GACb,IAAI,CAKN;AAED,2CAA2C;AAC3C,wBAAgB,YAAY,CAAC,OAAO,EAAE,GAAG,GAAG,GAAG,EAAE,CAIhD;AAED,wBAAgB,QAAQ,CAAC,OAAO,EAAE,GAAG,EAAE,IAAI,EAAE,MAAM,GAAG,GAAG,GAAG,SAAS,CAEpE;AAED,wBAAgB,UAAU,CAAC,OAAO,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,GAAG,IAAI,CAIxD;AAED,wBAAgB,OAAO,CAAC,OAAO,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,GAAG,IAAI,CAGrD;AAED,MAAM,MAAM,YAAY,GACpB;IAAE,IAAI,EAAE,SAAS,CAAC;IAAC,KAAK,EAAE,OAAO,CAAA;CAAE,GACnC;IAAE,IAAI,EAAE,QAAQ,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,GACjC;IAAE,IAAI,EAAE,QAAQ,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,GACjC;IAAE,IAAI,EAAE,YAAY,CAAA;CAAE,CAAC;AAE3B,mEAAmE;AACnE,wBAAgB,cAAc,CAAC,IAAI,EAAE,GAAG,GAAG,YAAY,CActD;AAED,MAAM,MAAM,UAAU,GAAG,QAAQ,GAAG,OAAO,GAAG,YAAY,CAAC;AAE3D;;;;;;;;GAQG;AACH,wBAAgB,iBAAiB,CAAC,IAAI,EAAE,GAAG,GAAG,UAAU,CAIvD;AAED,gEAAgE;AAChE,wBAAgB,QAAQ,CAAC,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,MAAM,GAAG,GAAG,CAK1E;AAED;;;;;GAKG;AACH,wBAAgB,OAAO,CACrB,GAAG,EAAE,gBAAgB,EACrB,IAAI,EAAE,OAAO,CAAC,GAAG,CAAC,EAClB,IAAI,EAAE,MAAM,EACZ,OAAO,EAAE,MAAM,GACd,IAAI,CAuCN;AAED,uEAAuE;AACvE,wBAAgB,YAAY,CAC1B,CAAC,EAAE,WAAW,EACd,GAAG,EAAE,MAAM,EACX,SAAS,EAAE,MAAM,GAAG,SAAS,EAC7B,KAAK,EAAE,GAAG,EAAE,EACZ,QAAQ,EAAE,GAAG,EAAE,EACf,WAAW,UAAQ,GAClB,GAAG,CAcL"}

package/dist/sources/react-bulma-components/jsx-utils.js

@@ -70,6 +70,21 @@ export function literalValueOf(attr) {
     }
     return { kind: 'expression' };
 }
+/**
+ * Classify a JSX attribute by the truthiness RBC evaluates it with at
+ * runtime (`!!value`): every statically-known literal (boolean, string, or
+ * number) resolves to `'truthy'`/`'falsy'`; only a genuine expression is
+ * `'expression'`. This is the single place the six call sites that branch on
+ * a "booleanish" prop (Button `remove`, Heading `heading`/`subtitle`,
+ * `booleanToProp`, the `active` ladders, Field `multiline`) agree on what
+ * counts as resolvable, so they can't drift out of sync with each other.
+ */
+export function resolveBooleanish(attr) {
+    const literal = literalValueOf(attr);
+    if (literal.kind === 'expression')
+        return 'expression';
+    return literal.value ? 'truthy' : 'falsy';
+}
 /** Create `name` (bare boolean) or `name="value"` attribute. */
 export function makeAttr(j, name, value) {
     return j.jsxAttribute(j.jsxIdentifier(name), value === undefined ? null : j.stringLiteral(value));

package/dist/sources/react-bulma-components/props.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"props.d.ts","sourceRoot":"","sources":["../../../src/sources/react-bulma-components/props.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAC3C,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,gBAAgB,CAAC;AAEjD,OAAO,EAQL,KAAK,gBAAgB,EACtB,MAAM,gBAAgB,CAAC;AA6BxB,mDAAmD;AACnD,wBAAgB,eAAe,CAC7B,GAAG,EAAE,gBAAgB,EACrB,IAAI,EAAE,OAAO,CAAC,GAAG,CAAC,EAClB,OAAO,EAAE,GAAG,EACZ,IAAI,EAAE,GAAG,EACT,MAAM,EAAE,UAAU,GACjB,IAAI,CA4HN;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CACjC,GAAG,EAAE,gBAAgB,EACrB,IAAI,EAAE,OAAO,CAAC,GAAG,CAAC,EAClB,OAAO,EAAE,GAAG,EACZ,OAAO,EAAE,GAAG,CAAC,MAAM,CAAC,GACnB,IAAI,CAQN"}
+{"version":3,"file":"props.d.ts","sourceRoot":"","sources":["../../../src/sources/react-bulma-components/props.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAC3C,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,gBAAgB,CAAC;AAEjD,OAAO,EASL,KAAK,gBAAgB,EACtB,MAAM,gBAAgB,CAAC;AA6BxB,mDAAmD;AACnD,wBAAgB,eAAe,CAC7B,GAAG,EAAE,gBAAgB,EACrB,IAAI,EAAE,OAAO,CAAC,GAAG,CAAC,EAClB,OAAO,EAAE,GAAG,EACZ,IAAI,EAAE,GAAG,EACT,MAAM,EAAE,UAAU,GACjB,IAAI,CA6HN;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CACjC,GAAG,EAAE,gBAAgB,EACrB,IAAI,EAAE,OAAO,CAAC,GAAG,CAAC,EAClB,OAAO,EAAE,GAAG,EACZ,OAAO,EAAE,GAAG,CAAC,MAAM,CAAC,GACnB,IAAI,CAQN"}

package/dist/sources/react-bulma-components/props.js

@@ -3,7 +3,7 @@
  * universal modifier-prop pass shared by every mapped component.
  */
 import { UNIVERSAL_PROPS } from './mapping.js';
-import { addAttr, addTodo, attributesOf, findAttr, literalValueOf, makeAttr, removeAttr, } from './jsx-utils.js';
+import { addAttr, addTodo, attributesOf, findAttr, literalValueOf, makeAttr, removeAttr, resolveBooleanish, } from './jsx-utils.js';
 /* eslint-disable @typescript-eslint/no-explicit-any */
 /**
  * Add a converted attribute unless the target name is already present — two
@@ -56,12 +56,13 @@ export function applyPropAction(ctx, path, element, attr, action) {
     }
     if (action.booleanToProp) {
         const { name, value } = action.booleanToProp;
-        if (literal.kind === 'boolean' && literal.value === true) {
+        const resolved = resolveBooleanish(attr);
+        if (resolved === 'truthy') {
             removeAttr(element, attr);
             addConverted(ctx, path, element, originalName, name, value);
             ctx.dirty = true;
         }
-        else if (literal.kind === 'boolean' && literal.value === false) {
+        else if (resolved === 'falsy') {
             removeAttr(element, attr);
             ctx.dirty = true;
         }

package/dist/sources/react-bulma-components/specials.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"specials.d.ts","sourceRoot":"","sources":["../../../src/sources/react-bulma-components/specials.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAE3C,OAAO,EAUL,KAAK,gBAAgB,EACtB,MAAM,gBAAgB,CAAC;AAIxB,MAAM,WAAW,aAAa;IAC5B,2DAA2D;IAC3D,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,oEAAoE;IACpE,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,yEAAyE;IACzE,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;CACzB;AA21BD,wBAAgB,UAAU,CACxB,IAAI,EAAE,MAAM,EACZ,GAAG,EAAE,gBAAgB,EACrB,IAAI,EAAE,OAAO,CAAC,GAAG,CAAC,EAClB,OAAO,EAAE,GAAG,GACX,aAAa,CAIf;AAED,kEAAkE;AAClE,eAAO,MAAM,gBAAgB,EAAE,MAAM,CAAC,MAAM,EAAE,SAAS,GAAG,QAAQ,CAGjE,CAAC"}
+{"version":3,"file":"specials.d.ts","sourceRoot":"","sources":["../../../src/sources/react-bulma-components/specials.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAE3C,OAAO,EAWL,KAAK,gBAAgB,EACtB,MAAM,gBAAgB,CAAC;AAIxB,MAAM,WAAW,aAAa;IAC5B,2DAA2D;IAC3D,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,oEAAoE;IACpE,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,yEAAyE;IACzE,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;CACzB;AAo3BD,wBAAgB,UAAU,CACxB,IAAI,EAAE,MAAM,EACZ,GAAG,EAAE,gBAAgB,EACrB,IAAI,EAAE,OAAO,CAAC,GAAG,CAAC,EAClB,OAAO,EAAE,GAAG,GACX,aAAa,CAIf;AAED,kEAAkE;AAClE,eAAO,MAAM,gBAAgB,EAAE,MAAM,CAAC,MAAM,EAAE,SAAS,GAAG,QAAQ,CAGjE,CAAC"}

package/dist/sources/react-bulma-components/specials.js

@@ -8,7 +8,7 @@
  * element `replaced` when it substituted the node itself.
  */
 import { RESPONSIVE_BREAKPOINTS, UNIVERSAL_PROPS } from './mapping.js';
-import { addAttr, addTodo, attributesOf, buildJsxName, findAttr, literalValueOf, makeAttr, plainElement, removeAttr, } from './jsx-utils.js';
+import { addAttr, addTodo, attributesOf, buildJsxName, findAttr, literalValueOf, makeAttr, plainElement, removeAttr, resolveBooleanish, } from './jsx-utils.js';
 /** Pick a target based on a literal align-style prop, removing the prop. */
 function alignTarget(ctx, path, element, prop, targets, fallback) {
     const attr = findAttr(element, prop);
@@ -141,14 +141,14 @@ const SPECIALS = {
         const attr = findAttr(element, 'remove');
         if (!attr)
             return {};
-        const literal = literalValueOf(attr);
-        if (literal.kind !== 'expression') {
+        const resolved = resolveBooleanish(attr);
+        if (resolved !== 'expression') {
             // A statically-known value always renders the same at runtime: truthy
             // (`remove`, `remove={true}`, `remove="true"`) → `<Delete />`; falsy
             // (`remove={false}`, `remove=""`, `remove={0}`) → just drop the prop.
             removeAttr(element, attr);
             ctx.dirty = true;
-            return literal.value ? { target: 'Delete' } : {};
+            return resolved === 'truthy' ? { target: 'Delete' } : {};
         }
         addTodo(ctx, path, 'prop:remove', '`remove` has a dynamic value; this can render as either a Button or a Delete cross — split the branch by hand');
         return {};
@@ -172,14 +172,18 @@ const SPECIALS = {
     heading(ctx, path, element) {
         const headingAttr = findAttr(element, 'heading');
         const subtitleAttr = findAttr(element, 'subtitle');
-        const headingLit = headingAttr ? literalValueOf(headingAttr) : undefined;
-        const subtitleLit = subtitleAttr ? literalValueOf(subtitleAttr) : undefined;
-        const headingDynamic = headingLit?.kind === 'expression';
-        const subtitleDynamic = subtitleLit?.kind === 'expression';
+        const headingResolved = headingAttr
+            ? resolveBooleanish(headingAttr)
+            : undefined;
+        const subtitleResolved = subtitleAttr
+            ? resolveBooleanish(subtitleAttr)
+            : undefined;
+        const headingDynamic = headingResolved === 'expression';
+        const subtitleDynamic = subtitleResolved === 'expression';
         // A resolvable literal (boolean/string/number) collapses to its runtime
         // truthiness; an expression stays dynamic.
-        const headingTruthy = !!headingLit && headingLit.kind !== 'expression' && !!headingLit.value;
-        const subtitleTruthy = !!subtitleLit && subtitleLit.kind !== 'expression' && !!subtitleLit.value;
+        const headingTruthy = headingResolved === 'truthy';
+        const subtitleTruthy = subtitleResolved === 'truthy';
         // Dynamic values can't be resolved to a target: leave the prop on the
         // element (so its expression survives for hand-splitting) plus a TODO.
         if (subtitleDynamic) {
@@ -205,11 +209,11 @@ const SPECIALS = {
         // A resolvable literal `heading` that didn't collapse (falsy, or truthy
         // but blocked by a dynamic subtitle) behaves as if the prop were absent —
         // drop it. A dynamic `heading` stays put (its TODO is already filed above).
-        if (headingLit && !headingDynamic) {
+        if (headingAttr && !headingDynamic) {
             removeAttr(element, headingAttr);
             ctx.dirty = true;
         }
-        if (subtitleLit && !subtitleDynamic) {
+        if (subtitleAttr && !subtitleDynamic) {
             removeAttr(element, subtitleAttr);
             ctx.dirty = true;
             return { target: subtitleTruthy ? 'SubTitle' : 'Title' };
@@ -313,35 +317,55 @@ const SPECIALS = {
         if ((kind && kind.kind !== 'string') ||
             (align && align.kind !== 'string')) {
             addTodo(ctx, path, 'prop:kind', 'dynamic Field kind/align; map to the bestax `grouped` / `hasAddons` props by hand');
-            return {};
+            // This handler owns `multiline` end-to-end; without this, the mapping's
+            // fallback `multiline: { drop: true }` prop action would silently strip
+            // a dynamic `multiline` the moment kind/align themselves are dynamic.
+            return { handledProps: ['multiline'] };
         }
-        for (const attr of [kindAttr, alignAttr, multilineAttr]) {
+        // `multiline` is a value, not just presence: a falsy literal drops it
+        // (same as it being absent) and a dynamic value must survive on the
+        // element — so, unlike `kind`/`align`, it's only stripped once resolved.
+        const multilineResolved = multilineAttr
+            ? resolveBooleanish(multilineAttr)
+            : null;
+        const multilineDynamic = multilineResolved === 'expression';
+        const multilineTruthy = multilineResolved === 'truthy';
+        const kindOrAlignPresent = Boolean(kindAttr || alignAttr);
+        for (const attr of [kindAttr, alignAttr]) {
             if (attr)
                 removeAttr(element, attr);
         }
-        ctx.dirty = true;
+        if (multilineAttr && !multilineDynamic) {
+            removeAttr(element, multilineAttr);
+        }
+        if (kindOrAlignPresent || (multilineAttr && !multilineDynamic)) {
+            ctx.dirty = true;
+        }
         const alignValue = align && align.kind === 'string'
             ? { center: 'centered', right: 'right' }[align.value]
             : undefined;
+        if (multilineDynamic) {
+            addTodo(ctx, path, 'prop:multiline', '`multiline` has a dynamic value; resolve the bestax `grouped`/`hasAddons` combination by hand');
+        }
         if (kind && kind.kind === 'string' && kind.value === 'addons') {
             addAttr(element, makeAttr(ctx.j, 'hasAddons', alignValue));
-            if (multilineAttr) {
+            if (multilineTruthy) {
                 addTodo(ctx, path, 'prop:multiline', "`multiline` only combines with kind='group' in Bulma; dropped from this addons Field");
             }
-            return {};
+            return { handledProps: ['multiline'] };
         }
         // kind='group' (or a stray multiline/align without kind)
-        if (multilineAttr) {
+        if (multilineTruthy) {
             addAttr(element, makeAttr(ctx.j, 'grouped', 'multiline'));
             if (alignValue) {
                 addTodo(ctx, path, 'prop:align', 'bestax `grouped` takes one value; choose between multiline and ' +
                     alignValue);
             }
         }
-        else {
+        else if (kindOrAlignPresent) {
             addAttr(element, makeAttr(ctx.j, 'grouped', alignValue));
         }
-        return {};
+        return { handledProps: ['multiline'] };
     },
     /** Form.Label → plain <label className="label">. */
     'plain-label'(ctx, path, element) {
@@ -472,18 +496,14 @@ const SPECIALS = {
         const activeAttr = findAttr(element, 'active');
         let className;
         if (activeAttr) {
-            const literal = literalValueOf(activeAttr);
-            if (literal.kind === 'boolean' && literal.value === true) {
+            const resolved = resolveBooleanish(activeAttr);
+            if (resolved === 'truthy') {
                 className = 'is-active';
-                removeAttr(element, activeAttr);
-            }
-            else if (literal.kind === 'boolean') {
-                removeAttr(element, activeAttr);
             }
-            else {
+            else if (resolved === 'expression') {
                 addTodo(ctx, path, 'prop:active', "dynamic Panel.Tabs.Tab active; set className={active ? 'is-active' : undefined} by hand");
-                removeAttr(element, activeAttr);
             }
+            removeAttr(element, activeAttr);
         }
         className = mergeClassName(ctx, path, element, className, 'Panel.Tabs.Tab');
         const rest = stripModifierProps(ctx, path, attributesOf(element), 'Panel.Tabs.Tab');
@@ -497,10 +517,10 @@ const SPECIALS = {
         const activeAttr = findAttr(element, 'active');
         let liClass;
         if (activeAttr) {
-            const literal = literalValueOf(activeAttr);
-            if (literal.kind === 'boolean' && literal.value === true)
+            const resolved = resolveBooleanish(activeAttr);
+            if (resolved === 'truthy')
                 liClass = 'is-active';
-            else if (literal.kind !== 'boolean') {
+            else if (resolved === 'expression') {
                 addTodo(ctx, path, 'prop:active', "dynamic Breadcrumb.Item active; set the li className={active ? 'is-active' : undefined} by hand");
             }
             removeAttr(element, activeAttr);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bestax-migrate",
-  "version": "2.1.3",
+  "version": "2.1.4",
   "description": "Codemods for migrating existing React apps from other Bulma libraries to @allxsmith/bestax-bulma",
   "type": "module",
   "bin": {
@@ -51,7 +51,7 @@
     "react-dom": "^19.2.8",
     "ts-jest": "^29.4.12",
     "typescript": "^6.0.3",
-    "@allxsmith/bestax-bulma": "^5.11.2"
+    "@allxsmith/bestax-bulma": "^5.11.4"
   },
   "publishConfig": {
     "access": "public"