@iceinvein/agent-skills 0.1.39 → 0.2.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. Six rules live as one-liners in the router; the full treatment sits in references read only on friction. 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

@@ -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;
@@ -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 rmSync4 } 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()) {
+    rmSync4(fullPath, { recursive: true, force: true });
+  } else {
+    unlinkSync4(fullPath);
+  }
+}
 async function removeSkill(cwd, skillName) {
   const lockfile = await readLockfile(cwd);
   const entry = lockfile.skills[skillName];
@@ -993,18 +1021,28 @@ async function removeSkill(cwd, skillName) {
       });
       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 +1064,30 @@ async function infoSkill(skillName) {
 }
 
 // src/cli/commands/remove.ts
+import { existsSync as existsSync7, statSync as statSync3, unlinkSync as unlinkSync5, rmSync as rmSync5 } 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()) {
+    rmSync5(fullPath, { recursive: true, force: true });
+  } else {
+    unlinkSync5(fullPath);
+  }
+}
 async function removeSkill2(cwd, skillName) {
   const lockfile = await readLockfile(cwd);
   const entry = lockfile.skills[skillName];
@@ -1050,18 +1112,28 @@ async function removeSkill2(cwd, skillName) {
       });
       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 +1203,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 +1224,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 +1295,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 +1351,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 +3234,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 +3338,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 +3442,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.39",
+  "version": "0.2.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.8.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 six rules as one-liners in the router and the full treatment in references read only on friction. Claude Code only; conflicts with the superpowers plugin.",
+    "type": "prompt",
+    "version": "1.0.0"
+  },
   {
     "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",

package/skills/magpie/README.md

@@ -12,6 +12,7 @@ Given a GitHub PR number, dispatches five specialist subagents in parallel (secu
 - `gh` on PATH, authenticated (`gh auth status`)
 - `git` on PATH
 - `codex` on PATH, authenticated (optional; if absent the peer-review stage falls back to a Claude second-opinion subagent)
+- the code-intelligence MCP server, with a completed index for the repo under review (optional; if absent the specialists review from the diff and worktree alone)
 
 ## Install
 
@@ -54,7 +55,7 @@ The fixture lives at `fixtures/example-pr/` (pr.json + findings.final.json + pos
 ## Layout
 
 - `SKILL.md` is the agent-facing prompt: the stage walkthrough and nothing else. Installed by the agent-skills CLI.
-- `references/` holds the prompt bodies the walkthrough loads on demand, one file per stage that needs one: `specialists.md` (stage 3, the five focus blocks plus the shared output contract), `critic.md` (stage 5), `peer-review.md` (stage 6, including the Claude-fallback preamble). They ship in the bundle and sit next to `SKILL.md` once installed.
+- `references/` holds the prompt bodies the walkthrough loads on demand, one file per stage that needs one: `scout.md` (stage 3, the PR-brief prompt and the `brief.json` contract), `specialists.md` (stage 4, the five focus blocks plus the shared output contract and the codebase-intelligence block), `critic.md` (stage 6), `peer-review.md` (stage 7, including the Claude-fallback preamble). They ship in the bundle and sit next to `SKILL.md` once installed.
 - `skill.json` is the agent-skills manifest.
 - `bin/magpie` is the CLI invoked by the agent during stages; symlinked onto PATH by `install.sh`.
 - `scripts/` holds the implementation (server, dedupe, render, setup, cleanup, etc.).