@iceinvein/agent-skills 0.1.40 → 0.3.0

package/README.md

@@ -24,6 +24,12 @@ Auto-detects Claude Code, Cursor, Codex, or Gemini CLI. Use `--tool claude` to t
 
 ## Skills
 
+### Migration
+
+| Skill | What it does |
+|-------|--------------|
+| **migrate** | Walks a legacy codebase through probe, enumerate, seam, extract, parity, and queue: measured surface coverage, an empirically derived capability seam, cited requirements, a parity plan, and a batch decision queue. Bundles a Bun CLI that enforces the coverage arithmetic and phase ordering instead of trusting it. |
+
 ### Code Architecture
 
 Skills that analyze how code is structured: module boundaries, coupling, complexity, contracts, and evolution over time.
@@ -91,6 +97,14 @@ Skills for producing job application materials that read like a human wrote them
 | **cover-letter-rewrite** | Reviser | Audit-driven targeted rewrite of an existing letter. Focus modes: humanize, align, tighten, structure, tone. Preserves voice where it already works. |
 | **cover-letter-persona** | Voice profile | Reusable writing personas using the NNGroup 4-dimension tone framework (funny-serious, formal-casual, respectful-irreverent, enthusiastic-matter-of-fact) adapted for professional correspondence. |
 
+### Workflow
+
+Skills that shape how work gets done rather than analyzing code.
+
+| Skill | What it does |
+|-------|--------------|
+| **sluice** | Routes work by change shape into four channels (`bypass`, `fast`, `main`, `deep`) and applies only the rules each channel needs, so a one-line fix does not pay the cost of a multi-subsystem build. Seven rules live as one-liners in the router; the full treatment sits in references read only on friction. Each run closes with a ledger read out of the session transcript: elapsed, tools, tokens, and what every dispatched agent cost. Claude Code only. Conflicts with the superpowers plugin, which requires its own fixed pipeline up front for anything that adds to or changes what the software does, not just code edits; disable superpowers before installing. |
+
 ### Orchestration
 
 Skills that compose the other audit skills into higher-level workflows.
@@ -145,11 +159,13 @@ To pin to the current version, do nothing: skills are not auto-updated. `update`
 
 ## Activation Modes (Claude Code)
 
-Some skills (like `terse`) support activation modes. Pick one at install time:
+Some skills (like `terse` and `sluice`) support activation modes. Pick one at install time:
 
-- **session** (default): invoke the skill manually with `/<skill>` each session
+- **session**: invoke the skill manually with `/<skill>` each session
 - **global**: auto-activate every Claude Code session via a `SessionStart` hook in `.claude/settings.json`
 
+Each skill declares its own default: `terse` defaults to session, `sluice` defaults to global because a router that is not always on does not route.
+
 If a skill declares activation support and you're installing for Claude Code interactively, the CLI prompts you. Use `--activation session` or `--activation global` for scripted installs. `remove` strips the hook cleanly; `update` preserves your choice across version bumps.
 
 ## Supported Tools

package/dist/cli/index.js

@@ -402,7 +402,7 @@ function resolveBundlePaths(entries, bundle) {
 }
 
 // src/cli/adapters/claude.ts
-import { chmodSync, existsSync as existsSync2, mkdirSync, rmSync, statSync, unlinkSync } from "node:fs";
+import { chmodSync, existsSync as existsSync2, mkdirSync, rmSync, rmdirSync, statSync, unlinkSync } from "node:fs";
 import { dirname, join as join2 } from "node:path";
 function shouldBeExecutable(relPath, content) {
   if (relPath.endsWith(".sh"))
@@ -423,8 +423,10 @@ function runScript(scriptPath, cwd) {
   });
   return { ok: result.exitCode === 0, code: result.exitCode ?? 1 };
 }
