@savvy-web/cli 3.2.8 → 3.3.0

package/cli/index.js

@@ -122,12 +122,15 @@ const LocalExecLive = Workspaces.localExecLayer();
 const ToolDiscoveryGroupLive = ToolDiscovery.layer.pipe(Layer.provide(LocalExecLive), Layer.provide(WorkspaceLive));
 /**
 * The merged runtime Layer stack satisfying every command's service
-* requirements. Provided to `rootCommand`'s assembled `Command.run` Effect by
-* `main()` in `../main.js`; this module never calls `NodeRuntime.runMain`
-* itself.
+* requirements, still open on the platform services. Provided to
+* `rootCommand`'s assembled `Command.run` Effect by `main()` in `../main.js`,
+* which hands {@link CliPlatform} to `CliRuntime.main` so the platform is
+* provided once, at the edge; this module never runs anything itself.
 */
-const AppLive = Layer.mergeAll(ToolDiscoveryGroupLive, InspectorAndAnalyzerLive, ReposGroupLive).pipe(Layer.provideMerge(BaseLive), Layer.provideMerge(NodeServices.layer));
+const AppLive = Layer.mergeAll(ToolDiscoveryGroupLive, InspectorAndAnalyzerLive, ReposGroupLive).pipe(Layer.provideMerge(BaseLive));
+/** The platform `main()` passes to `CliRuntime.main`: Node's filesystem, path, stdio, terminal and spawner. */
+const CliPlatform = NodeServices.layer;
 /* v8 ignore stop */
 
 //#endregion
-export { AppLive, rootCommand };
+export { AppLive, CliPlatform, rootCommand };

package/commands/changeset/commands/check.js

@@ -1,5 +1,7 @@
+import { Output } from "../../../internal/output.js";
 import { Argument, Command } from "effect/unstable/cli";
 import { resolve } from "node:path";
+import { CliExit } from "@effected/cli";
 import { Changesets } from "@savvy-web/silk-effects";
 import { Effect } from "effect";
 
@@ -15,7 +17,7 @@ import { Effect } from "effect";
 * The command resolves the directory argument, delegates to
 * {@link ChangesetLinter.validate}, groups the resulting
 * {@link LintMessage} objects by file path, and logs each file's errors
-* indented under the file name. Sets `process.exitCode = 1` when errors
+* indented under the file name. Sets exit code 1 through `CliExit.set` when errors
 * are found.
 *
 * @example
@@ -32,7 +34,7 @@ const dirArg = Argument.Directory("dir").pipe(Argument.withDefault(".changeset")
 * Run the check validation pipeline on all changeset files in `dir`.
 *
 * Groups lint messages by file and logs a human-readable summary. Sets
-* `process.exitCode = 1` when one or more errors are found.
+* exit code 1 through `CliExit.set` when one or more errors are found.
 *
 * @param dir - Path to the changeset directory (resolved relative to cwd)
 * @returns An Effect that performs validation and logs results
@@ -53,15 +55,17 @@ function runChangesetCheck(dir) {
 			else byFile.set(msg.file, [msg]);
 		}
 		for (const [file, fileMessages] of byFile) {
-			yield* Effect.log(`\n${file}`);
-			for (const msg of fileMessages) yield* Effect.log(`  ${msg.line}:${msg.column}  ${msg.rule}  ${msg.message}`);
+			yield* Output.line("");
+			yield* Output.heading(file);
+			for (const msg of fileMessages) yield* Output.detail(`${msg.line}:${msg.column}  ${msg.rule}  ${msg.message}`);
 		}
 		const errorCount = messages.length;
 		const filesWithErrors = byFile.size;
 		if (errorCount > 0) {
-			yield* Effect.log(`\n${filesWithErrors} file(s) with errors, ${errorCount} error(s) found`);
-			process.exitCode = 1;
-		} else yield* Effect.log("All changeset files passed validation.");
+			yield* Output.line("");
+			yield* Output.fail(`${filesWithErrors} file(s) with errors, ${errorCount} error(s) found`);
+			yield* CliExit.set(1);
+		} else yield* Output.ok("All changeset files passed validation");
 	});
 }
 /* v8 ignore next 3 -- CLI registration; handler tested via runChangesetCheck */

package/commands/changeset/commands/config-validate.js

@@ -1,5 +1,7 @@
+import { Output } from "../../../internal/output.js";
 import { Argument, Command } from "effect/unstable/cli";
 import { resolve } from "node:path";
+import { CliExit } from "@effected/cli";
 import { Changesets } from "@savvy-web/silk-effects";
 import { Effect } from "effect";
 
