@planningcenter/tapestry-migration-cli 2.8.0-rc.1 → 2.8.0-rc.10

package/dist/tapestry-react-shim.cjs

@@ -2925,8 +2925,8 @@ const tokens = {
   "--t-fill-color-control-neutral-off": "hsl(0, 0%, 58%)",
   "--t-fill-color-control-neutral-on": "hsl(0, 0%, 24%)",
   "--t-fill-color-control": "hsl(204, 100%, 40%)",
-  "--t-fill-color-control-secondary": "hsl(97, 57%, 40%)",
   "--t-fill-color-control-error": "hsl(8, 60%, 47%)",
+  "--t-fill-color-control-secondary": "hsl(97, 57%, 40%)",
   "--t-fill-color-control-disabled": "hsl(0, 0%, 81%)",
   "--t-fill-color-status-neutral": "hsl(0, 0%, 42%)",
   "--t-fill-color-status-neutral-ghost": "hsl(0, 0%, 93%)",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@planningcenter/tapestry-migration-cli",
-  "version": "2.8.0-rc.1",
+  "version": "2.8.0-rc.10",
   "description": "CLI tool for Tapestry migrations",
   "type": "module",
   "main": "dist/index.js",
@@ -32,6 +32,7 @@
   },
   "devDependencies": {
     "@emotion/react": "^11.14.0",
+    "@planningcenter/tapestry": "^2.7.0",
     "@planningcenter/tapestry-react": "^4.11.5",
     "@types/jscodeshift": "^17.3.0",
     "@types/node": "^20.0.0",
@@ -51,5 +52,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "142fc65c6b21edc636e8adbd6795d410d35e7d8d"
+  "gitHead": "c9c65ff2b6027e6d3df85f166a6cc0c2abc084b7"
 }

package/src/components/checkbox/transforms/sizeMapping.test.ts

@@ -51,7 +51,7 @@ function Test() {
       )
     })
 
-    it("should not transform xs size", () => {
+    it("should transform xs to sm", () => {
       const input = `
 import { Checkbox } from "@planningcenter/tapestry-react"
 
@@ -61,8 +61,11 @@ function Test() {
       `.trim()
 
       const result = applyTransform(input)
-      expect(result).toContain('size="xs"')
-      expect(result).not.toContain("TODO: tapestry-migration")
+      expect(result).toContain('size="sm"')
+      expect(result).not.toContain('size="xs"')
+      expect(result).toContain(
+        'TODO: tapestry-migration (size): Size "xs" was mapped to "sm"'
+      )
     })
 
     it("should not transform supported sizes", () => {
@@ -74,7 +77,6 @@ function Test() {
     <div>
       <Checkbox size="md" label="Medium checkbox" />
       <Checkbox size="sm" label="Small checkbox" />
-      <Checkbox size="xs" label="Extra small checkbox" />
     </div>
   )
 }
@@ -83,7 +85,6 @@ function Test() {
       const result = applyTransform(input)
       expect(result).toContain('size="md"')
       expect(result).toContain('size="sm"')
-      expect(result).toContain('size="xs"')
       expect(result).not.toContain("TODO: tapestry-migration")
     })
   })
@@ -139,12 +140,13 @@ function Test() {
 
       const result = applyTransform(input)
       expect(result).toContain('size="md"')
-      expect(result).toContain('size="xs"')
+      expect(result).toContain('size="sm"')
       expect(result).not.toContain('size="lg"')
       expect(result).not.toContain('size="xl"')
-      // Should have 2 TODO comments (one for lg->md, one for xl->md)
+      expect(result).not.toContain('size="xs"')
+      // Should have 3 TODO comments (one for lg->md, one for xl->md, one for xs->sm)
       const todoMatches = result.match(/TODO: tapestry-migration \(size\)/g)
-      expect(todoMatches).toHaveLength(2)
+      expect(todoMatches).toHaveLength(3)
     })
 
     it("should preserve other props", () => {

package/src/components/checkbox/transforms/sizeMapping.ts

@@ -7,6 +7,7 @@ import { attributeTransformFactory } from "../../shared/transformFactories/attri
 const SIZE_MAPPING = {
   lg: "md",
   xl: "md",
+  xs: "sm",
 }
 
 const transform: Transform = attributeTransformFactory({