happyskills 0.11.0 → 0.12.0

package/CHANGELOG.md

@@ -7,6 +7,21 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.12.0] - 2026-03-12
+
+### Added
+- Add `_kit-` naming convention for kits — `init --kit` and `fork` auto-prepend the `_kit-` prefix to the name if not already present
+- Add `KIT_PREFIX` constant (`_kit-`) exported from `constants.js`
+- Add `kit_prefix` validation rule — errors when a kit name is missing the `_kit-` prefix, or when a non-kit name uses the reserved prefix
+- Add `KIT_NAME_PATTERN` regex (`/^_kit-[a-z0-9][a-z0-9-]*$/`) for validating kit names
+
+## [0.11.1] - 2026-03-12
+
+### Fixed
+- Fix `init` not setting `type: "skill"` in skill.json for regular skills — only kits were getting the `type` field; now all scaffolded skills explicitly set `type`
+- Fix `convert` not setting `type` in skill.json — converted skills now preserve existing `type` or default to `"skill"`
+- Fix `fork` not setting `type` in skill.json — forked skills now preserve the original skill's `type` or default to `"skill"`
+
 ## [0.11.0] - 2026-03-11
 
 ### Added

package/package.json

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

package/src/commands/convert.js

@@ -16,7 +16,7 @@ const { file_exists } = require('../utils/fs')
 const { create_spinner } = require('../ui/spinner')
 const { print_help, print_success, print_error, print_info, print_warn, print_label, print_json } = require('../ui/output')
 const { exit_with_error, UsageError, CliError } = require('../utils/errors')
-const { EXIT_CODES, SKILL_MD } = require('../constants')
+const { EXIT_CODES, SKILL_MD, SKILL_TYPES } = require('../constants')
 
 const HELP_TEXT = `Usage: happyskills convert <skill-name> [options]
 
@@ -162,6 +162,7 @@ const run = (args) => catch_errors('Convert failed', async () => {
 		...(existing_manifest || {}),
 		name: skill_name,
 		version: merged_version,
+		type: (existing_manifest?.type) || SKILL_TYPES.SKILL,
 		description: merged_description,
 		keywords: merged_keywords
 	}

package/src/commands/fork.js

@@ -8,7 +8,8 @@ const { write_manifest } = require('../manifest/writer')
 const { create_spinner } = require('../ui/spinner')
 const { print_help, print_success, print_hint, print_json, code } = require('../ui/output')
 const { exit_with_error, UsageError } = require('../utils/errors')
-const { EXIT_CODES } = require('../constants')
+const { EXIT_CODES, SKILL_TYPES, KIT_PREFIX } = require('../constants')
+const { read_manifest } = require('../manifest/reader')
 
 const HELP_TEXT = `Usage: happyskills fork <owner/skill> [options]
 
