happyskills 0.22.0 → 0.23.0

package/CHANGELOG.md

@@ -7,6 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.23.0] - 2026-03-30
+
+### Added
+- Add renamed file detection to `pull` merge — files renamed on one side (same content, different path) are now classified as `remote_renamed`/`local_renamed` instead of delete + add, and applied as renames on disk. Only 1-to-1 SHA matches are detected; ambiguous many-to-many cases are safely skipped.
+
+## [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.23.0",
 	"description": "Package manager for AI agent skills",
 	"license": "SEE LICENSE IN LICENSE",
 	"author": "Nicolas Dao <nic@cloudlesslabs.com> (https://cloudlesslabs.com)",

package/src/commands/pull.js

@@ -360,7 +360,16 @@ const run = (args) => catch_errors('Pull failed', async () => {
 	const [del_err] = await remove_files(skill_dir, classified.remote_only_deleted.map(f => f.path))
 	if (del_err) { spinner.fail('Failed to remove deleted files'); throw del_err[0] }
 
-	// Local-only modified/added → keep (no action needed)
+	// Remote-only renamed → rename local files
+	for (const entry of classified.remote_renamed) {
+		const old_full = path.join(skill_dir, entry.old_path)
+		const new_full = path.join(skill_dir, entry.new_path)
+		const [dir_err] = await ensure_dir(path.dirname(new_full))
+		if (dir_err) { spinner.fail(`Failed to create dir for ${entry.new_path}`); throw dir_err[0] }
+		await fs.promises.rename(old_full, new_full)
+	}
+
+	// Local-only modified/added/renamed → keep (no action needed)
 	// Local-only deleted → keep deleted (no action needed)
 	if (full_report) {
 		for (const entry of [...classified.local_only_modified, ...classified.local_only_added]) {

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

package/src/merge/comparator.js

@@ -7,6 +7,37 @@
  * All inputs are arrays of { path, sha } entries.
  */
 
+/**
+ * Detect 1-to-1 renames by matching SHAs between deleted and added files.
+ * Ambiguous matches (multiple deletes or adds sharing a SHA) are skipped.
+ */
+const detect_renames = (deleted, added, get_del_sha, get_add_sha) => {
+	const del_by_sha = new Map()
+	for (const entry of deleted) {
+		const sha = get_del_sha(entry)
+		if (!sha) continue
+		del_by_sha.set(sha, del_by_sha.has(sha) ? null : entry)
+	}
+	const add_by_sha = new Map()
+	for (const entry of added) {
+		const sha = get_add_sha(entry)
+		if (!sha) continue
+		add_by_sha.set(sha, add_by_sha.has(sha) ? null : entry)
+	}
+	const renames = []
+	const del_matched = new Set()
+	const add_matched = new Set()
+	for (const [sha, del_entry] of del_by_sha) {
+		if (!del_entry) continue
+		const add_entry = add_by_sha.get(sha)
+		if (!add_entry) continue
+		renames.push({ old_path: del_entry.path, new_path: add_entry.path, sha })
+		del_matched.add(del_entry.path)
+		add_matched.add(add_entry.path)
+	}
+	return { renames, del_matched, add_matched }
+}
+
 const classify_changes = (base_files, local_files, remote_files) => {
 	const base_map = new Map(base_files.map(e => [e.path, e.sha]))
 	const local_map = new Map(local_files.map(e => [e.path, e.sha]))
@@ -83,14 +114,26 @@ const classify_changes = (base_files, local_files, remote_files) => {
 		}
 	}
 
+	// Detect renames: 1-to-1 SHA matches between deleted and added on the same side
+	const r = detect_renames(
+		remote_only_deleted, remote_only_added,
+		e => base_map.get(e.path), e => e.remote_sha
+	)
+	const l = detect_renames(
+		local_only_deleted, local_only_added,
+		e => base_map.get(e.path), e => e.local_sha
+	)
+
 	return {
 		remote_only_modified,
 		local_only_modified,
 		both_modified,
-		remote_only_added,
-		local_only_added,
-		remote_only_deleted,
-		local_only_deleted,
+		remote_only_added: remote_only_added.filter(e => !r.add_matched.has(e.path)),
+		local_only_added: local_only_added.filter(e => !l.add_matched.has(e.path)),
+		remote_only_deleted: remote_only_deleted.filter(e => !r.del_matched.has(e.path)),
+		local_only_deleted: local_only_deleted.filter(e => !l.del_matched.has(e.path)),
+		remote_renamed: r.renames,
+		local_renamed: l.renames,
 		unchanged
 	}
 }

package/src/merge/comparator.test.js

@@ -158,4 +158,97 @@ describe('classify_changes', () => {
 		assert.strictEqual(r.both_modified.length, 1)
 		assert.deepStrictEqual(r.both_modified[0], { path: 'a.md', base_sha: '111', local_sha: '222', remote_sha: null })
 	})
+
+	it('detects remote rename (1-to-1 SHA match)', () => {
+		const base = [{ path: 'old.md', sha: 'aaa' }]
+		const local = [{ path: 'old.md', sha: 'aaa' }]
+		const remote = [{ path: 'new.md', sha: 'aaa' }]
+		const r = classify_changes(base, local, remote)
+		assert.strictEqual(r.remote_renamed.length, 1)
+		assert.deepStrictEqual(r.remote_renamed[0], { old_path: 'old.md', new_path: 'new.md', sha: 'aaa' })
+		assert.strictEqual(r.remote_only_deleted.length, 0)
+		assert.strictEqual(r.remote_only_added.length, 0)
+	})
+
+	it('detects local rename (1-to-1 SHA match)', () => {
+		const base = [{ path: 'old.md', sha: 'bbb' }]
+		const local = [{ path: 'new.md', sha: 'bbb' }]
+		const remote = [{ path: 'old.md', sha: 'bbb' }]
+		const r = classify_changes(base, local, remote)
+		assert.strictEqual(r.local_renamed.length, 1)
+		assert.deepStrictEqual(r.local_renamed[0], { old_path: 'old.md', new_path: 'new.md', sha: 'bbb' })
+		assert.strictEqual(r.local_only_deleted.length, 0)
+		assert.strictEqual(r.local_only_added.length, 0)
+	})
+
+	it('skips rename detection for many-to-many SHA matches', () => {
+		// Two deleted files with same SHA + two added files with same SHA → ambiguous
+		const base = [
+			{ path: 'a.md', sha: 'xxx' },
+			{ path: 'b.md', sha: 'xxx' },
+		]
+		const local = [
+			{ path: 'a.md', sha: 'xxx' },
+			{ path: 'b.md', sha: 'xxx' },
+		]
+		const remote = [
+			{ path: 'c.md', sha: 'xxx' },
+			{ path: 'd.md', sha: 'xxx' },
+		]
+		const r = classify_changes(base, local, remote)
+		assert.strictEqual(r.remote_renamed.length, 0)
+		assert.strictEqual(r.remote_only_deleted.length, 2)
+		assert.strictEqual(r.remote_only_added.length, 2)
+	})
+
+	it('skips rename when only one side is ambiguous (two deletes, one add)', () => {
+		const base = [
+			{ path: 'a.md', sha: 'xxx' },
+			{ path: 'b.md', sha: 'xxx' },
+		]
+		const local = [
+			{ path: 'a.md', sha: 'xxx' },
+			{ path: 'b.md', sha: 'xxx' },
+		]
+		const remote = [{ path: 'c.md', sha: 'xxx' }]
+		const r = classify_changes(base, local, remote)
+		assert.strictEqual(r.remote_renamed.length, 0)
+		assert.strictEqual(r.remote_only_deleted.length, 2)
+		assert.strictEqual(r.remote_only_added.length, 1)
+	})
+
+	it('does not detect rename when content changed (different SHAs)', () => {
+		const base = [{ path: 'old.md', sha: 'aaa' }]
+		const local = [{ path: 'old.md', sha: 'aaa' }]
+		const remote = [{ path: 'new.md', sha: 'bbb' }]
+		const r = classify_changes(base, local, remote)
+		assert.strictEqual(r.remote_renamed.length, 0)
+		assert.strictEqual(r.remote_only_deleted.length, 1)
+		assert.strictEqual(r.remote_only_added.length, 1)
+	})
+
+	it('handles mixed scenario with renames alongside other changes', () => {
+		const base = [
+			{ path: 'unchanged.md', sha: '000' },
+			{ path: 'old_name.md', sha: 'rrr' },
+			{ path: 'remote_mod.md', sha: '111' },
+		]
+		const local = [
+			{ path: 'unchanged.md', sha: '000' },
+			{ path: 'old_name.md', sha: 'rrr' },
+			{ path: 'remote_mod.md', sha: '111' },
+		]
+		const remote = [
+			{ path: 'unchanged.md', sha: '000' },
+			{ path: 'new_name.md', sha: 'rrr' },
+			{ path: 'remote_mod.md', sha: 'R11' },
+		]
+		const r = classify_changes(base, local, remote)
+		assert.strictEqual(r.unchanged.length, 1)
+		assert.strictEqual(r.remote_renamed.length, 1)
+		assert.deepStrictEqual(r.remote_renamed[0], { old_path: 'old_name.md', new_path: 'new_name.md', sha: 'rrr' })
+		assert.strictEqual(r.remote_only_modified.length, 1)
+		assert.strictEqual(r.remote_only_deleted.length, 0)
+		assert.strictEqual(r.remote_only_added.length, 0)
+	})
 })

package/src/merge/report.js

@@ -14,6 +14,7 @@ const CLASSIFICATIONS = [
 	'remote_only_modified', 'local_only_modified', 'both_modified',
 	'remote_only_added', 'local_only_added',
 	'remote_only_deleted', 'local_only_deleted',
+	'remote_renamed', 'local_renamed',
 	'unchanged'
 ]
 
@@ -41,14 +42,17 @@ const build_report = (skill, base_version, remote_version, classified) => {
 			if (is_conflict) conflicted++
 			else auto_merged++
 
-			files.push({
-				path: entry.path,
+			const is_rename = classification === 'remote_renamed' || classification === 'local_renamed'
+			const file_entry = {
+				path: is_rename ? entry.new_path : entry.path,
 				classification,
 				conflict_written: false,
-				base_sha: entry.base_sha || null,
+				base_sha: entry.base_sha || entry.sha || null,
 				local_sha: entry.local_sha || null,
 				remote_sha: entry.remote_sha || null
-			})
+			}
+			if (is_rename) file_entry.old_path = entry.old_path
+			files.push(file_entry)
 		}
 	}