happyskills 0.7.0 → 0.7.1

package/CHANGELOG.md

@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.7.1] - 2026-03-06
+
+### Fixed
+- Fix `uninstall` incorrectly orphaning all user-installed skills — `find_orphans` now treats `__root__` as a protected requester, so only skills that were installed solely as transitive dependencies are pruned
+
 ## [0.7.0] - 2026-03-06
 
 ### Added

package/package.json

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

package/src/engine/uninstaller.js

@@ -9,11 +9,8 @@ const find_orphans = (skills, removed_skill) => {
 	const orphans = []
 	for (const [name, data] of Object.entries(skills)) {
 		if (name === removed_skill) continue
-		if (!data.requested_by || data.requested_by.length === 0) {
-			orphans.push(name)
-			continue
-		}
-		const remaining = data.requested_by.filter(r => r !== removed_skill && skills[r])
+		const requested_by = data.requested_by || []
+		const remaining = requested_by.filter(r => r === '__root__' || (r !== removed_skill && skills[r]))
 		if (remaining.length === 0) {
 			orphans.push(name)
 		}

package/src/engine/uninstaller.test.js

@@ -87,12 +87,12 @@ describe('find_orphans', () => {
 		assert.ok(result.includes('acme/auth'))
 	})
 
-	it('treats __root__ like any other requester (not in skills map)', () => {
+	it('protects skills installed by the user (requested_by includes __root__)', () => {
 		const skills = {
 			'acme/deploy': { requested_by: ['__root__'] }
 		}
-		// __root__ is not a key in skills, so deploy is considered orphaned
+		// __root__ marks a user-installed (root) package — should never be orphaned
 		const result = find_orphans(skills, 'acme/other')
-		assert.ok(result.includes('acme/deploy'))
+		assert.ok(!result.includes('acme/deploy'))
 	})
 })