@atlaskit/codemod-cli 0.32.0 → 0.32.2

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @atlaskit/codemod-cli
 
+## 0.32.2
+
+### Patch Changes
+
+- [`a60a82196851a`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/a60a82196851a) -
+  Internal refactors to remove unused variables. No functional or public changes.
+- Updated dependencies
+
+## 0.32.1
+
+### Patch Changes
+
+- [`b866a7f72dab4`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/b866a7f72dab4) -
+  Updated lozenge-to-tag-migration codemod to handle variable lozenge children
+
 ## 0.32.0
 
 ### Minor Changes

package/dist/cjs/main.js

@@ -186,7 +186,7 @@ var runTransform = /*#__PURE__*/function () {
           _fs.default.writeFileSync(jscodeshift, jscodeshiftContentNew);
           try {
             transformModule = require(transformPath);
-          } catch (error) {
+          } catch (_unused) {
             // eslint-disable-next-line no-console
             console.warn("Error loading transform module: ".concat(transformPath, ". Skipping lifecycle hooks."));
           }
@@ -362,7 +362,7 @@ function _main() {
         case 4:
           _yield$parseArgs = _context6.sent;
           packages = _yield$parseArgs.packages;
-          _process$env$_PACKAGE = "0.31.2", _PACKAGE_VERSION_ = _process$env$_PACKAGE === void 0 ? '0.0.0-dev' : _process$env$_PACKAGE;
+          _process$env$_PACKAGE = "0.32.1", _PACKAGE_VERSION_ = _process$env$_PACKAGE === void 0 ? '0.0.0-dev' : _process$env$_PACKAGE;
           logger.log(_chalk.default.bgBlue(_chalk.default.black("\uD83D\uDCDA Atlassian-Frontend codemod library @ ".concat(_PACKAGE_VERSION_, " \uD83D\uDCDA"))));
           if (packages && packages.length > 0) {
             logger.log(_chalk.default.gray("Searching for codemods for newer versions of the following packages: ".concat(packages.map(function (pkg) {

package/dist/cjs/presets/lozenge-to-tag-migration/codemods/lozenge-to-tag-migration.js

@@ -99,6 +99,8 @@ var transformer = function transformer(file, api) {
     var unknownAppearanceValue = '';
 
     // Process children to extract text
+    var hasVariableChild = false;
+    var variableChildExpression = null;
     if (element.children && element.children.length > 0) {
       if (element.children.length === 1) {
         var child = element.children[0];
@@ -109,7 +111,21 @@ var transformer = function transformer(file, api) {
           }
         } else if (j.JSXExpressionContainer.check(child) && j.StringLiteral.check(child.expression)) {
           textContent = child.expression.value;
+        } else if (j.JSXExpressionContainer.check(child)) {
+          // Check if it's a simple identifier (variable) or member expression
+          if (j.Identifier.check(child.expression)) {
+            hasVariableChild = true;
+            variableChildExpression = child.expression;
+          } else if (j.MemberExpression.check(child.expression)) {
+            // Handle member expressions like obj.prop
+            hasVariableChild = true;
+            variableChildExpression = child.expression;
+          } else {
+            // Complex expression or JSX element
+            hasComplexChildren = true;
+          }
         } else {
+          // JSX elements or other complex children
           hasComplexChildren = true;
         }
       } else {
@@ -159,8 +175,11 @@ var transformer = function transformer(file, api) {
     });
 
     // Assemble attributes in correct order: text, other props, color
-    if (textContent && !hasComplexChildren) {
+    if (textContent && !hasComplexChildren && !hasVariableChild) {
       newAttributes.push(j.jsxAttribute(j.jsxIdentifier('text'), j.stringLiteral(textContent)));
+    } else if (hasVariableChild && variableChildExpression) {
+      // Use the variable expression as the text prop
+      newAttributes.push(j.jsxAttribute(j.jsxIdentifier('text'), j.jsxExpressionContainer(variableChildExpression)));
     }
 
     // Add other attributes
@@ -182,6 +201,9 @@ var transformer = function transformer(file, api) {
     if (hasComplexChildren) {
       warnings.push("FIXME: This Tag component has complex children that couldn't be automatically migrated to the text prop.\nTag component only supports simple text via the text prop. Please manually convert the children content.");
     }
+    if (hasVariableChild) {
+      warnings.push('FIXME: This Tag component uses a variable as the text prop. Please verify that the variable contains a string value.');
+    }
     if (hasMaxWidthProp) {
       warnings.push('FIXME: maxWidth prop was removed during migration from Lozenge to Tag.\nTag component does not support maxWidth. Please review if width constraints are needed.');
     }

package/dist/cjs/presets/remove-token-fallbacks/utils/token-processor.js

@@ -601,7 +601,7 @@ var TokenProcessor = exports.TokenProcessor = /*#__PURE__*/function () {
         });
         this.logVerbose("Resolved module path: ".concat(_chalk.default.green(resolvedPath), " for ").concat(_chalk.default.cyan(moduleName)));
         return resolvedPath;
-      } catch (error) {
+      } catch (_unused) {
         return null;
       }
     }

package/dist/es2019/main.js

@@ -115,7 +115,7 @@ const runTransform = async (filePaths, transform, flags) => {
   let transformModule;
   try {
     transformModule = require(transformPath);
-  } catch (error) {
+  } catch {
     // eslint-disable-next-line no-console
     console.warn(`Error loading transform module: ${transformPath}. Skipping lifecycle hooks.`);
   }

package/dist/es2019/presets/lozenge-to-tag-migration/codemods/lozenge-to-tag-migration.js

@@ -90,6 +90,8 @@ If isBold is typically false, consider migrating to <Tag /> from '@atlaskit/tag'
     let unknownAppearanceValue = '';
 
     // Process children to extract text
+    let hasVariableChild = false;
+    let variableChildExpression = null;
     if (element.children && element.children.length > 0) {
       if (element.children.length === 1) {
         const child = element.children[0];
@@ -100,7 +102,21 @@ If isBold is typically false, consider migrating to <Tag /> from '@atlaskit/tag'
           }
         } else if (j.JSXExpressionContainer.check(child) && j.StringLiteral.check(child.expression)) {
           textContent = child.expression.value;
+        } else if (j.JSXExpressionContainer.check(child)) {
+          // Check if it's a simple identifier (variable) or member expression
+          if (j.Identifier.check(child.expression)) {
+            hasVariableChild = true;
+            variableChildExpression = child.expression;
+          } else if (j.MemberExpression.check(child.expression)) {
+            // Handle member expressions like obj.prop
+            hasVariableChild = true;
+            variableChildExpression = child.expression;
+          } else {
+            // Complex expression or JSX element
+            hasComplexChildren = true;
+          }
         } else {
+          // JSX elements or other complex children
           hasComplexChildren = true;
         }
       } else {
@@ -150,8 +166,11 @@ If isBold is typically false, consider migrating to <Tag /> from '@atlaskit/tag'
     });
 
     // Assemble attributes in correct order: text, other props, color
-    if (textContent && !hasComplexChildren) {
+    if (textContent && !hasComplexChildren && !hasVariableChild) {
       newAttributes.push(j.jsxAttribute(j.jsxIdentifier('text'), j.stringLiteral(textContent)));
+    } else if (hasVariableChild && variableChildExpression) {
+      // Use the variable expression as the text prop
+      newAttributes.push(j.jsxAttribute(j.jsxIdentifier('text'), j.jsxExpressionContainer(variableChildExpression)));
     }
 
     // Add other attributes
@@ -173,6 +192,9 @@ If isBold is typically false, consider migrating to <Tag /> from '@atlaskit/tag'
     if (hasComplexChildren) {
       warnings.push("FIXME: This Tag component has complex children that couldn't be automatically migrated to the text prop.\nTag component only supports simple text via the text prop. Please manually convert the children content.");
     }
+    if (hasVariableChild) {
+      warnings.push('FIXME: This Tag component uses a variable as the text prop. Please verify that the variable contains a string value.');
+    }
     if (hasMaxWidthProp) {
       warnings.push('FIXME: maxWidth prop was removed during migration from Lozenge to Tag.\nTag component does not support maxWidth. Please review if width constraints are needed.');
     }

package/dist/es2019/presets/remove-token-fallbacks/utils/token-processor.js

@@ -402,7 +402,7 @@ export class TokenProcessor {
       });
       this.logVerbose(`Resolved module path: ${chalk.green(resolvedPath)} for ${chalk.cyan(moduleName)}`);
       return resolvedPath;
-    } catch (error) {
+    } catch {
       return null;
     }
   }

package/dist/es2019/sinceRef.js

@@ -23,7 +23,7 @@ export const getPackagesSinceRef = async ref => {
   let commit;
   try {
     commit = await git.revparse(['--verify', ref]);
-  } catch (e) {
+  } catch {
     throw new ValidationError(`Invalid git ref "${ref}"`);
   }
   const diff = await git.diff([commit, '--', 'package.json']);

package/dist/esm/main.js

@@ -179,7 +179,7 @@ var runTransform = /*#__PURE__*/function () {
           fs.writeFileSync(jscodeshift, jscodeshiftContentNew);
           try {
             transformModule = require(transformPath);
-          } catch (error) {
+          } catch (_unused) {
             // eslint-disable-next-line no-console
             console.warn("Error loading transform module: ".concat(transformPath, ". Skipping lifecycle hooks."));
           }
@@ -355,7 +355,7 @@ function _main() {
         case 4:
           _yield$parseArgs = _context6.sent;
           packages = _yield$parseArgs.packages;
-          _process$env$_PACKAGE = "0.31.2", _PACKAGE_VERSION_ = _process$env$_PACKAGE === void 0 ? '0.0.0-dev' : _process$env$_PACKAGE;
+          _process$env$_PACKAGE = "0.32.1", _PACKAGE_VERSION_ = _process$env$_PACKAGE === void 0 ? '0.0.0-dev' : _process$env$_PACKAGE;
           logger.log(chalk.bgBlue(chalk.black("\uD83D\uDCDA Atlassian-Frontend codemod library @ ".concat(_PACKAGE_VERSION_, " \uD83D\uDCDA"))));
           if (packages && packages.length > 0) {
             logger.log(chalk.gray("Searching for codemods for newer versions of the following packages: ".concat(packages.map(function (pkg) {

package/dist/esm/presets/lozenge-to-tag-migration/codemods/lozenge-to-tag-migration.js

@@ -93,6 +93,8 @@ var transformer = function transformer(file, api) {
     var unknownAppearanceValue = '';
 
     // Process children to extract text
+    var hasVariableChild = false;
+    var variableChildExpression = null;
     if (element.children && element.children.length > 0) {
       if (element.children.length === 1) {
         var child = element.children[0];
@@ -103,7 +105,21 @@ var transformer = function transformer(file, api) {
           }
         } else if (j.JSXExpressionContainer.check(child) && j.StringLiteral.check(child.expression)) {
           textContent = child.expression.value;
+        } else if (j.JSXExpressionContainer.check(child)) {
+          // Check if it's a simple identifier (variable) or member expression
+          if (j.Identifier.check(child.expression)) {
+            hasVariableChild = true;
+            variableChildExpression = child.expression;
+          } else if (j.MemberExpression.check(child.expression)) {
+            // Handle member expressions like obj.prop
+            hasVariableChild = true;
+            variableChildExpression = child.expression;
+          } else {
+            // Complex expression or JSX element
+            hasComplexChildren = true;
+          }
         } else {
+          // JSX elements or other complex children
           hasComplexChildren = true;
         }
       } else {
@@ -153,8 +169,11 @@ var transformer = function transformer(file, api) {
     });
 
     // Assemble attributes in correct order: text, other props, color
-    if (textContent && !hasComplexChildren) {
+    if (textContent && !hasComplexChildren && !hasVariableChild) {
       newAttributes.push(j.jsxAttribute(j.jsxIdentifier('text'), j.stringLiteral(textContent)));
+    } else if (hasVariableChild && variableChildExpression) {
+      // Use the variable expression as the text prop
+      newAttributes.push(j.jsxAttribute(j.jsxIdentifier('text'), j.jsxExpressionContainer(variableChildExpression)));
     }
 
     // Add other attributes
@@ -176,6 +195,9 @@ var transformer = function transformer(file, api) {
     if (hasComplexChildren) {
       warnings.push("FIXME: This Tag component has complex children that couldn't be automatically migrated to the text prop.\nTag component only supports simple text via the text prop. Please manually convert the children content.");
     }
+    if (hasVariableChild) {
+      warnings.push('FIXME: This Tag component uses a variable as the text prop. Please verify that the variable contains a string value.');
+    }
     if (hasMaxWidthProp) {
       warnings.push('FIXME: maxWidth prop was removed during migration from Lozenge to Tag.\nTag component does not support maxWidth. Please review if width constraints are needed.');
     }

package/dist/esm/presets/remove-token-fallbacks/utils/token-processor.js

@@ -594,7 +594,7 @@ export var TokenProcessor = /*#__PURE__*/function () {
         });
         this.logVerbose("Resolved module path: ".concat(chalk.green(resolvedPath), " for ").concat(chalk.cyan(moduleName)));
         return resolvedPath;
-      } catch (error) {
+      } catch (_unused) {
         return null;
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/codemod-cli",
-	"version": "0.32.0",
+	"version": "0.32.2",
 	"description": "A cli for distributing codemods for atlassian-frontend components and services",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -29,7 +29,7 @@
 	"bin": "./bin/codemod-cli.js",
 	"dependencies": {
 		"@atlaskit/codemod-utils": "^4.2.0",
-		"@atlaskit/tokens": "^8.0.0",
+		"@atlaskit/tokens": "^8.4.0",
 		"@babel/runtime": "^7.0.0",
 		"@codeshift/utils": "^0.2.4",
 		"@hypermod/utils": "^0.4.2",