@planningcenter/tapestry-migration-cli 3.2.3-rc.4 → 3.2.3-rc.6

package/src/components/time-field/transforms/unsupportedProps.ts

@@ -1,37 +1,10 @@
-import { JSXAttribute, Transform } from "jscodeshift"
-
-import { addCommentToUnsupportedProps } from "../../shared/actions/addCommentToUnsupportedProps"
 import { TIME_FIELD_SUPPORTED_PROPS } from "../../shared/helpers/unsupportedPropsHelpers"
-import { attributeTransformFactory } from "../../shared/transformFactories/attributeTransformFactory"
+import { unsupportedPropsFactory } from "../../shared/transformFactories/unsupportedPropsFactory"
 
-const transform: Transform = attributeTransformFactory({
+const transform = unsupportedPropsFactory({
+  supportedProps: TIME_FIELD_SUPPORTED_PROPS,
   targetComponent: "TimeField",
   targetPackage: "@planningcenter/tapestry-react",
-  transform: (element, { j }) => {
-    const attrs = element.openingElement.attributes || []
-
-    const UNSUPPORTED_PROPS = attrs
-      .filter(
-        (attr) =>
-          attr.type === "JSXAttribute" &&
-          !TIME_FIELD_SUPPORTED_PROPS.includes(attr.name.name as string) &&
-          !(attr.name.name as string).startsWith("aria-") &&
-          !(attr.name.name as string).startsWith("data-")
-      )
-      .map((attr) => (attr as JSXAttribute).name.name as string)
-
-    return addCommentToUnsupportedProps({
-      element,
-      j,
-      messageSuffix: (prop) => {
-        if (prop === "css") {
-          return "\n * CSS prop is not supported. Use className or style prop instead.\n"
-        }
-        return ""
-      },
-      props: UNSUPPORTED_PROPS,
-    })
-  },
 })
 
 export default transform

package/src/components/toggle-switch/transforms/unsupportedProps.ts

@@ -1,8 +1,5 @@
-import { JSXAttribute, Transform } from "jscodeshift"
-
-import { addCommentToUnsupportedProps } from "../../shared/actions/addCommentToUnsupportedProps"
 import { CHECKBOX_RADIO_SUPPORTED_PROPS } from "../../shared/helpers/unsupportedPropsHelpers"
-import { attributeTransformFactory } from "../../shared/transformFactories/attributeTransformFactory"
+import { unsupportedPropsFactory } from "../../shared/transformFactories/unsupportedPropsFactory"
 
 const TOGGLE_SWITCH_SPECIFIC_PROPS = ["hideLabel"]
 
@@ -11,32 +8,10 @@ const SUPPORTED_PROPS = [
   ...TOGGLE_SWITCH_SPECIFIC_PROPS,
 ]
 
-const transform: Transform = attributeTransformFactory({
+const transform = unsupportedPropsFactory({
+  supportedProps: SUPPORTED_PROPS,
   targetComponent: "ToggleSwitch",
   targetPackage: "@planningcenter/tapestry-react",
-  transform: (element, { j }) => {
-    const UNSUPPORTED_PROPS = (element.openingElement.attributes || [])
-      .filter(
-        (attr) =>
-          attr.type === "JSXAttribute" &&
-          !SUPPORTED_PROPS.includes(attr.name.name as string) &&
-          !(attr.name.name as string).startsWith("aria-") &&
-          !(attr.name.name as string).startsWith("data-")
-      )
-      .map((attr) => (attr as JSXAttribute).name.name as string)
-
-    return addCommentToUnsupportedProps({
-      element,
-      j,
-      messageSuffix: (prop) => {
-        if (prop === "css") {
-          return "\n * CSS prop is not supported. Use className or style prop instead.\n"
-        }
-        return ""
-      },
-      props: UNSUPPORTED_PROPS,
-    })
-  },
 })
 
 export default transform

package/src/index.ts

@@ -14,6 +14,7 @@ program
 const COMPONENTS_SET = new Set([
   "button",
   "checkbox",
+  "date-picker",
   "input",
   "link",
   "radio",
@@ -23,12 +24,17 @@ const COMPONENTS_SET = new Set([
   "toggle-switch",
 ])
 
+const COMPONENTS_SENTENCE = new Intl.ListFormat("en", {
+  style: "long",
+  type: "conjunction",
+}).format([...COMPONENTS_SET])
+
 program
   .command("run")
   .description("Run a migration of a component from Tapestry React to Tapestry")
   .argument(
     "<component-name>",
-    "The name of the component to migrate (button, checkbox, input, link, radio, select, text-area, time-field, toggle-switch)"
+    `The name of the component to migrate (${COMPONENTS_SENTENCE})`
   )
   .requiredOption(
     "-p, --path <path>",