@@ -29,7 +31,7 @@ const { ConfigInspector } = Changesets;
 const dirArg = Argument.Directory("dir").pipe(Argument.withDefault("."));
 /**
 * Run validation. Logs a one-line OK on success; logs the error and sets
-* `process.exitCode = 1` on failure.
+* exit code 1 through `CliExit.set` on failure.
 *
 * @internal
 */
@@ -49,11 +51,11 @@ function runConfigValidate(dir) {
 			const { config } = result;
 			const pkgCount = config.packages.length;
 			const note = config.legacyVersionFilesUsed ? " (warning: legacy versionFiles in use)" : "";
-			yield* Effect.log(`OK  ${config.configPath} — ${pkgCount} package${pkgCount === 1 ? "" : "s"} declared${note}`);
+			yield* Output.ok(`${config.configPath} — ${pkgCount} package${pkgCount === 1 ? "" : "s"} declared${note}`);
 			return;
 		}
-		yield* Effect.log(`FAIL  ${result.field}: ${result.reason}`);
-		process.exitCode = 1;
+		yield* Output.fail(`${result.field}: ${result.reason}`);
+		yield* CliExit.set(1);
 	});
 }
 /* v8 ignore next 5 -- CLI registration */

package/commands/changeset/commands/deps-detect.js

@@ -1,5 +1,7 @@
+import { Output } from "../../../internal/output.js";
 import { Command, Flag } from "effect/unstable/cli";
 import { resolve } from "node:path";
+import { CliExit } from "@effected/cli";
 import { Changesets } from "@savvy-web/silk-effects";
 import { Console, Effect, Option } from "effect";
 
@@ -65,11 +67,9 @@ function runDepsDetect(cwd, from, to, pkg, json, markdown) {
 			...Option.isSome(pkg) ? { package: pkg.value } : {},
 			...Option.isSome(from) ? { from: from.value } : {},
 			...Option.isSome(to) ? { to: to.value } : {}
-		}).pipe(Effect.tapError(() => Effect.sync(() => {
-			process.exitCode = 1;
-		})))).toWrite.map((entry) => entry.diff);
+		}).pipe(Effect.tapError(() => CliExit.set(1)))).toWrite.map((entry) => entry.diff);
 		if (markdown && !json) {
-			yield* Effect.log(renderMarkdownBlocks(diffs));
+			yield* Output.line(renderMarkdownBlocks(diffs));
 			return;
 		}
 		yield* Console.log(JSON.stringify(diffs, null, 2));

package/commands/changeset/commands/deps-regen.js

@@ -1,5 +1,7 @@
+import { Output } from "../../../internal/output.js";
 import { Command, Flag } from "effect/unstable/cli";
 import { resolve } from "node:path";
+import { CliExit } from "@effected/cli";
 import { Changesets } from "@savvy-web/silk-effects";
 import { Console, Effect, Option } from "effect";
 
@@ -66,9 +68,7 @@ function runDepsRegen(cwd, base, pkg, dryRun, json) {
 			cwd,
 			...Option.isSome(base) ? { base: base.value } : {},
 			...Option.isSome(pkg) ? { package: pkg.value } : {}
-		}).pipe(Effect.tapError(() => Effect.sync(() => {
-			process.exitCode = 1;
-		})));
+		}).pipe(Effect.tapError(() => CliExit.set(1)));
 		if (!dryRun) yield* service.execute(plan);
 		if (json) yield* Console.log(JSON.stringify(plan, null, 2));
 		else yield* renderHumanPlan(plan);
