@atlaskit/section-message 6.5.2 → 6.5.4

package/codemods/__tests__/map-actions-prop.tsx

@@ -7,11 +7,11 @@ const transformer = createTransformer([mapActionsProp]);
 const defineInlineTest = require('jscodeshift/dist/testUtils').defineInlineTest;
 
 describe('map actions prop to components', () => {
-  describe('for default & named imports', () => {
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+	describe('for default & named imports', () => {
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
       import SectionMessage from "@atlaskit/section-message";
 
@@ -28,7 +28,7 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      `
+			`
       import React from "react";
       import SectionMessage, { SectionMessageAction } from "@atlaskit/section-message";
 
@@ -50,13 +50,13 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      'should map actions correctly',
-    );
+			'should map actions correctly',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps } from "@atlaskit/section-message";
 
@@ -71,7 +71,7 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      `
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps, SectionMessageAction } from "@atlaskit/section-message";
 
@@ -91,13 +91,13 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      'should map actions correctly if defined inline',
-    );
+			'should map actions correctly if defined inline',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps } from "@atlaskit/section-message";
 
@@ -118,7 +118,7 @@ describe('map actions prop to components', () => {
 
       export default Component;
       `,
-      `
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps, SectionMessageAction as AtlaskitSectionMessageAction } from "@atlaskit/section-message";
 
@@ -144,13 +144,13 @@ describe('map actions prop to components', () => {
 
       export default Component;
       `,
-      'should map actions correctly - with alias',
-    );
+			'should map actions correctly - with alias',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
       import SectionMessage from "@atlaskit/section-message";
 
@@ -165,7 +165,7 @@ describe('map actions prop to components', () => {
         <SectionMessage />
       );
       `,
-      `
+			`
       import React from "react";
       import SectionMessage from "@atlaskit/section-message";
 
@@ -180,13 +180,13 @@ describe('map actions prop to components', () => {
         <SectionMessage />
       );
       `,
-      'should only map when "actions" prop is defined',
-    );
+			'should only map when "actions" prop is defined',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
       import SectionMessage from "@atlaskit/section-message";
 
@@ -203,7 +203,7 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      `
+			`
       import React from "react";
       import SectionMessage, { SectionMessageAction } from "@atlaskit/section-message";
 
@@ -225,13 +225,13 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      'should map actions correctly if result coming from a function call',
-    );
+			'should map actions correctly if result coming from a function call',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps } from "@atlaskit/section-message";
 
@@ -246,7 +246,7 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      `
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps, SectionMessageAction } from "@atlaskit/section-message";
 
@@ -266,13 +266,13 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      'should map actions correctly if defined inline based on a condition',
-    );
+			'should map actions correctly if defined inline based on a condition',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps } from "@atlaskit/section-message";
 
@@ -290,7 +290,7 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      `
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps, SectionMessageAction } from "@atlaskit/section-message";
 
@@ -311,13 +311,13 @@ describe('map actions prop to components', () => {
           ) => <SectionMessageAction linkComponent={CustomLink} {...restAction}>{text}</SectionMessageAction>)} />
       );
       `,
-      'should move "linkComponent" from "SectionMessage" to "SectionMessageAction" if pointing to a component',
-    );
+			'should move "linkComponent" from "SectionMessage" to "SectionMessageAction" if pointing to a component',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps } from "@atlaskit/section-message";
 
@@ -333,7 +333,7 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      `
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps, SectionMessageAction } from "@atlaskit/section-message";
 
@@ -352,13 +352,13 @@ describe('map actions prop to components', () => {
           ) => <SectionMessageAction linkComponent={() => <a></a>} {...restAction}>{text}</SectionMessageAction>)} />
       );
       `,
-      'should move "linkComponent" from "SectionMessage" to "SectionMessageAction" if component is defined inline',
-    );
+			'should move "linkComponent" from "SectionMessage" to "SectionMessageAction" if component is defined inline',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps } from "@atlaskit/section-message";
 
@@ -370,7 +370,7 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      `
+			`
       import React from "react";
       import SectionMessage, { SectionMessageProps } from "@atlaskit/section-message";
 
@@ -380,15 +380,15 @@ describe('map actions prop to components', () => {
         <SectionMessage />
       );
       `,
