@iservu-inc/adf-cli 0.4.34 → 0.4.35

package/CHANGELOG.md

@@ -5,6 +5,94 @@ All notable changes to `@iservu-inc/adf-cli` will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.4.35] - 2025-10-04
+
+### 🔧 MAJOR FIX: Windsurf Configuration - Activation Modes & Content Generation
+
+**Fixed: Empty/Useless Windsurf Files**
+- **Problem:** Generated Windsurf files had wrong activation modes and empty content
+- **Issues:**
+  1. Rules had `Manual` activation (useless) instead of `Always On`
+  2. Workflows had no activation mode frontmatter
+  3. Content showed "Not provided" instead of actual answers
+  4. Not reading session data to extract meaningful information
+
+**Solution:**
+1. **Added Proper Frontmatter with Activation Modes:**
+   - Rules: `trigger: always_on` (default for most rules)
+   - Workflows: `trigger: manual` with descriptions
+   - Follows Windsurf best practices
+
+2. **Reads Actual Session Answers:**
+   - New `loadSessionAnswers()` loads `_progress.json`
+   - Extracts real answers from interview session
+   - Falls back gracefully if data unavailable
+
+3. **Intelligent Content Parsing:**
+   - New `parseImplementationBlueprint()` extracts structured sections
+   - New `extractSubsection()` finds content by keywords
+   - New `extractTechStackFromAnswers()` finds tech mentions
+   - New `extractWhatBuildingFromAnswers()` finds project goals
+
+**Before v0.4.35 (Broken):**
+```markdown
+# Architecture - adf-test-project
+
+## Implementation Blueprint
+Not provided
+
+## Data Model
+Not provided
+```
+
+**After v0.4.35 (Fixed):**
+```markdown
+---
+trigger: always_on
+---
+
+# Architecture - adf-test-project
+
+## Implementation Blueprint
+
+### Implementation Steps
+**Project Goal:**
+A performance website with nextjs frontend, postgres database...
+
+### Technical Approach
+**Tech Stack:**
+Next.js for frontend and backend, PostgreSQL for database...
+```
+
+**Code Changes:**
+- windsurf-generator.js:217-289 - Added frontmatter to project-context rule
+- windsurf-generator.js:296-359 - Added frontmatter & parsing to architecture rule
+- windsurf-generator.js:381-437 - Added frontmatter to coding-standards rule
+- windsurf-generator.js:465-545 - Added frontmatter & descriptions to workflows
+- windsurf-generator.js:633-754 - NEW helper methods for answer parsing
+
+**New Helper Methods:**
+- `loadSessionAnswers()` - Reads _progress.json
+- `parseImplementationBlueprint()` - Parses blueprint into sections
+- `extractSubsection()` - Finds content by keyword matching
+- `extractTechStackFromAnswers()` - Detects tech stack in answers
+- `extractWhatBuildingFromAnswers()` - Finds project goals
+
+**Activation Modes Guide:**
+- **Always On:** Default for rules - always active
+- **Glob:** For file/pattern matching (not used yet)
+- **Manual:** For workflows - user must activate
+- **Model Decide:** AI chooses when to activate (requires description)
+
+**Impact:**
+- ✅ Windsurf rules now work properly with `Always On` activation
+- ✅ Files contain actual project information, not placeholders
+- ✅ Workflows have proper frontmatter with descriptions
+- ✅ Better content extraction from session answers
+- ✅ Fallback to PRP/specification if answers unavailable
+
+---
+
 ## [0.4.34] - 2025-10-04
 
 ### ✨ Feature: Multi-Tool Deployment Selection

package/lib/generators/windsurf-generator.js

