5-phase-workflow 1.7.0 → 1.7.1

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.1",
   "description": "A 5-phase feature development workflow for Claude Code",
   "bin": {
     "5-phase-workflow": "bin/install.js"

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/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`;
       }