@@ -76,20 +76,21 @@ function runDepsRegen(cwd, base, pkg, dryRun, json) {
 }
 function renderHumanPlan(plan) {
 	return Effect.gen(function* () {
-		if (plan.toDelete.length === 0 && plan.toWrite.length === 0) yield* Effect.log("No dependency changes to regenerate.");
+		if (plan.toDelete.length === 0 && plan.toWrite.length === 0) yield* Output.ok("No dependency changes to regenerate");
 		else {
 			if (plan.toDelete.length > 0) {
-				yield* Effect.log(`Deleted ${plan.toDelete.length} pure dependency changeset(s):`);
-				for (const entry of plan.toDelete) yield* Effect.log(`  - ${entry.file}  (${entry.package})`);
+				yield* Output.ok(`Deleted ${plan.toDelete.length} pure dependency changeset(s):`);
+				for (const entry of plan.toDelete) yield* Output.detail(`${entry.file}  (${entry.package})`);
 			}
 			if (plan.toWrite.length > 0) {
-				yield* Effect.log(`Wrote ${plan.toWrite.length} fresh dependency changeset(s):`);
-				for (const entry of plan.toWrite) yield* Effect.log(`  + ${entry.file}  (${entry.package} — ${entry.diff.rows.length} row${entry.diff.rows.length === 1 ? "" : "s"})`);
+				yield* Output.ok(`Wrote ${plan.toWrite.length} fresh dependency changeset(s):`);
+				for (const entry of plan.toWrite) yield* Output.detail(`+ ${entry.file}  (${entry.package} — ${entry.diff.rows.length} row${entry.diff.rows.length === 1 ? "" : "s"})`);
 			}
 		}
 		if (plan.skippedMixed.length > 0) {
-			yield* Effect.log(`\nSkipped ${plan.skippedMixed.length} mixed changeset(s) (have Dependencies but also other content):`);
-			for (const file of plan.skippedMixed) yield* Effect.log(`  ~ ${file}`);
+			yield* Output.line("");
+			yield* Output.skip(`Skipped ${plan.skippedMixed.length} mixed changeset(s) (have Dependencies but also other content):`);
+			for (const file of plan.skippedMixed) yield* Output.detail(`~ ${file}`);
 		}
 	});
 }

package/commands/changeset/commands/init.js

@@ -1,4 +1,6 @@
+import { Output } from "../../../internal/output.js";
 import { join } from "node:path";
+import { CliExit } from "@effected/cli";
 import { Changesets } from "@savvy-web/silk-effects";
 import { Data, Effect, Option, Result, Schema } from "effect";
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
@@ -568,7 +570,7 @@ function runChangesetInit(opts) {
 	return Effect.gen(function* () {
 		const root = yield* resolveWorkspaceRoot(process.cwd());
 		const repo = yield* detectGitHubRepo(root);
-		if (!repo && !quiet) yield* Effect.log("Warning: could not detect GitHub repo from git remote, using placeholder");
+		if (!repo && !quiet) yield* Effect.logWarning("could not detect GitHub repo from git remote, using placeholder");
 		const repoSlug = repo ?? "owner/repo";
 		if (check) {
 			const changesetDir = join(root, ".changeset");
@@ -579,39 +581,39 @@ function runChangesetInit(opts) {
 				...checkChangesetMarkdownlint(changesetDir)
 			];
 			if (issues.length === 0) {
-				yield* Effect.log("All @savvy-web/changesets config files are up to date.");
+				yield* Output.ok("All @savvy-web/changesets config files are up to date");
 				return;
 			}
-			for (const issue of issues) yield* Effect.logWarning(`${issue.file}: ${issue.message}`);
-			yield* Effect.logWarning("Run \"savvy init --force\" to fix.");
+			for (const issue of issues) yield* Output.warn(`${issue.file}: ${issue.message}`);
+			yield* Output.detail("Run \"savvy init --force\" to fix");
 			return;
 		}
 		const changesetDir = yield* ensureChangesetDir(root);
-		yield* Effect.log("Ensured .changeset/ directory");
+		yield* Output.ok("Ensured .changeset/ directory");
 		const errors = [];
 		const configResult = yield* handleConfig(changesetDir, repoSlug, force).pipe(Effect.result);
 		if (Result.isSuccess(configResult)) {
-			yield* Effect.log(configResult.success);
+			yield* Output.ok(configResult.success);
 			if (!quiet) yield* warnIfLegacyVersionFiles(changesetDir);
 		} else errors.push(configResult.failure);
 		if (!skipMarkdownlint) {
 			const baseResult = yield* handleBaseMarkdownlint(root).pipe(Effect.result);
-			if (Result.isSuccess(baseResult)) yield* Effect.log(baseResult.success);
+			if (Result.isSuccess(baseResult)) yield* Output.ok(baseResult.success);
 			else errors.push(baseResult.failure);
 		}
 		const mdlintResult = yield* handleChangesetMarkdownlint(changesetDir, root, force).pipe(Effect.result);
-		if (Result.isSuccess(mdlintResult)) yield* Effect.log(mdlintResult.success);
+		if (Result.isSuccess(mdlintResult)) yield* Output.ok(mdlintResult.success);
 		else errors.push(mdlintResult.failure);
 		if (errors.length > 0) {
 			for (const err of errors) yield* Effect.logError(err.message);
-			if (!quiet) process.exitCode = 1;
+			if (!quiet) yield* CliExit.set(1);
 			return;
 		}
-		yield* Effect.log("Init complete.");
+		yield* Output.ok("Init complete");
 	}).pipe(Effect.catch((error) => Effect.gen(function* () {
 		if (!quiet) {
 			yield* Effect.logError(error instanceof InitError ? error.message : `Init failed: ${String(error)}`);
-			process.exitCode = 1;
+			yield* CliExit.set(1);
 		}
 	})));
 }