@@ -79,9 +80,13 @@ const run = (args) => catch_errors('Fork failed', async () => {
 	const [ext_err] = await extract(clone_data, dest)
 	if (ext_err) { spinner.fail('Extract failed'); throw e('Extract failed', ext_err) }
 
+	const [, original_manifest] = await read_manifest(dest)
+	const is_kit = (original_manifest?.type) === SKILL_TYPES.KIT
+	const forked_name = is_kit && !name.startsWith(KIT_PREFIX) ? `${KIT_PREFIX}${name}` : name
 	const forked_manifest = {
-		name,
+		name: forked_name,
 		version: '0.1.0',
+		type: is_kit ? SKILL_TYPES.KIT : SKILL_TYPES.SKILL,
 		description: '',
 		keywords: [],
 		forked_from: {

package/src/commands/init.js

@@ -4,7 +4,7 @@ const { write_manifest } = require('../manifest/writer')
 const { write_file, file_exists, ensure_dir } = require('../utils/fs')
 const { print_success, print_error, print_help, print_hint, print_json, code } = require('../ui/output')
 const { exit_with_error, CliError, UsageError } = require('../utils/errors')
-const { SKILL_JSON, SKILL_MD, EXIT_CODES, SKILL_TYPES } = require('../constants')
+const { SKILL_JSON, SKILL_MD, EXIT_CODES, SKILL_TYPES, KIT_PREFIX } = require('../constants')
 const { find_project_root, skills_dir } = require('../config/paths')
 
 const HELP_TEXT = `Usage: happyskills init <name> [options]
@@ -58,7 +58,7 @@ const create_manifest = (name, is_kit = false) => ({
 	version: '0.1.0',
 	description: '',
 	keywords: [],
-	...(is_kit ? { type: SKILL_TYPES.KIT } : {}),
+	type: is_kit ? SKILL_TYPES.KIT : SKILL_TYPES.SKILL,
 	dependencies: {}
 })
 
@@ -75,9 +75,10 @@ const run = (args) => catch_errors('Init failed', async () => {
 
 	const is_global = args.flags.global || false
 	const is_kit = args.flags.kit || false
+	const final_name = is_kit && !name.startsWith(KIT_PREFIX) ? `${KIT_PREFIX}${name}` : name
 	const project_root = find_project_root()
 	const base_skills_dir = skills_dir(is_global, project_root)
-	const dir = path.join(base_skills_dir, name)
+	const dir = path.join(base_skills_dir, final_name)
 
 	const [, json_exists] = await file_exists(path.join(dir, SKILL_JSON))
 	if (json_exists) {
@@ -87,7 +88,7 @@ const run = (args) => catch_errors('Init failed', async () => {
 	const [dir_err] = await ensure_dir(dir)
 	if (dir_err) throw e('Failed to create skill directory', dir_err)
 
-	const manifest = create_manifest(name, is_kit)
+	const manifest = create_manifest(final_name, is_kit)
 	const [write_err] = await write_manifest(dir, manifest)
 	if (write_err) throw e('Failed to write manifest', write_err)
 
@@ -95,7 +96,7 @@ const run = (args) => catch_errors('Init failed', async () => {
 
 	const [, md_exists] = await file_exists(path.join(dir, SKILL_MD))
 	if (!md_exists) {
-		const md_content = is_kit ? create_kit_md(name) : create_skill_md(name)
+		const md_content = is_kit ? create_kit_md(final_name) : create_skill_md(final_name)
 		const [md_err] = await write_file(path.join(dir, SKILL_MD), md_content)
 		if (md_err) throw e('Failed to write SKILL.md', md_err)
 		files_created.push(SKILL_MD)
@@ -104,11 +105,11 @@ const run = (args) => catch_errors('Init failed', async () => {
 	const label = is_kit ? 'kit' : 'skill'
 
 	if (args.flags.json) {
-		print_json({ data: { name, type: is_kit ? SKILL_TYPES.KIT : SKILL_TYPES.SKILL, files_created, directory: dir } })
+		print_json({ data: { name: final_name, type: is_kit ? SKILL_TYPES.KIT : SKILL_TYPES.SKILL, files_created, directory: dir } })
 		return
 	}
 
-	print_success(`Initialized ${label} "${name}" at ${dir}`)
+	print_success(`Initialized ${label} "${final_name}" at ${dir}`)
 	console.log(`  ${SKILL_JSON}  — manifest`)
 	console.log(`  ${SKILL_MD}    — ${is_kit ? 'kit description' : 'instructions'}`)
 	console.log()

package/src/constants.js

@@ -13,6 +13,7 @@ const EXIT_CODES = {
 }
 
 const SKILL_TYPES = { SKILL: 'skill', KIT: 'kit' }
+const KIT_PREFIX = '_kit-'
 const VALID_SKILL_TYPES = ['skill', 'kit']
 
 const LOCK_VERSION = 1
@@ -63,6 +64,7 @@ module.exports = {
 	CLI_VERSION,
 	EXIT_CODES,
 	SKILL_TYPES,
+	KIT_PREFIX,
 	VALID_SKILL_TYPES,
 	LOCK_VERSION,
 	SKILL_JSON,

package/src/validation/skill_json_rules.js

@@ -2,9 +2,10 @@ const path = require('path')
 const { error: { catch_errors } } = require('puffy-core')
 const { file_exists, read_file } = require('../utils/fs')
 const { valid } = require('../utils/semver')
-const { SKILL_JSON, VALID_SKILL_TYPES, SKILL_TYPES } = require('../constants')
+const { SKILL_JSON, VALID_SKILL_TYPES, SKILL_TYPES, KIT_PREFIX } = require('../constants')
 
 const NAME_PATTERN = /^[a-z0-9][a-z0-9_-]*$/
+const KIT_NAME_PATTERN = /^_kit-[a-z0-9][a-z0-9-]*$/
 const CANONICAL_SLUGS = ['deployment', 'database', 'security', 'ai', 'api', 'monitoring', 'testing', 'devops', 'cloud', 'analytics']
 const REQUIRED_PLATFORMS = ['darwin', 'linux', 'win32']
 
@@ -15,6 +16,7 @@ const result = (field, rule, severity, message, value) => ({
 const validate_name = (manifest) => {
 	const results = []
 	const name = manifest.name
+	const is_kit = manifest.type === SKILL_TYPES.KIT
 
 	if (!name) {
 		results.push(result('name', 'present', 'error', 'skill.json must include a "name" field'))
@@ -23,10 +25,26 @@ const validate_name = (manifest) => {
 
 	results.push(result('name', 'present', 'pass', `name: "${name}"`, name))
 
-	if (!NAME_PATTERN.test(name)) {
-		results.push(result('name', 'format', 'error', 'Name must match /^[a-z0-9][a-z0-9_-]*$/', name))
+	if (is_kit) {
+		if (!KIT_NAME_PATTERN.test(name)) {
+			results.push(result('name', 'format', 'error', `Kit name must match ${KIT_NAME_PATTERN}`, name))
+		} else {
+			results.push(result('name', 'format', 'pass', 'Name format valid', name))
+		}
+		if (!name.startsWith(KIT_PREFIX)) {
+			results.push(result('name', 'kit_prefix', 'error', `Kit name "${name}" must start with "${KIT_PREFIX}". Expected: "${KIT_PREFIX}${name}"`, name))
+		} else {
+			results.push(result('name', 'kit_prefix', 'pass', `Name starts with "${KIT_PREFIX}"`, name))
+		}
 	} else {
-		results.push(result('name', 'format', 'pass', 'Name format valid', name))
+		if (!NAME_PATTERN.test(name)) {
+			results.push(result('name', 'format', 'error', 'Name must match /^[a-z0-9][a-z0-9_-]*$/', name))
+		} else {
+			results.push(result('name', 'format', 'pass', 'Name format valid', name))
+		}
+		if (name.startsWith(KIT_PREFIX)) {
+			results.push(result('name', 'kit_prefix', 'error', `Name "${name}" uses the reserved "${KIT_PREFIX}" prefix but type is "${manifest.type || 'skill'}". The "${KIT_PREFIX}" prefix is reserved for kits (type: "kit")`, name))
+		}
 	}
 
 	return results

package/src/validation/skill_json_rules.test.js

@@ -205,7 +205,7 @@ describe('validate_skill_json — type', () => {
 	})
 
 	it('passes for type "kit"', async () => {
-		write_json(tmp, { name: 'my-kit', version: '1.0.0', type: 'kit' })
+		write_json(tmp, { name: '_kit-my-bundle', version: '1.0.0', type: 'kit' })
 		const [err, data] = await validate_skill_json(tmp)
 		assert.ifError(err)
 		const check = data.results.find(r => r.field === 'type' && r.rule === 'valid_value')
@@ -224,7 +224,7 @@ describe('validate_skill_json — type', () => {
 
 describe('validate_skill_json — kit dependencies', () => {
 	it('warns when kit has no dependencies', async () => {
-		write_json(tmp, { name: 'my-kit', version: '1.0.0', type: 'kit' })
+		write_json(tmp, { name: '_kit-my-bundle', version: '1.0.0', type: 'kit' })
 		const [err, data] = await validate_skill_json(tmp)
 		assert.ifError(err)
 		const check = data.results.find(r => r.rule === 'kit_has_deps')
@@ -233,7 +233,7 @@ describe('validate_skill_json — kit dependencies', () => {
 	})
 
 	it('warns when kit has empty dependencies object', async () => {
-		write_json(tmp, { name: 'my-kit', version: '1.0.0', type: 'kit', dependencies: {} })
+		write_json(tmp, { name: '_kit-my-bundle', version: '1.0.0', type: 'kit', dependencies: {} })
 		const [err, data] = await validate_skill_json(tmp)
 		assert.ifError(err)
 		const check = data.results.find(r => r.rule === 'kit_has_deps')
@@ -241,7 +241,7 @@ describe('validate_skill_json — kit dependencies', () => {
 	})
 
 	it('does not warn when kit has dependencies', async () => {
-		write_json(tmp, { name: 'my-kit', version: '1.0.0', type: 'kit', dependencies: { 'acme/deploy': '^1.0.0' } })
+		write_json(tmp, { name: '_kit-my-bundle', version: '1.0.0', type: 'kit', dependencies: { 'acme/deploy': '^1.0.0' } })
 		const [err, data] = await validate_skill_json(tmp)
 		assert.ifError(err)
 		assert.ok(!data.results.some(r => r.rule === 'kit_has_deps'))
@@ -255,6 +255,61 @@ describe('validate_skill_json — kit dependencies', () => {
 	})
 })
 
+describe('validate_skill_json — kit prefix', () => {
+	it('passes for kit with _kit- prefix', async () => {
+		write_json(tmp, { name: '_kit-my-bundle', version: '1.0.0', type: 'kit' })
+		const [err, data] = await validate_skill_json(tmp)
+		assert.ifError(err)
+		const format_check = data.results.find(r => r.field === 'name' && r.rule === 'format')
+		assert.strictEqual(format_check.severity, 'pass')
+		const prefix_check = data.results.find(r => r.field === 'name' && r.rule === 'kit_prefix')
+		assert.strictEqual(prefix_check.severity, 'pass')
+	})
+
+	it('errors for kit without _kit- prefix', async () => {
+		write_json(tmp, { name: 'my-bundle', version: '1.0.0', type: 'kit' })
+		const [err, data] = await validate_skill_json(tmp)
+		assert.ifError(err)
+		const check = data.results.find(r => r.field === 'name' && r.rule === 'kit_prefix')
+		assert.strictEqual(check.severity, 'error')
+		assert.ok(check.message.includes('must start with'))
+		assert.ok(check.message.includes('_kit-my-bundle'))
+	})
+
+	it('errors for non-kit with _kit- prefix', async () => {
+		write_json(tmp, { name: '_kit-foo', version: '1.0.0', type: 'skill' })
+		const [err, data] = await validate_skill_json(tmp)
+		assert.ifError(err)
+		const check = data.results.find(r => r.field === 'name' && r.rule === 'kit_prefix')
+		assert.strictEqual(check.severity, 'error')
+		assert.ok(check.message.includes('reserved'))
+		assert.ok(check.message.includes('type is "skill"'))
+	})
+
+	it('errors for _kit- alone as kit name', async () => {
+		write_json(tmp, { name: '_kit-', version: '1.0.0', type: 'kit' })
+		const [err, data] = await validate_skill_json(tmp)
+		assert.ifError(err)
+		const check = data.results.find(r => r.field === 'name' && r.rule === 'format')
+		assert.strictEqual(check.severity, 'error')
+	})
+
+	it('errors for kit name with uppercase after prefix', async () => {
+		write_json(tmp, { name: '_kit-A', version: '1.0.0', type: 'kit' })
+		const [err, data] = await validate_skill_json(tmp)
+		assert.ifError(err)
+		const check = data.results.find(r => r.field === 'name' && r.rule === 'format')
+		assert.strictEqual(check.severity, 'error')
+	})
+
+	it('does not produce kit_prefix error for non-kit without _kit- prefix', async () => {
+		write_json(tmp, { name: 'my-skill', version: '1.0.0', type: 'skill' })
+		const [err, data] = await validate_skill_json(tmp)
+		assert.ifError(err)
+		assert.ok(!data.results.some(r => r.rule === 'kit_prefix'))
+	})
+})
+
 describe('validate_skill_json — systemDependencies', () => {
 	it('errors when systemDependencies is an array', async () => {
 		write_json(tmp, { name: 'my-skill', version: '1.0.0', systemDependencies: [] })