happyskills 0.52.0 → 0.52.1

package/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.52.1] - 2026-05-25
+
+### Fixed
+
+- **`release` now validates kits against the kit contract (README.md) instead of the skill contract** (`cli/src/commands/release.js`). `run_validation()` was calling `validate_skill_md(dir, basename, null)` — hard-coding the third argument to `null` instead of reading the `type` from `skill.json`. For kit repos this meant the skill branch fired even though the kit had no `SKILL.md`, so `release` failed with `"SKILL.md not found"` immediately after CLI v0.52.0 moved kits to `README.md`. Discovered while migrating `nicolasdao/_kit-doc-essentials` from the old to the new format. Now reads `manifest.type` from `skill.json` first and passes it through to both `validate_skill_md` and `validate_cross`. The standalone `validate` command was unaffected — it was already wiring `skill_type` through correctly.
+
 ## [0.52.0] - 2026-05-25
 
 ### Added

package/package.json

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

package/src/commands/release.js

@@ -122,11 +122,14 @@ const compute_bump = (base_version, bump) => {
 }
 
 const run_validation = (dir, skill_name) => catch_errors('Validation failed', async () => {
-	const [md_err, md_data] = await validate_skill_md(dir, path.basename(dir), null)
-	if (md_err) throw md_err
+	// Read skill.json first so we know whether to validate against the skill
+	// (SKILL.md + frontmatter) or kit (README.md, no frontmatter) contract.
 	const [json_err, json_data] = await validate_skill_json(dir)
 	if (json_err) throw json_err
-	const [cross_err, cross_results] = await validate_cross(dir, md_data.frontmatter, json_data.manifest, md_data.content, json_data.manifest?.type)
+	const skill_type = json_data.manifest?.type
+	const [md_err, md_data] = await validate_skill_md(dir, path.basename(dir), skill_type)
+	if (md_err) throw md_err
+	const [cross_err, cross_results] = await validate_cross(dir, md_data.frontmatter, json_data.manifest, md_data.content, skill_type)
 	if (cross_err) throw cross_err
 	const [marker_err, marker_results] = await validate_no_conflict_markers(dir)
 	if (marker_err) throw marker_err