5-phase-workflow 1.2.0 → 1.2.1

package/package.json

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

package/src/commands/5/plan-feature.md

@@ -101,9 +101,14 @@ Use Task tool with subagent_type=Explore for complex exploration.
 
 **Goal:** Understand what already exists so you can ask informed questions.
 
-### Step 4: Intensive Collaboration (5-10 Questions)
+### Step 4: Intensive Collaboration (5-10 Questions, ONE AT A TIME)
 
-**CRITICAL:** After exploring the codebase, engage in intensive Q&A using the AskUserQuestion tool. Ask 5-10 clarifying questions based on your findings. This is NOT optional.
+**CRITICAL:** After exploring the codebase, engage in intensive Q&A. This is NOT optional.
+
+- Ask 5-10 clarifying questions using AskUserQuestion
+- **ONE question at a time** - wait for answer before next question
+- Do NOT list multiple questions in one message
+- Do NOT skip to creating feature.md before asking at least 5 questions
 
 **Question categories to explore:**
 
@@ -154,7 +159,7 @@ Use Task tool with subagent_type=Explore for complex exploration.
 - "Could we use existing Z component instead?"
 - "Is a full factory needed or just simple creation?"
 
-Use AskUserQuestion to present options and trade-offs. Multiple questions can be asked in batches.
+**Ask questions ONE AT A TIME.** Use AskUserQuestion for each question. For clarifying questions, provide 2-4 options where meaningful. Wait for the user's answer before asking the next question. Open questions (like feature description) can use free text.
 
 ### Step 5: Determine Feature Name
 

package/src/commands/5/plan-implementation.md

@@ -39,7 +39,9 @@ Understand the project structure:
 
 This is a quick scan, not deep analysis. The executor agent will do detailed pattern matching.
 
-### Step 3: Ask 2-3 Technical Questions
+### Step 3: Ask 2-3 Technical Questions (ONE AT A TIME)
+
+**CRITICAL:** Ask questions ONE AT A TIME. Wait for the user's answer before asking the next question.
 
 Use AskUserQuestion to clarify:
 - Data layer decisions (if applicable)
@@ -48,7 +50,11 @@ Use AskUserQuestion to clarify:
 - Software Architecture
 - Testing behaviour
 
-Keep it brief. Don't over-question. Don't ask feature questions already answered in the plan-feature phase. 
+Keep it brief. Don't over-question. Don't ask feature questions already answered in the plan-feature phase.
+
+- **ONE question at a time** - wait for answer before next question
+- Do NOT list multiple questions in one message
+- Do NOT skip to creating plan.md before asking your questions 
 
 ### Step 4: Design Components
 
@@ -212,3 +218,5 @@ Components in the same step run in parallel. Structure your plan accordingly:
 - Don't over-analyze the codebase - the executor will do detailed pattern matching
 - Don't ask more than 3 questions
 - Don't create unnecessary sequential steps - group independent work together
+- **Don't skip to implementation** - This command ONLY creates the plan
+- **Don't batch questions** - Ask one question at a time

package/src/commands/5/quick-implement.md

@@ -84,6 +84,8 @@ feature_name="${TICKET_ID}-${slug}"
 - Which existing patterns to follow?
 - Any edge cases to handle?
 
+**Ask questions ONE AT A TIME.** Wait for the user's answer before asking the next question. Do NOT list multiple questions in one message.
+
 ### Step 5: Create Plan
 
 Write plan to `.5/${feature_name}/plan.md` using the template structure.

package/src/hooks/check-updates.js

@@ -9,20 +9,20 @@ process.stdin.on('data', (chunk) => {
   inputData += chunk;
 });
 
-process.stdin.on('end', () => {
+process.stdin.on('end', async () => {
   try {
     // Parse hook input (contains workspace info)
     const hookData = JSON.parse(inputData);
     const workspaceDir = hookData.workingDirectory || process.cwd();
 
-    checkForUpdates(workspaceDir);
+    await checkForUpdates(workspaceDir);
   } catch (e) {
     // Silent failure - don't block on errors
     process.exit(0);
   }
 });
 
-function checkForUpdates(workspaceDir) {
+async function checkForUpdates(workspaceDir) {
   const versionFile = path.join(workspaceDir, '.claude', '.5', 'version.json');
 
   // Check if version.json exists
@@ -57,38 +57,53 @@ function checkForUpdates(workspaceDir) {
 
   // Compare versions
   const installed = versionData.installedVersion;
-  const packageVersion = getPackageVersion(workspaceDir);
+  const latestVersion = await getLatestVersion();
 
-  if (!packageVersion || installed === packageVersion) {
+  if (!latestVersion || installed === latestVersion) {
     // No update available
     process.exit(0);
   }
 
-  // Check if update is available (installed < package)
-  if (compareVersions(installed, packageVersion) < 0) {
+  // Check if update is available (installed < latest)
+  if (compareVersions(installed, latestVersion) < 0) {
     // Show update notification
-    console.log(`\n\x1b[34mℹ\x1b[0m Update available: ${installed} → ${packageVersion}`);
+    console.log(`\n\x1b[34mℹ\x1b[0m Update available: ${installed} → ${latestVersion}`);
     console.log(`  Run: \x1b[1mnpx 5-phase-workflow --upgrade\x1b[0m\n`);
   }
 
   process.exit(0);
 }
 
-// Get package version from local package.json
-function getPackageVersion(workspaceDir) {
-  // Try to find package.json in node_modules/5-phase-workflow
-  const pkgPath = path.join(workspaceDir, 'node_modules', '5-phase-workflow', 'package.json');
-
-  if (fs.existsSync(pkgPath)) {
-    try {
-      const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
-      return pkg.version;
-    } catch (e) {
-      return null;
-    }
-  }
-
-  return null;
+// Get latest version from npm registry
+async function getLatestVersion() {
+  return new Promise((resolve) => {
+    const https = require('https');
+    const req = https.get(
+      'https://registry.npmjs.org/5-phase-workflow/latest',
+      { timeout: 3000 },
+      (res) => {
+        if (res.statusCode !== 200) {
+          resolve(null);
+          return;
+        }
+        let data = '';
+        res.on('data', (chunk) => (data += chunk));
+        res.on('end', () => {
+          try {
+            const pkg = JSON.parse(data);
+            resolve(pkg.version);
+          } catch (e) {
+            resolve(null);
+          }
+        });
+      }
+    );
+    req.on('error', () => resolve(null));
+    req.on('timeout', () => {
+      req.destroy();
+      resolve(null);
+    });
+  });
 }
 
 // Compare semver versions