claude-devkit-cli 1.2.0 → 1.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-devkit-cli",
-  "version": "1.2.0",
+  "version": "1.2.3",
   "description": "CLI toolkit for spec-first development with Claude Code — hooks, commands, guards, and test runners",
   "bin": {
     "claude-devkit": "./bin/devkit.js",

package/src/commands/upgrade.js

@@ -83,10 +83,25 @@ export async function upgradeCommand(path, opts) {
     updated++;
   }
 
-  // Check for files in manifest that no longer exist in kit
+  // Remove files in manifest that no longer exist in kit
+  let removed = 0;
   for (const file of Object.keys(manifest.files)) {
     if (!allFiles.includes(file)) {
-      log.warn(`${file} — no longer in kit (keeping)`);
+      const filePath = resolve(targetDir, file);
+      if (!opts.dryRun) {
+        try {
+          const { unlink } = await import('node:fs/promises');
+          await unlink(filePath);
+          delete manifest.files[file];
+          removed++;
+          log.del(file);
+        } catch {
+          log.warn(`${file} — no longer in kit (could not delete)`);
+        }
+      } else {
+        log.del(`${file} (would remove)`);
+        removed++;
+      }
     }
   }
 
@@ -100,7 +115,9 @@ export async function upgradeCommand(path, opts) {
 
   // Summary
   log.blank();
-  log.pass(`Updated ${updated}, added ${added}, unchanged ${unchanged}, skipped ${skippedCustomized} customized.`);
+  const parts = [`Updated ${updated}`, `added ${added}`, `removed ${removed}`, `unchanged ${unchanged}`];
+  if (skippedCustomized > 0) parts.push(`skipped ${skippedCustomized} customized`);
+  log.pass(parts.join(', ') + '.');
 
   if (skippedCustomized > 0) {
     log.warn(`${skippedCustomized} customized file(s) skipped. Run with --force to overwrite.`);

package/templates/.claude/hooks/self-review.sh

@@ -21,9 +21,6 @@ cat > /dev/null 2>&1 || true
 cat <<'REVIEW_JSON'
 {
   "continue": true,
-  "hookSpecificOutput": {
-    "hookEventName": "Stop",
-    "additionalContext": "Self-review before finishing:\n1. Did you leave any TODO/FIXME comments that should be resolved now?\n2. Did you create mock or fake implementations just to pass tests?\n3. Did you replace real code with placeholder comments like '// ... existing code'?\n4. Do all changed files compile and typecheck cleanly?\n5. Did you run the full test suite, not just the new tests?\n6. Are there any files you modified but forgot to include in the summary?"
-  }
+  "systemMessage": "Self-review before finishing:\n1. Did you leave any TODO/FIXME comments that should be resolved now?\n2. Did you create mock or fake implementations just to pass tests?\n3. Did you replace real code with placeholder comments like '// ... existing code'?\n4. Do all changed files compile and typecheck cleanly?\n5. Did you run the full test suite, not just the new tests?\n6. Are there any files you modified but forgot to include in the summary?"
 }
 REVIEW_JSON