@savvy-web/silk-effects 10.0.1 → 10.2.0

package/changesets/api/linter.js

@@ -1,10 +1,9 @@
-import { parseKitMarkdown } from "../utils/markdown-emit.js";
+import { parseKitMarkdownBody } from "../utils/markdown-emit.js";
 import { contentStructure } from "../rules/content-structure.js";
 import { dependencyTableFormat } from "../rules/dependency-table-format.js";
 import { headingHierarchy } from "../rules/heading-hierarchy.js";
 import { requiredSections } from "../rules/required-sections.js";
 import { uncategorizedContent } from "../rules/uncategorized-content.js";
-import { stripFrontmatter } from "../utils/strip-frontmatter.js";
 import { readFileSync, readdirSync } from "node:fs";
 import { join } from "node:path";
 
@@ -141,7 +140,7 @@ var ChangesetLinter = class ChangesetLinter {
 	* @returns Array of {@link LintMessage} diagnostics (empty if the content is valid)
 	*/
 	static validateContent(content, filePath = "<input>") {
-		const root = parseKitMarkdown(stripFrontmatter(content));
+		const root = parseKitMarkdownBody(content);
 		return RULES.flatMap((rule) => rule(root).map((finding) => ({
 			file: filePath,
 			rule: finding.rule,

package/changesets/utils/markdown-emit.js

@@ -1,5 +1,5 @@
 import { Result } from "effect";
-import { Code, Markdown, MarkdownTransform, Mdast } from "@effected/markdown";
+import { Code, Markdown, MarkdownParseOptions, MarkdownTransform, Mdast, Root } from "@effected/markdown";
 
 // src/changesets/utils/markdown-emit.ts
 /**
@@ -65,6 +65,31 @@ function parseKitMarkdown(text) {
 	const parsed = unwrap(Markdown.parseResult(text), "markdown parse: canonical parse failed");
 	return unwrap(Mdast.fromMdastResult(Mdast.toMdast(parsed)), "markdown parse: fidelity strip failed");
 }
+const FRONTMATTER_CAPTURE = MarkdownParseOptions.make({ frontmatter: true });
+/**
+* Parse a changeset file into a kit root with its frontmatter block removed.
+*
+* @remarks
+* Parses with the kit's frontmatter capture on, so a `---` block at offset 0
+* is recognized whatever its line endings (CRLF included), then drops the
+* `frontmatter` head node. The text is never sliced, so every remaining
+* node's position is relative to the whole file — the lines a linter reports
+* match the lines an editor shows. A fence pair later in the document is not
+* frontmatter and parses as ordinary content.
+*
+* @param text - The raw changeset source, frontmatter included
+* @returns The parsed kit root, without its frontmatter node
+*
+* @internal
+*/
+function parseKitMarkdownBody(text) {
+	const parsed = unwrap(Markdown.parseResult(text, FRONTMATTER_CAPTURE), "markdown parse: canonical parse failed");
+	const body = new Root({
+		children: parsed.children.filter((node) => node.type !== "frontmatter"),
+		position: parsed.position
+	});
+	return unwrap(Mdast.fromMdastResult(Mdast.toMdast(body)), "markdown parse: fidelity strip failed");
+}
 /**
 * Serialize a kit root, or a single kit block node, to canonical markdown.
 *
@@ -81,4 +106,4 @@ function emitKitMarkdown(node) {
 	return unwrap(Markdown.stringifyResult(ensureFencedCode(node)), "markdown emit: canonical stringify failed");
 }
 
-export { emitKitMarkdown, parseKitMarkdown };
+export { emitKitMarkdown, parseKitMarkdown, parseKitMarkdownBody };

package/commitlint/hook/diagnostics/open-issues.js

@@ -1,22 +1,29 @@
 import { readCache, writeCache } from "./cache.js";
-import { Effect } from "effect";
+import { Effect, Schema } from "effect";
 import { resolve } from "node:path";
 import { Git } from "@effected/git";
-import { ChildProcess, ChildProcessSpawner } from "effect/process";
+import { Run } from "@effected/commands";
+import { ChildProcess } from "effect/process";
 
 // src/commitlint/hook/diagnostics/open-issues.ts
 /**
 * Open-issue lookup via gh CLI, cached on disk.
 *
 * @remarks
-* The `gh` invocations are not git, so they stay hand-rolled — the spawn
-* mechanism moved from promisified `node:child_process.execFile` onto
-* `effect/process` `ChildProcess`. Any failure (gh missing, not
-* logged in, no repo, malformed JSON) degrades to `null`, preserving the
-* never-fails contract of the v3 implementation.
+* The `gh` invocations run through `@effected/commands`' `Run`: the repo
+* lookup is `Run.text`, the issue list is `Run.json` decoded against
+* {@link OpenIssue}. Any failure (gh missing, not logged in, no repo, a
+* non-zero exit, output that is not JSON or not an issue list) degrades to
+* `null` and caches nothing, preserving the never-fails contract.
 *
 * @internal
 */
+/** One open issue as `gh issue list --json number,title` prints it. */
+const OpenIssue = Schema.Struct({
+	number: Schema.Number,
+	title: Schema.String
+});
+const OpenIssues = Schema.Array(OpenIssue);
 const ISSUES_CACHE_TTL_SECONDS = 600;
 /** Relative path under the project root where the open-issues cache lives. */
 const ISSUES_CACHE_RELATIVE_PATH = ".claude/cache/issues.json";
@@ -43,17 +50,16 @@ function readOpenIssuesFromCache(cachePath, ttlSeconds = 600) {
 }
 function fetchAndCacheOpenIssues(cachePath) {
 	return Effect.gen(function* () {
-		const spawner = yield* ChildProcessSpawner.ChildProcessSpawner;
-		const repo = (yield* spawner.string(ChildProcess.make("gh", [
+		const repo = yield* Run.text(ChildProcess.make("gh", [
 			"repo",
 			"view",
 			"--json",
 			"nameWithOwner",
 			"-q",
 			".nameWithOwner"
-		]))).trim();
+		]));
 		if (!repo) return null;
-		const listStdout = yield* spawner.string(ChildProcess.make("gh", [
+		const parsed = [...yield* Run.json(ChildProcess.make("gh", [
 			"issue",
 			"list",
 			"--repo",
@@ -64,8 +70,7 @@ function fetchAndCacheOpenIssues(cachePath) {
 			"20",
 			"--json",
 			"number,title"
-		]));
-		const parsed = yield* Effect.try(() => JSON.parse(listStdout));
+		]), OpenIssues)];
 		yield* writeCache(cachePath, parsed);
 		return parsed;
 	}).pipe(Effect.orElseSucceed(() => null));
@@ -78,4 +83,4 @@ function readOrFetchOpenIssues(cachePath) {
 	});
 }
 
-export { ISSUES_CACHE_RELATIVE_PATH, fetchAndCacheOpenIssues, readOpenIssuesFromCache, readOrFetchOpenIssues, resolveIssuesCachePath };
+export { ISSUES_CACHE_RELATIVE_PATH, OpenIssue, fetchAndCacheOpenIssues, readOpenIssuesFromCache, readOrFetchOpenIssues, resolveIssuesCachePath };

package/commitlint/hook/diagnostics/package-manager.js

@@ -1,62 +1,35 @@
-import { existsSync } from "node:fs";
-import { join } from "node:path";
-import { readFile } from "node:fs/promises";
+import { Result } from "effect";
+import { detectPackageManagerSync } from "@effected/workspaces";
+import { nodeSyncOps } from "@effected/workspaces/node-sync";
 
 // src/commitlint/hook/diagnostics/package-manager.ts
 /**
 * Package manager detection for the commit-time hooks.
 *
-* Mirrors the husky hook detection: prefer `package.json#devEngines.packageManager`
-* (the first entry when it is an array), then the legacy `packageManager` field,
-* then lockfile presence in priority order pnpm \> yarn \> bun \> npm.
+* Delegates to `@effected/workspaces`' `detectPackageManagerSync` — the same
+* detector `Lint`'s `Command.detectPackageManager` runs — so silk-effects has
+* one answer to "which manager owns this root". The kit refuses to guess when
+* no evidence matches (and fails on an unreadable root `package.json`); the
+* hook hints still need a command to print, so those cases fall back to `npm`.
 *
 * @internal
 */
-const VALID_PMS = /* @__PURE__ */ new Set([
-	"pnpm",
-	"yarn",
-	"bun",
-	"npm"
-]);
-const toPackageManager = (name) => typeof name === "string" && VALID_PMS.has(name) ? name : null;
-const devEnginesName = (manifest) => {
-	const devEngines = manifest.devEngines;
-	if (typeof devEngines !== "object" || devEngines === null) return void 0;
-	const declared = devEngines.packageManager;
-	const first = Array.isArray(declared) ? declared[0] : declared;
-	return typeof first === "object" && first !== null ? first.name : void 0;
-};
-function parsePackageManagerField(packageJsonContent) {
-	let parsed;
-	try {
-		parsed = JSON.parse(packageJsonContent);
-	} catch {
-		return null;
-	}
-	if (typeof parsed !== "object" || parsed === null) return null;
-	const manifest = parsed;
-	const fromDevEngines = toPackageManager(devEnginesName(manifest));
-	if (fromDevEngines !== null) return fromDevEngines;
-	const field = manifest.packageManager;
-	if (typeof field !== "string" || field.length === 0) return null;
-	return toPackageManager(field.split("@")[0]);
-}
-function detectFromLockfiles(presence) {
-	if (presence.pnpm) return "pnpm";
-	if (presence.yarn) return "yarn";
-	if (presence.bun) return "bun";
-	return "npm";
-}
+/**
+* Detect the package manager owning `root`, falling back to `npm`.
+*
+* @remarks
+* Precedence is the kit's (`PackageManagerDetector`): `pnpm-workspace.yaml`,
+* then a bun or yarn lockfile only when a manifest field also names that
+* manager, then `package.json#workspaces` (npm), `pnpm-lock.yaml`,
+* `package-lock.json`, and finally `devEngines.packageManager` over the
+* top-level `packageManager` field. `root` is probed as given, without ascent.
+*
+* @param root - The directory to probe
+* @returns The detected manager, or `"npm"` when none can be attributed
+*/
 async function detectPackageManager(root) {
-	try {
-		const fromField = parsePackageManagerField(await readFile(join(root, "package.json"), "utf8"));
-		if (fromField !== null) return fromField;
-	} catch {}
-	return detectFromLockfiles({
-		pnpm: existsSync(join(root, "pnpm-lock.yaml")),
-		yarn: existsSync(join(root, "yarn.lock")),
-		bun: existsSync(join(root, "bun.lock"))
-	});
+	const detected = detectPackageManagerSync(root, nodeSyncOps);
+	return Result.isSuccess(detected) ? detected.success.name : "npm";
 }
 
-export { detectFromLockfiles, detectPackageManager, parsePackageManagerField };
+export { detectPackageManager };

package/commitlint/index.js

@@ -18,7 +18,7 @@ import { inferTicketId, readBranchInfo } from "./hook/diagnostics/branch.js";
 import { readCache, writeCache } from "./hook/diagnostics/cache.js";
 import { parseHuskyConfigPath, readCommitlintConfigPath } from "./hook/diagnostics/commitlint-config.js";
 import { ISSUES_CACHE_RELATIVE_PATH, fetchAndCacheOpenIssues, readOpenIssuesFromCache, readOrFetchOpenIssues, resolveIssuesCachePath } from "./hook/diagnostics/open-issues.js";
-import { detectFromLockfiles, detectPackageManager, parsePackageManagerField } from "./hook/diagnostics/package-manager.js";
+import { detectPackageManager } from "./hook/diagnostics/package-manager.js";
 import { buildSigningDiagnostic, parseGpgKeyExpiry, readSigningDiagnostic } from "./hook/diagnostics/signing.js";
 import { closesTrailerRule, hasClosingTrailer } from "./hook/rules/closes-trailer.js";
 import { forbiddenContentRule } from "./hook/rules/forbidden-content.js";
@@ -69,7 +69,6 @@ var commitlint_exports = /* @__PURE__ */ __exportAll({
 	default: () => CommitlintConfig,
 	defaultPromptConfig: () => defaultPromptConfig,
 	detectDCO: () => detectDCO,
-	detectFromLockfiles: () => detectFromLockfiles,
 	detectPackageManager: () => detectPackageManager,
 	detectScopes: () => detectScopes,
 	emojiPromptConfig: () => emojiPromptConfig,
@@ -84,7 +83,6 @@ var commitlint_exports = /* @__PURE__ */ __exportAll({
 	parseBashCommand: () => parseBashCommand,
 	parseGpgKeyExpiry: () => parseGpgKeyExpiry,
 	parseHuskyConfigPath: () => parseHuskyConfigPath,
-	parsePackageManagerField: () => parsePackageManagerField,
 	partitionHits: () => partitionHits,
 	planLeakageRule: () => planLeakageRule,
 	postToolUseAdvise: () => postToolUseAdvise,
@@ -141,4 +139,4 @@ var CommitlintConfig = class {
 	constructor() {}
 };
 
-export { COMMIT_TYPES, COMMIT_TYPE_DEFINITIONS, CommitlintConfig, CommitlintConfig as default, DCO_SIGNOFF_TEXT, ERROR_EXPLANATIONS, ERROR_SUGGESTIONS, HookSilencer, ISSUES_CACHE_RELATIVE_PATH, PostToolUseEnvelope, PreToolUseEnvelope, SessionStartEnvelope, TDD_SCOPE_PATTERN, TDD_STATES, TYPE_EMOJIS, TYPE_EMOJIS_UNICODE, buildSigningDiagnostic, closesTrailerRule, commitlint_exports, createPromptConfig, createTypeEnum, defaultPromptConfig, detectDCO, detectFromLockfiles, detectPackageManager, detectScopes, emojiPromptConfig, fetchAndCacheOpenIssues, forbiddenContentRule, format, getExplanation, getSuggestion, getTypeEmoji, hasClosingTrailer, inferTicketId, parseBashCommand, parseGpgKeyExpiry, parseHuskyConfigPath, parsePackageManagerField, partitionHits, planLeakageRule, postToolUseAdvise, preToolUseAdvise, preToolUseAllow, preToolUseDeny, preToolUseSilent, prompter, readBranchInfo, readCache, readCommitlintConfigPath, readOpenIssuesFromCache, readOrFetchOpenIssues, readSigningDiagnostic, resolveIssuesCachePath, sessionStartContext, signingFlagConflictRule, softWrapRule, staticConfig, verbosityRule, writeCache };
+export { COMMIT_TYPES, COMMIT_TYPE_DEFINITIONS, CommitlintConfig, CommitlintConfig as default, DCO_SIGNOFF_TEXT, ERROR_EXPLANATIONS, ERROR_SUGGESTIONS, HookSilencer, ISSUES_CACHE_RELATIVE_PATH, PostToolUseEnvelope, PreToolUseEnvelope, SessionStartEnvelope, TDD_SCOPE_PATTERN, TDD_STATES, TYPE_EMOJIS, TYPE_EMOJIS_UNICODE, buildSigningDiagnostic, closesTrailerRule, commitlint_exports, createPromptConfig, createTypeEnum, defaultPromptConfig, detectDCO, detectPackageManager, detectScopes, emojiPromptConfig, fetchAndCacheOpenIssues, forbiddenContentRule, format, getExplanation, getSuggestion, getTypeEmoji, hasClosingTrailer, inferTicketId, parseBashCommand, parseGpgKeyExpiry, parseHuskyConfigPath, partitionHits, planLeakageRule, postToolUseAdvise, preToolUseAdvise, preToolUseAllow, preToolUseDeny, preToolUseSilent, prompter, readBranchInfo, readCache, readCommitlintConfigPath, readOpenIssuesFromCache, readOrFetchOpenIssues, readSigningDiagnostic, resolveIssuesCachePath, sessionStartContext, signingFlagConflictRule, softWrapRule, staticConfig, verbosityRule, writeCache };

package/index.d.ts

@@ -975,8 +975,8 @@ interface LintMessage {
    * Line number where the issue was detected (1-based).
    *
    * @remarks
-   * Line numbers are relative to the content after YAML frontmatter
-   * stripping (the blank line after the closing `---` counts as line 1).
+   * Line numbers are relative to the whole file, frontmatter block
+   * included, so they match the lines an editor shows.
    */
   line: number;
   /**
@@ -5456,10 +5456,13 @@ declare function readCache<T>(path: string, ttlSeconds: number): Effect.Effect<T
 declare function writeCache<T>(path: string, data: T, when?: Date): Effect.Effect<void>;
 declare function parseHuskyConfigPath(huskyContent: string, root: string): string | null;
 declare function readCommitlintConfigPath(root: string): Promise<string | null>;
-interface OpenIssue {
-  number: number;
-  title: string;
-}
+/** One open issue as `gh issue list --json number,title` prints it. */
+declare const OpenIssue: Schema.Struct<{
+  readonly number: Schema.Number;
+  readonly title: Schema.String;
+}>;
+/** The decoded {@link (OpenIssue:variable)}. */
+type OpenIssue = typeof OpenIssue.Type;
 declare const ISSUES_CACHE_TTL_SECONDS = 600;
 /** Relative path under the project root where the open-issues cache lives. */
 declare const ISSUES_CACHE_RELATIVE_PATH = ".claude/cache/issues.json";
@@ -5480,13 +5483,19 @@ declare function readOpenIssuesFromCache(cachePath: string, ttlSeconds?: number)
 declare function fetchAndCacheOpenIssues(cachePath: string): Effect.Effect<OpenIssue[] | null, never, ChildProcessSpawner.ChildProcessSpawner>;
 declare function readOrFetchOpenIssues(cachePath: string): Effect.Effect<OpenIssue[] | null, never, ChildProcessSpawner.ChildProcessSpawner>;
 type PackageManager$1 = "pnpm" | "yarn" | "bun" | "npm";
-interface LockfilePresence {
-  pnpm: boolean;
-  yarn: boolean;
-  bun: boolean;
-}
-declare function parsePackageManagerField(packageJsonContent: string): PackageManager$1 | null;
-declare function detectFromLockfiles(presence: LockfilePresence): PackageManager$1;
+/**
+ * Detect the package manager owning `root`, falling back to `npm`.
+ *
+ * @remarks
+ * Precedence is the kit's (`PackageManagerDetector`): `pnpm-workspace.yaml`,
+ * then a bun or yarn lockfile only when a manifest field also names that
+ * manager, then `package.json#workspaces` (npm), `pnpm-lock.yaml`,
+ * `package-lock.json`, and finally `devEngines.packageManager` over the
+ * top-level `packageManager` field. `root` is probed as given, without ascent.
+ *
+ * @param root - The directory to probe
+ * @returns The detected manager, or `"npm"` when none can be attributed
+ */
 declare function detectPackageManager(root: string): Promise<PackageManager$1>;
 interface SigningDiagnostic {
   format: "gpg" | "ssh" | "none";
@@ -5591,7 +5600,7 @@ declare const VERBOSITY_LINE_THRESHOLD = 12;
 declare const VERBOSITY_WORD_THRESHOLD = 150;
 declare const verbosityRule: Rule<VerbosityInput, never>;
 declare namespace index_d_exports$1 {
-  export { BranchInfo, COMMIT_TYPES, COMMIT_TYPE_DEFINITIONS, ClosesTrailerCtx, ClosesTrailerInput, CommitType, CommitTypeDefinition, CommitlintConfig, CommitlintPlugin, CommitlintUserConfig, ConfigOptions, DCO_SIGNOFF_TEXT, DEFAULT_BODY_MAX_LINE_LENGTH, ERROR_EXPLANATIONS, ERROR_SUGGESTIONS, ForbiddenContentInput, FormatterResult, HookOutput, RuleSeverity$1 as HookRuleSeverity, HookSilencer, ISSUES_CACHE_RELATIVE_PATH, ISSUES_CACHE_TTL_SECONDS, Inquirer, LintOutcome, LockfilePresence, OpenIssue, PackageManager$1 as PackageManager, ParsedCommit, ParsedKind, PlanLeakageInput, PostToolUseEnvelope, PostToolUseEnvelope as PostToolUseEnvelopeType, PreToolUseEnvelope, PreToolUseEnvelope as PreToolUseEnvelopeType, PromptConfig, PromptConfigOptions, PromptQuestion, PromptSettings, PrompterOptions, Question, RawSigningInputs, ReleaseFormat, ResolvedPromptConfig, Rule, RuleApplicability, RuleConfigTuple, RuleHit, RuleResult, RuleSeverity, RulesConfig, SessionStartEnvelope, SessionStartEnvelope as SessionStartEnvelopeType, SigningDiagnostic, SigningFlagCtx, SigningFlagInput, SoftWrapInput, TDD_SCOPE_PATTERN, TDD_STATES, TYPE_EMOJIS, TYPE_EMOJIS_UNICODE, TypeEnumEntry, VERBOSITY_LINE_THRESHOLD, VERBOSITY_WORD_THRESHOLD, VerbosityInput, buildSigningDiagnostic, closesTrailerRule, createPromptConfig, createTypeEnum, CommitlintConfig as default, defaultPromptConfig, detectDCO, detectFromLockfiles, detectPackageManager, detectScopes, emojiPromptConfig, fetchAndCacheOpenIssues, forbiddenContentRule, format, getExplanation, getSuggestion, getTypeEmoji, hasClosingTrailer, inferTicketId, parseBashCommand, parseGpgKeyExpiry, parseHuskyConfigPath, parsePackageManagerField, partitionHits, planLeakageRule, postToolUseAdvise, preToolUseAdvise, preToolUseAllow, preToolUseDeny, preToolUseSilent, prompter, readBranchInfo, readCache, readCommitlintConfigPath, readOpenIssuesFromCache, readOrFetchOpenIssues, readSigningDiagnostic, resolveIssuesCachePath, sessionStartContext, signingFlagConflictRule, softWrapRule, staticConfig, verbosityRule, writeCache };
+  export { BranchInfo, COMMIT_TYPES, COMMIT_TYPE_DEFINITIONS, ClosesTrailerCtx, ClosesTrailerInput, CommitType, CommitTypeDefinition, CommitlintConfig, CommitlintPlugin, CommitlintUserConfig, ConfigOptions, DCO_SIGNOFF_TEXT, DEFAULT_BODY_MAX_LINE_LENGTH, ERROR_EXPLANATIONS, ERROR_SUGGESTIONS, ForbiddenContentInput, FormatterResult, HookOutput, RuleSeverity$1 as HookRuleSeverity, HookSilencer, ISSUES_CACHE_RELATIVE_PATH, ISSUES_CACHE_TTL_SECONDS, Inquirer, LintOutcome, OpenIssue, PackageManager$1 as PackageManager, ParsedCommit, ParsedKind, PlanLeakageInput, PostToolUseEnvelope, PostToolUseEnvelope as PostToolUseEnvelopeType, PreToolUseEnvelope, PreToolUseEnvelope as PreToolUseEnvelopeType, PromptConfig, PromptConfigOptions, PromptQuestion, PromptSettings, PrompterOptions, Question, RawSigningInputs, ReleaseFormat, ResolvedPromptConfig, Rule, RuleApplicability, RuleConfigTuple, RuleHit, RuleResult, RuleSeverity, RulesConfig, SessionStartEnvelope, SessionStartEnvelope as SessionStartEnvelopeType, SigningDiagnostic, SigningFlagCtx, SigningFlagInput, SoftWrapInput, TDD_SCOPE_PATTERN, TDD_STATES, TYPE_EMOJIS, TYPE_EMOJIS_UNICODE, TypeEnumEntry, VERBOSITY_LINE_THRESHOLD, VERBOSITY_WORD_THRESHOLD, VerbosityInput, buildSigningDiagnostic, closesTrailerRule, createPromptConfig, createTypeEnum, CommitlintConfig as default, defaultPromptConfig, detectDCO, detectPackageManager, detectScopes, emojiPromptConfig, fetchAndCacheOpenIssues, forbiddenContentRule, format, getExplanation, getSuggestion, getTypeEmoji, hasClosingTrailer, inferTicketId, parseBashCommand, parseGpgKeyExpiry, parseHuskyConfigPath, partitionHits, planLeakageRule, postToolUseAdvise, preToolUseAdvise, preToolUseAllow, preToolUseDeny, preToolUseSilent, prompter, readBranchInfo, readCache, readCommitlintConfigPath, readOpenIssuesFromCache, readOrFetchOpenIssues, readSigningDiagnostic, resolveIssuesCachePath, sessionStartContext, signingFlagConflictRule, softWrapRule, staticConfig, verbosityRule, writeCache };
 }
 /**
  * Dynamic commitlint configuration factory.
@@ -6939,6 +6948,9 @@ declare class Command {
   private static cachedPackageManager;
   /** Cached project root path */
   private static cachedRoot;
+  /** Memoizing tool discovery, created lazily and dropped by `clearCache` */
+  private static discovery;
+  private static getDiscovery;
   /**
    * Find the project root directory by walking up from `cwd`.
    *
@@ -6963,9 +6975,8 @@ declare class Command {
   /**
    * Detect the package manager declared by the root package.json.
    *
-   * Reads `devEngines.packageManager.name` (the first entry when it is an array), then the
-   * legacy `packageManager` field (e.g., `pnpm\@9.0.0`), then lockfile presence in priority
-   * order pnpm, yarn, bun. Falls back to "npm" when none of them names a manager.
+   * Delegates to the kit's `detectPackageManagerSync` (manifest fields, then workspace and
+   * lockfile markers) at `cwd` without walking up. Falls back to "npm" when none names a manager.
    *
    * @param cwd - Directory to search for package.json (defaults to `Command.findRoot()`)
    * @returns The detected package manager
@@ -6986,7 +6997,7 @@ declare class Command {
    * @example
    * ```typescript
    * Command.getExecPrefix('pnpm'); // ['pnpm', 'exec']
-   * Command.getExecPrefix('npm');  // ['npx', '--no']
+   * Command.getExecPrefix('npm');  // ['npx', '--no', '--']
    * Command.getExecPrefix('yarn'); // ['yarn', 'exec']
    * Command.getExecPrefix('bun');  // ['bun', 'x', "--no-install"]
    * ```
@@ -7061,6 +7072,8 @@ declare class Command {
   /**
    * Execute a command and return its output.
    *
+   * @deprecated No in-repo consumers; will be removed in a future major.
+   *
    * @param command - The command to execute
    * @returns The command output as a string (trimmed)
    * @throws If the command fails
@@ -7075,6 +7088,8 @@ declare class Command {
   /**
    * Execute a command silently (ignore output).
    *
+   * @deprecated No in-repo consumers; will be removed in a future major.
+   *
    * @param command - The command to execute
    * @returns `true` if successful, `false` if failed
    */
@@ -8015,6 +8030,15 @@ declare class ReposLockdown extends ReposLockdown_base {
    */
   static readonly layer: Layer.Layer<ReposLockdown, never, FileSystem.FileSystem | Path.Path>;
 }
+/**
+ * Default repo name for `add`: the last segment of the remote's repository
+ * path, as `@effected/git`'s `GitRemote.repositoryPath` reads it (one trailing
+ * `/` and one trailing `.git` dropped). Every remote form git accepts is
+ * covered, scp without a slash included — `git@host:repo.git` is `repo`.
+ *
+ * @internal
+ */
+declare const repoSlug: (url: string) => string;
 /**
  * Minimum age (in milliseconds) a `.lock` file must reach before `sync` will
  * remove it. An ACTIVE git process can legitimately hold
@@ -8145,7 +8169,7 @@ declare class ReposManager extends ReposManager_base {
   static readonly layer: Layer.Layer<ReposManager, never, ReposConfigStore | Git | FileSystem.FileSystem | Path.Path | ReposLockdown>;
 }
 declare namespace index_d_exports$3 {
-  export { DriftKind, GitSubmoduleError, GitSubmoduleErrorBase, MANIFEST_PATH, NOTE_LIMIT, NoteNotFoundError, NoteNotFoundErrorBase, REPOS_DIR, RepoDrift, RepoEntry, RepoName, RepoNotFoundError, RepoNotFoundErrorBase, RepoNote, RepoOrientation, RepoStatusEntry, ReposAddResult, ReposConfigError, ReposConfigErrorBase, ReposConfigStore, ReposConfigStoreShape, ReposDeregisterResult, ReposDrift, ReposDriftReport, ReposDriftShape, ReposLockdown, ReposLockdownError, ReposLockdownErrorBase, ReposLockdownShape, ReposManager, ReposManagerShape, ReposManifestFile, ReposNoteResult, ReposPinResult, ReposRemoveResult, ReposRenameResult, ReposRestoreResult, ReposStatusReport, ReposSyncReport, STALE_LOCK_MAX_AGE_MS, resolveModuleDir, submoduleNameFromKey };
+  export { DriftKind, GitSubmoduleError, GitSubmoduleErrorBase, MANIFEST_PATH, NOTE_LIMIT, NoteNotFoundError, NoteNotFoundErrorBase, REPOS_DIR, RepoDrift, RepoEntry, RepoName, RepoNotFoundError, RepoNotFoundErrorBase, RepoNote, RepoOrientation, RepoStatusEntry, ReposAddResult, ReposConfigError, ReposConfigErrorBase, ReposConfigStore, ReposConfigStoreShape, ReposDeregisterResult, ReposDrift, ReposDriftReport, ReposDriftShape, ReposLockdown, ReposLockdownError, ReposLockdownErrorBase, ReposLockdownShape, ReposManager, ReposManagerShape, ReposManifestFile, ReposNoteResult, ReposPinResult, ReposRemoveResult, ReposRenameResult, ReposRestoreResult, ReposStatusReport, ReposSyncReport, STALE_LOCK_MAX_AGE_MS, repoSlug, resolveModuleDir, submoduleNameFromKey };
 }
 /**
  * Strip leading semver range operators from a version string.

package/lint/handlers/TextFiles.js

@@ -1,7 +1,8 @@
 import { Command } from "../utils/Command.js";
 import { Filter } from "../utils/Filter.js";
-import { Data, Effect, FileSystem, Path, Schema, Stream } from "effect";
+import { Data, Effect, FileSystem, Option, Path, Schema, Stream } from "effect";
 import { Git } from "@effected/git";
+import { Walker } from "@effected/walker";
 import { ChildProcess, ChildProcessSpawner } from "effect/process";
 
 // src/lint/handlers/TextFiles.ts
@@ -279,12 +280,8 @@ var TextFiles = class TextFiles {
 			const spawner = yield* ChildProcessSpawner.ChildProcessSpawner;
 			const fs = yield* FileSystem.FileSystem;
 			const pathService = yield* Path.Path;
-			let cwd = pathService.dirname(path);
-			while (!(yield* fs.exists(cwd))) {
-				const parent = pathService.dirname(cwd);
-				if (parent === cwd) break;
-				cwd = parent;
-			}
+			const chain = yield* Walker.ascend(pathService.dirname(path));
+			const cwd = Option.getOrElse(yield* Walker.firstMatch(chain, (dir) => fs.exists(dir)), () => chain[chain.length - 1] ?? pathService.dirname(path));
 			const relative = pathService.relative(cwd, path).split(pathService.sep).join("/");
 			const command = ChildProcess.make("git", [
 				"cat-file",

package/lint/utils/Command.js

@@ -1,7 +1,11 @@
-import { detectFromLockfiles, parsePackageManagerField } from "../../commitlint/hook/diagnostics/package-manager.js";
-import { existsSync, readFileSync } from "node:fs";
+import { Result } from "effect";
+import { existsSync } from "node:fs";
 import { dirname, join, resolve } from "node:path";
+import { detectPackageManagerSync } from "@effected/workspaces";
+import { LocalExec, Tool, makeToolDiscoverySync } from "@effected/commands";
+import { nodeSyncOps } from "@effected/workspaces/node-sync";
 import { execSync } from "node:child_process";
+import { nodeSpawnSync } from "@effected/commands/node-sync";
 
 // src/lint/utils/Command.ts
 /**
@@ -46,6 +50,18 @@ var Command = class Command {
 	static cachedPackageManager = null;
 	/** Cached project root path */
 	static cachedRoot = null;
+	/** Memoizing tool discovery, created lazily and dropped by `clearCache` */
+	static discovery = null;
+	static getDiscovery() {
+		if (Command.discovery === null) {
+			const context = LocalExec.contextFor(Command.detectPackageManager(), { directory: Command.findRoot() });
+			Command.discovery = makeToolDiscoverySync({
+				spawn: nodeSpawnSync,
+				context
+			});
+		}
+		return Command.discovery;
+	}
 	/**
 	* Find the project root directory by walking up from `cwd`.
 	*
@@ -84,9 +100,8 @@ var Command = class Command {
 	/**
 	* Detect the package manager declared by the root package.json.
 	*
-	* Reads `devEngines.packageManager.name` (the first entry when it is an array), then the
-	* legacy `packageManager` field (e.g., `pnpm\@9.0.0`), then lockfile presence in priority
-	* order pnpm, yarn, bun. Falls back to "npm" when none of them names a manager.
+	* Delegates to the kit's `detectPackageManagerSync` (manifest fields, then workspace and
+	* lockfile markers) at `cwd` without walking up. Falls back to "npm" when none names a manager.
 	*
 	* @param cwd - Directory to search for package.json (defaults to `Command.findRoot()`)
 	* @returns The detected package manager
@@ -99,16 +114,8 @@ var Command = class Command {
 	*/
 	static detectPackageManager(cwd = Command.findRoot()) {
 		if (Command.cachedPackageManager !== null) return Command.cachedPackageManager;
-		const packageJsonPath = join(cwd, "package.json");
-		let declared = null;
-		try {
-			declared = parsePackageManagerField(readFileSync(packageJsonPath, "utf-8"));
-		} catch {}
-		Command.cachedPackageManager = declared ?? detectFromLockfiles({
-			pnpm: existsSync(join(cwd, "pnpm-lock.yaml")),
-			yarn: existsSync(join(cwd, "yarn.lock")),
-			bun: existsSync(join(cwd, "bun.lock"))
-		});
+		const detected = detectPackageManagerSync(cwd, nodeSyncOps);
+		Command.cachedPackageManager = Result.isSuccess(detected) ? detected.success.name : "npm";
 		return Command.cachedPackageManager;
 	}
 	/**
@@ -120,22 +127,13 @@ var Command = class Command {
 	* @example
 	* ```typescript
 	* Command.getExecPrefix('pnpm'); // ['pnpm', 'exec']
-	* Command.getExecPrefix('npm');  // ['npx', '--no']
+	* Command.getExecPrefix('npm');  // ['npx', '--no', '--']
 	* Command.getExecPrefix('yarn'); // ['yarn', 'exec']
 	* Command.getExecPrefix('bun');  // ['bun', 'x', "--no-install"]
 	* ```
 	*/
 	static getExecPrefix(packageManager) {
-		switch (packageManager) {
-			case "pnpm": return ["pnpm", "exec"];
-			case "yarn": return ["yarn", "exec"];
-			case "bun": return [
-				"bun",
-				"x",
-				"--no-install"
-			];
-			default: return ["npx", "--no"];
-		}
+		return [...LocalExec.contextFor(packageManager).prefix];
 	}
 	/**
 	* Clear the cached package manager and project root detection.
@@ -144,6 +142,8 @@ var Command = class Command {
 	static clearCache() {
 		Command.cachedPackageManager = null;
 		Command.cachedRoot = null;
+		Command.discovery?.invalidateAll();
+		Command.discovery = null;
 	}
 	/**
 	* Check if a command is available in the system PATH.
@@ -160,12 +160,7 @@ var Command = class Command {
 	*/
 	static isAvailable(command) {
 		validateCommandName(command);
-		try {
-			execSync(`command -v ${command}`, { stdio: "ignore" });
-			return true;
-		} catch {
-			return false;
-		}
+		return Command.getDiscovery().isAvailable(Tool.named(command, { source: "global" }));
 	}
 	/**
 	* Find a tool, checking global installation first, then the project's package manager.
@@ -188,26 +183,18 @@ var Command = class Command {
 	*/
 	static findTool(tool) {
 		validateCommandName(tool);
-		if (Command.isAvailable(tool)) return {
-			available: true,
-			command: tool,
-			source: "global"
-		};
-		const pm = Command.detectPackageManager();
-		const execCmd = [...Command.getExecPrefix(pm), tool].join(" ");
-		try {
-			execSync(`${execCmd} --version`, { stdio: "ignore" });
-			return {
-				available: true,
-				command: execCmd,
-				source: pm
-			};
-		} catch {}
-		return {
+		const resolved = Command.getDiscovery().resolve(Tool.named(tool, { prefer: "global" }));
+		if (Result.isFailure(resolved)) return {
 			available: false,
 			command: void 0,
 			source: void 0
 		};
+		const found = resolved.success;
+		return {
+			available: true,
+			command: found.argv().join(" "),
+			source: found.source === "global" ? "global" : Command.detectPackageManager()
+		};
 	}
 	/**
 	* Find a tool or throw an error if not available.
@@ -247,6 +234,8 @@ var Command = class Command {
 	/**
 	* Execute a command and return its output.
 	*
+	* @deprecated No in-repo consumers; will be removed in a future major.
+	*
 	* @param command - The command to execute
 	* @returns The command output as a string (trimmed)
 	* @throws If the command fails
@@ -263,6 +252,8 @@ var Command = class Command {
 	/**
 	* Execute a command silently (ignore output).
 	*
+	* @deprecated No in-repo consumers; will be removed in a future major.
+	*
 	* @param command - The command to execute
 	* @returns `true` if successful, `false` if failed
 	*/

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@savvy-web/silk-effects",
-	"version": "10.0.1",
+	"version": "10.2.0",
 	"private": false,
 	"description": "Shared Effect library for Silk Suite conventions",
 	"homepage": "https://github.com/savvy-web/systems/tree/main/packages/silk-effects",
@@ -38,22 +38,22 @@
 		"@changesets/pre": "^3.0.0",
 		"@changesets/read": "^1.0.1",
 		"@effected/github-references": "^0.11.0",
-		"@effected/glob": "^0.14.0",
-		"@effected/jsonc": "^0.20.0",
+		"@effected/glob": "^0.15.0",
+		"@effected/jsonc": "^0.20.1",
 		"@effected/markdown": "^0.23.0",
-		"@effected/package-json": "^0.25.2",
+		"@effected/package-json": "^0.26.1",
 		"@effected/toml": "^0.16.0",
-		"@effected/walker": "^0.20.0",
+		"@effected/walker": "^0.21.0",
 		"@effected/yaml": "^0.25.1",
 		"@manypkg/get-packages": "^3.1.0",
-		"@savvy-web/silk-core": "^0.5.1",
+		"@savvy-web/silk-core": "^0.5.2",
 		"shell-quote": "^1.12.0"
 	},
 	"peerDependencies": {
-		"@effected/commands": "^0.15.0",
-		"@effected/git": "^0.25.0",
+		"@effected/commands": "^0.17.0",
+		"@effected/git": "^0.26.0",
 		"@effected/templates": "^0.14.0",
-		"@effected/workspaces": "^0.38.0",
+		"@effected/workspaces": "^0.39.0",
 		"effect": "^4.0.0"
 	}
 }

package/repos/index.js

@@ -7,7 +7,7 @@ import { RepoStatusEntry, ReposAddResult, ReposDeregisterResult, ReposNoteResult
 import { ReposConfigStore } from "./services/config-store.js";
 import { ReposLockdown, resolveModuleDir } from "./services/lockdown.js";
 import { ReposDrift, submoduleNameFromKey } from "./services/drift.js";
-import { ReposManager, STALE_LOCK_MAX_AGE_MS } from "./services/manager.js";
+import { ReposManager, STALE_LOCK_MAX_AGE_MS, repoSlug } from "./services/manager.js";
 
 // src/repos/index.ts
 var repos_exports = /* @__PURE__ */ __exportAll({
@@ -47,8 +47,9 @@ var repos_exports = /* @__PURE__ */ __exportAll({
 	ReposStatusReport: () => ReposStatusReport,
 	ReposSyncReport: () => ReposSyncReport,
 	STALE_LOCK_MAX_AGE_MS: () => STALE_LOCK_MAX_AGE_MS,
+	repoSlug: () => repoSlug,
 	resolveModuleDir: () => resolveModuleDir,
 	submoduleNameFromKey: () => submoduleNameFromKey
 });
 
-export { DriftKind, GitSubmoduleError, GitSubmoduleErrorBase, MANIFEST_PATH, NoteNotFoundError, NoteNotFoundErrorBase, REPOS_DIR, RepoDrift, RepoEntry, RepoName, RepoNotFoundError, RepoNotFoundErrorBase, RepoNote, RepoOrientation, RepoStatusEntry, ReposAddResult, ReposConfigError, ReposConfigErrorBase, ReposConfigStore, ReposDeregisterResult, ReposDrift, ReposDriftReport, ReposLockdown, ReposLockdownError, ReposLockdownErrorBase, ReposManager, ReposManifestFile, ReposNoteResult, ReposPinResult, ReposRemoveResult, ReposRenameResult, ReposRestoreResult, ReposStatusReport, ReposSyncReport, STALE_LOCK_MAX_AGE_MS, repos_exports, resolveModuleDir, submoduleNameFromKey };
+export { DriftKind, GitSubmoduleError, GitSubmoduleErrorBase, MANIFEST_PATH, NoteNotFoundError, NoteNotFoundErrorBase, REPOS_DIR, RepoDrift, RepoEntry, RepoName, RepoNotFoundError, RepoNotFoundErrorBase, RepoNote, RepoOrientation, RepoStatusEntry, ReposAddResult, ReposConfigError, ReposConfigErrorBase, ReposConfigStore, ReposDeregisterResult, ReposDrift, ReposDriftReport, ReposLockdown, ReposLockdownError, ReposLockdownErrorBase, ReposManager, ReposManifestFile, ReposNoteResult, ReposPinResult, ReposRemoveResult, ReposRenameResult, ReposRestoreResult, ReposStatusReport, ReposSyncReport, STALE_LOCK_MAX_AGE_MS, repoSlug, repos_exports, resolveModuleDir, submoduleNameFromKey };

package/repos/services/manager.js

@@ -5,7 +5,7 @@ import { ReposConfigStore } from "./config-store.js";
 import { ReposLockdown, resolveModuleDir } from "./lockdown.js";
 import { submoduleNameFromKey } from "./drift.js";
 import { Clock, Context, Effect, Exit, FileSystem, Layer, Option, Path, Result, Schema } from "effect";
-import { Git, GitConfig, Gitmodules, LsRemoteEntry } from "@effected/git";
+import { Git, GitConfig, GitRemote, Gitmodules, LsRemoteEntry } from "@effected/git";
 import { createHash } from "node:crypto";
 
 // src/repos/services/manager.ts
@@ -15,6 +15,22 @@ import { createHash } from "node:crypto";
 */
 const STALE_LOCKS = ["index.lock", "shallow.lock"];
 /**
+* Default repo name for `add`: the last segment of the remote's repository
+* path, as `@effected/git`'s `GitRemote.repositoryPath` reads it (one trailing
+* `/` and one trailing `.git` dropped). Every remote form git accepts is
+* covered, scp without a slash included — `git@host:repo.git` is `repo`.
+*
+* @internal
+*/
+const repoSlug = (url) => {
+	const trimmed = url.trim();
+	const path = Option.match(GitRemote.parse(trimmed), {
+		onNone: () => trimmed,
+		onSome: GitRemote.repositoryPath
+	});
+	return path.split("/").filter((segment) => segment.length > 0).pop() ?? path;
+};
+/**
 * Minimum age (in milliseconds) a `.lock` file must reach before `sync` will
 * remove it. An ACTIVE git process can legitimately hold
 * `index.lock`/`shallow.lock` for the duration of its own run; removing a
@@ -191,11 +207,6 @@ var ReposManager = class extends Context.Service()("@savvy-web/silk-effects/Repo
 				boundaryMarked
 			};
 		});
-		/** Last path segment of a repo URL, with a trailing `.git` stripped. */
-		const repoSlug = (url) => {
-			const last = url.split("/").filter((segment) => segment.length > 0).pop() ?? url;
-			return last.endsWith(".git") ? last.slice(0, -4) : last;
-		};
 		/**
 		* Fetch `ref` shallow into a submodule via the kit's `Git.fetchAny`:
 		* tag-form fetch first (tags need the explicit `fetch origin tag <ref>`
@@ -728,4 +739,4 @@ var ReposManager = class extends Context.Service()("@savvy-web/silk-effects/Repo
 	}));
 };
 
-export { ReposManager, STALE_LOCK_MAX_AGE_MS };
+export { ReposManager, STALE_LOCK_MAX_AGE_MS, repoSlug };

package/services/SilkPublishability.js

@@ -1,10 +1,35 @@
 import { ChangesetConfig } from "./ChangesetConfig.js";
 import { PublishTargetBindingError, ReleaseChannelError, trimTrailingSlashes } from "@savvy-web/silk-core";
-import { Effect, FileSystem, Layer } from "effect";
+import { Effect, FileSystem, Layer, Schema } from "effect";
 import { isAbsolute, join, relative } from "node:path";
 import { PublishTarget, PublishabilityDetector, WorkspaceDiscovery } from "@effected/workspaces";
 
 // src/services/SilkPublishability.ts
+/**
+* Structural decoder for {@link TargetsBinding}.
+*
+* @remarks
+* `releases` is decoded as `unknown` on purpose: an unrecognised channel must surface as a typed
+* `ReleaseChannelError` from `releaseChannelOf` (which names the package and target), not collapse
+* into a missing binding here. Every other field is validated structurally.
+*/
+const TargetsBindingSchema = Schema.Struct({
+	groups: Schema.Array(Schema.Struct({
+		id: Schema.String,
+		name: Schema.String,
+		dir: Schema.String
+	})),
+	targets: Schema.Array(Schema.Struct({
+		id: Schema.String,
+		group: Schema.String,
+		name: Schema.String,
+		registry: Schema.String,
+		releases: Schema.optionalKey(Schema.Unknown),
+		provenance: Schema.optionalKey(Schema.Boolean)
+	}))
+});
+/** `targets.json` file contents (a JSON string) decoded to a {@link TargetsBinding}. */
+const TargetsBindingFromJson = Schema.fromJsonString(TargetsBindingSchema);
 const NPM_DEFAULT = "https://registry.npmjs.org/";
 /**
 * Default registry endpoints for the well-known target keys. Kept in sync with the
@@ -413,9 +438,6 @@ const readRaw = (fs, packageJsonPath) => fs.readFileString(packageJsonPath).pipe
 * @since 1.0.0
 * @public
 */
-const readTargetsBinding = (fs, pkgPath) => fs.readFileString(join(pkgPath, "dist", "prod", "targets.json")).pipe(Effect.flatMap((content) => Effect.try({
-	try: () => JSON.parse(content),
-	catch: () => /* @__PURE__ */ new Error("invalid targets.json")
-})), Effect.orElseSucceed(() => null));
+const readTargetsBinding = (fs, pkgPath) => fs.readFileString(join(pkgPath, "dist", "prod", "targets.json")).pipe(Effect.flatMap(Schema.decodeUnknownEffect(TargetsBindingFromJson)), Effect.map((binding) => binding), Effect.orElseSucceed(() => null));
 
 export { SilkPublishability, readTargetsBinding };

package/turbo/services/TurboInspector.js

@@ -3,8 +3,8 @@ import { DryRunParseError, NotATurboRepoError, TurboExecError, TurboNotInstalled
 import { TurboDryRun } from "../schemas/DryRun.js";
 import { Context, Effect, FileSystem, Layer, Schema } from "effect";
 import { Git } from "@effected/git";
-import { ChildProcess, ChildProcessSpawner } from "effect/process";
 import { Run, Tool, ToolDiscovery } from "@effected/commands";
+import { ChildProcess, ChildProcessSpawner } from "effect/process";
 
 // src/turbo/services/TurboInspector.ts
 /**

package/changesets/utils/strip-frontmatter.js

@@ -1,26 +0,0 @@
-// src/changesets/utils/strip-frontmatter.ts
-/**
-* Strip leading YAML frontmatter from a markdown string.
-*
-* Removes the `---\n...\n---\n` block at the start of the content.
-* If no frontmatter is present, the content is returned as-is.
-*
-* @param content - Raw markdown string potentially containing frontmatter
-* @returns The markdown content with frontmatter removed
-*
-* @example
-* ```typescript
-* import { stripFrontmatter } from "../utils/strip-frontmatter.js";
-*
-* const raw = "---\n\"pkg\": minor\n---\n\n## Features\n\n- New API";
-* const body = stripFrontmatter(raw);
-* // "\n## Features\n\n- New API"
-* ```
-*
-* @internal
-*/
-function stripFrontmatter(content) {
-	return content.replace(/^---\n[\s\S]*?\n---\n?/, "");
-}
-
-export { stripFrontmatter };