happyskills 0.4.2 → 0.4.3

package/CHANGELOG.md

@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.4.3] - 2026-03-05
+
+### Fixed
+- Fix `find_project_root()` still treating the home directory as a project root via `~/skills-lock.json`; the home directory is now fully excluded from all project root detection checks
+
 ## [0.4.2] - 2026-03-05
 
 ### Fixed

package/package.json

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

package/src/config/paths.js

@@ -37,20 +37,21 @@ const skill_install_dir = (base_skills_dir, name) => path.join(base_skills_dir,
 const find_project_root = (start_dir = process.cwd()) => {
 	let dir = path.resolve(start_dir)
 	while (true) {
-		const skills = path.join(dir, '.claude', 'skills')
-		const lock = path.join(dir, 'skills-lock.json')
-		// Skip if this would match the global skills dir (~/.claude/skills)
-		// to avoid treating the home directory as a project root
-		if (skills !== global_skills_dir()) {
+		// Never treat the home directory as a project root —
+		// ~/.claude/skills is the global skills dir, not a project,
+		// and ~/skills-lock.json is an artifact of old buggy behaviour.
+		if (dir !== home_dir) {
+			const skills = path.join(dir, '.claude', 'skills')
+			const lock = path.join(dir, 'skills-lock.json')
 			try {
 				fs.statSync(skills)
 				return dir
 			} catch {}
+			try {
+				fs.statSync(lock)
+				return dir
+			} catch {}
 		}
-		try {
-			fs.statSync(lock)
-			return dir
-		} catch {}
 		const parent = path.dirname(dir)
 		if (parent === dir) break
 		dir = parent

package/src/config/paths.test.js

@@ -157,5 +157,20 @@ describe('paths', () => {
 				fs.rmSync(tmp, { recursive: true })
 			}
 		})
+
+		it('does not treat ~/skills-lock.json as a project marker', () => {
+			// The home directory itself must never be returned as project root,
+			// even if a stale skills-lock.json exists there from old buggy behaviour.
+			// We verify this by running from a temp dir that has no project markers
+			// of its own — the result must be the temp dir, not the home dir.
+			const tmp = fs.realpathSync(fs.mkdtempSync(path.join(os.tmpdir(), 'hs-test-')))
+			try {
+				const result = paths.find_project_root(tmp)
+				assert.notStrictEqual(result, os.homedir())
+				assert.strictEqual(result, tmp)
+			} finally {
+				fs.rmSync(tmp, { recursive: true })
+			}
+		})
 	})
 })