package/commands/changeset/commands/lint.js

@@ -1,5 +1,6 @@
 import { Argument, Command, Flag } from "effect/unstable/cli";
 import { resolve } from "node:path";
+import { CliExit } from "@effected/cli";
 import { Changesets } from "@savvy-web/silk-effects";
 import { Console, Effect } from "effect";
 
@@ -15,7 +16,7 @@ import { Console, Effect } from "effect";
 * The command resolves the directory argument, delegates to
 * {@link ChangesetLinter.validate}, and logs each {@link LintMessage} as a
 * single colon-delimited line. When no errors are found and `--quiet` is not
-* set, a success message is printed. Sets `process.exitCode = 1` when errors
+* set, a success message is printed. Sets exit code 1 through `CliExit.set` when errors
 * are found.
 *
 * @example
@@ -35,7 +36,7 @@ const quietOption = Flag.Boolean("quiet").pipe(Flag.withAlias("q"), Flag.withDes
 * Run machine-readable lint validation on all changeset files in `dir`.
 *
 * Outputs one line per error in `file:line:col rule message` format. Sets
-* `process.exitCode = 1` when one or more errors are found.
+* exit code 1 through `CliExit.set` when one or more errors are found.
 *
 * @param dir - Path to the changeset directory (resolved relative to cwd)
 * @param quiet - When `true`, suppress the "No lint errors found" message
@@ -49,7 +50,7 @@ function runLint(dir, quiet) {
 		const messages = yield* Effect.try(() => ChangesetLinter.validate(resolved));
 		for (const msg of messages) yield* Console.log(`${msg.file}:${msg.line}:${msg.column} ${msg.rule} ${msg.message}`);
 		if (!quiet && messages.length === 0) yield* Console.log("No lint errors found.");
-		if (messages.length > 0) process.exitCode = 1;
+		if (messages.length > 0) yield* CliExit.set(1);
 	});
 }
 /* v8 ignore next 3 -- CLI registration; handler tested via runLint */

package/commands/changeset/commands/transform.js

@@ -1,6 +1,8 @@
+import { Output } from "../../../internal/output.js";
 import { requireValidConfig } from "../utils/config-gate.js";
 import { Argument, Command, Flag } from "effect/unstable/cli";
 import { dirname, resolve } from "node:path";
+import { CliExit } from "@effected/cli";
 import { Changesets } from "@savvy-web/silk-effects";
 import { Effect } from "effect";
 import { readFileSync, writeFileSync } from "node:fs";
@@ -63,18 +65,18 @@ function runTransform(file, dryRun, check) {
 		const content = yield* Effect.try(() => readFileSync(resolved, "utf-8"));
 		const result = ChangelogTransformer.transformContent(content);
 		if (dryRun) {
-			yield* Effect.log(result);
+			yield* Output.line(result);
 			return;
 		}
 		if (check) {
 			if (result !== content) {
-				yield* Effect.log(`${resolved} would be modified by transform.`);
-				process.exitCode = 1;
-			} else yield* Effect.log(`${resolved} is already formatted.`);
+				yield* Output.warn(`${resolved} would be modified by transform`);
+				yield* CliExit.set(1);
+			} else yield* Output.ok(`${resolved} is already formatted`);
 			return;
 		}
 		yield* Effect.try(() => writeFileSync(resolved, result, "utf-8"));
-		yield* Effect.log(`Transformed ${resolved}`);
+		yield* Output.ok(`Transformed ${resolved}`);
 	});
 }
 /* v8 ignore next 5 -- CLI registration; handler tested via runTransform */

package/commands/changeset/commands/validate-file.js

@@ -1,4 +1,6 @@
+import { Output } from "../../../internal/output.js";
 import { Argument, Command } from "effect/unstable/cli";
+import { CliExit } from "@effected/cli";
 import { Changesets } from "@savvy-web/silk-effects";
 import { Effect } from "effect";
 
@@ -24,7 +26,7 @@ const fileArg = Argument.File("file");
 * Run lint validation on a single changeset file.
 *
 * Outputs one line per error in `file:line:col rule message` format.
-* Logs "Valid." when the file passes. Sets `process.exitCode = 1`
+* Logs "Valid." when the file passes. Sets exit code 1 through `CliExit.set`
 * when errors are found or the file cannot be read.
 *
 * @param filePath - Path to the changeset `.md` file
