happyskills 0.35.0 → 0.35.1

package/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.35.1] - 2026-04-10
+
+### Fixed
+- Fix multi-agent skill symlinks using absolute paths instead of relative — absolute symlinks break when the repo is cloned or pulled on a different machine; now uses `path.relative()` for portable symlinks
+- Fix enable skip-check failing to recognize existing relative symlinks — resolves both relative and absolute link targets before comparing
+
 ## [0.35.0] - 2026-04-10
 
 ### Added

package/package.json

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

package/src/agents/linker.js

@@ -43,8 +43,10 @@ const _exists = async (target_path) => {
  */
 const _link_or_copy = async (source, target) => {
 	try {
+		// Use relative path so symlinks work across machines (clones/pulls)
+		const relative_source = path.relative(path.dirname(target), source)
 		// 'junction' works on Windows without admin; ignored on macOS/Linux
-		await fs.promises.symlink(source, target, 'junction')
+		await fs.promises.symlink(relative_source, target, 'junction')
 		return { method: 'symlink' }
 	} catch (err) {
 		if (err.code === 'EPERM' || err.code === 'ENOTSUP') {
@@ -77,7 +79,9 @@ const link_to_agents = (source_dir, agents, options = {}) => catch_errors('Agent
 		if (is_link) {
 			try {
 				const link_target = await fs.promises.readlink(target)
-				if (link_target === source_dir) {
+				// Resolve relative or absolute symlink to compare against source
+				const resolved_link = path.resolve(path.dirname(target), link_target)
+				if (resolved_link === source_dir) {
 					results.push({ agent_id: agent.id, path: target, method: 'symlink', skipped: true })
 					continue
 				}

package/src/agents/linker.test.js

@@ -71,7 +71,9 @@ describe('link_to_agents', () => {
 		const link_path = path.join(tmp, '.claude', 'skills', 'deploy-aws')
 		const stat = fs.lstatSync(link_path)
 		assert.ok(stat.isSymbolicLink())
-		assert.equal(fs.readlinkSync(link_path), source)
+		// Symlink should be relative, not absolute
+		const link_target = fs.readlinkSync(link_path)
+		assert.equal(link_target, path.relative(path.dirname(link_path), source))
 	})
 
 	it('creates symlinks for multiple agents', async () => {
@@ -126,7 +128,9 @@ describe('link_to_agents', () => {
 		assert.equal(results[0].skipped, undefined)
 
 		const link_path = path.join(tmp, '.claude', 'skills', 'deploy-aws')
-		assert.equal(fs.readlinkSync(link_path), source)
+		// Symlink should be relative, not absolute
+		const link_target = fs.readlinkSync(link_path)
+		assert.equal(link_target, path.relative(path.dirname(link_path), source))
 	})
 })