@g-abhishek/gitx 0.1.2 → 0.1.5

package/dist/workflows/pr.js

@@ -0,0 +1,291 @@
+/**
+ * PR Workflow helpers
+ *
+ * Shared logic used by the `pr review` and `pr fix-comments` CLI commands.
+ */
+import ora from "ora";
+import { createProvider } from "../providers/factory.js";
+import { readRepoFile, listTrackedFiles } from "../utils/gitOps.js";
+import { logger } from "../logger/logger.js";
+// Files that add no review value — skip them even if they appear in the diff
+const SKIP_FILE_PATTERNS = [
+    /package-lock\.json$/,
+    /yarn\.lock$/,
+    /pnpm-lock\.yaml$/,
+    /bun\.lockb$/,
+    /\.lock$/,
+    /dist\//,
+    /build\//,
+    /\.min\.(js|css)$/,
+    /\.map$/,
+    /node_modules\//,
+    /\.d\.ts$/, // generated TypeScript declarations
+    /generated\//,
+    /migrations\/.*\.sql$/,
+];
+function isReviewableFile(path) {
+    return !SKIP_FILE_PATTERNS.some((re) => re.test(path));
+}
+/**
+ * Parse a unified diff and return the set of changed file paths.
+ */
+function parseChangedPathsFromDiff(diff) {
+    const paths = new Set();
+    for (const line of diff.split("\n")) {
+        // Match "+++ b/src/foo.ts" lines
+        const m = line.match(/^\+\+\+ b\/(.+)/);
+        if (m?.[1] && m[1] !== "/dev/null") {
+            paths.add(m[1].trim());
+        }
+    }
+    return [...paths];
+}
+/**
+ * Given a list of changed file paths, find supporting context files that are
+ * closely related (imported by or importing changed files).
+ * Returns at most `maxFiles` file paths.
+ */
+async function findContextFiles(changedPaths, allTrackedFiles, cwd, maxFiles = 8) {
+    const contextPaths = new Set();
+    for (const changedPath of changedPaths.slice(0, 10)) {
+        const content = await readRepoFile(changedPath, cwd);
+        if (!content)
+            continue;
+        // Extract relative import paths from TypeScript/JS files
+        const importMatches = content.matchAll(/from\s+['"]([^'"]+)['"]/g);
+        for (const [, importPath] of importMatches) {
+            if (!importPath || importPath.startsWith("node:") || !importPath.startsWith("."))
+                continue;
+            // Resolve the import path relative to the file's directory
+            const dir = changedPath.split("/").slice(0, -1).join("/");
+            const candidates = [
+                `${dir}/${importPath}.ts`,
+                `${dir}/${importPath}.tsx`,
+                `${dir}/${importPath}/index.ts`,
+                `${dir}/${importPath}`,
+            ].map((p) => p.replace(/\/\//g, "/").replace(/^\//, ""));
+            for (const candidate of candidates) {
+                if (allTrackedFiles.includes(candidate) && !changedPaths.includes(candidate)) {
+                    contextPaths.add(candidate);
+                    break;
+                }
+            }
+        }
+        if (contextPaths.size >= maxFiles)
+            break;
+    }
+    return [...contextPaths].slice(0, maxFiles);
+}
+/**
+ * Fetch a PR, its full diff, and all related codebase context;
+ * run a senior-developer quality AI review;
+ * submit as a formal review (with inline comments) to the hosting provider.
+ */
+export async function runReviewWorkflow(gitx, prNumber, postComment = true) {
+    const ctx = await gitx.getRepoContext();
+    const provider = createProvider(ctx);
+    const cwd = gitx.cwd;
+    const fetchSpinner = ora("Fetching PR info, diff and comments…").start();
+    const pr = await provider.getPR(ctx.repoSlug, prNumber);
+    const [comments, diff] = await Promise.all([
+        provider.getPRComments(ctx.repoSlug, prNumber),
+        provider.getPRDiff(ctx.repoSlug, prNumber),
+    ]);
+    fetchSpinner.succeed(`PR #${prNumber}: "${pr.title}"  (${pr.head} → ${pr.base})`);
+    // ── Build codebase context ─────────────────────────────────────────────────
+    const ctxSpinner = ora("Building codebase context from changed files…").start();
+    const allTracked = await listTrackedFiles(cwd);
+    const allChangedPaths = parseChangedPathsFromDiff(diff);
+    // Filter out lockfiles / generated files — they waste tokens and add no review value.
+    // No file count cap — per-file budgets in buildSeniorReviewPrompt already control
+    // total size. Dropping files arbitrarily is worse than sending all of them.
+    const changedPaths = allChangedPaths.filter(isReviewableFile);
+    const skippedCount = allChangedPaths.length - changedPaths.length;
+    // Read full content of changed files
+    const changedFiles = {};
+    for (const p of changedPaths) {
+        const content = await readRepoFile(p, cwd);
+        if (content)
+            changedFiles[p] = content;
+    }
+    // Read supporting context files (imported by the changed files)
+    // 10 context files gives the AI a solid picture of the codebase structure
+    const ctxPaths = await findContextFiles(changedPaths, allTracked, cwd, 10);
+    const contextFiles = {};
+    for (const p of ctxPaths) {
+        const content = await readRepoFile(p, cwd);
+        if (content)
+            contextFiles[p] = content;
+    }
+    const skippedNote = skippedCount > 0 ? `, ${skippedCount} lock/generated files skipped` : "";
+    ctxSpinner.succeed(`Context: ${changedPaths.length} changed files, ${ctxPaths.length} context files${skippedNote}`);
+    const reviewSpinner = ora(`Running senior-dev AI review… (this may take up to 5 min for large PRs)`).start();
+    let review;
+    try {
+        review = await gitx.ai.reviewPRDetailed({
+            prTitle: pr.title,
+            prBody: pr.body,
+            author: pr.author,
+            headBranch: pr.head,
+            baseBranch: pr.base,
+            diff,
+            changedFiles,
+            contextFiles,
+            repoFileList: allTracked,
+            existingComments: comments.map((c) => ({
+                author: c.author,
+                body: c.body,
+                path: c.path,
+                line: c.line,
+            })),
+        });
+        reviewSpinner.succeed(`Review complete — verdict: ${review.verdict}  |  ${review.inlineComments.length} inline comment(s)`);
+    }
+    catch (err) {
+        reviewSpinner.fail("AI review failed.");
+        throw err;
+    }
+    // ── Build the formatted review body (for the summary comment) ─────────────
+    const verdictIcon = review.verdict === "approve" ? "✅" :
+        review.verdict === "request_changes" ? "🔴" : "💬";
+    const checklistLines = review.checklist.map((c) => {
+        const icon = c.status === "pass" ? "✅" : c.status === "warn" ? "⚠️" : "❌";
+        return `| ${icon} | **${c.area}** | ${c.note} |`;
+    });
+    const issueLines = review.issues.map((i) => {
+        const sev = i.severity === "critical" ? "🔴" : i.severity === "warning" ? "🟡" : "💡";
+        const loc = i.file ? ` (\`${i.file}${i.line ? `:${i.line}` : ""}\`)` : "";
+        return `${sev} ${i.description}${loc}`;
+    });
+    const posLines = review.positives.map((p) => `✔ ${p}`);
+    const summaryBody = [
+        `## ${verdictIcon} Senior Dev AI Review (gitx) — ${review.verdict.replace("_", " ")}`,
+        "",
+        review.summary,
+        ...(checklistLines.length > 0
+            ? ["", "### Review Checklist", "| Status | Area | Note |", "|--------|------|------|", ...checklistLines]
+            : []),
+        ...(review.issues.length > 0 ? ["", "### Issues Found", ...issueLines] : []),
+        ...(review.positives.length > 0 ? ["", "### Positives", ...posLines] : []),
+        ...(review.testingNotes ? ["", "### How to Test", review.testingNotes] : []),
+        ...(review.inlineComments.length > 0
+            ? [`\n> 💬 ${review.inlineComments.length} inline comment(s) posted on specific lines.`]
+            : []),
+        "",
+        "*Generated by [gitx](https://github.com/g-abhishek/gitx)*",
+    ].join("\n");
+    // ── Post the formal review (inline comments + verdict) ────────────────────
+    let reviewPosted = false;
+    let inlineDelivery = "none";
+    if (postComment) {
+        const inlineCount = review.inlineComments.length;
+        const postSpinner = ora(`Submitting review to PR${inlineCount > 0 ? ` with ${inlineCount} inline comment(s)` : ""}…`).start();
+        try {
+            await provider.submitPRReview(ctx.repoSlug, prNumber, {
+                body: summaryBody,
+                event: review.verdict,
+                comments: review.inlineComments.map((c) => ({
+                    path: c.path,
+                    line: c.line,
+                    body: c.suggestion
+                        ? `${c.body}\n\n**Suggestion:**\n\`\`\`suggestion\n${c.suggestion}\n\`\`\``
+                        : c.body,
+                })),
+            });
+            reviewPosted = true;
+            // Determine how inline comments were delivered.
+            // GitHub's submitPRReview will attempt inline first, then fall back to
+            // plain comments. We can't detect which path was taken from here, so
+            // we report "inline" optimistically — the user sees the real result on GitHub.
+            inlineDelivery = inlineCount > 0 ? "inline" : "none";
+            postSpinner.succeed(`Review submitted to PR.${inlineCount > 0 ? ` (${inlineCount} inline comment(s) — see PR for delivery method)` : ""}`);
+        }
+        catch (err) {
+            postSpinner.fail(`Could not post review to PR: ${String(err.message ?? err)}`);
+            // Don't rethrow — still return the review so the user can see it locally
+        }
+    }
+    return { pr, comments, aiSummary: summaryBody, review, reviewPosted, inlineDelivery };
+}
+/**
+ * Fetch PR review comments, ask AI to suggest fixes, apply them, and
+ * commit + push the changes.
+ */
+export async function runFixCommentsWorkflow(gitx, prNumber, dryRun = false, noCommit = false) {
+    const { applyUnifiedDiff, stageAll, hasStagedChanges, commitChanges } = await import("../utils/gitOps.js");
+    const cwd = gitx.cwd;
+    const ctx = await gitx.getRepoContext();
+    const provider = createProvider(ctx);
+    logger.info(`🔍 Fetching PR #${prNumber}…`);
+    const pr = await provider.getPR(ctx.repoSlug, prNumber);
+    logger.info("💬 Fetching review comments…");
+    const comments = await provider.getPRComments(ctx.repoSlug, prNumber);
+    if (comments.length === 0) {
+        logger.info("No review comments found.");
+        return { pr, comments, appliedFixes: [], skippedFixes: [] };
+    }
+    // Gather file contents for files mentioned in comments
+    const mentionedPaths = [...new Set(comments.map((c) => c.path).filter(Boolean))];
+    const trackedFiles = await listTrackedFiles(cwd);
+    // Also include files mentioned in the PR body / title
+    const allSourceFiles = trackedFiles
+        .filter((f) => /\.(ts|js|tsx|jsx|py|go|rs|java|rb|cs|cpp|c|h)$/.test(f))
+        .slice(0, 15);
+    const relevantFiles = [...new Set([...mentionedPaths, ...allSourceFiles])].slice(0, 15);
+    const fileContents = {};
+    for (const f of relevantFiles) {
+        const content = await readRepoFile(f, cwd);
+        if (content)
+            fileContents[f] = content.slice(0, 4000);
+    }
+    logger.info("🧠 Generating AI-suggested fixes…");
+    const fixResult = await gitx.ai.suggestFixes({
+        comments: comments.map((c) => ({
+            body: c.body,
+            author: c.author,
+            path: c.path,
+            line: c.line,
+        })),
+        prTitle: pr.title,
+        prBody: pr.body,
+        fileContents,
+    });
+    const appliedFixes = [];
+    const skippedFixes = [];
+    for (const edit of fixResult.suggestedEdits) {
+        if (dryRun) {
+            logger.info(`  ↳ [dry-run] Would apply fix to: ${edit.path}`);
+            logger.info(`     Rationale: ${edit.rationale}`);
+            appliedFixes.push({ path: edit.path, rationale: edit.rationale });
+            continue;
+        }
+        const applyResult = await applyUnifiedDiff(edit.unifiedDiff, cwd);
+        if (applyResult.ok) {
+            logger.info(`  ↳ Applied fix to: ${edit.path}`);
+            appliedFixes.push({ path: edit.path, rationale: edit.rationale });
+        }
+        else {
+            logger.warn(`  ↳ Could not apply fix to ${edit.path}: ${applyResult.error ?? "unknown"}`);
+            skippedFixes.push({
+                path: edit.path,
+                reason: applyResult.error ?? "git apply failed",
+            });
+        }
+    }
+    if (!dryRun && !noCommit && appliedFixes.length > 0) {
+        await stageAll(cwd);
+        if (await hasStagedChanges(cwd)) {
+            const msg = `fix: address PR #${prNumber} review comments\n\n${appliedFixes.map((f) => `- ${f.path}: ${f.rationale}`).join("\n")}`;
+            const sha = await commitChanges(msg, cwd);
+            logger.success(`Committed fixes: ${sha.slice(0, 8)}`);
+            // Post summary comment
+            const fixSummary = appliedFixes.map((f) => `- \`${f.path}\`: ${f.rationale}`).join("\n");
+            await provider.addPRComment(ctx.repoSlug, prNumber, `## 🤖 Auto-fixes applied (gitx)\n\n${fixSummary}\n\nCommit: \`${sha.slice(0, 8)}\``);
+        }
+    }
+    if (!dryRun && noCommit && appliedFixes.length > 0) {
+        logger.info("\n💡 Fixes applied to working tree. Review the changes and commit when ready.");
+    }
+    return { pr, comments, appliedFixes, skippedFixes };
+}
+//# sourceMappingURL=pr.js.map

package/dist/workflows/pr.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr.js","sourceRoot":"","sources":["../../src/workflows/pr.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,GAAG,MAAM,KAAK,CAAC;AAGtB,OAAO,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAC;AACzD,OAAO,EAAE,YAAY,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAC;AACpE,OAAO,EAAE,MAAM,EAAE,MAAM,qBAAqB,CAAC;AAE7C,6EAA6E;AAC7E,MAAM,kBAAkB,GAAG;IACzB,qBAAqB;IACrB,aAAa;IACb,kBAAkB;IAClB,aAAa;IACb,SAAS;IACT,QAAQ;IACR,SAAS;IACT,kBAAkB;IAClB,QAAQ;IACR,gBAAgB;IAChB,UAAU,EAAM,oCAAoC;IACpD,aAAa;IACb,sBAAsB;CACvB,CAAC;AAEF,SAAS,gBAAgB,CAAC,IAAY;IACpC,OAAO,CAAC,kBAAkB,CAAC,IAAI,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;AACzD,CAAC;AAeD;;GAEG;AACH,SAAS,yBAAyB,CAAC,IAAY;IAC7C,MAAM,KAAK,GAAG,IAAI,GAAG,EAAU,CAAC;IAChC,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;QACpC,iCAAiC;QACjC,MAAM,CAAC,GAAG,IAAI,CAAC,KAAK,CAAC,iBAAiB,CAAC,CAAC;QACxC,IAAI,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,KAAK,WAAW,EAAE,CAAC;YACnC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;QACzB,CAAC;IACH,CAAC;IACD,OAAO,CAAC,GAAG,KAAK,CAAC,CAAC;AACpB,CAAC;AAED;;;;GAIG;AACH,KAAK,UAAU,gBAAgB,CAC7B,YAAsB,EACtB,eAAyB,EACzB,GAAW,EACX,QAAQ,GAAG,CAAC;IAEZ,MAAM,YAAY,GAAG,IAAI,GAAG,EAAU,CAAC;IAEvC,KAAK,MAAM,WAAW,IAAI,YAAY,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC;QACpD,MAAM,OAAO,GAAG,MAAM,YAAY,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;QACrD,IAAI,CAAC,OAAO;YAAE,SAAS;QAEvB,yDAAyD;QACzD,MAAM,aAAa,GAAG,OAAO,CAAC,QAAQ,CAAC,0BAA0B,CAAC,CAAC;QACnE,KAAK,MAAM,CAAC,EAAE,UAAU,CAAC,IAAI,aAAa,EAAE,CAAC;YAC3C,IAAI,CAAC,UAAU,IAAI,UAAU,CAAC,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,GAAG,CAAC;gBAAE,SAAS;YAC3F,2DAA2D;YAC3D,MAAM,GAAG,GAAG,WAAW,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YAC1D,MAAM,UAAU,GAAG;gBACjB,GAAG,GAAG,IAAI,UAAU,KAAK;gBACzB,GAAG,GAAG,IAAI,UAAU,MAAM;gBAC1B,GAAG,GAAG,IAAI,UAAU,WAAW;gBAC/B,GAAG,GAAG,IAAI,UAAU,EAAE;aACvB,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,CAAC;YAEzD,KAAK,MAAM,SAAS,IAAI,UAAU,EAAE,CAAC;gBACnC,IAAI,eAAe,CAAC,QAAQ,CAAC,SAAS,CAAC,IAAI,CAAC,YAAY,CAAC,QAAQ,CAAC,SAAS,CAAC,EAAE,CAAC;oBAC7E,YAAY,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC;oBAC5B,MAAM;gBACR,CAAC;YACH,CAAC;QACH,CAAC;QAED,IAAI,YAAY,CAAC,IAAI,IAAI,QAAQ;YAAE,MAAM;IAC3C,CAAC;IAED,OAAO,CAAC,GAAG,YAAY,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,QAAQ,CAAC,CAAC;AAC9C,CAAC;AAED;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,iBAAiB,CACrC,IAAU,EACV,QAAgB,EAChB,WAAW,GAAG,IAAI;IAElB,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,cAAc,EAAE,CAAC;IACxC,MAAM,QAAQ,GAAG,cAAc,CAAC,GAAG,CAAC,CAAC;IACrC,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,CAAC;IAErB,MAAM,YAAY,GAAG,GAAG,CAAC,sCAAsC,CAAC,CAAC,KAAK,EAAE,CAAC;IACzE,MAAM,EAAE,GAAG,MAAM,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;IACxD,MAAM,CAAC,QAAQ,EAAE,IAAI,CAAC,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC;QACzC,QAAQ,CAAC,aAAa,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC;QAC9C,QAAQ,CAAC,SAAS,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC;KAC3C,CAAC,CAAC;IACH,YAAY,CAAC,OAAO,CAAC,OAAO,QAAQ,MAAM,EAAE,CAAC,KAAK,OAAO,EAAE,CAAC,IAAI,MAAM,EAAE,CAAC,IAAI,GAAG,CAAC,CAAC;IAElF,8EAA8E;IAC9E,MAAM,UAAU,GAAG,GAAG,CAAC,+CAA+C,CAAC,CAAC,KAAK,EAAE,CAAC;IAChF,MAAM,UAAU,GAAG,MAAM,gBAAgB,CAAC,GAAG,CAAC,CAAC;IAC/C,MAAM,eAAe,GAAG,yBAAyB,CAAC,IAAI,CAAC,CAAC;IAExD,sFAAsF;IACtF,kFAAkF;IAClF,4EAA4E;IAC5E,MAAM,YAAY,GAAG,eAAe,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC;IAC9D,MAAM,YAAY,GAAG,eAAe,CAAC,MAAM,GAAG,YAAY,CAAC,MAAM,CAAC;IAElE,qCAAqC;IACrC,MAAM,YAAY,GAA2B,EAAE,CAAC;IAChD,KAAK,MAAM,CAAC,IAAI,YAAY,EAAE,CAAC;QAC7B,MAAM,OAAO,GAAG,MAAM,YAAY,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;QAC3C,IAAI,OAAO;YAAE,YAAY,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC;IACzC,CAAC;IAED,gEAAgE;IAChE,0EAA0E;IAC1E,MAAM,QAAQ,GAAG,MAAM,gBAAgB,CAAC,YAAY,EAAE,UAAU,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC;IAC3E,MAAM,YAAY,GAA2B,EAAE,CAAC;IAChD,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;QACzB,MAAM,OAAO,GAAG,MAAM,YAAY,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;QAC3C,IAAI,OAAO;YAAE,YAAY,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC;IACzC,CAAC;IAED,MAAM,WAAW,GAAG,YAAY,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,YAAY,+BAA+B,CAAC,CAAC,CAAC,EAAE,CAAC;IAC7F,UAAU,CAAC,OAAO,CAChB,YAAY,YAAY,CAAC,MAAM,mBAAmB,QAAQ,CAAC,MAAM,iBAAiB,WAAW,EAAE,CAChG,CAAC;IAEF,MAAM,aAAa,GAAG,GAAG,CACvB,yEAAyE,CAC1E,CAAC,KAAK,EAAE,CAAC;IAEV,IAAI,MAA4D,CAAC;IACjE,IAAI,CAAC;QACH,MAAM,GAAG,MAAM,IAAI,CAAC,EAAE,CAAC,gBAAgB,CAAC;YACtC,OAAO,EAAE,EAAE,CAAC,KAAK;YACjB,MAAM,EAAE,EAAE,CAAC,IAAI;YACf,MAAM,EAAE,EAAE,CAAC,MAAM;YACjB,UAAU,EAAE,EAAE,CAAC,IAAI;YACnB,UAAU,EAAE,EAAE,CAAC,IAAI;YACnB,IAAI;YACJ,YAAY;YACZ,YAAY;YACZ,YAAY,EAAE,UAAU;YACxB,gBAAgB,EAAE,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;gBACrC,MAAM,EAAE,CAAC,CAAC,MAAM;gBAChB,IAAI,EAAE,CAAC,CAAC,IAAI;gBACZ,IAAI,EAAE,CAAC,CAAC,IAAI;gBACZ,IAAI,EAAE,CAAC,CAAC,IAAI;aACb,CAAC,CAAC;SACJ,CAAC,CAAC;QACH,aAAa,CAAC,OAAO,CACnB,8BAA8B,MAAM,CAAC,OAAO,QAAQ,MAAM,CAAC,cAAc,CAAC,MAAM,oBAAoB,CACrG,CAAC;IACJ,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,aAAa,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC;QACxC,MAAM,GAAG,CAAC;IACZ,CAAC;IAED,6EAA6E;IAC7E,MAAM,WAAW,GACf,MAAM,CAAC,OAAO,KAAK,SAAS,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC;QACpC,MAAM,CAAC,OAAO,KAAK,iBAAiB,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC;IAErD,MAAM,cAAc,GAAG,MAAM,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE;QAChD,MAAM,IAAI,GAAG,CAAC,CAAC,MAAM,KAAK,MAAM,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,KAAK,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC;QAC1E,OAAO,KAAK,IAAI,QAAQ,CAAC,CAAC,IAAI,QAAQ,CAAC,CAAC,IAAI,IAAI,CAAC;IACnD,CAAC,CAAC,CAAC;IAEH,MAAM,UAAU,GAAG,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE;QACzC,MAAM,GAAG,GAAG,CAAC,CAAC,QAAQ,KAAK,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,QAAQ,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC;QACtF,MAAM,GAAG,GAAG,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC;QAC1E,OAAO,GAAG,GAAG,IAAI,CAAC,CAAC,WAAW,GAAG,GAAG,EAAE,CAAC;IACzC,CAAC,CAAC,CAAC;IAEH,MAAM,QAAQ,GAAG,MAAM,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;IAEvD,MAAM,WAAW,GAAG;QAClB,MAAM,WAAW,kCAAkC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,GAAG,EAAE,GAAG,CAAC,EAAE;QACrF,EAAE;QACF,MAAM,CAAC,OAAO;QACd,GAAG,CAAC,cAAc,CAAC,MAAM,GAAG,CAAC;YAC3B,CAAC,CAAC,CAAC,EAAE,EAAE,sBAAsB,EAAE,0BAA0B,EAAE,0BAA0B,EAAE,GAAG,cAAc,CAAC;YACzG,CAAC,CAAC,EAAE,CAAC;QACP,GAAG,CAAC,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,EAAE,kBAAkB,EAAE,GAAG,UAAU,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;QAC5E,GAAG,CAAC,MAAM,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,EAAE,eAAe,EAAE,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;QAC1E,GAAG,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,EAAE,EAAE,iBAAiB,EAAE,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;QAC5E,GAAG,CAAC,MAAM,CAAC,cAAc,CAAC,MAAM,GAAG,CAAC;YAClC,CAAC,CAAC,CAAC,UAAU,MAAM,CAAC,cAAc,CAAC,MAAM,8CAA8C,CAAC;YACxF,CAAC,CAAC,EAAE,CAAC;QACP,EAAE;QACF,2DAA2D;KAC5D,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAEb,6EAA6E;IAC7E,IAAI,YAAY,GAAG,KAAK,CAAC;IACzB,IAAI,cAAc,GAAmC,MAAM,CAAC;IAE5D,IAAI,WAAW,EAAE,CAAC;QAChB,MAAM,WAAW,GAAG,MAAM,CAAC,cAAc,CAAC,MAAM,CAAC;QACjD,MAAM,WAAW,GAAG,GAAG,CACrB,0BAA0B,WAAW,GAAG,CAAC,CAAC,CAAC,CAAC,SAAS,WAAW,oBAAoB,CAAC,CAAC,CAAC,EAAE,GAAG,CAC7F,CAAC,KAAK,EAAE,CAAC;QACV,IAAI,CAAC;YACH,MAAM,QAAQ,CAAC,cAAc,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,EAAE;gBACpD,IAAI,EAAE,WAAW;gBACjB,KAAK,EAAE,MAAM,CAAC,OAAO;gBACrB,QAAQ,EAAE,MAAM,CAAC,cAAc,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;oBAC1C,IAAI,EAAE,CAAC,CAAC,IAAI;oBACZ,IAAI,EAAE,CAAC,CAAC,IAAI;oBACZ,IAAI,EAAE,CAAC,CAAC,UAAU;wBAChB,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,0CAA0C,CAAC,CAAC,UAAU,UAAU;wBAC3E,CAAC,CAAC,CAAC,CAAC,IAAI;iBACX,CAAC,CAAC;aACJ,CAAC,CAAC;YACH,YAAY,GAAG,IAAI,CAAC;YACpB,gDAAgD;YAChD,uEAAuE;YACvE,qEAAqE;YACrE,+EAA+E;YAC/E,cAAc,GAAG,WAAW,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC;YACrD,WAAW,CAAC,OAAO,CACjB,0BAA0B,WAAW,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,WAAW,kDAAkD,CAAC,CAAC,CAAC,EAAE,EAAE,CACtH,CAAC;QACJ,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,WAAW,CAAC,IAAI,CAAC,gCAAgC,MAAM,CAAE,GAAa,CAAC,OAAO,IAAI,GAAG,CAAC,EAAE,CAAC,CAAC;YAC1F,yEAAyE;QAC3E,CAAC;IACH,CAAC;IAED,OAAO,EAAE,EAAE,EAAE,QAAQ,EAAE,SAAS,EAAE,WAAW,EAAE,MAAM,EAAE,YAAY,EAAE,cAAc,EAAE,CAAC;AACxF,CAAC;AAWD;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,sBAAsB,CAC1C,IAAU,EACV,QAAgB,EAChB,MAAM,GAAG,KAAK,EACd,QAAQ,GAAG,KAAK;IAEhB,MAAM,EAAE,gBAAgB,EAAE,QAAQ,EAAE,gBAAgB,EAAE,aAAa,EAAE,GAAG,MAAM,MAAM,CAClF,oBAAoB,CACrB,CAAC;IACF,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,CAAC;IACrB,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,cAAc,EAAE,CAAC;IACxC,MAAM,QAAQ,GAAG,cAAc,CAAC,GAAG,CAAC,CAAC;IAErC,MAAM,CAAC,IAAI,CAAC,mBAAmB,QAAQ,GAAG,CAAC,CAAC;IAC5C,MAAM,EAAE,GAAG,MAAM,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;IAExD,MAAM,CAAC,IAAI,CAAC,8BAA8B,CAAC,CAAC;IAC5C,MAAM,QAAQ,GAAG,MAAM,QAAQ,CAAC,aAAa,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;IAEtE,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC1B,MAAM,CAAC,IAAI,CAAC,2BAA2B,CAAC,CAAC;QACzC,OAAO,EAAE,EAAE,EAAE,QAAQ,EAAE,YAAY,EAAE,EAAE,EAAE,YAAY,EAAE,EAAE,EAAE,CAAC;IAC9D,CAAC;IAED,uDAAuD;IACvD,MAAM,cAAc,GAAG,CAAC,GAAG,IAAI,GAAG,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,OAAO,CAAa,CAAC,CAAC,CAAC;IAC7F,MAAM,YAAY,GAAG,MAAM,gBAAgB,CAAC,GAAG,CAAC,CAAC;IAEjD,sDAAsD;IACtD,MAAM,cAAc,GAAG,YAAY;SAChC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,gDAAgD,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;SACvE,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;IAEhB,MAAM,aAAa,GAAG,CAAC,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,cAAc,EAAE,GAAG,cAAc,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;IACxF,MAAM,YAAY,GAA2B,EAAE,CAAC;IAChD,KAAK,MAAM,CAAC,IAAI,aAAa,EAAE,CAAC;QAC9B,MAAM,OAAO,GAAG,MAAM,YAAY,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;QAC3C,IAAI,OAAO;YAAE,YAAY,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC;IACxD,CAAC;IAED,MAAM,CAAC,IAAI,CAAC,mCAAmC,CAAC,CAAC;IACjD,MAAM,SAAS,GAAG,MAAM,IAAI,CAAC,EAAE,CAAC,YAAY,CAAC;QAC3C,QAAQ,EAAE,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YAC7B,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,MAAM,EAAE,CAAC,CAAC,MAAM;YAChB,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,IAAI,EAAE,CAAC,CAAC,IAAI;SACb,CAAC,CAAC;QACH,OAAO,EAAE,EAAE,CAAC,KAAK;QACjB,MAAM,EAAE,EAAE,CAAC,IAAI;QACf,YAAY;KACb,CAAC,CAAC;IAEH,MAAM,YAAY,GAA+C,EAAE,CAAC;IACpE,MAAM,YAAY,GAA4C,EAAE,CAAC;IAEjE,KAAK,MAAM,IAAI,IAAI,SAAS,CAAC,cAAc,EAAE,CAAC;QAC5C,IAAI,MAAM,EAAE,CAAC;YACX,MAAM,CAAC,IAAI,CAAC,qCAAqC,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC;YAC9D,MAAM,CAAC,IAAI,CAAC,mBAAmB,IAAI,CAAC,SAAS,EAAE,CAAC,CAAC;YACjD,YAAY,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE,SAAS,EAAE,IAAI,CAAC,SAAS,EAAE,CAAC,CAAC;YAClE,SAAS;QACX,CAAC;QAED,MAAM,WAAW,GAAG,MAAM,gBAAgB,CAAC,IAAI,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;QAClE,IAAI,WAAW,CAAC,EAAE,EAAE,CAAC;YACnB,MAAM,CAAC,IAAI,CAAC,uBAAuB,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC;YAChD,YAAY,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE,SAAS,EAAE,IAAI,CAAC,SAAS,EAAE,CAAC,CAAC;QACpE,CAAC;aAAM,CAAC;YACN,MAAM,CAAC,IAAI,CAAC,8BAA8B,IAAI,CAAC,IAAI,KAAK,WAAW,CAAC,KAAK,IAAI,SAAS,EAAE,CAAC,CAAC;YAC1F,YAAY,CAAC,IAAI,CAAC;gBAChB,IAAI,EAAE,IAAI,CAAC,IAAI;gBACf,MAAM,EAAE,WAAW,CAAC,KAAK,IAAI,kBAAkB;aAChD,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,IAAI,CAAC,MAAM,IAAI,CAAC,QAAQ,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACpD,MAAM,QAAQ,CAAC,GAAG,CAAC,CAAC;QACpB,IAAI,MAAM,gBAAgB,CAAC,GAAG,CAAC,EAAE,CAAC;YAChC,MAAM,GAAG,GAAG,oBAAoB,QAAQ,uBAAuB,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,CAAC,IAAI,KAAK,CAAC,CAAC,SAAS,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YACnI,MAAM,GAAG,GAAG,MAAM,aAAa,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;YAC1C,MAAM,CAAC,OAAO,CAAC,oBAAoB,GAAG,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,CAAC,CAAC;YAEtD,uBAAuB;YACvB,MAAM,UAAU,GAAG,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,CAAC,IAAI,OAAO,CAAC,CAAC,SAAS,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACzF,MAAM,QAAQ,CAAC,YAAY,CACzB,GAAG,CAAC,QAAQ,EACZ,QAAQ,EACR,sCAAsC,UAAU,iBAAiB,GAAG,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CACrF,CAAC;QACJ,CAAC;IACH,CAAC;IAED,IAAI,CAAC,MAAM,IAAI,QAAQ,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACnD,MAAM,CAAC,IAAI,CAAC,+EAA+E,CAAC,CAAC;IAC/F,CAAC;IAED,OAAO,EAAE,EAAE,EAAE,QAAQ,EAAE,YAAY,EAAE,YAAY,EAAE,CAAC;AACtD,CAAC","sourcesContent":["/**\n * PR Workflow helpers\n *\n * Shared logic used by the `pr review` and `pr fix-comments` CLI commands.\n */\n\nimport ora from \"ora\";\nimport type { Gitx } from \"../core/gitx.js\";\nimport type { PullRequest, PullRequestComment } from \"../providers/base.js\";\nimport { createProvider } from \"../providers/factory.js\";\nimport { readRepoFile, listTrackedFiles } from \"../utils/gitOps.js\";\nimport { logger } from \"../logger/logger.js\";\n\n// Files that add no review value — skip them even if they appear in the diff\nconst SKIP_FILE_PATTERNS = [\n  /package-lock\\.json$/,\n  /yarn\\.lock$/,\n  /pnpm-lock\\.yaml$/,\n  /bun\\.lockb$/,\n  /\\.lock$/,\n  /dist\\//,\n  /build\\//,\n  /\\.min\\.(js|css)$/,\n  /\\.map$/,\n  /node_modules\\//,\n  /\\.d\\.ts$/,     // generated TypeScript declarations\n  /generated\\//,\n  /migrations\\/.*\\.sql$/,\n];\n\nfunction isReviewableFile(path: string): boolean {\n  return !SKIP_FILE_PATTERNS.some((re) => re.test(path));\n}\n\n// ─── Review workflow ──────────────────────────────────────────────────────────\n\nexport interface ReviewResult {\n  pr: PullRequest;\n  comments: PullRequestComment[];\n  aiSummary: string;\n  review?: import(\"../ai/types.js\").AiDetailedReviewResponse;\n  /** true = review was actually posted to the hosting provider */\n  reviewPosted: boolean;\n  /** How inline comments were delivered: formal inline, plain comments, or not posted */\n  inlineDelivery: \"inline\" | \"plain-comments\" | \"none\";\n}\n\n/**\n * Parse a unified diff and return the set of changed file paths.\n */\nfunction parseChangedPathsFromDiff(diff: string): string[] {\n  const paths = new Set<string>();\n  for (const line of diff.split(\"\\n\")) {\n    // Match \"+++ b/src/foo.ts\" lines\n    const m = line.match(/^\\+\\+\\+ b\\/(.+)/);\n    if (m?.[1] && m[1] !== \"/dev/null\") {\n      paths.add(m[1].trim());\n    }\n  }\n  return [...paths];\n}\n\n/**\n * Given a list of changed file paths, find supporting context files that are\n * closely related (imported by or importing changed files).\n * Returns at most `maxFiles` file paths.\n */\nasync function findContextFiles(\n  changedPaths: string[],\n  allTrackedFiles: string[],\n  cwd: string,\n  maxFiles = 8\n): Promise<string[]> {\n  const contextPaths = new Set<string>();\n\n  for (const changedPath of changedPaths.slice(0, 10)) {\n    const content = await readRepoFile(changedPath, cwd);\n    if (!content) continue;\n\n    // Extract relative import paths from TypeScript/JS files\n    const importMatches = content.matchAll(/from\\s+['\"]([^'\"]+)['\"]/g);\n    for (const [, importPath] of importMatches) {\n      if (!importPath || importPath.startsWith(\"node:\") || !importPath.startsWith(\".\")) continue;\n      // Resolve the import path relative to the file's directory\n      const dir = changedPath.split(\"/\").slice(0, -1).join(\"/\");\n      const candidates = [\n        `${dir}/${importPath}.ts`,\n        `${dir}/${importPath}.tsx`,\n        `${dir}/${importPath}/index.ts`,\n        `${dir}/${importPath}`,\n      ].map((p) => p.replace(/\\/\\//g, \"/\").replace(/^\\//, \"\"));\n\n      for (const candidate of candidates) {\n        if (allTrackedFiles.includes(candidate) && !changedPaths.includes(candidate)) {\n          contextPaths.add(candidate);\n          break;\n        }\n      }\n    }\n\n    if (contextPaths.size >= maxFiles) break;\n  }\n\n  return [...contextPaths].slice(0, maxFiles);\n}\n\n/**\n * Fetch a PR, its full diff, and all related codebase context;\n * run a senior-developer quality AI review;\n * submit as a formal review (with inline comments) to the hosting provider.\n */\nexport async function runReviewWorkflow(\n  gitx: Gitx,\n  prNumber: number,\n  postComment = true\n): Promise<ReviewResult> {\n  const ctx = await gitx.getRepoContext();\n  const provider = createProvider(ctx);\n  const cwd = gitx.cwd;\n\n  const fetchSpinner = ora(\"Fetching PR info, diff and comments…\").start();\n  const pr = await provider.getPR(ctx.repoSlug, prNumber);\n  const [comments, diff] = await Promise.all([\n    provider.getPRComments(ctx.repoSlug, prNumber),\n    provider.getPRDiff(ctx.repoSlug, prNumber),\n  ]);\n  fetchSpinner.succeed(`PR #${prNumber}: \"${pr.title}\"  (${pr.head} → ${pr.base})`);\n\n  // ── Build codebase context ─────────────────────────────────────────────────\n  const ctxSpinner = ora(\"Building codebase context from changed files…\").start();\n  const allTracked = await listTrackedFiles(cwd);\n  const allChangedPaths = parseChangedPathsFromDiff(diff);\n\n  // Filter out lockfiles / generated files — they waste tokens and add no review value.\n  // No file count cap — per-file budgets in buildSeniorReviewPrompt already control\n  // total size. Dropping files arbitrarily is worse than sending all of them.\n  const changedPaths = allChangedPaths.filter(isReviewableFile);\n  const skippedCount = allChangedPaths.length - changedPaths.length;\n\n  // Read full content of changed files\n  const changedFiles: Record<string, string> = {};\n  for (const p of changedPaths) {\n    const content = await readRepoFile(p, cwd);\n    if (content) changedFiles[p] = content;\n  }\n\n  // Read supporting context files (imported by the changed files)\n  // 10 context files gives the AI a solid picture of the codebase structure\n  const ctxPaths = await findContextFiles(changedPaths, allTracked, cwd, 10);\n  const contextFiles: Record<string, string> = {};\n  for (const p of ctxPaths) {\n    const content = await readRepoFile(p, cwd);\n    if (content) contextFiles[p] = content;\n  }\n\n  const skippedNote = skippedCount > 0 ? `, ${skippedCount} lock/generated files skipped` : \"\";\n  ctxSpinner.succeed(\n    `Context: ${changedPaths.length} changed files, ${ctxPaths.length} context files${skippedNote}`\n  );\n\n  const reviewSpinner = ora(\n    `Running senior-dev AI review… (this may take up to 5 min for large PRs)`\n  ).start();\n\n  let review: Awaited<ReturnType<typeof gitx.ai.reviewPRDetailed>>;\n  try {\n    review = await gitx.ai.reviewPRDetailed({\n      prTitle: pr.title,\n      prBody: pr.body,\n      author: pr.author,\n      headBranch: pr.head,\n      baseBranch: pr.base,\n      diff,\n      changedFiles,\n      contextFiles,\n      repoFileList: allTracked,\n      existingComments: comments.map((c) => ({\n        author: c.author,\n        body: c.body,\n        path: c.path,\n        line: c.line,\n      })),\n    });\n    reviewSpinner.succeed(\n      `Review complete — verdict: ${review.verdict}  |  ${review.inlineComments.length} inline comment(s)`\n    );\n  } catch (err) {\n    reviewSpinner.fail(\"AI review failed.\");\n    throw err;\n  }\n\n  // ── Build the formatted review body (for the summary comment) ─────────────\n  const verdictIcon =\n    review.verdict === \"approve\" ? \"✅\" :\n    review.verdict === \"request_changes\" ? \"🔴\" : \"💬\";\n\n  const checklistLines = review.checklist.map((c) => {\n    const icon = c.status === \"pass\" ? \"✅\" : c.status === \"warn\" ? \"⚠️\" : \"❌\";\n    return `| ${icon} | **${c.area}** | ${c.note} |`;\n  });\n\n  const issueLines = review.issues.map((i) => {\n    const sev = i.severity === \"critical\" ? \"🔴\" : i.severity === \"warning\" ? \"🟡\" : \"💡\";\n    const loc = i.file ? ` (\\`${i.file}${i.line ? `:${i.line}` : \"\"}\\`)` : \"\";\n    return `${sev} ${i.description}${loc}`;\n  });\n\n  const posLines = review.positives.map((p) => `✔ ${p}`);\n\n  const summaryBody = [\n    `## ${verdictIcon} Senior Dev AI Review (gitx) — ${review.verdict.replace(\"_\", \" \")}`,\n    \"\",\n    review.summary,\n    ...(checklistLines.length > 0\n      ? [\"\", \"### Review Checklist\", \"| Status | Area | Note |\", \"|--------|------|------|\", ...checklistLines]\n      : []),\n    ...(review.issues.length > 0 ? [\"\", \"### Issues Found\", ...issueLines] : []),\n    ...(review.positives.length > 0 ? [\"\", \"### Positives\", ...posLines] : []),\n    ...(review.testingNotes ? [\"\", \"### How to Test\", review.testingNotes] : []),\n    ...(review.inlineComments.length > 0\n      ? [`\\n> 💬 ${review.inlineComments.length} inline comment(s) posted on specific lines.`]\n      : []),\n    \"\",\n    \"*Generated by [gitx](https://github.com/g-abhishek/gitx)*\",\n  ].join(\"\\n\");\n\n  // ── Post the formal review (inline comments + verdict) ────────────────────\n  let reviewPosted = false;\n  let inlineDelivery: ReviewResult[\"inlineDelivery\"] = \"none\";\n\n  if (postComment) {\n    const inlineCount = review.inlineComments.length;\n    const postSpinner = ora(\n      `Submitting review to PR${inlineCount > 0 ? ` with ${inlineCount} inline comment(s)` : \"\"}…`\n    ).start();\n    try {\n      await provider.submitPRReview(ctx.repoSlug, prNumber, {\n        body: summaryBody,\n        event: review.verdict,\n        comments: review.inlineComments.map((c) => ({\n          path: c.path,\n          line: c.line,\n          body: c.suggestion\n            ? `${c.body}\\n\\n**Suggestion:**\\n\\`\\`\\`suggestion\\n${c.suggestion}\\n\\`\\`\\``\n            : c.body,\n        })),\n      });\n      reviewPosted = true;\n      // Determine how inline comments were delivered.\n      // GitHub's submitPRReview will attempt inline first, then fall back to\n      // plain comments. We can't detect which path was taken from here, so\n      // we report \"inline\" optimistically — the user sees the real result on GitHub.\n      inlineDelivery = inlineCount > 0 ? \"inline\" : \"none\";\n      postSpinner.succeed(\n        `Review submitted to PR.${inlineCount > 0 ? ` (${inlineCount} inline comment(s) — see PR for delivery method)` : \"\"}`\n      );\n    } catch (err) {\n      postSpinner.fail(`Could not post review to PR: ${String((err as Error).message ?? err)}`);\n      // Don't rethrow — still return the review so the user can see it locally\n    }\n  }\n\n  return { pr, comments, aiSummary: summaryBody, review, reviewPosted, inlineDelivery };\n}\n\n// ─── Fix-comments workflow ────────────────────────────────────────────────────\n\nexport interface FixCommentsResult {\n  pr: PullRequest;\n  comments: PullRequestComment[];\n  appliedFixes: Array<{ path: string; rationale: string }>;\n  skippedFixes: Array<{ path: string; reason: string }>;\n}\n\n/**\n * Fetch PR review comments, ask AI to suggest fixes, apply them, and\n * commit + push the changes.\n */\nexport async function runFixCommentsWorkflow(\n  gitx: Gitx,\n  prNumber: number,\n  dryRun = false,\n  noCommit = false\n): Promise<FixCommentsResult> {\n  const { applyUnifiedDiff, stageAll, hasStagedChanges, commitChanges } = await import(\n    \"../utils/gitOps.js\"\n  );\n  const cwd = gitx.cwd;\n  const ctx = await gitx.getRepoContext();\n  const provider = createProvider(ctx);\n\n  logger.info(`🔍 Fetching PR #${prNumber}…`);\n  const pr = await provider.getPR(ctx.repoSlug, prNumber);\n\n  logger.info(\"💬 Fetching review comments…\");\n  const comments = await provider.getPRComments(ctx.repoSlug, prNumber);\n\n  if (comments.length === 0) {\n    logger.info(\"No review comments found.\");\n    return { pr, comments, appliedFixes: [], skippedFixes: [] };\n  }\n\n  // Gather file contents for files mentioned in comments\n  const mentionedPaths = [...new Set(comments.map((c) => c.path).filter(Boolean) as string[])];\n  const trackedFiles = await listTrackedFiles(cwd);\n\n  // Also include files mentioned in the PR body / title\n  const allSourceFiles = trackedFiles\n    .filter((f) => /\\.(ts|js|tsx|jsx|py|go|rs|java|rb|cs|cpp|c|h)$/.test(f))\n    .slice(0, 15);\n\n  const relevantFiles = [...new Set([...mentionedPaths, ...allSourceFiles])].slice(0, 15);\n  const fileContents: Record<string, string> = {};\n  for (const f of relevantFiles) {\n    const content = await readRepoFile(f, cwd);\n    if (content) fileContents[f] = content.slice(0, 4000);\n  }\n\n  logger.info(\"🧠 Generating AI-suggested fixes…\");\n  const fixResult = await gitx.ai.suggestFixes({\n    comments: comments.map((c) => ({\n      body: c.body,\n      author: c.author,\n      path: c.path,\n      line: c.line,\n    })),\n    prTitle: pr.title,\n    prBody: pr.body,\n    fileContents,\n  });\n\n  const appliedFixes: Array<{ path: string; rationale: string }> = [];\n  const skippedFixes: Array<{ path: string; reason: string }> = [];\n\n  for (const edit of fixResult.suggestedEdits) {\n    if (dryRun) {\n      logger.info(`  ↳ [dry-run] Would apply fix to: ${edit.path}`);\n      logger.info(`     Rationale: ${edit.rationale}`);\n      appliedFixes.push({ path: edit.path, rationale: edit.rationale });\n      continue;\n    }\n\n    const applyResult = await applyUnifiedDiff(edit.unifiedDiff, cwd);\n    if (applyResult.ok) {\n      logger.info(`  ↳ Applied fix to: ${edit.path}`);\n      appliedFixes.push({ path: edit.path, rationale: edit.rationale });\n    } else {\n      logger.warn(`  ↳ Could not apply fix to ${edit.path}: ${applyResult.error ?? \"unknown\"}`);\n      skippedFixes.push({\n        path: edit.path,\n        reason: applyResult.error ?? \"git apply failed\",\n      });\n    }\n  }\n\n  if (!dryRun && !noCommit && appliedFixes.length > 0) {\n    await stageAll(cwd);\n    if (await hasStagedChanges(cwd)) {\n      const msg = `fix: address PR #${prNumber} review comments\\n\\n${appliedFixes.map((f) => `- ${f.path}: ${f.rationale}`).join(\"\\n\")}`;\n      const sha = await commitChanges(msg, cwd);\n      logger.success(`Committed fixes: ${sha.slice(0, 8)}`);\n\n      // Post summary comment\n      const fixSummary = appliedFixes.map((f) => `- \\`${f.path}\\`: ${f.rationale}`).join(\"\\n\");\n      await provider.addPRComment(\n        ctx.repoSlug,\n        prNumber,\n        `## 🤖 Auto-fixes applied (gitx)\\n\\n${fixSummary}\\n\\nCommit: \\`${sha.slice(0, 8)}\\``\n      );\n    }\n  }\n\n  if (!dryRun && noCommit && appliedFixes.length > 0) {\n    logger.info(\"\\n💡 Fixes applied to working tree. Review the changes and commit when ready.\");\n  }\n\n  return { pr, comments, appliedFixes, skippedFixes };\n}\n"]}

