@crustjs/skills 0.3.0 → 0.3.2

package/dist/index.d.ts

@@ -1,4 +1,5 @@
 import { CommandDefinition, CommandSnapshot, ExtensionFactory } from "@crustjs/core";
+import "@crustjs/utils/primitive";
 //#region src/agents.d.ts
 type AgentClass = "universal" | "additional";
 type Scope = "global" | "project";

package/dist/index.js

@@ -1,13 +1,17 @@
 import { lstat, mkdir, readFile, readdir, readlink, realpath, rm, stat, symlink, unlink, writeFile } from "node:fs/promises";
-import { basename, delimiter, dirname, extname, isAbsolute, join, posix, relative, resolve, sep } from "node:path";
-import { accessSync, constants, existsSync, readFileSync, readdirSync, realpathSync, statSync } from "node:fs";
-import { fileURLToPath } from "node:url";
+import { basename, dirname, join, posix, relative, resolve } from "node:path";
+import { isWithin } from "@crustjs/utils/path";
+import { resolveSourceDir } from "@crustjs/utils/source";
 import { buildCommandDocumentation, formatDefault, sectionsFor } from "@crustjs/core/tooling";
 import { defineCommand, defineExtension, defineExtensionId } from "@crustjs/core";
 import { spinner } from "@crustjs/progress";
 import { confirm, multiselect, select } from "@crustjs/prompts";
 import { bold, dim, yellow } from "@crustjs/style";
+import { resolveArtifactDir } from "@crustjs/utils/artifacts";
 import { homedir } from "node:os";
+import { which } from "@crustjs/utils/process";
+import { isErrnoException } from "@crustjs/utils/error";
+import { existsSync, readFileSync, readdirSync } from "node:fs";
 //#region \0rolldown/runtime.js
 var __defProp = Object.defineProperty;
 var __exportAll = (all, no_symbols) => {
@@ -20,95 +24,6 @@ var __exportAll = (all, no_symbols) => {
 	return target;
 };
 //#endregion
-//#region ../utils/src/path.ts
-/** Returns whether child is parent itself or lies below it. */
-function isWithin(parent, child) {
-	const path = relative(parent, child);
-	return path === "" || !isAbsolute(path) && path !== ".." && !path.startsWith(`..${sep}`);
-}
-//#endregion
-//#region ../utils/src/source.ts
-/**
-* Find the nearest directory containing `package.json` by walking up the
-* filesystem from `startPath`.
-*
-* If `startPath` points at an existing file, its parent directory is used as
-* the starting point. The directory walk uses `path.resolve()` (lexical), not
-* `fs.realpath()`, so a symlink's parent chain is walked rather than the
-* symlink target's parent chain.
-*
-* @param startPath - Directory or file path to start walking from.
-* @returns The absolute path of the nearest enclosing directory containing
-*   `package.json`, or `null` if the filesystem root is reached first.
-*
-* @internal Shared by {@link resolveSourceDir} and `resolveArtifactDir`.
-*/
-function findNearestPackageRoot(startPath) {
-	let current = resolve(startPath);
-	if (existsSync(current) && !statSync(current).isDirectory()) current = dirname(current);
-	while (true) {
-		if (existsSync(join(current, "package.json"))) return current;
-		const parent = dirname(current);
-		if (parent === current) return null;
-		current = parent;
-	}
-}
-/**
-* Resolves an absolute filesystem path from a caller-supplied source
-* directory descriptor. Designed for tools that ship reference assets
-* (templates, skill bundles, etc.) alongside their published package and
-* need a uniform way to locate them at runtime regardless of how the
-* consumer expressed the path.
-*
-* Three input modes are supported:
-* - **`URL`** — must use the `file:` protocol. Resolved via
-*   `url.fileURLToPath()`. The intended idiom is
-*   `new URL("./relative/path", import.meta.url)`.
-* - **Absolute string path** — returned as `path.resolve(input)`.
-* - **Relative string path** — resolved against the nearest `package.json`
-*   directory walking up from `process.argv[1]`. This makes
-*   `"templates/base"` resolve to `<consumer-package-root>/templates/base`
-*   regardless of cwd.
-*
-* Three failure modes throw descriptive `Error`s:
-* - URL with non-`file:` protocol — message names the offending protocol.
-* - Relative string path with `process.argv[1]` unset — message suggests
-*   switching to an absolute path or a `file:` URL.
-* - Relative string path but no `package.json` found walking up — message
-*   names the entrypoint and the relative input, and suggests switching to
-*   an absolute path or a `file:` URL.
-*
-* @param input - File URL, absolute path, or package-relative path.
-* @returns Absolute filesystem path.
-* @throws {Error} For any of the three failure modes above.
-*
-* @example
-* ```ts
-* import { resolveSourceDir } from "@crustjs/utils/source";
-*
-* // 1. file: URL — relative to the calling module
-* const a = resolveSourceDir(new URL("../templates/base", import.meta.url));
-*
-* // 2. Absolute path
-* const b = resolveSourceDir("/abs/path/to/templates/base");
-*
-* // 3. Relative path — resolved from the consuming package's root
-* const c = resolveSourceDir("templates/base");
-* ```
-*/
-function resolveSourceDir(input) {
-	if (input instanceof URL) {
-		if (input.protocol !== "file:") throw new Error(`sourceDir URL must use file: protocol, got "${input.protocol}".`);
-		return fileURLToPath(input);
-	}
-	if (isAbsolute(input)) return resolve(input);
-	const entrypoint = process.argv[1];
-	if (!entrypoint) throw new Error(`Could not resolve relative sourceDir "${input}" because process.argv[1] is not set. Pass an absolute path or a file: URL.`);
-	const packageRoot = findNearestPackageRoot(resolve(entrypoint));
-	if (!packageRoot) throw new Error(`Could not resolve relative sourceDir "${input}" from entrypoint "${entrypoint}" because no package.json was found in its parent directories. Pass an absolute path or a file: URL.`);
-	return resolve(packageRoot, input);
-}
-//#endregion
 //#region src/bundle.ts
 /** Filename of the entrypoint markdown file required at the bundle root. */
 const SKILL_MD = "SKILL.md";
@@ -230,12 +145,13 @@ async function loadBundleFiles(sourceDir) {
 	const collected = await collectBundleEntries(canonicalRoot, canonicalRoot, "", /* @__PURE__ */ new Set([canonicalRoot]));
 	const skillMd = collected.find((f) => f.relPath === SKILL_MD);
 	if (!skillMd) throw new Error(`Extra skill directory is missing SKILL.md at its root "${canonicalRoot}". Every extra skill directory must contain a top-level SKILL.md file.`);
+	const files = await Promise.all(collected.map(async (entry) => ({
+		path: entry.relPath,
+		content: await readFile(entry.absPath)
+	})));
 	return {
-		files: await Promise.all(collected.map(async (entry) => ({
-			path: entry.relPath,
-			content: await readFile(entry.absPath)
-		}))),
-		frontmatter: requireSkillFrontmatter(probeFrontmatter(await readFile(skillMd.absPath, "utf-8")), `Extra skill SKILL.md at "${join(canonicalRoot, SKILL_MD)}"`)
+		files,
+		frontmatter: requireSkillFrontmatter(probeFrontmatter(files[collected.indexOf(skillMd)].content.toString("utf-8")), `Extra skill SKILL.md at "${join(canonicalRoot, SKILL_MD)}"`)
 	};
 }
 //#endregion
@@ -258,77 +174,6 @@ var SkillConflictError = class extends Error {
 	}
 };
 //#endregion