@@ -214,7 +214,11 @@ ${this.extractSection(this.outputs.architecture || '', 'System Overview') || 'Se
     let content;
     if (this.framework === 'rapid') {
       const sections = this.outputs.sections || {};
-      content = `# Project Context - ${projectName}
+      content = `---
+trigger: always_on
+---
+
+# Project Context - ${projectName}
 
 ## Goal
 
@@ -235,7 +239,11 @@ All requirements are documented in: \`.adf/sessions/${this.getSessionId()}/outpu
 Read this file before implementing any features.
 `;
     } else if (this.framework === 'balanced') {
-      content = `# Project Context - ${projectName}
+      content = `---
+trigger: always_on
+---
+
+# Project Context - ${projectName}
 
 ## Overview
 
@@ -257,7 +265,11 @@ Review these files before implementing:
 - Technical Plan: \`.adf/sessions/${this.getSessionId()}/outputs/plan.md\`
 `;
     } else {
-      content = `# Project Context - ${projectName}
+      content = `---
+trigger: always_on
+---
+
+# Project Context - ${projectName}
 
 ## Product Overview
 
@@ -282,21 +294,52 @@ Review these files before implementing:
   async generateArchitectureRule() {
     const projectName = this.getProjectName();
 
+    // Load actual session answers for better content
+    const answers = await this.loadSessionAnswers();
+
     let content;
     if (this.framework === 'rapid') {
       const sections = this.outputs.sections || {};
-      content = `# Architecture - ${projectName}
+      const blueprint = sections['4._implementation_blueprint'] || sections['implementation_blueprint'];
+
+      // Parse blueprint into structured sections
+      const parsed = this.parseImplementationBlueprint(blueprint || '', answers);
+
+      content = `---
+trigger: always_on
+---
+
+# Architecture - ${projectName}
 
 ## Implementation Blueprint
 
-${sections['4._implementation_blueprint'] || sections['implementation_blueprint'] || 'See PRP'}
+### Implementation Steps
+${parsed.steps || 'Not provided'}
+
+### Data Model
+${parsed.dataModel || 'Not provided'}
+
+### API Design
+${parsed.apiDesign || 'Not provided'}
+
+### Edge Cases & Error Handling
+${parsed.errorHandling || 'Not provided'}
+
+### Technical Approach
+${parsed.technicalApproach || 'Not provided'}
+
+---
 
 ## Reference
 
 Full implementation details: \`.adf/sessions/${this.getSessionId()}/outputs/prp.md\`
 `;
     } else if (this.framework === 'balanced') {
-      content = `# Architecture - ${projectName}
+      content = `---
+trigger: always_on
+---
+
+# Architecture - ${projectName}
 
 ${this.extractSection(this.outputs.specification || '', 'Architecture') || 'See specification'}
 
@@ -309,7 +352,11 @@ ${this.extractSection(this.outputs.plan || '', 'Architecture') || 'See technical
 Full architecture: \`.adf/sessions/${this.getSessionId()}/outputs/specification.md\`
 `;
     } else {
-      content = `# Architecture - ${projectName}
+      content = `---
+trigger: always_on
+---
+
+# Architecture - ${projectName}
 
 ${this.extractSection(this.outputs.architecture || '', 'System Overview') || 'See architecture.md'}
 
@@ -331,7 +378,11 @@ Full architecture: \`.adf/sessions/${this.getSessionId()}/outputs/architecture.m
 
     let content;
     if (this.framework === 'rapid') {
-      content = `# Coding Standards - ${projectName}
+      content = `---
+trigger: always_on
+---
+
+# Coding Standards - ${projectName}
 
 ## General Guidelines
 
@@ -346,7 +397,11 @@ Full architecture: \`.adf/sessions/${this.getSessionId()}/outputs/architecture.m
 See implementation blueprint in: \`.adf/sessions/${this.getSessionId()}/outputs/prp.md\`
 `;
     } else if (this.framework === 'balanced') {
-      content = `# Coding Standards - ${projectName}
+      content = `---
+trigger: always_on
+---
+
+# Coding Standards - ${projectName}
 
 ${this.extractSection(this.outputs.plan || '', 'Code Style') || this.extractSection(this.outputs.plan || '', 'Coding Standards') || ''}
 
@@ -359,7 +414,11 @@ ${this.extractSection(this.outputs.constitution || '', 'Core Principles') || 'Fo
 Full standards: \`.adf/sessions/${this.getSessionId()}/outputs/plan.md\`
 `;
     } else {
-      content = `# Coding Standards - ${projectName}
+      content = `---
+trigger: always_on
+---
+
+# Coding Standards - ${projectName}
 
 ${this.extractSection(this.outputs.prd || '', 'Code Quality') || ''}
 
@@ -403,7 +462,12 @@ Full requirements: \`.adf/sessions/${this.getSessionId()}/outputs/prd.md\`
     let content;
 
     if (this.framework === 'rapid') {
-      content = `# Review Requirements
+      content = `---
+trigger: manual
+description: Review project requirements before implementing
+---
+
+# Review Requirements
 
 Use this workflow to review requirements before implementing.
 
@@ -424,7 +488,12 @@ Use this workflow to review requirements before implementing.
 - How will success be measured?
 `;
     } else if (this.framework === 'balanced') {
-      content = `# Review Requirements
+      content = `---
+trigger: manual
+description: Review project requirements before implementing
+---
+
+# Review Requirements
 
 Use this workflow to review requirements before implementing.
 
@@ -447,7 +516,12 @@ Use this workflow to review requirements before implementing.
 - Are there any dependencies?
 `;
     } else {
-      content = `# Review Requirements
+      content = `---
+trigger: manual
+description: Review project requirements before implementing
+---
+
+# Review Requirements
 
 Use this workflow to review requirements before implementing.
 
@@ -477,7 +551,12 @@ Use this workflow to review requirements before implementing.
     let content;
 
     if (this.framework === 'balanced') {
-      content = `# Implement Feature
+      content = `---
+trigger: manual
+description: Workflow for implementing a new feature
+---
+
+# Implement Feature
 
 Use this workflow when implementing a feature.
 
@@ -511,7 +590,12 @@ Use this workflow when implementing a feature.
 - Tasks: \`.adf/sessions/${this.getSessionId()}/outputs/tasks.md\`
 `;
     } else {
-      content = `# Implement User Story
+      content = `---
+trigger: manual
+description: Workflow for implementing a user story
+---
+
+# Implement User Story
 
 Use this workflow when implementing a user story.
 
@@ -591,6 +675,132 @@ Use this workflow when implementing a user story.
       return '0.3.0';
     }
   }
+
+  /**
+   * Load session answers from _progress.json
+   */
+  async loadSessionAnswers() {
+    const fs = require('fs-extra');
+    const progressPath = path.join(this.sessionPath, '_progress.json');
+
+    try {
+      if (await fs.pathExists(progressPath)) {
+        const progress = await fs.readJson(progressPath);
+        return progress.answers || {};
+      }
+    } catch (error) {
+      // Fall back to empty if can't load
+    }
+
+    return {};
+  }
+
+  /**
+   * Parse implementation blueprint into structured sections
+   */
+  parseImplementationBlueprint(blueprint, answers) {
+    const parsed = {
+      steps: null,
+      dataModel: null,
+      apiDesign: null,
+      errorHandling: null,
+      technicalApproach: null
+    };
+
+    // If blueprint exists, parse it
+    if (blueprint && blueprint.length > 20) {
+      parsed.steps = this.extractSubsection(blueprint, ['implementation steps', 'steps to implement', 'development steps']);
+      parsed.dataModel = this.extractSubsection(blueprint, ['data model', 'database schema', 'data structure']);
+      parsed.apiDesign = this.extractSubsection(blueprint, ['api design', 'api endpoints', 'routes']);
+      parsed.errorHandling = this.extractSubsection(blueprint, ['error handling', 'edge cases']);
+      parsed.technicalApproach = this.extractSubsection(blueprint, ['technical approach', 'architecture', 'approach']);
+    }
+
+    // If still empty, try to extract from answers
+    if (!parsed.steps && answers) {
+      // Look for tech stack, implementation details in any answer
+      const techStack = this.extractTechStackFromAnswers(answers);
+      const whatBuilding = this.extractWhatBuildingFromAnswers(answers);
+
+      if (techStack) {
+        parsed.technicalApproach = `**Tech Stack:**\n${techStack}`;
+      }
+
+      if (whatBuilding) {
+        parsed.steps = `**Project Goal:**\n${whatBuilding}`;
+      }
+    }
+
+    return parsed;
+  }
+
+  /**
+   * Extract subsection from content by looking for keywords
+   */
+  extractSubsection(content, keywords) {
+    const lines = content.split('\n');
+    let inSection = false;
+    let sectionContent = [];
+
+    for (let i = 0; i < lines.length; i++) {
+      const line = lines[i];
+      const lowerLine = line.toLowerCase();
+
+      // Check if this line is a heading matching our keywords
+      if (line.match(/^#{2,4}\s+/)) {
+        const headingText = line.replace(/^#{2,4}\s+/, '').toLowerCase();
+
+        if (keywords.some(kw => headingText.includes(kw))) {
+          inSection = true;
+          continue;
+        } else if (inSection) {
+          // Hit a new heading, stop
+          break;
+        }
+      }
+
+      if (inSection && line.trim()) {
+        sectionContent.push(line);
+      }
+    }
+
+    const result = sectionContent.join('\n').trim();
+    return result.length > 10 ? result : null;
+  }
+
+  /**
+   * Extract tech stack from any answer
+   */
+  extractTechStackFromAnswers(answers) {
+    for (const [questionId, answer] of Object.entries(answers)) {
+      if (typeof answer === 'string') {
+        const lower = answer.toLowerCase();
+        // Look for mentions of frameworks, languages, databases
+        if (lower.includes('react') || lower.includes('vue') || lower.includes('angular') ||
+            lower.includes('node') || lower.includes('python') || lower.includes('next') ||
+            lower.includes('postgres') || lower.includes('mongo') || lower.includes('mysql')) {
+          return answer;
+        }
+      }
+    }
+    return null;
+  }
+
+  /**
+   * Extract "what are you building" from any answer
+   */
+  extractWhatBuildingFromAnswers(answers) {
+    for (const [questionId, answer] of Object.entries(answers)) {
+      if (questionId.toLowerCase().includes('goal') ||
+          questionId.toLowerCase().includes('building') ||
+          questionId.toLowerCase().includes('project')) {
+        if (typeof answer === 'string' && answer.length > 20) {
+          return answer;
+        }
+      }
+    }
+    return null;
+  }
 }
 
 module.exports = WindsurfGenerator;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iservu-inc/adf-cli",
-  "version": "0.4.34",
+  "version": "0.4.35",
   "description": "CLI tool for AgentDevFramework - AI-assisted development framework with multi-provider AI support",
   "main": "index.js",
   "bin": {