happyskills 0.34.0 → 0.35.1

package/CHANGELOG.md

@@ -7,6 +7,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.35.1] - 2026-04-10
+
+### Fixed
+- Fix multi-agent skill symlinks using absolute paths instead of relative — absolute symlinks break when the repo is cloned or pulled on a different machine; now uses `path.relative()` for portable symlinks
+- Fix enable skip-check failing to recognize existing relative symlinks — resolves both relative and absolute link targets before comparing
+
+## [0.35.0] - 2026-04-10
+
+### Added
+- Add multi-skill `install` — pass multiple skills in one command (e.g., `happyskills install acme/foo acme/bar@1.0 teamb/baz`)
+- Add multi-skill `uninstall` — remove multiple skills in one command (e.g., `happyskills uninstall acme/foo acme/bar`)
+- Add per-skill inline `@version` pinning for multi-skill install (e.g., `acme/foo@1.2.0 acme/bar@latest`)
+- Add graceful error handling for batch operations — individual failures print a warning and do not interrupt remaining skills
+- Add `--version` flag guard — rejects ambiguous usage with multiple skills, directs to inline `@version` syntax
+- Add `errors` array in JSON output for partial failures alongside successful `data`
+
 ## [0.34.0] - 2026-04-09
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "happyskills",
-	"version": "0.34.0",
+	"version": "0.35.1",
 	"description": "Package manager for AI agent skills",
 	"license": "SEE LICENSE IN LICENSE",
 	"author": "Nicolas Dao <nic@cloudlesslabs.com> (https://cloudlesslabs.com)",

package/src/agents/linker.js

