happyskills 0.4.3 → 0.5.0

package/CHANGELOG.md

@@ -7,6 +7,23 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.5.0] - 2026-03-05
+
+### Added
+- Add `--mine` flag to `search` command to browse skills across all user's workspaces
+- Add `--personal` flag to `search` command to browse skills in the user's personal workspace only
+- Add `-w` / `--workspace <slug>` flag to `search` command to search within specific workspace(s)
+- Add `--tags <tags>` flag to `search` command to filter results by tags
+- Add `scope` and `visibility` fields to `search` JSON output
+
+### Changed
+- Change `search` query argument from required to optional when `--mine`, `--personal`, or `--workspace` is provided (browse mode)
+
+## [0.4.4] - 2026-03-05
+
+### Fixed
+- Fix all commands (`install`, `list`, `setup`, `update`, `uninstall`, `check`, `bump`, `convert`) inheriting skills from a parent directory; `find_project_root()` now always uses the current working directory and never walks up the directory tree
+
 ## [0.4.3] - 2026-03-05
 
 ### Fixed

package/package.json

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

package/src/api/repos.js

@@ -2,10 +2,15 @@ const { error: { catch_errors } } = require('puffy-core')
 const client = require('./client')
 
 const search = (query, options = {}) => catch_errors('Search failed', async () => {
-	const params = new URLSearchParams({ q: query })
+	const params = new URLSearchParams()
+	if (query) params.set('q', query)
 	if (options.limit) params.set('limit', options.limit)
 	if (options.offset) params.set('offset', options.offset)
-	const [errors, data] = await client.get(`/repos/search?${params}`, { auth: false })
+	if (options.scope) params.set('scope', options.scope)
+	if (options.workspace) params.set('workspace', options.workspace)
+	if (options.tags) params.set('tags', options.tags)
+	const needs_auth = (options.scope && options.scope !== 'public') || options.workspace
+	const [errors, data] = await client.get(`/repos/search?${params}`, { auth: needs_auth || false })
 	if (errors) throw errors[errors.length - 1]
 	return data
 })

package/src/commands/search.js

@@ -1,24 +1,31 @@
 const { error: { catch_errors, wrap_errors: e } } = require('puffy-core')
 const repos_api = require('../api/repos')
 const { print_help, print_table, print_json, print_info } = require('../ui/output')
-const { exit_with_error, UsageError } = require('../utils/errors')
+const { exit_with_error, UsageError, AuthError } = require('../utils/errors')
 const { EXIT_CODES } = require('../constants')
+const { load_token } = require('../auth/token_store')
 
-const HELP_TEXT = `Usage: happyskills search <query> [options]
+const HELP_TEXT = `Usage: happyskills search [query] [options]
 
 Search the skill registry.
 
 Arguments:
-  query             Search term
+  query               Search term (optional with --mine, --personal, or --workspace)
 
 Options:
-  --json            Output as JSON
+  -w, --workspace <slug>  Search within specific workspace(s) (comma-separated)
+  --mine                  Search across all your workspaces
+  --personal              Search only your personal workspace
+  --tags <tags>           Filter by tags (comma-separated)
+  --json                  Output as JSON
 
 Aliases: s
 
 Examples:
   happyskills search deploy
