happyskills 1.2.0 → 1.2.2

package/CHANGELOG.md

@@ -7,6 +7,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [1.2.2] - 2026-06-02
+
+### Changed
+
+- `validate` no longer warns when `skill.json` `keywords` is missing or omits a canonical slug. The `keywords` field is deprecated — it is not used for search, ranking, or discovery — so the warning and the canonical-slug check were removed. The field is still accepted in the manifest.
+
+## [1.2.1] - 2026-06-02
+
+### Fixed
+
+- Emit tailored `next_step.instructions` for the three actions added in 1.2.0 (`retry_or_abandon`, `resolve_unknown_drift`, `review_publish_error`). They were registered in the enum but missing from `DEFAULT_INSTRUCTIONS`, so the envelope fell back to the generic "Follow the protocol step indicated by `action`." placeholder. Each now carries actionable guidance.
+
 ## [1.2.0] - 2026-06-02
 
 ### Added

package/package.json

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

package/src/ui/envelope.js

@@ -34,6 +34,9 @@ const DEFAULT_INSTRUCTIONS = {
 	retry:              'Transient failure. Retry shortly.',
 	reconcile_first:    'Drift must be resolved before this operation. Run reconcile, follow its next_step, then retry.',
 	pull_rebase_first:  'The local skill has diverged from the registry. Pull with --rebase, resolve any rejected patches, then retry.',
+	retry_or_abandon:   'The rebase step failed before any change landed. Retry the pull, or abandon and restore the snapshot.',
+	resolve_unknown_drift: 'Unrecognized drift subtype — surface the details to the principal for a manual decision.',
+	review_publish_error: 'The publish step failed. Inspect the wrapped publish error in context, resolve it, then retry.',
 	fix_validation_errors: 'Fix the listed validation errors and re-run.',
 	provide_changelog:  'CHANGELOG.md is missing the target version entry. Add it and re-run.',
 	self_update:        'Upgrade the happyskills CLI and re-run.',

package/src/validation/skill_json_rules.js

@@ -6,7 +6,6 @@ const { SKILL_JSON, VALID_SKILL_TYPES, SKILL_TYPES, KIT_PREFIX } = require('../c
 
 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']
 
 const result = (field, rule, severity, message, value) => ({
@@ -90,27 +89,6 @@ const validate_description = (manifest) => {
 	return results
 }
 
-const validate_keywords = (manifest) => {
-	const results = []
-	const kw = manifest.keywords
-
-	if (!kw || !Array.isArray(kw)) {
-		results.push(result('keywords', 'recommended_field', 'warning', 'Keywords array is recommended — include at least one canonical slug'))
-		return results
-	}
-
-	results.push(result('keywords', 'recommended_field', 'pass', `keywords: ${kw.length} entries`))
-
-	const has_slug = kw.some(k => CANONICAL_SLUGS.includes(k))
-	if (!has_slug) {
-		results.push(result('keywords', 'canonical_slug', 'warning', `Keywords should include at least one canonical slug: ${CANONICAL_SLUGS.join(', ')}`))
-	} else {
-		results.push(result('keywords', 'canonical_slug', 'pass', 'Contains canonical slug'))
-	}
-
-	return results
-}
-
 const validate_type = (manifest) => {
 	const results = []
 	const type = manifest.type
@@ -229,7 +207,6 @@ const validate_skill_json = (skill_dir) => catch_errors('Failed to validate skil
 	results.push(...validate_version(manifest))
 	results.push(...validate_type(manifest))
 	results.push(...validate_description(manifest))
-	results.push(...validate_keywords(manifest))
 	results.push(...validate_dependencies(manifest))
 	results.push(...validate_system_dependencies(manifest))
 

package/src/validation/skill_json_rules.test.js

@@ -121,32 +121,6 @@ describe('validate_skill_json — description', () => {
 	})
 })
 
-describe('validate_skill_json — keywords', () => {
-	it('warns when keywords are missing', async () => {
-		write_json(tmp, { name: 'my-skill', version: '1.0.0' })
-		const [err, data] = await validate_skill_json(tmp)
-		assert.ifError(err)
-		const check = data.results.find(r => r.field === 'keywords' && r.rule === 'recommended_field')
-		assert.strictEqual(check.severity, 'warning')
-	})
-
-	it('warns when no canonical slug is present', async () => {
-		write_json(tmp, { name: 'my-skill', version: '1.0.0', keywords: ['custom'] })
-		const [err, data] = await validate_skill_json(tmp)
-		assert.ifError(err)
-		const check = data.results.find(r => r.rule === 'canonical_slug')
-		assert.strictEqual(check.severity, 'warning')
-	})
-
-	it('passes when a canonical slug is present', async () => {
-		write_json(tmp, { name: 'my-skill', version: '1.0.0', keywords: ['deployment', 'custom'] })
-		const [err, data] = await validate_skill_json(tmp)
-		assert.ifError(err)
-		const check = data.results.find(r => r.rule === 'canonical_slug')
-		assert.strictEqual(check.severity, 'pass')
-	})
-})
-
 describe('validate_skill_json — dependencies', () => {
 	it('errors when dependencies is an array', async () => {
 		write_json(tmp, { name: 'my-skill', version: '1.0.0', dependencies: ['acme/deploy'] })