@ryanatkn/gro 0.129.15 → 0.130.1

package/dist/build.task.js

@@ -4,8 +4,8 @@ import { clean_fs } from './clean_fs.js';
 export const Args = z
     .object({
     install: z.boolean({ description: 'dual of no-install' }).default(true),
-    'no-install': z
-        .boolean({ description: 'opt out of npm installing before building' })
+    'no-install': z // convenience, same as `gro build -- gro sync --no-install` but the latter takes precedence
+        .boolean({ description: 'opt out of `npm install` before building' })
         .default(false),
 })
     .strict();

package/dist/changeset.task.d.ts

@@ -10,31 +10,31 @@ export declare const Args: z.ZodObject<{
     dir: z.ZodDefault<z.ZodString>;
     access: z.ZodOptional<z.ZodEnum<["restricted", "public"]>>;
     changelog: z.ZodDefault<z.ZodString>;
-    install: z.ZodDefault<z.ZodBoolean>;
-    'no-install': z.ZodDefault<z.ZodBoolean>;
+    dep: z.ZodDefault<z.ZodBoolean>;
+    'no-dep': z.ZodDefault<z.ZodBoolean>;
     origin: z.ZodDefault<z.ZodBranded<z.ZodString, "Git_Origin">>;
     changeset_cli: z.ZodDefault<z.ZodString>;
 }, "strict", z.ZodTypeAny, {
     _: string[];
     dir: string;
-    install: boolean;
-    'no-install': boolean;
     origin: string & z.BRAND<"Git_Origin">;
     minor: boolean;
     major: boolean;
     changelog: string;
+    dep: boolean;
+    'no-dep': boolean;
     changeset_cli: string;
     access?: "public" | "restricted" | undefined;
 }, {
     _?: string[] | undefined;
     dir?: string | undefined;
-    install?: boolean | undefined;
-    'no-install'?: boolean | undefined;
     origin?: string | undefined;
     minor?: boolean | undefined;
     major?: boolean | undefined;
     access?: "public" | "restricted" | undefined;
     changelog?: string | undefined;
+    dep?: boolean | undefined;
+    'no-dep'?: boolean | undefined;
     changeset_cli?: string | undefined;
 }>;
 export type Args = z.infer<typeof Args>;

package/dist/changeset.task.js

@@ -21,10 +21,10 @@ export const Args = z
     dir: z.string({ description: 'changeset dir' }).default(CHANGESET_DIR),
     access: Changeset_Access.describe("changeset 'access' config value, the default depends on package.json#private").optional(),
     changelog: z
-        .string({ description: 'changeset "changelog" config value' })
+        .string({ description: 'changelog dep package name, used as changeset\'s "changelog" config' })
         .default('@changesets/changelog-git'),
-    install: z.boolean({ description: 'dual of no-install' }).default(true),
-    'no-install': z
+    dep: z.boolean({ description: 'dual of no-dep' }).default(true),
+    'no-dep': z
         .boolean({ description: 'opt out of npm installing the changelog package' })
         .default(false),
     origin: Git_Origin.describe('git origin to deploy to').default('origin'),
@@ -44,7 +44,7 @@ export const task = {
     summary: 'call changeset with gro patterns',
     Args,
     run: async (ctx) => {
-        const { invoke_task, args: { _, minor, major, dir, access: access_arg, changelog, install, origin, changeset_cli }, log, sveltekit_config, } = ctx;
+        const { invoke_task, args: { _, minor, major, dir, access: access_arg, changelog, dep, origin, changeset_cli }, log, sveltekit_config, } = ctx;
         const message = _.join(' ');
         if (!message && (minor || major))
             throw new Task_Error('cannot bump version without a message');
@@ -76,12 +76,12 @@ export const task = {
                 });
             }
             await spawn('git', ['add', dir]);
-            if (install) {
+            if (dep) {
                 await spawn('npm', ['i', '-D', changelog]);
             }
         }
         // TODO small problem here where generated files don't get committed
