specweave 0.28.17 → 0.28.20

package/plugins/specweave-github/lib/github-status-sync.ts

@@ -53,6 +53,9 @@ export class GitHubStatusSync {
   /**
    * Update GitHub issue status
    *
+   * Preserves existing labels that are not status-related.
+   * Only replaces labels that start with "status:" prefix.
+   *
    * @param issueNumber - GitHub issue number
    * @param status - New status (state and labels)
    */
@@ -60,21 +63,74 @@ export class GitHubStatusSync {
     issueNumber: number,
     status: ExternalStatus
   ): Promise<void> {
-    const updateData: any = {
+    const updateData: {
+      owner: string;
+      repo: string;
+      issue_number: number;
+      state: 'open' | 'closed';
+      labels?: string[];
+    } = {
       owner: this.owner,
       repo: this.repo,
       issue_number: issueNumber,
-      state: status.state
+      state: status.state as 'open' | 'closed'
     };
 
-    // Add labels if provided
+    // Merge labels - preserve non-status labels, replace status labels
     if (status.labels && status.labels.length > 0) {
-      updateData.labels = status.labels;
+      // Fetch current labels to preserve non-status ones
+      const currentLabels = await this.getCurrentLabels(issueNumber);
+
+      // Filter out status-related labels (start with "status:")
+      const preservedLabels = currentLabels.filter(
+        label => !label.startsWith('status:')
+      );
+
+      // Get new status labels from the provided status
+      const newStatusLabels = status.labels.filter(
+        label => label.startsWith('status:')
+      );
+
+      // Get non-status labels from the provided status (e.g., priority, type)
+      const newOtherLabels = status.labels.filter(
+        label => !label.startsWith('status:')
+      );
+
+      // Combine: preserved non-status + new status + new other labels
+      const mergedLabels = [...new Set([
+        ...preservedLabels,
+        ...newStatusLabels,
+        ...newOtherLabels
+      ])];
+
+      updateData.labels = mergedLabels;
     }
 
     await this.octokit.rest.issues.update(updateData);
   }
 
+  /**
+   * Get current labels from GitHub issue
+   *
+   * @param issueNumber - GitHub issue number
+   * @returns Array of current label names
+   */
+  private async getCurrentLabels(issueNumber: number): Promise<string[]> {
+    try {
+      const response = await this.octokit.rest.issues.get({
+        owner: this.owner,
+        repo: this.repo,
+        issue_number: issueNumber
+      });
+
+      return response.data.labels
+        .map((label: any) => (typeof label === 'string' ? label : label.name))
+        .filter(Boolean) as string[];
+    } catch {
+      return [];
+    }
+  }
+
   /**
    * Post status change comment to GitHub issue
    *

package/plugins/specweave-github/lib/increment-issue-builder.js

@@ -0,0 +1,389 @@
+import { readFile } from "fs/promises";
+import { existsSync } from "fs";
+import * as path from "path";
+import * as yaml from "yaml";
+class IncrementIssueBuilder {
+  constructor(incrementPath, projectRoot) {
+    this.incrementPath = incrementPath;
+    this.projectRoot = projectRoot;
+  }
+  /**
+   * Parse increment spec.md and extract all data
+   */
+  async parse() {
+    const specPath = path.join(this.incrementPath, "spec.md");
+    if (!existsSync(specPath)) {
+      throw new Error(`spec.md not found at ${specPath}`);
+    }
+    const content = await readFile(specPath, "utf-8");
+    const frontmatterMatch = content.match(/^---\n([\s\S]*?)\n---/);
+    if (!frontmatterMatch) {
+      throw new Error("spec.md missing YAML frontmatter");
+    }
+    const frontmatter = yaml.parse(frontmatterMatch[1]);
+    const bodyContent = content.slice(frontmatterMatch[0].length).trim();
+    const titleMatch = bodyContent.match(/^#\s+(.+)$/m);
+    const title = titleMatch ? titleMatch[1].trim() : frontmatter.increment;
+    const problemStatement = this.extractSection(bodyContent, "Problem Statement");
+    const userStories = this.extractUserStories(bodyContent);
+    const outOfScopeSection = this.extractSection(bodyContent, "Out of Scope");
+    const outOfScope = outOfScopeSection.split("\n").filter((line) => line.startsWith("-")).map((line) => line.replace(/^-\s*/, "").trim());
+    const tasks = await this.extractTasks();
+    return {
+      frontmatter,
+      title,
+      problemStatement,
+      userStories,
+      tasks,
+      outOfScope
+    };
+  }
+  /**
+   * Extract tasks from tasks.md
+   */
+  async extractTasks() {
+    const tasksPath = path.join(this.incrementPath, "tasks.md");
+    if (!existsSync(tasksPath)) {
+      return [];
+    }
+    try {
+      const content = await readFile(tasksPath, "utf-8");
+      const tasks = [];
+      const taskBlocks = content.split(/(?=###\s+T-\d+)/);
+      for (const block of taskBlocks) {
+        const headerMatch = block.match(/###\s+(T-\d+):\s*(.+)/);
+        if (!headerMatch) continue;
+        const id = headerMatch[1];
+        const title = headerMatch[2].trim();
+        const statusMatch = block.match(/\*\*Status\*\*:\s*\[([x\s])\]/i);
+        const completed = statusMatch ? statusMatch[1].toLowerCase() === "x" : false;
+        const userStoryMatch = block.match(/\*\*User Story\*\*:\s*([^\n]+)/i);
+        const satisfiesMatch = block.match(/\*\*Satisfies ACs?\*\*:\s*([^\n]+)/i);
+        let userStories = [];
+        if (userStoryMatch) {
+          userStories = userStoryMatch[1].split(",").map((s) => s.trim());
+        } else if (satisfiesMatch) {
+          const acIds = satisfiesMatch[1].split(",").map((s) => s.trim());
+          const usIds = /* @__PURE__ */ new Set();
+          for (const ac of acIds) {
+            const usMatch = ac.match(/AC-(US\d+)-/i);
+            if (usMatch) {
+              usIds.add(usMatch[1]);
+            }
+          }
+          userStories = Array.from(usIds);
+        }
+        const priorityMatch = block.match(/\*\*Priority\*\*:\s*(P\d)/i);
+        const priority = priorityMatch ? priorityMatch[1] : void 0;
+        tasks.push({
+          id,
+          title,
+          completed,
+          userStories,
+          priority
+        });
+      }
+      return tasks;
+    } catch {
+      return [];
+    }
+  }
+  /**
+   * Extract a section by heading
+   */
+  extractSection(content, heading) {
+    const regex = new RegExp(`##\\s+${heading}\\s*\\n([\\s\\S]*?)(?=\\n##\\s|$)`, "i");
+    const match = content.match(regex);
+    return match ? match[1].trim() : "";
+  }
+  /**
+   * Extract user stories from spec.md
+   */
+  extractUserStories(content) {
+    const stories = [];
+    const userStoriesSection = this.extractSection(content, "User Stories");
+    if (!userStoriesSection) {
+      return stories;
+    }
+    const storyBlocks = userStoriesSection.split(/(?=###\s+US-\d+)/);
+    for (const block of storyBlocks) {
+      const headerMatch = block.match(/###\s+(US-\d+):\s*(.+)/);
+      if (!headerMatch) continue;
+      const id = headerMatch[1];
+      const title = headerMatch[2].trim();
+      const asAMatch = block.match(/\*\*As a\*\*\s+([^\n]+)/i);
+      const iWantMatch = block.match(/\*\*I want\*\*\s+([^\n]+)/i);
+      const soThatMatch = block.match(/\*\*So that\*\*\s+([^\n]+)/i);
+      const acceptanceCriteria = this.extractAcceptanceCriteria(block, id);
+      stories.push({
+        id,
+        title,
+        asA: asAMatch ? asAMatch[1].trim().replace(/,$/, "") : "",
+        iWant: iWantMatch ? iWantMatch[1].trim().replace(/,$/, "") : "",
+        soThat: soThatMatch ? soThatMatch[1].trim().replace(/\.$/, "") : "",
+        acceptanceCriteria
+      });
+    }
+    return stories;
+  }
+  /**
+   * Extract acceptance criteria from a user story block
+   */
+  extractAcceptanceCriteria(block, userStoryId) {
+    const criteria = [];
+    const acPattern = /-\s*\[([ x])\]\s*\*\*AC-(US\d+)-(\d+)\*\*:\s*(.+)/gi;
+    let match;
+    while ((match = acPattern.exec(block)) !== null) {
+      const completed = match[1].toLowerCase() === "x";
+      const usNum = match[2];
+      const acNum = match[3];
+      const description = match[4].trim();
+      criteria.push({
+        id: `AC-${usNum}-${acNum}`,
+        description,
+        completed
+      });
+    }
+    return criteria;
+  }
+  /**
+   * Build GitHub issue for a single user story
+   */
+  buildUserStoryIssue(story, incrementData, githubRepo) {
+    const featureId = incrementData.frontmatter.feature_id || this.generateFeatureId(incrementData);
+    const incrementId = incrementData.frontmatter.increment;
+    const title = `[${featureId}][${story.id}] ${story.title}`;
+    let body = "";
+    body += `**Feature**: ${featureId}
+`;
+    body += `**Status**: ${incrementData.frontmatter.status || "planning"}
+`;
+    body += `**Priority**: P1
+`;
+    body += `
+---
+
+`;
+    body += `## User Story
+
+`;
+    if (story.asA && story.iWant && story.soThat) {
+      body += `**As a** ${story.asA}
+`;
+      body += `**I want** ${story.iWant}
+`;
+      body += `**So that** ${story.soThat}
+
+`;
+    } else {
+      body += `${story.title}
+
+`;
+    }
+    body += `---
+
+`;
+    body += `## Acceptance Criteria
+
+`;
+    if (story.acceptanceCriteria.length > 0) {
+      const completed = story.acceptanceCriteria.filter((ac) => ac.completed).length;
+      const total = story.acceptanceCriteria.length;
+      const percentage = total > 0 ? Math.round(completed / total * 100) : 0;
+      body += `Progress: ${completed}/${total} criteria met (${percentage}%)
+
+`;
+      for (const ac of story.acceptanceCriteria) {
+        const checkbox = ac.completed ? "[x]" : "[ ]";
+        body += `- ${checkbox} **${ac.id}**: ${ac.description}
+`;
+      }
+      body += "\n";
+    } else {
+      body += `*No acceptance criteria defined*
+
+`;
+    }
+    body += `---
+
+`;
+    const storyTasks = incrementData.tasks.filter(
+      (t) => t.userStories.includes(story.id) || t.userStories.some((us) => us.toUpperCase() === story.id.toUpperCase())
+    );
+    if (storyTasks.length > 0) {
+      body += `## Tasks
+
+`;
+      const completedTasks = storyTasks.filter((t) => t.completed).length;
+      body += `Progress: ${completedTasks}/${storyTasks.length} tasks
+
+`;
+      for (const task of storyTasks) {
+        const checkbox = task.completed ? "[x]" : "[ ]";
+        body += `- ${checkbox} **${task.id}**: ${task.title}
+`;
+      }
+      body += "\n---\n\n";
+    }
+    body += `## Implementation
+
+`;
+    if (githubRepo) {
+      body += `**Increment**: [${incrementId}](https://github.com/${githubRepo}/tree/develop/.specweave/increments/${incrementId})
+
+`;
+    } else {
+      body += `**Increment**: ${incrementId}
+
+`;
+    }
+    body += `---
+
+`;
+    body += `\u{1F916} Auto-synced by SpecWeave Increment Sync`;
+    const labels = ["specweave", "user-story"];
+    if (incrementData.frontmatter.type) {
+      labels.push(incrementData.frontmatter.type.toLowerCase());
+    }
+    const priority = story.priority?.toLowerCase() || incrementData.frontmatter.priority?.toLowerCase() || "p2";
+    labels.push(priority);
+    return { title, body, labels };
+  }
+  /**
+   * Build GitHub issue for the entire increment (epic-style)
+   */
+  buildIncrementIssue(incrementData, githubRepo) {
+    const featureId = incrementData.frontmatter.feature_id || this.generateFeatureId(incrementData);
+    const incrementId = incrementData.frontmatter.increment;
+    const title = `[${featureId}] ${incrementData.title}`;
+    let body = "";
+    body += `**Increment**: ${incrementId}
+`;
+    body += `**Status**: ${incrementData.frontmatter.status || "planning"}
+`;
+    body += `**Priority**: P0 (Critical)
+`;
+    const totalACs = incrementData.userStories.reduce((sum, us) => sum + us.acceptanceCriteria.length, 0);
+    const completedACs = incrementData.userStories.reduce(
+      (sum, us) => sum + us.acceptanceCriteria.filter((ac) => ac.completed).length,
+      0
+    );
+    const percentage = totalACs > 0 ? Math.round(completedACs / totalACs * 100) : 0;
+    body += `**Progress**: ${completedACs}/${totalACs} ACs (${percentage}%)
+`;
+    body += `
+---
+
+`;
+    body += `## Overview
+
+`;
+    body += incrementData.problemStatement || incrementData.title;
+    body += `
+
+---
+
+`;
+    body += `## User Stories
+
+`;
+    for (const story of incrementData.userStories) {
+      const usCompleted = story.acceptanceCriteria.filter((ac) => ac.completed).length;
+      const usTotal = story.acceptanceCriteria.length;
+      const usCheckbox = usCompleted === usTotal && usTotal > 0 ? "[x]" : "[ ]";
+      body += `### ${usCheckbox} ${story.id}: ${story.title}
+
+`;
+      if (story.asA && story.iWant && story.soThat) {
+        body += `> **As a** ${story.asA}, **I want** ${story.iWant}, **So that** ${story.soThat}
+
+`;
+      }
+      body += `**Acceptance Criteria:**
+`;
+      for (const ac of story.acceptanceCriteria) {
+        const checkbox = ac.completed ? "[x]" : "[ ]";
+        body += `- ${checkbox} **${ac.id}**: ${ac.description}
+`;
+      }
+      body += "\n";
+    }
+    body += `---
+
+`;
+    if (incrementData.tasks.length > 0) {
+      body += `## Tasks
+
+`;
+      const completedTasks = incrementData.tasks.filter((t) => t.completed).length;
+      const totalTasks = incrementData.tasks.length;
+      const taskPercentage = totalTasks > 0 ? Math.round(completedTasks / totalTasks * 100) : 0;
+      body += `Progress: ${completedTasks}/${totalTasks} tasks (${taskPercentage}%)
+
+`;
+      for (const task of incrementData.tasks) {
+        const checkbox = task.completed ? "[x]" : "[ ]";
+        body += `- ${checkbox} **${task.id}**: ${task.title}
+`;
+        if (task.priority || task.userStories.length > 0) {
+          const parts = [];
+          if (task.priority) parts.push(`Priority: ${task.priority}`);
+          if (task.userStories.length > 0) parts.push(`User ${task.userStories.length === 1 ? "Story" : "Stories"}: ${task.userStories.join(", ")}`);
+          body += `  - ${parts.join(" | ")}
+`;
+        }
+      }
+      body += "\n---\n\n";
+    }
+    body += `## SpecWeave Increment
+
+`;
+    if (githubRepo) {
+      body += `- **Spec**: [\`spec.md\`](https://github.com/${githubRepo}/blob/develop/.specweave/increments/${incrementId}/spec.md)
+`;
+      body += `- **Plan**: [\`plan.md\`](https://github.com/${githubRepo}/blob/develop/.specweave/increments/${incrementId}/plan.md)
+`;
+      body += `- **Tasks**: [\`tasks.md\`](https://github.com/${githubRepo}/blob/develop/.specweave/increments/${incrementId}/tasks.md)
+`;
+    } else {
+      body += `- **Spec**: \`spec.md\`
+`;
+      body += `- **Plan**: \`plan.md\`
+`;
+      body += `- **Tasks**: \`tasks.md\`
+`;
+    }
+    body += `
+---
+
+`;
+    body += `\u{1F916} Auto-synced by SpecWeave Increment Sync`;
+    const labels = ["specweave", "increment"];
+    const typeLabel = incrementData.frontmatter.type?.toLowerCase() || "enhancement";
+    labels.push(typeLabel);
+    const priority = incrementData.frontmatter.priority?.toLowerCase() || "p2";
+    labels.push(priority);
+    return { title, body, labels };
+  }
+  /**
+   * Generate a feature ID if not present in frontmatter
+   * Uses date-based format: FS-YY-MM-DD
+   */
+  generateFeatureId(incrementData) {
+    const incrementNum = incrementData.frontmatter.increment.match(/^(\d+)/)?.[1];
+    if (incrementNum) {
+      return `FS-${incrementNum.padStart(3, "0")}`;
+    }
+    const created = incrementData.frontmatter.created;
+    if (created) {
+      const match = created.match(/^(\d{4})-(\d{2})-(\d{2})/);
+      if (match) {
+        return `FS-${match[1].slice(2)}-${match[2]}-${match[3]}`;
+      }
+    }
+    return "FS-UNKNOWN";
+  }
+}
+export {
+  IncrementIssueBuilder
+};