package/dist/workflows/prAddress.d.ts

@@ -0,0 +1,55 @@
+/**
+ * PR Address Workflow
+ *
+ * Reads unresolved review comments on a PR, generates AI fixes for each one,
+ * applies them to local files, and optionally commits, pushes, and replies
+ * to each thread marking it as addressed.
+ *
+ * Used by:
+ *   - `gitx pr review <n>`  — offered interactively after the review output
+ *   - `gitx sync`           — offered when unresolved comments are detected
+ */
+import type { Gitx } from "../core/gitx.js";
+import type { PullRequestComment } from "../providers/base.js";
+import type { AiFixResponse } from "../ai/types.js";
+export interface AddressedComment {
+    comment: PullRequestComment;
+    fix: AiFixResponse;
+    applied: boolean;
+    skipped: boolean;
+    /** Reason for skipping (user choice, low-confidence rejection, patch error) */
+    skipReason?: string;
+}
+export interface AddressWorkflowResult {
+    addressed: AddressedComment[];
+    filesChanged: string[];
+    pushed: boolean;
+    commitSha?: string;
+    repliedCount: number;
+}
+/**
+ * Given the full flat list of PR comments, return only the root inline comments
+ * that are genuinely unresolved — i.e.:
+ *
+ *   1. They are inline (have path + line > 0).
+ *   2. They are root comments, not replies (no inReplyToId).
+ *   3. None of their replies contain "✅ Addressed" — our bot's resolution marker.
+ *
+ * This is the single source of truth used by both the address workflow and the
+ * sync pre-check. Replacing the old body-prefix heuristic which was fragile and
+ * incorrectly filtered out valid comments.
+ */
+export declare function filterUnresolvedInlineComments(all: PullRequestComment[]): PullRequestComment[];
+export interface AddressWorkflowOptions {
+    /**
+     * "interactive"      (default) — show each fix and ask to apply; ask to commit+push at the end.
+     * "auto"             — apply all high-confidence fixes silently, commit and push.
+     * "no-push"          — interactive but never commits or pushes (local changes only).
+     * "commit-no-push"   — interactive, commits the fixes but does NOT push.
+     *                      Used by `gitx sync` so sync can push everything together
+     *                      after rebasing/merging onto the base branch.
+     */
+    mode?: "interactive" | "auto" | "no-push" | "commit-no-push";
+}
+export declare function runAddressWorkflow(gitx: Gitx, prNumber: number, opts?: AddressWorkflowOptions): Promise<AddressWorkflowResult>;
+//# sourceMappingURL=prAddress.d.ts.map

