happyskills 0.35.4 → 0.35.5

package/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.35.5] - 2026-04-11
+
+### Fixed
+- Fix kits being symlinked to agent folders during `install`, `refresh`, and `enable` — kits are meta-packages not invocable by agents, only their dependency skills should be linked
+- Block `enable` command for kits with a clear warning instead of silently creating useless symlinks
+
 ## [0.35.4] - 2026-04-11
 
 ### Fixed

package/package.json

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

package/src/commands/enable.js

@@ -6,7 +6,7 @@ const { is_skill_enabled } = require('../agents/status')
 const { file_exists } = require('../utils/fs')
 const { print_help, print_json, print_success, print_warn, print_info } = require('../ui/output')
 const { exit_with_error, UsageError } = require('../utils/errors')
-const { EXIT_CODES } = require('../constants')
+const { EXIT_CODES, SKILL_TYPES } = require('../constants')
 
 const HELP_TEXT = `Usage: happyskills enable <skill> [skill2 ...] [options]
 
@@ -77,6 +77,13 @@ const run = (args) => catch_errors('Enable failed', async () => {
 
 		const { full, short } = resolved
 
+		// Kits are not agent-invocable — skip linking
+		if (locked_skills[full]?.type === SKILL_TYPES.KIT) {
+			print_warn(`${full} is a kit — kits are not linked to agent folders`)
+			results.push({ skill: full, status: 'kit_skipped' })
+			continue
+		}
+
 		// Verify the skill directory exists on disk
 		const dir = skill_install_dir(base_dir, short)
 		const [, exists] = await file_exists(dir)

package/src/commands/refresh.js

@@ -9,7 +9,7 @@ const { green, yellow, red } = require('../ui/colors')
 const { create_spinner } = require('../ui/spinner')
 const { exit_with_error } = require('../utils/errors')
 const { find_project_root, lock_root, skills_dir, skill_install_dir } = require('../config/paths')
-const { EXIT_CODES } = require('../constants')
+const { EXIT_CODES, SKILL_TYPES } = require('../constants')
 const { resolve_agents, verify_and_repair_symlinks } = require('../agents')
 const { is_skill_enabled } = require('../agents/status')
 
@@ -123,7 +123,8 @@ const run = (args) => catch_errors('Refresh failed', async () => {
 	let symlink_repairs = []
 	if (detected_agents.length > 0) {
 		const skills_to_check = []
-		for (const [name] of entries) {
+		for (const [name, data] of entries) {
+			if (data.type === SKILL_TYPES.KIT) continue
 			const short_name = name.split('/')[1] || name
 			const source_dir = skill_install_dir(base_dir, short_name)
 			const [, enabled] = await is_skill_enabled(short_name, detected_agents, is_global, project_root)

package/src/engine/installer.js

@@ -10,7 +10,7 @@ const { read_lock, get_locked_skill } = require('../lock/reader')
 const { write_lock, update_lock_skills } = require('../lock/writer')
 const { skills_dir, tmp_dir, skill_install_dir, lock_root } = require('../config/paths')
 const { ensure_dir, remove_dir, file_exists, read_json } = require('../utils/fs')
-const { SKILL_JSON } = require('../constants')
+const { SKILL_JSON, SKILL_TYPES } = require('../constants')
 const { resolve_agents, link_to_agents, verify_and_repair_symlinks } = require('../agents')
 const { is_skill_enabled } = require('../agents/status')
 const { create_spinner } = require('../ui/spinner')
@@ -42,8 +42,8 @@ const install = (skill, options = {}) => catch_errors('Install failed', async ()
 			if (exists) {
 				const [, valid] = locked.integrity ? await verify_integrity(install_dir, locked.integrity) : [null, true]
 				if (valid !== false) {
-					// Verify and repair symlinks even for already-installed skills
-					if (agents.length > 0) {
+					// Verify and repair symlinks even for already-installed skills (skip kits — not agent-invocable)
+					if (agents.length > 0 && locked.type !== SKILL_TYPES.KIT) {
 						const name = skill.split('/')[1]
 						const [, enabled] = await is_skill_enabled(name, agents, is_global, project_root)
 						if (enabled) {
@@ -103,10 +103,12 @@ const install = (skill, options = {}) => catch_errors('Install failed', async ()
 	}
 
 	if (packages_to_install.length === 0) {
-		// Verify and repair symlinks for all skipped packages
+		// Verify and repair symlinks for all skipped packages (skip kits — not agent-invocable)
 		if (agents.length > 0) {
 			const skills_to_verify = []
 			for (const pkg of packages) {
+				const locked = get_locked_skill(lock_data, pkg.skill)
+				if (locked?.type === SKILL_TYPES.KIT) continue
 				const name = pkg.skill.split('/')[1]
 				const [, enabled] = await is_skill_enabled(name, agents, is_global, project_root)
 				if (enabled) {
@@ -190,10 +192,22 @@ const install = (skill, options = {}) => catch_errors('Install failed', async ()
 			await fs.promises.rename(tmp_path, final_dir)
 		}
 
+		// Read type from each installed package's skill.json (used for linking + lock writing)
+		const pkg_types = {}
+		for (const { pkg } of downloaded) {
+			const name = pkg.skill.split('/')[1]
+			const final_dir = skill_install_dir(base_dir, name)
+			const pkg_json_path = path.join(final_dir, SKILL_JSON)
+			const [, pkg_manifest] = await read_json(pkg_json_path)
+			if (pkg_manifest?.type) pkg_types[pkg.skill] = pkg_manifest.type
+		}
+
 		// Link to detected agents (non-fatal — warnings only)
-		// Skip linking for skills that were disabled before this install/update
+		// Skip linking for kits (not invocable by agents) and skills that were disabled before this install/update
 		if (agents.length > 0) {
-			const to_link = downloaded.filter(({ pkg }) => !disabled_skills.has(pkg.skill.split('/')[1]))
+			const to_link = downloaded.filter(({ pkg }) =>
+				pkg_types[pkg.skill] !== SKILL_TYPES.KIT && !disabled_skills.has(pkg.skill.split('/')[1])
+			)
 			if (to_link.length > 0) {
 				spinner.update(`Linking to ${agents.length} agent(s)...`)
 				for (const { pkg } of to_link) {
@@ -220,12 +234,7 @@ const install = (skill, options = {}) => catch_errors('Install failed', async ()
 			const final_dir = skill_install_dir(base_dir, name)
 			const [, integrity] = await hash_directory(final_dir)
 
-			// Read type from the installed package's skill.json
-			let pkg_type
-			const pkg_json_path = path.join(final_dir, SKILL_JSON)
-			const [, pkg_manifest] = await read_json(pkg_json_path)
-			if (pkg_manifest?.type) pkg_type = pkg_manifest.type
-
+			const pkg_type = pkg_types[pkg.skill]
 			updates[pkg.skill] = {
 				version: pkg.version,
 				ref: pkg.ref,