@atlaskit/codemod-cli 0.34.1 → 0.34.3

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @atlaskit/codemod-cli
 
+## 0.34.3
+
+### Patch Changes
+
+- Updated dependencies
+
+## 0.34.2
+
+### Patch Changes
+
+- [`1aa97b0f3b4a8`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/1aa97b0f3b4a8) -
+  update lozenge-to-tag-migration to include the migration_fallback prop so that teams can adopt the
+  Tag API without triggering an immediate visual change.
+
 ## 0.34.1
 
 ### Patch Changes

package/dist/cjs/main.js

@@ -362,7 +362,7 @@ function _main() {
         case 4:
           _yield$parseArgs = _context6.sent;
           packages = _yield$parseArgs.packages;
-          _process$env$_PACKAGE = "0.34.0", _PACKAGE_VERSION_ = _process$env$_PACKAGE === void 0 ? '0.0.0-dev' : _process$env$_PACKAGE;
+          _process$env$_PACKAGE = "0.34.2", _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

@@ -101,6 +101,7 @@ var transformer = function transformer(file, api) {
     // Process children to extract text
     var hasVariableChild = false;
     var variableChildExpression = null;
+    var isFormatMessageCall = false;
     if (element.children && element.children.length > 0) {
       if (element.children.length === 1) {
         var child = element.children[0];
@@ -112,7 +113,7 @@ 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
+          // Check if it's a simple identifier (variable), member expression, or call expression
           if (j.Identifier.check(child.expression)) {
             hasVariableChild = true;
             variableChildExpression = child.expression;
@@ -120,6 +121,14 @@ var transformer = function transformer(file, api) {
             // Handle member expressions like obj.prop
             hasVariableChild = true;
             variableChildExpression = child.expression;
+          } else if (j.CallExpression.check(child.expression)) {
+            // Handle call expressions like formatMessage(messages.label)
+            hasVariableChild = true;
+            variableChildExpression = child.expression;
+            // Check if it's a formatMessage call (which always returns a string)
+            if (j.Identifier.check(child.expression.callee) && child.expression.callee.name === 'formatMessage') {
+              isFormatMessageCall = true;
+            }
           } else {
             // Complex expression or JSX element
             hasComplexChildren = true;
@@ -174,7 +183,7 @@ var transformer = function transformer(file, api) {
       }
     });
 
-    // Assemble attributes in correct order: text, other props, color
+    // Assemble attributes in correct order: text, other props, color, migration props
     if (textContent && !hasComplexChildren && !hasVariableChild) {
       newAttributes.push(j.jsxAttribute(j.jsxIdentifier('text'), j.stringLiteral(textContent)));
     } else if (hasVariableChild && variableChildExpression) {
@@ -185,7 +194,7 @@ var transformer = function transformer(file, api) {
     // Add other attributes
     newAttributes.push.apply(newAttributes, otherAttributes);
 
-    // Add color attribute last (default to 'standard' if no appearance was found)
+    // Add color attribute (default to 'standard' if no appearance was found)
     if (colorAttribute) {
       newAttributes.push(colorAttribute);
     } else {
@@ -193,6 +202,10 @@ var transformer = function transformer(file, api) {
       newAttributes.push(j.jsxAttribute(j.jsxIdentifier('color'), j.stringLiteral('standard')));
     }
 
+    // Add migration-specific props
+    newAttributes.push(j.jsxAttribute(j.jsxIdentifier('isRemovable'), j.jsxExpressionContainer(j.booleanLiteral(false))));
+    newAttributes.push(j.jsxAttribute(j.jsxIdentifier('migration_fallback'), j.stringLiteral('lozenge')));
+
     // Create new Tag element
     var newElement = j.jsxElement(j.jsxOpeningElement(j.jsxIdentifier('Tag'), newAttributes, true), null, []);
 
@@ -201,7 +214,7 @@ 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) {
+    if (hasVariableChild && !isFormatMessageCall) {
       warnings.push('FIXME: This Tag component uses a variable as the text prop. Please verify that the variable contains a string value.');
     }
     if (hasMaxWidthProp) {

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

@@ -92,6 +92,7 @@ If isBold is typically false, consider migrating to <Tag /> from '@atlaskit/tag'
     // Process children to extract text
     let hasVariableChild = false;
     let variableChildExpression = null;
+    let isFormatMessageCall = false;
     if (element.children && element.children.length > 0) {
       if (element.children.length === 1) {
         const child = element.children[0];
@@ -103,7 +104,7 @@ 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
+          // Check if it's a simple identifier (variable), member expression, or call expression
           if (j.Identifier.check(child.expression)) {
             hasVariableChild = true;
             variableChildExpression = child.expression;
@@ -111,6 +112,14 @@ If isBold is typically false, consider migrating to <Tag /> from '@atlaskit/tag'
             // Handle member expressions like obj.prop
             hasVariableChild = true;
             variableChildExpression = child.expression;
+          } else if (j.CallExpression.check(child.expression)) {
+            // Handle call expressions like formatMessage(messages.label)
+            hasVariableChild = true;
+            variableChildExpression = child.expression;
+            // Check if it's a formatMessage call (which always returns a string)
+            if (j.Identifier.check(child.expression.callee) && child.expression.callee.name === 'formatMessage') {
+              isFormatMessageCall = true;
+            }
           } else {
             // Complex expression or JSX element
             hasComplexChildren = true;
@@ -165,7 +174,7 @@ If isBold is typically false, consider migrating to <Tag /> from '@atlaskit/tag'
       }
     });
 
-    // Assemble attributes in correct order: text, other props, color
+    // Assemble attributes in correct order: text, other props, color, migration props
     if (textContent && !hasComplexChildren && !hasVariableChild) {
       newAttributes.push(j.jsxAttribute(j.jsxIdentifier('text'), j.stringLiteral(textContent)));
     } else if (hasVariableChild && variableChildExpression) {
@@ -176,7 +185,7 @@ If isBold is typically false, consider migrating to <Tag /> from '@atlaskit/tag'
     // Add other attributes
     newAttributes.push(...otherAttributes);
 
-    // Add color attribute last (default to 'standard' if no appearance was found)
+    // Add color attribute (default to 'standard' if no appearance was found)
     if (colorAttribute) {
       newAttributes.push(colorAttribute);
     } else {
@@ -184,6 +193,10 @@ If isBold is typically false, consider migrating to <Tag /> from '@atlaskit/tag'
       newAttributes.push(j.jsxAttribute(j.jsxIdentifier('color'), j.stringLiteral('standard')));
     }
 
+    // Add migration-specific props
+    newAttributes.push(j.jsxAttribute(j.jsxIdentifier('isRemovable'), j.jsxExpressionContainer(j.booleanLiteral(false))));
+    newAttributes.push(j.jsxAttribute(j.jsxIdentifier('migration_fallback'), j.stringLiteral('lozenge')));
+
     // Create new Tag element
     const newElement = j.jsxElement(j.jsxOpeningElement(j.jsxIdentifier('Tag'), newAttributes, true), null, []);
 
@@ -192,7 +205,7 @@ 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) {
+    if (hasVariableChild && !isFormatMessageCall) {
       warnings.push('FIXME: This Tag component uses a variable as the text prop. Please verify that the variable contains a string value.');
     }
     if (hasMaxWidthProp) {

package/dist/esm/main.js

@@ -355,7 +355,7 @@ function _main() {
         case 4:
           _yield$parseArgs = _context6.sent;
           packages = _yield$parseArgs.packages;
-          _process$env$_PACKAGE = "0.34.0", _PACKAGE_VERSION_ = _process$env$_PACKAGE === void 0 ? '0.0.0-dev' : _process$env$_PACKAGE;
+          _process$env$_PACKAGE = "0.34.2", _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

@@ -95,6 +95,7 @@ var transformer = function transformer(file, api) {
     // Process children to extract text
     var hasVariableChild = false;
     var variableChildExpression = null;
+    var isFormatMessageCall = false;
     if (element.children && element.children.length > 0) {
       if (element.children.length === 1) {
         var child = element.children[0];
@@ -106,7 +107,7 @@ 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
+          // Check if it's a simple identifier (variable), member expression, or call expression
           if (j.Identifier.check(child.expression)) {
             hasVariableChild = true;
             variableChildExpression = child.expression;
@@ -114,6 +115,14 @@ var transformer = function transformer(file, api) {
             // Handle member expressions like obj.prop
             hasVariableChild = true;
             variableChildExpression = child.expression;
+          } else if (j.CallExpression.check(child.expression)) {
+            // Handle call expressions like formatMessage(messages.label)
+            hasVariableChild = true;
+            variableChildExpression = child.expression;
+            // Check if it's a formatMessage call (which always returns a string)
+            if (j.Identifier.check(child.expression.callee) && child.expression.callee.name === 'formatMessage') {
+              isFormatMessageCall = true;
+            }
           } else {
             // Complex expression or JSX element
             hasComplexChildren = true;
@@ -168,7 +177,7 @@ var transformer = function transformer(file, api) {
       }
     });
 
-    // Assemble attributes in correct order: text, other props, color
+    // Assemble attributes in correct order: text, other props, color, migration props
     if (textContent && !hasComplexChildren && !hasVariableChild) {
       newAttributes.push(j.jsxAttribute(j.jsxIdentifier('text'), j.stringLiteral(textContent)));
     } else if (hasVariableChild && variableChildExpression) {
@@ -179,7 +188,7 @@ var transformer = function transformer(file, api) {
     // Add other attributes
     newAttributes.push.apply(newAttributes, otherAttributes);
 
-    // Add color attribute last (default to 'standard' if no appearance was found)
+    // Add color attribute (default to 'standard' if no appearance was found)
     if (colorAttribute) {
       newAttributes.push(colorAttribute);
     } else {
@@ -187,6 +196,10 @@ var transformer = function transformer(file, api) {
       newAttributes.push(j.jsxAttribute(j.jsxIdentifier('color'), j.stringLiteral('standard')));
     }
 
+    // Add migration-specific props
+    newAttributes.push(j.jsxAttribute(j.jsxIdentifier('isRemovable'), j.jsxExpressionContainer(j.booleanLiteral(false))));
+    newAttributes.push(j.jsxAttribute(j.jsxIdentifier('migration_fallback'), j.stringLiteral('lozenge')));
+
     // Create new Tag element
     var newElement = j.jsxElement(j.jsxOpeningElement(j.jsxIdentifier('Tag'), newAttributes, true), null, []);
 
@@ -195,7 +208,7 @@ 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) {
+    if (hasVariableChild && !isFormatMessageCall) {
       warnings.push('FIXME: This Tag component uses a variable as the text prop. Please verify that the variable contains a string value.');
     }
     if (hasMaxWidthProp) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/codemod-cli",
-	"version": "0.34.1",
+	"version": "0.34.3",
 	"description": "A cli for distributing codemods for atlassian-frontend components and services",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -29,9 +29,9 @@
 	"bin": "./bin/codemod-cli.js",
 	"dependencies": {
 		"@atlaskit/codemod-utils": "^4.2.0",
-		"@atlaskit/icon": "^29.4.0",
+		"@atlaskit/icon": "^30.0.0",
 		"@atlaskit/icon-lab": "^5.14.0",
-		"@atlaskit/tokens": "^10.0.0",
+		"@atlaskit/tokens": "^10.1.0",
 		"@babel/runtime": "^7.0.0",
 		"@codeshift/utils": "^0.2.4",
 		"@hypermod/utils": "^0.4.2",