package/dist/workflows/prAddress.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"prAddress.d.ts","sourceRoot":"","sources":["../../src/workflows/prAddress.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAQH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,iBAAiB,CAAC;AAC5C,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,sBAAsB,CAAC;AAC/D,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAC;AAsBpD,MAAM,WAAW,gBAAgB;IAC/B,OAAO,EAAE,kBAAkB,CAAC;IAC5B,GAAG,EAAE,aAAa,CAAC;IACnB,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,OAAO,CAAC;IACjB,+EAA+E;IAC/E,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,MAAM,WAAW,qBAAqB;IACpC,SAAS,EAAE,gBAAgB,EAAE,CAAC;IAC9B,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,MAAM,EAAE,OAAO,CAAC;IAChB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,YAAY,EAAE,MAAM,CAAC;CACtB;AAqDD;;;;;;;;;;;GAWG;AACH,wBAAgB,8BAA8B,CAC5C,GAAG,EAAE,kBAAkB,EAAE,GACxB,kBAAkB,EAAE,CAoBtB;AAID,MAAM,WAAW,sBAAsB;IACrC;;;;;;;OAOG;IACH,IAAI,CAAC,EAAE,aAAa,GAAG,MAAM,GAAG,SAAS,GAAG,gBAAgB,CAAC;CAC9D;AAED,wBAAsB,kBAAkB,CACtC,IAAI,EAAE,IAAI,EACV,QAAQ,EAAE,MAAM,EAChB,IAAI,GAAE,sBAA2B,GAChC,OAAO,CAAC,qBAAqB,CAAC,CA8PhC"}