-//#region ../utils/src/artifacts.ts
-/**
-* Set by `crust build` while it prepares the Command Snapshot: the absolute
-* entry-isolated directory Extension build hooks write into. Core reads it
-* to run the hooks; `resolveArtifactDir` reads it so sections evaluated during
-* that run see the artifacts being built instead of the wiped `.crust/root`.
-*/
-const BUILD_OUT_DIR_ENV = "CRUST_INTERNAL_BUILD_OUT_DIR";
-/** True inside a `bun build --compile` or `deno compile` executable. */
-function isCompiledExecutable() {
-	const { Bun, Deno } = globalThis;
-	const bunMain = Bun?.main ?? "";
-	return bunMain.startsWith("/$bunfs/") || /^[A-Za-z]:[\\/]~BUN[\\/]/.test(bunMain) || Deno?.build?.standalone === true;
-}
-/**
-* Absolute path of a build artifact or `crust.include` directory shipped with
-* this CLI. `name` is a top-level directory name such as `"skills"`.
-*
-* The artifact path is computed from how the CLI is running — never probed:
-* - Compiled executable (Bun or Deno): `<dir of the executable>/<name>`, which
-*   is a platform package's `bin/` or wherever the binary was placed.
-* - Crust-built Node bundle: `<name>` next to the bundle's `bin/` directory,
-*   i.e. `.crust/root/<name>` in place and `<installed root>/<name>` after install.
-* - Snapshot preparation inside `crust build`: `<build output dir>/<name>`, the
-*   artifacts earlier Extension build hooks wrote in this same build.
-* - Source (`bun run`, `node`, `deno run`): `.crust/root/<name>` under the
-*   nearest package root of the real `process.argv[1]` entrypoint (following source
-*   links) — the output of the last `crust build`.
-*
-* @throws {Error} when `name` is not a single path segment, or in source mode
-*   when the entrypoint cannot be resolved or has no enclosing `package.json`.
-*/
-function resolveArtifactDir(name) {
-	if (name === "" || name === "." || name === ".." || /[\\/]/.test(name)) throw new Error(`Artifact name must be a single directory name, got ${JSON.stringify(name)}.`);
-	if (isCompiledExecutable()) return join(dirname(process.execPath), name);
-	if (process.env.CRUST_INTERNAL_BUILD === "1") return resolve(fileURLToPath(import.meta.url), "..", "..", name);
-	const buildOutDir = process.env[BUILD_OUT_DIR_ENV];
-	if (buildOutDir) return join(buildOutDir, name);
-	const entrypoint = process.argv[1];
-	let sourceEntrypoint = entrypoint;
-	if (entrypoint) try {
-		sourceEntrypoint = realpathSync(entrypoint);
-	} catch (cause) {
-		throw new Error(`Could not resolve artifact "${name}": could not resolve source entrypoint "${entrypoint}".`, { cause });
-	}
-	const packageRoot = sourceEntrypoint ? findNearestPackageRoot(sourceEntrypoint) : null;
-	if (!packageRoot) throw new Error(`Could not resolve artifact "${name}": no package.json was found above ${entrypoint ? `entrypoint "${entrypoint}"` : "process.argv[1] (unset)"}.`);
-	return join(packageRoot, ".crust", "root", name);
-}
-//#endregion
-//#region ../utils/src/process.ts
-/** Resolve a bare executable name from PATH (and PATHEXT on Windows). */
-function which(command) {
-	if (command.includes(sep) || command.includes("/")) {
-		try {
-			accessSync(command, constants.X_OK);
-			if (statSync(command).isFile()) return command;
-		} catch {}
-		return null;
-	}
-	const extensions = process.platform === "win32" && !extname(command) ? (process.env.PATHEXT ?? ".EXE;.CMD;.BAT;.COM").split(";") : [""];
-	for (const directory of process.env.PATH?.split(delimiter) ?? []) for (const extension of extensions) {
-		const candidate = resolve(join(directory, command + extension));
-		try {
-			accessSync(candidate, constants.X_OK);
-			if (statSync(candidate).isFile()) return candidate;
-		} catch {}
-	}
-	return null;
-}
-//#endregion
 //#region src/agents.ts
 const PROJECT_UNIVERSAL_SKILLS_DIR = join(".agents", "skills");
 function configHome(home) {
@@ -617,12 +462,6 @@ function resolveAgentPath(agent, scope, name) {
 	return join(cfg.globalSkillsDir(homedir()), name);
 }
 //#endregion
-//#region ../utils/src/error.ts
-/** Narrows a caught Node.js system error to its stable errno contract. */
-function isErrnoException(error) {
-	return error instanceof Error && "code" in error && typeof error.code === "string";
-}
-//#endregion
 //#region src/link.ts
 /** Returns whether a symlink target carries Crust's skill ownership signature. */
 function isOwnedSkillLink(target, name) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@crustjs/skills",
-	"version": "0.3.0",
+	"version": "0.3.2",
 	"description": "Package and install agent skills for AI coding assistants.",
 	"type": "module",
 	"sideEffects": false,
@@ -45,20 +45,20 @@
 		"postpack": "rm -f LICENSE"
 	},
 	"dependencies": {
-		"@crustjs/progress": "^0.1.1",
-		"@crustjs/prompts": "^0.2.2",
-		"@crustjs/style": "^0.3.2"
+		"@crustjs/progress": "^0.1.2",
+		"@crustjs/prompts": "^0.2.3",
+		"@crustjs/style": "^0.3.3",
+		"@crustjs/utils": "^0.1.0"
 	},
 	"devDependencies": {
 		"@crustjs/config": "0.0.0",
-		"@crustjs/core": "0.3.0",
-		"@crustjs/extensions": "0.3.0",
+		"@crustjs/core": "0.3.2",
+		"@crustjs/extensions": "0.3.2",
 		"@crustjs/testing": "0.1.2",
-		"@crustjs/utils": "0.0.0",
 		"tsdown": "^0.23.0"
 	},
 	"peerDependencies": {
-		"@crustjs/core": "^0.3.0",
+		"@crustjs/core": "^0.3.2",
 		"typescript": "^7.0.0"
 	},
 	"peerDependenciesMeta": {