-  happyskills s aws --json`
+  happyskills search --mine
+  happyskills search deploy --workspace acme
+  happyskills s --personal --json`
 
 const run = (args) => catch_errors('Search failed', async () => {
 	if (args.flags._show_help) {
@@ -26,32 +33,52 @@ const run = (args) => catch_errors('Search failed', async () => {
 		return process.exit(EXIT_CODES.SUCCESS)
 	}
 
-	const query = args._.join(' ')
-	if (!query) {
-		throw new UsageError('Please provide a search query (e.g., happyskills search deploy).')
+	const query = args._.join(' ') || null
+	const { mine, personal, workspace, tags } = args.flags
+	const has_scope_flag = mine || personal || workspace
+
+	if (!query && !has_scope_flag && !tags) {
+		throw new UsageError('Please provide a search query or use --mine, --personal, or --workspace.')
+	}
+
+	const scope = mine ? 'mine' : personal ? 'personal' : undefined
+
+	if (has_scope_flag) {
+		const [, token_data] = await load_token()
+		if (!token_data) {
+			throw new AuthError('Authentication required. Run `happyskills login` first.')
+		}
 	}
 
-	const [errors, results] = await repos_api.search(query)
+	const options = {}
+	if (scope) options.scope = scope
+	if (workspace) options.workspace = workspace
+	if (tags) options.tags = tags
+
+	const [errors, results] = await repos_api.search(query, options)
 	if (errors) throw e('Search failed', errors)
 
 	const items = Array.isArray(results) ? results : (results?.repos || results?.items || [])
+	const effective_scope = scope || (has_scope_flag ? undefined : undefined)
 
 	if (items.length === 0) {
 		if (args.flags.json) {
-			print_json({ data: { query, results: [], count: 0 } })
+			print_json({ data: { query, scope: effective_scope || 'all', results: [], count: 0 } })
 			return
 		}
-		print_info(`No skills found for "${query}".`)
+		const context = query ? ` for "${query}"` : ''
+		print_info(`No skills found${context}.`)
 		return
 	}
 
 	if (args.flags.json) {
-		const results = items.map(item => ({
+		const mapped = items.map(item => ({
 			skill: `${item.owner || item.workspace_slug}/${item.name}`,
 			description: item.description || '',
-			version: item.latest_version || item.version || '-'
+			version: item.latest_version || item.version || '-',
+			visibility: item.visibility || 'public'
 		}))
-		print_json({ data: { query, results, count: results.length } })
+		print_json({ data: { query, scope: effective_scope || 'all', results: mapped, count: mapped.length } })
 		return
 	}
 

package/src/config/paths.js

@@ -34,30 +34,7 @@ const lock_file_path = (project_root = process.cwd()) => path.join(project_root,
 
 const skill_install_dir = (base_skills_dir, name) => path.join(base_skills_dir, name)
 
-const find_project_root = (start_dir = process.cwd()) => {
-	let dir = path.resolve(start_dir)
-	while (true) {
-		// 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 {}
-		}
-		const parent = path.dirname(dir)
-		if (parent === dir) break
-		dir = parent
-	}
-	return start_dir
-}
+const find_project_root = (start_dir = process.cwd()) => path.resolve(start_dir)
 
 module.exports = {
 	home_dir,

package/src/config/paths.test.js

@@ -100,77 +100,47 @@ describe('paths', () => {
 	})
 
 	describe('find_project_root', () => {
-		it('finds project root by .claude/skills dir', () => {
+		it('returns the given directory as-is', () => {
 			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', () => {
+		it('does not walk up to a parent with .claude/skills', () => {
 			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')
+				const sub = path.join(tmp, 'sub')
 				fs.mkdirSync(sub, { recursive: true })
-				assert.strictEqual(paths.find_project_root(sub), tmp)
+				// should return sub, not tmp
+				assert.strictEqual(paths.find_project_root(sub), sub)
 			} finally {
 				fs.rmSync(tmp, { recursive: true })
 			}
 		})
 
-		it('falls back to start_dir when no project markers found', () => {
+		it('does not walk up to a parent with skills-lock.json', () => {
 			const tmp = fs.realpathSync(fs.mkdtempSync(path.join(os.tmpdir(), 'hs-test-')))
 			try {
+				fs.writeFileSync(path.join(tmp, 'skills-lock.json'), '{}')
 				const sub = path.join(tmp, 'sub')
 				fs.mkdirSync(sub, { recursive: true })
+				// should return sub, not tmp
 				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 })
-			}
+		it('returns home dir only when explicitly called with it', () => {
+			assert.strictEqual(paths.find_project_root(os.homedir()), os.homedir())
 		})
 
-		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 })
-			}
+		it('defaults to cwd when no argument given', () => {
+			assert.strictEqual(paths.find_project_root(), path.resolve(process.cwd()))
 		})
 	})
 })

package/src/index.js

@@ -62,6 +62,7 @@ const parse_args = (argv) => {
 const SHORT_FLAGS = {
 	g: 'global',
 	y: 'yes',
+	w: 'workspace',
 }
 
 const normalize_flags = (flags) => {