happyskills 0.3.1 → 0.4.2

package/CHANGELOG.md

@@ -7,6 +7,24 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.4.2] - 2026-03-05
+
+### Fixed
+- Fix `list`, `install`, `uninstall`, `update`, `setup`, and `convert` incorrectly treating the home directory as a project root when `~/.claude/skills/` exists; `find_project_root()` now excludes the global skills directory from its upward search
+
+## [0.4.1] - 2026-03-05
+
+### Fixed
+- Fix `setup` command still installing globally on npm-published v0.4.0 (re-release of the project-level default and `-g` flag)
+
+## [0.4.0] - 2026-03-05
+
+### Added
+- Add `-g` / `--global` flag to `setup` command to opt into installing `happyskillsai/happyskills-cli` globally (`~/.claude/skills/`)
+
+### Changed
+- Change `setup` default install scope from global to project-level (`.claude/skills/`); pass `-g` to restore the previous global behaviour
+
 ## [0.3.1] - 2026-03-04
 
 ### Fixed

package/package.json

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

package/src/commands/setup.js

@@ -7,18 +7,20 @@ const { EXIT_CODES } = require('../constants')
 
 const SKILL_NAME = 'happyskillsai/happyskills-cli'
 
-const HELP_TEXT = `Usage: happyskills setup
+const HELP_TEXT = `Usage: happyskills setup [options]
 
-Install (or update) the official HappySkills CLI skill globally.
+Install (or update) the official HappySkills CLI skill.
 
-Installs happyskillsai/happyskills-cli into ~/.claude/skills/ so AI agents
+Installs happyskillsai/happyskills-cli into .claude/skills/ so AI agents
 (Claude Code, etc.) can interact with HappySkills using natural language.
 
 Options:
+  -g, --global  Install globally (~/.claude/skills/) instead of project-level
   --json        Output as JSON
 
 Examples:
-  happyskills setup`
+  happyskills setup
+  happyskills setup -g`
 
 const run = (args) => catch_errors('Setup failed', async () => {
 	if (args.flags._show_help) {
@@ -27,7 +29,7 @@ const run = (args) => catch_errors('Setup failed', async () => {
 	}
 
 	const options = {
-		global: true,
+		global: args.flags.global || false,
 		yes: true,
 		version: 'latest',
 		project_root: find_project_root()

package/src/config/paths.js

@@ -39,10 +39,14 @@ const find_project_root = (start_dir = process.cwd()) => {
 	while (true) {
 		const skills = path.join(dir, '.claude', 'skills')
 		const lock = path.join(dir, 'skills-lock.json')
-		try {
-			fs.statSync(skills)
-			return dir
-		} catch {}
+		// 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()) {
+			try {
+				fs.statSync(skills)
+				return dir
+			} catch {}
+		}
 		try {
 			fs.statSync(lock)
 			return dir

package/src/config/paths.test.js

@@ -1,5 +1,6 @@
 const { describe, it, beforeEach, afterEach } = require('node:test')
 const assert = require('node:assert')
+const fs = require('fs')
 const path = require('path')
 const os = require('os')
 
@@ -97,4 +98,64 @@ describe('paths', () => {
 			assert.strictEqual(result, path.join('/base/skills', '.install.lock'))
 		})
 	})
+
+	describe('find_project_root', () => {
+		it('finds project root by .claude/skills dir', () => {
+			const tmp = fs.realpathSync(fs.mkdtempSync(path.join(os.tmpdir(), 'hs-test-')))
+			try {
+				fs.mkdirSync(path.join(tmp, '.claude', 'skills'), { recursive: true })
+				assert.strictEqual(paths.find_project_root(tmp), tmp)
+			} finally {
+				fs.rmSync(tmp, { recursive: true })
+			}
+		})
+
+		it('finds project root by skills-lock.json', () => {
+			const tmp = fs.realpathSync(fs.mkdtempSync(path.join(os.tmpdir(), 'hs-test-')))
+			try {
+				fs.writeFileSync(path.join(tmp, 'skills-lock.json'), '{}')
+				assert.strictEqual(paths.find_project_root(tmp), tmp)
+			} finally {
+				fs.rmSync(tmp, { recursive: true })
+			}
+		})
+
+		it('walks up to find project root from nested dir', () => {
+			const tmp = fs.realpathSync(fs.mkdtempSync(path.join(os.tmpdir(), 'hs-test-')))
+			try {
+				fs.mkdirSync(path.join(tmp, '.claude', 'skills'), { recursive: true })
+				const sub = path.join(tmp, 'src', 'deep')
+				fs.mkdirSync(sub, { recursive: true })
+				assert.strictEqual(paths.find_project_root(sub), tmp)
+			} finally {
+				fs.rmSync(tmp, { recursive: true })
+			}
+		})
+
+		it('falls back to start_dir when no project markers found', () => {
+			const tmp = fs.realpathSync(fs.mkdtempSync(path.join(os.tmpdir(), 'hs-test-')))
+			try {
+				const sub = path.join(tmp, 'sub')
+				fs.mkdirSync(sub, { recursive: true })
+				assert.strictEqual(paths.find_project_root(sub), sub)
+			} finally {
+				fs.rmSync(tmp, { recursive: true })
+			}
+		})
+
+		it('does not treat ~/.claude/skills as a project marker', () => {
+			const global_skills = path.join(os.homedir(), '.claude', 'skills')
+			let global_exists = false
+			try { fs.statSync(global_skills); global_exists = true } catch {}
+			if (!global_exists) return // skip if global dir not present on this machine
+
+			const tmp = fs.realpathSync(fs.mkdtempSync(path.join(os.tmpdir(), 'hs-test-')))
+			try {
+				// tmp has no project markers — should fall back to tmp, not home dir
+				assert.strictEqual(paths.find_project_root(tmp), tmp)
+			} finally {
+				fs.rmSync(tmp, { recursive: true })
+			}
+		})
+	})
 })