@atlaskit/popup 1.17.2 → 1.18.0

package/codemods/1.0.0-lite-mode.tsx

@@ -1,211 +1,178 @@
 import {
-  type API,
-  type ASTPath,
-  type default as core,
-  type FileInfo,
-  type JSXElement,
-  JSXExpressionContainer,
-  type Literal,
-  type Options,
+	type API,
+	type ASTPath,
+	type default as core,
+	type FileInfo,
+	type JSXElement,
+	JSXExpressionContainer,
+	type Literal,
+	type Options,
 } from 'jscodeshift';
 import { type Collection } from 'jscodeshift/src/Collection';
 
 import {
-  addCommentToStartOfFile,
-  getDefaultSpecifierName,
-  getJSXAttributesByName,
-  getSpecifierName,
-  hasImportDeclaration,
-  isUsingProp,
-  updateRenderProps,
+	addCommentToStartOfFile,
+	getDefaultSpecifierName,
+	getJSXAttributesByName,
+	getSpecifierName,
+	hasImportDeclaration,
+	isUsingProp,
+	updateRenderProps,
 } from './utils/helpers';
 
-function updateBoundariesProps(
-  j: core.JSCodeshift,
-  source: Collection<any>,
-  specifier: string,
-) {
-  source.findJSXElements(specifier).forEach((path: ASTPath<JSXElement>) => {
-    if (
-      isUsingProp({
-        j,
-        base: source,
-        element: path,
-        propName: 'boundariesElement',
-      })
-    ) {
-      // Get value from prop
-      getJSXAttributesByName({
-        j,
-        element: path,
-        attributeName: 'boundariesElement',
-      }).forEach((attribute) => {
-        const expression = attribute.node.value;
-        if (expression && expression.type === 'StringLiteral') {
-          const value = expression && expression.value;
-          if (value === 'window') {
-            j(attribute).replaceWith(
-              j.jsxAttribute(
-                j.jsxIdentifier('rootBoundary'),
-                j.stringLiteral('document'),
-              ),
-            );
-          } else if (value === 'viewport') {
-            j(attribute).replaceWith(
-              j.jsxAttribute(
-                j.jsxIdentifier('rootBoundary'),
-                j.stringLiteral('viewport'),
-              ),
-            );
-          } else if (value === 'scrollParents') {
-            j(attribute).replaceWith(
-              j.jsxAttribute(
-                j.jsxIdentifier('boundary'),
-                j.stringLiteral('clippingParents'),
-              ),
-            );
-          }
-        }
-      });
-      // Add relevant replacement props
+function updateBoundariesProps(j: core.JSCodeshift, source: Collection<any>, specifier: string) {
+	source.findJSXElements(specifier).forEach((path: ASTPath<JSXElement>) => {
+		if (
+			isUsingProp({
+				j,
+				base: source,
+				element: path,
+				propName: 'boundariesElement',
+			})
+		) {
+			// Get value from prop
+			getJSXAttributesByName({
+				j,
+				element: path,
+				attributeName: 'boundariesElement',
+			}).forEach((attribute) => {
+				const expression = attribute.node.value;
+				if (expression && expression.type === 'StringLiteral') {
+					const value = expression && expression.value;
+					if (value === 'window') {
+						j(attribute).replaceWith(
+							j.jsxAttribute(j.jsxIdentifier('rootBoundary'), j.stringLiteral('document')),
+						);
+					} else if (value === 'viewport') {
+						j(attribute).replaceWith(
+							j.jsxAttribute(j.jsxIdentifier('rootBoundary'), j.stringLiteral('viewport')),
+						);
+					} else if (value === 'scrollParents') {
+						j(attribute).replaceWith(
+							j.jsxAttribute(j.jsxIdentifier('boundary'), j.stringLiteral('clippingParents')),
+						);
+					}
+				}
+			});
+			// Add relevant replacement props
 
-      // Remove old prop
-      getJSXAttributesByName({
-        j,
-        element: path,
-        attributeName: 'boundariesElement',
-      }).remove();
-    }
-  });
+			// Remove old prop
+			getJSXAttributesByName({
+				j,
+				element: path,
+				attributeName: 'boundariesElement',
+			}).remove();
+		}
+	});
 }
 
-function updateOffset(
-  j: core.JSCodeshift,
-  source: Collection<any>,
-  specifier: string,
-) {
-  source.findJSXElements(specifier).forEach((path: ASTPath<JSXElement>) => {
-    getJSXAttributesByName({
-      j,
-      element: path,
-      attributeName: 'offset',
-    })
-      .find(JSXExpressionContainer)
-      .forEach((attribute) => {
-        const expression = attribute.value.expression;
-        if (expression.type === 'StringLiteral') {
-          const value = expression.value;
-          // Not testing for cases like '10 + 10%' because I assume if you're
-          // adding or taking numbers it's with units that are not supported
-          // and will be picked up by the first case
-          if (
-            value.includes('%') ||
-            value.includes('vw') ||
-            value.includes('vh')
-          ) {
-            addCommentToStartOfFile({
-              j,
-              base: source,
-              message: `
+function updateOffset(j: core.JSCodeshift, source: Collection<any>, specifier: string) {
+	source.findJSXElements(specifier).forEach((path: ASTPath<JSXElement>) => {
+		getJSXAttributesByName({
+			j,
+			element: path,
+			attributeName: 'offset',
+		})
+			.find(JSXExpressionContainer)
+			.forEach((attribute) => {
+				const expression = attribute.value.expression;
+				if (expression.type === 'StringLiteral') {
+					const value = expression.value;
+					// Not testing for cases like '10 + 10%' because I assume if you're
+					// adding or taking numbers it's with units that are not supported
+					// and will be picked up by the first case
+					if (value.includes('%') || value.includes('vw') || value.includes('vh')) {
+						addCommentToStartOfFile({
+							j,
+							base: source,
+							message: `
                 Popper.js has been upgraded from 1.14.1 to 2.4.2,
                 and as a result the offset prop has changed to be an array. e.g '0px 8px' -> [0, 8]
                 Along with this change you cannot use vw, vh or % units or addition or multiplication
                 Change the offset value to use pixel values
                 Further details can be found in the popper docs https://popper.js.org/docs/v2/modifiers/offset/
                 `,
-            });
-          } else if (value.includes(',')) {
-            // Split by comma
-            const offsetArray: Literal[] = expression.value
-              .split(',')
-              //@ts-ignore
-              .map((elem) => j.literal(parseInt(elem.replace(/\D/g, ''))));
-            if (offsetArray.length === 2) {
-              j(attribute).replaceWith(
-                j.jsxExpressionContainer(j.arrayExpression(offsetArray)),
-              );
-            }
-          } else {
-            // Split by space but check if it is a single number
-            const offsetArray: Literal[] = expression.value
-              .split(' ')
-              .filter((elem) => elem.length)
-              .map((elem) => j.literal(parseInt(elem.replace(/\D/g, ''))));
-            if (offsetArray.length === 2) {
-              j(attribute).replaceWith(
-                j.jsxExpressionContainer(j.arrayExpression(offsetArray)),
-              );
-            } else if (offsetArray.length === 1) {
-              j(attribute).replaceWith(
-                j.jsxExpressionContainer(
-                  j.arrayExpression([offsetArray[0], j.literal(0)]),
-                ),
-              );
-            }
-          }
-        } else if (expression.type === 'NumericLiteral') {
-          // If it is a single number convert to [number, 0]
-          j(attribute).replaceWith(
-            j.jsxExpressionContainer(
-              j.arrayExpression([expression, j.literal(0)]),
-            ),
-          );
-        } else if (expression.type === 'Identifier') {
-          // If there is a variable add this comment
-          addCommentToStartOfFile({
-            j,
-            base: source,
-            message: `
+						});
+					} else if (value.includes(',')) {
+						// Split by comma
+						const offsetArray: Literal[] = expression.value
+							.split(',')
+							//@ts-ignore
+							.map((elem) => j.literal(parseInt(elem.replace(/\D/g, ''))));
+						if (offsetArray.length === 2) {
+							j(attribute).replaceWith(j.jsxExpressionContainer(j.arrayExpression(offsetArray)));
+						}
+					} else {
+						// Split by space but check if it is a single number
+						const offsetArray: Literal[] = expression.value
+							.split(' ')
+							.filter((elem) => elem.length)
+							.map((elem) => j.literal(parseInt(elem.replace(/\D/g, ''))));
+						if (offsetArray.length === 2) {
+							j(attribute).replaceWith(j.jsxExpressionContainer(j.arrayExpression(offsetArray)));
+						} else if (offsetArray.length === 1) {
+							j(attribute).replaceWith(
+								j.jsxExpressionContainer(j.arrayExpression([offsetArray[0], j.literal(0)])),
+							);
+						}
+					}
+				} else if (expression.type === 'NumericLiteral') {
+					// If it is a single number convert to [number, 0]
+					j(attribute).replaceWith(
+						j.jsxExpressionContainer(j.arrayExpression([expression, j.literal(0)])),
+					);
+				} else if (expression.type === 'Identifier') {
+					// If there is a variable add this comment
+					addCommentToStartOfFile({
+						j,
+						base: source,
+						message: `
               Popper.js has been upgraded from 1.14.1 to 2.4.2, and as a result the offset
               prop has changed to be an array. e.g '0px 8px' -> [0, 8]
               As you are using a variable, you will have change the offset prop manually
               Further details can be found in the popper docs https://popper.js.org/docs/v2/modifiers/offset/
               `,
-          });
-        }
-      });
-  });
+					});
+				}
+			});
+	});
 }
 
