jettypod 4.4.77 → 4.4.79

package/jettypod.js

@@ -705,6 +705,77 @@ async function generateClaude(options = {}) {
   console.log('📝 CLAUDE.md generated');
 }
 
+// Ensure Claude Code hooks are up to date (called on every jettypod launch)
+function ensureClaudeHooks() {
+  // Create .claude directory if needed
+  if (!fs.existsSync('.claude')) {
+    fs.mkdirSync('.claude', { recursive: true });
+  }
+
+  const claudeSettingsPath = path.join('.claude', 'settings.json');
+
+  // Required hooks configuration - always ensure these are present
+  const requiredHooks = {
+    Edit: [
+      '.jettypod/hooks/global-guardrails.js',
+      '.jettypod/hooks/protect-claude-md.js'
+    ],
+    Write: [
+      '.jettypod/hooks/global-guardrails.js',
+      '.jettypod/hooks/protect-claude-md.js'
+    ],
+    Bash: [
+      '.jettypod/hooks/global-guardrails.js',
+      '.jettypod/hooks/enforce-skill-activation.js'
+    ]
+  };
+
+  // Load existing settings or create empty structure
+  let claudeSettings = { hooks: { PreToolUse: [] } };
+  if (fs.existsSync(claudeSettingsPath)) {
+    try {
+      claudeSettings = JSON.parse(fs.readFileSync(claudeSettingsPath, 'utf-8'));
+      if (!claudeSettings.hooks) claudeSettings.hooks = {};
+      if (!claudeSettings.hooks.PreToolUse) claudeSettings.hooks.PreToolUse = [];
+    } catch {
+      // Invalid JSON, start fresh
+      claudeSettings = { hooks: { PreToolUse: [] } };
+    }
+  }
+
+  // Ensure required hooks are present for each matcher
+  let hooksUpdated = false;
+  for (const [matcher, requiredCommands] of Object.entries(requiredHooks)) {
+    // Find existing entry for this matcher
+    let matcherEntry = claudeSettings.hooks.PreToolUse.find(e => e.matcher === matcher);
+
+    if (!matcherEntry) {
+      // Create new entry for this matcher
+      matcherEntry = { matcher, hooks: [] };
+      claudeSettings.hooks.PreToolUse.push(matcherEntry);
+      hooksUpdated = true;
+    }
+
+    // Ensure each required command is present
+    for (const command of requiredCommands) {
+      const hasCommand = matcherEntry.hooks.some(h => h.command === command);
+      if (!hasCommand) {
+        // Add required hook at the beginning (guardrails should run first)
+        matcherEntry.hooks.unshift({ type: 'command', command });
+        hooksUpdated = true;
+      }
+    }
+  }
+
+  // Write updated settings
+  fs.writeFileSync(claudeSettingsPath, JSON.stringify(claudeSettings, null, 2));
+  if (hooksUpdated) {
+    console.log('⚙️  Claude Code hooks updated');
+  }
+
+  return hooksUpdated;
+}
+
 // Initialize project (used by both 'jettypod init' and 'jettypod' with no args)
 async function initializeProject() {
   const { showLogo } = require('./lib/terminal-logo');
@@ -805,74 +876,11 @@ async function initializeProject() {
     fs.chmodSync(guardrailsHookDest, 0o755);
   }
 
-  // Create Claude Code settings
-  if (!fs.existsSync('.claude')) {
-    fs.mkdirSync('.claude', { recursive: true });
-  }
-
   // Ensure .claude/session.md is gitignored and untracked
   ensureJettypodGitignores();
 
-  const claudeSettingsPath = path.join('.claude', 'settings.json');
-
-  // Required hooks configuration - always ensure these are present
-  const requiredHooks = {
-    Edit: [
-      '.jettypod/hooks/global-guardrails.js',
-      '.jettypod/hooks/protect-claude-md.js'
-    ],
-    Write: [
-      '.jettypod/hooks/global-guardrails.js',
-      '.jettypod/hooks/protect-claude-md.js'
-    ],
-    Bash: [
-      '.jettypod/hooks/global-guardrails.js',
-      '.jettypod/hooks/enforce-skill-activation.js'
-    ]
-  };
-
-  // Load existing settings or create empty structure
-  let claudeSettings = { hooks: { PreToolUse: [] } };
-  if (fs.existsSync(claudeSettingsPath)) {
-    try {
-      claudeSettings = JSON.parse(fs.readFileSync(claudeSettingsPath, 'utf-8'));
-      if (!claudeSettings.hooks) claudeSettings.hooks = {};
-      if (!claudeSettings.hooks.PreToolUse) claudeSettings.hooks.PreToolUse = [];
-    } catch {
-      // Invalid JSON, start fresh
-      claudeSettings = { hooks: { PreToolUse: [] } };
-    }
-  }
-
-  // Ensure required hooks are present for each matcher
-  let hooksUpdated = false;
-  for (const [matcher, requiredCommands] of Object.entries(requiredHooks)) {
-    // Find existing entry for this matcher
-    let matcherEntry = claudeSettings.hooks.PreToolUse.find(e => e.matcher === matcher);
-
-    if (!matcherEntry) {
-      // Create new entry for this matcher
-      matcherEntry = { matcher, hooks: [] };
-      claudeSettings.hooks.PreToolUse.push(matcherEntry);
-      hooksUpdated = true;
-    }
-
-    // Ensure each required command is present
-    for (const command of requiredCommands) {
-      const hasCommand = matcherEntry.hooks.some(h => h.command === command);
-      if (!hasCommand) {
-        // Add required hook at the beginning (guardrails should run first)
-        matcherEntry.hooks.unshift({ type: 'command', command });
-        hooksUpdated = true;
-      }
-    }
-  }
-
-  // Write updated settings
-  fs.writeFileSync(claudeSettingsPath, JSON.stringify(claudeSettings, null, 2));
-  if (hooksUpdated) {
-    console.log('⚙️  Claude Code hooks updated');
-  }
+  // Ensure Claude Code hooks are up to date
+  ensureClaudeHooks();
 
   // Copy skills directory
   // Allow tests to override skills source directory
@@ -1163,8 +1171,11 @@ switch (command) {
     const updateCommand = require('./lib/update-command');
     const success = await updateCommand.runUpdate();
 
-    // Always refresh skills in current project after update attempt
+    // Always refresh skills and hooks in current project after update attempt
     if (fs.existsSync('.jettypod')) {
+      // Update Claude Code hooks first
+      ensureClaudeHooks();
+
       console.log('');
       console.log('🔄 Refreshing skills in current project...');
 
@@ -2099,6 +2110,9 @@ switch (command) {
       const currentConfig = config.read();
       await generateClaude();
 
+      // Ensure Claude Code hooks are up to date on every launch
+      ensureClaudeHooks();
+
       // Launch dashboard
       const dashboardPath = path.join(__dirname, 'apps', 'dashboard');
       const BASE_PORT = 3456;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jettypod",
-  "version": "4.4.77",
+  "version": "4.4.79",
   "description": "AI-powered development workflow manager with TDD, BDD, and automatic test generation",
   "main": "jettypod.js",
   "bin": {

package/skills-templates/chore-planning/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: chore-planning
-description: Guide standalone chore planning with automatic type classification, taxonomy guidance loading, and routing to chore-mode for execution. Use when user starts a standalone chore (chore with no parent feature), mentions planning a chore, or says "help me with this chore".
+description: Guide standalone chore planning with automatic type classification and routing to chore-mode. Use when the implementation approach is obvious - refactoring, bug fixes, infrastructure, or simple enhancements where there's no UX decision to make. Key question - "Does this need UX exploration?" No → chore-planning. Yes → feature-planning instead.
 ---
 
 # Chore Planning Skill

package/skills-templates/feature-planning/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: feature-planning
-description: Guide feature planning with UX approach exploration and BDD scenario generation. Use when user asks to plan a feature, mentions designing/implementing a feature, says "help me plan feature", or wants to explore UX approaches for a specific feature.
+description: Guide feature planning with UX approach exploration and BDD scenario generation. Use when implementing NEW user-facing behavior that requires UX decisions (multiple valid approaches to discuss). NOT for simple/obvious changes like copy tweaks, styling fixes, or adding straightforward functionality. Key question - "Does this need UX exploration?" Yes → feature-planning.
 ---
 
 # Feature Planning Skill