fix: migration cleanup

.changeset/forty-ears-beg.md

@@ -0,0 +1,5 @@
+---
+'svelte-migrate': patch
+---
+
+fix: finalize svelte-4 migration

packages/migrate/migrations/svelte-4/index.js

@@ -3,14 +3,21 @@ import fs from 'node:fs';
 import prompts from 'prompts';
 import glob from 'tiny-glob/sync.js';
 import { bail, check_git } from '../../utils.js';
-import { update_js_file, update_svelte_file } from './migrate.js';
+import { update_js_file, update_pkg_json, update_svelte_file } from './migrate.js';
 
 export async function migrate() {
 	if (!fs.existsSync('package.json')) {
 		bail('Please re-run this script in a directory with a package.json');
 	}
 
-	console.log(colors.bold().yellow('\nThis will update files in the current directory\n'));
+	console.log(
+		colors
+			.bold()
+			.yellow(
+				'\nThis will update files in the current directory\n' +
+					"If you're inside a monorepo, don't run this in the root directory, rather run it in all projects independently.\n"
+			)
+	);
 
 	const use_git = check_git();
 
@@ -25,22 +32,49 @@ export async function migrate() {
 		process.exit(1);
 	}
 
+	const folders = await prompts({
+		type: 'multiselect',
+		name: 'value',
+		message: 'Which folders should be migrated?',
+		choices: fs
+			.readdirSync('.')
+			.filter(
+				(dir) => fs.statSync(dir).isDirectory() && dir !== 'node_modules' && !dir.startsWith('.')
+			)
+			.map((dir) => ({ title: dir, value: dir, selected: true }))
+	});
+
+	if (!folders.value?.length) {
+		process.exit(1);
+	}
+
+	const migrate_transition = await prompts({
+		type: 'confirm',
+		name: 'value',
+		message:
+			'Add the `|global` modifier to currently global transitions for backwards compatibility? More info at https://svelte.dev/docs/v4-migration-guide#transitions-are-local-by-default',
+		initial: true
+	});
+
+	update_pkg_json();
+
 	// const { default: config } = fs.existsSync('svelte.config.js')
 	// 	? await import(pathToFileURL(path.resolve('svelte.config.js')).href)
 	// 	: { default: {} };
 
 	/** @type {string[]} */
-	const svelte_extensions = /* config.extensions ?? - disabled because it would break .svx */ ['.svelte'];
+	const svelte_extensions = /* config.extensions ?? - disabled because it would break .svx */ [
+		'.svelte'
+	];
 	const extensions = [...svelte_extensions, '.ts', '.js'];
-	// TODO read tsconfig/jsconfig if available? src/** will be good for 99% of cases
-	const files = glob('src/**', { filesOnly: true, dot: true }).map((file) =>
-		file.replace(/\\/g, '/')
-	);
+	const files = glob(`{${folders.value.join(',')}}/**`, { filesOnly: true, dot: true })
+		.map((file) => file.replace(/\\/g, '/'))
+		.filter((file) => !file.includes('/node_modules/'));
 
 	for (const file of files) {
 		if (extensions.some((ext) => file.endsWith(ext))) {
 			if (svelte_extensions.some((ext) => file.endsWith(ext))) {
-				update_svelte_file(file);
+				update_svelte_file(file, migrate_transition.value);
 			} else {
 				update_js_file(file);
 			}
@@ -55,7 +89,7 @@ export async function migrate() {
 
 	const tasks = [
 		use_git && cyan('git commit -m "migration to Svelte 4"'),
-		'Review the migration guide at TODO',
+		'Review the migration guide at https://svelte.dev/docs/v4-migration-guide',
 		'Read the updated docs at https://svelte.dev/docs'
 	].filter(Boolean);