-export default function transformer(
-  file: FileInfo,
-  { jscodeshift: j }: API,
-  options: Options,
-) {
-  const source = j(file.source);
+export default function transformer(file: FileInfo, { jscodeshift: j }: API, options: Options) {
+	const source = j(file.source);
 
-  // Exit early if not relevant
-  // We are doing this so we don't touch the formatting of unrelated files
-  if (!hasImportDeclaration(j, file.source, '@atlaskit/popup')) {
-    return file.source;
-  }
+	// Exit early if not relevant
+	// We are doing this so we don't touch the formatting of unrelated files
+	if (!hasImportDeclaration(j, file.source, '@atlaskit/popup')) {
+		return file.source;
+	}
 
-  // Get imported name for the component
-  var specifier = getDefaultSpecifierName({
-    j,
-    base: source,
-    packageName: '@atlaskit/popup',
-  });
-  if (!specifier) {
-    specifier = getSpecifierName({
-      j,
-      base: source,
-      packageName: '@atlaskit/popup',
-      component: 'Popup',
-    });
-  }
-  if (!specifier) {
-    return file.source;
-  }
+	// Get imported name for the component
+	var specifier = getDefaultSpecifierName({
+		j,
+		base: source,
+		packageName: '@atlaskit/popup',
+	});
+	if (!specifier) {
+		specifier = getSpecifierName({
+			j,
+			base: source,
+			packageName: '@atlaskit/popup',
+			component: 'Popup',
+		});
+	}
+	if (!specifier) {
+		return file.source;
+	}
 
-  // Convert boundaries prop
-  updateBoundariesProps(j, source, specifier);
+	// Convert boundaries prop
+	updateBoundariesProps(j, source, specifier);
 
-  // Convert offset prop
-  updateOffset(j, source, specifier);
+	// Convert offset prop
+	updateOffset(j, source, specifier);
 
-  // Rename render props
-  updateRenderProps(j, source, specifier, 'scheduleUpdate', 'update');
+	// Rename render props
+	updateRenderProps(j, source, specifier, 'scheduleUpdate', 'update');
 
-  return source.toSource(options.printOptions || { quote: 'single' });
+	return source.toSource(options.printOptions || { quote: 'single' });
 }