specweave 0.17.0 → 0.17.2

package/plugins/specweave-github/lib/github-spec-commit-sync.ts

@@ -0,0 +1,285 @@
+/**
+ * GitHub Spec Commit Sync
+ *
+ * Posts commit/PR comments to GitHub issues linked to living docs specs.
+ * This enables traceability from spec user stories to actual code changes.
+ */
+
+import { GitHubClientV2 } from './github-client-v2.js';
+import {
+  getRecentCommits,
+  detectRepository,
+  getCommitUrl,
+  findPRForCommit,
+  getLastSyncCommit,
+  updateLastSyncCommit,
+  GitCommit,
+  GitRepository,
+  PullRequest,
+} from '../../../src/utils/git-utils.js';
+import {
+  getSpecTaskMapping,
+  getCompletedUserStories,
+  SpecTaskMapping,
+  UserStory,
+} from '../../../src/core/spec-task-mapper.js';
+import {
+  buildUserStoryComment,
+  buildCommitBatchComment,
+  buildShortComment,
+  generateSmartSummary,
+  CommentContent,
+} from '../../../src/core/comment-builder.js';
+import path from 'path';
+import fs from 'fs/promises';
+
+export interface SyncOptions {
+  incrementPath: string;
+  dryRun?: boolean;
+  verbose?: boolean;
+}
+
+export interface SyncResult {
+  success: boolean;
+  commentsPosted: number;
+  errors: string[];
+  commits: GitCommit[];
+  userStories: UserStory[];
+}
+
+/**
+ * Sync commits for completed user stories to GitHub issue
+ */
+export async function syncSpecCommitsToGitHub(
+  options: SyncOptions
+): Promise<SyncResult> {
+  const { incrementPath, dryRun = false, verbose = false } = options;
+
+  const result: SyncResult = {
+    success: true,
+    commentsPosted: 0,
+    errors: [],
+    commits: [],
+    userStories: [],
+  };
+
+  try {
+    // 1. Check if increment has GitHub issue linked
+    const metadataPath = path.join(incrementPath, 'metadata.json');
+    let metadata: any;
+
+    try {
+      metadata = JSON.parse(await fs.readFile(metadataPath, 'utf-8'));
+    } catch {
+      result.errors.push('No metadata.json found');
+      return result;
+    }
+
+    const githubIssue = metadata.github?.issue;
+    if (!githubIssue) {
+      if (verbose) {
+        console.log('No GitHub issue linked to increment');
+      }
+      return result;
+    }
+
+    // 2. Detect repository
+    const repo = await detectRepository('origin', incrementPath);
+    if (!repo || repo.provider !== 'github') {
+      result.errors.push('Not a GitHub repository or remote not found');
+      result.success = false;
+      return result;
+    }
+
+    // 3. Get spec-task mapping
+    const mapping = await getSpecTaskMapping(incrementPath);
+    if (!mapping) {
+      if (verbose) {
+        console.log('No spec mapping found for increment');
+      }
+      return result;
+    }
+
+    // 4. Get completed user stories
+    const completedUserStories = getCompletedUserStories(mapping);
+    if (completedUserStories.length === 0) {
+      if (verbose) {
+        console.log('No completed user stories');
+      }
+      return result;
+    }
+
+    result.userStories = completedUserStories;
+
+    // 5. Get recent commits (since last sync)
+    const lastSyncCommit = await getLastSyncCommit(metadataPath);
+    const commits = lastSyncCommit
+      ? await getRecentCommits(undefined, lastSyncCommit, incrementPath)
+      : await getRecentCommits('7.days', undefined, incrementPath);
+
+    if (commits.length === 0) {
+      if (verbose) {
+        console.log('No new commits since last sync');
+      }
+      return result;
+    }
+
+    result.commits = commits;
+
+    // Add commit URLs
+    for (const commit of commits) {
+      commit.url = getCommitUrl(commit, repo);
+    }
+
+    // 6. Initialize GitHub client
+    const client = GitHubClientV2.fromRepo(repo.owner, repo.repo);
+
+    // 7. Post comments for each completed user story
+    for (const userStory of completedUserStories) {
+      try {
+        // Get tasks for this user story
+        const relatedTasks = mapping.tasks.filter(t =>
+          t.userStories.includes(userStory.id)
+        );
+
+        // Find PRs for commits (optional, best effort)
+        const pullRequests: PullRequest[] = [];
+        for (const commit of commits.slice(0, 3)) { // Check first 3 commits only
+          const pr = await findPRForCommit(commit.sha, repo.owner, repo.repo);
+          if (pr && !pullRequests.find(p => p.number === pr.number)) {
+            pullRequests.push(pr);
+          }
+        }
+
+        // Build comment
+        const summary = generateSmartSummary(commits, relatedTasks);
+        const commentContent: CommentContent = {
+          userStory,
+          tasks: relatedTasks,
+          commits,
+          pullRequests,
+          summary,
+        };
+
+        const formattedComment = buildUserStoryComment(commentContent, repo);
+
+        if (verbose) {
+          console.log(`\nComment for ${userStory.id}:`);
+          console.log(formattedComment.markdown);
+          console.log('\n---\n');
+        }
+
+        // Post comment
+        if (!dryRun) {
+          await client.addComment(githubIssue, formattedComment.markdown);
+          result.commentsPosted++;
+        } else {
+          result.commentsPosted++; // Count as "would post"
+        }
+      } catch (error: any) {
+        result.errors.push(`Failed to post comment for ${userStory.id}: ${error.message}`);
+      }
+    }
+
+    // 8. If no user stories completed but there are commits, post a simple update
+    if (completedUserStories.length === 0 && commits.length > 0) {
+      try {
+        const summary = `Work in progress: ${commits.length} commit${commits.length > 1 ? 's' : ''}`;
+        const formattedComment = buildShortComment(commits, repo, summary);
+
+        if (verbose) {
+          console.log(`\nShort update comment:`);
+          console.log(formattedComment.markdown);
+          console.log('\n---\n');
+        }
+
+        if (!dryRun) {
+          await client.addComment(githubIssue, formattedComment.markdown);
+          result.commentsPosted++;
+        } else {
+          result.commentsPosted++;
+        }
+      } catch (error: any) {
+        result.errors.push(`Failed to post short update: ${error.message}`);
+      }
+    }
+
+    // 9. Update last sync commit
+    if (!dryRun && commits.length > 0) {
+      await updateLastSyncCommit(metadataPath, commits[0].sha);
+    }
+
+  } catch (error: any) {
+    result.success = false;
+    result.errors.push(error.message);
+  }
+
+  return result;
+}
+
+/**
+ * Post batch commit update to GitHub issue (for multiple commits)
+ */
+export async function postCommitBatchUpdate(
+  incrementPath: string,
+  commits: GitCommit[],
+  issueNumber: number,
+  repo: GitRepository,
+  dryRun: boolean = false
+): Promise<boolean> {
+  try {
+    const client = GitHubClientV2.fromRepo(repo.owner, repo.repo);
+
+    // Add commit URLs
+    for (const commit of commits) {
+      commit.url = getCommitUrl(commit, repo);
+    }
+
+    // Build and post comment
+    const formattedComment = buildCommitBatchComment(commits, repo, true);
+
+    if (!dryRun) {
+      await client.addComment(issueNumber, formattedComment.markdown);
+    }
+
+    return true;
+  } catch (error: any) {
+    console.error(`Failed to post batch update: ${error.message}`);
+    return false;
+  }
+}
+
+/**
+ * Check if spec commit sync is enabled in config
+ */
+export async function isSpecCommitSyncEnabled(
+  projectRoot: string
+): Promise<boolean> {
+  try {
+    const configPath = path.join(projectRoot, '.specweave', 'config.json');
+    const config = JSON.parse(await fs.readFile(configPath, 'utf-8'));
+
+    return config.sync?.settings?.postCommitComments !== false;
+  } catch {
+    return true; // Default: enabled
+  }
+}
+
+/**
+ * Get GitHub issue number from spec metadata
+ */
+export async function getGitHubIssueForSpec(
+  specPath: string
+): Promise<number | null> {
+  try {
+    const content = await fs.readFile(specPath, 'utf-8');
+
+    // Extract GitHub issue from frontmatter or body
+    // Pattern: **GitHub Project**: https://github.com/owner/repo/issues/123
+    const match = content.match(/github\.com\/[\w-]+\/[\w-]+\/issues\/(\d+)/);
+
+    return match ? parseInt(match[1], 10) : null;
+  } catch {
+    return null;
+  }
+}

