@lythos/skill-deck 0.9.24 → 0.9.26

package/README.md

@@ -9,7 +9,7 @@
 This package exposes a **CLI**. Invoke via:
 
 ```bash
-bunx @lythos/skill-deck@0.9.24 <command> [options]
+bunx @lythos/skill-deck@0.9.26 <command> [options]
 ```
 
 No installation required. `bunx` auto-downloads the package.
@@ -55,14 +55,14 @@ prompt = "Search for latest info, then generate professional document with diagr
 
 | Situation | Command |
 |-----------|---------|
-| Sync working set with `skill-deck.toml` | `bunx @lythos/skill-deck@0.9.24 link` |
-| Validate `skill-deck.toml` before committing | `bunx @lythos/skill-deck@0.9.24 validate` |
-| Download a skill to cold pool and add to deck | `bunx @lythos/skill-deck@0.9.24 add owner/repo` |
-| Pull latest versions of declared skills | `bunx @lythos/skill-deck@0.9.24 refresh` |
-| Refresh a single skill by alias | `bunx @lythos/skill-deck@0.9.24 refresh tdd` |
-| Remove a skill from deck and working set | `bunx @lythos/skill-deck@0.9.24 remove tdd` |
-| GC unreferenced repos from cold pool | `bunx @lythos/skill-deck@0.9.24 prune` |
-| Use a custom deck file or working dir | `bunx @lythos/skill-deck@0.9.24 link --deck ./my-deck.toml --workdir /path/to/project` |
+| Sync working set with `skill-deck.toml` | `bunx @lythos/skill-deck@0.9.26 link` |
+| Validate `skill-deck.toml` before committing | `bunx @lythos/skill-deck@0.9.26 validate` |
+| Download a skill to cold pool and add to deck | `bunx @lythos/skill-deck@0.9.26 add owner/repo` |
+| Pull latest versions of declared skills | `bunx @lythos/skill-deck@0.9.26 refresh` |
+| Refresh a single skill by alias | `bunx @lythos/skill-deck@0.9.26 refresh tdd` |
+| Remove a skill from deck and working set | `bunx @lythos/skill-deck@0.9.26 remove tdd` |
+| GC unreferenced repos from cold pool | `bunx @lythos/skill-deck@0.9.26 prune` |
+| Use a custom deck file or working dir | `bunx @lythos/skill-deck@0.9.26 link --deck ./my-deck.toml --workdir /path/to/project` |
 
 ### Commands
 
@@ -119,7 +119,7 @@ path = "github.com/lythos-labs/lythoskill/skills/lythoskill-deck"
 EOF
 
 # 2. Link — creates symlinks in .claude/skills/
-bunx @lythos/skill-deck@0.9.24 link
+bunx @lythos/skill-deck@0.9.26 link
 ```
 
 ### Key Concepts
@@ -148,7 +148,7 @@ Different agents look for skills in different directories. `skill-deck.toml` con
 
 | Symptom | Cause | Fix |
 |---------|-------|-----|
-| `❌ Skill not found: <name>` | Skill declared in deck but not in cold pool | `bunx @lythos/skill-deck@0.9.24 add github.com/owner/repo/skill` or clone manually into cold pool |
+| `❌ Skill not found: <name>` | Skill declared in deck but not in cold pool | `bunx @lythos/skill-deck@0.9.26 add github.com/owner/repo/skill` or clone manually into cold pool |
 | `link` skips entries with warnings | Real files/directories exist in working set (not symlinks) | Delete the real directories in `working_set` and re-run `link`. Never create directories manually there |
 | `refresh` reports "Not a git repository" | Skill was copied (not cloned) into cold pool | Re-clone with `git clone` or use `deck add` which clones by default |
 | `deck update` prints deprecation warning | `update` was renamed to `refresh` in v0.8+ | Use `deck refresh` instead |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lythos/skill-deck",
-  "version": "0.9.24",
+  "version": "0.9.26",
   "description": "Declarative skill deck governance — cold pool, working set, deny-by-default",
   "keywords": [
     "ai-agent",
@@ -26,7 +26,7 @@
   ],
   "dependencies": {
     "@iarna/toml": "^2.2.5",
-    "@lythos/cold-pool": "workspace:*",
+    "@lythos/cold-pool": "^0.9.25",
     "yaml": "^2.8.3",
     "zod": "^4.3.6"
   },

package/src/add.ts

@@ -148,12 +148,6 @@ export async function addSkill(
     return
   }
 
-  if (fetchPlan.alreadyExists) {
-    console.error(`❌ Already exists in cold pool: ${fetchPlan.targetDir}`)
-    console.error(`   To update: rm -rf ${fetchPlan.targetDir} and re-run`)
-    process.exit(1)
-  }
-
   if (!existsSync(coldPoolPath)) {
     console.log(`📁 Creating cold pool: ${coldPoolPath}`)
     mkdirSync(coldPoolPath, { recursive: true })
@@ -161,6 +155,13 @@ export async function addSkill(
   // git clone needs the parent of the target dir (e.g. host/owner/) to exist
   mkdirSync(dirname(fetchPlan.targetDir), { recursive: true })
 
+  // Note: if cold pool already has the repo (fetchPlan.alreadyExists),
+  // executeFetchPlan returns status: 'already-present' and skips clone.
+  // We still want to write deck.toml + link this skill — critical for
+  // monorepo case (second `deck add` from same repo, e.g. anthropics/skills/pdf
+  // then anthropics/skills/docx). Used to early-exit here, which broke
+  // the monorepo workflow and triggered post-compaction agent CPTSD
+  // (see: 2026-05-07 morning skill-deck.toml overwrite incident).
   const fetchResult = executeFetchPlan(fetchPlan, {
     log: (msg) => console.log(msg),
   })
@@ -171,6 +172,13 @@ export async function addSkill(
     process.exit(1)
   }
 
+  if (fetchResult.status === 'already-present') {
+    console.log(`✓ Repo already in cold pool — skipped clone.`)
+    console.log(`   To check for upstream updates without pulling:`)
+    console.log(`     bunx @lythos/skill-deck refresh ${parsed.host}/${parsed.owner}/${parsed.repo}`)
+    console.log(`   (per ADR-20260507110332805, refresh defaults to discover-only)`)
+  }
+
   const skillDir = findSkillDir(fetchPlan.targetDir, parsed.skill)
   if (!skillDir) {
     console.error(`❌ No SKILL.md found in downloaded repo`)