5-phase-workflow 1.7.0 → 1.7.2

package/bin/install.js

@@ -41,7 +41,7 @@ function getInstalledVersion(isGlobal) {
 
   try {
     const data = JSON.parse(fs.readFileSync(versionFile, 'utf8'));
-    return data.installedVersion;
+    return data.packageVersion;
   } catch (e) {
     return null; // Corrupted file, treat as missing
   }
@@ -386,6 +386,20 @@ function selectiveUpdate(targetPath, sourcePath) {
   log.success('Updated templates/ (workflow files only)');
 }
 
+// Ensure .5/.gitignore exists and contains .update-cache.json
+function ensureDotFiveGitignore(dataDir) {
+  const gitignorePath = path.join(dataDir, '.gitignore');
+  const entry = '.update-cache.json';
+  if (fs.existsSync(gitignorePath)) {
+    const content = fs.readFileSync(gitignorePath, 'utf8');
+    if (!content.includes(entry)) {
+      fs.appendFileSync(gitignorePath, '\n' + entry + '\n');
+    }
+  } else {
+    fs.writeFileSync(gitignorePath, entry + '\n');
+  }
+}
+
 // Initialize version.json after successful install
 function initializeVersionJson(isGlobal) {
   const dataDir = getDataPath(isGlobal);
@@ -400,13 +414,13 @@ function initializeVersionJson(isGlobal) {
 
   const versionData = {
     packageVersion: version,
-    installedVersion: version,
     installedAt: now,
     lastUpdated: now,
     installationType: isGlobal ? 'global' : 'local'
   };
 
   fs.writeFileSync(versionFile, JSON.stringify(versionData, null, 2));
+  ensureDotFiveGitignore(dataDir);
   log.success('Initialized version tracking');
 }
 
@@ -554,26 +568,23 @@ function performUpdate(targetPath, sourcePath, isGlobal, versionInfo) {
   // Update version.json
   const dataDir = getDataPath(isGlobal);
   const versionFile = path.join(dataDir, 'version.json');
+  const now = new Date().toISOString();
 
-  let versionData;
-  if (fs.existsSync(versionFile)) {
-    versionData = JSON.parse(fs.readFileSync(versionFile, 'utf8'));
-  } else {
-    // Legacy install, create version.json
-    versionData = {
-      installedAt: new Date().toISOString(),
-      installationType: isGlobal ? 'global' : 'local'
-    };
-  }
-
-  versionData.packageVersion = versionInfo.available;
-  versionData.installedVersion = versionInfo.available;
-  versionData.lastUpdated = new Date().toISOString();
+  const existing = fs.existsSync(versionFile)
+    ? JSON.parse(fs.readFileSync(versionFile, 'utf8'))
+    : {};
+  const versionData = {
+    packageVersion: versionInfo.available,
+    installedAt: existing.installedAt || now,
+    lastUpdated: now,
+    installationType: existing.installationType || (isGlobal ? 'global' : 'local')
+  };
 
   if (!fs.existsSync(dataDir)) {
     fs.mkdirSync(dataDir, { recursive: true });
   }
   fs.writeFileSync(versionFile, JSON.stringify(versionData, null, 2));
+  ensureDotFiveGitignore(dataDir);
 
   // Create features directory if it doesn't exist
   const featuresDir = path.join(dataDir, 'features');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "5-phase-workflow",
-  "version": "1.7.0",
+  "version": "1.7.2",
   "description": "A 5-phase feature development workflow for Claude Code",
   "bin": {
     "5-phase-workflow": "bin/install.js"

package/src/agents/feature-planner.md

@@ -18,6 +18,7 @@ HARD CONSTRAINTS — violations waste tokens and get blocked by plan-guard:
 - NEVER describe HOW something will be implemented (file contents, signatures, class structures)
 - NEVER spawn Task agents with subagent_type other than Explore
 - NEVER write to any file except .5/features/{name}/feature.md and .5/.planning-active
+- NEVER call EnterPlanMode — the workflow has its own planning process
 - The feature spec describes WHAT and WHY, never HOW
 - If you feel the urge to implement, STOP and ask a clarifying question instead
 - Your output is a SPECIFICATION, not a design document. No code. No file layouts. No API shapes.

package/src/agents/implementation-planner.md

@@ -16,6 +16,7 @@ After creating the plan, you are DONE.
 HARD CONSTRAINTS — violations waste tokens and get blocked by plan-guard:
 - NEVER write code, pseudo-code, or implementation snippets
 - NEVER create source files — you create ONE file: plan.md
+- NEVER call EnterPlanMode — the workflow has its own planning process
 - NEVER spawn Task agents with subagent_type other than Explore
 - The plan describes WHAT to build and WHERE. Agents figure out HOW by reading existing code.
 - Each component in the table gets: name, action, file path, one-sentence description, complexity

package/src/hooks/check-updates.js

@@ -40,7 +40,7 @@ async function checkForUpdates(workspaceDir) {
   }
 
   // Compare versions
-  const installed = versionData.installedVersion;
+  const installed = versionData.packageVersion;
   const latestVersion = await getLatestVersion();
 
   let newLatest = null;
@@ -48,16 +48,16 @@ async function checkForUpdates(workspaceDir) {
     newLatest = latestVersion;
   }
 
-  // Only write if latestAvailableVersion actually changed
-  const oldLatest = versionData.latestAvailableVersion || null;
+  // Read/write latestAvailableVersion from .update-cache.json (gitignored)
+  const cacheFile = path.join(path.dirname(versionFile), '.update-cache.json');
+  let cacheData = {};
+  if (fs.existsSync(cacheFile)) {
+    try { cacheData = JSON.parse(fs.readFileSync(cacheFile, 'utf8')); } catch(e) {}
+  }
+  const oldLatest = cacheData.latestAvailableVersion || null;
   if (newLatest !== oldLatest) {
-    versionData.latestAvailableVersion = newLatest;
-
-    // Clean up legacy throttling fields
-    delete versionData.updateCheckLastRun;
-    delete versionData.updateCheckFrequency;
-
-    fs.writeFileSync(versionFile, JSON.stringify(versionData, null, 2));
+    cacheData.latestAvailableVersion = newLatest;
+    fs.writeFileSync(cacheFile, JSON.stringify(cacheData, null, 2));
   }
 
   process.exit(0);

package/src/hooks/plan-guard.js

@@ -20,8 +20,8 @@ process.stdin.on('end', () => {
     const data = JSON.parse(input);
     const toolName = data.tool_name || '';
 
-    // Short-circuit: only check Task, Write, and Edit tools
-    if (toolName !== 'Task' && toolName !== 'Write' && toolName !== 'Edit') {
+    // Short-circuit: only check Task, Write, Edit, and EnterPlanMode tools
+    if (toolName !== 'Task' && toolName !== 'Write' && toolName !== 'Edit' && toolName !== 'EnterPlanMode') {
       process.exit(0);
     }
 
@@ -41,6 +41,20 @@ process.stdin.on('end', () => {
     }
 
     // Planning mode enforcement
+    if (toolName === 'EnterPlanMode') {
+      const blockCount = incrementBlockCount(workspaceDir);
+      const escalation = blockCount >= 3
+        ? ` WARNING: Block #${blockCount}. Repeated violations. Complete your planning artifact and STOP.`
+        : '';
+      process.stderr.write(
+        `BLOCKED: EnterPlanMode is not allowed during workflow planning phases. ` +
+        `The 5-phase workflow has its own planning process. ` +
+        `REDIRECT: Continue with your current planning task. ` +
+        `Write your output to .5/features/{name}/ and output the completion message when done.${escalation}`
+      );
+      process.exit(2);
+    }
+
     if (toolName === 'Task') {
       const agentType = toolInput.subagent_type || '';
       if (agentType && agentType !== 'Explore') {

package/src/hooks/statusline.js

@@ -50,9 +50,16 @@ process.stdin.on('end', () => {
       const versionFile = path.join(dir, '.5', 'version.json');
       const versionData = JSON.parse(fs.readFileSync(versionFile, 'utf8'));
 
-      // Update check
-      const latest = versionData.latestAvailableVersion;
-      const installed = versionData.installedVersion;
+      // Update check — read latestAvailableVersion from cache file (gitignored)
+      const cacheFile = path.join(dir, '.5', '.update-cache.json');
+      let latest = null;
+      if (fs.existsSync(cacheFile)) {
+        try {
+          const cache = JSON.parse(fs.readFileSync(cacheFile, 'utf8'));
+          latest = cache.latestAvailableVersion || null;
+        } catch(e) {}
+      }
+      const installed = versionData.packageVersion;
       if (latest && installed && compareVersions(installed, latest) < 0) {
         updateIndicator = ` | \x1b[33m↑${latest} → /5:update\x1b[0m`;
       }

package/src/settings.json

@@ -38,7 +38,7 @@
         ]
       },
       {
-        "matcher": "Task|Write|Edit",
+        "matcher": "Task|Write|Edit|EnterPlanMode",
         "hooks": [
           {
             "type": "command",