@astryxdesign/cli 0.1.8-canary.0c4d8d7 → 0.1.8-canary.0db5dfd

package/{src/codemods → codemods}/transforms/v0.1.0/migrate-xds-css-surfaces.mjs

@@ -9,19 +9,18 @@
  * consumer CSS that targeted those surfaces is now broken.
  *
  * This transform performs precise, targeted string replacements on CSS/SCSS
- * files for the documented surfaces only:
+ * files for the documented surfaces only (selectors, layer names, and package
+ * stylesheet imports):
  *
- *   .xds-*                  (class-name prefix in selectors — matched only
- *                            after a `.` so it is a class selector)
- *   [data-xds-theme         (theme attribute selector)
- *   [data-xds-theme-prose   (prose-theme attribute selector)
- *   [data-xds-media         (media attribute selector)
- *   @layer xds-theme        (cascade layer name — incl. comma-separated
- *   @layer xds-base          `@layer a, b;` statement lists and nested blocks)
- *   @import '@xds/...'      (package stylesheet imports — scope rewrite, plus
- *                            the @xds/core/xds.css -> @astryxdesign/core/
- *                            astryx.css file rename and the theme-default /
- *                            theme-daily -> theme-neutral collapse)
+ * - `.xds-*` class-name prefix in selectors (matched only after a `.` so it is
+ *   a class selector).
+ * - `[data-xds-theme`, `[data-xds-theme-prose`, `[data-xds-media` attribute
+ *   selectors.
+ * - `\@layer xds-theme` / `\@layer xds-base` cascade layer names, including
+ *   comma-separated `\@layer a, b;` statement lists and nested blocks.
+ * - `\@import '\@xds/...'` package stylesheet imports: scope rewrite, plus the
+ *   `\@xds/core/xds.css` -> `\@astryxdesign/core/astryx.css` file rename and the
+ *   `theme-default` / `theme-daily` -> `theme-neutral` collapse.
  *
  * It deliberately does NOT blindly replace every `xds` substring: a bare
  * `xds` in a comment, a custom-property value, or an unrelated identifier is
@@ -51,8 +50,8 @@ const XDS_LAYER_NAMES = new Map([
   ['xds-base', 'astryx-base'],
 ]);
 
-function rewriteLayerPrelude(prelude) {
-  return prelude.replace(/\bxds-(?:theme|base)\b/g, name =>
+function rewriteLayerPrelude(/** @type {any} */ prelude) {
+  return prelude.replace(/\bxds-(?:theme|base)\b/g, (/** @type {any} */ name) =>
     XDS_LAYER_NAMES.get(name) ?? name,
   );
 }
@@ -64,7 +63,7 @@ function rewriteLayerPrelude(prelude) {
 //   @xds/theme-default/*         -> @astryxdesign/theme-neutral/*   (collapse)
 //   @xds/theme-daily/*           -> @astryxdesign/theme-neutral/*   (collapse)
 // Returns the original value unchanged if it is not an @xds/* specifier.
-function rewriteImportSpecifier(spec) {
+function rewriteImportSpecifier(/** @type {any} */ spec) {
   if (!spec.startsWith('@xds/')) return spec;
   // Theme package collapse (default/daily -> neutral).
   let next = spec
@@ -85,6 +84,7 @@ function rewriteImportSpecifier(spec) {
 
 // Ordered, non-overlapping replacements. Each pattern is intentionally narrow
 // so we never touch a bare `xds` substring outside these surfaces.
+/** @type {Array<{re: RegExp, to: any}>} */
 const REPLACEMENTS = [
   // Class-name prefix: only when preceded by a `.` (a class selector).
   {re: /\.xds-/g, to: '.astryx-'},
@@ -92,18 +92,22 @@ const REPLACEMENTS = [
   {re: /\[data-xds-/g, to: '[data-astryx-'},
   // Cascade-layer preludes: `@layer <names>` up to the block `{` or `;`.
   // Rewrite only the recognized xds-* layer tokens inside the prelude.
-  {re: /@layer\b[^{;]*/g, to: match => rewriteLayerPrelude(match)},
+  {re: /@layer\b[^{;]*/g, to: (/** @type {any} */ match) => rewriteLayerPrelude(match)},
   // Package stylesheet imports: `@import '@xds/...'` or `@import url('@xds/...')`.
   // Rewrite only the quoted @xds/* specifier, preserving the quote style.
   {
     re: /@import\s+(?:url\(\s*)?(['"])(@xds\/[^'"]+)\1/g,
-    to: (match, quote, spec) => {
+    to: (/** @type {any} */ match, /** @type {any} */ quote, /** @type {any} */ spec) => {
       const next = rewriteImportSpecifier(spec);
       return next === spec ? match : match.replace(spec, next);
     },
   },
 ];
 
+/**
+ * @param {import('../../../types/codemod').AstryxCodemodFile} file
+ * @returns {string | null | undefined}
+ */
 export default function transformer(file /*, api */) {
   const source = file.source;
   if (typeof source !== 'string') return undefined;

package/{src/codemods → codemods}/transforms/v0.1.0/migrate-xds-declare-module.mjs

@@ -48,7 +48,7 @@ const PACKAGE_RENAMES = new Map([
   ['@xds/theme-y2k', '@astryxdesign/theme-y2k'],
 ]);
 
-function renamePackageSpecifier(value) {
+function renamePackageSpecifier(/** @type {any} */ value) {
   if (typeof value !== 'string') return value;
   for (const [from, to] of PACKAGE_RENAMES) {
     if (value === from) return to;
@@ -57,12 +57,17 @@ function renamePackageSpecifier(value) {
   return value;
 }
 
+/**
+ * @param {import('../../../types/codemod').AstryxCodemodFile} file
+ * @param {import('../../../types/codemod').CodemodTransformApi} api
+ * @returns {string | null | undefined}
+ */
 export default function transformer(file, api) {
   const j = api.jscodeshift;
   const root = j(file.source);
   let hasChanges = false;
 
-  root.find(j.TSModuleDeclaration).forEach(path => {
+  root.find(j.TSModuleDeclaration).forEach((/** @type {any} */ path) => {
     const id = path.node.id;
     // Module augmentations name the module with a string literal id; a
     // `namespace Foo {}` uses an Identifier id, which we ignore.

package/{src/codemods → codemods}/transforms/v0.1.0/migrate-xds-module-specifiers.mjs

@@ -35,7 +35,7 @@ const PACKAGE_RENAMES = new Map([
   ['@xds/theme-y2k', '@astryxdesign/theme-y2k'],
 ]);
 
-function renamePackageSpecifier(value) {
+function renamePackageSpecifier(/** @type {any} */ value) {
   if (typeof value !== 'string') return value;
   for (const [from, to] of PACKAGE_RENAMES) {
     if (value === from) return to;
@@ -44,7 +44,7 @@ function renamePackageSpecifier(value) {
   return value;
 }
 
-function rewriteLiteral(node) {
+function rewriteLiteral(/** @type {any} */ node) {
   if (!node || typeof node.value !== 'string') return false;
   const next = renamePackageSpecifier(node.value);
   if (next === node.value) return false;
@@ -66,7 +66,7 @@ const COLLAPSED_THEME_SOURCES = new Set([
   '@xds/theme-daily',
 ]);
 
-function isCollapsedThemeSource(value) {
+function isCollapsedThemeSource(/** @type {any} */ value) {
   if (typeof value !== 'string') return false;
   for (const src of COLLAPSED_THEME_SOURCES) {
     if (value === src || value.startsWith(src + '/')) return true;
@@ -84,7 +84,7 @@ function isCollapsedThemeSource(value) {
 const MOCK_METHOD_NAMES = new Set(['mock', 'doMock']);
 const MOCK_OBJECT_NAMES = new Set(['vi', 'jest']);
 
-function isMockCall(callee) {
+function isMockCall(/** @type {any} */ callee) {
   if (
     callee.type === 'MemberExpression' &&
     !callee.computed &&
@@ -98,9 +98,9 @@ function isMockCall(callee) {
   return callee.type === 'Identifier' && callee.name === 'mock';
 }
 
-function remapThemeExportNames(importPath, j) {
+function remapThemeExportNames(/** @type {any} */ importPath, /** @type {any} */ j) {
   let changed = false;
-  importPath.node.specifiers = (importPath.node.specifiers || []).map(spec => {
+  importPath.node.specifiers = (importPath.node.specifiers || []).map((/** @type {any} */ spec) => {
     if (spec.type !== 'ImportSpecifier') return spec;
     const importedName = spec.imported.name;
     const renamed = THEME_EXPORT_RENAMES.get(importedName);
@@ -116,12 +116,17 @@ function remapThemeExportNames(importPath, j) {
   return changed;
 }
 
+/**
+ * @param {import('../../../types/codemod').AstryxCodemodFile} file
+ * @param {import('../../../types/codemod').CodemodTransformApi} api
+ * @returns {string | null | undefined}
+ */
 export default function transformer(file, api) {
   const j = api.jscodeshift;
   const root = j(file.source);
   let hasChanges = false;
 
-  root.find(j.ImportDeclaration).forEach(path => {
+  root.find(j.ImportDeclaration).forEach((/** @type {any} */ path) => {
     // Remap collapsed-theme export names BEFORE rewriting the source specifier
     // (the check keys off the original @xds/theme-default|daily path).
     if (isCollapsedThemeSource(path.node.source.value)) {
@@ -130,15 +135,15 @@ export default function transformer(file, api) {
     hasChanges = rewriteLiteral(path.node.source) || hasChanges;
   });
 
-  root.find(j.ExportNamedDeclaration).forEach(path => {
+  root.find(j.ExportNamedDeclaration).forEach((/** @type {any} */ path) => {
     hasChanges = rewriteLiteral(path.node.source) || hasChanges;
   });
 
-  root.find(j.ExportAllDeclaration).forEach(path => {
+  root.find(j.ExportAllDeclaration).forEach((/** @type {any} */ path) => {
     hasChanges = rewriteLiteral(path.node.source) || hasChanges;
   });
 
-  root.find(j.CallExpression).forEach(path => {
+  root.find(j.CallExpression).forEach((/** @type {any} */ path) => {
     const isDynamicImport = path.node.callee.type === 'Import';
     const isRequire =
       path.node.callee.type === 'Identifier' &&
@@ -160,7 +165,7 @@ export default function transformer(file, api) {
   // position. Re-visiting a node the loop above handled is harmless: the
   // literal is already rewritten, so the second pass is a no-op.
   const seenTypeImportNodes = new Set();
-  const rewriteTSImportTypes = node => {
+  const rewriteTSImportTypes = (/** @type {any} */ node) => {
     if (!node || typeof node !== 'object') return;
     if (Array.isArray(node)) {
       for (const child of node) rewriteTSImportTypes(child);

package/{src/codemods → codemods}/transforms/v0.1.2/rename-text-color-active-to-accent.mjs

@@ -40,13 +40,18 @@ const NEW_VALUE = 'accent';
 /** Components whose `color` prop accepts a TextColor value. */
 const TARGET_COMPONENTS = new Set(['Text', 'Heading', 'Link', 'Timestamp']);
 
+/**
+ * @param {import('../../../types/codemod').AstryxCodemodFile} file
+ * @param {import('../../../types/codemod').CodemodTransformApi} api
+ * @returns {string | null | undefined}
+ */
 export default function transformer(file, api) {
   const j = api.jscodeshift;
   const root = j(file.source);
   let hasChanges = false;
 
   /** Rewrite a string-literal node when it equals the old value. */
-  function renameStringLiteral(node) {
+  function renameStringLiteral(/** @type {any} */ node) {
     if (!node) return false;
     if (
       (node.type === 'StringLiteral' || node.type === 'Literal') &&
@@ -60,7 +65,8 @@ export default function transformer(file, api) {
   }
 
   /** Recursively rewrite the old value inside ternary/logical expressions. */
-  function renameInExpression(node) {
+  /** @returns {boolean} */
+  function renameInExpression(/** @type {any} */ node) {
     if (!node) return false;
     let changed = false;
     if (node.type === 'StringLiteral' || node.type === 'Literal') {
@@ -79,12 +85,12 @@ export default function transformer(file, api) {
   }
 
   // --- 1. JSX attribute: color="active" / color={'active'} on target components ---
-  root.find(j.JSXOpeningElement).forEach(path => {
+  root.find(j.JSXOpeningElement).forEach((/** @type {any} */ path) => {
     const name = path.node.name;
     const componentName = name.type === 'JSXIdentifier' ? name.name : null;
     if (!componentName || !TARGET_COMPONENTS.has(componentName)) return;
 
-    path.node.attributes.forEach(attr => {
+    path.node.attributes.forEach((/** @type {any} */ attr) => {
       if (attr.type !== 'JSXAttribute') return;
       if (!attr.name || attr.name.name !== 'color') return;
 
@@ -104,27 +110,27 @@ export default function transformer(file, api) {
   const importsTarget =
     root
       .find(j.ImportSpecifier)
-      .filter(p => TARGET_COMPONENTS.has(p.node.imported?.name))
+      .filter((/** @type {any} */ p) => TARGET_COMPONENTS.has(p.node.imported?.name))
       .size() > 0;
 
   if (importsTarget) {
     const PropertyType = j.ObjectProperty ?? j.Property;
 
     // --- 2. Object property: { color: 'active' } / { color: 'active' as const } ---
-    root.find(PropertyType, {key: {name: 'color'}}).forEach(path => {
+    root.find(PropertyType, {key: {name: 'color'}}).forEach((/** @type {any} */ path) => {
       if (renameInExpression(path.node.value)) hasChanges = true;
     });
 
     // --- 3. Storybook argTypes: color: { options: [..., 'active', ...] } ---
-    root.find(PropertyType, {key: {name: 'color'}}).forEach(path => {
+    root.find(PropertyType, {key: {name: 'color'}}).forEach((/** @type {any} */ path) => {
       const value = path.node.value;
       if (!value || value.type !== 'ObjectExpression') return;
 
       const optionsProp = value.properties.find(
-        p => p.key && (p.key.name === 'options' || p.key.value === 'options'),
+        (/** @type {any} */ p) => p.key && (p.key.name === 'options' || p.key.value === 'options'),
       );
       if (optionsProp && optionsProp.value.type === 'ArrayExpression') {
-        optionsProp.value.elements.forEach(el => {
+        optionsProp.value.elements.forEach((/** @type {any} */ el) => {
           if (renameStringLiteral(el)) hasChanges = true;
         });
       }

package/{src/codemods → codemods}/transforms/v0.1.3/migrate-layout-components-to-experimental.mjs

@@ -58,6 +58,11 @@ const MANUAL =
  * @param {{jscodeshift: Function}} api
  * @returns {string|null}
  */
+/**
+ * @param {import('../../../types/codemod').AstryxCodemodFile} file
+ * @param {import('../../../types/codemod').CodemodTransformApi} api
+ * @returns {string | null | undefined}
+ */
 export default function transformer(file, api) {
   const j = api.jscodeshift;
   const root = j(file.source);
@@ -105,7 +110,7 @@ export default function transformer(file, api) {
   // `components` was the only documented `layout` key. Anything else is
   // unexpected and we cannot safely guess intent.
   const otherLayoutKeys = layoutProp.value.properties.filter(
-    p => p !== componentsProp,
+    (/** @type {any} */ p) => p !== componentsProp,
   );
   if (otherLayoutKeys.length > 0) {
     throw new Error(
@@ -148,7 +153,7 @@ export default function transformer(file, api) {
   // Remove the migrated data from `layout`. Since `components` was the only
   // key (asserted above), remove the entire `layout` property.
   configObject.properties = configObject.properties.filter(
-    p => p !== layoutProp,
+    (/** @type {any} */ p) => p !== layoutProp,
   );
 
   return root.toSource();
@@ -157,9 +162,9 @@ export default function transformer(file, api) {
 /**
  * Resolve the wrapped config object literal from a default-export declaration.
  * Handles `{ ... }` and `createConfig({ ... })`.
- * @returns {object|null} the ObjectExpression node, or null if not analyzable
+ * @returns {any} the ObjectExpression node, or null if not analyzable
  */
-function resolveConfigObject(j, declaration) {
+function resolveConfigObject(/** @type {any} */ j, /** @type {any} */ declaration) {
   if (!declaration) return null;
   if (j.ObjectExpression.check(declaration)) {
     return declaration;
@@ -177,10 +182,10 @@ function resolveConfigObject(j, declaration) {
 /**
  * Find a (non-computed, non-spread) property on an object literal by key name.
  * Supports Identifier and string-literal keys.
- * @returns {object|undefined} the property node
+ * @returns {any} the property node
  */
-function findObjectProperty(j, objectExpression, name) {
-  return objectExpression.properties.find(prop => {
+function findObjectProperty(/** @type {any} */ j, /** @type {any} */ objectExpression, /** @type {any} */ name) {
+  return objectExpression.properties.find((/** @type {any} */ prop) => {
     if (
       !j.ObjectProperty.check(prop) &&
       !j.Property.check(prop)
@@ -200,8 +205,8 @@ function findObjectProperty(j, objectExpression, name) {
  * Build the migrated `components` ObjectExpression from the old one.
  * @returns {object} ObjectExpression
  */
-function buildMigratedComponents(j, oldComponents) {
-  const newProps = oldComponents.properties.map(prop => {
+function buildMigratedComponents(/** @type {any} */ j, /** @type {any} */ oldComponents) {
+  const newProps = oldComponents.properties.map((/** @type {any} */ prop) => {
     if (
       (!j.ObjectProperty.check(prop) && !j.Property.check(prop)) ||
       prop.computed
@@ -239,14 +244,14 @@ function buildMigratedComponents(j, oldComponents) {
 }
 
 /** Build `{ components: <obj> }` for `experimental.xle`. */
-function buildXleObject(j, migratedComponents) {
+function buildXleObject(/** @type {any} */ j, /** @type {any} */ migratedComponents) {
   return j.objectExpression([
     j.objectProperty(j.identifier('components'), migratedComponents),
   ]);
 }
 
 /** Clone a property key node so we don't reuse a node across the tree. */
-function cloneKey(j, key) {
+function cloneKey(/** @type {any} */ j, /** @type {any} */ key) {
   if (j.Identifier.check(key)) return j.identifier(key.name);
   if (j.StringLiteral.check(key)) return j.stringLiteral(key.value);
   if (j.Literal.check(key)) return j.literal(key.value);
@@ -254,7 +259,7 @@ function cloneKey(j, key) {
 }
 
 /** Build a fresh string literal preserving the original raw quoting if possible. */
-function cloneStringLiteral(j, value) {
+function cloneStringLiteral(/** @type {any} */ j, /** @type {any} */ value) {
   if (j.StringLiteral.check(value)) return j.stringLiteral(value.value);
   return j.literal(value.value);
 }

package/{src/codemods → codemods}/transforms/v0.1.5/rename-switch-label-spacing-default-to-hug.mjs

@@ -44,13 +44,18 @@ const TARGET_PROP = 'labelSpacing';
 /** Components whose `labelSpacing` prop accepts a SwitchLabelSpacing value. */
 const TARGET_COMPONENTS = new Set(['Switch']);
 
+/**
+ * @param {import('../../../types/codemod').AstryxCodemodFile} file
+ * @param {import('../../../types/codemod').CodemodTransformApi} api
+ * @returns {string | null | undefined}
+ */
 export default function transformer(file, api) {
   const j = api.jscodeshift;
   const root = j(file.source);
   let hasChanges = false;
 
   /** Rewrite a string-literal node when it equals the old value. */
-  function renameStringLiteral(node) {
+  function renameStringLiteral(/** @type {any} */ node) {
     if (!node) return false;
     if (
       (node.type === 'StringLiteral' || node.type === 'Literal') &&
@@ -64,7 +69,8 @@ export default function transformer(file, api) {
   }
 
   /** Recursively rewrite the old value inside ternary/logical expressions. */
-  function renameInExpression(node) {
+  /** @returns {boolean} */
+  function renameInExpression(/** @type {any} */ node) {
     if (!node) return false;
     let changed = false;
     if (node.type === 'StringLiteral' || node.type === 'Literal') {
@@ -83,12 +89,12 @@ export default function transformer(file, api) {
   }
 
   // --- 1. JSX attribute: labelSpacing="default" / labelSpacing={'default'} on Switch ---
-  root.find(j.JSXOpeningElement).forEach(path => {
+  root.find(j.JSXOpeningElement).forEach((/** @type {any} */ path) => {
     const name = path.node.name;
     const componentName = name.type === 'JSXIdentifier' ? name.name : null;
     if (!componentName || !TARGET_COMPONENTS.has(componentName)) return;
 
-    path.node.attributes.forEach(attr => {
+    path.node.attributes.forEach((/** @type {any} */ attr) => {
       if (attr.type !== 'JSXAttribute') return;
       if (!attr.name || attr.name.name !== TARGET_PROP) return;
 
@@ -108,27 +114,27 @@ export default function transformer(file, api) {
   const importsTarget =
     root
       .find(j.ImportSpecifier)
-      .filter(p => TARGET_COMPONENTS.has(p.node.imported?.name))
+      .filter((/** @type {any} */ p) => TARGET_COMPONENTS.has(p.node.imported?.name))
       .size() > 0;
 
   if (importsTarget) {
     const PropertyType = j.ObjectProperty ?? j.Property;
 
     // --- 2. Object property: { labelSpacing: 'default' } / 'default' as const ---
-    root.find(PropertyType, {key: {name: TARGET_PROP}}).forEach(path => {
+    root.find(PropertyType, {key: {name: TARGET_PROP}}).forEach((/** @type {any} */ path) => {
       if (renameInExpression(path.node.value)) hasChanges = true;
     });
 
     // --- 3. Storybook argTypes: labelSpacing: { options: [..., 'default', ...] } ---
-    root.find(PropertyType, {key: {name: TARGET_PROP}}).forEach(path => {
+    root.find(PropertyType, {key: {name: TARGET_PROP}}).forEach((/** @type {any} */ path) => {
       const value = path.node.value;
       if (!value || value.type !== 'ObjectExpression') return;
 
       const optionsProp = value.properties.find(
-        p => p.key && (p.key.name === 'options' || p.key.value === 'options'),
+        (/** @type {any} */ p) => p.key && (p.key.name === 'options' || p.key.value === 'options'),
       );
       if (optionsProp && optionsProp.value.type === 'ArrayExpression') {
-        optionsProp.value.elements.forEach(el => {
+        optionsProp.value.elements.forEach((/** @type {any} */ el) => {
           if (renameStringLiteral(el)) hasChanges = true;
         });
       }

package/{src/codemods → codemods}/transforms/v0.1.7/migrate-table-tableprops-to-direct-props.mjs

@@ -54,7 +54,7 @@ const TODO_COMMENT =
  * property is not a simple liftable entry (spread, computed key, method,
  * getter/setter, non-string literal key).
  */
-function getPropertyKeyName(prop) {
+function getPropertyKeyName(/** @type {any} */ prop) {
   if (prop.type !== 'ObjectProperty' && prop.type !== 'Property') return null;
   // espree-style Property nodes: skip methods and accessors
   if (prop.method || (prop.kind != null && prop.kind !== 'init')) return null;
@@ -70,6 +70,11 @@ function getPropertyKeyName(prop) {
   return null;
 }
 
+/**
+ * @param {import('../../../types/codemod').AstryxCodemodFile} file
+ * @param {import('../../../types/codemod').CodemodTransformApi} api
+ * @returns {string | null | undefined}
+ */
 export default function transformer(file, api) {
   const j = api.jscodeshift;
   const root = j(file.source);
@@ -77,7 +82,7 @@ export default function transformer(file, api) {
 
   // --- 1. Track local names for the Table import (alias-aware) ---
   const tableLocals = new Set();
-  root.find(j.ImportDeclaration).forEach((path) => {
+  root.find(j.ImportDeclaration).forEach((/** @type {any} */ path) => {
     if (!TABLE_IMPORT_SOURCES.has(path.node.source.value)) return;
     for (const spec of path.node.specifiers ?? []) {
       if (spec.type === 'ImportSpecifier' && spec.imported.name === 'Table') {
@@ -88,14 +93,14 @@ export default function transformer(file, api) {
 
   if (tableLocals.size === 0) return undefined;
 
-  function attachTodo(attr) {
+  function attachTodo(/** @type {any} */ attr) {
     if (!attr.comments) attr.comments = [];
-    if (attr.comments.some((c) => c.value === TODO_COMMENT)) return;
+    if (attr.comments.some((/** @type {any} */ c) => c.value === TODO_COMMENT)) return;
     attr.comments.push(j.commentBlock(TODO_COMMENT, false, true));
     hasChanges = true;
   }
 
-  function buildAttributeValue(valueNode) {
+  function buildAttributeValue(/** @type {any} */ valueNode) {
     if (
       valueNode.type === 'StringLiteral' ||
       (valueNode.type === 'Literal' && typeof valueNode.value === 'string')
@@ -106,14 +111,14 @@ export default function transformer(file, api) {
   }
 
   // --- 2. Rewrite tableProps on tracked <Table> elements ---
-  root.find(j.JSXOpeningElement).forEach((path) => {
+  root.find(j.JSXOpeningElement).forEach((/** @type {any} */ path) => {
     const name = path.node.name;
     const componentName = name.type === 'JSXIdentifier' ? name.name : null;
     if (!componentName || !tableLocals.has(componentName)) return;
 
     const attrs = path.node.attributes;
     const tablePropsAttr = attrs.find(
-      (a) => a.type === 'JSXAttribute' && a.name?.name === 'tableProps',
+      (/** @type {any} */ a) => a.type === 'JSXAttribute' && a.name?.name === 'tableProps',
     );
     if (!tablePropsAttr) return;
 
@@ -135,22 +140,24 @@ export default function transformer(file, api) {
     // Objects containing spreads, computed keys, or methods are treated
     // as dynamic: no partial lift, just the TODO comment.
     const allSimple = obj.properties.every(
-      (prop) => getPropertyKeyName(prop) !== null,
+      (/** @type {any} */ prop) => getPropertyKeyName(prop) !== null,
     );
     if (!allSimple) {
       attachTodo(tablePropsAttr);
       return;
     }
 
+    /** @type {any[]} */
     const lifted = [];
+    /** @type {any[]} */
     const kept = [];
     for (const prop of obj.properties) {
       const keyName = getPropertyKeyName(prop);
       const collidesWithSibling = attrs.some(
-        (a) => a.type === 'JSXAttribute' && a.name?.name === keyName,
+        (/** @type {any} */ a) => a.type === 'JSXAttribute' && a.name?.name === keyName,
       );
       const collidesWithLifted = lifted.some(
-        (a) => a.name.name === keyName,
+        (/** @type {any} */ a) => a.name.name === keyName,
       );
       if (
         !LIFTABLE_KEY_RE.test(keyName) ||

package/{src/codemods → codemods}/transforms/v0.1.7/rename-table-renderprops-styles-to-xstyle.mjs

@@ -58,7 +58,7 @@ const RENDER_PROP_TYPES = new Set([
 ]);
 
 /** Resolve the base type name from a TS type annotation node. */
-function typeNameOf(typeAnnotation) {
+function typeNameOf(/** @type {any} */ typeAnnotation) {
   // `x: TableRenderProps`
   const t = typeAnnotation?.typeAnnotation ?? typeAnnotation;
   if (!t) return null;
@@ -68,6 +68,11 @@ function typeNameOf(typeAnnotation) {
   return null;
 }
 
+/**
+ * @param {import('../../../types/codemod').AstryxCodemodFile} file
+ * @param {import('../../../types/codemod').CodemodTransformApi} api
+ * @returns {string | null | undefined}
+ */
 export default function transformer(file, api) {
   const j = api.jscodeshift;
   const root = j(file.source);
@@ -77,7 +82,7 @@ export default function transformer(file, api) {
   // Function/arrow params: (props: TableRenderProps) => ...
   const renderPropBindings = new Set();
 
-  function collectParam(param) {
+  function collectParam(/** @type {any} */ param) {
     if (
       param.type === 'Identifier' &&
       RENDER_PROP_TYPES.has(typeNameOf(param.typeAnnotation))
@@ -88,19 +93,19 @@ export default function transformer(file, api) {
 
   root
     .find(j.Function)
-    .forEach((p) => (p.node.params ?? []).forEach(collectParam));
+    .forEach((/** @type {any} */ p) => (p.node.params ?? []).forEach(collectParam));
   root
     .find(j.FunctionDeclaration)
-    .forEach((p) => (p.node.params ?? []).forEach(collectParam));
+    .forEach((/** @type {any} */ p) => (p.node.params ?? []).forEach(collectParam));
   root
     .find(j.FunctionExpression)
-    .forEach((p) => (p.node.params ?? []).forEach(collectParam));
+    .forEach((/** @type {any} */ p) => (p.node.params ?? []).forEach(collectParam));
   root
     .find(j.ArrowFunctionExpression)
-    .forEach((p) => (p.node.params ?? []).forEach(collectParam));
+    .forEach((/** @type {any} */ p) => (p.node.params ?? []).forEach(collectParam));
 
   // Variable declarations: `const rp: BodyCellRenderProps = ...`
-  root.find(j.VariableDeclarator).forEach((p) => {
+  root.find(j.VariableDeclarator).forEach((/** @type {any} */ p) => {
     const id = p.node.id;
     if (
       id?.type === 'Identifier' &&
@@ -119,10 +124,10 @@ export default function transformer(file, api) {
   //  2. A spread of a tracked render-prop binding (`{...props, styles: [...]}`)
   //     — the common "carry the render-prop object forward, override styles"
   //     shape. (Only counts when `renderPropBindings` is non-empty.)
-  function isRenderPropShapedObject(objExpr) {
+  function isRenderPropShapedObject(/** @type {any} */ objExpr) {
     if (objExpr.type !== 'ObjectExpression') return false;
     const keyNames = objExpr.properties
-      .map((pr) =>
+      .map((/** @type {any} */ pr) =>
         pr.type === 'ObjectProperty' || pr.type === 'Property'
           ? pr.key?.name ?? pr.key?.value
           : null,
@@ -133,7 +138,7 @@ export default function transformer(file, api) {
     }
     // Spread of a tracked render-prop binding.
     const spreadsRenderProp = objExpr.properties.some(
-      (pr) =>
+      (/** @type {any} */ pr) =>
         (pr.type === 'SpreadElement' || pr.type === 'ExperimentalSpreadProperty') &&
         pr.argument?.type === 'Identifier' &&
         renderPropBindings.has(pr.argument.name),
@@ -146,7 +151,7 @@ export default function transformer(file, api) {
     // render-prop-shaped object literals (htmlProps + styles siblings);
     // never touch bare `styles` in this file (too ambiguous).
     let touched = false;
-    root.find(j.ObjectExpression).forEach((p) => {
+    root.find(j.ObjectExpression).forEach((/** @type {any} */ p) => {
       if (!isRenderPropShapedObject(p.node)) return;
       for (const prop of p.node.properties) {
         if (
@@ -165,7 +170,7 @@ export default function transformer(file, api) {
   }
 
   // --- 2. Rewrite `<binding>.styles` member access -> `.xstyle`. ---
-  root.find(j.MemberExpression).forEach((p) => {
+  root.find(j.MemberExpression).forEach((/** @type {any} */ p) => {
     const {object, property, computed} = p.node;
     if (computed) return;
     if (property?.type !== 'Identifier' || property.name !== 'styles') return;
@@ -177,7 +182,7 @@ export default function transformer(file, api) {
   });
 
   // --- 3. Rewrite `styles:` keys on render-prop-shaped object literals. ---
-  root.find(j.ObjectExpression).forEach((p) => {
+  root.find(j.ObjectExpression).forEach((/** @type {any} */ p) => {
     if (!isRenderPropShapedObject(p.node)) return;
     for (const prop of p.node.properties) {
       if (