bestax-migrate 2.2.1 → 2.3.0

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

@@ -16,7 +16,7 @@
 import { MAPPING, UNIVERSAL_PROPS, resolveMapping } from './mapping.js';
 import { addTodo, attributesOf, jsxNameParts, renameElement, } from '../_shared/jsx-utils.js';
 import { applyPropAction, applyUniversalProps } from '../_shared/props.js';
-import { collectBoundNames, makeReserve, nameOf, prefersTabs, resolvesToBinding, } from '../_shared/imports.js';
+import { collectBoundNames, makeAliasRegistry, makeReserve, nameOf, prefersTabs, resolvesToBinding, } from '../_shared/imports.js';
 import { flattenResponsiveProps } from './responsive.js';
 import { RESPONSIVE_KINDS, runSpecial } from './specials.js';
 /* eslint-disable @typescript-eslint/no-explicit-any */
@@ -88,6 +88,21 @@ export default function transform(fileInfo, api, options = {}) {
             addTodo(ctx, path, 'imports', 'deep react-bulma-components import paths are a v3 pattern; move to v4 named root imports first');
         }
     });
+    // A barrel re-export reaches the library without an import declaration,
+    // so nothing above saw it; it cannot be rewritten (a compound bestax
+    // target has no single name to re-export) and must not vanish silently.
+    root
+        .find(j.ExportNamedDeclaration)
+        .filter(p => String(p.node.source?.value ?? '') === RBC)
+        .forEach(p => {
+        addTodo(ctx, p, 'imports', `re-exports from '${RBC}' were not migrated; import the components, migrate them, then re-export the bestax ones by hand`);
+    });
+    root
+        .find(j.ExportAllDeclaration)
+        .filter(p => String(p.node.source?.value ?? '') === RBC)
+        .forEach(p => {
+        addTodo(ctx, p, 'imports', `\`export * from '${RBC}'\` was not migrated; re-export the bestax components you need by hand`);
+    });
     // ---- 1a. Stylesheet imports (mode-driven) -----------------------------
     // `bestax` (default): everything converges on the recommended combined
     // bundle. `bulma`: plain Bulma v1 CSS plus the separate extras file.
