hngr 0.2.0 → 0.4.0

package/dist/{skills-DwoDzsXd.mjs → skills-CxMeJ5PF.mjs}

@@ -1,5 +1,5 @@
-import { S as HngrError, g as resolveConnection, r as apiRequest, y as writeDirectoryAtomically } from "./http-2HWxLIOq.mjs";
-import { _ as resolveCapabilityVersion, a as failedResult, c as stringValues, d as homeDir, f as downloadContent, g as resolveCapability, h as parseCapabilityPath, i as errorBody, l as AGENTS, m as SKILL_KIND, n as readPublishFlags, o as positionalPaths, r as runPublish, s as runBatch, t as explicitName } from "./publish-D7zaQl9x.mjs";
+import { C as HngrError, b as writeDirectoryAtomically, g as resolveConnection, r as apiRequest } from "./http-DgHX7Fux.mjs";
+import { _ as resolveCapability, a as errorBody, c as runBatch, f as homeDir, g as parseCapabilityPath, h as SKILL_KIND, i as runPublish, l as stringValues, n as explicitName, o as failedResult, p as downloadContent, r as readPublishFlags, s as positionalPaths, t as resolveInstallScope, u as AGENTS, v as resolveCapabilityVersion } from "./scope-Ch6wZCgE.mjs";
 import { lstat, mkdir, readFile, readdir, rename, rm, symlink, writeFile } from "node:fs/promises";
 import { basename, dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
 import { randomUUID } from "node:crypto";
@@ -92,13 +92,6 @@ function downloadSkillPackage(connection, capabilityId, version, signal) {
 async function pathExists(path) {
 	return lstat(path).then(() => true, () => false);
 }
-function targetExistsError(path) {
-	return new HngrError({
-		type: "io",
-		title: `Target already exists: ${path}`,
-		detail: "Pass --force to replace it."
-	});
-}
 async function writeEntries(dest, entries) {
 	await mkdir(dest, { recursive: true });
 	for (const entry of entries) {
@@ -114,25 +107,22 @@ async function writeEntries(dest, entries) {
 }
 /**
 * Writes a skill through "temp directory → validate → rename": the target is
-* either absent or complete, never half-written. `--force` moves the old
-* directory aside before the rename so a failed swap puts it back.
+* either absent or complete, never half-written. Re-installing replaces the
+* previous directory, with the old one moved aside so a failed swap puts it
+* back.
 */
-async function installAtomically(dest, entries, force) {
-	await writeDirectoryAtomically(dest, (temp) => writeEntries(temp, entries), {
-		force,
-		existsError: () => targetExistsError(dest)
-	});
+async function installAtomically(dest, entries) {
+	await writeDirectoryAtomically(dest, (temp) => writeEntries(temp, entries));
 }
 /**
 * Creates a relative symlink to the canonical directory, falling back to a
 * copy when the platform refuses the link (Windows without Developer Mode).
 * Windows prefers a junction, which does not need elevated privileges; its
 * target must be absolute. A temp link plus rename replaces an existing link
-* atomically.
+* atomically; re-installing replaces an existing directory or link.
 */
-async function linkAtomically(canonical, target, entries, force, options = {}) {
+async function linkAtomically(canonical, target, entries, options = {}) {
 	const exists = await pathExists(target);
-	if (exists && !force) throw targetExistsError(target);
 	await mkdir(dirname(target), { recursive: true });
 	const temp = resolve(dirname(target), `.hngr-${basename(target)}-${randomUUID()}.link`);
 	const windows = (options.platform ?? process.platform) === "win32";
@@ -150,7 +140,7 @@ async function linkAtomically(canonical, target, entries, force, options = {}) {
 		return { mode: "symlink" };
 	} catch {
 		await rm(temp, { force: true }).catch(() => {});
-		await installAtomically(target, entries, false);
+		await installAtomically(target, entries);
 		return {
 			mode: "copy",
 			warning: `warning: could not create a link at ${target}; copied instead`
@@ -158,8 +148,8 @@ async function linkAtomically(canonical, target, entries, force, options = {}) {
 	}
 }
 /** Copies one target directory atomically, without a canonical store. */
-async function copyAtomically(target, entries, force) {
-	await installAtomically(target, entries, force);
+async function copyAtomically(target, entries) {
+	await installAtomically(target, entries);
 }
 // src/skills/resolve.ts
 /** Parses `<name>` or `<namespace>/<name>`, with an optional `@YYYY.MM.DD.N`. */
@@ -657,16 +647,12 @@ function planInstall(cwd, env, destName, agents, options) {
 async function install(ctx, plan, entries) {
 	const cwd = process.cwd();
 	const display = (path) => displayPath(path, cwd);
-	const destinations = [plan.canonical, ...plan.targets.map((target) => target.path)].filter((path) => path !== null);
-	if (!ctx.flags.force) {
-		for (const path of destinations) if (await pathExists(path)) throw targetExistsError(display(path));
-	}
-	if (plan.canonical) await installAtomically(plan.canonical, entries, ctx.flags.force);
+	if (plan.canonical) await installAtomically(plan.canonical, entries);
 	const installed = [];
 	let failure = null;
 	for (const target of plan.targets) try {
 		if (target.mode === "copy" || !plan.canonical) {
-			await copyAtomically(target.path, entries, ctx.flags.force);
+			await copyAtomically(target.path, entries);
 			installed.push({
 				agent: target.agent,
 				path: display(target.path),
@@ -675,7 +661,7 @@ async function install(ctx, plan, entries) {
 			});
 			continue;
 		}
-		const result = await linkAtomically(plan.canonical, target.path, entries, ctx.flags.force);
+		const result = await linkAtomically(plan.canonical, target.path, entries);
 		if (result.warning) ctx.summary(result.warning);
 		installed.push({
 			agent: target.agent,
@@ -708,9 +694,6 @@ async function addOne(ctx, connection, parsed, destName, agents, dir, global, co
 	});
 	const canonicalPath = plan.canonical ? displayPath(plan.canonical, process.cwd()) : null;
 	if (ctx.flags.dryRun) {
-		if (!ctx.flags.force) {
-			for (const path of [plan.canonical, ...plan.targets.map((target) => target.path)]) if (path && await pathExists(path)) throw targetExistsError(displayPath(path, process.cwd()));
-		}
 		ctx.summary(`resolved ${parsed.namespace}/${parsed.name} to version ${version}`);
 		return { result: {
 			name: capability.name,
@@ -784,9 +767,9 @@ function skillPrepared(pkg, metadata) {
 			name: "readme",
 			value: metadata.readme
 		});
-		if (metadata.tags.length > 0) parts.push({
-			name: "tags",
-			value: metadata.tags.join(",")
+		if (metadata.category !== void 0) parts.push({
+			name: "category",
+			value: metadata.category
 		});
 		parts.push(packagePart(pkg, metadata.name));
 		return parts;
@@ -809,7 +792,7 @@ function skillPrepared(pkg, metadata) {
 			if (metadata.description !== void 0) preview.description = metadata.description;
 			preview.visibility = metadata.visibility;
 			if (metadata.readme !== void 0) preview.readme = metadata.readme;
-			if (metadata.tags.length > 0) preview.tags = metadata.tags.join(",");
+			if (metadata.category !== void 0) preview.category = metadata.category;
 			preview.package = `<${pkg.fileCount} files, ${pkg.byteCount} bytes>`;
 			return preview;
 		},
@@ -828,7 +811,7 @@ function skillPrepared(pkg, metadata) {
 async function publish(ctx, argv) {
 	const rawPaths = positionalPaths(argv.paths);
 	if (rawPaths.length === 0) throw usage("No skill directory given", "Pass one or more local directories that contain SKILL.md.");
-	const flags = await readPublishFlags(argv, "skill");
+	const flags = await readPublishFlags(argv);
 	const forcedName = explicitName(argv, rawPaths.length);
 	const items = rawPaths.map((dir) => ({
 		name: forcedName ?? basename(resolve(dir)),
@@ -849,12 +832,13 @@ async function add(ctx, argv) {
 	const dir = stringFlag(argv, "dir");
 	const as = stringFlag(argv, "as");
 	if (as !== void 0 && (as.includes("/") || as.includes("\\") || as === "." || as === "..")) throw usage(`Invalid --as name ${JSON.stringify(as)}`, "Pass a single directory name.");
-	const global = argv.global === true;
+	const requestedGlobal = argv.global === true;
 	const copy = argv.copy === true;
-	if (dir !== void 0 && global) throw usage("--dir cannot be combined with --global");
+	if (dir !== void 0 && requestedGlobal) throw usage("--dir cannot be combined with --global");
 	if (dir !== void 0 && (stringValues(argv.agent).length > 0 || argv.all === true)) throw usage("--dir cannot be combined with --agent or --all");
 	if (as !== void 0) assertNotReserved(as);
 	for (const parsed of parsedPaths) assertNotReserved(as ?? parsed.name);
+	const global = dir === void 0 ? await resolveInstallScope(ctx, requestedGlobal) === "global" : false;
 	const agents = dir === void 0 ? await selectAgents(ctx, argv, rawPaths[0] ?? "", global) : [];
 	const connection = await resolveConnection(ctx.flags, ctx.env);
 	const { results, failed } = await runBatch({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hngr",
-  "version": "0.2.0",
+  "version": "0.4.0",
   "description": "The Hangar CLI: install skills and MCP servers into coding agents, and drive the Management API.",
   "keywords": [
     "hangar",