@@ -43,8 +43,10 @@ const _exists = async (target_path) => {
  */
 const _link_or_copy = async (source, target) => {
 	try {
+		// Use relative path so symlinks work across machines (clones/pulls)
+		const relative_source = path.relative(path.dirname(target), source)
 		// 'junction' works on Windows without admin; ignored on macOS/Linux
-		await fs.promises.symlink(source, target, 'junction')
+		await fs.promises.symlink(relative_source, target, 'junction')
 		return { method: 'symlink' }
 	} catch (err) {
 		if (err.code === 'EPERM' || err.code === 'ENOTSUP') {
@@ -77,7 +79,9 @@ const link_to_agents = (source_dir, agents, options = {}) => catch_errors('Agent
 		if (is_link) {
 			try {
 				const link_target = await fs.promises.readlink(target)
-				if (link_target === source_dir) {
+				// Resolve relative or absolute symlink to compare against source
+				const resolved_link = path.resolve(path.dirname(target), link_target)
+				if (resolved_link === source_dir) {
 					results.push({ agent_id: agent.id, path: target, method: 'symlink', skipped: true })
 					continue
 				}

package/src/agents/linker.test.js

@@ -71,7 +71,9 @@ describe('link_to_agents', () => {
 		const link_path = path.join(tmp, '.claude', 'skills', 'deploy-aws')
 		const stat = fs.lstatSync(link_path)
 		assert.ok(stat.isSymbolicLink())
-		assert.equal(fs.readlinkSync(link_path), source)
+		// Symlink should be relative, not absolute
+		const link_target = fs.readlinkSync(link_path)
+		assert.equal(link_target, path.relative(path.dirname(link_path), source))
 	})
 
 	it('creates symlinks for multiple agents', async () => {
@@ -126,7 +128,9 @@ describe('link_to_agents', () => {
 		assert.equal(results[0].skipped, undefined)
 
 		const link_path = path.join(tmp, '.claude', 'skills', 'deploy-aws')
-		assert.equal(fs.readlinkSync(link_path), source)
+		// Symlink should be relative, not absolute
+		const link_target = fs.readlinkSync(link_path)
+		assert.equal(link_target, path.relative(path.dirname(link_path), source))
 	})
 })
 

package/src/commands/install.js

@@ -2,14 +2,14 @@ const { error: { catch_errors, wrap_errors: e } } = require('puffy-core')
 const { install, install_from_manifest, install_from_lock } = require('../engine/installer')
 const { read_lock } = require('../lock/reader')
 const { read_manifest } = require('../manifest/reader')
-const { print_help, print_hint, print_json, code } = require('../ui/output')
+const { print_help, print_hint, print_json, print_warn, code } = require('../ui/output')
 const { exit_with_error, UsageError } = require('../utils/errors')
 const { find_project_root } = require('../config/paths')
 const { EXIT_CODES } = require('../constants')
 
-const HELP_TEXT = `Usage: happyskills install [<owner>/<skill>[@<version>]] [options]
+const HELP_TEXT = `Usage: happyskills install [<owner>/<skill>[@<version>]] [...] [options]
 
-Install a skill and its dependencies.
+Install one or more skills and their dependencies.
 Without arguments, installs all dependencies from skill.json (or skills-lock.json if no skill.json exists).
 
 Arguments:
@@ -18,8 +18,8 @@ Arguments:
   owner/skill@latest       Force install of the latest version (bypasses lock skip check)
 
 Options:
-  -g, --global        Install globally (~/.claude/skills/)
-  --version <ver>     Pin to specific version (overrides inline @version)
+  -g, --global        Install globally (~/.agents/skills/)
+  --version <ver>     Pin to specific version (single skill only, overrides inline @version)
   --force             Force install on dependency conflicts
   --fresh             Ignore lock file, re-resolve from scratch
   --agents <list>     Target specific agents (comma-separated, e.g., claude,cursor)
@@ -32,8 +32,8 @@ Aliases: i, add
 Examples:
   happyskills install
   happyskills install acme/deploy-aws
-  happyskills install acme/deploy-aws@1.2.0
-  happyskills install acme/deploy-aws@latest
+  happyskills install acme/deploy-aws acme/monitor acme/logging
+  happyskills install acme/deploy-aws@1.2.0 acme/monitor@latest
   happyskills install acme/deploy-aws --version 1.2.0
   happyskills i acme/deploy-aws -g`
 
@@ -43,20 +43,9 @@ const run = (args) => catch_errors('Install failed', async () => {
 		return process.exit(EXIT_CODES.SUCCESS)
 	}
 
-	let skill = args._[0]
-	let version = args.flags.version || undefined
-
-	// Support owner/skill@version inline syntax
-	if (skill && skill.includes('@')) {
-		const at_idx = skill.lastIndexOf('@')
-		const inline_version = skill.slice(at_idx + 1)
-		skill = skill.slice(0, at_idx)
-		if (inline_version && !args.flags.version) version = inline_version
-	}
-
-	const options = {
+	const skills_raw = args._
+	const base_options = {
 		global: args.flags.global || false,
-		version,
 		force: args.flags.force || false,
 		fresh: args.flags.fresh || false,
 		yes: args.flags.yes || false,
@@ -64,12 +53,12 @@ const run = (args) => catch_errors('Install failed', async () => {
 		project_root: find_project_root()
 	}
 
-	if (!skill) {
+	if (skills_raw.length === 0) {
 		const [manifest_err, manifest] = await read_manifest()
 		if (manifest_err) {
-			const [, lock_data] = await read_lock(options.project_root)
+			const [, lock_data] = await read_lock(base_options.project_root)
 			if (lock_data && lock_data.skills && Object.keys(lock_data.skills).length > 0) {
-				const [errors, result] = await install_from_lock(lock_data, options)
+				const [errors, result] = await install_from_lock(lock_data, base_options)
 				if (errors) throw e('Install from lock failed', errors)
 				if (args.flags.json) {
 					print_json({ data: result })
@@ -80,7 +69,7 @@ const run = (args) => catch_errors('Install failed', async () => {
 			throw new UsageError("No skill specified and no skill.json found. Run 'happyskills init' to create a skill.json, or specify a skill to install.")
 		}
 
-		const [errors, result] = await install_from_manifest(manifest, options)
+		const [errors, result] = await install_from_manifest(manifest, base_options)
 		if (errors) throw e('Install from manifest failed', errors)
 		if (args.flags.json) {
 			print_json({ data: result })
@@ -89,22 +78,62 @@ const run = (args) => catch_errors('Install failed', async () => {
 		return
 	}
 
-	if (!skill.includes('/')) {
-		throw new UsageError('Skill must be in owner/name format (e.g., acme/deploy-aws).')
+	// Parse each argument: extract skill name and optional inline @version
+	const parsed = skills_raw.map(raw => {
+		let skill = raw, version = undefined
+		if (skill.includes('@')) {
+			const at_idx = skill.lastIndexOf('@')
+			version = skill.slice(at_idx + 1)
+			skill = skill.slice(0, at_idx)
+		}
+		return { skill, version }
+	})
+
+	for (const { skill } of parsed) {
+		if (!skill.includes('/')) {
+			throw new UsageError(`Skill must be in owner/name format (e.g., acme/deploy-aws). Got: ${skill}`)
+		}
 	}
 
-	const [errors, result] = await install(skill, options)
-	if (errors) throw e(`Install ${skill} failed`, errors)
+	// --version flag only makes sense for a single skill
+	const flag_version = args.flags.version || undefined
+	if (flag_version && parsed.length > 1) {
+		throw new UsageError('--version flag cannot be used with multiple skills. Use inline @version syntax instead (e.g., acme/foo@1.2.0 acme/bar@2.0.0).')
+	}
+
+	const results = []
+	const failures = []
+	for (const { skill, version: inline_version } of parsed) {
+		const version = flag_version || inline_version
+		const [errors, result] = await install(skill, { ...base_options, version })
+		if (errors) {
+			const msg = errors[0]?.message || errors.message || String(errors)
+			print_warn(`Failed to install ${skill}: ${msg}`)
+			failures.push({ skill, error: msg })
+		} else {
+			results.push({ skill, result })
+		}
+	}
+
+	if (results.length === 0 && failures.length > 0) {
+		throw new Error(`All ${failures.length} install(s) failed.`)
+	}
 
 	if (args.flags.json) {
-		print_json({ data: {
+		const items = results.map(({ skill, result }) => ({
 			skill,
 			version: result.version,
 			installed: result.installed || [],
 			skipped: result.skipped || [],
 			warnings: result.warnings || [],
 			forced: result.forced || []
-		} })
+		}))
+		const data = results.length === 1 && failures.length === 0 ? items[0] : items
+		if (failures.length > 0) {
+			print_json({ data, errors: failures })
+		} else {
+			print_json({ data })
+		}
 		return
 	}
 

package/src/commands/uninstall.js

@@ -1,16 +1,16 @@
 const { error: { catch_errors, wrap_errors: e } } = require('puffy-core')
 const { uninstall } = require('../engine/uninstaller')
-const { print_help, print_json } = require('../ui/output')
+const { print_help, print_json, print_warn } = require('../ui/output')
 const { exit_with_error, UsageError } = require('../utils/errors')
 const { find_project_root } = require('../config/paths')
 const { EXIT_CODES } = require('../constants')
 
-const HELP_TEXT = `Usage: happyskills uninstall <owner/skill> [options]
+const HELP_TEXT = `Usage: happyskills uninstall <owner/skill> [...] [options]
 
-Remove a skill and prune orphaned dependencies.
+Remove one or more skills and prune orphaned dependencies.
 
 Arguments:
-  owner/skill       Skill to remove (e.g., acme/deploy-aws)
+  owner/skill       Skill(s) to remove (e.g., acme/deploy-aws teamb/logging)
 
 Options:
   -g, --global        Remove from global scope
@@ -23,6 +23,7 @@ Aliases: rm, remove
 
 Examples:
   happyskills uninstall acme/deploy-aws
+  happyskills uninstall acme/deploy-aws acme/monitor acme/logging
   happyskills rm acme/deploy-aws -g`
 
 const run = (args) => catch_errors('Uninstall failed', async () => {
@@ -31,13 +32,15 @@ const run = (args) => catch_errors('Uninstall failed', async () => {
 		return process.exit(EXIT_CODES.SUCCESS)
 	}
 
-	const skill = args._[0]
-	if (!skill) {
+	const skills_raw = args._
+	if (skills_raw.length === 0) {
 		throw new UsageError('Please specify a skill to uninstall (e.g., happyskills uninstall acme/deploy-aws).')
 	}
 
-	if (!skill.includes('/')) {
-		throw new UsageError('Skill must be in owner/name format (e.g., acme/deploy-aws).')
+	for (const skill of skills_raw) {
+		if (!skill.includes('/')) {
+			throw new UsageError(`Skill must be in owner/name format (e.g., acme/deploy-aws). Got: ${skill}`)
+		}
 	}
 
 	const options = {
@@ -46,15 +49,35 @@ const run = (args) => catch_errors('Uninstall failed', async () => {
 		project_root: find_project_root()
 	}
 
-	const [errors, result] = await uninstall(skill, options)
-	if (errors) throw e(`Uninstall ${skill} failed`, errors)
+	const results = []
+	const failures = []
+	for (const skill of skills_raw) {
+		const [errors, result] = await uninstall(skill, options)
+		if (errors) {
+			const msg = errors[0]?.message || errors.message || String(errors)
+			print_warn(`${skill}: ${msg}`)
+			failures.push({ skill, error: msg })
+		} else {
+			results.push({ skill, result })
+		}
+	}
+
+	if (results.length === 0 && failures.length > 0) {
+		throw new Error(`All ${failures.length} uninstall(s) failed.`)
+	}
 
 	if (args.flags.json) {
-		print_json({ data: {
+		const items = results.map(({ skill, result }) => ({
 			skill,
 			removed: result.removed || [],
 			orphans_pruned: result.orphans_pruned || []
-		} })
+		}))
+		const data = results.length === 1 && failures.length === 0 ? items[0] : items
+		if (failures.length > 0) {
+			print_json({ data, errors: failures })
+		} else {
+			print_json({ data })
+		}
 		return
 	}
 }).then(([errors]) => { if (errors) { exit_with_error(errors); return } })