happyskills 0.22.0 → 0.22.1

package/CHANGELOG.md

@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.22.1] - 2026-03-30
+
+### Fixed
+- Fix sequential `detect_status` bottleneck in `status`, `refresh`, and `update` — local modification detection now runs in parallel via `Promise.all` instead of awaiting each skill sequentially
+
 ## [0.22.0] - 2026-03-30
 
 ### Added

package/package.json

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

package/src/commands/refresh.js

@@ -126,16 +126,18 @@ const run = (args) => catch_errors('Refresh failed', async () => {
 		}
 	}
 
-	// 5. Detect local modifications before updating
+	// 5. Detect local modifications for all outdated skills in parallel
 	const base_dir = skills_dir(is_global, project_root)
-	const skipped = []
-	const safe_to_update = []
-
-	for (const r of outdated) {
+	const detections = await Promise.all(outdated.map(r => {
 		const lock_entry = skills[r.skill]
 		const short_name = r.skill.split('/')[1] || r.skill
 		const dir = skill_install_dir(base_dir, short_name)
-		const [, det] = await detect_status(lock_entry, dir)
+		return detect_status(lock_entry, dir).then(([, det]) => ({ r, det }))
+	}))
+
+	const skipped = []
+	const safe_to_update = []
+	for (const { r, det } of detections) {
 		if (det?.local_modified) {
 			skipped.push({ skill: r.skill, reason: 'local_modifications', suggestion: `happyskills pull ${r.skill}` })
 		} else {

package/src/commands/status.js

@@ -69,18 +69,18 @@ const run = (args) => catch_errors('Status failed', async () => {
 
 	const base_dir = skills_dir(is_global, project_root)
 
-	// Detect local modifications for each skill
-	const results = []
-	for (const [name, data] of entries) {
-		if (!data) {
-			results.push({ skill: name, status: 'not_found', local_modified: false, remote_updated: false })
-			continue
-		}
+	// Detect local modifications for all skills in parallel
+	const detections = await Promise.all(entries.map(([name, data]) => {
+		if (!data) return { name, data, det: null }
 		const short_name = name.split('/')[1] || name
 		const dir = skill_install_dir(base_dir, short_name)
-		const [, det] = await detect_status(data, dir)
+		return detect_status(data, dir).then(([, det]) => ({ name, data, det }))
+	}))
+
+	const results = detections.map(({ name, data, det }) => {
+		if (!data) return { skill: name, status: 'not_found', local_modified: false, remote_updated: false }
 		const has_conflicts = (data.conflict_files || []).length > 0
-		results.push({
+		return {
 			skill: name,
 			base_version: data.version || null,
 			base_commit: data.base_commit || null,
@@ -91,8 +91,8 @@ const run = (args) => catch_errors('Status failed', async () => {
 			remote_commit: null,
 			conflict_files: data.conflict_files || [],
 			status: has_conflicts ? 'conflicts' : 'clean'
-		})
-	}
+		}
+	})
 
 	// Check remote for updates
 	const skill_names = results.filter(r => r.status !== 'not_found').map(r => r.skill)

package/src/commands/update.js

@@ -72,16 +72,24 @@ const run = (args) => catch_errors('Update failed', async () => {
 	const base_dir = skills_dir(is_global, project_root)
 	const force = args.flags.force || false
 
-	for (const [name, data] of to_update) {
-		// Check for local modifications before overwriting
-		if (!force && data) {
+	// Detect local modifications for all skills in parallel
+	const modified_set = new Set()
+	if (!force) {
+		const detections = await Promise.all(to_update.map(([name, data]) => {
+			if (!data) return { name, modified: false }
 			const short_name = name.split('/')[1] || name
 			const dir = skill_install_dir(base_dir, short_name)
-			const [, det] = await detect_status(data, dir)
-			if (det?.local_modified) {
-				print_warn(`${name} has local modifications. Use --force to discard, or 'happyskills pull' to merge.`)
-				continue
-			}
+			return detect_status(data, dir).then(([, det]) => ({ name, modified: det?.local_modified || false }))
+		}))
+		for (const { name, modified } of detections) {
+			if (modified) modified_set.add(name)
+		}
+	}
+
+	for (const [name, data] of to_update) {
+		if (modified_set.has(name)) {
+			print_warn(`${name} has local modifications. Use --force to discard, or 'happyskills pull' to merge.`)
+			continue
 		}
 
 		const before_version = data?.version || null