feat(migrate): add multi dirs support for tsup migration script

lauigi · lauigi · commit 159ed8c24ce4 · 2026-01-05T13:45:12.000-05:00
Resolves #452
diff --git a/docs/guide/migrate-from-tsup.md b/docs/guide/migrate-from-tsup.md
@@ -10,14 +10,29 @@ If you're currently using `tsup` and want to migrate to `tsdown`, the process is
 npx tsdown-migrate
 ```
 
+For monorepos, you can specify directories using glob patterns:
+
+```bash
+npx tsdown-migrate packages/*
+```
+
+Or specify multiple directories explicitly:
+
+```bash
+npx tsdown-migrate packages/foo packages/bar
+```
+
 > [!WARNING]
 > Please save your changes before migration. The migration process may modify your configuration files, so it's important to ensure all your changes are committed or backed up beforehand.
 
+> [!TIP]
+> The migration tool will automatically install dependencies after migration. Make sure to run the command from within your project directory.
+
 ### Migration Options
 
 The `migrate` command supports the following options to customize the migration process:
 
-- `--cwd <dir>` (or `-c`): Specify the working directory for the migration.
+- `[...dirs]`: Specify directories to migrate. Supports glob patterns (e.g., `packages/*`). Defaults to the current directory if not specified.
 - `--dry-run` (or `-d`): Perform a dry run to preview the migration without making any changes.
 
 With these options, you can easily tailor the migration process to fit your specific project setup.
diff --git a/packages/migrate/package.json b/packages/migrate/package.json
@@ -51,6 +51,7 @@
     "ansis": "catalog:prod",
     "cac": "catalog:prod",
     "consola": "catalog:migrate",
-    "diff": "catalog:migrate"
+    "diff": "catalog:migrate",
+    "tinyglobby": "catalog:prod"
   }
 }
diff --git a/packages/migrate/src/cli.ts b/packages/migrate/src/cli.ts
@@ -7,13 +7,11 @@ import { migrate, type MigrateOptions } from './index.ts'
 const cli = cac(pkg.name).version(pkg.version).help()
 
 cli
-  .command('', 'Migrate a project to tsdown')
-  .option(
-    '-c, --cwd <path>',
-    'Current working directory to run the migration in',
-  )
+  .command('[...dirs]', 'Migrate a project to tsdown')
   .option('-d, --dry-run', 'Perform a dry run without making changes')
-  .action((options: MigrateOptions) => migrate(options))
+  .action((dirs: string[], options: Omit<MigrateOptions, 'dirs'>) =>
+    migrate({ ...options, dirs }),
+  )
 
 export async function runCLI(): Promise<void> {
   cli.parse(process.argv, { run: false })
diff --git a/packages/migrate/src/index.ts b/packages/migrate/src/index.ts
@@ -1,16 +1,18 @@
+import path from 'node:path'
 import process from 'node:process'
 import { getCliCommand, parseNi, run } from '@antfu/ni'
-import { green, underline } from 'ansis'
+import { green, greenBright, underline } from 'ansis'
 import consola from 'consola'
+import { glob } from 'tinyglobby'
 import { migratePackageJson } from './helpers/package-json.ts'
 import { migrateTsupConfig } from './helpers/tsup-config.ts'
 
 export interface MigrateOptions {
-  cwd?: string
+  dirs?: string[]
   dryRun?: boolean
 }
 
-export async function migrate({ cwd, dryRun }: MigrateOptions): Promise<void> {
+export async function migrate({ dirs, dryRun }: MigrateOptions): Promise<void> {
   if (dryRun) {
     consola.info('Dry run enabled. No changes were made.')
   } else {
@@ -27,13 +29,52 @@ export async function migrate({ cwd, dryRun }: MigrateOptions): Promise<void> {
     }
   }
 
-  if (cwd) process.chdir(cwd)
+  const baseCwd = process.cwd()
+  let cwds: string[]
+  if (!dirs || dirs.length === 0) {
+    cwds = [baseCwd]
+  } else {
+    cwds = await glob(dirs, {
+      cwd: baseCwd,
+      onlyDirectories: true,
+      absolute: true,
+      expandDirectories: false,
+    })
+    if (cwds.length === 0) {
+      consola.error(`No directories matched: ${dirs.join(', ')}`)
+      process.exitCode = 1
+      return
+    }
+  }
+
+  let migratedAny = false
 
-  let migrated = await migratePackageJson(dryRun)
-  if (await migrateTsupConfig(dryRun)) {
-    migrated = true
+  try {
+    for (const dir of cwds) {
+      process.chdir(dir)
+
+      const relativeDirLabel = greenBright(
+        path.relative(baseCwd, dir) || 'the current directory',
+      )
+      consola.info(`Processing ${relativeDirLabel}`)
+
+      let migrated = await migratePackageJson(dryRun)
+      if (await migrateTsupConfig(dryRun)) {
+        migrated = true
+      }
+
+      if (!migrated) {
+        consola.info(`No migrations to apply in ${relativeDirLabel}.`)
+        continue
+      }
+
+      migratedAny = true
+    }
+  } finally {
+    process.chdir(baseCwd)
   }
-  if (!migrated) {
+
+  if (!migratedAny) {
     consola.error('No migration performed.')
     process.exitCode = 1
     return
@@ -44,7 +85,7 @@ export async function migrate({ cwd, dryRun }: MigrateOptions): Promise<void> {
   if (dryRun) {
     consola.info('[dry-run] would run:', await getCliCommand(parseNi, []))
   } else {
-    await run(parseNi, [], { cwd })
+    await run(parseNi, [], { cwd: baseCwd })
     consola.success('Dependencies installed.')
   }
 }
diff --git a/packages/migrate/tests/fixtures/issue-452/apps/pkg3/package.json b/packages/migrate/tests/fixtures/issue-452/apps/pkg3/package.json
@@ -0,0 +1,12 @@
+{
+  "name": "standalone-pkg",
+  "version": "0.0.0",
+  "type": "module",
+  "packageManager": "pnpm@10.26.0",
+  "scripts": {
+    "build": "tsup src/index.ts"
+  },
+  "dependencies": {
+    "tsup": "^7.2.0"
+  }
+}
diff --git a/packages/migrate/tests/fixtures/issue-452/package.json b/packages/migrate/tests/fixtures/issue-452/package.json
@@ -0,0 +1,8 @@
+{
+  "name": "issue-452",
+  "version": "0.0.0",
+  "private": true,
+  "scripts": {
+    "build": "pnpm --filter \"*\" build"
+  }
+}
diff --git a/packages/migrate/tests/fixtures/issue-452/packages/pkg1/package.json b/packages/migrate/tests/fixtures/issue-452/packages/pkg1/package.json
@@ -0,0 +1,16 @@
+{
+  "name": "pkg-app",
+  "version": "1.0.0",
+  "private": true,
+  "type": "module",
+  "main": "./src/index.ts",
+  "dependencies": {
+    "pkg2": "workspace:*"
+  },
+  "devDependencies": {
+    "tsup": "catalog:"
+  },
+  "scripts": {
+    "build": "tsup"
+  }
+}
diff --git a/packages/migrate/tests/fixtures/issue-452/packages/pkg1/src/index.ts b/packages/migrate/tests/fixtures/issue-452/packages/pkg1/src/index.ts
@@ -0,0 +1,3 @@
+import { add } from 'pkg2'
+
+export const answer: number = add(40, 2)
diff --git a/packages/migrate/tests/fixtures/issue-452/packages/pkg1/tsup.config.ts b/packages/migrate/tests/fixtures/issue-452/packages/pkg1/tsup.config.ts
@@ -0,0 +1,7 @@
+import { defineConfig } from 'tsup'
+
+export default defineConfig({
+  entry: ['src/index.ts'],
+  format: ['esm'],
+  dts: true,
+})
diff --git a/packages/migrate/tests/fixtures/issue-452/packages/pkg2/package.json b/packages/migrate/tests/fixtures/issue-452/packages/pkg2/package.json
@@ -0,0 +1,21 @@
+{
+  "name": "pkg2",
+  "version": "1.0.0",
+  "private": true,
+  "type": "module",
+  "main": "./src/index.ts",
+  "exports": {
+    ".": "./src/index.ts"
+  },
+  "devDependencies": {
+    "tsup": "catalog:"
+  },
+  "scripts": {
+    "build": "tsup"
+  },
+  "tsup": {
+    "entry": [
+      "src/index.ts"
+    ]
+  }
+}
diff --git a/packages/migrate/tests/fixtures/issue-452/packages/pkg2/src/index.ts b/packages/migrate/tests/fixtures/issue-452/packages/pkg2/src/index.ts
@@ -0,0 +1 @@
+export const add = (a: number, b: number) => a + b
diff --git a/packages/migrate/tests/fixtures/issue-452/pnpm-workspace.yaml b/packages/migrate/tests/fixtures/issue-452/pnpm-workspace.yaml
@@ -0,0 +1,6 @@
+packages:
+  - apps/*
+  - packages/*
+
+catalog:
+  tsup: 8.5.1
diff --git a/packages/migrate/tests/issues.test.ts b/packages/migrate/tests/issues.test.ts
@@ -0,0 +1,110 @@
+import { realpathSync } from 'node:fs'
+import { join } from 'node:path'
+import process from 'node:process'
+import { fileURLToPath } from 'node:url'
+import { afterEach, describe, expect, test, vi } from 'vitest'
+
+const packageJsonCwds = vi.hoisted(() => [] as string[])
+const tsupConfigCwds = vi.hoisted(() => [] as string[])
+
+vi.mock('../src/helpers/package-json.ts', () => ({
+  migratePackageJson: vi.fn(() => {
+    packageJsonCwds.push(process.cwd())
+    return true
+  }),
+}))
+
+vi.mock('../src/helpers/tsup-config.ts', () => ({
+  migrateTsupConfig: vi.fn(() => {
+    tsupConfigCwds.push(process.cwd())
+    return true
+  }),
+}))
+
+describe('issue-452', () => {
+  const fixturesDir = fileURLToPath(new URL('.', import.meta.url))
+  const monorepoFixture = join(fixturesDir, 'fixtures/issue-452')
+  const originalCwd = process.cwd()
+
+  afterEach(() => {
+    packageJsonCwds.length = 0
+    tsupConfigCwds.length = 0
+    process.chdir(originalCwd)
+    vi.clearAllMocks()
+  })
+
+  test('migrates current directory when no dirs provided', async () => {
+    vi.resetModules()
+    const { migrate } = await import('../src/index.ts')
+
+    const pkg1Dir = realpathSync(join(monorepoFixture, 'packages/pkg1'))
+
+    process.chdir(pkg1Dir)
+    await migrate({ dryRun: true })
+
+    expect(packageJsonCwds.toSorted()).toEqual([pkg1Dir].toSorted())
+    expect(tsupConfigCwds.toSorted()).toEqual([pkg1Dir].toSorted())
+  })
+
+  test('migrates packages matching glob pattern', async () => {
+    vi.resetModules()
+    const { migrate } = await import('../src/index.ts')
+
+    const pkg1Dir = realpathSync(join(monorepoFixture, 'packages/pkg1'))
+    const pkg2Dir = realpathSync(join(monorepoFixture, 'packages/pkg2'))
+    const pkg3Dir = realpathSync(join(monorepoFixture, 'apps/pkg3'))
+
+    process.chdir(monorepoFixture)
+    await migrate({ dirs: ['packages/*', 'apps/*'], dryRun: true })
+
+    expect(packageJsonCwds.toSorted()).toEqual(
+      [pkg1Dir, pkg2Dir, pkg3Dir].toSorted(),
+    )
+    expect(tsupConfigCwds.toSorted()).toEqual(
+      [pkg1Dir, pkg2Dir, pkg3Dir].toSorted(),
+    )
+  })
+
+  test('migrates multiple explicitly specified dirs', async () => {
+    vi.resetModules()
+    const { migrate } = await import('../src/index.ts')
+
+    const pkg1Dir = realpathSync(join(monorepoFixture, 'packages/pkg1'))
+    const pkg3Dir = realpathSync(join(monorepoFixture, 'apps/pkg3'))
+
+    process.chdir(monorepoFixture)
+    await migrate({
+      dirs: ['packages/pkg1', 'apps/pkg3'],
+      dryRun: true,
+    })
+
+    expect(packageJsonCwds.toSorted()).toEqual([pkg1Dir, pkg3Dir].toSorted())
+    expect(tsupConfigCwds.toSorted()).toEqual([pkg1Dir, pkg3Dir].toSorted())
+  })
+
+  test('migrates single explicitly specified dir', async () => {
+    vi.resetModules()
+    const { migrate } = await import('../src/index.ts')
+
+    const pkg3Dir = realpathSync(join(monorepoFixture, 'apps/pkg3'))
+
+    process.chdir(monorepoFixture)
+    await migrate({ dirs: ['apps/pkg3'], dryRun: true })
+
+    expect(packageJsonCwds).toEqual([pkg3Dir])
+    expect(tsupConfigCwds).toEqual([pkg3Dir])
+  })
+
+  test('exits with error when dirs do not match any directory', async () => {
+    vi.resetModules()
+    const { migrate } = await import('../src/index.ts')
+
+    process.chdir(monorepoFixture)
+    process.exitCode = 0
+    await migrate({ dirs: ['non-existent/*', 'also-not-found'], dryRun: true })
+
+    expect(process.exitCode).toBe(1)
+    expect(packageJsonCwds).toEqual([])
+    expect(tsupConfigCwds).toEqual([])
+  })
+})
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,7 @@`
`51`	`51`	`"ansis": "catalog:prod",`
`52`	`52`	`"cac": "catalog:prod",`
`53`	`53`	`"consola": "catalog:migrate",`
`54`		`- "diff": "catalog:migrate"`
	`54`	`+ "diff": "catalog:migrate",`
	`55`	`+ "tinyglobby": "catalog:prod"`
`55`	`56`	`}`
`56`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+import { add } from 'pkg2'`
	`2`	`+`
	`3`	`+export const answer: number = add(40, 2)`