@spardutti/claude-skills 1.24.1 → 1.26.0

package/README.md

@@ -79,6 +79,7 @@ Portable slash commands for common git workflows. Installed to `.claude/commands
 | `/release` | Release flow — dev→main PR with semver, changelog, tag, and GitHub release |
 | `/refactor` | Detect size/complexity/duplication/coupling issues via 4 parallel Haiku subagents, then refactor |
 | `/deep-review` | Multi-agent deep code review — 5 parallel Sonnet subagents catch guard bypasses, lost async state, wrong-table queries, dead references, protocol violations |
+| `/plan-feature` | Integration-first feature planning — 3 parallel Haiku subagents scan for reusable code, established patterns, and touch points before producing a short integration plan |
 
 ## Quick Start
 

package/lib/setup-claude-md.mjs

@@ -1,65 +1,121 @@
 import { readFile, writeFile } from "node:fs/promises";
 import { join, resolve } from "node:path";
 
-const SKILL_EVAL_BLOCK = `
-skill_evaluation:
-  mandatory: true
-  rule: |
-    BEFORE writing ANY code, you MUST:
-    1. List EVERY skill from the system-reminder's available skills section
-    2. For each skill, write: [skill-name] → ACTIVATE / SKIP — [one-line reason]
-    3. Call Skill(name) for every skill marked ACTIVATE
-    4. Only THEN proceed to implementation
-    If you skip this evaluation, your response is INCOMPLETE and WRONG.`;
-
-const FILE_SIZE_BLOCK = `
-## File Size Enforcement
+const SKILL_BODY = `## Skills
+
+BEFORE writing ANY code, you MUST:
+
+1. List EVERY skill available: check \`.claude/skills/\` (project) and \`~/.claude/skills/\` (global). The system-reminder's available-skills section is a hint, not the source of truth — if it's missing or empty, still check the directories.
+2. For each skill, write: [skill-name] → ACTIVATE / SKIP — [one-line reason]
+3. Call Skill(name) for every skill marked ACTIVATE
+4. Only THEN proceed to implementation
+
+If you skip this evaluation, your response is INCOMPLETE and WRONG.`;
+
+const FILE_SIZE_BODY = `## File Size Enforcement
 
 - **Never write a file longer than 200 lines of code.** If a file would exceed 200 lines, split it into smaller modules before writing.
 - This rule applies during skill evaluation: if the code you're about to write would exceed 200 lines in any single file, refactor into multiple files first.
 - Skill evaluation must check this limit as part of every ACTIVATE decision.`;
 
-const EVAL_MARKER = "skill_evaluation:";
-const FILE_SIZE_MARKER = "## File Size Enforcement";
+const BLOCKS = [
+  {
+    id: "skill-evaluation",
+    body: SKILL_BODY,
+    legacyHeadings: ["## Skills"],
+    legacyYamlMarker: "skill_evaluation:",
+  },
+  {
+    id: "file-size",
+    body: FILE_SIZE_BODY,
+    legacyHeadings: ["## File Size Enforcement"],
+  },
+];
 
