5-phase-workflow 1.1.2 → 1.2.1

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

package/src/skills/configure-project/SKILL.md

@@ -57,7 +57,7 @@ It handles three distinct tasks, invoked with different parameters per component
       "type": null
     }
   },
-  "reviewTool": "coderabbit" or "none"
+  "reviewTool": "claude" or "coderabbit" or "none"
 }
 ```
 

package/src/templates/workflow/FEATURE-SPEC.md

@@ -0,0 +1,83 @@
+# Feature: {TICKET-ID} - {Title}
+
+## Ticket ID
+{TICKET-ID}
+
+## Summary
+{1-2 sentence overview of what will be implemented}
+
+## Problem Statement
+{Why is this feature needed? What problem does it solve?}
+
+## Requirements
+
+### Functional Requirements
+- {Requirement 1}
+- {Requirement 2}
+- ...
+
+### Non-Functional Requirements
+- {Performance requirements}
+- {Compatibility requirements}
+- ...
+
+## Constraints
+- {Business constraints}
+- {Technical constraints}
+- {Time/resource constraints}
+
+## Affected Components
+- **{component/module-1}** - {What changes here}
+- **{component/module-2}** - {What changes here}
+- **{component/module-3}** - {What changes here}
+- ...
+
+## Entity/Component Definitions
+
+### {EntityName} (if applicable)
+| Field | Type | Required | Description |
+|-------|------|----------|-------------|
+| id | {Entity}Id | Yes | Unique identifier |
+| name | String | Yes | Entity name |
+| ... | ... | ... | ... |
+
+### Business Rules
+- {Rule 1}
+- {Rule 2}
+- ...
+
+## Acceptance Criteria
+- [ ] {Criterion 1 - how to verify success}
+- [ ] {Criterion 2}
+- [ ] {Criterion 3}
+- ...
+
+## Alternatives Considered
+
+### Option 1: {Alternative approach}
+**Pros:** {Benefits}
+**Cons:** {Drawbacks}
+**Decision:** Rejected because {reason}
+
+### Option 2: {Another alternative}
+**Pros:** {Benefits}
+**Cons:** {Drawbacks}
+**Decision:** Rejected because {reason}
+
+### Chosen Approach: {Selected approach}
+**Rationale:** {Why this approach was chosen}
+
+## Questions & Answers
+
+### Q1: {Question from collaboration phase}
+**A:** {Answer from developer}
+
+### Q2: {Question}
+**A:** {Answer}
+
+...
+
+## Next Steps
+After approval:
+1. Run `/clear` to reset context
+2. Run `/5:plan-implementation {TICKET-ID}-{description}`

package/src/templates/workflow/FIX-PLAN.md

@@ -0,0 +1,55 @@
+# Fix Plan: {TICKET-ID}
+
+**Feature:** {feature-name}
+**Generated:** {timestamp}
+**Verification Status:** PARTIAL | FAILED
+
+---
+
+## Infrastructure Fixes
+
+{If no infrastructure issues: "No infrastructure issues found."}
+
+| # | File | Issue | Fix | Complexity |
+|---|------|-------|-----|------------|
+| 1 | {path} | {description} | {what to do} | simple / moderate / complex |
+
+## Feature Gap Fixes
+
+{If no feature gaps or Layer 2 was skipped: "No feature gaps found." or "Skipped — no feature.md available."}
+
+| # | File | Issue | Fix | Complexity |
+|---|------|-------|-----|------------|
+| 1 | {path} | {criterion or requirement not met} | {what to change} | simple / moderate / complex |
+
+## Quality Fixes
+
+{If no quality issues: "No quality issues found."}
+
+| # | File | Issue | Fix | Complexity |
+|---|------|-------|-----|------------|
+| 1 | {test-file-path} | No test file for {component} | Create test file | simple / moderate |
+
+---
+
+## Summary
+
+| Category | Count |
+|----------|-------|
+| Infrastructure | {N} |
+| Feature Gaps | {N} |
+| Quality | {N} |
+| **Total** | **{N}** |
+
+---
+
+## Results
+
+{Populated after fixes are applied}
+
+| # | File | Fix | Status |
+|---|------|----|--------|
+| 1 | {path} | {description} | APPLIED / FAILED |
+
+**Build after fixes:** {status}
+**Tests after fixes:** {status}

package/src/templates/workflow/PLAN.md

@@ -0,0 +1,30 @@
+---
+ticket: {TICKET-ID}
+feature: {feature-name}
+created: {ISO-timestamp}
+---
+
+# Implementation Plan: {TICKET-ID}
+
+{One sentence summary of what this plan builds}
+
+## Components
+
+| Step | Component | Action | File | Description | Complexity |
+|------|-----------|--------|------|-------------|------------|
+| 1 | {name} | create | {path} | {what it does} | simple |
+| 1 | {name} | create | {path} | {what it does} | simple |
+| 2 | {name} | create | {path} | {what it does} | moderate |
+| 2 | {name} | modify | {path} | {what to change} | moderate |
+| 3 | {name} | create | {path} | {what it does} | complex |
+
+## Implementation Notes
+
+- Follow the pattern from {existing-file} for {component-type}
+- {Key business rule to remember}
+- {Integration point to wire up}
+
+## Verification
+
+- Build: {command or "auto"}
+- Test: {command or "auto"}

package/src/templates/workflow/QUICK-PLAN.md

@@ -0,0 +1,17 @@
+# Quick Implementation: {TICKET-ID}
+
+## Task
+{DESCRIPTION}
+
+## Components
+
+| # | Type | Name | Skill | Module |
+|---|------|------|-------|--------|
+| 1 | {type} | {name} | {skill} | {module} |
+
+## Affected Modules
+- {module-1}
+- {module-2}
+
+## Execution
+{parallel | sequential | direct}

package/src/templates/workflow/REVIEW-FINDINGS.md

@@ -0,0 +1,58 @@
+# Code Review Findings
+
+**Generated:** {ISO-timestamp}
+**Scope:** {what was reviewed}
+**Total Findings:** {N}
+
+---
+
+## How to Use This File
+
+1. Review each finding below
+2. For each finding, choose an action:
+   - `[FIX]` - Apply this fix automatically (leave as-is)
+   - `[SKIP]` - Don't apply this fix (change FIX to SKIP)
+   - `[MANUAL]` - Custom instructions (change FIX to MANUAL and add instructions)
+3. Save this file
+4. Run: `/review-code apply`
+
+The apply command will read your annotations and apply marked fixes.
+
+---
+
+## Finding {N}/{total}
+
+**File:** {file-path}
+**Line:** {line-number}
+**Category:** {Fixable|Question|Manual}
+**Severity:** {error|warning|suggestion}
+
+**Description:**
+{what the reviewer found}
+
+**Suggested Fix:**
+{how to fix it - can be multi-line}
+
+**Original Reviewer Message:**
+```
+{raw output from reviewer}
+```
+
+**Action:** [FIX]
+
+**Custom Instructions:** (only if you selected [MANUAL])
+<!-- Add detailed instructions here if you want a custom fix -->
+
+
+---
+
+## Summary
+
+- Total: {N}
+- Fixable: {N}
+- Questions: {N}
+- Manual Review: {N}
+
+**Next Steps:**
+1. Edit this file to mark which findings to fix
+2. Run: `/review-code apply`

package/src/templates/workflow/REVIEW-SUMMARY.md

@@ -0,0 +1,35 @@
+# Code Review Summary
+
+**Reviewed:** {scope}
+**Timestamp:** {ISO-timestamp}
+**User Decisions:** {summary of user choices}
+
+## Summary
+
+- **Total Issues:** {N}
+- **Applied with User Approval:** {N}
+- **User-Resolved Questions:** {N}
+- **Manual Review Needed:** {N}
+- **Skipped by User:** {N}
+
+## Applied Fixes (User Approved)
+
+- `{file-path}:{line}` - {description of fix}
+- `{file-path}:{line}` - {description of fix}
+
+## User-Resolved Questions
+
+- `{file-path}:{line}` - {description} (user answered: {answer})
+
+## Manual Review Needed
+
+- `{file-path}:{line}` - {description}
+
+## Skipped Issues
+
+- `{file-path}:{line}` - User chose not to apply
+
+## Files Modified
+
+- {file-path} ({N} fixes applied)
+- {file-path} ({N} fixes applied)

package/src/templates/workflow/STATE.json

@@ -0,0 +1,9 @@
+{
+  "ticket": "{TICKET-ID}",
+  "feature": "{feature-name}",
+  "status": "in-progress",
+  "currentStep": 1,
+  "completed": [],
+  "failed": [],
+  "startedAt": "{ISO-timestamp}"
+}

package/src/templates/workflow/VERIFICATION-REPORT.md

@@ -0,0 +1,95 @@
+# Verification Report: {TICKET-ID}
+
+**Feature:** {feature-name}
+**Status:** PASSED | PARTIAL | FAILED
+**Verified:** {timestamp}
+
+---
+
+## Layer 1: Infrastructure
+
+### Files
+
+- [x] {path} — exists
+- [ ] {path} — MISSING
+
+**Result:** {N}/{M} files exist
+
+### Build
+
+**Command:** `{build-command}`
+**Status:** SUCCESS | FAILED
+
+{build errors if any}
+
+### Tests
+
+**Command:** `{test-command}`
+**Status:** SUCCESS | FAILED
+**Total:** {N} | **Passed:** {N} | **Failed:** {N}
+
+{test failure details if any}
+
+**Layer 1 Result:** PASSED | FAILED
+
+---
+
+## Layer 2: Feature Completeness
+
+{If feature.md was not found: "Skipped — no feature.md available (normal for quick-implement workflows)"}
+
+### Acceptance Criteria
+
+| # | Criterion | Status | Evidence |
+|---|-----------|--------|----------|
+| 1 | {criterion text} | SATISFIED | {file:line} |
+| 2 | {criterion text} | NOT SATISFIED | — |
+
+**Result:** {N}/{M} criteria satisfied
+
+### Functional Requirements
+
+| # | Requirement | Status | Evidence |
+|---|-------------|--------|----------|
+| 1 | {requirement text} | IMPLEMENTED | {file:line} |
+| 2 | {requirement text} | NOT IMPLEMENTED | — |
+
+**Result:** {N}/{M} requirements implemented
+
+### Component Completeness
+
+| Component | File | Status | Notes |
+|-----------|------|--------|-------|
+| {name} | {path} | COMPLETE | — |
+| {name} | {path} | PARTIAL | {what's missing} |
+
+**Result:** {N}/{M} components complete
+
+**Layer 2 Result:** PASSED | PARTIAL | SKIPPED
+
+---
+
+## Layer 3: Quality
+
+### Test Coverage for New Files
+
+| New File | Test File | Status |
+|----------|-----------|--------|
+| {src/path/File.ts} | {src/path/File.test.ts} | HAS TEST |
+| {src/path/Other.ts} | — | NO TEST |
+
+**Result:** {N}/{M} new files have tests
+
+**Layer 3 Result:** PASSED | PARTIAL
+
+---
+
+## Summary
+
+| Layer | Result | Details |
+|-------|--------|---------|
+| Infrastructure | PASSED / FAILED | {N}/{M} files, build {status}, tests {status} |
+| Feature Completeness | PASSED / PARTIAL / SKIPPED | {N}/{M} criteria, {N}/{M} requirements |
+| Quality | PASSED / PARTIAL | {N}/{M} new files have tests |
+
+**Overall:** PASSED | PARTIAL | FAILED