@@ -150,68 +165,20 @@ export default function transform(fileInfo, api, options = {}) {
     // The scope each binding was collected in, so a reference can be checked
     // against the binding it actually resolves to rather than matched by name.
     const programScope = root.find(j.Program).paths()[0]?.scope;
-    // Aliases are keyed by the AST node that owns the declaring scope, not by
-    // bare name. Two functions can each destructure `Header` from a different
-    // component; a name-keyed map let the second overwrite the first, and both
-    // declarations were then pruned while both `<Header…>` references were left
-    // resolving to nothing.
-    //
-    // Resolution walks the reference's ANCESTORS rather than consulting
-    // `scope.lookup`, because by the time references are rewritten the alias
-    // pass has already pruned the declaration -- so the binding is gone from
-    // the scope table, and `path.scope` objects are not identity-stable across
-    // traversals either.
-    const aliasesByOwner = new Map();
-    const ownersFor = (name) => {
-        const owners = [];
-        for (const [owner, perScope] of aliasesByOwner) {
-            if (perScope.has(name))
-                owners.push(owner);
-        }
-        return owners;
-    };
-    const aliasAt = (name, at) => {
-        const owners = ownersFor(name);
-        if (owners.length === 0)
-            return undefined;
-        // Without a reference location there is nothing to walk, so the only
-        // owner is the best answer available. WITH one, always walk: an unrelated
-        // `Header` parameter in another function must not resolve to the alias
-        // merely because the alias is the only one by that name. A single-owner
-        // shortcut here rewrote such a parameter to `{ Header: Card.Header }`,
-        // which is not even valid syntax.
-        if (at === undefined) {
-            return owners.length === 1
-                ? aliasesByOwner.get(owners[0]).get(name)
-                : undefined;
-        }
-        let cursor = at;
-        while (cursor) {
-            if (cursor.node !== undefined) {
-                const hit = aliasesByOwner.get(cursor.node)?.get(name);
-                if (hit) {
-                    // The walk finds the nearest OWNER, but a binding can intervene
-                    // between the reference and that owner: `function F(Header)` under
-                    // a module-level `const { Header } = Card` shadows the alias, and
-                    // resolving through it rewrote the parameter itself to
-                    // `F(Card.Header)`. If the scope table still binds the name
-                    // somewhere other than the owner, that nearer binding wins. (After
-                    // the alias declaration is pruned the lookup returns nothing for
-                    // the alias itself, which is the case this must keep allowing.)
-                    const declaring = at.scope?.lookup(name)?.path?.node;
-                    if (declaring !== undefined &&
-                        declaring !== null &&
-                        declaring !== cursor.node) {
-                        return undefined;
-                    }
-                    return hit;
-                }
-            }
-            cursor = cursor.parent;
-        }
-        return undefined;
-    };
-    const anyAlias = (name) => ownersFor(name).length > 0;
+    const { register: registerAlias, aliasAt, anyAlias } = makeAliasRegistry();
+    /**
+     * Locals a re-export names. Pruning the declaration that binds one of them
+     * (`const { Header } = Card; export { Header };`) leaves the specifier
+     * pointing at nothing, so those declarations are kept: the value-reference
+     * pass rewrites their initialiser to the bestax component, and the export
+     * goes on re-exporting the same member.
+     */
+    const reExported = new Set();
+    root.find(j.ExportSpecifier).forEach(p => {
+        const local = p.node.local ?? p.node.exported;
+        if (local?.type === 'Identifier')
+            reExported.add(local.name);
+    });
     // ---- 1b. Resolve `const { Input, Field: F } = Form` destructuring -----
     root.find(j.VariableDeclarator).forEach(path => {
         const node = path.node;
@@ -234,13 +201,8 @@ export default function transform(fileInfo, api, options = {}) {
                 prop.value?.type === 'Identifier') {
                 const owner = path.scope
                     ?.path?.node;
-                let perScope = aliasesByOwner.get(owner);
-                if (!perScope) {
-                    perScope = new Map();
-                    aliasesByOwner.set(owner, perScope);
-                }
                 const target = [...basePath, prop.key.name];
-                perScope.set(prop.value.name, target);
+                registerAlias(owner, prop.value.name, target);
                 aliases.set(prop.value.name, target);
                 // Pruning the declaration is only safe if every use of the alias will
                 // actually be rewritten. For a name the table cannot map, the JSX is
@@ -261,7 +223,10 @@ export default function transform(fileInfo, api, options = {}) {
                 allResolved = false;
             }
         }
-        if (allResolved) {
+        const exportsAnAlias = node.id.properties.some((prop) => (prop.type === 'ObjectProperty' || prop.type === 'Property') &&
+            prop.value?.type === 'Identifier' &&
+            reExported.has(prop.value.name));
+        if (allResolved && !exportsAnAlias) {
             const declaration = path.parent;
             if (declaration?.node?.type === 'VariableDeclaration' &&
                 declaration.node.declarations.length === 1) {
@@ -377,6 +342,16 @@ export default function transform(fileInfo, api, options = {}) {
             bound.add(head);
     });
     ctx.reserve = makeReserve(ctx, bound);
+    // Names the passes actually asked for. `ctx.needed` is also seeded from an
+    // existing bestax import's specifiers below (so JSX reuses their locals),
+    // and that seeding alone must not turn a type-only specifier into a value
+    // import nobody needed.
+    const requested = new Set();
+    const baseReserve = ctx.reserve;
+    ctx.reserve = root => {
+        requested.add(root);
+        return baseReserve(root);
+    };
     // Merge with an existing bestax import: reuse its locals verbatim.
     // Only a declaration made entirely of NAMED specifiers can be merged into.
     // `import * as Bulma from '@allxsmith/bestax-bulma'` matches the source too,
@@ -386,13 +361,24 @@ export default function transform(fileInfo, api, options = {}) {
     const existingBestax = root
         .find(j.ImportDeclaration, { source: { value: BESTAX } })
         .paths()
-        .find(path => (path.node.specifiers ?? []).every((spec) => spec.type === 'ImportSpecifier'));
+        .find(path => 
+    // A type-only declaration cannot take a value specifier: merging a
+    // component into `import type { … }` erases it at runtime.
+    path.node.importKind !== 'type' &&
+        (path.node.specifiers ?? []).every((spec) => spec.type === 'ImportSpecifier'));
     const preExistingImports = new Set();
     if (existingBestax) {
         for (const spec of existingBestax.node.specifiers ?? []) {
             if (spec.type === 'ImportSpecifier' && spec.local) {
+                // The local name is reused either way, so the JSX never needs an
+                // alias. But an inline `type` specifier (`import { type Box }`) is
+                // not a value binding: it is not counted as already imported, so the
+                // component is written as a value below — onto this specifier, which
+                // keeps its name, rather than as a duplicate.
                 ctx.needed.set(nameOf(spec.imported), nameOf(spec.local));
-                preExistingImports.add(nameOf(spec.imported));
+                if (spec.importKind !== 'type') {
+                    preExistingImports.add(nameOf(spec.imported));
+                }
             }
         }
     }
@@ -508,6 +494,10 @@ export default function transform(fileInfo, api, options = {}) {
             parentType === 'JSXMemberExpression') {
             return;
         }
+        // A TS qualified name (`RBC.ButtonProps` in a type position) is erased at
+        // runtime, so it is not a value use of the namespace at all.
+        if (parentType === 'TSQualifiedName')
+            return;
         // A namespace binding in a real value position. `RBC.Box` is still a
         // mappable component reference, so fall through to the member-expression
         // branch below with an empty prefix (the namespace itself is not part of
@@ -520,6 +510,46 @@ export default function transform(fileInfo, api, options = {}) {
                 parentNode.object === path.node &&
                 !parentNode.computed)) {
             namespaceStillReferenced = true;
+            addTodo(ctx, path, 'value-reference', `\`${name}\` is the library's namespace import used as a plain value (destructured, or passed around); nothing reached through it was migrated — reference its components as \`${name}.Component\` or import them by name, then re-run`);
+            return;
+        }
+        // Members, keys and signatures name a thing; they do not reference the
+        // import. An export specifier does, and is handled on its own: a flat
+        // target keeps the public name (\`export { SubTitle as Subtitle }\`), a
+        // dotted one cannot be re-exported under a member name and is flagged.
+        if (parentType === 'ExportSpecifier') {
+            if (parentNode.local !== path.node)
+                return;
+            const exportedName = nameOf(parentNode.exported ?? parentNode.local);
+            const mapping = aliasPath ? resolveMapping(aliasPath) : MAPPING[imported];
+            if (mapping &&
+                mapping.status !== 'todo' &&
+                mapping.target &&
+                !mapping.target.includes('.')) {
+                const local = ctx.reserve(mapping.target);
+                // Replace the node: recast reprints a specifier whose identifiers were
+                // swapped in place in its original shorthand form, dropping the `as`.
+                path.parent.replace(j.exportSpecifier.from({
+                    local: j.identifier(local),
+                    exported: j.identifier(exportedName),
+                }));
+                ctx.dirty = true;
+                return;
+            }
+            ctx.retained.add(imported);
+            addTodo(ctx, path, 'value-reference', `\`${name}\` is re-exported; ${mapping?.target ? `its bestax counterpart \`${mapping.target}\` is a member of \`${mapping.target.split('.')[0]}\` and cannot be re-exported under this name` : 'it has no bestax counterpart to re-export'} — migrate the consumers by hand`);
+            return;
+        }
+        if (parentType === 'TSEnumMember' ||
+            ((parentType === 'ObjectMethod' ||
+                parentType === 'ClassMethod' ||
+                parentType === 'ClassProperty' ||
+                parentType === 'PropertyDefinition' ||
+                parentType === 'MethodDefinition' ||
+                parentType === 'TSPropertySignature' ||
+                parentType === 'TSMethodSignature') &&
+                parentNode.key === path.node &&
+                !parentNode.computed)) {
             return;
         }
         // Non-reference positions: member property names and object keys.
@@ -712,15 +742,61 @@ export default function transform(fileInfo, api, options = {}) {
     if (imports.size > 0) {
         const retainedNames = [...ctx.retained].sort((a, b) => a.localeCompare(b));
         const freshNames = [...ctx.needed.entries()]
-            .filter(([imported]) => !preExistingImports.has(imported))
+            .filter(([imported]) => requested.has(imported) && !preExistingImports.has(imported))
             .sort((a, b) => a[0].localeCompare(b[0]));
         const bestaxImport = freshNames.length > 0
             ? j.importDeclaration(freshNames.map(([imported, local]) => j.importSpecifier(j.identifier(imported), j.identifier(local))), j.stringLiteral(BESTAX))
             : null;
+        /**
+    
+         * A pruned declaration takes its comments with it — a licence header, an
+    
+         * eslint directive. Hand them to what replaces it, or to the next statement.
+    
+         */
+        const carryComments = (node, importPath) => {
+            const comments = node.comments ?? [];
+            if (comments.length === 0)
+                return;
+            const body = importPath.parent?.node?.body ?? [];
+            const index = body.indexOf(node);
+            // The fresh specifiers are merged INTO an existing bestax import when there
+            // is one, and the `bestaxImport` node is then discarded — so the comments
+            // have to follow the declaration that survives, not the one that does not.
+            const carrier = (existingBestax ? existingBestax.node : bestaxImport) ??
+                (index >= 0 ? body[index + 1] : undefined);
+            if (carrier) {
+                carrier.comments = [...comments, ...(carrier.comments ?? [])];
+            }
+            else if (index > 0) {
+                // The import was the last statement: the comments stay where they
+                // were, after what precedes it.
+                const previous = body[index - 1];
+                previous.comments = [
+                    ...(previous.comments ?? []),
+                    ...comments.map((c) => ({
+                        ...c,
+                        leading: false,
+                        trailing: true,
+                    })),
+                ];
+            }
+            else {
+                // The import was the only statement: the comments become the file's.
+                const program = importPath.parent?.node;
+                if (program)
+                    program.comments = [...comments, ...(program.comments ?? [])];
+            }
+            node.comments = [];
+        };
         let inserted = false;
         // A retained RBC specifier must never collide with a bestax import local
         // (possible when one component is both JSX-migrated and value-retained).
-        const bestaxLocals = new Set(ctx.needed.values());
+        // Only a VALUE local can collide at runtime: a type-only specifier the
+        // seeding above recorded, and nothing promoted, binds no value.
+        const bestaxLocals = new Set([...ctx.needed.entries()]
+            .filter(([imported]) => requested.has(imported) || preExistingImports.has(imported))
+            .map(([, local]) => local));
         for (const path of rbcImportPaths) {
             const node = path.node;
             const keepSpecifiers = (node.specifiers ?? []).filter((spec) => {
@@ -736,10 +812,23 @@ export default function transform(fileInfo, api, options = {}) {
             });
             if (!inserted) {
                 if (existingBestax && bestaxImport) {
-                    existingBestax.node.specifiers = [
-                        ...(existingBestax.node.specifiers ?? []),
-                        ...bestaxImport.specifiers,
-                    ];
+                    const current = existingBestax.node.specifiers ?? [];
+                    const appended = [];
+                    for (const fresh of bestaxImport.specifiers) {
+                        // A type-only specifier for the same name becomes the value
+                        // import (a value import carries the type too); appending a
+                        // second `Box` beside `type Box` would be a duplicate identifier.
+                        const typeOnly = current.find((spec) => spec.type === 'ImportSpecifier' &&
+                            spec.importKind === 'type' &&
+                            nameOf(spec.imported) === nameOf(fresh.imported));
+                        if (typeOnly) {
+                            typeOnly.importKind = null;
+                        }
+                        else {
+                            appended.push(fresh);
+                        }
+                    }
+                    existingBestax.node.specifiers = [...current, ...appended];
                 }
                 else if (bestaxImport) {
                     path.insertBefore(bestaxImport);
@@ -755,6 +844,7 @@ export default function transform(fileInfo, api, options = {}) {
                 }
             }
             else {
+                carryComments(node, path);
                 path.prune();
             }
             ctx.dirty = true;

package/dist/sources/registry.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"registry.d.ts","sourceRoot":"","sources":["../../src/sources/registry.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAInD,eAAO,MAAM,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,eAAe,CAGnD,CAAC;AAEF,wBAAgB,SAAS,CAAC,IAAI,EAAE,MAAM,GAAG,eAAe,GAAG,SAAS,CAEnE;AAED,wBAAgB,WAAW,IAAI,MAAM,EAAE,CAEtC"}
+{"version":3,"file":"registry.d.ts","sourceRoot":"","sources":["../../src/sources/registry.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAKnD,eAAO,MAAM,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,eAAe,CAInD,CAAC;AAEF,wBAAgB,SAAS,CAAC,IAAI,EAAE,MAAM,GAAG,eAAe,GAAG,SAAS,CAEnE;AAED,wBAAgB,WAAW,IAAI,MAAM,EAAE,CAEtC"}

package/dist/sources/registry.js

@@ -4,9 +4,11 @@
  */
 import { reactBulmaComponents } from './react-bulma-components/index.js';
 import { rbx } from './rbx/index.js';
+import { bloomer } from './bloomer/index.js';
 export const SOURCES = {
     [reactBulmaComponents.name]: reactBulmaComponents,
     [rbx.name]: rbx,
+    [bloomer.name]: bloomer,
 };
 export function getSource(name) {
     return SOURCES[name];

package/dist/types.d.ts

@@ -38,6 +38,17 @@ export interface PropAction {
         name: string;
         value?: string;
     };
+    /**
+     * Replace a boolean (valueless / `true`) attribute with this Bulma class,
+     * merged into `className` — for a modifier bestax has no prop for but
+     * Bulma v1 still ships. A `false` value is dropped; a dynamic one is a TODO.
+     */
+    toClass?: string;
+    /**
+     * Replace a literal string attribute with `prefix + value` merged into
+     * `className` (`isSize="large"` → `is-large`). A dynamic value is a TODO.
+     */
+    toClassPrefix?: string;
     /** Remove the attribute entirely. */
     drop?: boolean;
     /** Leave the attribute as-is and attach a TODO with this hint. */

package/dist/types.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,KAAK,EAAE,GAAG,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AAEjD;;;;;;GAMG;AACH,MAAM,MAAM,aAAa,GAAG,QAAQ,GAAG,SAAS,GAAG,MAAM,CAAC;AAE1D,0DAA0D;AAC1D,MAAM,WAAW,UAAU;IACzB,+CAA+C;IAC/C,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,oEAAoE;IACpE,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAClC;;;;;OAKG;IACH,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,sEAAsE;IACtE,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB;;;OAGG;IACH,aAAa,CAAC,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,KAAK,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;IACjD,qCAAqC;IACrC,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,kEAAkE;IAClE,IAAI,CAAC,EAAE,MAAM,CAAC;IACd;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;CACpC;AAED,4EAA4E;AAC5E,MAAM,WAAW,gBAAgB;IAC/B,MAAM,EAAE,aAAa,CAAC;IACtB;;;;OAIG;IACH,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,iEAAiE;IACjE,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,+EAA+E;IAC/E,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,yEAAyE;IACzE,KAAK,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC;IACnC,kEAAkE;IAClE,IAAI,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,gBAAgB,CAAC,CAAC;CACzC;AAED,qEAAqE;AACrE,MAAM,WAAW,SAAS;IACxB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,2CAA2C;AAC3C,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,OAAO,CAAC;IACjB,KAAK,EAAE,SAAS,EAAE,CAAC;CACpB;AAED,sEAAsE;AACtE,MAAM,WAAW,aAAa;IAC5B,GAAG,CAAC,KAAK,EAAE,SAAS,GAAG,IAAI,CAAC;CAC7B;AAED;;;;;;GAMG;AACH,MAAM,MAAM,OAAO,GAAG,QAAQ,GAAG,OAAO,GAAG,MAAM,CAAC;AAElD,uEAAuE;AACvE,MAAM,WAAW,gBAAgB;IAC/B,SAAS,CAAC,EAAE,aAAa,CAAC;IAC1B,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB;;;;;OAKG;IACH,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC;CACxB;AAED,uDAAuD;AACvD,MAAM,WAAW,WAAW;IAC1B,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,uEAAuE;IACvE,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B;;;;;;OAMG;IACH,mBAAmB,CAAC,EAAE,OAAO,CAAC;CAC/B;AAED,mDAAmD;AACnD,MAAM,MAAM,SAAS,GAAG,CACtB,QAAQ,EAAE,QAAQ,EAClB,GAAG,EAAE,GAAG,EACR,OAAO,EAAE,gBAAgB,KACtB,MAAM,GAAG,SAAS,CAAC;AAExB,+DAA+D;AAC/D,MAAM,MAAM,eAAe,GAAG,CAC5B,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,MAAM,EACd,SAAS,EAAE,aAAa,GAAG,SAAS,EACpC,OAAO,EAAE,gBAAgB,KACtB,MAAM,GAAG,IAAI,CAAC;AAEnB,kEAAkE;AAClE,MAAM,MAAM,kBAAkB,GAAG,CAC/B,QAAQ,EAAE,MAAM,EAChB,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC5B,SAAS,EAAE,aAAa,GAAG,SAAS,EACpC,OAAO,EAAE,WAAW,KACjB,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,CAAC;AAEpC,kDAAkD;AAClD,MAAM,WAAW,eAAe;IAC9B,+CAA+C;IAC/C,IAAI,EAAE,MAAM,CAAC;IACb,kDAAkD;IAClD,KAAK,EAAE,MAAM,CAAC;IACd,SAAS,EAAE,SAAS,CAAC;IACrB,eAAe,CAAC,EAAE,eAAe,CAAC;IAClC,kBAAkB,CAAC,EAAE,kBAAkB,CAAC;CACzC"}
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,KAAK,EAAE,GAAG,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AAEjD;;;;;;GAMG;AACH,MAAM,MAAM,aAAa,GAAG,QAAQ,GAAG,SAAS,GAAG,MAAM,CAAC;AAE1D,0DAA0D;AAC1D,MAAM,WAAW,UAAU;IACzB,+CAA+C;IAC/C,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,oEAAoE;IACpE,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAClC;;;;;OAKG;IACH,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,sEAAsE;IACtE,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB;;;OAGG;IACH,aAAa,CAAC,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,KAAK,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;IACjD;;;;OAIG;IACH,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB;;;OAGG;IACH,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,qCAAqC;IACrC,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,kEAAkE;IAClE,IAAI,CAAC,EAAE,MAAM,CAAC;IACd;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;CACpC;AAED,4EAA4E;AAC5E,MAAM,WAAW,gBAAgB;IAC/B,MAAM,EAAE,aAAa,CAAC;IACtB;;;;OAIG;IACH,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,iEAAiE;IACjE,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,+EAA+E;IAC/E,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,yEAAyE;IACzE,KAAK,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC;IACnC,kEAAkE;IAClE,IAAI,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,gBAAgB,CAAC,CAAC;CACzC;AAED,qEAAqE;AACrE,MAAM,WAAW,SAAS;IACxB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,2CAA2C;AAC3C,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,OAAO,CAAC;IACjB,KAAK,EAAE,SAAS,EAAE,CAAC;CACpB;AAED,sEAAsE;AACtE,MAAM,WAAW,aAAa;IAC5B,GAAG,CAAC,KAAK,EAAE,SAAS,GAAG,IAAI,CAAC;CAC7B;AAED;;;;;;GAMG;AACH,MAAM,MAAM,OAAO,GAAG,QAAQ,GAAG,OAAO,GAAG,MAAM,CAAC;AAElD,uEAAuE;AACvE,MAAM,WAAW,gBAAgB;IAC/B,SAAS,CAAC,EAAE,aAAa,CAAC;IAC1B,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB;;;;;OAKG;IACH,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC;CACxB;AAED,uDAAuD;AACvD,MAAM,WAAW,WAAW;IAC1B,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,uEAAuE;IACvE,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B;;;;;;OAMG;IACH,mBAAmB,CAAC,EAAE,OAAO,CAAC;CAC/B;AAED,mDAAmD;AACnD,MAAM,MAAM,SAAS,GAAG,CACtB,QAAQ,EAAE,QAAQ,EAClB,GAAG,EAAE,GAAG,EACR,OAAO,EAAE,gBAAgB,KACtB,MAAM,GAAG,SAAS,CAAC;AAExB,+DAA+D;AAC/D,MAAM,MAAM,eAAe,GAAG,CAC5B,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,MAAM,EACd,SAAS,EAAE,aAAa,GAAG,SAAS,EACpC,OAAO,EAAE,gBAAgB,KACtB,MAAM,GAAG,IAAI,CAAC;AAEnB,kEAAkE;AAClE,MAAM,MAAM,kBAAkB,GAAG,CAC/B,QAAQ,EAAE,MAAM,EAChB,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC5B,SAAS,EAAE,aAAa,GAAG,SAAS,EACpC,OAAO,EAAE,WAAW,KACjB,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,CAAC;AAEpC,kDAAkD;AAClD,MAAM,WAAW,eAAe;IAC9B,+CAA+C;IAC/C,IAAI,EAAE,MAAM,CAAC;IACb,kDAAkD;IAClD,KAAK,EAAE,MAAM,CAAC;IACd,SAAS,EAAE,SAAS,CAAC;IACrB,eAAe,CAAC,EAAE,eAAe,CAAC;IAClC,kBAAkB,CAAC,EAAE,kBAAkB,CAAC;CACzC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bestax-migrate",
-  "version": "2.2.1",
+  "version": "2.3.0",
   "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.12.0"
+    "@allxsmith/bestax-bulma": "^5.15.0"
   },
   "publishConfig": {
     "access": "public"
@@ -66,10 +66,11 @@
     "typecheck": "tsc --noEmit",
     "validate:corpus": "pnpm run build && node scripts/validate-corpus.mjs",
     "validate:corpus:rbx": "pnpm run build && node scripts/validate-corpus-rbx.mjs",
+    "validate:corpus:bloomer": "pnpm run build && node scripts/validate-corpus-bloomer.mjs",
     "format": "prettier --write \"src/**/*.{ts,tsx}\" \"e2e/**/*.ts\"",
     "format:check": "prettier --check \"src/**/*.{ts,tsx}\" \"e2e/**/*.ts\"",
     "clean": "rimraf dist",
     "release": "npx semantic-release"
   },
-  "readme": "# bestax-migrate\n\n[![npm version](https://img.shields.io/npm/v/bestax-migrate.svg)](https://www.npmjs.com/package/bestax-migrate)\n[![npm downloads](https://img.shields.io/npm/dm/bestax-migrate.svg)](https://www.npmjs.com/package/bestax-migrate)\n[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)\n[![Socket Badge](https://socket.dev/api/badge/npm/package/bestax-migrate)](https://socket.dev/npm/package/bestax-migrate/overview)\n[![OpenSSF Scorecard](https://api.scorecard.dev/projects/github.com/allxsmith/bestax/badge)](https://scorecard.dev/viewer/?uri=github.com/allxsmith/bestax)\n[![OpenSSF Best Practices](https://www.bestpractices.dev/projects/14361/badge)](https://www.bestpractices.dev/projects/14361)\n[![npm provenance](https://img.shields.io/badge/npm-provenance-3fb950.svg)](https://www.npmjs.com/package/bestax-migrate#provenance)\n[![Security policy](https://img.shields.io/badge/security-policy-blue.svg)](https://github.com/allxsmith/bestax/blob/main/SECURITY.md)\n\nCodemods that migrate existing React apps to [`@allxsmith/bestax-bulma`](https://www.npmjs.com/package/@allxsmith/bestax-bulma) — the actively maintained React component library for **Bulma v1**.\n\nCurrently supported source libraries:\n\n| Source                                                                           | Status                                                              |\n| -------------------------------------------------------------------------------- | ------------------------------------------------------------------- |\n| [`react-bulma-components`](https://github.com/couds/react-bulma-components) (v4) | ✅ All 32 components mapped (a few patterns are flagged as TODOs)   |\n| [`rbx`](https://github.com/dfee/rbx) (v2)                                        | ✅ Full export surface mapped (a few patterns are flagged as TODOs) |\n\nPass the source library as the first argument: `bestax-migrate <source> <paths…>`.\n\n## Requirements\n\n**Node.js 22 or newer.** Node 18 and 20 are both past end-of-life. On an older runtime the CLI\nexits immediately with an explicit upgrade message.\n\nThis applies only to the Node version the codemod itself runs on. It places no requirement on\nthe app being migrated — the source is read as text and is never executed.\n\n## Usage\n\n```bash\n# Preview what would change (no writes)\npnpm dlx bestax-migrate react-bulma-components src/ --dry\n# …or, for an rbx app\npnpm dlx bestax-migrate rbx src/ --dry\n\n# Apply the migration\npnpm dlx bestax-migrate react-bulma-components src/\n```\n\nnpm and yarn work too: `npx bestax-migrate …` / `yarn dlx bestax-migrate …`.\n\nThe codemod uses [jscodeshift](https://github.com/facebook/jscodeshift) to:\n\n- rewrite the source library's imports to `@allxsmith/bestax-bulma` (including destructuring and namespace imports)\n- rename components and compound sub-components (`Form.Textarea` → `TextArea`, `Card.Footer.Item` → `Card.FooterItem`, `Hero.Footer` → `Hero.Foot`, …)\n- convert props (`renderAs` → `as`, `loading` → `isLoading`, numeric spacing/text sizes → string unions, `textAlign=\"center\"` → `textAlign=\"centered\"`, …)\n- flatten responsive breakpoint objects (`mobile={{ size: 4 }}` → `sizeMobile={4}`)\n- restructure patterns bestax models differently (`Table.Container`, Navbar dropdowns, `Form.Help`, icon-font children → `<Icon name=…>`)\n- migrate stylesheets: CSS imports converge on `@allxsmith/bestax-bulma/bestax.css`, and SCSS files move from Bulma 0.9's `@import` + `$var !default` overrides to `@use 'bulma/sass' with (…)` plus `@use '@allxsmith/bestax-bulma/scss/extras'`\n- update `package.json`: remove the source library, add `@allxsmith/bestax-bulma`, raise a declared pre-1.0 `bulma` to `^1` (adding it only when your sources import `bulma/…` directly — otherwise it arrives transitively), and swap the dead `node-sass` for dart `sass` (no install is ever run). Migrating **rbx** is what frees an app from Bulma 0.7.5: rbx pinned it as a _direct_ dependency, so the app could not choose its own Bulma version at all. The four Bulma extensions rbx also pulled in (`bulma-badge`, `bulma-divider`, `bulma-pageloader`, `bulma-tooltip`) are **reported rather than removed** — bestax replaces all four, but a manifest entry is a deliberate declaration and your own Sass may import them\n\nAnything without a safe automatic conversion is left in place with a `// TODO(bestax-migrate): …` comment, and the run ends with a report of every TODO by file and line. TODOs are expected output, not errors — resolve them with the migration guide ([react-bulma-components](https://bestax.io/docs/guides/getting-started/migration/react-bulma-components), [rbx](https://bestax.io/docs/guides/getting-started/migration/rbx)), or let the [`bestax-migrate` Agent Skill](https://bestax.io/docs/skills/intro) walk them for you:\n\n```bash\nnpx skills add https://github.com/allxsmith/bestax --skill bestax-migrate\n```\n\n## Options\n\n| Flag                 | Description                                                    |\n| -------------------- | -------------------------------------------------------------- |\n| `--dry`, `-d`        | Report what would change without writing files                 |\n| `--print`, `-p`      | Print transformed sources to stdout                            |\n| `--extensions`, `-e` | File extensions to include (default `js,jsx,ts,tsx,scss,sass`) |\n| `--css <mode>`       | Stylesheet target: `bestax` (default), `bulma`, or `keep`      |\n| `--no-deps`          | Skip updating package.json dependencies                        |\n\n## After the codemod\n\n1. Run your package manager's install (the codemod rewrote `package.json` but never installs)\n2. Search for `TODO(bestax-migrate)` and resolve each comment\n3. Typecheck/build and review the rendered app\n\nFull walkthrough: [react-bulma-components migration guide](https://bestax.io/docs/guides/getting-started/migration/react-bulma-components) · [rbx migration guide](https://bestax.io/docs/guides/getting-started/migration/rbx).\n\n## Hardened by default\n\nA codemod rewrites your source in place, so how it is built and published matters:\n\n- **Signed provenance** — every release carries a sigstore attestation linking the tarball to the exact commit and CI run that built it. Check the **Provenance** section on the [npm page](https://www.npmjs.com/package/bestax-migrate#provenance), or run `npm audit signatures`.\n- **npm OIDC trusted publishing** — short-lived, per-run credentials; no long-lived `NPM_TOKEN` exists to be stolen. Release commits and tags are GPG-signed.\n- **Socket.dev scans every PR** for malware, install scripts, obfuscated code, and privilege escalation before it can reach `main`.\n- **The libraries this tool migrates away from are never installed here** — source fixtures are read as text only, so no unmaintained third-party package enters the dependency tree.\n- **Dependencies are a deliberate act** — install scripts are blocked unless individually allow-listed, freshly published versions are refused for 3 days, and CI installs only what the reviewed lockfile resolves.\n- **Every GitHub Action is pinned to a full commit SHA**, and CodeQL, Dependency Review, and Dependabot run continuously alongside a high-severity `pnpm audit` gate.\n- **Layered AI review before merge** — [CodeRabbit](https://coderabbit.ai) plus an independent adversarial Claude review (a different model from the one writing AI-authored changes), on top of required green CI, an approving review, and a human merge.\n\nFull detail: [`SECURITY.md`](https://github.com/allxsmith/bestax/blob/main/SECURITY.md) · [Security guide](https://bestax.io/docs/guides/security)\n\n## Telemetry\n\n`bestax-migrate` can send one **anonymous** usage event after a successful\nrun — only if you opt in when asked (once, on an interactive terminal). The\nevent is just the run's shape — source library, CSS mode, the dry/deps flags, a\ncapped changed-file count, and per-rule TODO counts — plus the CLI version,\nNode major version, and OS platform name. Never file paths, file contents,\ncode, names, IPs, or machine IDs, and no identifier exists that could link two\nevents together. Opt out any time with `--no-telemetry`,\n`BESTAX_TELEMETRY=0`, or `DO_NOT_TRACK=1`.\n\nFull disclosure of every field and control: [Telemetry guide](https://bestax.io/docs/guides/telemetry)\n\n## License\n\nSource code licensed MIT\n"
+  "readme": "# bestax-migrate\n\n[![npm version](https://img.shields.io/npm/v/bestax-migrate.svg)](https://www.npmjs.com/package/bestax-migrate)\n[![npm downloads](https://img.shields.io/npm/dm/bestax-migrate.svg)](https://www.npmjs.com/package/bestax-migrate)\n[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)\n[![Socket Badge](https://socket.dev/api/badge/npm/package/bestax-migrate)](https://socket.dev/npm/package/bestax-migrate/overview)\n[![OpenSSF Scorecard](https://api.scorecard.dev/projects/github.com/allxsmith/bestax/badge)](https://scorecard.dev/viewer/?uri=github.com/allxsmith/bestax)\n[![OpenSSF Best Practices](https://www.bestpractices.dev/projects/14361/badge)](https://www.bestpractices.dev/projects/14361)\n[![npm provenance](https://img.shields.io/badge/npm-provenance-3fb950.svg)](https://www.npmjs.com/package/bestax-migrate#provenance)\n[![Security policy](https://img.shields.io/badge/security-policy-blue.svg)](https://github.com/allxsmith/bestax/blob/main/SECURITY.md)\n\nCodemods that migrate existing React apps to [`@allxsmith/bestax-bulma`](https://www.npmjs.com/package/@allxsmith/bestax-bulma) — the actively maintained React component library for **Bulma v1**.\n\nCurrently supported source libraries:\n\n| Source                                                                           | Status                                                                             |\n| -------------------------------------------------------------------------------- | ---------------------------------------------------------------------------------- |\n| [`react-bulma-components`](https://github.com/couds/react-bulma-components) (v4) | ✅ All 32 components mapped (a few patterns are flagged as TODOs)                  |\n| [`rbx`](https://github.com/dfee/rbx) (v2)                                        | ✅ Full export surface mapped (a few patterns are flagged as TODOs)                |\n| [`bloomer`](https://github.com/AlgusDark/bloomer) (0.6)                          | ✅ All 108 exports mapped (Font Awesome 4 icons and Bulma 0.4's `Nav` are flagged) |\n\nPass the source library as the first argument: `bestax-migrate <source> <paths…>`.\n\n## Requirements\n\n**Node.js 22 or newer.** Node 18 and 20 are both past end-of-life. On an older runtime the CLI\nexits immediately with an explicit upgrade message.\n\nThis applies only to the Node version the codemod itself runs on. It places no requirement on\nthe app being migrated — the source is read as text and is never executed.\n\n## Usage\n\n```bash\n# Preview what would change (no writes)\npnpm dlx bestax-migrate react-bulma-components src/ --dry\n# …or, for an rbx or bloomer app\npnpm dlx bestax-migrate rbx src/ --dry\npnpm dlx bestax-migrate bloomer src/ --dry\n\n# Apply the migration\npnpm dlx bestax-migrate react-bulma-components src/\n```\n\nnpm and yarn work too: `npx bestax-migrate …` / `yarn dlx bestax-migrate …`.\n\nThe codemod uses [jscodeshift](https://github.com/facebook/jscodeshift) to:\n\n- rewrite the source library's imports to `@allxsmith/bestax-bulma` (including destructuring and namespace imports)\n- rename components and compound sub-components (`Form.Textarea` → `TextArea`, `Card.Footer.Item` → `Card.FooterItem`, `Hero.Footer` → `Hero.Foot`, …) — and, for bloomer, its flat names onto bestax's compounds (`CardHeaderTitle` → `Card.Header.Title`)\n- convert props (`renderAs` → `as`, `loading` → `isLoading`, numeric spacing/text sizes → string unions, `textAlign=\"center\"` → `textAlign=\"centered\"`, …)\n- flatten responsive breakpoint objects (`mobile={{ size: 4 }}` → `sizeMobile={4}`) and bloomer's `isDisplay`/`isHidden` strings, arrays and objects (`isHidden=\"touch\"` → `visibilityTouch=\"hidden\"`)\n- restructure patterns bestax models differently (`Table.Container`, Navbar dropdowns, `Form.Help`, icon-font children → `<Icon name=…>`)\n- migrate stylesheets: CSS imports converge on `@allxsmith/bestax-bulma/bestax.css`, and SCSS files move from Bulma 0.9's `@import` + `$var !default` overrides to `@use 'bulma/sass' with (…)` plus `@use '@allxsmith/bestax-bulma/scss/extras'`\n- update `package.json`: remove the source library, add `@allxsmith/bestax-bulma`, raise a declared pre-1.0 `bulma` to `^1` (adding it only when your sources import `bulma/…` directly — otherwise it arrives transitively), and swap the dead `node-sass` for dart `sass` (no install is ever run). Migrating **rbx** is what frees an app from Bulma 0.7.5: rbx pinned it as a _direct_ dependency, so the app could not choose its own Bulma version at all. The four Bulma extensions rbx also pulled in (`bulma-badge`, `bulma-divider`, `bulma-pageloader`, `bulma-tooltip`) are **reported rather than removed** — bestax replaces all four, but a manifest entry is a deliberate declaration and your own Sass may import them. A **bloomer** app declared its own Bulma 0.6, which is what gets bumped\n\nAnything without a safe automatic conversion is left in place with a `// TODO(bestax-migrate): …` comment, and the run ends with a report of every TODO by file and line. TODOs are expected output, not errors — resolve them with the migration guide ([react-bulma-components](https://bestax.io/docs/guides/getting-started/migration/react-bulma-components), [rbx](https://bestax.io/docs/guides/getting-started/migration/rbx), [bloomer](https://bestax.io/docs/guides/getting-started/migration/bloomer)), or let the [`bestax-migrate` Agent Skill](https://bestax.io/docs/skills/intro) walk them for you:\n\n```bash\nnpx skills add https://github.com/allxsmith/bestax --skill bestax-migrate\n```\n\n## Options\n\n| Flag                 | Description                                                    |\n| -------------------- | -------------------------------------------------------------- |\n| `--dry`, `-d`        | Report what would change without writing files                 |\n| `--print`, `-p`      | Print transformed sources to stdout                            |\n| `--extensions`, `-e` | File extensions to include (default `js,jsx,ts,tsx,scss,sass`) |\n| `--css <mode>`       | Stylesheet target: `bestax` (default), `bulma`, or `keep`      |\n| `--no-deps`          | Skip updating package.json dependencies                        |\n\n## After the codemod\n\n1. Run your package manager's install (the codemod rewrote `package.json` but never installs)\n2. Search for `TODO(bestax-migrate)` and resolve each comment\n3. Typecheck/build and review the rendered app\n\nFull walkthrough: [react-bulma-components migration guide](https://bestax.io/docs/guides/getting-started/migration/react-bulma-components) · [rbx migration guide](https://bestax.io/docs/guides/getting-started/migration/rbx) · [bloomer migration guide](https://bestax.io/docs/guides/getting-started/migration/bloomer).\n\n## Hardened by default\n\nA codemod rewrites your source in place, so how it is built and published matters:\n\n- **Signed provenance** — every release carries a sigstore attestation linking the tarball to the exact commit and CI run that built it. Check the **Provenance** section on the [npm page](https://www.npmjs.com/package/bestax-migrate#provenance), or run `npm audit signatures`.\n- **npm OIDC trusted publishing** — short-lived, per-run credentials; no long-lived `NPM_TOKEN` exists to be stolen. Release commits and tags are GPG-signed.\n- **Socket.dev scans every PR** for malware, install scripts, obfuscated code, and privilege escalation before it can reach `main`.\n- **The libraries this tool migrates away from are never installed here** — source fixtures are read as text only, so no unmaintained third-party package enters the dependency tree.\n- **Dependencies are a deliberate act** — install scripts are blocked unless individually allow-listed, freshly published versions are refused for 3 days, and CI installs only what the reviewed lockfile resolves.\n- **Every GitHub Action is pinned to a full commit SHA**, and CodeQL, Dependency Review, and Dependabot run continuously alongside a high-severity `pnpm audit` gate.\n- **Layered AI review before merge** — [CodeRabbit](https://coderabbit.ai) plus an independent adversarial Claude review (a different model from the one writing AI-authored changes), on top of required green CI, an approving review, and a human merge.\n\nFull detail: [`SECURITY.md`](https://github.com/allxsmith/bestax/blob/main/SECURITY.md) · [Security guide](https://bestax.io/docs/guides/security)\n\n## Telemetry\n\n`bestax-migrate` can send one **anonymous** usage event after a successful\nrun — only if you opt in when asked (once, on an interactive terminal). The\nevent is just the run's shape — source library, CSS mode, the dry/deps flags, a\ncapped changed-file count, and per-rule TODO counts — plus the CLI version,\nNode major version, and OS platform name. Never file paths, file contents,\ncode, names, IPs, or machine IDs, and no identifier exists that could link two\nevents together. Opt out any time with `--no-telemetry`,\n`BESTAX_TELEMETRY=0`, or `DO_NOT_TRACK=1`.\n\nFull disclosure of every field and control: [Telemetry guide](https://bestax.io/docs/guides/telemetry)\n\n## License\n\nSource code licensed MIT\n"
 }