-        await invoke_task('sync'); // after the `npm i` above, and in all cases
+        await invoke_task('sync', { install: inited || !dep }); // after the `npm i` above, and in all cases
         if (message) {
             // TODO see the helper below, simplify this to CLI flags when support is added to Changesets
             const changeset_adder = create_changeset_adder(package_json.name, dir, message, bump);

package/dist/check.task.d.ts

@@ -15,11 +15,15 @@ export declare const Args: z.ZodObject<{
     'no-lint': z.ZodDefault<z.ZodBoolean>;
     sync: z.ZodDefault<z.ZodBoolean>;
     'no-sync': z.ZodDefault<z.ZodBoolean>;
+    install: z.ZodDefault<z.ZodBoolean>;
+    'no-install': z.ZodDefault<z.ZodBoolean>;
     workspace: z.ZodDefault<z.ZodBoolean>;
 }, "strict", z.ZodTypeAny, {
     sync: boolean;
     package_json: boolean;
     format: boolean;
+    install: boolean;
+    'no-install': boolean;
     typecheck: boolean;
     'no-typecheck': boolean;
     test: boolean;
@@ -36,6 +40,8 @@ export declare const Args: z.ZodObject<{
     sync?: boolean | undefined;
     package_json?: boolean | undefined;
     format?: boolean | undefined;
+    install?: boolean | undefined;
+    'no-install'?: boolean | undefined;
     typecheck?: boolean | undefined;
     'no-typecheck'?: boolean | undefined;
     test?: boolean | undefined;

package/dist/check.task.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"check.task.d.ts","sourceRoot":"../src/lib/","sources":["check.task.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAItB,OAAO,EAAa,KAAK,IAAI,EAAC,MAAM,WAAW,CAAC;AAIhD,eAAO,MAAM,IAAI;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAoBP,CAAC;AACX,MAAM,MAAM,IAAI,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;AAExC,eAAO,MAAM,IAAI,EAAE,IAAI,CAAC,IAAI,CAyD3B,CAAC"}
+{"version":3,"file":"check.task.d.ts","sourceRoot":"../src/lib/","sources":["check.task.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAItB,OAAO,EAAa,KAAK,IAAI,EAAC,MAAM,WAAW,CAAC;AAIhD,eAAO,MAAM,IAAI;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAsBP,CAAC;AACX,MAAM,MAAM,IAAI,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;AAExC,eAAO,MAAM,IAAI,EAAE,IAAI,CAAC,IAAI,CAyD3B,CAAC"}

package/dist/check.task.js

@@ -20,6 +20,8 @@ export const Args = z
     'no-lint': z.boolean({ description: 'opt out of linting' }).default(false),
     sync: z.boolean({ description: 'dual of no-sync' }).default(true),
     'no-sync': z.boolean({ description: 'opt out of syncing' }).default(false),
+    install: z.boolean({ description: 'dual of no-install' }).default(true),
+    'no-install': z.boolean({ description: 'opt out of `npm install` when syncing' }).default(false), // convenience, same as `gro check -- gro sync --no-install` but the latter takes precedence
     workspace: z
         .boolean({ description: 'ensure a clean git workspace, useful for CI, also implies --no-sync' })
         .default(false),
@@ -29,11 +31,11 @@ export const task = {
     summary: 'check that everything is ready to commit',
     Args,
     run: async ({ args, invoke_task, log, config }) => {
-        const { typecheck, test, gen, format, package_json, lint, sync, workspace } = args;
+        const { typecheck, test, gen, format, package_json, lint, sync, install, workspace } = args;
         // When checking the workspace, which was added for CI, never sync.
         // Setup like `npm i` and `sveltekit-sync` should be done in the CI setup.
         if (sync && !workspace) {
-            await invoke_task('sync', { gen: false }); // never generate because `gro gen --check` runs below
+            await invoke_task('sync', { install, gen: false }); // never generate because `gro gen --check` runs below
         }
         if (typecheck) {
             await invoke_task('typecheck');

package/dist/dev.task.d.ts

@@ -6,14 +6,20 @@ export declare const Args: z.ZodObject<{
     'no-watch': z.ZodDefault<z.ZodBoolean>;
     sync: z.ZodDefault<z.ZodBoolean>;
     'no-sync': z.ZodDefault<z.ZodBoolean>;
+    install: z.ZodDefault<z.ZodBoolean>;
+    'no-install': z.ZodDefault<z.ZodBoolean>;
 }, "strict", z.ZodTypeAny, {
     sync: boolean;
     watch: boolean;
+    install: boolean;
+    'no-install': boolean;
     'no-sync': boolean;
     'no-watch': boolean;
 }, {
     sync?: boolean | undefined;
     watch?: boolean | undefined;
+    install?: boolean | undefined;
+    'no-install'?: boolean | undefined;
     'no-sync'?: boolean | undefined;
     'no-watch'?: boolean | undefined;
 }>;

package/dist/dev.task.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"dev.task.d.ts","sourceRoot":"../src/lib/","sources":["dev.task.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAEtB,OAAO,KAAK,EAAC,IAAI,EAAC,MAAM,WAAW,CAAC;AACpC,OAAO,EAAU,KAAK,cAAc,EAAC,MAAM,aAAa,CAAC;AAGzD,eAAO,MAAM,IAAI;;;;;;;;;;;;;;;EAYP,CAAC;AACX,MAAM,MAAM,IAAI,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;AAExC,MAAM,MAAM,eAAe,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;AAEnD,eAAO,MAAM,IAAI,EAAE,IAAI,CAAC,IAAI,CAmB3B,CAAC"}
+{"version":3,"file":"dev.task.d.ts","sourceRoot":"../src/lib/","sources":["dev.task.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAEtB,OAAO,KAAK,EAAC,IAAI,EAAC,MAAM,WAAW,CAAC;AACpC,OAAO,EAAU,KAAK,cAAc,EAAC,MAAM,aAAa,CAAC;AAGzD,eAAO,MAAM,IAAI;;;;;;;;;;;;;;;;;;;;;EAgBP,CAAC;AACX,MAAM,MAAM,IAAI,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;AAExC,MAAM,MAAM,eAAe,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;AAEnD,eAAO,MAAM,IAAI,EAAE,IAAI,CAAC,IAAI,CAmB3B,CAAC"}

package/dist/dev.task.js

@@ -11,6 +11,10 @@ export const Args = z
         .default(false),
     sync: z.boolean({ description: 'dual of no-sync' }).default(true),
     'no-sync': z.boolean({ description: 'opt out of gro sync' }).default(false),
+    install: z.boolean({ description: 'dual of no-install' }).default(true),
+    'no-install': z // convenience, same as `gro dev -- gro sync --no-install` but the latter takes precedence
+        .boolean({ description: 'opt out of `npm install` before starting the dev server' })
+        .default(false),
 })
     .strict();
 export const task = {
@@ -18,10 +22,10 @@ export const task = {
     Args,
     run: async (ctx) => {
         const { args, invoke_task } = ctx;
-        const { watch, sync } = args;
+        const { watch, sync, install } = args;
         await clean_fs({ build_dev: true });
         if (sync) {
-            await invoke_task('sync');
+            await invoke_task('sync', { install });
         }
         const plugins = await Plugins.create({ ...ctx, dev: true, watch });
         await plugins.setup();

package/dist/package.js

@@ -1,7 +1,7 @@
 // generated by src/lib/package.gen.ts
 export const package_json = {
     name: '@ryanatkn/gro',
-    version: '0.129.15',
+    version: '0.130.1',
     description: 'task runner and toolkit extending SvelteKit',
     motto: 'generate, run, optimize',
     glyph: '🌰',
@@ -266,7 +266,7 @@ export const package_json = {
 };
 export const src_json = {
     name: '@ryanatkn/gro',
-    version: '0.129.15',
+    version: '0.130.1',
     modules: {
         '.': {
             path: 'index.ts',

package/dist/publish.task.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"publish.task.d.ts","sourceRoot":"../src/lib/","sources":["publish.task.ts"],"names":[],"mappings":"AACA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAItB,OAAO,EAAa,KAAK,IAAI,EAAC,MAAM,WAAW,CAAC;AAwBhD,eAAO,MAAM,IAAI;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EA2BP,CAAC;AACX,MAAM,MAAM,IAAI,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;AAExC,eAAO,MAAM,IAAI,EAAE,IAAI,CAAC,IAAI,CAsI3B,CAAC"}
+{"version":3,"file":"publish.task.d.ts","sourceRoot":"../src/lib/","sources":["publish.task.ts"],"names":[],"mappings":"AACA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAItB,OAAO,EAAa,KAAK,IAAI,EAAC,MAAM,WAAW,CAAC;AAwBhD,eAAO,MAAM,IAAI;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EA2BP,CAAC;AACX,MAAM,MAAM,IAAI,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;AAExC,eAAO,MAAM,IAAI,EAAE,IAAI,CAAC,IAAI,CA+I3B,CAAC"}

package/dist/publish.task.js

@@ -79,6 +79,7 @@ export const task = {
             await invoke_task('check', { workspace: true });
         }
         let version;
+        let optional_and_version_unchanged = false;
         // Bump the version so the package.json is updated before building:
         // TODO problem here is build may fail and put us in a bad state,
         // but I don't see how we could do this to robustly
@@ -116,16 +117,24 @@ export const task = {
                 // The version didn't change.
                 // For now this is the best detection we have for a no-op `changeset version`.
                 if (optional) {
-                    return; // exit gracefully
+                    optional_and_version_unchanged = true;
                 }
                 else {
+                    // Doesn't build if the version didn't change and publishing isn't optional.
                     throw new Task_Error(`\`${changeset_cli} version\` failed: are there any changes?`);
                 }
             }
         }
+        // Build after the version is bumped so the new version is in the build as needed.
         if (build) {
             await invoke_task('build');
         }
+        // Return early if there are no changes and publishing is optional, but after building,
+        // so if callers want to optimize away building
+        // they need to do so manually like in `gro release`.
+        // TODO this could be cleaned up if tasks had a return value to callers, it could specifiy that it didn't build
+        if (optional_and_version_unchanged)
+            return;
         if (dry) {
             log.info('publishing branch ' + branch);
             log.info(green('dry run complete!'));

package/dist/sync.task.d.ts

@@ -8,10 +8,12 @@ export declare const Args: z.ZodObject<{
     gen: z.ZodDefault<z.ZodBoolean>;
     'no-gen': z.ZodDefault<z.ZodBoolean>;
     install: z.ZodDefault<z.ZodBoolean>;
+    'no-install': z.ZodDefault<z.ZodBoolean>;
 }, "strict", z.ZodTypeAny, {
     package_json: boolean;
     sveltekit: boolean;
     install: boolean;
+    'no-install': boolean;
     gen: boolean;
     'no-gen': boolean;
     'no-package_json': boolean;
@@ -20,6 +22,7 @@ export declare const Args: z.ZodObject<{
     package_json?: boolean | undefined;
     sveltekit?: boolean | undefined;
     install?: boolean | undefined;
+    'no-install'?: boolean | undefined;
     gen?: boolean | undefined;
     'no-gen'?: boolean | undefined;
     'no-package_json'?: boolean | undefined;

package/dist/sync.task.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"sync.task.d.ts","sourceRoot":"../src/lib/","sources":["sync.task.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAGtB,OAAO,EAAa,KAAK,IAAI,EAAC,MAAM,WAAW,CAAC;AAIhD,eAAO,MAAM,IAAI;;;;;;;;;;;;;;;;;;;;;;;;EAUP,CAAC;AACX,MAAM,MAAM,IAAI,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;AAExC,eAAO,MAAM,IAAI,EAAE,IAAI,CAAC,IAAI,CA0B3B,CAAC"}
+{"version":3,"file":"sync.task.d.ts","sourceRoot":"../src/lib/","sources":["sync.task.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAGtB,OAAO,EAAa,KAAK,IAAI,EAAC,MAAM,WAAW,CAAC;AAIhD,eAAO,MAAM,IAAI;;;;;;;;;;;;;;;;;;;;;;;;;;;EAWP,CAAC;AACX,MAAM,MAAM,IAAI,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;AAExC,eAAO,MAAM,IAAI,EAAE,IAAI,CAAC,IAAI,CA0B3B,CAAC"}

package/dist/sync.task.js

@@ -10,8 +10,9 @@ export const Args = z
     package_json: z.boolean({ description: 'dual of no-package_json' }).default(true),
     'no-package_json': z.boolean({ description: 'opt out of package.json sync' }).default(false),
     gen: z.boolean({ description: 'dual of no-gen' }).default(true),
-    'no-gen': z.boolean({ description: 'opt out of gen sync' }).default(false),
-    install: z.boolean({ description: 'run npm install' }).default(false),
+    'no-gen': z.boolean({ description: 'opt out of running gen' }).default(false),
+    install: z.boolean({ description: 'dual of no-install' }).default(true),
+    'no-install': z.boolean({ description: 'opt out of `npm install`' }).default(false),
 })
     .strict();
 export const task = {

package/dist/upgrade.task.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"upgrade.task.d.ts","sourceRoot":"../src/lib/","sources":["upgrade.task.ts"],"names":[],"mappings":"AACA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAEtB,OAAO,EAAa,KAAK,IAAI,EAAC,MAAM,WAAW,CAAC;AAKhD,eAAO,MAAM,IAAI;;;;;;;;;;;;;;;;;;;;;;;;EAeP,CAAC;AACX,MAAM,MAAM,IAAI,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;AAExC,eAAO,MAAM,IAAI,EAAE,IAAI,CAAC,IAAI,CAiD3B,CAAC"}
+{"version":3,"file":"upgrade.task.d.ts","sourceRoot":"../src/lib/","sources":["upgrade.task.ts"],"names":[],"mappings":"AACA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAEtB,OAAO,EAAa,KAAK,IAAI,EAAC,MAAM,WAAW,CAAC;AAKhD,eAAO,MAAM,IAAI;;;;;;;;;;;;;;;;;;;;;;;;EAeP,CAAC;AACX,MAAM,MAAM,IAAI,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;AAExC,eAAO,MAAM,IAAI,EAAE,IAAI,CAAC,IAAI,CA8C3B,CAAC"}

package/dist/upgrade.task.js

@@ -52,7 +52,7 @@ export const task = {
         }
         await spawn('npm', install_args);
         // Sync in a new process to pick up any changes after installing, avoiding some errors.
-        await spawn_cli('gro', ['sync']);
+        await spawn_cli('gro', ['sync', '--no-install']); // don't install because we do above
     },
 };
 const to_deps = (package_json) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ryanatkn/gro",
-  "version": "0.129.15",
+  "version": "0.130.1",
   "description": "task runner and toolkit extending SvelteKit",
   "motto": "generate, run, optimize",
   "glyph": "🌰",

package/src/lib/build.task.ts

@@ -7,8 +7,8 @@ import {clean_fs} from './clean_fs.js';
 export const Args = z
 	.object({
 		install: z.boolean({description: 'dual of no-install'}).default(true),
-		'no-install': z
-			.boolean({description: 'opt out of npm installing before building'})
+		'no-install': z // convenience, same as `gro build -- gro sync --no-install` but the latter takes precedence
+			.boolean({description: 'opt out of `npm install` before building'})
 			.default(false),
 	})
 	.strict();

package/src/lib/changeset.task.ts

@@ -33,10 +33,10 @@ export const Args = z
 			"changeset 'access' config value, the default depends on package.json#private",
 		).optional(),
 		changelog: z
-			.string({description: 'changeset "changelog" config value'})
+			.string({description: 'changelog dep package name, used as changeset\'s "changelog" config'})
 			.default('@changesets/changelog-git'),
-		install: z.boolean({description: 'dual of no-install'}).default(true),
-		'no-install': z
+		dep: z.boolean({description: 'dual of no-dep'}).default(true),
+		'no-dep': z
 			.boolean({description: 'opt out of npm installing the changelog package'})
 			.default(false),
 		origin: Git_Origin.describe('git origin to deploy to').default('origin'),
@@ -60,7 +60,7 @@ export const task: Task<Args> = {
 	run: async (ctx): Promise<void> => {
 		const {
 			invoke_task,
-			args: {_, minor, major, dir, access: access_arg, changelog, install, origin, changeset_cli},
+			args: {_, minor, major, dir, access: access_arg, changelog, dep, origin, changeset_cli},
 			log,
 			sveltekit_config,
 		} = ctx;
@@ -111,13 +111,13 @@ export const task: Task<Args> = {
 
 			await spawn('git', ['add', dir]);
 
-			if (install) {
+			if (dep) {
 				await spawn('npm', ['i', '-D', changelog]);
 			}
 		}
 
 		// TODO small problem here where generated files don't get committed
-		await invoke_task('sync'); // after the `npm i` above, and in all cases
+		await invoke_task('sync', {install: inited || !dep}); // after the `npm i` above, and in all cases
 
 		if (message) {
 			// TODO see the helper below, simplify this to CLI flags when support is added to Changesets

package/src/lib/check.task.ts

@@ -22,6 +22,8 @@ export const Args = z
 		'no-lint': z.boolean({description: 'opt out of linting'}).default(false),
 		sync: z.boolean({description: 'dual of no-sync'}).default(true),
 		'no-sync': z.boolean({description: 'opt out of syncing'}).default(false),
+		install: z.boolean({description: 'dual of no-install'}).default(true),
+		'no-install': z.boolean({description: 'opt out of `npm install` when syncing'}).default(false), // convenience, same as `gro check -- gro sync --no-install` but the latter takes precedence
 		workspace: z
 			.boolean({description: 'ensure a clean git workspace, useful for CI, also implies --no-sync'})
 			.default(false),
@@ -33,12 +35,12 @@ export const task: Task<Args> = {
 	summary: 'check that everything is ready to commit',
 	Args,
 	run: async ({args, invoke_task, log, config}) => {
-		const {typecheck, test, gen, format, package_json, lint, sync, workspace} = args;
+		const {typecheck, test, gen, format, package_json, lint, sync, install, workspace} = args;
 
 		// When checking the workspace, which was added for CI, never sync.
 		// Setup like `npm i` and `sveltekit-sync` should be done in the CI setup.
 		if (sync && !workspace) {
-			await invoke_task('sync', {gen: false}); // never generate because `gro gen --check` runs below
+			await invoke_task('sync', {install, gen: false}); // never generate because `gro gen --check` runs below
 		}
 
 		if (typecheck) {

package/src/lib/dev.task.ts

@@ -15,6 +15,10 @@ export const Args = z
 			.default(false),
 		sync: z.boolean({description: 'dual of no-sync'}).default(true),
 		'no-sync': z.boolean({description: 'opt out of gro sync'}).default(false),
+		install: z.boolean({description: 'dual of no-install'}).default(true),
+		'no-install': z // convenience, same as `gro dev -- gro sync --no-install` but the latter takes precedence
+			.boolean({description: 'opt out of `npm install` before starting the dev server'})
+			.default(false),
 	})
 	.strict();
 export type Args = z.infer<typeof Args>;
@@ -26,12 +30,12 @@ export const task: Task<Args> = {
 	Args,
 	run: async (ctx) => {
 		const {args, invoke_task} = ctx;
-		const {watch, sync} = args;
+		const {watch, sync, install} = args;
 
 		await clean_fs({build_dev: true});
 
 		if (sync) {
-			await invoke_task('sync');
+			await invoke_task('sync', {install});
 		}
 
 		const plugins = await Plugins.create({...ctx, dev: true, watch});

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.129.15',
+	version: '0.130.1',
 	description: 'task runner and toolkit extending SvelteKit',
 	motto: 'generate, run, optimize',
 	glyph: '🌰',
@@ -272,7 +272,7 @@ export const package_json = {
 
 export const src_json = {
 	name: '@ryanatkn/gro',
-	version: '0.129.15',
+	version: '0.130.1',
 	modules: {
 		'.': {
 			path: 'index.ts',

package/src/lib/publish.task.ts

@@ -114,6 +114,7 @@ export const task: Task<Args> = {
 		}
 
 		let version!: string;
+		let optional_and_version_unchanged = false;
 
 		// Bump the version so the package.json is updated before building:
 		// TODO problem here is build may fail and put us in a bad state,
@@ -159,17 +160,25 @@ export const task: Task<Args> = {
 				// The version didn't change.
 				// For now this is the best detection we have for a no-op `changeset version`.
 				if (optional) {
-					return; // exit gracefully
+					optional_and_version_unchanged = true;
 				} else {
+					// Doesn't build if the version didn't change and publishing isn't optional.
 					throw new Task_Error(`\`${changeset_cli} version\` failed: are there any changes?`);
 				}
 			}
 		}
 
+		// Build after the version is bumped so the new version is in the build as needed.
 		if (build) {
 			await invoke_task('build');
 		}
 
+		// Return early if there are no changes and publishing is optional, but after building,
+		// so if callers want to optimize away building
+		// they need to do so manually like in `gro release`.
+		// TODO this could be cleaned up if tasks had a return value to callers, it could specifiy that it didn't build
+		if (optional_and_version_unchanged) return;
+
 		if (dry) {
 			log.info('publishing branch ' + branch);
 			log.info(green('dry run complete!'));

package/src/lib/sync.task.ts

@@ -12,8 +12,9 @@ export const Args = z
 		package_json: z.boolean({description: 'dual of no-package_json'}).default(true),
 		'no-package_json': z.boolean({description: 'opt out of package.json sync'}).default(false),
 		gen: z.boolean({description: 'dual of no-gen'}).default(true),
-		'no-gen': z.boolean({description: 'opt out of gen sync'}).default(false),
-		install: z.boolean({description: 'run npm install'}).default(false),
+		'no-gen': z.boolean({description: 'opt out of running gen'}).default(false),
+		install: z.boolean({description: 'dual of no-install'}).default(true),
+		'no-install': z.boolean({description: 'opt out of `npm install`'}).default(false),
 	})
 	.strict();
 export type Args = z.infer<typeof Args>;

package/src/lib/upgrade.task.ts

@@ -58,20 +58,17 @@ export const task: Task<Args> = {
 		log.info(`upgrading:`, upgrade_items.join(' '));
 
 		const install_args = ['install'].concat(upgrade_items);
-
 		if (dry) {
 			install_args.push('--dry-run');
 			log.info(`deps`, deps);
 		}
-
 		if (force) {
 			install_args.push('--force');
 		}
-
 		await spawn('npm', install_args);
 
 		// Sync in a new process to pick up any changes after installing, avoiding some errors.
-		await spawn_cli('gro', ['sync']);
+		await spawn_cli('gro', ['sync', '--no-install']); // don't install because we do above
 	},
 };