-      'should remove "linkComponent" from "SectionMessage" if no actions are present',
-    );
-  });
-
-  describe('for dynamic imports', () => {
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+			'should remove "linkComponent" from "SectionMessage" if no actions are present',
+		);
+	});
+
+	describe('for dynamic imports', () => {
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
 
       const SectionMessageDynamicImport = React.lazy(() => import('@atlaskit/section-message'));
@@ -406,7 +406,7 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      `
+			`
       import React from "react";
 
       const SectionMessageDynamicImport = React.lazy(() => import('@atlaskit/section-message'));
@@ -432,13 +432,13 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      'should map actions correctly',
-    );
+			'should map actions correctly',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
 
       const SectionMessageDynamicImport = React.lazy(() => import('@atlaskit/section-message'));
@@ -454,7 +454,7 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      `
+			`
       import React from "react";
 
       const SectionMessageDynamicImport = React.lazy(() => import('@atlaskit/section-message'));
@@ -478,13 +478,13 @@ describe('map actions prop to components', () => {
         />
       );
       `,
-      'should map actions correctly if defined inline',
-    );
+			'should map actions correctly if defined inline',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
 
       const SectionMessageDynamicImport = React.lazy(() => import('@atlaskit/section-message'));
@@ -506,7 +506,7 @@ describe('map actions prop to components', () => {
 
       export default Component;
       `,
-      `
+			`
       import React from "react";
 
       const SectionMessageDynamicImport = React.lazy(() => import('@atlaskit/section-message'));
@@ -536,13 +536,13 @@ describe('map actions prop to components', () => {
 
       export default Component;
       `,
-      'should map actions correctly - with alias',
-    );
+			'should map actions correctly - with alias',
+		);
 
-    defineInlineTest(
-      { default: transformer, parser: 'tsx' },
-      {},
-      `
+		defineInlineTest(
+			{ default: transformer, parser: 'tsx' },
+			{},
+			`
       import React from "react";
 
       const SectionMessageDynamicImport = React.lazy(() => import('@atlaskit/section-message'));
@@ -558,7 +558,7 @@ describe('map actions prop to components', () => {
         <SectionMessageDynamicImport />
       );
       `,
-      `
+			`
       import React from "react";
 
       const SectionMessageDynamicImport = React.lazy(() => import('@atlaskit/section-message'));
@@ -574,7 +574,7 @@ describe('map actions prop to components', () => {
         <SectionMessageDynamicImport />
       );
       `,
-      'should only map when "actions" prop is defined',
-    );
-  });
+			'should only map when "actions" prop is defined',
+		);
+	});
 });

package/codemods/internal/change-appearance-prop.tsx

@@ -1,121 +1,101 @@
 import {
-  type ASTPath,
-  type default as core,
-  type JSXAttribute,
-  type JSXElement,
+	type ASTPath,
+	type default as core,
+	type JSXAttribute,
+	type JSXElement,
 } from 'jscodeshift';
 import { type Collection } from 'jscodeshift/src/Collection';
 
 import {
-  addCommentBefore,
-  getDefaultSpecifier,
-  getJSXAttributesByName,
+	addCommentBefore,
+	getDefaultSpecifier,
+	getJSXAttributesByName,
 } from '@atlaskit/codemod-utils';
 
 import {
-  APPEARANCE_OLD_TO_NEW_MAPPING,
-  APPEARANCE_PROP_NAME,
-  SECTION_MESSAGE_PACKAGE_NAME,
+	APPEARANCE_OLD_TO_NEW_MAPPING,
+	APPEARANCE_PROP_NAME,
+	SECTION_MESSAGE_PACKAGE_NAME,
 } from './constants';
 
-export const changeAppearanceProp = (
-  j: core.JSCodeshift,
-  source: Collection<Node>,
-) => {
-  const defaultSpecifier = getDefaultSpecifier(
-    j,
-    source,
-    SECTION_MESSAGE_PACKAGE_NAME,
-  );
+export const changeAppearanceProp = (j: core.JSCodeshift, source: Collection<Node>) => {
+	const defaultSpecifier = getDefaultSpecifier(j, source, SECTION_MESSAGE_PACKAGE_NAME);
 
-  if (!defaultSpecifier) {
-    return;
-  }
+	if (!defaultSpecifier) {
+		return;
+	}
 
-  source
-    .findJSXElements(defaultSpecifier)
-    .forEach((element: ASTPath<JSXElement>) => {
-      getJSXAttributesByName(j, element, APPEARANCE_PROP_NAME).forEach(
-        (attribute: ASTPath<JSXAttribute>) => {
-          const { value } = attribute.node;
-          if (!value) {
-            return;
-          }
-          // appearance prop can be provided in multiple ways. Handling different cases here
-          switch (value.type) {
-            // case when object value is provided
-            case 'JSXExpressionContainer':
-              const { expression } = value;
+	source.findJSXElements(defaultSpecifier).forEach((element: ASTPath<JSXElement>) => {
+		getJSXAttributesByName(j, element, APPEARANCE_PROP_NAME).forEach(
+			(attribute: ASTPath<JSXAttribute>) => {
+				const { value } = attribute.node;
+				if (!value) {
+					return;
+				}
+				// appearance prop can be provided in multiple ways. Handling different cases here
+				switch (value.type) {
+					// case when object value is provided
+					case 'JSXExpressionContainer':
+						const { expression } = value;
 
-              // case when string is provided inside JSX expression
-              // e.g.: <SectionMessage appearance={"information"} />
-              if (expression.type === 'StringLiteral') {
-                replaceAppearanceStringValue(j, attribute, expression.value);
-              }
-              // case when a variable is provided as value
-              // e.g.: <SectionMessage appearance={someVariable} />
-              else if (expression.type !== 'JSXEmptyExpression') {
-                const mappingValue = j.memberExpression(
-                  j.objectExpression(
-                    Object.entries(APPEARANCE_OLD_TO_NEW_MAPPING).map(
-                      ([key, value]) =>
-                        j.objectProperty(
-                          j.identifier(key),
-                          j.stringLiteral(value),
-                        ),
-                    ),
-                  ),
-                  expression,
-                );
-                mappingValue.computed = true;
+						// case when string is provided inside JSX expression
+						// e.g.: <SectionMessage appearance={"information"} />
+						if (expression.type === 'StringLiteral') {
+							replaceAppearanceStringValue(j, attribute, expression.value);
+						}
+						// case when a variable is provided as value
+						// e.g.: <SectionMessage appearance={someVariable} />
+						else if (expression.type !== 'JSXEmptyExpression') {
+							const mappingValue = j.memberExpression(
+								j.objectExpression(
+									Object.entries(APPEARANCE_OLD_TO_NEW_MAPPING).map(([key, value]) =>
+										j.objectProperty(j.identifier(key), j.stringLiteral(value)),
+									),
+								),
+								expression,
+							);
+							mappingValue.computed = true;
 
-                const propValue = j.logicalExpression(
-                  '||',
-                  mappingValue,
-                  expression,
-                );
+							const propValue = j.logicalExpression('||', mappingValue, expression);
 
-                j(attribute).replaceWith(
-                  j.jsxAttribute(
-                    j.jsxIdentifier(APPEARANCE_PROP_NAME),
-                    j.jsxExpressionContainer(propValue),
-                  ),
-                );
+							j(attribute).replaceWith(
+								j.jsxAttribute(
+									j.jsxIdentifier(APPEARANCE_PROP_NAME),
+									j.jsxExpressionContainer(propValue),
+								),
+							);
 
-                addCommentBefore(
-                  j,
-                  j(attribute),
-                  `We have added this temporary appearance mapping here to make things work. Feel free to change it accordingly. We have also added @ts-ignore for typescript files.
+							addCommentBefore(
+								j,
+								j(attribute),
+								`We have added this temporary appearance mapping here to make things work. Feel free to change it accordingly. We have also added @ts-ignore for typescript files.
                    @ts-ignore
                   `,
-                );
-              }
+							);
+						}
 
-              break;
+						break;
 
-            // case when string value is provided
-            // e.g.: <SectionMessage appearance="information" />
-            case 'StringLiteral':
-              replaceAppearanceStringValue(j, attribute, value.value);
-          }
-        },
-      );
-    });
+					// case when string value is provided
+					// e.g.: <SectionMessage appearance="information" />
+					case 'StringLiteral':
+						replaceAppearanceStringValue(j, attribute, value.value);
+				}
+			},
+		);
+	});
 };
 
 const replaceAppearanceStringValue = (
-  j: core.JSCodeshift,
-  attribute: core.ASTPath<JSXAttribute>,
-  propValue: string,
+	j: core.JSCodeshift,
+	attribute: core.ASTPath<JSXAttribute>,
+	propValue: string,
 ) => {
-  const newPropValue: string = APPEARANCE_OLD_TO_NEW_MAPPING[propValue];
+	const newPropValue: string = APPEARANCE_OLD_TO_NEW_MAPPING[propValue];
 
-  if (newPropValue) {
-    j(attribute).replaceWith(
-      j.jsxAttribute(
-        j.jsxIdentifier(APPEARANCE_PROP_NAME),
-        j.stringLiteral(newPropValue),
-      ),
-    );
-  }
+	if (newPropValue) {
+		j(attribute).replaceWith(
+			j.jsxAttribute(j.jsxIdentifier(APPEARANCE_PROP_NAME), j.stringLiteral(newPropValue)),
+		);
+	}
 };

package/codemods/internal/constants.tsx

@@ -1,11 +1,11 @@
 export const SECTION_MESSAGE_PACKAGE_NAME = '@atlaskit/section-message';
 export const APPEARANCE_PROP_NAME = 'appearance';
 export const APPEARANCE_OLD_TO_NEW_MAPPING: {
-  [index: string]: string;
+	[index: string]: string;
 } = {
-  info: 'information',
-  confirmation: 'success',
-  change: 'discovery',
+	info: 'information',
+	confirmation: 'success',
+	change: 'discovery',
 };
 export const SECTION_MESSAGE_ACTION_PACKAGE_NAME = `${SECTION_MESSAGE_PACKAGE_NAME}/section-message-action`;
 export const SECTION_MESSAGE_ACTION_COMPONENT_NAME = 'SectionMessageAction';