@@ -35,14 +37,14 @@ const fileArg = Argument.File("file");
 function runValidateFile(filePath) {
 	return Effect.gen(function* () {
 		const result = yield* Effect.try(() => ChangesetLinter.validateFile(filePath)).pipe(Effect.catch((error) => Effect.gen(function* () {
-			yield* Effect.log(`Error: ${error instanceof Error ? error.message : String(error)}`);
-			process.exitCode = 1;
+			yield* Effect.logError(`Error: ${error instanceof Error ? error.message : String(error)}`);
+			yield* CliExit.set(1);
 			return null;
 		})));
 		if (result === null) return;
-		for (const msg of result) yield* Effect.log(`${msg.file}:${msg.line}:${msg.column} ${msg.rule} ${msg.message}`);
-		if (result.length > 0) process.exitCode = 1;
-		else yield* Effect.log("Valid.");
+		for (const msg of result) yield* Output.line(`${msg.file}:${msg.line}:${msg.column} ${msg.rule} ${msg.message}`);
+		if (result.length > 0) yield* CliExit.set(1);
+		else yield* Output.ok("Valid");
 	});
 }
 /* v8 ignore next 3 -- CLI registration; handler tested via runValidateFile */

package/commands/changeset/commands/version.js

@@ -1,3 +1,4 @@
+import { Output } from "../../../internal/output.js";
 import { requireValidConfig } from "../utils/config-gate.js";
 import { Command, Flag } from "effect/unstable/cli";
 import { Changesets } from "@savvy-web/silk-effects";
@@ -29,13 +30,13 @@ function runVersion(dryRun) {
 		yield* requireValidConfig(cwd);
 		const result = yield* (yield* Changesets.ReleasePlanner).apply(cwd, { dryRun });
 		if (result.releases.length === 0) {
-			yield* Effect.log("No pending changesets.");
+			yield* Output.ok("No pending changesets");
 			return;
 		}
 		const verb = dryRun ? "Would release" : "Released";
-		for (const r of result.releases) yield* Effect.log(`${verb} ${r.name}: ${r.oldVersion} -> ${r.newVersion} (${r.type})`);
-		if (!dryRun) yield* Effect.log(`Touched ${result.touchedFiles.length} file(s)`);
-		for (const u of result.versionFileUpdates) yield* Effect.log(`${dryRun ? "Would update" : "Updated"} ${u.filePath} -> ${u.version}`);
+		for (const r of result.releases) yield* Output.ok(`${verb} ${r.name}: ${r.oldVersion} -> ${r.newVersion} (${r.type})`);
+		if (!dryRun) yield* Output.detail(`Touched ${result.touchedFiles.length} file(s)`);
+		for (const u of result.versionFileUpdates) yield* Output.detail(`${dryRun ? "Would update" : "Updated"} ${u.filePath} -> ${u.version}`);
 	});
 }
 /* v8 ignore next 4 -- CLI registration; handler tested via runVersion */

package/commands/changeset/utils/config-gate.js

@@ -1,4 +1,5 @@
 import { join, resolve } from "node:path";
+import { CliExit } from "@effected/cli";
 import { Changesets } from "@savvy-web/silk-effects";
 import { Effect } from "effect";
 import { existsSync } from "node:fs";
@@ -26,7 +27,7 @@ import { existsSync } from "node:fs";
 *   projects that have not yet been bootstrapped.
 * - **Config present**: the gate invokes
 *   {@link ConfigInspector.inspect}. On `ConfigurationError`, it sets
-*   `process.exitCode = 1` and propagates the error so the caller's
+*   exit code 1 through `CliExit.set` and propagates the error so the caller's
 *   `Effect.gen` short-circuits.
 *
 * @internal
@@ -50,8 +51,7 @@ function requireValidConfig(cwd) {
 		const configPath = join(projectDir, ".changeset", "config.json");
 		if (!existsSync(configPath)) return;
 		yield* (yield* ConfigInspector).inspect(projectDir).pipe(Effect.catchTag("ConfigurationError", (err) => {
-			process.exitCode = 1;
-			return Effect.fail(err);
+			return CliExit.set(1).pipe(Effect.andThen(Effect.fail(err)));
 		}));
 	});
 }

package/commands/check.js

@@ -1,3 +1,4 @@
+import { Output } from "../internal/output.js";
 import { runChangesetCheck } from "./changeset/commands/check.js";
 import "./changeset/index.js";
 import { runCommitCheck } from "./commit/check.js";
