@ryanatkn/gro 0.141.0 → 0.142.0

package/dist/watch_dir.js

@@ -1,6 +1,7 @@
 import { watch } from 'chokidar';
 import { relative } from 'node:path';
 import { statSync } from 'node:fs';
+import { create_deferred } from '@ryanatkn/belt/async.js';
 /**
  * Watch for changes on the filesystem using chokidar.
  */
@@ -10,9 +11,8 @@ export const watch_dir = ({ dir, on_change, filter, absolute = true, chokidar, }
     return {
         init: async () => {
             if (initing)
-                return initing;
-            let resolve;
-            initing = new Promise((r) => (resolve = r)); // TODO `create_deferred`?// cwd: chokidar?.cwd ?? process.cwd()
+                return initing.promise;
+            initing = create_deferred();
             watcher = watch(dir, { ...chokidar });
             watcher.on('add', (path) => {
                 const final_path = absolute ? path : relative(dir, path);
@@ -46,8 +46,8 @@ export const watch_dir = ({ dir, on_change, filter, absolute = true, chokidar, }
                 on_change({ type: 'delete', path: final_path, is_directory: true });
             });
             // wait until ready
-            watcher.once('ready', () => resolve());
-            await initing;
+            watcher.once('ready', () => initing?.resolve());
+            await initing.promise;
         },
         close: async () => {
             initing = undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ryanatkn/gro",
-  "version": "0.141.0",
+  "version": "0.142.0",
   "description": "task runner and toolkit extending SvelteKit",
   "motto": "generate, run, optimize",
   "glyph": "🌰",
@@ -58,7 +58,7 @@
     "prettier": "^3.3.3",
     "prettier-plugin-svelte": "^3.2.7",
     "ts-morph": "^24.0.0",
-    "tslib": "^2.7.0",
+    "tslib": "^2.8.0",
     "zod": "^3.23.8"
   },
   "peerDependencies": {
@@ -69,21 +69,21 @@
     "@changesets/changelog-git": "^0.2.0",
     "@changesets/types": "^6.0.0",
     "@ryanatkn/eslint-config": "^0.5.5",
-    "@ryanatkn/fuz": "^0.129.2",
-    "@ryanatkn/moss": "^0.18.0",
-    "@sveltejs/adapter-static": "^3.0.5",
-    "@sveltejs/kit": "^2.6.4",
-    "@sveltejs/package": "^2.3.5",
-    "@sveltejs/vite-plugin-svelte": "^4.0.0-next.6",
+    "@ryanatkn/fuz": "^0.129.5",
+    "@ryanatkn/moss": "^0.18.2",
+    "@sveltejs/adapter-static": "^3.0.6",
+    "@sveltejs/kit": "^2.7.3",
+    "@sveltejs/package": "^2.3.7",
+    "@sveltejs/vite-plugin-svelte": "^4.0.0",
     "@types/fs-extra": "^11.0.4",
-    "@types/node": "^22.7.5",
+    "@types/node": "^22.8.1",
     "esbuild": "^0.21.5",
-    "eslint": "^9.12.0",
-    "eslint-plugin-svelte": "^2.44.1",
-    "svelte": "^5.0.0-next.264",
-    "svelte-check": "^4.0.4",
+    "eslint": "^9.13.0",
+    "eslint-plugin-svelte": "^2.46.0",
+    "svelte": "^5.1.3",
+    "svelte-check": "^4.0.5",
     "typescript": "^5.6.3",
-    "typescript-eslint": "^8.8.1",
+    "typescript-eslint": "^8.11.0",
     "uvu": "^0.5.6"
   },
   "prettier": {

package/src/lib/changeset.task.ts

@@ -63,6 +63,7 @@ export const task: Task<Args> = {
 			args: {_, minor, major, dir, access: access_arg, changelog, dep, origin, changeset_cli},
 			log,
 			sveltekit_config,
+			config,
 		} = ctx;
 
 		const message = _.join(' ');
@@ -114,7 +115,7 @@ export const task: Task<Args> = {
 			await spawn('git', ['add', dir]);
 
 			if (dep) {
-				await spawn('npm', ['i', '-D', changelog]);
+				await spawn(config.pm_cli, ['i', '-D', changelog]);
 			}
 		}
 

package/src/lib/filer.ts

@@ -18,6 +18,8 @@ import {resolve_specifier} from './resolve_specifier.js';
 import {default_sveltekit_config} from './sveltekit_config.js';
 import {map_sveltekit_aliases} from './sveltekit_helpers.js';
 import {Unreachable_Error} from '@ryanatkn/belt/error.js';
+import {resolve_node_specifier} from './resolve_node_specifier.js';
+import type {Package_Json} from './package_json.js';
 // TODO see below
 // import {resolve_node_specifier} from './resolve_node_specifier.js';
 
@@ -31,6 +33,11 @@ export interface Source_File {
 	 * We create the file in memory to track its dependents regardless of its existence on disk.
 	 */
 	contents: string | null;
+	/**
+	 * Is the source file outside of the `root_dir` or excluded by `watch_dir_options.filter`?
+	 */
+	external: boolean;
+	ctime: number | null;
 	mtime: number | null;
 	dependents: Map<Path_Id, Source_File>;
 	dependencies: Map<Path_Id, Source_File>;
@@ -43,6 +50,7 @@ export type On_Filer_Change = (change: Watcher_Change, source_file: Source_File)
 export interface Options {
 	watch_dir?: typeof watch_dir;
 	watch_dir_options?: Partial<Omit_Strict<Watch_Dir_Options, 'on_change'>>;
+	package_json_cache?: Record<string, Package_Json>;
 }
 
 export class Filer {
@@ -53,10 +61,13 @@ export class Filer {
 	#watch_dir: typeof watch_dir;
 	#watch_dir_options: Partial<Watch_Dir_Options>;
 
+	#package_json_cache: Record<string, Package_Json>;
+
 	constructor(options: Options = EMPTY_OBJECT) {
 		this.#watch_dir = options.watch_dir ?? watch_dir;
 		this.#watch_dir_options = options.watch_dir_options ?? EMPTY_OBJECT;
 		this.root_dir = resolve(options.watch_dir_options?.dir ?? paths.source);
+		this.#package_json_cache = options.package_json_cache ?? {};
 	}
 
 	#watching: Watch_Node_Fs | undefined;
@@ -74,11 +85,17 @@ export class Filer {
 		const file: Source_File = {
 			id,
 			contents: null,
+			external: this.#is_external(id), // TODO maybe filter externals by default? the user needs to configure the filer then
+			ctime: null,
 			mtime: null,
 			dependents: new Map(),
 			dependencies: new Map(),
 		};
 		this.files.set(id, file);
+		// TODO this may need to be batched/deferred
+		if (file.external) {
+			this.#on_change({type: 'add', path: file.id, is_directory: false});
+		}
 		return file;
 	};
 
@@ -86,8 +103,7 @@ export class Filer {
 		const file = this.get_or_create(id);
 
 		const stats = existsSync(id) ? statSync(id) : null;
-		// const mtime_prev = file.mtime;
-		// const mtime_changed = mtime_prev !== (stats?.mtimeMs ?? null);
+		file.ctime = stats?.ctimeMs ?? null;
 		file.mtime = stats?.mtimeMs ?? null;
 
 		const new_contents = stats ? readFileSync(id, 'utf8') : null;
@@ -109,36 +125,25 @@ export class Filer {
 			const path = map_sveltekit_aliases(specifier, aliases);
 
 			// The specifier `path` has now been mapped to its final form, so we can inspect it.
-			if (path[0] === '.' || path[0] === '/') {
-				const {path_id} = resolve_specifier(path, dir);
-				dependencies_removed.delete(path_id);
-				if (!dependencies_before.has(path_id)) {
-					const d = this.get_or_create(path_id);
-					file.dependencies.set(d.id, d);
-					d.dependents.set(file.id, file);
-				}
+			const resolved =
+				path[0] === '.' || path[0] === '/'
+					? resolve_specifier(path, dir)
+					: resolve_node_specifier(path, undefined, file.id, this.#package_json_cache, false);
+			if (!resolved) continue; // ignore any missing imports like Node identifiers
+			const {path_id} = resolved;
+			dependencies_removed.delete(path_id);
+			if (!dependencies_before.has(path_id)) {
+				const d = this.get_or_create(path_id);
+				file.dependencies.set(d.id, d);
+				d.dependents.set(file.id, file);
 			}
-			// TODO this doesn't work
-			// const resolved =
-			// 	path[0] === '.' || path[0] === '/'
-			// 		? resolve_specifier(path, dir)
-			// 		: resolve_node_specifier(path, dir);
-			// const {path_id} = resolved;
-			// dependencies_removed.delete(path_id);
-			// if (!dependencies_before.has(path_id)) {
-			// 	const d = this.get_or_create(path_id);
-			// 	file.dependencies.set(d.id, d);
-			// 	d.dependents.set(file.id, file);
-			// }
 		}
 
 		// update any removed dependencies
 		for (const dependency_removed of dependencies_removed) {
-			const deleted1 = file.dependencies.delete(dependency_removed);
-			if (!deleted1) throw Error('expected to delete1 ' + file.id); // TODO @many delete if correct
+			file.dependencies.delete(dependency_removed);
 			const dependency_removed_file = this.get_or_create(dependency_removed);
-			const deleted2 = dependency_removed_file.dependents.delete(file.id);
-			if (!deleted2) throw Error('expected to delete2 ' + file.id); // TODO @many delete if correct
+			dependency_removed_file.dependents.delete(file.id);
 		}
 
 		return file;
@@ -203,7 +208,7 @@ export class Filer {
 	}
 
 	#on_change: Watcher_Change_Callback = (change) => {
-		if (change.is_directory) return;
+		if (change.is_directory) return; // TODO manage directories?
 		let source_file: Source_File | null;
 		switch (change.type) {
 			case 'add':
@@ -236,4 +241,9 @@ export class Filer {
 			this.#watching = undefined;
 		}
 	}
+
+	#is_external(id: string): boolean {
+		const {filter} = this.#watch_dir_options;
+		return !id.startsWith(this.root_dir + '/') || (!!filter && !filter(id, false));
+	}
 }

package/src/lib/gro_config.ts

@@ -40,6 +40,10 @@ export interface Gro_Config {
 	 * directories and files are included if they pass all of these filters.
 	 */
 	search_filters: Path_Filter[];
+	/**
+	 * The CLI to use that's compatible with `npm install` and `npm link`. Defaults to `'npm'`.
+	 */
+	pm_cli: string;
 }
 
 /**
@@ -52,6 +56,7 @@ export interface Raw_Gro_Config {
 	map_package_json?: Map_Package_Json | null;
 	task_root_dirs?: string[];
 	search_filters?: Path_Filter | Path_Filter[] | null;
+	pm_cli?: string;
 }
 
 export type Create_Gro_Config = (
@@ -68,6 +73,7 @@ export const create_empty_gro_config = (): Gro_Config => ({
 		IS_THIS_GRO ? null : GRO_DIST_DIR,
 	].filter((v) => v !== null),
 	search_filters: [(id) => !DEFAULT_SEARCH_EXCLUDER.test(id)],
+	pm_cli: 'npm',
 });
 
 /**
@@ -112,6 +118,7 @@ export const normalize_gro_config = (raw_config: Raw_Gro_Config): Gro_Config =>
 		map_package_json = empty_config.map_package_json,
 		task_root_dirs = empty_config.task_root_dirs,
 		search_filters = empty_config.search_filters,
+		pm_cli = empty_config.pm_cli,
 	} = raw_config;
 	return {
 		plugins,
@@ -122,6 +129,7 @@ export const normalize_gro_config = (raw_config: Raw_Gro_Config): Gro_Config =>
 			: search_filters
 				? [search_filters]
 				: [],
+		pm_cli,
 	};
 };
 

package/src/lib/gro_plugin_sveltekit_library.ts

@@ -33,12 +33,12 @@ export const gro_plugin_sveltekit_library = ({
 				await run_svelte_package(svelte_package_options, svelte_package_cli, log);
 			}
 		},
-		adapt: async ({log, timings}) => {
+		adapt: async ({log, timings, config}) => {
 			const package_json = load_package_json();
 
 			// `npm link`
 			if (package_json.bin) {
-				const timing_to_npm_link = timings.start('npm link');
+				const timing_to_link = timings.start(`${config.pm_cli} link`);
 				await Promise.all(
 					Object.values(package_json.bin).map(async (bin_path) => {
 						const chmod_result = await spawn('chmod', ['+x', bin_path]);
@@ -47,11 +47,11 @@ export const gro_plugin_sveltekit_library = ({
 					}),
 				);
 				log.info(`linking`);
-				const link_result = await spawn('npm', ['link', '-f']); // TODO don't use `-f` unless necessary or at all?
+				const link_result = await spawn(config.pm_cli, ['link', '-f']); // TODO don't use `-f` unless necessary or at all?
 				if (!link_result.ok) {
 					throw new Task_Error(`Failed to link. ${print_spawn_result(link_result)}`);
 				}
-				timing_to_npm_link();
+				timing_to_link();
 			}
 		},
 	};

package/src/lib/loader.ts

@@ -16,7 +16,7 @@ import {
 } from './sveltekit_shim_app.js';
 import {default_sveltekit_config} from './sveltekit_config.js';
 import {SVELTE_MATCHER, SVELTE_RUNES_MATCHER} from './svelte_helpers.js';
-import {paths} from './paths.js';
+import {IS_THIS_GRO, paths} from './paths.js';
 import {JSON_MATCHER, NODE_MODULES_DIRNAME, TS_MATCHER} from './path_constants.js';
 import {to_define_import_meta_env, default_ts_transform_options} from './esbuild_helpers.js';
 import {resolve_specifier} from './resolve_specifier.js';
@@ -173,51 +173,59 @@ export const load: LoadHook = async (url, context, nextLoad) => {
 };
 
 export const resolve: ResolveHook = async (specifier, context, nextResolve) => {
+	let s = specifier;
+
+	// Support SvelteKit `$env` imports
 	if (
-		specifier === '$env/static/public' ||
-		specifier === '$env/static/private' ||
-		specifier === '$env/dynamic/public' ||
-		specifier === '$env/dynamic/private'
+		s === '$env/static/public' ||
+		s === '$env/static/private' ||
+		s === '$env/dynamic/public' ||
+		s === '$env/dynamic/private'
 	) {
 		// The returned `url` is validated before `load` is called,
 		// so we need a slightly roundabout strategy to pass through the specifier for virtual files.
 		return {
-			url: pathToFileURL(join(dir, 'src/lib', specifier)).href,
+			url: pathToFileURL(join(dir, 'src/lib', s)).href,
 			format: 'module',
 			shortCircuit: true,
 		};
 	}
 
+	// Special case for Gro's dependencies that import into Gro.
+	// Without this, we'd need to add a dev dep to Gro for Gro, which causes problems.
+	if (IS_THIS_GRO && s.startsWith('@ryanatkn/gro')) {
+		s = join(dir, 'dist', s.substring(13));
+	}
+
 	const parent_url = context.parentURL;
 	if (!parent_url || NODE_MODULES_MATCHER.test(parent_url)) {
-		return nextResolve(specifier, context);
+		return nextResolve(s, context);
 	}
 
-	const shimmed = sveltekit_shim_app_specifiers.get(specifier);
+	const shimmed = sveltekit_shim_app_specifiers.get(s);
 	if (shimmed !== undefined) {
 		return nextResolve(shimmed, context);
 	}
 
-	const path = map_sveltekit_aliases(specifier, aliases);
+	s = map_sveltekit_aliases(s, aliases);
 
-	// The specifier `path` has now been mapped to its final form, so we can inspect it.
-	if (path[0] !== '.' && path[0] !== '/') {
+	// The specifier has now been mapped to its final form, so we can inspect it.
+	if (s[0] !== '.' && s[0] !== '/') {
 		// Resolve to `node_modules`.
-		if (SVELTE_MATCHER.test(path) || JSON_MATCHER.test(path)) {
+		if (SVELTE_MATCHER.test(s) || JSON_MATCHER.test(s)) {
 			// Match the behavior of Vite and esbuild for Svelte and JSON imports.
-			// TODO `.ts` too
-			const resolved = resolve_node_specifier(path, dir, parent_url, package_json_cache);
+			const resolved = resolve_node_specifier(s, dir, parent_url, package_json_cache)!; // `node:` specifiers shouldn't reach this point, so the assertion is safe
 			return {
 				url: pathToFileURL(resolved.path_id_with_querystring).href,
 				format: 'module',
 				shortCircuit: true,
 			};
 		} else {
-			return nextResolve(path, context);
+			return nextResolve(s, context);
 		}
 	}
 
-	const resolved = resolve_specifier(path, dirname(fileURLToPath(parent_url)));
+	const resolved = resolve_specifier(s, dirname(fileURLToPath(parent_url)));
 
 	return {
 		url: pathToFileURL(resolved.path_id_with_querystring).href,

package/src/lib/moss_helpers.ts

@@ -3,6 +3,7 @@ import {existsSync} from 'node:fs';
 import {resolve} from 'node:path';
 
 import {has_dep, type Package_Json} from './package_json.js';
+import {NODE_MODULES_DIRNAME} from './path_constants.js';
 
 export const MOSS_PACKAGE_DEP_NAME = '@ryanatkn/moss';
 
@@ -10,7 +11,7 @@ export const MOSS_PACKAGE_DEP_NAME = '@ryanatkn/moss';
 export const load_moss_plugin = async (
 	package_json?: Package_Json,
 	dep_name = MOSS_PACKAGE_DEP_NAME,
-	plugin_path = `node_modules/${dep_name}/dist/gro_plugin_moss.js`, // TODO maybe lookup from its `package_json.exports`? kinda unnecessary
+	plugin_path = `${NODE_MODULES_DIRNAME}/${dep_name}/dist/gro_plugin_moss.js`, // TODO maybe lookup from its `package_json.exports`? kinda unnecessary
 	local_plugin_path = 'src/lib/gro_plugin_moss.ts',
 ): Promise<Result<{gro_plugin_moss: any}, {message: string}>> => {
 	if (!has_dep(dep_name, package_json)) {

package/src/lib/package.ts

@@ -5,7 +5,7 @@ import type {Src_Json} from './src_json.js';
 
 export const package_json = {
 	name: '@ryanatkn/gro',
-	version: '0.141.0',
+	version: '0.142.0',
 	description: 'task runner and toolkit extending SvelteKit',
 	motto: 'generate, run, optimize',
 	glyph: '🌰',
@@ -53,7 +53,7 @@ export const package_json = {
 		prettier: '^3.3.3',
 		'prettier-plugin-svelte': '^3.2.7',
 		'ts-morph': '^24.0.0',
-		tslib: '^2.7.0',
+		tslib: '^2.8.0',
 		zod: '^3.23.8',
 	},
 	peerDependencies: {esbuild: '^0.21.0', svelte: '^5.0.0-next.0'},
@@ -61,21 +61,21 @@ export const package_json = {
 		'@changesets/changelog-git': '^0.2.0',
 		'@changesets/types': '^6.0.0',
 		'@ryanatkn/eslint-config': '^0.5.5',
-		'@ryanatkn/fuz': '^0.129.2',
-		'@ryanatkn/moss': '^0.18.0',
-		'@sveltejs/adapter-static': '^3.0.5',
-		'@sveltejs/kit': '^2.6.4',
-		'@sveltejs/package': '^2.3.5',
-		'@sveltejs/vite-plugin-svelte': '^4.0.0-next.6',
+		'@ryanatkn/fuz': '^0.129.5',
+		'@ryanatkn/moss': '^0.18.2',
+		'@sveltejs/adapter-static': '^3.0.6',
+		'@sveltejs/kit': '^2.7.3',
+		'@sveltejs/package': '^2.3.7',
+		'@sveltejs/vite-plugin-svelte': '^4.0.0',
 		'@types/fs-extra': '^11.0.4',
-		'@types/node': '^22.7.5',
+		'@types/node': '^22.8.1',
 		esbuild: '^0.21.5',
-		eslint: '^9.12.0',
-		'eslint-plugin-svelte': '^2.44.1',
-		svelte: '^5.0.0-next.264',
-		'svelte-check': '^4.0.4',
+		eslint: '^9.13.0',
+		'eslint-plugin-svelte': '^2.46.0',
+		svelte: '^5.1.3',
+		'svelte-check': '^4.0.5',
 		typescript: '^5.6.3',
-		'typescript-eslint': '^8.8.1',
+		'typescript-eslint': '^8.11.0',
 		uvu: '^0.5.6',
 	},
 	prettier: {
@@ -270,7 +270,7 @@ export const package_json = {
 
 export const src_json = {
 	name: '@ryanatkn/gro',
-	version: '0.141.0',
+	version: '0.142.0',
 	modules: {
 		'.': {
 			path: 'index.ts',

package/src/lib/package_json.ts

@@ -21,7 +21,7 @@ export const Email = z.string();
 export type Email = Flavored<z.infer<typeof Email>, 'Email'>;
 
 // TODO move this where?
-export const transform_empty_object_to_undefined = (val: any): any => {
+export const transform_empty_object_to_undefined = <T>(val: T): T | undefined => {
 	if (val && Object.keys(val).length === 0) {
 		return;
 	}
@@ -63,8 +63,19 @@ export const Package_Json_Funding = z.union([
 ]);
 export type Package_Json_Funding = z.infer<typeof Package_Json_Funding>;
 
+// exports: {
+// 	'./': './index.js',
+// 	'./record': {default: './record.js'},
+//  './export_condition': {default: {development: './ec1', default: './ec2.js'}},
+// }
 export const Package_Json_Exports = z.record(
-	z.union([z.string(), z.record(z.string())]).optional(),
+	z
+		.union([
+			z.string(),
+			z.record(z.string().optional()),
+			z.record(z.record(z.string().optional()).optional()),
+		])
+		.optional(),
 );
 export type Package_Json_Exports = z.infer<typeof Package_Json_Exports>;
 
@@ -131,6 +142,7 @@ export const Package_Json = z
 		bin: z.record(z.string()).optional(),
 		sideEffects: z.array(z.string()).optional(),
 		files: z.array(z.string()).optional(),
+		main: z.string().optional(),
 		exports: Package_Json_Exports.transform(transform_empty_object_to_undefined).optional(),
 	})
 	.passthrough();
@@ -145,6 +157,7 @@ export const EMPTY_PACKAGE_JSON: Package_Json = {name: '', version: ''};
 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
 ): Package_Json => {
 	let package_json: Package_Json;
 	if (cache && dir in cache) {
@@ -155,8 +168,12 @@ export const load_package_json = (
 	} catch (_err) {
 		return EMPTY_PACKAGE_JSON;
 	}
-	package_json = parse_package_json(Package_Json, package_json);
-	if (cache) cache[dir] = package_json;
+	if (parse) {
+		package_json = parse_package_json(Package_Json, package_json);
+	}
+	if (cache) {
+		cache[dir] = package_json;
+	}
 	return package_json;
 };
 

package/src/lib/parse_imports.ts

@@ -2,6 +2,8 @@ import {init, parse} from 'es-module-lexer';
 import type {Flavored} from '@ryanatkn/belt/types.js';
 
 import type {Path_Id} from './path.js';
+import {SVELTE_MATCHER} from './svelte_helpers.js';
+import {JS_MATCHER, TS_MATCHER} from './path_constants.js';
 
 export const init_lexer = (): Promise<void> => init;
 
@@ -29,12 +31,12 @@ export const parse_imports = (
 		}
 	};
 
-	if (id.endsWith('.svelte')) {
+	if (SVELTE_MATCHER.test(id)) {
 		const matches = contents.matchAll(script_matcher);
 		for (const m of matches) {
 			parse_from(m[1]);
 		}
-	} else {
+	} else if (TS_MATCHER.test(id) || JS_MATCHER.test(id)) {
 		parse_from(contents);
 	}
 

package/src/lib/publish.task.ts

@@ -6,7 +6,6 @@ import {existsSync} from 'node:fs';
 import {Task_Error, type Task} from './task.js';
 import {load_package_json, parse_repo_url} from './package_json.js';
 import {find_cli, spawn_cli} from './cli.js';
-import {IS_THIS_GRO} from './paths.js';
 import {has_sveltekit_library} from './sveltekit_helpers.js';
 import {update_changelog} from './changelog.js';
 import {load_from_env} from './env.js';
@@ -84,11 +83,6 @@ export const task: Task<Args> = {
 			);
 		}
 
-		// TODO hacky, ensures Gro bootstraps itself
-		if (IS_THIS_GRO) {
-			await spawn('npm', ['run', 'build']);
-		}
-
 		const changelog_exists = existsSync(changelog);
 
 		const found_changeset_cli = find_cli(changeset_cli);
@@ -139,9 +133,9 @@ export const task: Task<Args> = {
 
 			// This is the first line that alters the repo.
 
-			const npmVersionResult = await spawn_cli(found_changeset_cli, ['version'], log);
-			if (!npmVersionResult?.ok) {
-				throw Error('npm version failed: no commits were made: see the error above');
+			const changeset_ersion_result = await spawn_cli(found_changeset_cli, ['version'], log);
+			if (!changeset_ersion_result?.ok) {
+				throw Error('changeset version failed: no commits were made: see the error above');
 			}
 
 			if (!preserve_changelog) {

package/src/lib/reinstall.task.ts

@@ -11,9 +11,9 @@ export type Args = z.infer<typeof Args>;
 export const task: Task<Args> = {
 	summary: `refreshes ${LOCKFILE_FILENAME} with the latest and cleanest deps`,
 	Args,
-	run: async ({log}): Promise<void> => {
+	run: async ({log, config}): Promise<void> => {
 		log.info('running the initial npm install');
-		const initial_install_result = await spawn('npm', ['i']);
+		const initial_install_result = await spawn(config.pm_cli, ['i']);
 		if (!initial_install_result.ok) {
 			throw new Task_Error('Failed initial npm install');
 		}
@@ -23,7 +23,7 @@ export const task: Task<Args> = {
 		log.info(
 			`running npm install after deleting ${LOCKFILE_FILENAME} and ${NODE_MODULES_DIRNAME}, this can take a while...`,
 		);
-		const second_install_result = await spawn('npm', ['i']);
+		const second_install_result = await spawn(config.pm_cli, ['i']);
 		if (!second_install_result.ok) {
 			throw new Task_Error(
 				`Failed npm install after deleting ${LOCKFILE_FILENAME} and ${NODE_MODULES_DIRNAME}`,
@@ -34,7 +34,7 @@ export const task: Task<Args> = {
 		// like esbuild's many packages for each platform.
 		await rm(LOCKFILE_FILENAME);
 		log.info(`running npm install one last time to clean ${LOCKFILE_FILENAME}`);
-		const final_install_result = await spawn('npm', ['i']);
+		const final_install_result = await spawn(config.pm_cli, ['i']);
 		if (!final_install_result.ok) {
 			throw new Task_Error('Failed npm install');
 		}