specweave 2.3.0 → 3.0.0

package/plugins/specweave/lib/integrations/github/CodeValidator.ts

@@ -1,284 +0,0 @@
-/**
- * Code Validator
- *
- * Validates that completed tasks have actual code implementation.
- * Prevents marking tasks as complete when:
- * - Files don't exist
- * - Files are empty or have trivial content
- * - Implementation is incomplete
- *
- * Used by GitHub sync to enforce code-completion discipline.
- *
- * @module CodeValidator
- */
-
-import * as fs from '../../../../../src/utils/fs-native.js';
-import path from 'path';
-
-/**
- * File validation result
- */
-export interface FileValidationResult {
-  path: string;
-  exists: boolean;
-  hasContent: boolean;
-  lineCount: number;
-  reason?: string; // Why validation failed
-}
-
-/**
- * Task validation result
- */
-export interface TaskValidationResult {
-  taskId: string;
-  valid: boolean;
-  files: FileValidationResult[];
-  reason?: string; // Summary of why task validation failed
-}
-
-/**
- * CodeValidator options
- */
-export interface CodeValidatorOptions {
-  minLines?: number; // Minimum lines for a file to be considered non-empty (default: 3)
-  minChars?: number; // Minimum characters for meaningful content (default: 50)
-  projectRoot?: string; // Project root for resolving relative paths
-}
-
-export class CodeValidator {
-  private options: Required<CodeValidatorOptions>;
-
-  constructor(options: CodeValidatorOptions = {}) {
-    this.options = {
-      minLines: options.minLines ?? 3,
-      minChars: options.minChars ?? 50,
-      projectRoot: options.projectRoot ?? process.cwd()
-    };
-  }
-
-  /**
-   * Validate that code exists for a task
-   *
-   * Extracts file paths from task description and verifies:
-   * 1. Files exist
-   * 2. Files have meaningful content
-   * 3. Files are not just stubs
-   *
-   * @param taskDescription - Task description with file paths
-   * @param taskId - Task ID for error messages
-   * @returns Validation result
-   */
-  async validateTask(taskDescription: string, taskId: string): Promise<TaskValidationResult> {
-    const filePaths = this.extractFilePaths(taskDescription);
-
-    if (filePaths.length === 0) {
-      // No file paths specified - consider it valid (task might be non-code)
-      return {
-        taskId,
-        valid: true,
-        files: [],
-        reason: 'No file paths specified in task description'
-      };
-    }
-
-    const fileResults: FileValidationResult[] = [];
-    let allValid = true;
-    const reasons: string[] = [];
-
-    for (const filePath of filePaths) {
-      const result = await this.validateFile(filePath);
-      fileResults.push(result);
-
-      if (!result.exists) {
-        allValid = false;
-        reasons.push(`File not found: ${filePath}`);
-      } else if (!result.hasContent) {
-        allValid = false;
-        reasons.push(`File has no meaningful content: ${filePath} (${result.reason})`);
-      }
-    }
-
-    return {
-      taskId,
-      valid: allValid,
-      files: fileResults,
-      reason: reasons.length > 0 ? reasons.join('; ') : undefined
-    };
-  }
-
-  /**
-   * Validate a single file
-   *
-   * @param filePath - Path to file (relative or absolute)
-   * @returns File validation result
-   */
-  async validateFile(filePath: string): Promise<FileValidationResult> {
-    // Resolve relative paths
-    const absolutePath = path.isAbsolute(filePath)
-      ? filePath
-      : path.join(this.options.projectRoot, filePath);
-
-    // Check if file exists
-    const exists = await fs.pathExists(absolutePath);
-    if (!exists) {
-      return {
-        path: filePath,
-        exists: false,
-        hasContent: false,
-        lineCount: 0,
-        reason: 'File does not exist'
-      };
-    }
-
-    // Read file content
-    const content = await fs.readFile(absolutePath, 'utf-8');
-    const lines = content.split('\n');
-    const nonEmptyLines = lines.filter((line: string) => line.trim().length > 0);
-
-    // Check line count
-    if (nonEmptyLines.length < this.options.minLines) {
-      return {
-        path: filePath,
-        exists: true,
-        hasContent: false,
-        lineCount: nonEmptyLines.length,
-        reason: `Only ${nonEmptyLines.length} non-empty lines (minimum: ${this.options.minLines})`
-      };
-    }
-
-    // Check character count
-    const trimmedContent = content.trim();
-    if (trimmedContent.length < this.options.minChars) {
-      return {
-        path: filePath,
-        exists: true,
-        hasContent: false,
-        lineCount: nonEmptyLines.length,
-        reason: `Only ${trimmedContent.length} characters (minimum: ${this.options.minChars})`
-      };
-    }
-
-    // Check for stub patterns (common placeholder patterns)
-    const stubPatterns = [
-      /^\/\/\s*TODO:/i,
-      /^#\s*TODO:/i,
-      /^\s*throw new Error\(['"]Not implemented['"]\)/i,
-      /^\s*return null;?\s*$/m,
-      /^\s*pass\s*$/m, // Python
-      /^\s*\.\.\.$/m    // TypeScript
-    ];
-
-    const isStub = stubPatterns.some(pattern => pattern.test(trimmedContent));
-    if (isStub) {
-      return {
-        path: filePath,
-        exists: true,
-        hasContent: false,
-        lineCount: nonEmptyLines.length,
-        reason: 'File contains stub/placeholder code'
-      };
-    }
-
-    // All checks passed
-    return {
-      path: filePath,
-      exists: true,
-      hasContent: true,
-      lineCount: nonEmptyLines.length
-    };
-  }
-
-  /**
-   * Extract file paths from task description
-   *
-   * Supports multiple formats:
-   * - **Files**: src/foo.ts, src/bar.ts
-   * - **Files to create**: src/foo.ts
-   * - **Files to modify**: src/bar.ts
-   * - Inline code blocks with file paths
-   *
-   * @param description - Task description text
-   * @returns Array of file paths
-   */
-  extractFilePaths(description: string): string[] {
-    const paths: Set<string> = new Set();
-
-    // Pattern 1: **Files**: path1, path2, path3
-    const filesMatch = description.match(/\*\*Files\*\*:\s*([^\n]+)/i);
-    if (filesMatch) {
-      const filePaths = filesMatch[1].split(',').map(p => p.trim());
-      filePaths.forEach(p => paths.add(p));
-    }
-
-    // Pattern 2: **Files to create**: path1, path2
-    const createMatch = description.match(/\*\*Files to create\*\*:\s*([^\n]+)/i);
-    if (createMatch) {
-      const filePaths = createMatch[1].split(',').map(p => p.trim());
-      filePaths.forEach(p => paths.add(p));
-    }
-
-    // Pattern 3: **Files to modify**: path1, path2
-    const modifyMatch = description.match(/\*\*Files to modify\*\*:\s*([^\n]+)/i);
-    if (modifyMatch) {
-      const filePaths = modifyMatch[1].split(',').map(p => p.trim());
-      filePaths.forEach(p => paths.add(p));
-    }
-
-    // Pattern 4: Inline file references (e.g., `src/foo/bar.ts`)
-    const inlineMatches = description.matchAll(/`([a-zA-Z0-9_\-./]+\.(ts|js|tsx|jsx|py|java|go|rs|cpp|c|h))`/g);
-    for (const match of inlineMatches) {
-      paths.add(match[1]);
-    }
-
-    // Pattern 5: Markdown list items with file paths
-    const listMatches = description.matchAll(/^[-*]\s+([a-zA-Z0-9_\-./]+\.(ts|js|tsx|jsx|py|java|go|rs|cpp|c|h))/gm);
-    for (const match of listMatches) {
-      paths.add(match[1]);
-    }
-
-    return Array.from(paths);
-  }
-
-  /**
-   * Batch validate multiple tasks
-   *
-   * @param tasks - Array of {taskId, description}
-   * @returns Array of validation results
-   */
-  async validateTasks(tasks: Array<{ taskId: string; description: string }>): Promise<TaskValidationResult[]> {
-    // Use parallel validation for performance
-    const validationPromises = tasks.map(task =>
-      this.validateTask(task.description, task.taskId)
-    );
-
-    return Promise.all(validationPromises);
-  }
-
-  /**
-   * Get summary of validation results
-   *
-   * @param results - Array of task validation results
-   * @returns Summary statistics
-   */
-  summarizeResults(results: TaskValidationResult[]): {
-    total: number;
-    valid: number;
-    invalid: number;
-    noFiles: number;
-    invalidTasks: string[];
-  } {
-    const total = results.length;
-    const valid = results.filter(r => r.valid).length;
-    const invalid = results.filter(r => !r.valid).length;
-    const noFiles = results.filter(r => r.files.length === 0).length;
-    const invalidTasks = results.filter(r => !r.valid).map(r => r.taskId);
-
-    return {
-      total,
-      valid,
-      invalid,
-      noFiles,
-      invalidTasks
-    };
-  }
-}

package/plugins/specweave/lib/integrations/github/github-batch-sync.js

@@ -1,116 +0,0 @@
-import { glob } from "glob";
-import { readFile } from "fs/promises";
-import { join } from "path";
-import { GitHubSyncOrchestrator } from "./github-sync-orchestrator.js";
-async function batchSyncAllSpecs(config) {
-  const specsDir = join(config.workspaceRoot, ".specweave/docs/internal/specs");
-  const specFiles = await glob("**/spec-*.md", { cwd: specsDir });
-  const result = {
-    specsProcessed: 0,
-    specsFailed: 0,
-    totalIssuesCreated: 0,
-    totalIssuesUpdated: 0,
-    errors: [],
-    summary: ""
-  };
-  if (specFiles.length === 0) {
-    result.summary = "No specs found";
-    return result;
-  }
-  const orchestrator = new GitHubSyncOrchestrator({
-    owner: config.owner,
-    repo: config.repo,
-    token: config.token,
-    dryRun: config.dryRun,
-    projectV2Enabled: config.projectV2Enabled,
-    projectV2Number: config.projectV2Number,
-    projectV2Id: config.projectV2Id,
-    statusFieldMapping: config.statusFieldMapping,
-    priorityFieldMapping: config.priorityFieldMapping
-  });
-  for (const specFile of specFiles) {
-    const specPath = join(specsDir, specFile);
-    result.specsProcessed++;
-    try {
-      const content = await readFile(specPath, "utf-8");
-      const userStories = parseUserStoriesFromSpec(content);
-      const syncResult = await orchestrator.syncSpec(specPath, userStories);
-      result.totalIssuesCreated += syncResult.pushResult.created.length;
-      result.totalIssuesUpdated += syncResult.pushResult.updated.length;
-    } catch (err) {
-      result.specsFailed++;
-      result.errors.push({
-        specPath,
-        error: err instanceof Error ? err.message : String(err)
-      });
-    }
-  }
-  const synced = result.specsProcessed - result.specsFailed;
-  result.summary = `Synced ${synced}/${result.specsProcessed} specs, ${result.totalIssuesCreated} issues created, ${result.totalIssuesUpdated} updated`;
-  return result;
-}
-const US_HEADER_RE = /^###?\s+(US-\d{3,}E?):\s*(.+)$/;
-const AC_CHECKBOX_RE = /^-\s+\[([ xX])\]\s+\*\*(?<acId>AC-US\d+E?-\d{2})\*\*:\s*(?<desc>.+)$/;
-const PRIORITY_RE = /\*\*Priority\*\*:\s*(P[0-3])/;
-const AC_SECTION_RE = /^\*\*(?:Acceptance Criteria|ACs?)\*\*:/i;
-const METADATA_RE = /^\*\*(?:Project|Board|External)\*\*:/i;
-function parseUserStoriesFromSpec(content) {
-  const lines = content.split("\n");
-  const userStories = [];
-  let current = null;
-  let collectingDescription = false;
-  for (const line of lines) {
-    const usMatch = line.match(US_HEADER_RE);
-    if (usMatch) {
-      if (current) {
-        current.description = current.description.trim();
-        userStories.push(current);
-      }
-      current = {
-        id: usMatch[1],
-        title: usMatch[2],
-        description: "",
-        priority: "P3",
-        status: "planned",
-        acceptanceCriteria: []
-      };
-      collectingDescription = true;
-      continue;
-    }
-    if (!current) continue;
-    const acMatch = line.match(AC_CHECKBOX_RE);
-    if (acMatch?.groups) {
-      collectingDescription = false;
-      current.acceptanceCriteria.push({
-        id: acMatch.groups.acId,
-        description: acMatch.groups.desc.trim(),
-        completed: acMatch[1] !== " "
-      });
-      continue;
-    }
-    const prioMatch = line.match(PRIORITY_RE);
-    if (prioMatch) {
-      collectingDescription = false;
-      current.priority = prioMatch[1];
-      continue;
-    }
-    if (AC_SECTION_RE.test(line)) {
-      collectingDescription = false;
-      continue;
-    }
-    if (collectingDescription) {
-      if (METADATA_RE.test(line)) continue;
-      if (line.trim() !== "" || current.description.length > 0) {
-        current.description += line + "\n";
-      }
-    }
-  }
-  if (current) {
-    current.description = current.description.trim();
-    userStories.push(current);
-  }
-  return userStories;
-}
-export {
-  batchSyncAllSpecs
-};

package/plugins/specweave/lib/integrations/github/github-batch-sync.ts

@@ -1,180 +0,0 @@
-/**
- * GitHub Batch Sync — Sync all specs in workspace
- *
- * Discovers all spec files and syncs them sequentially
- * using the GitHubSyncOrchestrator.
- *
- * @module github-batch-sync
- */
-
-import { glob } from 'glob';
-import { readFile } from 'fs/promises';
-import { join } from 'path';
-import { GitHubSyncOrchestrator } from './github-sync-orchestrator.js';
-import type { UserStoryForSync } from './github-push-sync.js';
-
-export interface BatchSyncConfig {
-  owner: string;
-  repo: string;
-  workspaceRoot: string;
-  token?: string;
-  dryRun?: boolean;
-  projectV2Enabled?: boolean;
-  projectV2Number?: number;
-  projectV2Id?: string;
-  statusFieldMapping?: Record<string, string>;
-  priorityFieldMapping?: Record<string, string>;
-}
-
-export interface BatchSyncResult {
-  specsProcessed: number;
-  specsFailed: number;
-  totalIssuesCreated: number;
-  totalIssuesUpdated: number;
-  errors: Array<{ specPath: string; error: string }>;
-  summary: string;
-}
-
-/**
- * Discover all specs and sync them sequentially.
- */
-export async function batchSyncAllSpecs(config: BatchSyncConfig): Promise<BatchSyncResult> {
-  const specsDir = join(config.workspaceRoot, '.specweave/docs/internal/specs');
-
-  // Discover specs
-  const specFiles = await glob('**/spec-*.md', { cwd: specsDir });
-
-  const result: BatchSyncResult = {
-    specsProcessed: 0,
-    specsFailed: 0,
-    totalIssuesCreated: 0,
-    totalIssuesUpdated: 0,
-    errors: [],
-    summary: '',
-  };
-
-  if (specFiles.length === 0) {
-    result.summary = 'No specs found';
-    return result;
-  }
-
-  // Create orchestrator
-  const orchestrator = new GitHubSyncOrchestrator({
-    owner: config.owner,
-    repo: config.repo,
-    token: config.token,
-    dryRun: config.dryRun,
-    projectV2Enabled: config.projectV2Enabled,
-    projectV2Number: config.projectV2Number,
-    projectV2Id: config.projectV2Id,
-    statusFieldMapping: config.statusFieldMapping,
-    priorityFieldMapping: config.priorityFieldMapping,
-  });
-
-  // Sync each spec sequentially
-  for (const specFile of specFiles) {
-    const specPath = join(specsDir, specFile);
-    result.specsProcessed++;
-
-    try {
-      const content = await readFile(specPath, 'utf-8');
-      const userStories = parseUserStoriesFromSpec(content);
-      const syncResult = await orchestrator.syncSpec(specPath, userStories);
-
-      result.totalIssuesCreated += syncResult.pushResult.created.length;
-      result.totalIssuesUpdated += syncResult.pushResult.updated.length;
-    } catch (err) {
-      result.specsFailed++;
-      result.errors.push({
-        specPath,
-        error: err instanceof Error ? err.message : String(err),
-      });
-    }
-  }
-
-  const synced = result.specsProcessed - result.specsFailed;
-  result.summary = `Synced ${synced}/${result.specsProcessed} specs, ${result.totalIssuesCreated} issues created, ${result.totalIssuesUpdated} updated`;
-
-  return result;
-}
-
-// US header: ### US-001: Title
-const US_HEADER_RE = /^###?\s+(US-\d{3,}E?):\s*(.+)$/;
-// AC checkbox: - [x] **AC-US1-01**: Description
-const AC_CHECKBOX_RE = /^-\s+\[([ xX])\]\s+\*\*(?<acId>AC-US\d+E?-\d{2})\*\*:\s*(?<desc>.+)$/;
-// Priority: **Priority**: P1
-const PRIORITY_RE = /\*\*Priority\*\*:\s*(P[0-3])/;
-// Section headers that end description collection
-const AC_SECTION_RE = /^\*\*(?:Acceptance Criteria|ACs?)\*\*:/i;
-// Metadata lines to exclude from description
-const METADATA_RE = /^\*\*(?:Project|Board|External)\*\*:/i;
-
-/**
- * Parse user stories from spec.md content.
- */
-function parseUserStoriesFromSpec(content: string): UserStoryForSync[] {
-  const lines = content.split('\n');
-  const userStories: UserStoryForSync[] = [];
-  let current: UserStoryForSync | null = null;
-  let collectingDescription = false;
-
-  for (const line of lines) {
-    const usMatch = line.match(US_HEADER_RE);
-    if (usMatch) {
-      if (current) {
-        current.description = current.description.trim();
-        userStories.push(current);
-      }
-      current = {
-        id: usMatch[1],
-        title: usMatch[2],
-        description: '',
-        priority: 'P3',
-        status: 'planned',
-        acceptanceCriteria: [],
-      };
-      collectingDescription = true;
-      continue;
-    }
-
-    if (!current) continue;
-
-    const acMatch = line.match(AC_CHECKBOX_RE);
-    if (acMatch?.groups) {
-      collectingDescription = false;
-      current.acceptanceCriteria.push({
-        id: acMatch.groups.acId,
-        description: acMatch.groups.desc.trim(),
-        completed: acMatch[1] !== ' ',
-      });
-      continue;
-    }
-
-    const prioMatch = line.match(PRIORITY_RE);
-    if (prioMatch) {
-      collectingDescription = false;
-      current.priority = prioMatch[1];
-      continue;
-    }
-
-    if (AC_SECTION_RE.test(line)) {
-      collectingDescription = false;
-      continue;
-    }
-
-    // Collect description lines between US header and first AC/Priority
-    if (collectingDescription) {
-      if (METADATA_RE.test(line)) continue;
-      if (line.trim() !== '' || current.description.length > 0) {
-        current.description += line + '\n';
-      }
-    }
-  }
-
-  if (current) {
-    current.description = current.description.trim();
-    userStories.push(current);
-  }
-
-  return userStories;
-}

package/plugins/specweave/lib/integrations/github/github-board-resolver-v2.js

@@ -1,47 +0,0 @@
-class GitHubBoardResolverV2 {
-  constructor(client, config) {
-    this.client = client;
-    this.config = config;
-  }
-  /**
-   * Find an existing project or create a new one.
-   *
-   * Priority:
-   * 1. If projectV2Id is configured, use it directly (skip lookup)
-   * 2. If projectV2Number is configured, return with that number
-   * 3. Otherwise, create a new project
-   */
-  async findOrCreateProject(title) {
-    if (this.config.projectV2Id) {
-      return {
-        id: this.config.projectV2Id,
-        number: this.config.projectV2Number || 0
-      };
-    }
-    if (this.config.projectV2Number) {
-      const ownerId2 = await this.client.getOwnerNodeId(this.config.owner);
-      return {
-        id: ownerId2,
-        // Will be resolved to actual project ID during operations
-        number: this.config.projectV2Number
-      };
-    }
-    const ownerId = await this.client.getOwnerNodeId(this.config.owner);
-    return this.client.createProjectV2(ownerId, title);
-  }
-  /**
-   * Add issues (by node ID) to a Projects V2 board.
-   * Returns the project item IDs for each added issue.
-   */
-  async addIssuesToProject(projectId, issueNodeIds) {
-    const itemIds = [];
-    for (const nodeId of issueNodeIds) {
-      const itemId = await this.client.addProjectV2Item(projectId, nodeId);
-      itemIds.push(itemId);
-    }
-    return itemIds;
-  }
-}
-export {
-  GitHubBoardResolverV2
-};

package/plugins/specweave/lib/integrations/github/github-board-resolver-v2.ts

@@ -1,73 +0,0 @@
-/**
- * GitHub Board Resolver V2 — Projects V2 Integration
- *
- * Finds or creates GitHub Projects V2 boards and adds issues to them.
- * Replaces the deprecated V1 Classic Projects board resolver.
- *
- * @module github-board-resolver-v2
- */
-
-import { GitHubGraphQLClient } from './github-graphql-client.js';
-
-export interface BoardResolverConfig {
-  owner: string;
-  projectV2Number?: number;
-  projectV2Id?: string;
-}
-
-export class GitHubBoardResolverV2 {
-  private client: GitHubGraphQLClient;
-  private config: BoardResolverConfig;
-
-  constructor(client: GitHubGraphQLClient, config: BoardResolverConfig) {
-    this.client = client;
-    this.config = config;
-  }
-
-  /**
-   * Find an existing project or create a new one.
-   *
-   * Priority:
-   * 1. If projectV2Id is configured, use it directly (skip lookup)
-   * 2. If projectV2Number is configured, return with that number
-   * 3. Otherwise, create a new project
-   */
-  async findOrCreateProject(title: string): Promise<{ id: string; number: number }> {
-    // If we have a direct project ID, use it
-    if (this.config.projectV2Id) {
-      return {
-        id: this.config.projectV2Id,
-        number: this.config.projectV2Number || 0,
-      };
-    }
-
-    // If we have a project number, look it up by querying the owner
-    if (this.config.projectV2Number) {
-      const ownerId = await this.client.getOwnerNodeId(this.config.owner);
-      // Return the project reference — the number is known, ID will be resolved on first use
-      return {
-        id: ownerId, // Will be resolved to actual project ID during operations
-        number: this.config.projectV2Number,
-      };
-    }
-
-    // No project configured — create a new one
-    const ownerId = await this.client.getOwnerNodeId(this.config.owner);
-    return this.client.createProjectV2(ownerId, title);
-  }
-
-  /**
-   * Add issues (by node ID) to a Projects V2 board.
-   * Returns the project item IDs for each added issue.
-   */
-  async addIssuesToProject(projectId: string, issueNodeIds: string[]): Promise<string[]> {
-    const itemIds: string[] = [];
-
-    for (const nodeId of issueNodeIds) {
-      const itemId = await this.client.addProjectV2Item(projectId, nodeId);
-      itemIds.push(itemId);
-    }
-
-    return itemIds;
-  }
-}