@@ -44,8 +45,8 @@ function runCheck(steps) {
 	return Effect.gen(function* () {
 		const results = yield* Effect.all([
 			Effect.result(steps.changeset),
-			Effect.result(steps.commit),
-			Effect.result(steps.lint)
+			Output.line("").pipe(Effect.andThen(Effect.result(steps.commit))),
+			Output.line("").pipe(Effect.andThen(Effect.result(steps.lint)))
 		], { concurrency: 1 });
 		for (const result of results) if (Result.isFailure(result)) return yield* Effect.fail(result.failure);
 	});

package/commands/clean.js

@@ -1,3 +1,4 @@
+import { Output } from "../internal/output.js";
 import { Command, Flag } from "effect/unstable/cli";
 import { join, sep } from "node:path";
 import { Data, Effect } from "effect";
@@ -97,10 +98,6 @@ function removeTargets(targets, dryRun) {
 		};
 	});
 }
-/** Unicode symbols for output. */
-const CHECK_MARK = "✓";
-const BULLET = "•";
-const WARN_MARK = "⚠";
 /** Split the comma-separated `--globs` value; fall back to defaults when empty. */
 function parseGlobs(raw) {
 	const parts = raw.split(",").map((s) => s.trim()).filter((s) => s.length > 0);
@@ -147,14 +144,16 @@ function runClean(opts) {
 			}))));
 			for (const { g, report } of reports) {
 				if (g.targets.length === 0) continue;
-				yield* Effect.log(`\n${g.pkg.relativePath === "." ? "<root>" : g.pkg.relativePath}`);
-				for (const t of report.removed) yield* Effect.log(`  ${BULLET} ${verb} [${t.kind}] ${t.path}`);
-				for (const f of report.failed) yield* Effect.log(`  ${WARN_MARK} failed [${f.target.kind}] ${f.target.path}: ${f.reason}`);
+				yield* Output.line("");
+				yield* Output.heading(g.pkg.relativePath === "." ? "<root>" : g.pkg.relativePath);
+				for (const t of report.removed) yield* Output.detail(`${verb} [${t.kind}] ${t.path}`);
+				for (const f of report.failed) yield* Output.warn(`failed [${f.target.kind}] ${f.target.path}: ${f.reason}`);
 				total += report.removed.length;
 				failures.push(...report.failed);
 			}
 		}
