@ryanatkn/gro 0.161.1 → 0.162.0

package/src/lib/package.ts

@@ -5,7 +5,7 @@ import type {Src_Json} from '@ryanatkn/belt/src_json.js';
 
 export const package_json: Package_Json = {
 	name: '@ryanatkn/gro',
-	version: '0.161.1',
+	version: '0.162.0',
 	description: 'task runner and toolkit extending SvelteKit',
 	motto: 'generate, run, optimize',
 	glyph: '🌰',
@@ -44,39 +44,46 @@ export const package_json: Package_Json = {
 		'typescript',
 	],
 	dependencies: {
-		'@ryanatkn/belt': '^0.33.0',
+		'@ryanatkn/belt': '^0.34.1',
 		chokidar: '^4.0.3',
 		dotenv: '^17.2.1',
 		'esm-env': '^1.2.2',
 		mri: '^1.2.0',
-		'oxc-parser': '^0.79.1',
+		'oxc-parser': '^0.82.3',
 		prettier: '^3.6.2',
 		'prettier-plugin-svelte': '^3.4.0',
-		'ts-blank-space': '^0.6.1',
+		'ts-blank-space': '^0.6.2',
 		tslib: '^2.8.1',
-		zod: '^4.0.14',
+		zod: '^4.1.4',
 	},
-	peerDependencies: {esbuild: '^0.25', svelte: '^5', typescript: '^5'},
-	optionalDependencies: {'@ryanatkn/moss': '>=0.31.0'},
+	peerDependencies: {
+		'@sveltejs/kit': '^2',
+		esbuild: '^0.25',
+		svelte: '^5',
+		typescript: '^5',
+		vitest: '^3',
+	},
+	peerDependenciesMeta: {'@sveltejs/kit': {optional: true}, vitest: {optional: true}},
+	optionalDependencies: {'@ryanatkn/moss': '>=0.31.0', vitest: '^3'},
 	devDependencies: {
 		'@changesets/changelog-git': '^0.2.1',
 		'@changesets/types': '^6.1.0',
 		'@ryanatkn/eslint-config': '^0.8.0',
 		'@ryanatkn/fuz': '^0.145.0',
 		'@ryanatkn/moss': '^0.32.0',
-		'@sveltejs/adapter-static': '^3.0.8',
-		'@sveltejs/kit': '^2.27.0',
-		'@sveltejs/package': '^2.4.0',
-		'@sveltejs/vite-plugin-svelte': '^6.1.0',
-		'@types/node': '^24.1.0',
-		esbuild: '^0.25.5',
-		eslint: '^9.32.0',
+		'@sveltejs/adapter-static': '^3.0.9',
+		'@sveltejs/kit': '^2.36.3',
+		'@sveltejs/package': '^2.5.0',
+		'@sveltejs/vite-plugin-svelte': '^6.1.3',
+		'@types/node': '^24.3.0',
+		esbuild: '^0.25.9',
+		eslint: '^9.34.0',
 		'eslint-plugin-svelte': '^3.11.0',
-		svelte: '^5.37.2',
-		'svelte-check': '^4.3.0',
-		typescript: '^5.8.3',
-		'typescript-eslint': '^8.38.0',
-		uvu: '^0.5.6',
+		svelte: '^5.38.6',
+		'svelte-check': '^4.3.1',
+		typescript: '^5.9.2',
+		'typescript-eslint': '^8.41.0',
+		vitest: '^3.2.4',
 	},
 	prettier: {
 		plugins: ['prettier-plugin-svelte'],
@@ -114,6 +121,7 @@ export const package_json: Package_Json = {
 		'./constants.js': {types: './dist/constants.d.ts', default: './dist/constants.js'},
 		'./deploy.task.js': {types: './dist/deploy.task.d.ts', default: './dist/deploy.task.js'},
 		'./dev.task.js': {types: './dist/dev.task.d.ts', default: './dist/dev.task.js'},
+		'./disknode.js': {types: './dist/disknode.d.ts', default: './dist/disknode.js'},
 		'./env.js': {types: './dist/env.d.ts', default: './dist/env.js'},
 		'./esbuild_helpers.js': {
 			types: './dist/esbuild_helpers.d.ts',
@@ -264,7 +272,7 @@ export const package_json: Package_Json = {
 
 export const src_json: Src_Json = {
 	name: '@ryanatkn/gro',
-	version: '0.161.1',
+	version: '0.162.0',
 	modules: {
 		'.': {
 			path: 'index.ts',
@@ -377,11 +385,12 @@ export const src_json: Src_Json = {
 				{name: 'SOURCE_DIR', kind: 'variable'},
 				{name: 'GRO_DIR', kind: 'variable'},
 				{name: 'GRO_DEV_DIR', kind: 'variable'},
-				{name: 'GRO_CONFIG_PATH', kind: 'variable'},
+				{name: 'GRO_CONFIG_FILENAME', kind: 'variable'},
 				{name: 'README_FILENAME', kind: 'variable'},
 				{name: 'SVELTE_CONFIG_FILENAME', kind: 'variable'},
 				{name: 'VITE_CONFIG_FILENAME', kind: 'variable'},
 				{name: 'NODE_MODULES_DIRNAME', kind: 'variable'},
+				{name: 'PACKAGE_JSON_FILENAME', kind: 'variable'},
 				{name: 'LOCKFILE_FILENAME', kind: 'variable'},
 				{name: 'SVELTEKIT_DEV_DIRNAME', kind: 'variable'},
 				{name: 'SVELTEKIT_BUILD_DIRNAME', kind: 'variable'},
@@ -396,10 +405,18 @@ export const src_json: Src_Json = {
 				{name: 'SVELTE_MATCHER', kind: 'variable'},
 				{name: 'SVELTE_RUNES_MATCHER', kind: 'variable'},
 				{name: 'SVELTE_SCRIPT_MATCHER', kind: 'variable'},
+				{name: 'SVELTEKIT_ENV_MATCHER', kind: 'variable'},
+				{name: 'SVELTEKIT_GLOBAL_SPECIFIER', kind: 'variable'},
 				{name: 'EVERYTHING_MATCHER', kind: 'variable'},
 				{name: 'JS_CLI_DEFAULT', kind: 'variable'},
 				{name: 'PM_CLI_DEFAULT', kind: 'variable'},
 				{name: 'PRETTIER_CLI_DEFAULT', kind: 'variable'},
+				{name: 'SVELTEKIT_CLI', kind: 'variable'},
+				{name: 'SVELTE_CHECK_CLI', kind: 'variable'},
+				{name: 'SVELTE_PACKAGE_CLI', kind: 'variable'},
+				{name: 'SVELTE_PACKAGE_DEP_NAME', kind: 'variable'},
+				{name: 'VITE_CLI', kind: 'variable'},
+				{name: 'VITEST_CLI', kind: 'variable'},
 			],
 		},
 		'./deploy.task.js': {
@@ -417,6 +434,7 @@ export const src_json: Src_Json = {
 				{name: 'task', kind: 'variable'},
 			],
 		},
+		'./disknode.js': {path: 'disknode.ts', declarations: [{name: 'Disknode', kind: 'type'}]},
 		'./env.js': {
 			path: 'env.ts',
 			declarations: [
@@ -477,7 +495,6 @@ export const src_json: Src_Json = {
 		'./filer.js': {
 			path: 'filer.ts',
 			declarations: [
-				{name: 'Source_File', kind: 'type'},
 				{name: 'Cleanup_Watch', kind: 'type'},
 				{name: 'On_Filer_Change', kind: 'type'},
 				{name: 'Filer_Options', kind: 'type'},
@@ -584,7 +601,7 @@ export const src_json: Src_Json = {
 				{name: 'create_empty_gro_config', kind: 'function'},
 				{name: 'SEARCH_EXCLUDER_DEFAULT', kind: 'variable'},
 				{name: 'EXPORTS_EXCLUDER_DEFAULT', kind: 'variable'},
-				{name: 'normalize_gro_config', kind: 'function'},
+				{name: 'cook_gro_config', kind: 'function'},
 				{name: 'Gro_Config_Module', kind: 'type'},
 				{name: 'load_gro_config', kind: 'function'},
 				{name: 'validate_gro_config_module', kind: 'function'},
@@ -872,13 +889,6 @@ export const src_json: Src_Json = {
 		'./sveltekit_helpers.js': {
 			path: 'sveltekit_helpers.ts',
 			declarations: [
-				{name: 'SVELTEKIT_CLI', kind: 'variable'},
-				{name: 'SVELTE_CHECK_CLI', kind: 'variable'},
-				{name: 'SVELTE_PACKAGE_CLI', kind: 'variable'},
-				{name: 'SVELTE_PACKAGE_DEP_NAME', kind: 'variable'},
-				{name: 'VITE_CLI', kind: 'variable'},
-				{name: 'SVELTEKIT_ENV_MATCHER', kind: 'variable'},
-				{name: 'SVELTEKIT_GLOBAL_SPECIFIER', kind: 'variable'},
 				{name: 'has_sveltekit_app', kind: 'function'},
 				{name: 'has_sveltekit_library', kind: 'function'},
 				{name: 'sveltekit_sync', kind: 'function'},
@@ -994,6 +1004,7 @@ export const src_json: Src_Json = {
 		'./test_helpers.js': {
 			path: 'test_helpers.ts',
 			declarations: [
+				{name: 'TEST_TIMEOUT_MD', kind: 'variable'},
 				{name: 'SOME_PUBLIC_ENV_VAR_NAME', kind: 'variable'},
 				{name: 'SOME_PUBLIC_ENV_VAR_VALUE', kind: 'variable'},
 				{name: 'init_test_env', kind: 'function'},

package/src/lib/package_json.ts

@@ -7,7 +7,7 @@ import {styleText as st} from 'node:util';
 import {Package_Json, Package_Json_Exports} from '@ryanatkn/belt/package_json.js';
 
 import {paths, gro_paths, IS_THIS_GRO, replace_extension} from './paths.ts';
-import {SVELTEKIT_DIST_DIRNAME} from './constants.ts';
+import {PACKAGE_JSON_FILENAME, SVELTEKIT_DIST_DIRNAME} from './constants.ts';
 import {search_fs} from './search_fs.ts';
 import {has_sveltekit_library} from './sveltekit_helpers.ts';
 import {GITHUB_REPO_MATCHER} from './github.ts';
@@ -22,6 +22,7 @@ export const load_package_json = (
 	dir = IS_THIS_GRO ? gro_paths.root : paths.root,
 	cache?: Record<string, Package_Json>,
 	parse = true, // TODO pass `false` here in more places, especially anything perf-sensitive like work on startup
+	log?: Logger,
 ): Package_Json => {
 	let package_json: Package_Json;
 	if (cache && dir in cache) {
@@ -29,7 +30,8 @@ export const load_package_json = (
 	}
 	try {
 		package_json = JSON.parse(load_package_json_contents(dir));
-	} catch (_err) {
+	} catch (err) {
+		log?.error(st('yellow', `Failed to load package.json in ${dir}`), err);
 		return EMPTY_PACKAGE_JSON;
 	}
 	if (parse) {
@@ -44,14 +46,13 @@ export const load_package_json = (
 export const sync_package_json = async (
 	map_package_json: Map_Package_Json,
 	log: Logger,
-	check = false,
+	write = true,
 	dir = paths.root,
 	exports_dir = paths.lib,
 ): Promise<{package_json: Package_Json | null; changed: boolean}> => {
 	const exported_files = search_fs(exports_dir);
 	const exported_paths = exported_files.map((f) => f.path);
 	const updated = await update_package_json(
-		dir,
 		async (package_json) => {
 			if (has_sveltekit_library(package_json).ok) {
 				const exports = to_package_exports(exported_paths);
@@ -60,7 +61,8 @@ export const sync_package_json = async (
 			const mapped = await map_package_json(package_json);
 			return mapped ? parse_package_json(Package_Json, mapped) : mapped;
 		},
-		!check,
+		dir,
+		write,
 	);
 
 	const exports_count =
@@ -80,10 +82,10 @@ export const load_gro_package_json = (): Package_Json => load_package_json(gro_p
 
 // TODO probably make this nullable and make callers handle failures
 const load_package_json_contents = (dir: string): string =>
-	readFileSync(join(dir, 'package.json'), 'utf8');
+	readFileSync(join(dir, PACKAGE_JSON_FILENAME), 'utf8');
 
 export const write_package_json = (serialized_package_json: string): void => {
-	writeFileSync(join(paths.root, 'package.json'), serialized_package_json);
+	writeFileSync(join(paths.root, PACKAGE_JSON_FILENAME), serialized_package_json);
 };
 
 export const serialize_package_json = (package_json: Package_Json): string =>
@@ -93,8 +95,8 @@ export const serialize_package_json = (package_json: Package_Json): string =>
  * Updates package.json. Writes to the filesystem only when contents change.
  */
 export const update_package_json = async (
-	dir = paths.root,
 	update: (package_json: Package_Json) => Package_Json | null | Promise<Package_Json | null>,
+	dir = paths.root,
 	write = true,
 ): Promise<{package_json: Package_Json | null; changed: boolean}> => {
 	const original_contents = load_package_json_contents(dir);
@@ -186,7 +188,7 @@ export const parse_repo_url = (
  * Parses a `Package_Json` object but preserves the order of the original keys.
  */
 const parse_package_json = (schema: typeof Package_Json, value: any): Package_Json => {
-	const parsed = parse_or_throw_formatted_error('package.json', schema, value);
+	const parsed = parse_or_throw_formatted_error(PACKAGE_JSON_FILENAME, schema, value);
 	const keys = Object.keys(value);
 	return Object.fromEntries(
 		Object.entries(parsed).sort(([a], [b]) => keys.indexOf(a) - keys.indexOf(b)),

package/src/lib/parse_imports.ts

@@ -124,7 +124,7 @@ export const parse_imports = (
 
 				// eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
 				if (node.source.type === 'Literal') {
-					const value = String(node.source.value);
+					const value = node.source.value;
 					if (value) {
 						specifiers.push(value);
 					}
@@ -136,7 +136,7 @@ export const parse_imports = (
 	};
 
 	if (is_svelte) {
-		// Reset the regex state between calls
+		// Reset the regexp state between calls
 		SVELTE_SCRIPT_MATCHER.lastIndex = 0;
 
 		// Capture script tags at the top level (not nested in HTML)

package/src/lib/paths.ts

@@ -4,7 +4,7 @@ import {ensure_end, strip_end} from '@ryanatkn/belt/string.js';
 import {styleText as st} from 'node:util';
 
 import {
-	GRO_CONFIG_PATH,
+	GRO_CONFIG_FILENAME,
 	GRO_DEV_DIR,
 	GRO_DIR,
 	SOURCE_DIR,
@@ -49,7 +49,7 @@ export const create_paths = (root_dir: string): Paths => {
 		lib: root + LIB_DIR,
 		build: root + GRO_DIR,
 		build_dev: root + GRO_DEV_DIR,
-		config: root + GRO_CONFIG_PATH,
+		config: root + GRO_CONFIG_FILENAME,
 	};
 };
 

package/src/lib/reinstall.task.ts

@@ -30,7 +30,7 @@ export const task: Task<Args> = {
 			);
 		}
 
-		// TODO this relies on npm behavior that changed in v11
+		// TODO @many this relies on npm behavior that changed in v11
 		// Deleting the lockfile and reinstalling cleans the lockfile of unnecessary dep noise,
 		// like esbuild's many packages for each platform.
 		await rm(LOCKFILE_FILENAME);

package/src/lib/resolve_specifier.ts

@@ -4,6 +4,8 @@ import {existsSync} from 'node:fs';
 import {replace_extension} from './paths.ts';
 import type {Path_Id} from './path.ts';
 
+// TODO ideally this module doesnt exist, but import.meta.resolve doesn't work in loaders last I tried
+
 export interface Resolved_Specifier {
 	/**
 	 * The resolved filesystem path for the specifier.

package/src/lib/src_json.ts

@@ -7,6 +7,7 @@ import {Src_Json, Src_Modules} from '@ryanatkn/belt/src_json.js';
 
 import {paths, replace_extension} from './paths.ts';
 import {parse_exports} from './parse_exports.ts';
+import {TS_MATCHER} from './constants.ts';
 
 export type Map_Src_Json = (src_json: Src_Json) => Src_Json | null | Promise<Src_Json | null>;
 
@@ -48,6 +49,7 @@ export const to_src_modules = (
 				continue;
 			}
 
+			// TODO still an error for unknown files running `gro gen`, can it use the filer to invalidate correctly?
 			throw Error(
 				`Failed to infer source file from package.json export path ${k} - the inferred file ${source_file_id} does not exist`,
 			);
@@ -58,7 +60,7 @@ export const to_src_modules = (
 
 	// Create a TypeScript program for all TypeScript files
 	const ts_files = file_paths
-		.filter(({file_path}) => file_path.endsWith('.ts') || file_path.endsWith('.tsx'))
+		.filter(({file_path}) => TS_MATCHER.test(file_path))
 		.map(({file_path}) => file_path);
 
 	let program: ts.Program | undefined;

package/src/lib/svelte_config.ts

@@ -5,6 +5,9 @@ import {EMPTY_OBJECT} from '@ryanatkn/belt/object.js';
 
 import {SVELTE_CONFIG_FILENAME} from './constants.ts';
 
+/* eslint-disable @typescript-eslint/no-deprecated */
+// see https://github.com/sveltejs/kit/discussions/14240
+
 /*
 
 This module is intended to have minimal dependencies to avoid over-imports in the CLI.

package/src/lib/sveltekit_helpers.ts

@@ -6,23 +6,17 @@ import type {Package_Json} from '@ryanatkn/belt/package_json.js';
 
 import {has_dep} from './package_json.ts';
 import {default_svelte_config, type Parsed_Svelte_Config} from './svelte_config.ts';
-import {SVELTE_CONFIG_FILENAME, SVELTEKIT_DEV_DIRNAME, PM_CLI_DEFAULT} from './constants.ts';
+import {
+	SVELTE_CONFIG_FILENAME,
+	SVELTEKIT_DEV_DIRNAME,
+	PM_CLI_DEFAULT,
+	SVELTE_PACKAGE_DEP_NAME,
+	SVELTEKIT_CLI,
+} from './constants.ts';
 import {find_cli, spawn_cli, to_cli_name, type Cli} from './cli.ts';
 import {Task_Error} from './task.ts';
 import {serialize_args, to_forwarded_args} from './args.ts';
 
-export const SVELTEKIT_CLI = 'svelte-kit';
-
-export const SVELTE_CHECK_CLI = 'svelte-check';
-
-export const SVELTE_PACKAGE_CLI = 'svelte-package';
-export const SVELTE_PACKAGE_DEP_NAME = '@sveltejs/package';
-
-export const VITE_CLI = 'vite';
-
-export const SVELTEKIT_ENV_MATCHER = /^\$env\/(static|dynamic)\/(public|private)$/;
-export const SVELTEKIT_GLOBAL_SPECIFIER = /^\$(env|app)\//;
-
 export const has_sveltekit_app = (
 	svelte_config_path: string = SVELTE_CONFIG_FILENAME,
 ): Result<object, {message: string}> => {

package/src/lib/sync.task.ts

@@ -12,8 +12,7 @@ export const Args = z.strictObject({
 	'no-package_json': z.boolean().meta({description: 'opt out of package.json sync'}).default(false),
 	gen: z.boolean().meta({description: 'dual of no-gen'}).default(true),
 	'no-gen': z.boolean().meta({description: 'opt out of running gen'}).default(false),
-	install: z.boolean().meta({description: 'dual of no-install'}).default(true),
-	'no-install': z.boolean().meta({description: 'opt out of installing packages'}).default(false),
+	install: z.boolean().meta({description: 'opt into installing packages'}).default(false),
 });
 export type Args = z.infer<typeof Args>;
 

package/src/lib/task.ts

@@ -37,8 +37,6 @@ export interface Task_Context<T_Args = object> {
 	config: Gro_Config;
 	svelte_config: Parsed_Svelte_Config;
 	filer: Filer;
-	// TODO should this go here or on `config` for convenience?
-	// svelte_config: Parsed_Svelte_Config;
 	log: Logger;
 	timings: Timings;
 	invoke_task: (task_name: string, args?: Args, config?: Gro_Config) => Promise<void>;

package/src/lib/test.task.ts

@@ -1,50 +1,49 @@
-import {styleText as st} from 'node:util';
 import {z} from 'zod';
+import {spawn_cli} from '@ryanatkn/gro/cli.js';
 
 import {Task_Error, type Task} from './task.ts';
-import {paths} from './paths.ts';
 import {find_cli} from './cli.ts';
+import {has_dep} from './package_json.ts';
+import {serialize_args, to_forwarded_args} from './args.ts';
+import {VITEST_CLI} from './constants.ts';
+import {paths} from './paths.ts';
 
 export const Args = z.strictObject({
-	_: z.array(z.string()).meta({description: 'file patterns to test'}).default([`\\.test\\.ts$`]), // TODO maybe use uvu's default instead of being restrictive?
-	bail: z
-		.boolean()
-		.meta({description: 'the bail option to uvu run, exit immediately on failure'})
-		.default(false),
-	cwd: z.string().meta({description: 'the cwd option to uvu parse'}).optional(),
-	ignore: z
-		.union([z.string(), z.array(z.string())])
-		.meta({description: 'the ignore option to uvu parse'})
+	_: z.array(z.string()).meta({description: 'file patterns to test'}).default(['.test.']),
+	dir: z.string().meta({description: 'working directory for tests'}).default(paths.source),
+	t: z
+		.string()
+		.meta({description: 'search pattern for test names, same as vitest -t and --testNamePattern'})
 		.optional(),
 });
 export type Args = z.infer<typeof Args>;
 
 export const task: Task<Args> = {
-	summary: 'run tests with uvu',
+	summary: 'run tests with vitest',
 	Args,
-	run: async ({args, log}): Promise<void> => {
-		const {_: patterns, bail, cwd, ignore} = args;
+	run: async ({args}): Promise<void> => {
+		const {_: patterns, dir, t} = args;
 
-		if (!find_cli('uvu')) {
-			log.warn(st('yellow', 'uvu is not installed, skipping tests'));
-			return;
-		}
+		if (has_dep(VITEST_CLI)) {
+			if (!find_cli(VITEST_CLI)) {
+				throw new Task_Error('vitest is a dependency but not installed; run `npm i`?');
+			}
 
-		const [{run}, {parse}] = await Promise.all([import('uvu/run'), import('uvu/parse')]);
-
-		// uvu doesn't work with esm loaders and TypeScript files,
-		// so we use its `parse` and `run` APIs directly instead of its CLI.
-		// To avoid surprises, we allow any number of patterns in the rest args,
-		// so we call `parse` multiple times because it supports only one.
-		const suites = [];
-		for (const pattern of patterns) {
-			const parsed = await parse(paths.source, pattern, {cwd, ignore}); // eslint-disable-line no-await-in-loop
-			suites.push(...parsed.suites);
-		}
-		await run(suites, {bail});
+			const vitest_args = ['run', ...patterns];
+			if (dir) {
+				vitest_args.push('--dir', dir);
+			}
+			if (t) {
+				vitest_args.push('-t', t);
+			}
+			vitest_args.push(...serialize_args(to_forwarded_args(VITEST_CLI)));
 
-		if (process.exitCode) {
-			throw new Task_Error('Tests failed.');
+			const spawned = await spawn_cli(VITEST_CLI, vitest_args); // TODO proper forwarding
+			if (!spawned?.ok) {
+				throw new Task_Error(`vitest failed with exit code ${spawned?.code}`);
+			}
+		} else {
+			throw new Task_Error('no test runner found, install vitest');
 		}
 	},
 };

package/src/lib/test_helpers.ts

@@ -2,6 +2,8 @@ import {existsSync, readFileSync, writeFileSync} from 'node:fs';
 import {join} from 'node:path';
 import ts from 'typescript';
 
+export const TEST_TIMEOUT_MD = 20_000;
+
 export const SOME_PUBLIC_ENV_VAR_NAME = 'PUBLIC_SOME_PUBLIC_ENV_VAR';
 export const SOME_PUBLIC_ENV_VAR_VALUE = 'SOME_PUBLIC_ENV_VAR';
 const name_equals = SOME_PUBLIC_ENV_VAR_NAME + '=';

package/src/lib/typecheck.task.ts

@@ -4,9 +4,10 @@ import {z} from 'zod';
 import {Task_Error, type Task} from './task.ts';
 import {serialize_args, to_forwarded_args} from './args.ts';
 import {find_cli, spawn_cli, spawn_cli_process} from './cli.ts';
-import {SVELTE_CHECK_CLI, sveltekit_sync_if_available} from './sveltekit_helpers.ts';
+import {sveltekit_sync_if_available} from './sveltekit_helpers.ts';
 import {configure_colored_output_with_path_replacement} from './child_process_logging.ts';
 import {paths} from './paths.ts';
+import {SVELTE_CHECK_CLI} from './constants.ts';
 
 export const Args = z.strictObject({
 	svelte_check_cli: z

package/src/lib/upgrade.task.ts

@@ -110,6 +110,7 @@ export const task: Task<Args> = {
 		install_args.push(...serialize_args(to_forwarded_args(config.pm_cli)));
 		await spawn(config.pm_cli, install_args);
 
+		// TODO @many this relies on npm behavior that changed in v11
 		// If we deleted the lockfile or node modules, `npm install` again
 		// to fix a recurring npm bug getting the lockfile to its final state.
 		if (!dry && (delete_node_modules || delete_lockfile)) {
@@ -118,7 +119,7 @@ export const task: Task<Args> = {
 		}
 
 		// Sync in a new process to pick up any changes after installing, avoiding some errors.
-		await spawn_cli('gro', ['sync', '--no-install']); // don't install because we do above
+		await spawn_cli('gro', ['sync']); // don't install because we do above
 	},
 };