@lythos/skill-deck 0.14.2 → 0.14.4

package/README.md

@@ -8,16 +8,16 @@
 
 ```bash
 # Add a skill from skills.sh (owner/repo syntax — no conversion needed)
-bunx @lythos/skill-deck@0.14.2 add vercel-labs/agent-skills
+bunx @lythos/skill-deck@0.14.4 add vercel-labs/agent-skills
 
 # Or with @skill filter (same as npx skills add):
-bunx @lythos/skill-deck@0.14.2 add mattpocock/skills@tdd
+bunx @lythos/skill-deck@0.14.4 add mattpocock/skills@tdd
 
 # Or FQ locator:
-bunx @lythos/skill-deck@0.14.2 add github.com/anthropics/skills/skills/frontend-design
+bunx @lythos/skill-deck@0.14.4 add github.com/anthropics/skills/skills/frontend-design
 
 # Sync working set (deny-by-default):
-bunx @lythos/skill-deck@0.14.2 link
+bunx @lythos/skill-deck@0.14.4 link
 ```
 
 ## For AI Agents
@@ -25,7 +25,7 @@ bunx @lythos/skill-deck@0.14.2 link
 This package exposes a **CLI**. Invoke via:
 
 ```bash
-bunx @lythos/skill-deck@0.14.2 <command> [options]
+bunx @lythos/skill-deck@0.14.4 <command> [options]
 ```
 
 No installation required. `bunx` auto-downloads the package.
@@ -73,15 +73,15 @@ 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.14.2 link` |
-| Validate `skill-deck.toml` before committing | `bunx @lythos/skill-deck@0.14.2 validate` |
-| Download a skill to cold pool and add to deck | `bunx @lythos/skill-deck@0.14.2 add owner/repo` |
-| Pull latest versions of declared skills | `bunx @lythos/skill-deck@0.14.2 refresh` |
-| Refresh a single skill by alias | `bunx @lythos/skill-deck@0.14.2 refresh tdd` |
-| Remove a skill from deck and working set | `bunx @lythos/skill-deck@0.14.2 remove tdd` |
-| Switch skill to symlink mode (live) | `bunx @lythos/skill-deck@0.14.2 to-symlink tdd` |
-| Switch skill to snapshot mode (pinned) | `bunx @lythos/skill-deck@0.14.2 to-snapshot tdd` |
-| Use a custom deck file or working dir | `bunx @lythos/skill-deck@0.14.2 link --deck ./my-deck.toml --workdir /path/to/project` |
+| Sync working set with `skill-deck.toml` | `bunx @lythos/skill-deck@0.14.4 link` |
+| Validate `skill-deck.toml` before committing | `bunx @lythos/skill-deck@0.14.4 validate` |
+| Download a skill to cold pool and add to deck | `bunx @lythos/skill-deck@0.14.4 add owner/repo` |
+| Pull latest versions of declared skills | `bunx @lythos/skill-deck@0.14.4 refresh` |
+| Refresh a single skill by alias | `bunx @lythos/skill-deck@0.14.4 refresh tdd` |
+| Remove a skill from deck and working set | `bunx @lythos/skill-deck@0.14.4 remove tdd` |
+| Switch skill to symlink mode (live) | `bunx @lythos/skill-deck@0.14.4 to-symlink tdd` |
+| Switch skill to snapshot mode (pinned) | `bunx @lythos/skill-deck@0.14.4 to-snapshot tdd` |
+| Use a custom deck file or working dir | `bunx @lythos/skill-deck@0.14.4 link --deck ./my-deck.toml --workdir /path/to/project` |
 
 ### Commands
 
@@ -143,7 +143,7 @@ source = "https://github.com/lythos-labs/lythoskill/blob/HEAD/skills/lythoskill-
 EOF
 
 # 2. Link — creates symlinks in .claude/skills/
-bunx @lythos/skill-deck@0.14.2 link
+bunx @lythos/skill-deck@0.14.4 link
 ```
 
 ### Key Concepts
@@ -229,7 +229,7 @@ Caution: deck's deny-by-default will remove any skills not declared in your deck
 
 | Symptom | Cause | Fix |
 |---------|-------|-----|