package/plugins/specweave-github/lib/github-spec-content-sync.js

@@ -0,0 +1,169 @@
+import { GitHubClientV2 } from "./github-client-v2.js";
+import {
+  parseSpecContent,
+  detectContentChanges,
+  buildExternalDescription,
+  hasExternalLink,
+  updateSpecWithExternalLink
+} from "../../../src/core/spec-content-sync.js";
+import path from "path";
+import fs from "fs/promises";
+async function syncSpecContentToGitHub(options) {
+  const { specPath, owner, repo, dryRun = false, verbose = false } = options;
+  try {
+    const spec = await parseSpecContent(specPath);
+    if (!spec) {
+      return {
+        success: false,
+        action: "error",
+        error: "Failed to parse spec content"
+      };
+    }
+    if (verbose) {
+      console.log(`\u{1F4C4} Parsed spec: ${spec.id}`);
+      console.log(`   Title: ${spec.title}`);
+      console.log(`   User Stories: ${spec.userStories.length}`);
+    }
+    const existingIssueNumber = await hasExternalLink(specPath, "github");
+    const client = GitHubClientV2.fromRepo(owner, repo);
+    if (existingIssueNumber) {
+      return await updateGitHubIssue(client, spec, parseInt(existingIssueNumber), options);
+    } else {
+      return await createGitHubIssue(client, spec, options);
+    }
+  } catch (error) {
+    return {
+      success: false,
+      action: "error",
+      error: error.message
+    };
+  }
+}
+async function createGitHubIssue(client, spec, options) {
+  const { specPath, dryRun, verbose } = options;
+  try {
+    const title = `[${spec.id.toUpperCase()}] ${spec.title}`;
+    const body = buildExternalDescription(spec);
+    if (verbose) {
+      console.log(`
+\u{1F4DD} Creating GitHub issue:`);
+      console.log(`   Title: ${title}`);
+      console.log(`   Body length: ${body.length} chars`);
+    }
+    if (dryRun) {
+      console.log("\n\u{1F50D} Dry run - would create issue:");
+      console.log(`   Title: ${title}`);
+      console.log(`   Body:
+${body}`);
+      return {
+        success: true,
+        action: "created",
+        externalId: "DRY-RUN",
+        externalUrl: "https://github.com/DRY-RUN"
+      };
+    }
+    const labels = ["specweave", "spec", spec.metadata.priority || "P2"].filter(Boolean);
+    const issue = await client.createEpicIssue(title, body, void 0, labels);
+    if (verbose) {
+      console.log(`\u2705 Created issue #${issue.number}`);
+      console.log(`   URL: ${issue.html_url}`);
+    }
+    await updateSpecWithExternalLink(
+      specPath,
+      "github",
+      issue.number.toString(),
+      issue.html_url
+    );
+    return {
+      success: true,
+      action: "created",
+      externalId: issue.number.toString(),
+      externalUrl: issue.html_url
+    };
+  } catch (error) {
+    return {
+      success: false,
+      action: "error",
+      error: `Failed to create GitHub issue: ${error.message}`
+    };
+  }
+}
+async function updateGitHubIssue(client, spec, issueNumber, options) {
+  const { specPath, dryRun, verbose } = options;
+  try {
+    const issue = await client.getIssue(issueNumber);
+    if (verbose) {
+      console.log(`
+\u{1F504} Checking for changes in issue #${issueNumber}`);
+    }
+    const changes = detectContentChanges(spec, {
+      title: issue.title.replace(/^\[SPEC-\d+\]\s*/, ""),
+      description: issue.body || "",
+      userStoryCount: countUserStoriesInBody(issue.body || "")
+    });
+    if (!changes.hasChanges) {
+      if (verbose) {
+        console.log("   \u2139\uFE0F  No changes detected");
+      }
+      return {
+        success: true,
+        action: "no-change",
+        externalId: issueNumber.toString(),
+        externalUrl: issue.html_url
+      };
+    }
+    if (verbose) {
+      console.log("   \u{1F4DD} Changes detected:");
+      for (const change of changes.changes) {
+        console.log(`      - ${change}`);
+      }
+    }
+    const newTitle = `[${spec.id.toUpperCase()}] ${spec.title}`;
+    const newBody = buildExternalDescription(spec);
+    if (dryRun) {
+      console.log("\n\u{1F50D} Dry run - would update issue:");
+      console.log(`   Title: ${newTitle}`);
+      console.log(`   Body:
+${newBody}`);
+      return {
+        success: true,
+        action: "updated",
+        externalId: issueNumber.toString(),
+        externalUrl: issue.html_url
+      };
+    }
+    await client.updateIssueBody(issueNumber, newBody);
+    if (verbose) {
+      console.log(`\u2705 Updated issue #${issueNumber}`);
+    }
+    return {
+      success: true,
+      action: "updated",
+      externalId: issueNumber.toString(),
+      externalUrl: issue.html_url
+    };
+  } catch (error) {
+    return {
+      success: false,
+      action: "error",
+      error: `Failed to update GitHub issue: ${error.message}`
+    };
+  }
+}
+function countUserStoriesInBody(body) {
+  const matches = body.match(/###\s+US-\d+:/g);
+  return matches ? matches.length : 0;
+}
+async function isContentSyncEnabled(projectRoot) {
+  try {
+    const configPath = path.join(projectRoot, ".specweave", "config.json");
+    const config = JSON.parse(await fs.readFile(configPath, "utf-8"));
+    return config.sync?.settings?.syncSpecContent !== false;
+  } catch {
+    return true;
+  }
+}
+export {
+  isContentSyncEnabled,
+  syncSpecContentToGitHub
+};

package/plugins/specweave-github/lib/github-spec-content-sync.ts

@@ -0,0 +1,255 @@
+/**
+ * GitHub Spec Content Sync
+ *
+ * Syncs spec CONTENT (title, description, user stories) to GitHub Issues.
+ * Does NOT sync STATUS - that's managed in GitHub.
+ *
+ * Sync Direction:
+ * - Title/Description: SpecWeave → GitHub (we update)
+ * - Status (open/closed): GitHub → SpecWeave (we read)
+ */
+
+import { GitHubClientV2 } from './github-client-v2.js';
+import {
+  parseSpecContent,
+  detectContentChanges,
+  buildExternalDescription,
+  hasExternalLink,
+  updateSpecWithExternalLink,
+  wasSpecModifiedSinceSync,
+  SpecContent,
+  ContentSyncResult,
+} from '../../../src/core/spec-content-sync.js';
+import path from 'path';
+import fs from 'fs/promises';
+
+export interface GitHubContentSyncOptions {
+  specPath: string;
+  owner: string;
+  repo: string;
+  dryRun?: boolean;
+  verbose?: boolean;
+}
+
+/**
+ * Sync spec content to GitHub issue
+ * Creates issue if it doesn't exist, updates if it does
+ */
+export async function syncSpecContentToGitHub(
+  options: GitHubContentSyncOptions
+): Promise<ContentSyncResult> {
+  const { specPath, owner, repo, dryRun = false, verbose = false } = options;
+
+  try {
+    // 1. Parse spec content
+    const spec = await parseSpecContent(specPath);
+    if (!spec) {
+      return {
+        success: false,
+        action: 'error',
+        error: 'Failed to parse spec content',
+      };
+    }
+
+    if (verbose) {
+      console.log(`📄 Parsed spec: ${spec.id}`);
+      console.log(`   Title: ${spec.title}`);
+      console.log(`   User Stories: ${spec.userStories.length}`);
+    }
+
+    // 2. Check if issue already exists
+    const existingIssueNumber = await hasExternalLink(specPath, 'github');
+
+    const client = GitHubClientV2.fromRepo(owner, repo);
+
+    if (existingIssueNumber) {
+      // UPDATE existing issue
+      return await updateGitHubIssue(client, spec, parseInt(existingIssueNumber), options);
+    } else {
+      // CREATE new issue
+      return await createGitHubIssue(client, spec, options);
+    }
+  } catch (error: any) {
+    return {
+      success: false,
+      action: 'error',
+      error: error.message,
+    };
+  }
+}
+
+/**
+ * Create new GitHub issue from spec
+ */
+async function createGitHubIssue(
+  client: GitHubClientV2,
+  spec: SpecContent,
+  options: GitHubContentSyncOptions
+): Promise<ContentSyncResult> {
+  const { specPath, dryRun, verbose } = options;
+
+  try {
+    // Build issue title and body
+    const title = `[${spec.id.toUpperCase()}] ${spec.title}`;
+    const body = buildExternalDescription(spec);
+
+    if (verbose) {
+      console.log(`\n📝 Creating GitHub issue:`);
+      console.log(`   Title: ${title}`);
+      console.log(`   Body length: ${body.length} chars`);
+    }
+
+    if (dryRun) {
+      console.log('\n🔍 Dry run - would create issue:');
+      console.log(`   Title: ${title}`);
+      console.log(`   Body:\n${body}`);
+
+      return {
+        success: true,
+        action: 'created',
+        externalId: 'DRY-RUN',
+        externalUrl: 'https://github.com/DRY-RUN',
+      };
+    }
+
+    // Create issue
+    const labels = ['specweave', 'spec', spec.metadata.priority || 'P2'].filter(Boolean);
+    const issue = await client.createEpicIssue(title, body, undefined, labels);
+
+    if (verbose) {
+      console.log(`✅ Created issue #${issue.number}`);
+      console.log(`   URL: ${issue.html_url}`);
+    }
+
+    // Update spec with GitHub link
+    await updateSpecWithExternalLink(
+      specPath,
+      'github',
+      issue.number.toString(),
+      issue.html_url
+    );
+
+    return {
+      success: true,
+      action: 'created',
+      externalId: issue.number.toString(),
+      externalUrl: issue.html_url,
+    };
+  } catch (error: any) {
+    return {
+      success: false,
+      action: 'error',
+      error: `Failed to create GitHub issue: ${error.message}`,
+    };
+  }
+}
+
+/**
+ * Update existing GitHub issue with spec content
+ */
+async function updateGitHubIssue(
+  client: GitHubClientV2,
+  spec: SpecContent,
+  issueNumber: number,
+  options: GitHubContentSyncOptions
+): Promise<ContentSyncResult> {
+  const { specPath, dryRun, verbose } = options;
+
+  try {
+    // Get current issue
+    const issue = await client.getIssue(issueNumber);
+
+    if (verbose) {
+      console.log(`\n🔄 Checking for changes in issue #${issueNumber}`);
+    }
+
+    // Detect changes
+    const changes = detectContentChanges(spec, {
+      title: issue.title.replace(/^\[SPEC-\d+\]\s*/, ''),
+      description: issue.body || '',
+      userStoryCount: countUserStoriesInBody(issue.body || ''),
+    });
+
+    if (!changes.hasChanges) {
+      if (verbose) {
+        console.log('   ℹ️  No changes detected');
+      }
+
+      return {
+        success: true,
+        action: 'no-change',
+        externalId: issueNumber.toString(),
+        externalUrl: issue.html_url,
+      };
+    }
+
+    if (verbose) {
+      console.log('   📝 Changes detected:');
+      for (const change of changes.changes) {
+        console.log(`      - ${change}`);
+      }
+    }
+
+    // Build updated content
+    const newTitle = `[${spec.id.toUpperCase()}] ${spec.title}`;
+    const newBody = buildExternalDescription(spec);
+
+    if (dryRun) {
+      console.log('\n🔍 Dry run - would update issue:');
+      console.log(`   Title: ${newTitle}`);
+      console.log(`   Body:\n${newBody}`);
+
+      return {
+        success: true,
+        action: 'updated',
+        externalId: issueNumber.toString(),
+        externalUrl: issue.html_url,
+      };
+    }
+
+    // Update issue (ONLY title and description, NOT status!)
+    await client.updateIssueBody(issueNumber, newBody);
+
+    // Note: We do NOT update status/state - that's managed in GitHub
+    // We also do NOT close/reopen issues - that's done by humans/automation in GitHub
+
+    if (verbose) {
+      console.log(`✅ Updated issue #${issueNumber}`);
+    }
+
+    return {
+      success: true,
+      action: 'updated',
+      externalId: issueNumber.toString(),
+      externalUrl: issue.html_url,
+    };
+  } catch (error: any) {
+    return {
+      success: false,
+      action: 'error',
+      error: `Failed to update GitHub issue: ${error.message}`,
+    };
+  }
+}
+
+/**
+ * Count user stories in issue body
+ */
+function countUserStoriesInBody(body: string): number {
+  const matches = body.match(/###\s+US-\d+:/g);
+  return matches ? matches.length : 0;
+}
+
+/**
+ * Check if spec content sync is enabled
+ */
+export async function isContentSyncEnabled(projectRoot: string): Promise<boolean> {
+  try {
+    const configPath = path.join(projectRoot, '.specweave', 'config.json');
+    const config = JSON.parse(await fs.readFile(configPath, 'utf-8'));
+
+    return config.sync?.settings?.syncSpecContent !== false;
+  } catch {
+    return true; // Default: enabled
+  }
+}