specweave 1.0.318 → 1.0.320

package/dist/src/sync/sync-coordinator.d.ts

@@ -4,7 +4,6 @@
  * High-level coordinator that integrates FormatPreservationSyncService
  * with living docs sync workflow. Called by post-task-completion hook.
  */
-import { GitHubIssue } from '../../plugins/specweave-github/lib/types.js';
 import { Logger } from '../utils/logger.js';
 import { ResolvedAdoProfile } from '../integrations/ado/ado-client-factory.js';
 import { ClosureMetrics } from './closure-metrics.js';
@@ -48,36 +47,13 @@ export declare class SyncCoordinator {
      */
     getFormattedClosureMetrics(): string;
     /**
-     * Create GitHub issues for all User Stories in the feature (NEW in v0.25.0)
-     *
-     * This is the AUTOMATIC GitHub sync that runs when increment completes.
-     * Creates issues for User Stories that don't have GitHub issues yet.
-     *
-     * Uses 3-layer idempotency:
-     * - Layer 1: Check user story frontmatter (fastest, <1ms)
-     * - Layer 2: Check increment metadata.json (fast, <5ms)
-     * - Layer 3: Query GitHub API (slow but authoritative, 500-2000ms)
-     *
-     * @param config - Project configuration
-     * @returns Array of created issues
+     * @deprecated (0348) GitHub issue creation now handled by GitHubFeatureSync via LivingDocsSync chain.
      */
-    createGitHubIssuesForUserStories(config: SpecWeaveConfig): Promise<GitHubIssue[]>;
+    createGitHubIssuesForUserStories(_config: SpecWeaveConfig): Promise<any[]>;
     /**
-     * Close GitHub issues for all User Stories when increment completes (NEW in v0.28.1)
-     *
-     * This is the CRITICAL missing feature that was causing GitHub issues to remain
-     * open after increments were closed.
-     *
-     * Called by: syncIncrementClosure() when increment status changes to "completed"
-     *
-     * Flow:
-     * 1. Get feature ID from increment spec
-     * 2. Load all user stories for that feature
-     * 3. For each user story with a GitHub issue, close it with completion comment
-     *
-     * @param config - Project configuration
-     * @returns Array of closed issue numbers
+     * @deprecated (0348) GitHub closure now handled by GitHubFeatureSync via LivingDocsSync chain.
      */
+    closeGitHubIssuesForUserStories(_config: SpecWeaveConfig): Promise<number[]>;
     /**
      * Close JIRA issues for completed user stories
      *
@@ -114,31 +90,6 @@ export declare class SyncCoordinator {
     syncIncrementClosure(): Promise<SyncResult & {
         closedIssues: number[];
     }>;
-    /**
-     * Detect duplicate issues with different feature ID formats
-     *
-     * Searches for issues that match the user story but have a different feature ID format.
-     * This prevents creating duplicates like:
-     * - [FS-0128][US-001] (old bug format with leading zeros)
-     * - [FS-128][US-001] (correct format)
-     *
-     * The search uses a regex pattern that matches any FS-XXX format for the same US-XXX.
-     *
-     * @param client - GitHub client
-     * @param featureId - Current feature ID (e.g., "FS-128")
-     * @param userStoryId - User story ID (e.g., "US-001")
-     * @returns Duplicate info if found, null otherwise
-     */
-    private detectDuplicateIssue;
-    /**
-     * Update issue if it has placeholder content
-     * Fetches issue from GitHub, checks for placeholder, and updates with rich content
-     */
-    private updateIssueIfPlaceholder;
-    /**
-     * Format user story content for GitHub issue body
-     */
-    private formatUserStoryBody;
     /**
      * Sync increment completion to external tools using format preservation
      */
@@ -182,18 +133,5 @@ export declare class SyncCoordinator {
      * Emoji checkmarks work in both plain text and ADF contexts.
      */
     private formatJiraCompletionComment;
-    /**
-     * Sync AC checkbox status to GitHub issues
-     *
-     * @deprecated (0348) Delegates to GitHubACCheckboxSync in the GitHub plugin.
-     * Kept for backward compatibility with callers that haven't migrated yet.
-     */
-    syncACCheckboxesToGitHub(config: SpecWeaveConfig, options?: {
-        addComment?: boolean;
-    }): Promise<{
-        success: boolean;
-        updated: number;
-        issues: number[];
-    }>;
 }
 //# sourceMappingURL=sync-coordinator.d.ts.map

package/dist/src/sync/sync-coordinator.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"sync-coordinator.d.ts","sourceRoot":"","sources":["../../../src/sync/sync-coordinator.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAQH,OAAO,EAAE,WAAW,EAAE,MAAM,6CAA6C,CAAC;AAC1E,OAAO,EAAE,MAAM,EAAiB,MAAM,oBAAoB,CAAC;AAK3D,OAAO,EAAE,kBAAkB,EAAE,MAAM,2CAA2C,CAAC;AAG/E,OAAO,EAAE,cAAc,EAAwB,MAAM,sBAAsB,CAAC;AAE5E,OAAO,KAAK,EACV,eAAe,EAEhB,MAAM,yBAAyB,CAAC;AAcjC,OAAO,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAC;AAiCvD,MAAM,WAAW,sBAAsB;IACrC,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB;;;OAGG;IACH,UAAU,CAAC,EAAE,kBAAkB,CAAC;CACjC;AAED,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,OAAO,CAAC;IACjB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,QAAQ,EAAE,cAAc,GAAG,WAAW,GAAG,WAAW,GAAG,aAAa,GAAG,kBAAkB,GAAG,mBAAmB,CAAC;IAChH,MAAM,EAAE,MAAM,EAAE,CAAC;CAClB;AAED,qBAAa,eAAe;IAC1B,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,MAAM,CAAS;IACvB,OAAO,CAAC,kBAAkB,CAAqB;IAC/C,OAAO,CAAC,SAAS,CAAS;IAC1B,OAAO,CAAC,UAAU,CAAC,CAAqB;IACxC,OAAO,CAAC,OAAO,CAAiB;IAChC,OAAO,CAAC,cAAc,CAAiB;gBAE3B,OAAO,EAAE,sBAAsB;IAkB3C;;;;;OAKG;IACH,iBAAiB,IAAI,UAAU,CAAC,cAAc,CAAC,YAAY,CAAC,CAAC;IAI7D;;OAEG;IACH,0BAA0B,IAAI,MAAM;IAIpC;;;;;;;;;;;;;OAaG;IACG,gCAAgC,CAAC,MAAM,EAAE,eAAe,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;IA+WvF;;;;;;;;;;;;;;;OAeG;IACH;;;;;;;;OAQG;IACG,6BAA6B,CAAC,MAAM,EAAE,eAAe,GAAG,OAAO,CAAC,MAAM,CAAC;IA+E7E;;;;;;;;OAQG;IACG,+BAA+B,CAAC,MAAM,EAAE,eAAe,GAAG,OAAO,CAAC,MAAM,CAAC;IA0G/E;;;;;;;;;;;;OAYG;IACG,oBAAoB,IAAI,OAAO,CAAC,UAAU,GAAG;QAAE,YAAY,EAAE,MAAM,EAAE,CAAA;KAAE,CAAC;IA0J9E;;;;;;;;;;;;;;OAcG;YACW,oBAAoB;IAsDlC;;;OAGG;YACW,wBAAwB;IA2DtC;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAiB3B;;OAEG;IACG,uBAAuB,IAAI,OAAO,CAAC,UAAU,CAAC;IAuHpD;;OAEG;YACW,aAAa;IA6L3B;;OAEG;YACW,kBAAkB;IAmEhC;;OAEG;YACW,2BAA2B;IAyGzC;;;;OAIG;IACH,OAAO,CAAC,wBAAwB;IA0BhC;;OAEG;YACW,UAAU;IAWxB;;;OAGG;YACW,gBAAgB;IAI9B;;OAEG;IACH,OAAO,CAAC,0BAA0B;IA0ClC;;;;;;OAMG;IACH,OAAO,CAAC,2BAA2B;IA0CnC;;;;;OAKG;IACG,wBAAwB,CAC5B,MAAM,EAAE,eAAe,EACvB,OAAO,GAAE;QAAE,UAAU,CAAC,EAAE,OAAO,CAAA;KAAO,GACrC,OAAO,CAAC;QAAE,OAAO,EAAE,OAAO,CAAC;QAAC,OAAO,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,EAAE,CAAA;KAAE,CAAC;CAWpE"}
+{"version":3,"file":"sync-coordinator.d.ts","sourceRoot":"","sources":["../../../src/sync/sync-coordinator.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAQH,OAAO,EAAE,MAAM,EAAiB,MAAM,oBAAoB,CAAC;AAI3D,OAAO,EAAE,kBAAkB,EAAE,MAAM,2CAA2C,CAAC;AAG/E,OAAO,EAAE,cAAc,EAAwB,MAAM,sBAAsB,CAAC;AAE5E,OAAO,KAAK,EACV,eAAe,EAEhB,MAAM,yBAAyB,CAAC;AAcjC,OAAO,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAC;AAWvD,MAAM,WAAW,sBAAsB;IACrC,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB;;;OAGG;IACH,UAAU,CAAC,EAAE,kBAAkB,CAAC;CACjC;AAED,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,OAAO,CAAC;IACjB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,QAAQ,EAAE,cAAc,GAAG,WAAW,GAAG,WAAW,GAAG,aAAa,GAAG,kBAAkB,GAAG,mBAAmB,CAAC;IAChH,MAAM,EAAE,MAAM,EAAE,CAAC;CAClB;AAED,qBAAa,eAAe;IAC1B,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,MAAM,CAAS;IACvB,OAAO,CAAC,kBAAkB,CAAqB;IAC/C,OAAO,CAAC,SAAS,CAAS;IAC1B,OAAO,CAAC,UAAU,CAAC,CAAqB;IACxC,OAAO,CAAC,OAAO,CAAiB;IAChC,OAAO,CAAC,cAAc,CAAiB;gBAE3B,OAAO,EAAE,sBAAsB;IAkB3C;;;;;OAKG;IACH,iBAAiB,IAAI,UAAU,CAAC,cAAc,CAAC,YAAY,CAAC,CAAC;IAI7D;;OAEG;IACH,0BAA0B,IAAI,MAAM;IAKpC;;OAEG;IACG,gCAAgC,CAAC,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;IAKhF;;OAEG;IACG,+BAA+B,CAAC,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC;IAKlF;;;;;;;;OAQG;IACG,6BAA6B,CAAC,MAAM,EAAE,eAAe,GAAG,OAAO,CAAC,MAAM,CAAC;IA+E7E;;;;;;;;OAQG;IACG,+BAA+B,CAAC,MAAM,EAAE,eAAe,GAAG,OAAO,CAAC,MAAM,CAAC;IA0G/E;;;;;;;;;;;;OAYG;IACG,oBAAoB,IAAI,OAAO,CAAC,UAAU,GAAG;QAAE,YAAY,EAAE,MAAM,EAAE,CAAA;KAAE,CAAC;IA2J9E;;OAEG;IACG,uBAAuB,IAAI,OAAO,CAAC,UAAU,CAAC;IAuHpD;;OAEG;YACW,aAAa;IA6L3B;;OAEG;YACW,kBAAkB;IAmEhC;;OAEG;YACW,2BAA2B;IAyGzC;;;;OAIG;IACH,OAAO,CAAC,wBAAwB;IA0BhC;;OAEG;YACW,UAAU;IAWxB;;;OAGG;YACW,gBAAgB;IAI9B;;OAEG;IACH,OAAO,CAAC,0BAA0B;IA0ClC;;;;;;OAMG;IACH,OAAO,CAAC,2BAA2B;CA0CpC"}

package/dist/src/sync/sync-coordinator.js

@@ -13,7 +13,6 @@ import { GitHubClientV2 } from '../../plugins/specweave-github/lib/github-client
 import { consoleLogger } from '../utils/logger.js';
 import { FrontmatterUpdater } from './frontmatter-updater.js';
 import { autoDetectProjectIdSync } from '../utils/project-detection.js';
-import { UserStoryContentBuilder } from '../../plugins/specweave-github/lib/user-story-content-builder.js';
 import { AdoClient } from '../integrations/ado/ado-client.js';
 import { getAdoPat } from '../integrations/ado/ado-pat-provider.js';
 import { deriveFeatureId } from '../utils/feature-id-derivation.js';
@@ -58,333 +57,19 @@ export class SyncCoordinator {
         return this.metrics.formatSummary();
     }
     /**
-     * Create GitHub issues for all User Stories in the feature (NEW in v0.25.0)
-     *
-     * This is the AUTOMATIC GitHub sync that runs when increment completes.
-     * Creates issues for User Stories that don't have GitHub issues yet.
-     *
-     * Uses 3-layer idempotency:
-     * - Layer 1: Check user story frontmatter (fastest, <1ms)
-     * - Layer 2: Check increment metadata.json (fast, <5ms)
-     * - Layer 3: Query GitHub API (slow but authoritative, 500-2000ms)
-     *
-     * @param config - Project configuration
-     * @returns Array of created issues
+     * @deprecated (0348) GitHub issue creation now handled by GitHubFeatureSync via LivingDocsSync chain.
      */
-    async createGitHubIssuesForUserStories(config) {
-        const createdIssues = [];
-        try {
-            // Load user stories for this increment
-            const userStories = await this.loadUserStoriesForIncrement();
-            if (userStories.length === 0) {
-                this.logger.log('📚 No user stories found for this increment');
-                return createdIssues;
-            }
-            this.logger.log(`📚 Found ${userStories.length} user story/stories for GitHub sync`);
-            // Get GitHub config
-            const githubConfig = config.sync?.github || {};
-            const repoInfo = await this.detectGitHubRepo(githubConfig);
-            if (!repoInfo) {
-                throw new Error('GitHub repository not configured');
-            }
-            const client = GitHubClientV2.fromRepo(repoInfo.owner, repoInfo.repo);
-            // Get feature ID from increment spec
-            const specFile = path.join(this.projectRoot, '.specweave/increments', this.incrementId, 'spec.md');
-            let featureId = '';
-            if (existsSync(specFile)) {
-                const content = await fs.readFile(specFile, 'utf-8');
-                const frontmatterMatch = content.match(/^---\n([\s\S]*?)\n---/);
-                if (frontmatterMatch) {
-                    const frontmatter = yaml.parse(frontmatterMatch[1]);
-                    featureId = frontmatter.feature_id || frontmatter.epic || frontmatter.feature || '';
-                }
-            }
-            if (!featureId) {
-                // AUTO-GENERATE feature ID using deriveFeatureId() (ADR-0139)
-                // CRITICAL FIX: Must use deriveFeatureId() to get correct format (FS-128, not FS-0128)
-                // The old code used raw regex match which preserved leading zeros, causing duplicates
-                try {
-                    featureId = deriveFeatureId(this.incrementId);
-                    this.logger.log(`📝 Auto-generated feature ID: ${featureId} (no epic/feature_id in spec.md)`);
-                }
-                catch {
-                    this.logger.log('⚠️  No feature ID found and could not auto-generate - skipping GitHub sync');
-                    this.logger.log('   💡 Add epic: FS-XXX or feature_id: FS-XXX to spec.md frontmatter');
-                    return createdIssues;
-                }
-            }
-            // Try to find or create milestone for the feature
-            let milestoneNumber = null;
-            try {
-                const milestone = await client.createOrGetMilestone(`${featureId}: Automatic GitHub Sync`, `Feature milestone for ${featureId}`, 30 // 30 days from now
-                );
-                milestoneNumber = milestone.number;
-                this.logger.log(`🎯 Using milestone: ${milestone.title} (#${milestone.number})`);
-            }
-            catch (error) {
-                this.logger.log('⚠️  Could not create/get milestone, continuing without it');
-            }
-            // Create issue for each user story (with idempotency + atomic locking)
-            // Ensure locks directory exists (once per batch, not per user story)
-            const locksDir = path.join(this.projectRoot, '.specweave/state/.locks');
-            if (!existsSync(locksDir)) {
-                await fs.mkdir(locksDir, { recursive: true });
-            }
-            for (const usFile of userStories) {
-                // ATOMIC LOCK: Prevent race conditions when multiple syncs run concurrently
-                // Lock is per-user-story to allow parallel processing of different stories
-                const lockDir = path.join(locksDir, `github-issue-${featureId}-${usFile.id}`.toLowerCase().replace(/[^a-z0-9-]/g, '-'));
-                const lockManager = new LockManager(lockDir, 60, { logger: this.logger }); // 60s stale threshold
-                let lockAcquired = false;
-                try {
-                    lockAcquired = await lockManager.acquire();
-                    if (!lockAcquired) {
-                        this.logger.log(`  ⏳ ${usFile.id} - Another sync in progress, skipping (lock not acquired)`);
-                        continue;
-                    }
-                    // LAYER 1: Check user story frontmatter for existing GitHub issue
-                    const cachedIssue = await this.frontmatterUpdater.getGitHubIssueFromFrontmatter(this.projectRoot, featureId, usFile.id);
-                    if (cachedIssue) {
-                        this.logger.log(`  🔍 ${usFile.id} - Issue #${cachedIssue.number} exists (cached)`);
-                        // Check if issue needs content update (has placeholder body)
-                        await this.updateIssueIfPlaceholder(cachedIssue.number, usFile.id, featureId, client, repoInfo);
-                        continue;
-                    }
-                    // LAYER 2: Check increment metadata.json
-                    const metadataFile = path.join(this.projectRoot, '.specweave/increments', this.incrementId, 'metadata.json');
-                    let existingIssue = null;
-                    if (existsSync(metadataFile)) {
-                        const metadata = JSON.parse(await fs.readFile(metadataFile, 'utf-8'));
-                        // Check OLD format (metadata.github.issues[])
-                        const githubIssues = metadata.github?.issues || [];
-                        let found = githubIssues.find((i) => i.userStory === usFile.id);
-                        // v1.0.240 FIX: Also check NEW format (externalLinks.github.issues{})
-                        if (!found) {
-                            const extLinks = metadata.externalLinks?.github?.issues;
-                            if (extLinks && extLinks[usFile.id]?.issueNumber) {
-                                found = {
-                                    userStory: usFile.id,
-                                    number: extLinks[usFile.id].issueNumber,
-                                    url: extLinks[usFile.id].issueUrl || '',
-                                    createdAt: new Date().toISOString(),
-                                };
-                                this.logger.log(`  🔍 ${usFile.id} - Issue #${found.number} exists (externalLinks)`);
-                            }
-                        }
-                        if (found) {
-                            this.logger.log(`  🔍 ${usFile.id} - Issue #${found.number} exists (metadata)`);
-                            // Check if issue needs content update
-                            await this.updateIssueIfPlaceholder(found.number, usFile.id, featureId, client, repoInfo);
-                            existingIssue = found.number;
-                            continue;
-                        }
-                    }
-                    // LAYER 3: Query GitHub API to check for existing issue
-                    const searchTitle = `[${featureId}][${usFile.id}]`;
-                    const existingOnGitHub = await client.searchIssueByTitle(searchTitle);
-                    if (existingOnGitHub) {
-                        this.logger.log(`  🔍 ${usFile.id} - Issue #${existingOnGitHub.number} already exists on GitHub`);
-                        // CHECK: Does issue have placeholder content that needs updating?
-                        const hasPlaceholderBody = existingOnGitHub.body?.includes('This issue was auto-created by SpecWeave')
-                            && !existingOnGitHub.body?.includes('## Acceptance Criteria');
-                        if (hasPlaceholderBody) {
-                            this.logger.log(`  📝 Issue has placeholder content - updating with rich body...`);
-                            try {
-                                // Find the user story file in living docs
-                                const featurePath = path.join(this.projectRoot, '.specweave/docs/internal/specs', this.projectId, featureId);
-                                const usIdNumber = usFile.id.toLowerCase().replace('us-', '');
-                                const featureFiles = existsSync(featurePath) ? await fs.readdir(featurePath) : [];
-                                const usFileName = featureFiles.find(f => f.startsWith(`us-${usIdNumber}-`) && f.endsWith('.md'));
-                                if (usFileName) {
-                                    const usFilePath = path.join(featurePath, usFileName);
-                                    const contentBuilder = new UserStoryContentBuilder(usFilePath, this.projectRoot);
-                                    const richBody = await contentBuilder.buildIssueBody(`${repoInfo.owner}/${repoInfo.repo}`);
-                                    // Update the issue body via gh CLI
-                                    await client.updateIssueBody(existingOnGitHub.number, richBody);
-                                    this.logger.log(`  ✅ Updated issue #${existingOnGitHub.number} with rich content (ACs, tasks)`);
-                                }
-                                else {
-                                    this.logger.log(`  ⚠️ User story file not found - keeping placeholder content`);
-                                }
-                            }
-                            catch (error) {
-                                this.logger.log(`  ⚠️ Failed to update issue body: ${error}`);
-                            }
-                        }
-                        else {
-                            this.logger.log(`  ⏭️ Issue already has rich content - skipping update`);
-                        }
-                        // Backfill Layer 1 (frontmatter) and Layer 2 (metadata)
-                        await this.frontmatterUpdater.updateUserStoryFrontmatter({
-                            projectRoot: this.projectRoot,
-                            featureId,
-                            userStoryId: usFile.id,
-                            githubIssue: {
-                                number: existingOnGitHub.number,
-                                url: existingOnGitHub.html_url,
-                                createdAt: new Date().toISOString(),
-                            },
-                        });
-                        // Backfill Layer 2 (metadata.json)
-                        if (existsSync(metadataFile)) {
-                            const metadata = JSON.parse(await fs.readFile(metadataFile, 'utf-8'));
-                            if (!metadata.github) {
-                                metadata.github = {};
-                            }
-                            if (!metadata.github.issues) {
-                                metadata.github.issues = [];
-                            }
-                            // Check if not already in metadata
-                            const existsInMetadata = metadata.github.issues.find((i) => i.userStory === usFile.id);
-                            if (!existsInMetadata) {
-                                metadata.github.issues.push({
-                                    userStory: usFile.id,
-                                    number: existingOnGitHub.number,
-                                    url: existingOnGitHub.html_url,
-                                    createdAt: new Date().toISOString(),
-                                });
-                                metadata.github.lastSync = new Date().toISOString();
-                                await fs.writeFile(metadataFile, JSON.stringify(metadata, null, 2), 'utf-8');
-                            }
-                        }
-                        continue;
-                    }
-                    // All 3 layers miss - but check for DUPLICATES with wrong format first!
-                    // ========================================================================
-                    // DUPLICATE DETECTION: Prevent FS-0128 vs FS-128 duplicates
-                    // ========================================================================
-                    // Before creating, search for issues with similar titles but different feature ID formats.
-                    // This catches cases where an old bug created issues with leading zeros (FS-0128)
-                    // but the correct format is without (FS-128).
-                    const duplicateCheck = await this.detectDuplicateIssue(client, featureId, usFile.id);
-                    if (duplicateCheck) {
-                        this.logger.log(`  ⚠️  DUPLICATE DETECTED: Issue #${duplicateCheck.number} exists with format "${duplicateCheck.format}"`);
-                        this.logger.log(`     Current format: [${featureId}][${usFile.id}]`);
-                        this.logger.log(`     Existing issue: ${duplicateCheck.title}`);
-                        this.logger.log(`     ⏭️  Skipping creation to avoid duplicate. Use existing issue #${duplicateCheck.number}`);
-                        // Backfill metadata with the existing issue (even if wrong format)
-                        await this.frontmatterUpdater.updateUserStoryFrontmatter({
-                            projectRoot: this.projectRoot,
-                            featureId,
-                            userStoryId: usFile.id,
-                            githubIssue: {
-                                number: duplicateCheck.number,
-                                url: duplicateCheck.url,
-                                createdAt: new Date().toISOString(),
-                            },
-                        });
-                        continue;
-                    }
-                    this.logger.log(`  📝 Creating GitHub issue for ${usFile.id}...`);
-                    // Format issue body - use UserStoryContentBuilder for rich content with ACs
-                    let issueBody;
-                    try {
-                        // Find the user story file in living docs
-                        const featurePath = path.join(this.projectRoot, '.specweave/docs/internal/specs', this.projectId, featureId);
-                        // Search for file matching us-{number}-*.md pattern
-                        const usIdNumber = usFile.id.toLowerCase().replace('us-', '');
-                        const featureFiles = existsSync(featurePath) ? await fs.readdir(featurePath) : [];
-                        const usFileName = featureFiles.find(f => f.startsWith(`us-${usIdNumber}-`) && f.endsWith('.md'));
-                        if (usFileName) {
-                            const usFilePath = path.join(featurePath, usFileName);
-                            const contentBuilder = new UserStoryContentBuilder(usFilePath, this.projectRoot);
-                            issueBody = await contentBuilder.buildIssueBody(`${repoInfo.owner}/${repoInfo.repo}`);
-                            this.logger.log(`  📄 Built rich issue body with ACs from ${usFileName}`);
-                        }
-                        else {
-                            // Fallback to basic body if file not found
-                            this.logger.log(`  ⚠️ User story file not found for ${usFile.id}, using basic body`);
-                            issueBody = this.formatUserStoryBody(usFile);
-                        }
-                    }
-                    catch (error) {
-                        // Fallback to basic body on any error
-                        this.logger.log(`  ⚠️ Failed to build rich body: ${error}, using fallback`);
-                        issueBody = this.formatUserStoryBody(usFile);
-                    }
-                    // Create issue
-                    const issue = await client.createUserStoryIssue({
-                        featureId,
-                        userStoryId: usFile.id,
-                        title: usFile.title,
-                        body: issueBody,
-                        labels: [],
-                        milestone: milestoneNumber,
-                    });
-                    this.logger.log(`  ✅ Created issue #${issue.number}: ${issue.html_url}`);
-                    createdIssues.push(issue);
-                    // Update increment metadata.json (Layer 2)
-                    if (existsSync(metadataFile)) {
-                        const metadata = JSON.parse(await fs.readFile(metadataFile, 'utf-8'));
-                        if (!metadata.github) {
-                            metadata.github = {};
-                        }
-                        if (!metadata.github.issues) {
-                            metadata.github.issues = [];
-                        }
-                        metadata.github.issues.push({
-                            userStory: usFile.id,
-                            number: issue.number,
-                            url: issue.html_url,
-                            createdAt: new Date().toISOString(),
-                        });
-                        metadata.github.lastSync = new Date().toISOString();
-                        await fs.writeFile(metadataFile, JSON.stringify(metadata, null, 2), 'utf-8');
-                    }
-                    // Update user story frontmatter (Layer 1 backfill)
-                    await this.frontmatterUpdater.updateUserStoryFrontmatter({
-                        projectRoot: this.projectRoot,
-                        featureId,
-                        userStoryId: usFile.id,
-                        githubIssue: {
-                            number: issue.number,
-                            url: issue.html_url,
-                            createdAt: new Date().toISOString(),
-                        },
-                    });
-                }
-                catch (error) {
-                    this.logger.error(`  ❌ Failed to create issue for ${usFile.id}:`, error);
-                }
-                finally {
-                    // ALWAYS release lock, even on error or early continue
-                    if (lockAcquired) {
-                        await lockManager.release();
-                    }
-                }
-            }
-            if (createdIssues.length > 0) {
-                this.logger.log(`\n✅ Created ${createdIssues.length} GitHub issue(s) for ${featureId}`);
-                createdIssues.forEach(issue => {
-                    this.logger.log(`   - Issue #${issue.number}: ${issue.html_url}`);
-                });
-            }
-            else {
-                this.logger.log('\n✅ All GitHub issues already exist (0 new issues created)');
-            }
-            return createdIssues;
-        }
-        catch (error) {
-            this.logger.error('❌ Failed to create GitHub issues:', error);
-            throw error;
-        }
+    async createGitHubIssuesForUserStories(_config) {
+        this.logger.log('GitHub issue creation handled by GitHubFeatureSync pipeline');
+        return [];
     }
     /**
-     * Close GitHub issues for all User Stories when increment completes (NEW in v0.28.1)
-     *
-     * This is the CRITICAL missing feature that was causing GitHub issues to remain
-     * open after increments were closed.
-     *
-     * Called by: syncIncrementClosure() when increment status changes to "completed"
-     *
-     * Flow:
-     * 1. Get feature ID from increment spec
-     * 2. Load all user stories for that feature
-     * 3. For each user story with a GitHub issue, close it with completion comment
-     *
-     * @param config - Project configuration
-     * @returns Array of closed issue numbers
+     * @deprecated (0348) GitHub closure now handled by GitHubFeatureSync via LivingDocsSync chain.
      */
+    async closeGitHubIssuesForUserStories(_config) {
+        this.logger.log('GitHub closure handled by GitHubFeatureSync pipeline');
+        return [];
+    }
     /**
      * Close JIRA issues for completed user stories
      *
@@ -712,121 +397,6 @@ export class SyncCoordinator {
             }
         }
     }
-    /**
-     * Detect duplicate issues with different feature ID formats
-     *
-     * Searches for issues that match the user story but have a different feature ID format.
-     * This prevents creating duplicates like:
-     * - [FS-0128][US-001] (old bug format with leading zeros)
-     * - [FS-128][US-001] (correct format)
-     *
-     * The search uses a regex pattern that matches any FS-XXX format for the same US-XXX.
-     *
-     * @param client - GitHub client
-     * @param featureId - Current feature ID (e.g., "FS-128")
-     * @param userStoryId - User story ID (e.g., "US-001")
-     * @returns Duplicate info if found, null otherwise
-     */
-    async detectDuplicateIssue(client, featureId, userStoryId) {
-        try {
-            // Extract the numeric part of the feature ID (e.g., "128" from "FS-128" or "FS-0128")
-            const featureNumMatch = featureId.match(/FS-0*(\d+)E?/i);
-            if (!featureNumMatch) {
-                return null;
-            }
-            const featureNum = featureNumMatch[1];
-            // Search for issues with ANY format of this feature ID + user story
-            // Patterns to check:
-            // - [FS-128][US-001]   (correct, no leading zeros)
-            // - [FS-0128][US-001]  (bug format, with leading zeros)
-            // - [FS-00128][US-001] (edge case, multiple leading zeros)
-            const searchPatterns = [
-                `[FS-${featureNum}][${userStoryId}]`, // FS-128
-                `[FS-0${featureNum}][${userStoryId}]`, // FS-0128
-                `[FS-00${featureNum}][${userStoryId}]`, // FS-00128
-                `[FS-${featureNum}E][${userStoryId}]`, // FS-128E (external)
-                `[FS-0${featureNum}E][${userStoryId}]`, // FS-0128E
-            ];
-            // Filter out the exact current format (we already checked that)
-            const currentFormat = `[${featureId}][${userStoryId}]`;
-            const alternatePatterns = searchPatterns.filter(p => p !== currentFormat);
-            for (const pattern of alternatePatterns) {
-                const existingIssue = await client.searchIssueByTitle(pattern, true); // Include closed
-                if (existingIssue) {
-                    // Extract the format from the issue title
-                    const formatMatch = existingIssue.title.match(/\[(FS-\d+E?)\]/i);
-                    const detectedFormat = formatMatch ? formatMatch[1] : 'unknown';
-                    return {
-                        number: existingIssue.number,
-                        url: existingIssue.html_url,
-                        title: existingIssue.title,
-                        format: detectedFormat,
-                    };
-                }
-            }
-            return null;
-        }
-        catch (error) {
-            // Don't block issue creation on duplicate detection failure
-            this.logger.log(`    ⚠️ Duplicate detection failed (non-blocking): ${error}`);
-            return null;
-        }
-    }
-    /**
-     * Update issue if it has placeholder content
-     * Fetches issue from GitHub, checks for placeholder, and updates with rich content
-     */
-    async updateIssueIfPlaceholder(issueNumber, userStoryId, featureId, client, repoInfo) {
-        try {
-            // Fetch issue from GitHub to check body content
-            const issue = await client.getIssue(issueNumber);
-            if (!issue) {
-                this.logger.log(`    ⚠️ Could not fetch issue #${issueNumber}`);
-                return;
-            }
-            // Check if issue has placeholder content
-            const hasPlaceholderBody = issue.body?.includes('This issue was auto-created by SpecWeave')
-                && !issue.body?.includes('## Acceptance Criteria');
-            if (!hasPlaceholderBody) {
-                this.logger.log(`    ✓ Issue already has rich content`);
-                return;
-            }
-            this.logger.log(`    📝 Updating placeholder content with rich body...`);
-            // Find the user story file in living docs
-            const featurePath = path.join(this.projectRoot, '.specweave/docs/internal/specs', this.projectId, featureId);
-            const usIdNumber = userStoryId.toLowerCase().replace('us-', '');
-            const featureFiles = existsSync(featurePath) ? await fs.readdir(featurePath) : [];
-            const usFileName = featureFiles.find(f => f.startsWith(`us-${usIdNumber}-`) && f.endsWith('.md'));
-            if (!usFileName) {
-                this.logger.log(`    ⚠️ User story file not found - keeping placeholder`);
-                return;
-            }
-            const usFilePath = path.join(featurePath, usFileName);
-            const contentBuilder = new UserStoryContentBuilder(usFilePath, this.projectRoot);
-            const richBody = await contentBuilder.buildIssueBody(`${repoInfo.owner}/${repoInfo.repo}`);
-            // Update the issue body
-            await client.updateIssueBody(issueNumber, richBody);
-            this.logger.log(`    ✅ Updated issue #${issueNumber} with ACs and tasks`);
-        }
-        catch (error) {
-            this.logger.log(`    ⚠️ Failed to update: ${error}`);
-        }
-    }
-    /**
-     * Format user story content for GitHub issue body
-     */
-    formatUserStoryBody(usFile) {
-        // Simple body for now - can be enhanced later
-        const parts = [];
-        parts.push(`## User Story: ${usFile.id}`);
-        parts.push('');
-        if (usFile.external_title) {
-            parts.push(`**Original Title**: ${usFile.external_title}`);
-            parts.push('');
-        }
-        parts.push('For detailed acceptance criteria, tasks, and technical specifications, see the living docs in the repository.');
-        return parts.join('\n');
-    }
     /**
      * Sync increment completion to external tools using format preservation
      */
@@ -1350,20 +920,5 @@ export class SyncCoordinator {
         lines.push(`🤖 Auto-synced by SpecWeave`);
         return lines.join('\n');
     }
-    /**
-     * Sync AC checkbox status to GitHub issues
-     *
-     * @deprecated (0348) Delegates to GitHubACCheckboxSync in the GitHub plugin.
-     * Kept for backward compatibility with callers that haven't migrated yet.
-     */
-    async syncACCheckboxesToGitHub(config, options = {}) {
-        const { GitHubACCheckboxSync } = await import('../../plugins/specweave-github/lib/github-ac-checkbox-sync.js');
-        const sync = new GitHubACCheckboxSync({
-            projectRoot: this.projectRoot,
-            incrementId: this.incrementId,
-            logger: this.logger,
-        });
-        return sync.syncACCheckboxesToGitHub(config, options);
-    }
 }
 //# sourceMappingURL=sync-coordinator.js.map