@planningcenter/tapestry-migration-cli 4.10.0 → 4.10.1-qa-1450.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@planningcenter/tapestry-migration-cli",
-  "version": "4.10.0",
+  "version": "4.10.1-qa-1450.0",
   "description": "CLI tool for Tapestry migrations",
   "type": "module",
   "main": "dist/index.js",
@@ -31,7 +31,7 @@
   "devDependencies": {
     "@emotion/react": "^11.14.0",
     "@planningcenter/tapestry-react": "^4.11.5",
-    "@planningcenter/tapestry-tokens": "^4.4.0",
+    "@planningcenter/tapestry-tokens": "^4.10.1-qa-1450.0",
     "@types/jscodeshift": "^17.3.0",
     "@types/node": "^20.0.0",
     "typescript": "^5.8.3",
@@ -50,5 +50,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "3457b28c41e489b5e2fce83c8afa2e67b8df54ed"
+  "gitHead": "c2f44c24a37c0284f83986a1fb540cb24bdbb425"
 }

package/src/index.test.ts

@@ -13,67 +13,80 @@ import { describe, expect, it } from "vitest"
 import { buildMigrationList } from "./migrationList"
 
 const CLI_PATH = fileURLToPath(new URL("./index.ts", import.meta.url))
+// tsx cold-start can take a few seconds; allow up to 30 s.
+const CLI_TIMEOUT_MS = 30_000
 
 function runCli(args: string[]) {
   return spawnSync("npx", ["tsx", CLI_PATH, ...args], {
     encoding: "utf8",
-    // tsx cold-start can take a few seconds; allow up to 30 s.
-    timeout: 30_000,
+    timeout: CLI_TIMEOUT_MS,
   })
 }
 
-describe("list --json (stdout/exit-code contract)", () => {
-  it("exits with code 0", () => {
-    const result = runCli(["list", "--json"])
-    expect(result.status).toBe(0)
-  })
+describe(
+  "list --json (stdout/exit-code contract)",
+  { timeout: CLI_TIMEOUT_MS },
+  () => {
+    it("exits with code 0", () => {
+      const result = runCli(["list", "--json"])
+      expect(result.status).toBe(0)
+    })
 
-  it("emits valid JSON as the entire stdout (no stray lines)", () => {
-    const result = runCli(["list", "--json"])
-    // JSON.parse throws if there is ANY non-JSON content on stdout.
-    expect(() => JSON.parse(result.stdout)).not.toThrow()
-  })
+    it("emits valid JSON as the entire stdout (no stray lines)", () => {
+      const result = runCli(["list", "--json"])
+      // JSON.parse throws if there is ANY non-JSON content on stdout.
+      expect(() => JSON.parse(result.stdout)).not.toThrow()
+    })
 
-  it("payload matches buildMigrationList() (schemaVersion + migrations)", () => {
-    const result = runCli(["list", "--json"])
-    const payload = JSON.parse(result.stdout)
-    expect(payload).toEqual(buildMigrationList())
-  })
+    it("payload matches buildMigrationList() (schemaVersion + migrations)", () => {
+      const result = runCli(["list", "--json"])
+      const payload = JSON.parse(result.stdout)
+      expect(payload).toEqual(buildMigrationList())
+    })
 
-  it("schemaVersion is exactly the integer 1", () => {
-    const result = runCli(["list", "--json"])
-    const { schemaVersion } = JSON.parse(result.stdout)
-    expect(schemaVersion).toBe(1)
-    expect(Number.isInteger(schemaVersion)).toBe(true)
-  })
+    it("schemaVersion is exactly the integer 1", () => {
+      const result = runCli(["list", "--json"])
+      const { schemaVersion } = JSON.parse(result.stdout)
+      expect(schemaVersion).toBe(1)
+      expect(Number.isInteger(schemaVersion)).toBe(true)
+    })
 
-  it("stderr is empty on success", () => {
-    const result = runCli(["list", "--json"])
-    expect(result.stderr).toBe("")
-  })
+    it("stderr is empty on success", () => {
+      const result = runCli(["list", "--json"])
+      expect(result.stderr).toBe("")
+    })
 
-  it("does not require --path and makes no filesystem writes", () => {
-    // If --path were required, commander would write an error to stderr and exit 1.
-    // We assert the happy path — exit 0 with no --path — proving the flag is absent.
-    const result = runCli(["list", "--json"])
-    expect(result.status).toBe(0)
-  })
-})
+    it("does not require --path and makes no filesystem writes", () => {
+      // If --path were required, commander would write an error to stderr and exit 1.
+      // We assert the happy path — exit 0 with no --path — proving the flag is absent.
+      const result = runCli(["list", "--json"])
+      expect(result.status).toBe(0)
+    })
+  }
+)
 
-describe("list (human-readable, non-contractual)", () => {
-  it("exits with code 0 and writes component ids to stdout", () => {
-    const result = runCli(["list"])
-    expect(result.status).toBe(0)
-    expect(result.stdout).toContain("button")
-    expect(result.stdout).toContain("toggle-switch")
-  })
-})
+describe(
+  "list (human-readable, non-contractual)",
+  { timeout: CLI_TIMEOUT_MS },
+  () => {
+    it("exits with code 0 and writes component ids to stdout", () => {
+      const result = runCli(["list"])
+      expect(result.status).toBe(0)
+      expect(result.stdout).toContain("button")
+      expect(result.stdout).toContain("toggle-switch")
+    })
+  }
+)
 
-describe("diagnostics contract (nonzero exit → stderr)", () => {
-  it("exits nonzero and writes to stderr for an unknown subcommand", () => {
-    const result = runCli(["unknown-subcommand"])
-    // Commander writes "error: unknown command 'unknown-subcommand'" to stderr.
-    expect(result.status).not.toBe(0)
-    expect(result.stderr.length).toBeGreaterThan(0)
-  })
-})
+describe(
+  "diagnostics contract (nonzero exit → stderr)",
+  { timeout: CLI_TIMEOUT_MS },
+  () => {
+    it("exits nonzero and writes to stderr for an unknown subcommand", () => {
+      const result = runCli(["unknown-subcommand"])
+      // Commander writes "error: unknown command 'unknown-subcommand'" to stderr.
+      expect(result.status).not.toBe(0)
+      expect(result.stderr.length).toBeGreaterThan(0)
+    })
+  }
+)