@planningcenter/tapestry-migration-cli 2.4.1 → 2.5.0-rc.1

package/README.md

@@ -14,25 +14,25 @@ yarn add --dev @planningcenter/tapestry-migration-cli
 
 For optimal results, migrate components in this order:
 
-1. **Button components first**: `npx @planningcenter/tapestry-migration-cli run button -p ./src/components --fix`
-2. **Link components second**: `npx @planningcenter/tapestry-migration-cli run link -p ./src/components --fix`
+1. **Button components first**: `npx @planningcenter/tapestry-migration-cli run button -p ./src/components`
+2. **Link components second**: `npx @planningcenter/tapestry-migration-cli run link -p ./src/components`
 
 This order ensures that Button components with navigation props are properly handled before Link components are migrated.
 
 ### Basic Commands
 
 ```bash
-# Run a dry-run migration (preview changes)
+# Apply the migration changes (default behavior)
 npx @planningcenter/tapestry-migration-cli run button -p ./src/components
 
-# Apply the migration changes
-npx @planningcenter/tapestry-migration-cli run button -p ./src/components --fix
+# Preview changes without writing (dry run)
+npx @planningcenter/tapestry-migration-cli run button -p ./src/components --dry-run
 
 # Run with verbose output
-npx @planningcenter/tapestry-migration-cli run button -p ./src/components --fix --verbose
+npx @planningcenter/tapestry-migration-cli run button -p ./src/components --verbose
 
 # Generate a migration report
-npx @planningcenter/tapestry-migration-cli run button -p ./src/components --fix --report-path ./migration-report.md
+npx @planningcenter/tapestry-migration-cli run button -p ./src/components --report-path ./migration-report.md
 ```
 
 ## Available Components
@@ -46,7 +46,7 @@ npx @planningcenter/tapestry-migration-cli run button -p ./src/components --fix
 
 ## Optional Arguments
 
-- `-f, --fix` - Write changes to files (without this flag, it's a dry run)
+- `-d, --dry-run` - Preview changes without writing to files (default: writes changes)
 - `-v, --verbose` - Show detailed output
 - `-j, --js-theme <path>` - Path to JavaScript theme file
 - `-r, --report-path <path>` - Path for migration report (default: MIGRATION_REPORT.md)
@@ -101,6 +101,6 @@ yarn add --dev @planningcenter/tapestry-migration-cli
 }
 
 # Then run (following recommended order)
-yarn migrate run button -p app/javascript/components --fix
-yarn migrate run link -p app/javascript/components --fix
+yarn migrate run button -p app/javascript/components
+yarn migrate run link -p app/javascript/components
 ```

package/dist/tapestry-react-shim.cjs

@@ -2571,6 +2571,7 @@ const COMPONENT_KIND_CLASS_MAP = {
   "ghost-interaction": "tds-btn--ghost-interaction",
   "inline-text": "tds-btn--inline-text",
   neutral: "tds-btn--neutral",
+  "neutral-inline": "tds-btn--neutral-inline",
   pill: "tds-btn--pill",
   primary: "tds-btn--interaction",
   "primary-page-header": "tds-btn--primary-page-header",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@planningcenter/tapestry-migration-cli",
-  "version": "2.4.1",
+  "version": "2.5.0-rc.1",
   "description": "CLI tool for Tapestry migrations",
   "type": "module",
   "main": "dist/index.js",
@@ -51,5 +51,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "465fa2ffd3103c546f66e368859061daf3c7b6c1"
+  "gitHead": "e5ff2455e4766e1a91998ab096ea8c77f00f9bae"
 }

package/src/index.ts

@@ -23,7 +23,7 @@ program
     "-p, --path <path>",
     "REQUIRED: The path to the folder/file to migrate"
   )
-  .option("-f, --fix", "Write the changes to files (default: dry run)")
+  .option("-d, --dry-run", "Preview changes without writing to files")
   .option("-v, --verbose", "Verbose output")
   .option(
     "-j, --js-theme <path>",
@@ -31,7 +31,7 @@ program
   )
   .option(
     "-r, --report-path <path>",
-    "Path for the migration report. Only runs with --fix and does not create a report if no additional changes are required.",
+    "Path for the migration report. Creates a report when running with changes applied and does not create a report if no additional changes are required.",
     "MIGRATION_REPORT.md"
   )
   .action((componentName, options) => {
@@ -41,7 +41,7 @@ program
       console.log(`🎨 Migrating ${componentName} components...`)
       console.log(`📁 Target: ${options.path}`)
       console.log(
-        `🔧 Mode: ${options.fix ? "Apply changes" : "Dry run (preview only)"}`
+        `🔧 Mode: ${options.dryRun ? "Dry run (preview only)" : "Apply changes"}`
       )
       runTransforms(key, options)
     } else {

package/src/jscodeshiftRunner.ts

@@ -34,7 +34,7 @@ export function runTransforms(key: string, options: Options): void {
       "-t",
       transformPath,
       targetPath,
-      options.fix ? "" : "--dry",
+      options.dryRun ? "--dry" : "",
       options.verbose ? "--verbose=2" : "",
       "--parser=tsx",
     ]

package/src/shared/types.ts

@@ -1,5 +1,5 @@
 export interface Options {
-  fix?: boolean
+  dryRun?: boolean
   jsTheme?: string
   path: string
   reportPath?: string