-		yield* Effect.log(`\n${CHECK_MARK} ${opts.dryRun ? "Would remove" : "Removed"} ${total} item(s).`);
+		yield* Output.line("");
+		yield* Output.ok(`${opts.dryRun ? "Would remove" : "Removed"} ${total} item(s)`);
 		if (failures.length > 0) {
 			for (const f of failures) yield* Effect.logError(`Failed to remove ${f.target.path}: ${f.reason}`);
 			return yield* Effect.fail(new CleanError({

package/commands/commit/check.js

@@ -1,3 +1,4 @@
+import { Output } from "../../internal/output.js";
 import { HUSKY_HOOK_PATH, POST_CHECKOUT_HOOK_PATH, POST_COMMIT_HOOK_PATH, POST_MERGE_HOOK_PATH } from "./constants.js";
 import { SECTION_DEF, savvyCommitBlock } from "./init.js";
 import { ChangesetConfigReader, Commitlint, SavvyBaseSection, SavvyHooksSection, SavvyToolchainSection, savvyBasePreamble, savvyHooksHygiene, savvyInstallBlock, savvyToolchainCheck } from "@savvy-web/silk-effects";
@@ -6,10 +7,6 @@ import { VersioningStrategy } from "@effected/workspaces";
 import { CheckOutcome, ManagedSection } from "@effected/templates";
 
 //#region src/commands/commit/check.ts
-/** Unicode cross symbol. */
-const CROSS_MARK = "✗";
-/** Unicode bullet symbol. */
-const BULLET = "•";
 /** Possible commitlint configuration file names, in priority order. */
 const CONFIG_FILES = [
 	"commitlint.config.ts",
@@ -98,41 +95,41 @@ function runCommitCheck() {
 	return Effect.gen(function* () {
 		const fs = yield* FileSystem.FileSystem;
 		const ms = yield* ManagedSection;
-		yield* Effect.log("Checking commitlint configuration...\n");
+		yield* Output.heading("commitlint configuration");
 		const foundConfig = yield* findConfigFile(fs);
-		if (foundConfig) yield* Effect.log(`${"✓"} Config file: ${foundConfig}`);
-		else yield* Effect.log(`${CROSS_MARK} No commitlint config file found`);
+		if (foundConfig) yield* Output.ok(`Config file: ${foundConfig}`);
+		else yield* Output.fail(`No commitlint config file found`);
 		const hasHuskyHook = yield* fs.exists(HUSKY_HOOK_PATH);
-		if (hasHuskyHook) yield* Effect.log(`${"✓"} Husky hook: ${HUSKY_HOOK_PATH}`);
-		else yield* Effect.log(`${CROSS_MARK} No husky commit-msg hook found`);
+		if (hasHuskyHook) yield* Output.ok(`Husky hook: ${HUSKY_HOOK_PATH}`);
+		else yield* Output.fail(`No husky commit-msg hook found`);
 		let sectionsHealthy = true;
 		if (hasHuskyHook) {
 			const baseStatus = yield* ms.check(HUSKY_HOOK_PATH, SavvyBaseSection.section(savvyBasePreamble()));
-			if (CheckOutcome.$is("UpToDate")(baseStatus)) yield* Effect.log(`${"✓"} Base section: up-to-date`);
+			if (CheckOutcome.$is("UpToDate")(baseStatus)) yield* Output.ok(`Base section: up-to-date`);
 			else if (CheckOutcome.$is("Drifted")(baseStatus)) {
 				sectionsHealthy = false;
-				yield* Effect.log(`${"⚠"} Base section: outdated (run 'savvy init' to update)`);
+				yield* Output.warn(`Base section: outdated (run 'savvy init' to update)`);
 			} else {
 				sectionsHealthy = false;
-				yield* Effect.log(`${BULLET} Base section: not found (run 'savvy init' to add)`);
+				yield* Output.skip(`Base section: not found (run 'savvy init' to add)`);
 			}
 			const block = yield* ms.read(HUSKY_HOOK_PATH, SECTION_DEF);
 			if (Option.isSome(block)) {
 				const configPath = extractConfigPathFromManaged(block.value.content);
 				if (configPath) {
 					const status = yield* ms.check(HUSKY_HOOK_PATH, savvyCommitBlock(configPath));
-					if (CheckOutcome.$is("UpToDate")(status)) yield* Effect.log(`${"✓"} Commit section: up-to-date`);
+					if (CheckOutcome.$is("UpToDate")(status)) yield* Output.ok(`Commit section: up-to-date`);
 					else {
 						sectionsHealthy = false;
-						yield* Effect.log(`${"⚠"} Commit section: outdated (run 'savvy init' to update)`);
+						yield* Output.warn(`Commit section: outdated (run 'savvy init' to update)`);
 					}
 				} else {
 					sectionsHealthy = false;
-					yield* Effect.log(`${"⚠"} Commit section: outdated (run 'savvy init' to update)`);
+					yield* Output.warn(`Commit section: outdated (run 'savvy init' to update)`);
 				}
 			} else {
 				sectionsHealthy = false;
-				yield* Effect.log(`${BULLET} Commit section: not found (run 'savvy init' to add)`);
+				yield* Output.skip(`Commit section: not found (run 'savvy init' to add)`);
 			}
 		}
 		for (const hookPath of [
@@ -142,51 +139,52 @@ function runCommitCheck() {
 		]) {
 			if (!(yield* fs.exists(hookPath))) {
 				sectionsHealthy = false;
-				yield* Effect.log(`${BULLET} Hygiene hook: ${hookPath} not found (run 'savvy init' to add)`);
+				yield* Output.skip(`Hygiene hook: ${hookPath} not found (run 'savvy init' to add)`);
 				continue;
 			}
 			const hygieneStatus = yield* ms.check(hookPath, SavvyHooksSection.section(savvyHooksHygiene()));
-			if (CheckOutcome.$is("UpToDate")(hygieneStatus)) yield* Effect.log(`${"✓"} Hygiene hook: ${hookPath}`);
+			if (CheckOutcome.$is("UpToDate")(hygieneStatus)) yield* Output.ok(`Hygiene hook: ${hookPath}`);
 			else if (CheckOutcome.$is("Drifted")(hygieneStatus)) {
 				sectionsHealthy = false;
-				yield* Effect.log(`${"⚠"} Hygiene hook: ${hookPath} outdated (run 'savvy init' to update)`);
+				yield* Output.warn(`Hygiene hook: ${hookPath} outdated (run 'savvy init' to update)`);
 			} else {
 				sectionsHealthy = false;
-				yield* Effect.log(`${BULLET} Hygiene hook: ${hookPath} section not found (run 'savvy init' to add)`);
+				yield* Output.skip(`Hygiene hook: ${hookPath} section not found (run 'savvy init' to add)`);
 			}
 			if (hookPath === ".husky/post-commit") continue;
 			const installHook = hookPath === ".husky/post-checkout" ? "post-checkout" : "post-merge";
 			const installStatus = yield* ms.check(hookPath, savvyInstallBlock(installHook));
-			if (CheckOutcome.$is("UpToDate")(installStatus)) yield* Effect.log(`${"✓"} Dependency install: ${hookPath}`);
+			if (CheckOutcome.$is("UpToDate")(installStatus)) yield* Output.ok(`Dependency install: ${hookPath}`);
 			else if (CheckOutcome.$is("Drifted")(installStatus)) {
 				sectionsHealthy = false;
-				yield* Effect.log(`${"⚠"} Dependency install: ${hookPath} outdated (run 'savvy init' to update)`);
+				yield* Output.warn(`Dependency install: ${hookPath} outdated (run 'savvy init' to update)`);
 			} else {
 				sectionsHealthy = false;
-				yield* Effect.log(`${BULLET} Dependency install: ${hookPath} section not found (run 'savvy init' to add)`);
+				yield* Output.skip(`Dependency install: ${hookPath} section not found (run 'savvy init' to add)`);
 			}
 			const toolchainStatus = yield* ms.check(hookPath, SavvyToolchainSection.section(savvyToolchainCheck()));
-			if (CheckOutcome.$is("UpToDate")(toolchainStatus)) yield* Effect.log(`${"✓"} Toolchain check: ${hookPath}`);
+			if (CheckOutcome.$is("UpToDate")(toolchainStatus)) yield* Output.ok(`Toolchain check: ${hookPath}`);
 			else if (CheckOutcome.$is("Drifted")(toolchainStatus)) {
 				sectionsHealthy = false;
-				yield* Effect.log(`${"⚠"} Toolchain check: ${hookPath} outdated (run 'savvy init' to update)`);
+				yield* Output.warn(`Toolchain check: ${hookPath} outdated (run 'savvy init' to update)`);
 			} else {
 				sectionsHealthy = false;
-				yield* Effect.log(`${BULLET} Toolchain check: ${hookPath} section not found (run 'savvy init' to add)`);
+				yield* Output.skip(`Toolchain check: ${hookPath} section not found (run 'savvy init' to add)`);
 			}
 		}
-		if (yield* fs.exists(DCO_FILE_PATH)) yield* Effect.log(`${"✓"} DCO file: ${DCO_FILE_PATH}`);
-		else yield* Effect.log(`${BULLET} No DCO file (signoff not required)`);
-		yield* Effect.log("\nDetected settings:");
-		yield* Effect.log(`  DCO required: ${Commitlint.detectDCO()}`);
+		if (yield* fs.exists(DCO_FILE_PATH)) yield* Output.ok(`DCO file: ${DCO_FILE_PATH}`);
+		else yield* Output.skip(`No DCO file (signoff not required)`);
+		yield* Output.line("");
+		yield* Output.heading("Detected settings");
+		yield* Output.detail(`DCO required: ${Commitlint.detectDCO()}`);
 		const releaseFormat = yield* detectReleaseFormat;
-		yield* Effect.log(`  Release format: ${releaseFormat}`);
+		yield* Output.detail(`Release format: ${releaseFormat}`);
 		const scopes = yield* Effect.catch(Commitlint.detectScopes, () => Effect.succeed([]));
 		const scopeDisplay = scopes.length > 0 ? scopes.join(", ") : "(none - not a monorepo or no packages found)";
-		yield* Effect.log(`  Detected scopes: ${scopeDisplay}`);
-		yield* Effect.log("");
-		if (!foundConfig || !hasHuskyHook || !sectionsHealthy) yield* Effect.log(`${CROSS_MARK} Commitlint needs configuration. Run: savvy init`);
-		else yield* Effect.log(`${"✓"} Commitlint is configured correctly.`);
+		yield* Output.detail(`Detected scopes: ${scopeDisplay}`);
+		yield* Output.line("");
+		if (!foundConfig || !hasHuskyHook || !sectionsHealthy) yield* Output.fail(`Commitlint needs configuration. Run: savvy init`);
+		else yield* Output.ok(`Commitlint is configured correctly.`);
 	});
 }
 

package/commands/commit/constants.js

@@ -1,4 +1,9 @@
 //#region src/commands/commit/constants.ts
+/**
+* Shared constants for CLI commands.
+*
+* @internal
+*/
 /** Husky commit-msg hook path (savvy-base + savvy-commit sections). */
 const HUSKY_HOOK_PATH = ".husky/commit-msg";
 /** Husky post-checkout hook path (savvy-hooks hygiene). */