-export async function setupClaudeMd(targetDir = process.cwd()) {
-  const resolved = resolve(targetDir);
-  const claudeMdPath = join(resolved, "CLAUDE.md");
+function wrap(id, body) {
+  return `<!-- claude-skills:${id}:start -->\n${body}\n<!-- claude-skills:${id}:end -->`;
+}
 
-  let existing = "";
-  try {
-    existing = await readFile(claudeMdPath, "utf-8");
-  } catch {
-    // File doesn't exist — will create
-  }
+function spliceSentinels(content, id, replacement) {
+  const start = `<!-- claude-skills:${id}:start -->`;
+  const end = `<!-- claude-skills:${id}:end -->`;
+  const startIdx = content.indexOf(start);
+  if (startIdx === -1) return null;
+  const endIdx = content.indexOf(end, startIdx);
+  if (endIdx === -1) return null;
+  return content.slice(0, startIdx) + replacement + content.slice(endIdx + end.length);
+}
 
-  const hasEval = existing.includes(EVAL_MARKER);
-  const hasFileSize = existing.includes(FILE_SIZE_MARKER);
+function spliceLegacyHeading(content, heading, replacement) {
+  const lines = content.split("\n");
+  const startIdx = lines.findIndex((l) => l.trim() === heading);
+  if (startIdx === -1) return null;
+  let endIdx = lines.length;
+  for (let i = startIdx + 1; i < lines.length; i++) {
+    if (lines[i].startsWith("## ")) {
+      endIdx = i;
+      break;
+    }
+  }
+  return joinSplice(lines, startIdx, endIdx, replacement);
+}
 
-  if (hasEval && hasFileSize) {
-    console.log("  CLAUDE.md already has skill_evaluation and file size rules — skipped.");
-    return;
+function spliceLegacyYaml(content, marker, replacement) {
+  const lines = content.split("\n");
+  const startIdx = lines.findIndex((l) => l.startsWith(marker));
+  if (startIdx === -1) return null;
+  let endIdx = lines.length;
+  for (let i = startIdx + 1; i < lines.length; i++) {
+    const line = lines[i];
+    if (line.length > 0 && !/^\s/.test(line)) {
+      endIdx = i;
+      break;
+    }
   }
+  return joinSplice(lines, startIdx, endIdx, replacement);
+}
 
-  let content = existing.trimEnd();
+function joinSplice(lines, startIdx, endIdx, replacement) {
+  const before = lines.slice(0, startIdx).join("\n").replace(/\s+$/, "");
+  const after = lines.slice(endIdx).join("\n").replace(/^\s+/, "");
+  const parts = [before, replacement, after].filter((s) => s.length > 0);
+  return parts.join("\n\n");
+}
 
-  if (!hasEval) {
-    content = content.length > 0
-      ? content + "\n" + SKILL_EVAL_BLOCK
-      : SKILL_EVAL_BLOCK.trimStart();
+function applyBlock(content, block) {
+  const wrapped = wrap(block.id, block.body);
+  let next = spliceSentinels(content, block.id, wrapped);
+  if (next !== null) return { content: next, action: `replaced ${block.id}` };
+  for (const heading of block.legacyHeadings ?? []) {
+    next = spliceLegacyHeading(content, heading, wrapped);
+    if (next !== null) return { content: next, action: `migrated ${block.id}` };
   }
-
-  if (!hasFileSize) {
-    content = content + "\n" + FILE_SIZE_BLOCK;
+  if (block.legacyYamlMarker) {
+    next = spliceLegacyYaml(content, block.legacyYamlMarker, wrapped);
+    if (next !== null) return { content: next, action: `migrated ${block.id}` };
   }
+  const base = content.replace(/\s+$/, "");
+  const merged = base.length > 0 ? base + "\n\n" + wrapped : wrapped;
+  return { content: merged, action: `added ${block.id}` };
+}
+
+export async function setupClaudeMd(targetDir = process.cwd()) {
+  const claudeMdPath = join(resolve(targetDir), "CLAUDE.md");
 
-  await writeFile(claudeMdPath, content + "\n", { mode: 0o644 });
+  let content = "";
+  try {
+    content = await readFile(claudeMdPath, "utf-8");
+  } catch {
+    // File doesn't exist — will create
+  }
 
-  if (!hasEval && !hasFileSize) {
-    console.log("  CLAUDE.md updated with skill_evaluation and file size rules.");
-  } else if (!hasEval) {
-    console.log("  CLAUDE.md updated with skill_evaluation block.");
-  } else {
-    console.log("  CLAUDE.md updated with file size enforcement rule.");
+  const actions = [];
+  for (const block of BLOCKS) {
+    const result = applyBlock(content, block);
+    content = result.content;
+    actions.push(result.action);
   }
+
+  await writeFile(claudeMdPath, content.replace(/\s+$/, "") + "\n", { mode: 0o644 });
+  console.log(`  CLAUDE.md: ${actions.join(", ")}.`);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spardutti/claude-skills",
-  "version": "1.24.1",
+  "version": "1.26.0",
   "description": "CLI to install Claude Code skills from the claude-skills collection",
   "type": "module",
   "bin": {