-| `❌ Skill not found: <name>` | Skill declared in deck but not in cold pool | `bunx @lythos/skill-deck@0.14.2 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.14.4 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.14.2",
+  "version": "0.14.4",
   "description": "Declarative skill deck governance — cold pool, working set, deny-by-default",
   "keywords": [
     "ai-agent",
@@ -31,8 +31,8 @@
   ],
   "dependencies": {
     "@iarna/toml": "^2.2.5",
-    "@lythos/cold-pool": "^0.14.2",
-    "@lythos/infra": "^0.14.2",
+    "@lythos/cold-pool": "^0.14.4",
+    "@lythos/infra": "^0.14.4",
     "yaml": "^2.8.3",
     "zod": "^4.3.6"
   },

package/src/add.ts

@@ -183,7 +183,7 @@ function exitInvalidLocator(locator: string): never {
   console.error(`     owner/repo@skill                — skills.sh syntax`)
   console.error(`     owner/repo/subpath              — subdirectory`)
   console.error(`     github:owner/repo               — explicit GitHub prefix`)
-  console.error(`     localhost/<name>                — local-only skill`)
+  console.error(`     localhost/me/<skill>             — local-only skill (host/owner/repo aligned)`)
   console.error(``)
   console.error(`   Note: FQ locators look like URLs but map to cold pool paths:`)
   console.error(`     github.com/o/r/skills/s → ~/.agents/skill-repos/github.com/o/r/skills/s/SKILL.md`)

package/src/link.ts

@@ -76,7 +76,7 @@ export function findSource(name: string, coldPool: string, _projectDir: string):
   if (!locator) {
     return {
       path: null,
-      error: `Locator "${name}" is not FQ. Expected: host.tld/owner/repo[/skill] or localhost/<name>. Bare names rejected per ADR-20260502012643244.`,
+      error: `Locator "${name}" is not FQ. Expected: host.tld/owner/repo[/skill] or localhost/me/<skill>. Bare names rejected per ADR-20260502012643244.`,
     };
   }
 
@@ -188,6 +188,7 @@ interface DeclaredSkill {
   alias: string;       // working-set flat symlink name
   type: "innate" | "tool" | "combo" | "transient";
   sourcePath: string;
+  mode?: 'symlink' | 'snapshot';
   expires?: string;
 }
 
@@ -234,7 +235,8 @@ for (const entry of parsedEntries) {
       continue
     }
   }
-  declared.push({ name: entry.path, alias: entry.alias, type: entry.type, sourcePath: result.path });
+  const entryMode = (entry as any).mode as 'symlink' | 'snapshot' | undefined;
+  declared.push({ name: entry.path, alias: entry.alias, type: entry.type, sourcePath: result.path, mode: entryMode });
 }
 
 // transient: sub-tables with path field (kept backward-compat; future ADR may unify)
@@ -427,7 +429,8 @@ for (const item of declared) {
 
   try {
     mkdirSync(dirname(dest), { recursive: true });
-    if (MODE === 'snapshot') {
+    const linkMode = item.mode ?? MODE;
+	    if (linkMode === 'snapshot') {
       cpSync(item.sourcePath, dest, { recursive: true });
     } else {
       symlinkSync(item.sourcePath, dest);
@@ -463,7 +466,7 @@ for (const item of declared) {
     type: item.type,
     source: sourceRel,
     dest: relative(PROJECT_DIR, dest),
-    mode: MODE,
+    mode: item.mode ?? MODE,
     content_hash: contentHash,
     linked_at: new Date().toISOString(),
     ...(item.expires ? { expires: item.expires } : {}),

package/src/refresh-plan.test.ts

@@ -135,6 +135,25 @@ describe('buildRefreshPlan', () => {
     expect(plan.workdir).toBe('/custom/work')
     expect(plan.coldPool).toBe('/custom/pool')
   })
+  test('plan-mode: all declared skills appear in plan structure', () => {
+    const plan = buildRefreshPlan(deckAliasDict, { coldPool: '/pool' })
+    expect(plan.allDeclared).toHaveLength(2)
+    expect(plan.targets.length).toBeGreaterThanOrEqual(0)
+    // Plan structure is correct — path/alias mapping verified.
+    // Type detection (git/localhost/missing) depends on filesystem;
+    // those cases are covered by detectGitRoot tests above.
+  })
+
+  test('plan-mode: plan carries correct config paths through', () => {
+    const plan = buildRefreshPlan(deckAliasDict, {
+      deckPath: '/custom/deck.toml',
+      workdir: '/custom/work',
+      coldPool: '/custom/pool',
+    })
+    expect(plan.deckPath).toBe('/custom/deck.toml')
+    expect(plan.workdir).toBe('/custom/work')
+    expect(plan.coldPool).toBe('/custom/pool')
+  })
 })
 
 // ── executeRefreshPlan (IO-injected plan execution) ────────────────