specweave 0.18.1 → 0.20.1

package/plugins/specweave-github/lib/github-feature-sync.js

@@ -0,0 +1,381 @@
+import { readdir, readFile, writeFile } from "fs/promises";
+import { existsSync } from "fs";
+import * as path from "path";
+import * as yaml from "yaml";
+import { UserStoryIssueBuilder } from "./user-story-issue-builder.js";
+import { CompletionCalculator } from "./completion-calculator.js";
+import { execFileNoThrow } from "../../../src/utils/execFileNoThrow.js";
+class GitHubFeatureSync {
+  constructor(client, specsDir, projectRoot) {
+    this.client = client;
+    this.specsDir = specsDir;
+    this.projectRoot = projectRoot;
+    this.calculator = new CompletionCalculator(projectRoot);
+  }
+  /**
+   * Sync Feature folder to GitHub (Milestone + User Story Issues)
+   *
+   * Process:
+   * 1. Create/update GitHub Milestone for Feature
+   * 2. Find all us-*.md files across all projects
+   * 3. Create/update GitHub Issue for EACH user story
+   * 4. Update frontmatter with GitHub issue links
+   */
+  async syncFeatureToGitHub(featureId) {
+    console.log(`
+\u{1F504} Syncing Feature ${featureId} to GitHub...`);
+    const featureFolder = await this.findFeatureFolder(featureId);
+    if (!featureFolder) {
+      throw new Error(`Feature ${featureId} not found in ${this.specsDir}`);
+    }
+    const featurePath = path.join(featureFolder, "FEATURE.md");
+    const featureData = await this.parseFeatureMd(featurePath);
+    console.log(`   \u{1F4E6} Feature: ${featureData.title}`);
+    console.log(`   \u{1F4CA} Status: ${featureData.status}`);
+    let milestoneNumber = featureData.external_tools?.github?.id;
+    let milestoneUrl = featureData.external_tools?.github?.url;
+    if (!milestoneNumber) {
+      console.log(`   \u{1F680} Creating GitHub Milestone...`);
+      const milestone = await this.createMilestone(featureData);
+      milestoneNumber = milestone.number;
+      milestoneUrl = milestone.url;
+      console.log(`   \u2705 Created Milestone #${milestoneNumber}`);
+      await this.updateFeatureMd(featurePath, {
+        type: "milestone",
+        id: milestoneNumber,
+        url: milestoneUrl
+      });
+    } else {
+      console.log(`   \u267B\uFE0F  Using existing Milestone #${milestoneNumber}`);
+      milestoneUrl = featureData.external_tools?.github?.url || milestoneUrl;
+    }
+    const userStories = await this.findUserStories(featureId);
+    console.log(`
+   \u{1F4DD} Found ${userStories.length} User Stories to sync...`);
+    let issuesCreated = 0;
+    let issuesUpdated = 0;
+    for (const userStory of userStories) {
+      console.log(`
+   \u{1F539} Processing ${userStory.id}: ${userStory.title}`);
+      const repoInfo = {
+        owner: this.client.getOwner(),
+        repo: this.client.getRepo(),
+        branch: "develop"
+        // TODO: detect from git
+      };
+      const builder = new UserStoryIssueBuilder(
+        userStory.filePath,
+        this.projectRoot,
+        featureId,
+        repoInfo
+      );
+      const issueContent = await builder.buildIssueBody();
+      issueContent.status = userStory.status;
+      if (userStory.existingIssue) {
+        console.log(`      \u267B\uFE0F  Issue #${userStory.existingIssue} exists in frontmatter`);
+        try {
+          await this.client.getIssue(userStory.existingIssue);
+          await this.updateUserStoryIssue(userStory.existingIssue, issueContent, userStory.filePath);
+          issuesUpdated++;
+          console.log(`      \u2705 Updated Issue #${userStory.existingIssue}`);
+          continue;
+        } catch (err) {
+          console.log(`      \u26A0\uFE0F  Issue #${userStory.existingIssue} deleted on GitHub, creating new`);
+        }
+      }
+      const existingByTitle = await this.client.searchIssueByTitle(issueContent.title);
+      if (existingByTitle) {
+        console.log(`      \u267B\uFE0F  Found existing issue by title: #${existingByTitle.number}`);
+        await this.updateUserStoryFrontmatter(userStory.filePath, existingByTitle.number);
+        await this.updateUserStoryIssue(existingByTitle.number, issueContent, userStory.filePath);
+        issuesUpdated++;
+        console.log(`      \u2705 Linked and updated Issue #${existingByTitle.number}`);
+        continue;
+      }
+      console.log(`      \u{1F680} Creating new issue: ${issueContent.title}`);
+      const milestoneTitle = `${featureData.id}: ${featureData.title}`;
+      const issueNumber = await this.createUserStoryIssue(
+        issueContent,
+        milestoneTitle,
+        userStory.filePath
+      );
+      issuesCreated++;
+      console.log(`      \u2705 Created Issue #${issueNumber}`);
+      await this.updateUserStoryFrontmatter(userStory.filePath, issueNumber);
+    }
+    console.log(`
+\u2705 Feature sync complete!`);
+    console.log(`   Milestone: ${milestoneUrl}`);
+    console.log(`   User Stories: ${userStories.length}`);
+    console.log(`   Issues created: ${issuesCreated}`);
+    console.log(`   Issues updated: ${issuesUpdated}`);
+    return {
+      milestoneNumber,
+      milestoneUrl,
+      issuesCreated,
+      issuesUpdated,
+      userStoriesProcessed: userStories.length
+    };
+  }
+  /**
+   * Find Feature folder in specs directory
+   */
+  async findFeatureFolder(featureId) {
+    const featuresFolder = path.join(this.specsDir, "_features", featureId);
+    if (existsSync(featuresFolder)) {
+      return featuresFolder;
+    }
+    return null;
+  }
+  /**
+   * Parse FEATURE.md frontmatter
+   */
+  async parseFeatureMd(featurePath) {
+    const content = await readFile(featurePath, "utf-8");
+    const match = content.match(/^---\n([\s\S]*?)\n---/);
+    if (!match) {
+      throw new Error(`${featurePath}: Missing YAML frontmatter`);
+    }
+    return yaml.parse(match[1]);
+  }
+  /**
+   * Find all User Story files for this feature across all projects
+   */
+  async findUserStories(featureId) {
+    const userStories = [];
+    const projectFolders = await this.findProjectFolders();
+    for (const projectFolder of projectFolders) {
+      const featureSpecsFolder = path.join(projectFolder, featureId);
+      if (!existsSync(featureSpecsFolder)) {
+        continue;
+      }
+      const files = await readdir(featureSpecsFolder);
+      const usFiles = files.filter((f) => f.startsWith("us-") && f.endsWith(".md"));
+      for (const file of usFiles.sort()) {
+        const filePath = path.join(featureSpecsFolder, file);
+        const content = await readFile(filePath, "utf-8");
+        const match = content.match(/^---\n([\s\S]*?)\n---/);
+        if (!match) {
+          console.warn(`   \u26A0\uFE0F  ${file}: Missing frontmatter, skipping`);
+          continue;
+        }
+        const frontmatter = yaml.parse(match[1]);
+        const projectName = path.basename(projectFolder);
+        userStories.push({
+          id: frontmatter.id || file.match(/us-(\d+)/)?.[0]?.toUpperCase() || "UNKNOWN",
+          title: frontmatter.title || "Untitled User Story",
+          filePath,
+          project: projectName,
+          status: frontmatter.status || "not-started",
+          existingIssue: frontmatter.external?.github?.issue || null
+        });
+      }
+    }
+    return userStories;
+  }
+  /**
+   * Find all project folders (default, backend, frontend, etc.)
+   */
+  async findProjectFolders() {
+    const folders = [];
+    const specsRoot = this.specsDir;
+    const entries = await readdir(specsRoot, { withFileTypes: true });
+    for (const entry of entries) {
+      if (entry.isDirectory() && !entry.name.startsWith("_")) {
+        folders.push(path.join(specsRoot, entry.name));
+      }
+    }
+    return folders;
+  }
+  /**
+   * Create GitHub Milestone for Feature
+   */
+  async createMilestone(featureData) {
+    const title = `${featureData.id}: ${featureData.title}`;
+    const description = `Feature ${featureData.id}
+
+Status: ${featureData.status}
+Created: ${featureData.created}`;
+    const result = await execFileNoThrow("gh", [
+      "api",
+      "repos/:owner/:repo/milestones",
+      "-X",
+      "POST",
+      "-f",
+      `title=${title}`,
+      "-f",
+      `description=${description}`,
+      "-f",
+      "state=open"
+    ]);
+    if (result.exitCode !== 0) {
+      throw new Error(`Failed to create Milestone: ${result.stderr || result.stdout}`);
+    }
+    const milestone = JSON.parse(result.stdout);
+    return {
+      number: milestone.number,
+      url: milestone.html_url
+    };
+  }
+  /**
+   * Create GitHub Issue for User Story with AC/Task Verification
+   *
+   * ✅ VERIFICATION GATE FIX:
+   * - Verifies actual completion before closing
+   * - Prevents premature closure on creation
+   */
+  async createUserStoryIssue(issueContent, milestoneTitle, userStoryPath) {
+    const result = await execFileNoThrow("gh", [
+      "issue",
+      "create",
+      "--title",
+      issueContent.title,
+      "--body",
+      issueContent.body,
+      "--milestone",
+      milestoneTitle,
+      ...issueContent.labels.flatMap((label) => ["--label", label])
+    ]);
+    if (result.exitCode !== 0) {
+      throw new Error(`Failed to create GitHub Issue: ${result.stderr || result.stdout}`);
+    }
+    const match = result.stdout.match(/issues\/(\d+)/);
+    if (!match) {
+      throw new Error(`Failed to parse issue number from: ${result.stdout}`);
+    }
+    const issueNumber = parseInt(match[1], 10);
+    const completion = await this.calculator.calculateCompletion(userStoryPath);
+    if (completion.overallComplete) {
+      await execFileNoThrow("gh", [
+        "issue",
+        "close",
+        issueNumber.toString(),
+        "--comment",
+        this.calculator.buildCompletionComment(completion)
+      ]);
+      console.log(
+        `      \u2705 Created and verified complete: ${completion.acsCompleted}/${completion.acsTotal} ACs, ${completion.tasksCompleted}/${completion.tasksTotal} tasks`
+      );
+    } else {
+      await execFileNoThrow("gh", [
+        "issue",
+        "comment",
+        issueNumber.toString(),
+        "--body",
+        this.calculator.buildProgressComment(completion)
+      ]);
+      console.log(
+        `      \u{1F4CA} Created: ${completion.acsPercentage.toFixed(0)}% ACs, ${completion.tasksPercentage.toFixed(0)}% tasks`
+      );
+    }
+    return issueNumber;
+  }
+  /**
+   * Update existing GitHub Issue for User Story with AC/Task Verification
+   *
+   * ✅ VERIFICATION GATE FIX:
+   * - OLD: Closed based on frontmatter `status: complete`
+   * - NEW: Closes ONLY if all ACs and tasks are [x]
+   *
+   * This prevents Issue #574 type bugs (premature closure)
+   */
+  async updateUserStoryIssue(issueNumber, issueContent, userStoryPath) {
+    await execFileNoThrow("gh", [
+      "issue",
+      "edit",
+      issueNumber.toString(),
+      "--title",
+      issueContent.title,
+      "--body",
+      issueContent.body
+    ]);
+    const completion = await this.calculator.calculateCompletion(userStoryPath);
+    const issueData = await this.client.getIssue(issueNumber);
+    const currentlyClosed = issueData.state === "closed";
+    if (completion.overallComplete) {
+      if (!currentlyClosed) {
+        await execFileNoThrow("gh", [
+          "issue",
+          "close",
+          issueNumber.toString(),
+          "--comment",
+          this.calculator.buildCompletionComment(completion)
+        ]);
+        console.log(
+          `      \u2705 Verified complete: ${completion.acsCompleted}/${completion.acsTotal} ACs, ${completion.tasksCompleted}/${completion.tasksTotal} tasks`
+        );
+      }
+    } else {
+      if (currentlyClosed) {
+        await execFileNoThrow("gh", [
+          "issue",
+          "reopen",
+          issueNumber.toString(),
+          "--comment",
+          this.calculator.buildReopenComment(completion, "Work verification failed")
+        ]);
+        console.log(
+          `      \u26A0\uFE0F Reopened: ${completion.blockingAcs.length + completion.blockingTasks.length} items incomplete`
+        );
+      } else {
+        await execFileNoThrow("gh", [
+          "issue",
+          "comment",
+          issueNumber.toString(),
+          "--body",
+          this.calculator.buildProgressComment(completion)
+        ]);
+        console.log(
+          `      \u{1F4CA} Progress: ${completion.acsPercentage.toFixed(0)}% ACs, ${completion.tasksPercentage.toFixed(0)}% tasks`
+        );
+      }
+    }
+  }
+  /**
+   * Update FEATURE.md with GitHub Milestone link
+   */
+  async updateFeatureMd(featurePath, milestone) {
+    const content = await readFile(featurePath, "utf-8");
+    const match = content.match(/^---\n([\s\S]*?)\n---/);
+    if (!match) {
+      throw new Error(`${featurePath}: Missing YAML frontmatter`);
+    }
+    const frontmatter = yaml.parse(match[1]);
+    if (!frontmatter.external_tools) {
+      frontmatter.external_tools = {};
+    }
+    frontmatter.external_tools.github = milestone;
+    const newFrontmatter = yaml.stringify(frontmatter);
+    const bodyContent = content.slice(match[0].length);
+    const newContent = `---
+${newFrontmatter}---${bodyContent}`;
+    await writeFile(featurePath, newContent, "utf-8");
+  }
+  /**
+   * Update User Story frontmatter with GitHub issue link
+   */
+  async updateUserStoryFrontmatter(userStoryPath, issueNumber) {
+    const content = await readFile(userStoryPath, "utf-8");
+    const match = content.match(/^---\n([\s\S]*?)\n---/);
+    if (!match) {
+      throw new Error(`${userStoryPath}: Missing YAML frontmatter`);
+    }
+    const frontmatter = yaml.parse(match[1]);
+    if (!frontmatter.external) {
+      frontmatter.external = {};
+    }
+    if (!frontmatter.external.github) {
+      frontmatter.external.github = {};
+    }
+    frontmatter.external.github.issue = issueNumber;
+    frontmatter.external.github.url = `https://github.com/anton-abyzov/specweave/issues/${issueNumber}`;
+    const newFrontmatter = yaml.stringify(frontmatter);
+    const bodyContent = content.slice(match[0].length);
+    const newContent = `---
+${newFrontmatter}---${bodyContent}`;
+    await writeFile(userStoryPath, newContent, "utf-8");
+  }
+}
+export {
+  GitHubFeatureSync
+};