-function matchesSkillDirective(command, skillName) {
-  return command.includes(`Activate ${skillName} skill`);
+function matchesSkillDirective(hook, skillName) {
+  if (hook.skill !== undefined)
+    return hook.skill === skillName;
+  return hook.command.includes(`Activate ${skillName} skill`);
 }
 async function wireSessionStartHook(settingsPath, skillName, directive) {
   let settings = {};
@@ -438,12 +440,12 @@ async function wireSessionStartHook(settingsPath, skillName, directive) {
   const command = `echo '${directive}'`;
   for (const group of settings.hooks.SessionStart) {
     for (const hook of group.hooks ?? []) {
-      if (matchesSkillDirective(hook.command, skillName))
+      if (matchesSkillDirective(hook, skillName))
         return;
     }
   }
   settings.hooks.SessionStart.push({
-    hooks: [{ type: "command", command }]
+    hooks: [{ type: "command", command, skill: skillName }]
   });
   mkdirSync(dirname(settingsPath), { recursive: true });
   await Bun.write(settingsPath, JSON.stringify(settings, null, 2) + `
@@ -457,7 +459,7 @@ async function unwireSessionStartHook(settingsPath, skillName) {
   if (!sessionStart)
     return;
   const filteredGroups = sessionStart.map((group) => ({
-    hooks: (group.hooks ?? []).filter((h) => !matchesSkillDirective(h.command, skillName))
+    hooks: (group.hooks ?? []).filter((h) => !matchesSkillDirective(h, skillName))
   })).filter((group) => group.hooks.length > 0);
   if (filteredGroups.length === 0) {
     delete settings.hooks.SessionStart;
@@ -567,7 +569,7 @@ var claudeAdapter = {
         const stopAt = config.bundleRoot ? join2(cwd, config.bundleRoot, "..") : cwd;
         while (dir !== stopAt && dir !== "/" && dir.startsWith(cwd)) {
           try {
-            rmSync(dir, { recursive: false });
+            rmdirSync(dir);
           } catch {
             break;
           }
@@ -596,7 +598,7 @@ var claudeAdapter = {
 };
 
 // src/cli/adapters/cursor.ts
-import { existsSync as existsSync3, mkdirSync as mkdirSync2, unlinkSync as unlinkSync2, rmSync as rmSync2 } from "node:fs";
+import { existsSync as existsSync3, mkdirSync as mkdirSync2, unlinkSync as unlinkSync2, rmdirSync as rmdirSync2 } from "node:fs";
 import { dirname as dirname2, join as join3 } from "node:path";
 var cursorAdapter = {
   name: "cursor",
@@ -649,7 +651,7 @@ var cursorAdapter = {
       if (existsSync3(fullPath)) {
         unlinkSync2(fullPath);
         try {
-          rmSync2(dirname2(fullPath), { recursive: false });
+          rmdirSync2(dirname2(fullPath));
         } catch {}
       }
     }
@@ -726,7 +728,7 @@ var codexAdapter = {
 };
 
 // src/cli/adapters/gemini.ts
-import { existsSync as existsSync5, mkdirSync as mkdirSync3, unlinkSync as unlinkSync3, rmSync as rmSync3 } from "node:fs";
+import { existsSync as existsSync5, mkdirSync as mkdirSync3, unlinkSync as unlinkSync3, rmdirSync as rmdirSync3 } from "node:fs";
 import { dirname as dirname3, join as join5 } from "node:path";
 var geminiAdapter = {
   name: "gemini",
@@ -779,7 +781,7 @@ var geminiAdapter = {
       if (existsSync5(fullPath)) {
         unlinkSync3(fullPath);
         try {
-          rmSync3(dirname3(fullPath), { recursive: false });
+          rmdirSync3(dirname3(fullPath));
         } catch {}
       }
     }
@@ -865,6 +867,10 @@ async function installSkill(cwd, manifest, files, targetTools, activation) {
   return { ok: true, installed, skipped };
 }
 
+// src/cli/commands/remove.ts
+import { existsSync as existsSync6, statSync as statSync2, unlinkSync as unlinkSync4, rmSync as rmSync2 } from "node:fs";
+import { basename, join as join7 } from "node:path";
+
 // src/cli/github.ts
 var REPO2 = "iceinvein/agent-skills";
 var BRANCH2 = "master";
@@ -969,6 +975,28 @@ function resolveBundlePaths2(entries, bundle) {
 }
 
 // src/cli/commands/remove.ts
+var SHARED_CONFIG_FILES = new Set([
+  ".claude/settings.json",
+  ".claude/settings.local.json",
+  ".cursor/mcp.json",
+  ".gemini/settings.json"
+]);
+function isSharedConfigFile(relPath) {
+  if (SHARED_CONFIG_FILES.has(relPath))
+    return true;
+  const parts = relPath.split("/");
+  const name = basename(relPath);
+  const isConfigName = name === "settings.json" || name === "settings.local.json" || name === "mcp.json";
+  return parts.length === 2 && parts[0].startsWith(".") && isConfigName;
+}
+function removeFileOrDir(fullPath) {
+  const stat = statSync2(fullPath);
+  if (stat.isDirectory()) {
+    rmSync2(fullPath, { recursive: true, force: true });
+  } else {
+    unlinkSync4(fullPath);
+  }
+}
 async function removeSkill(cwd, skillName) {
   const lockfile = await readLockfile(cwd);
   const entry = lockfile.skills[skillName];
@@ -987,24 +1015,36 @@ async function removeSkill(cwd, skillName) {
           return true;
         if (config.supporting && Object.values(config.supporting).includes(f))
           return true;
+        if (config.bundleRoot && f.startsWith(`${config.bundleRoot}/`) && !isSharedConfigFile(f))
+          return true;
         if (config.mcpServers && (f.includes(".claude/settings") || f.includes(".cursor/mcp") || f.includes(".gemini/settings")))
           return true;
         return false;
       });
       await adapter.remove(cwd, manifestResult.manifest, toolFiles);
     }
-  } else {
-    const { unlinkSync: unlinkSync4, existsSync: existsSync6 } = await import("node:fs");
-    const { join: join7 } = await import("node:path");
-    for (const file of entry.files) {
-      const fullPath = join7(cwd, file);
-      if (existsSync6(fullPath)) {
-        unlinkSync4(fullPath);
+    await removeSkillFromLockfile(cwd, skillName);
+    return { ok: true, removed: entry.files };
+  }
+  const removed = [];
+  const warnings = [];
+  for (const file of entry.files) {
+    if (isSharedConfigFile(file)) {
+      const fullPath2 = join7(cwd, file);
+      if (existsSync6(fullPath2)) {
+        await unwireSessionStartHook(fullPath2, skillName);
       }
+      warnings.push(`Manifest unavailable (${manifestResult.error}). Left '${file}' in place, only removing '${skillName}'s SessionStart hook from it; any MCP server entries it owns were not touched.`);
+      continue;
     }
+    const fullPath = join7(cwd, file);
+    if (!existsSync6(fullPath))
+      continue;
+    removeFileOrDir(fullPath);
+    removed.push(file);
   }
   await removeSkillFromLockfile(cwd, skillName);
-  return { ok: true, removed: entry.files };
+  return { ok: true, removed, ...warnings.length > 0 ? { warnings } : {} };
 }
 
 // src/cli/commands/list.ts
@@ -1026,6 +1066,30 @@ async function infoSkill(skillName) {
 }
 
 // src/cli/commands/remove.ts
+import { existsSync as existsSync7, statSync as statSync3, unlinkSync as unlinkSync5, rmSync as rmSync3 } from "node:fs";
+import { basename as basename2, join as join8 } from "node:path";
+var SHARED_CONFIG_FILES2 = new Set([
+  ".claude/settings.json",
+  ".claude/settings.local.json",
+  ".cursor/mcp.json",
+  ".gemini/settings.json"
+]);
+function isSharedConfigFile2(relPath) {
+  if (SHARED_CONFIG_FILES2.has(relPath))
+    return true;
+  const parts = relPath.split("/");
+  const name = basename2(relPath);
+  const isConfigName = name === "settings.json" || name === "settings.local.json" || name === "mcp.json";
+  return parts.length === 2 && parts[0].startsWith(".") && isConfigName;
+}
+function removeFileOrDir2(fullPath) {
+  const stat = statSync3(fullPath);
+  if (stat.isDirectory()) {
+    rmSync3(fullPath, { recursive: true, force: true });
+  } else {
+    unlinkSync5(fullPath);
+  }
+}
 async function removeSkill2(cwd, skillName) {
   const lockfile = await readLockfile(cwd);
   const entry = lockfile.skills[skillName];
@@ -1044,24 +1108,36 @@ async function removeSkill2(cwd, skillName) {
           return true;
         if (config.supporting && Object.values(config.supporting).includes(f))
           return true;
+        if (config.bundleRoot && f.startsWith(`${config.bundleRoot}/`) && !isSharedConfigFile2(f))
+          return true;
         if (config.mcpServers && (f.includes(".claude/settings") || f.includes(".cursor/mcp") || f.includes(".gemini/settings")))
           return true;
         return false;
       });
       await adapter.remove(cwd, manifestResult.manifest, toolFiles);
     }
-  } else {
-    const { unlinkSync: unlinkSync4, existsSync: existsSync6 } = await import("node:fs");
-    const { join: join7 } = await import("node:path");
-    for (const file of entry.files) {
-      const fullPath = join7(cwd, file);
-      if (existsSync6(fullPath)) {
-        unlinkSync4(fullPath);
+    await removeSkillFromLockfile(cwd, skillName);
+    return { ok: true, removed: entry.files };
+  }
+  const removed = [];
+  const warnings = [];
+  for (const file of entry.files) {
+    if (isSharedConfigFile2(file)) {
+      const fullPath2 = join8(cwd, file);
+      if (existsSync7(fullPath2)) {
+        await unwireSessionStartHook(fullPath2, skillName);
       }
+      warnings.push(`Manifest unavailable (${manifestResult.error}). Left '${file}' in place, only removing '${skillName}'s SessionStart hook from it; any MCP server entries it owns were not touched.`);
+      continue;
     }
+    const fullPath = join8(cwd, file);
+    if (!existsSync7(fullPath))
+      continue;
+    removeFileOrDir2(fullPath);
+    removed.push(file);
   }
   await removeSkillFromLockfile(cwd, skillName);
-  return { ok: true, removed: entry.files };
+  return { ok: true, removed, ...warnings.length > 0 ? { warnings } : {} };
 }
 
 // src/cli/commands/install.ts
@@ -1131,7 +1207,7 @@ async function updateAllSkills(cwd) {
 }
 
 // src/cli/commands/bump.ts
-import { join as join7 } from "node:path";
+import { join as join9 } from "node:path";
 
 // src/cli/semver.ts
 function bumpVersion(current, level) {
@@ -1152,7 +1228,7 @@ function bumpVersion(current, level) {
 
 // src/cli/commands/bump.ts
 async function bumpSkill(repoRoot, skillName, level) {
-  const manifestPath = join7(repoRoot, "skills", skillName, "skill.json");
+  const manifestPath = join9(repoRoot, "skills", skillName, "skill.json");
   const file = Bun.file(manifestPath);
   if (!await file.exists()) {
     return { ok: false, error: `Skill '${skillName}' not found at skills/${skillName}/skill.json` };
@@ -1223,7 +1299,7 @@ async function bumpAllChanged(repoRoot, level, dryRun) {
     if (skillVersionChanged(repoRoot, skillName, tag))
       continue;
     if (dryRun) {
-      const manifestPath = join7(repoRoot, "skills", skillName, "skill.json");
+      const manifestPath = join9(repoRoot, "skills", skillName, "skill.json");
       const file = Bun.file(manifestPath);
       if (!await file.exists())
         continue;
@@ -1279,10 +1355,10 @@ async function checkForUpdates(cwd) {
 }
 
 // src/cli/lockfile.ts
-import { join as join8 } from "node:path";
+import { join as join10 } from "node:path";
 var LOCKFILE_NAME2 = ".agent-skills.lock";
 async function readLockfile2(cwd) {
-  const path = join8(cwd, LOCKFILE_NAME2);
+  const path = join10(cwd, LOCKFILE_NAME2);
   const file = Bun.file(path);
   if (!await file.exists()) {
     return { skills: {} };
@@ -3162,7 +3238,7 @@ async function pickActivation(skillName, modes) {
 
 // src/cli/index.ts
 import { mkdirSync as mkdirSync4 } from "fs";
-import { join as join9 } from "path";
+import { join as join11 } from "path";
 import { homedir } from "os";
 async function otherScopeSkillCount(currentDir) {
   const home = homedir();
@@ -3266,7 +3342,7 @@ async function resolveTargetTools(installDir, isGlobal, flags) {
   for (const tool of picked) {
     const dir = TOOL_DIRS[tool];
     if (dir)
-      mkdirSync4(join9(installDir, dir), { recursive: true });
+      mkdirSync4(join11(installDir, dir), { recursive: true });
   }
   return picked;
 }
@@ -3370,6 +3446,11 @@ Installing ${names.length} skills...
         console.error(`Error: ${result.error}`);
         process.exit(1);
       }
+      if (result.warnings) {
+        for (const warning of result.warnings) {
+          console.warn(`\u26A0 ${warning}`);
+        }
+      }
       console.log(`\u2713 Removed '${skillName}'`);
       break;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iceinvein/agent-skills",
-  "version": "0.1.40",
+  "version": "0.3.0",
   "description": "Install agent skills into AI coding tools",
   "author": "iceinvein",
   "license": "MIT",

package/skills/index.json

@@ -219,9 +219,15 @@
   },
   {
     "name": "magpie",
-    "description": "Interactive PR review pipeline. Runs five parallel specialist subagents (security, bugs, performance, code-smells, architecture), dedupes findings, applies a critic rubric, peer-reviews via codex exec (falling back to a Claude second opinion when codex is unavailable), and serves an interactive HTML report for selecting findings to post via gh. Bundles a Bun CLI installed onto PATH via the skill's postinstall step. Use when the user asks to review a GitHub pull request.",
+    "description": "Interactive PR review pipeline. Runs five parallel specialist subagents (security, bugs, performance, code-smells, architecture), dedupes findings, applies a critic rubric, peer-reviews via codex exec (falling back to a Claude second opinion when codex is unavailable), and serves an interactive HTML report for selecting findings to post via gh. Bundles a Bun CLI installed onto PATH via the skill's postinstall step. Use when the user asks to review a GitHub pull request. Splits oversized diffs into budgeted shards and rebuilds the diff from the local clone when gh pr diff refuses it.",
     "type": "prompt",
-    "version": "0.9.0"
+    "version": "0.10.0"
+  },
+  {
+    "name": "migrate",
+    "description": "Source-agnostic legacy migration mapping. Walks a legacy codebase through probe, enumerate, seam, extract, parity, and queue, building an auditable requirements ledger with mandatory citations and a `migrate check` gate in place of self-reported completeness. Use when the user asks to migrate, re-specify, replatform, or map a legacy system onto a new stack, or to resume, check, or report on a mapping run already under way.",
+    "type": "prompt",
+    "version": "0.2.0"
   },
   {
     "name": "module-secret-auditor",
@@ -273,6 +279,12 @@
     ],
     "quick": false
   },
+  {
+    "name": "sluice",
+    "description": "Routes work by change shape into four channels (bypass, fast, main, deep) and applies only the rules each channel needs, so a one-line fix does not pay the cost of a multi-subsystem build. Carries seven rules as one-liners in the router and the full treatment in references read only on friction, and closes each run with a ledger read out of the session transcript: elapsed, tools, tokens, and what every dispatched agent cost. Claude Code only; conflicts with the superpowers plugin.",
+    "type": "prompt",
+    "version": "0.1.1"
+  },
   {
     "name": "temporal-coupling-detector",
     "description": "Hidden ordering dependency analysis: detect two-phase init, method order dependencies, invisible preconditions, and resource lifecycle violations; fix with types, parameters, and factory patterns",