@g-abhishek/gitx 0.1.2 → 0.1.5

package/dist/cli/commands/pr/list.js

@@ -1,12 +1,32 @@
 import { logger } from "../../../logger/logger.js";
 import { Gitx } from "../../../core/gitx.js";
+import { createProvider } from "../../../providers/factory.js";
 export function registerPrListCommand(pr) {
     pr.command("list")
-        .description("📋 List pull requests")
-        .action(async () => {
+        .description("📋 List open pull requests")
+        .option("--state <state>", "Filter: open|closed|all", "open")
+        .action(async (options) => {
         const gitx = await Gitx.fromCwd();
-        await gitx.getRepoContext();
-        logger.warn("Not implemented yet (provider system next).");
+        const ctx = await gitx.getRepoContext();
+        logger.info(`📋 Fetching PRs for ${ctx.repoSlug} (${ctx.provider})…`);
+        const provider = createProvider(ctx);
+        const prs = await provider.listPRs(ctx.repoSlug);
+        const filtered = options.state === "all"
+            ? prs
+            : prs.filter((p) => p.state === options.state);
+        if (filtered.length === 0) {
+            logger.info(`No ${options.state} pull requests found.`);
+            return;
+        }
+        logger.info(`\nFound ${filtered.length} pull request(s):\n`);
+        for (const p of filtered) {
+            const stateIcon = p.state === "open" ? "🟢" : p.state === "merged" ? "🟣" : "🔴";
+            logger.info(`  ${stateIcon} #${p.number}  ${p.title}`);
+            logger.info(`        Branch: ${p.head} → ${p.base}`);
+            logger.info(`        Author: ${p.author}  |  Updated: ${new Date(p.updatedAt).toLocaleDateString()}`);
+            logger.info(`        URL:    ${p.url}`);
+            logger.info("");
+        }
     });
 }
 //# sourceMappingURL=list.js.map

package/dist/cli/commands/pr/list.js.map

@@ -1 +1 @@
-{"version":3,"file":"list.js","sourceRoot":"","sources":["../../../../src/cli/commands/pr/list.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,MAAM,EAAE,MAAM,2BAA2B,CAAC;AACnD,OAAO,EAAE,IAAI,EAAE,MAAM,uBAAuB,CAAC;AAE7C,MAAM,UAAU,qBAAqB,CAAC,EAAW;IAC/C,EAAE,CAAC,OAAO,CAAC,MAAM,CAAC;SACf,WAAW,CAAC,uBAAuB,CAAC;SACpC,MAAM,CAAC,KAAK,IAAI,EAAE;QACjB,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,OAAO,EAAE,CAAC;QAClC,MAAM,IAAI,CAAC,cAAc,EAAE,CAAC;QAC5B,MAAM,CAAC,IAAI,CAAC,6CAA6C,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;AACP,CAAC","sourcesContent":["import type { Command } from \"commander\";\nimport { logger } from \"../../../logger/logger.js\";\nimport { Gitx } from \"../../../core/gitx.js\";\n\nexport function registerPrListCommand(pr: Command): void {\n  pr.command(\"list\")\n    .description(\"📋 List pull requests\")\n    .action(async () => {\n      const gitx = await Gitx.fromCwd();\n      await gitx.getRepoContext();\n      logger.warn(\"Not implemented yet (provider system next).\");\n    });\n}\n"]}
+{"version":3,"file":"list.js","sourceRoot":"","sources":["../../../../src/cli/commands/pr/list.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,MAAM,EAAE,MAAM,2BAA2B,CAAC;AACnD,OAAO,EAAE,IAAI,EAAE,MAAM,uBAAuB,CAAC;AAC7C,OAAO,EAAE,cAAc,EAAE,MAAM,+BAA+B,CAAC;AAG/D,MAAM,UAAU,qBAAqB,CAAC,EAAW;IAC/C,EAAE,CAAC,OAAO,CAAC,MAAM,CAAC;SACf,WAAW,CAAC,4BAA4B,CAAC;SACzC,MAAM,CAAC,iBAAiB,EAAE,yBAAyB,EAAE,MAAM,CAAC;SAC5D,MAAM,CAAC,KAAK,EAAE,OAA0B,EAAE,EAAE;QAC3C,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,OAAO,EAAE,CAAC;QAClC,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,cAAc,EAAE,CAAC;QAExC,MAAM,CAAC,IAAI,CAAC,uBAAuB,GAAG,CAAC,QAAQ,KAAK,GAAG,CAAC,QAAQ,IAAI,CAAC,CAAC;QAEtE,MAAM,QAAQ,GAAG,cAAc,CAAC,GAAG,CAAC,CAAC;QACrC,MAAM,GAAG,GAAkB,MAAM,QAAQ,CAAC,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;QAEhE,MAAM,QAAQ,GACZ,OAAO,CAAC,KAAK,KAAK,KAAK;YACrB,CAAC,CAAC,GAAG;YACL,CAAC,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,OAAO,CAAC,KAAK,CAAC,CAAC;QAEnD,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC1B,MAAM,CAAC,IAAI,CAAC,MAAM,OAAO,CAAC,KAAK,uBAAuB,CAAC,CAAC;YACxD,OAAO;QACT,CAAC;QAED,MAAM,CAAC,IAAI,CAAC,WAAW,QAAQ,CAAC,MAAM,qBAAqB,CAAC,CAAC;QAC7D,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;YACzB,MAAM,SAAS,GAAG,CAAC,CAAC,KAAK,KAAK,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC;YACjF,MAAM,CAAC,IAAI,CACT,KAAK,SAAS,KAAK,CAAC,CAAC,MAAM,KAAK,CAAC,CAAC,KAAK,EAAE,CAC1C,CAAC;YACF,MAAM,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC,IAAI,MAAM,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;YACrD,MAAM,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC,MAAM,iBAAiB,IAAI,IAAI,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,kBAAkB,EAAE,EAAE,CAAC,CAAC;YACtG,MAAM,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC,GAAG,EAAE,CAAC,CAAC;YACxC,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAClB,CAAC;IACH,CAAC,CAAC,CAAC;AACP,CAAC","sourcesContent":["import type { Command } from \"commander\";\nimport { logger } from \"../../../logger/logger.js\";\nimport { Gitx } from \"../../../core/gitx.js\";\nimport { createProvider } from \"../../../providers/factory.js\";\nimport type { PullRequest } from \"../../../providers/base.js\";\n\nexport function registerPrListCommand(pr: Command): void {\n  pr.command(\"list\")\n    .description(\"📋 List open pull requests\")\n    .option(\"--state <state>\", \"Filter: open|closed|all\", \"open\")\n    .action(async (options: { state: string }) => {\n      const gitx = await Gitx.fromCwd();\n      const ctx = await gitx.getRepoContext();\n\n      logger.info(`📋 Fetching PRs for ${ctx.repoSlug} (${ctx.provider})…`);\n\n      const provider = createProvider(ctx);\n      const prs: PullRequest[] = await provider.listPRs(ctx.repoSlug);\n\n      const filtered =\n        options.state === \"all\"\n          ? prs\n          : prs.filter((p) => p.state === options.state);\n\n      if (filtered.length === 0) {\n        logger.info(`No ${options.state} pull requests found.`);\n        return;\n      }\n\n      logger.info(`\\nFound ${filtered.length} pull request(s):\\n`);\n      for (const p of filtered) {\n        const stateIcon = p.state === \"open\" ? \"🟢\" : p.state === \"merged\" ? \"🟣\" : \"🔴\";\n        logger.info(\n          `  ${stateIcon} #${p.number}  ${p.title}`\n        );\n        logger.info(`        Branch: ${p.head} → ${p.base}`);\n        logger.info(`        Author: ${p.author}  |  Updated: ${new Date(p.updatedAt).toLocaleDateString()}`);\n        logger.info(`        URL:    ${p.url}`);\n        logger.info(\"\");\n      }\n    });\n}\n"]}

package/dist/cli/commands/pr/merge.d.ts

@@ -0,0 +1,23 @@
+/**
+ * gitx pr merge <number>
+ *
+ * Merge a pull request via the provider API.
+ *
+ * Flow:
+ *   1. Fetch PR details and show a summary
+ *   2. Guard: PR must be open (not already merged / closed)
+ *   3. Confirm (or skip with --force)
+ *   4. Merge via API using the chosen strategy
+ *   5. Optionally delete the source branch (local + remote)
+ *   6. Optionally checkout the base branch and pull
+ *
+ * Usage:
+ *   gitx pr merge 42                    # squash-merge (default), asks to confirm
+ *   gitx pr merge 42 --method merge     # regular merge commit
+ *   gitx pr merge 42 --method rebase    # rebase and merge
+ *   gitx pr merge 42 --delete-branch    # delete source branch after merging
+ *   gitx pr merge 42 --force            # skip confirmation prompt
+ */
+import type { Command } from "commander";
+export declare function registerPrMergeCommand(pr: Command): void;
+//# sourceMappingURL=merge.d.ts.map

package/dist/cli/commands/pr/merge.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"merge.d.ts","sourceRoot":"","sources":["../../../../src/cli/commands/pr/merge.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;GAmBG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAmBzC,wBAAgB,sBAAsB,CAAC,EAAE,EAAE,OAAO,GAAG,IAAI,CAkLxD"}

package/dist/cli/commands/pr/merge.js

@@ -0,0 +1,191 @@
+/**
+ * gitx pr merge <number>
+ *
+ * Merge a pull request via the provider API.
+ *
+ * Flow:
+ *   1. Fetch PR details and show a summary
+ *   2. Guard: PR must be open (not already merged / closed)
+ *   3. Confirm (or skip with --force)
+ *   4. Merge via API using the chosen strategy
+ *   5. Optionally delete the source branch (local + remote)
+ *   6. Optionally checkout the base branch and pull
+ *
+ * Usage:
+ *   gitx pr merge 42                    # squash-merge (default), asks to confirm
+ *   gitx pr merge 42 --method merge     # regular merge commit
+ *   gitx pr merge 42 --method rebase    # rebase and merge
+ *   gitx pr merge 42 --delete-branch    # delete source branch after merging
+ *   gitx pr merge 42 --force            # skip confirmation prompt
+ */
+import inquirer from "inquirer";
+import ora from "ora";
+import { execFile } from "node:child_process";
+import { promisify } from "node:util";
+import { logger } from "../../../logger/logger.js";
+import { Gitx } from "../../../core/gitx.js";
+import { createProvider } from "../../../providers/factory.js";
+import { GitxError } from "../../../utils/errors.js";
+import { getCurrentBranch } from "../../../utils/gitOps.js";
+const execFileAsync = promisify(execFile);
+async function gitLocal(args, cwd) {
+    const { stdout } = await execFileAsync("git", args, { cwd });
+    return stdout.trim();
+}
+export function registerPrMergeCommand(pr) {
+    pr.command("merge <number>")
+        .description("🔀 Merge a pull request")
+        .option("--method <method>", "Merge strategy: squash | merge | rebase (default: squash)", "squash")
+        .option("--delete-branch", "Delete the source branch after merging", false)
+        .option("-f, --force", "Skip confirmation prompt", false)
+        .action(async (numberArg, opts) => {
+        const prNumber = parseInt(numberArg, 10);
+        if (isNaN(prNumber) || prNumber <= 0) {
+            throw new GitxError(`Invalid PR number: "${numberArg}"`, { exitCode: 2 });
+        }
+        const validMethods = ["squash", "merge", "rebase"];
+        if (!validMethods.includes(opts.method)) {
+            throw new GitxError(`Invalid merge method "${opts.method}". Choose: squash | merge | rebase`, { exitCode: 2 });
+        }
+        const method = opts.method;
+        const cwd = process.cwd();
+        const gitx = await Gitx.fromCwd(cwd);
+        const ctx = await gitx.getRepoContext();
+        const provider = createProvider(ctx);
+        // ── Fetch PR details ───────────────────────────────────────────────────
+        const fetchSpinner = ora(`Fetching PR #${prNumber}…`).start();
+        let pr_;
+        try {
+            pr_ = await provider.getPR(ctx.repoSlug, prNumber);
+            fetchSpinner.stop();
+        }
+        catch (err) {
+            fetchSpinner.fail();
+            throw err;
+        }
+        // ── Guards ─────────────────────────────────────────────────────────────
+        if (pr_.state === "merged") {
+            logger.warn(`PR #${prNumber} is already merged.`);
+            logger.info(`   ${pr_.url}`);
+            return;
+        }
+        if (pr_.state === "closed") {
+            throw new GitxError(`PR #${prNumber} is closed and cannot be merged. Reopen it first.`, { exitCode: 1 });
+        }
+        // ── Show PR summary ────────────────────────────────────────────────────
+        const methodLabel = {
+            squash: "Squash and merge",
+            merge: "Create a merge commit",
+            rebase: "Rebase and merge",
+        };
+        logger.info(`\n  #${pr_.number}  ${pr_.title}`);
+        logger.info(`  Branch:  ${pr_.head} → ${pr_.base}`);
+        logger.info(`  Author:  ${pr_.author}`);
+        logger.info(`  Method:  ${methodLabel[method]}`);
+        logger.info(`  URL:     ${pr_.url}\n`);
+        // ── Confirm ────────────────────────────────────────────────────────────
+        if (!opts.force) {
+            const { confirmed } = await inquirer.prompt([
+                {
+                    type: "confirm",
+                    name: "confirmed",
+                    message: `Merge PR #${prNumber}${opts.deleteBranch ? " and delete source branch" : ""}?`,
+                    default: true,
+                },
+            ]);
+            if (!confirmed) {
+                logger.info("Aborted — PR not merged.");
+                return;
+            }
+        }
+        // ── Merge via API ──────────────────────────────────────────────────────
+        const mergeSpinner = ora(`Merging PR #${prNumber} (${method})…`).start();
+        try {
+            await provider.mergePR(ctx.repoSlug, prNumber, {
+                method,
+                commitTitle: pr_.title,
+                deleteSourceBranch: opts.deleteBranch,
+            });
+        }
+        catch (err) {
+            mergeSpinner.fail("Merge failed.");
+            const msg = err instanceof Error ? err.message : String(err);
+            // GitHub 405 / GitLab 405/406 = merge conflicts or branch not mergeable
+            const isConflict = msg.includes("merge conflict") ||
+                msg.includes("405") ||
+                msg.includes("not mergeable") ||
+                msg.toLowerCase().includes("conflict");
+            if (isConflict) {
+                logger.error(`\n❌ PR #${prNumber} has merge conflicts.\n`);
+                logger.info(`  The branch "${pr_.head}" is out of sync with "${pr_.base}".`);
+                logger.info(`  Sync and resolve conflicts, then retry:\n`);
+                logger.info(`    gitx sync              ← rebase onto ${pr_.base} and push`);
+                logger.info(`    gitx pr merge ${prNumber}    ← retry after sync\n`);
+                logger.info(`  Or resolve manually:`);
+                logger.info(`    git fetch origin`);
+                logger.info(`    git rebase origin/${pr_.base}`);
+                logger.info(`    # fix conflicts in editor, then:`);
+                logger.info(`    git add . && git rebase --continue`);
+                logger.info(`    git push --force-with-lease`);
+                process.exitCode = 1;
+                return;
+            }
+            throw err; // re-throw non-conflict errors
+        }
+        mergeSpinner.succeed(`PR #${prNumber} merged ✓`);
+        // ── Delete source branch locally if requested ──────────────────────────
+        if (opts.deleteBranch) {
+            const currentBranch = await getCurrentBranch(cwd);
+            // If we're ON the merged branch, switch to base first
+            if (currentBranch === pr_.head) {
+                const switchSpinner = ora(`Switching to ${pr_.base}…`).start();
+                try {
+                    await gitLocal(["checkout", pr_.base], cwd);
+                    await gitLocal(["pull", "--ff-only"], cwd);
+                    switchSpinner.succeed(`Switched to ${pr_.base} and pulled latest.`);
+                }
+                catch {
+                    switchSpinner.fail(`Could not switch to ${pr_.base} — switch manually before deleting.`);
+                }
+            }
+            // Delete local branch
+            try {
+                await gitLocal(["branch", "-d", pr_.head], cwd);
+                logger.success(`   Deleted local branch: ${pr_.head}`);
+            }
+            catch {
+                // -d refuses to delete if not fully merged locally; use -D
+                try {
+                    await gitLocal(["branch", "-D", pr_.head], cwd);
+                    logger.success(`   Force-deleted local branch: ${pr_.head}`);
+                }
+                catch {
+                    logger.warn(`   Could not delete local branch "${pr_.head}" — delete it manually.`);
+                }
+            }
+            // Delete remote branch
+            try {
+                await gitLocal(["push", "origin", "--delete", pr_.head], cwd);
+                logger.success(`   Deleted remote branch: origin/${pr_.head}`);
+            }
+            catch {
+                logger.warn(`   Could not delete remote branch "origin/${pr_.head}" — it may already be gone.`);
+            }
+        }
+        else {
+            // Even without --delete-branch, offer to pull base if we're on it
+            const currentBranch = await getCurrentBranch(cwd);
+            if (currentBranch === pr_.base) {
+                try {
+                    await gitLocal(["pull", "--ff-only"], cwd);
+                    logger.success(`   Pulled latest ${pr_.base}.`);
+                }
+                catch {
+                    // Non-fatal
+                }
+            }
+        }
+        logger.success(`\n✅ Done! ${pr_.url}`);
+    });
+}
+//# sourceMappingURL=merge.js.map

package/dist/cli/commands/pr/merge.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"merge.js","sourceRoot":"","sources":["../../../../src/cli/commands/pr/merge.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;GAmBG;AAGH,OAAO,QAAQ,MAAM,UAAU,CAAC;AAChC,OAAO,GAAG,MAAM,KAAK,CAAC;AACtB,OAAO,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAC9C,OAAO,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AACtC,OAAO,EAAE,MAAM,EAAE,MAAM,2BAA2B,CAAC;AACnD,OAAO,EAAE,IAAI,EAAE,MAAM,uBAAuB,CAAC;AAC7C,OAAO,EAAE,cAAc,EAAE,MAAM,+BAA+B,CAAC;AAE/D,OAAO,EAAE,SAAS,EAAE,MAAM,0BAA0B,CAAC;AACrD,OAAO,EAAE,gBAAgB,EAAE,MAAM,0BAA0B,CAAC;AAE5D,MAAM,aAAa,GAAG,SAAS,CAAC,QAAQ,CAAC,CAAC;AAE1C,KAAK,UAAU,QAAQ,CAAC,IAAc,EAAE,GAAW;IACjD,MAAM,EAAE,MAAM,EAAE,GAAG,MAAM,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,GAAG,EAAE,CAAC,CAAC;IAC7D,OAAO,MAAM,CAAC,IAAI,EAAE,CAAC;AACvB,CAAC;AAED,MAAM,UAAU,sBAAsB,CAAC,EAAW;IAChD,EAAE,CAAC,OAAO,CAAC,gBAAgB,CAAC;SACzB,WAAW,CAAC,yBAAyB,CAAC;SACtC,MAAM,CACL,mBAAmB,EACnB,2DAA2D,EAC3D,QAAQ,CACT;SACA,MAAM,CAAC,iBAAiB,EAAE,wCAAwC,EAAE,KAAK,CAAC;SAC1E,MAAM,CAAC,aAAa,EAAE,0BAA0B,EAAE,KAAK,CAAC;SACxD,MAAM,CAAC,KAAK,EAAE,SAAiB,EAAE,IAIjC,EAAE,EAAE;QACH,MAAM,QAAQ,GAAG,QAAQ,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC;QACzC,IAAI,KAAK,CAAC,QAAQ,CAAC,IAAI,QAAQ,IAAI,CAAC,EAAE,CAAC;YACrC,MAAM,IAAI,SAAS,CAAC,uBAAuB,SAAS,GAAG,EAAE,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QAC5E,CAAC;QAED,MAAM,YAAY,GAAG,CAAC,QAAQ,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAC;QACnD,IAAI,CAAC,YAAY,CAAC,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC;YACxC,MAAM,IAAI,SAAS,CACjB,yBAAyB,IAAI,CAAC,MAAM,oCAAoC,EACxE,EAAE,QAAQ,EAAE,CAAC,EAAE,CAChB,CAAC;QACJ,CAAC;QACD,MAAM,MAAM,GAAG,IAAI,CAAC,MAAkC,CAAC;QAEvD,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;QAC1B,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;QACrC,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,cAAc,EAAE,CAAC;QACxC,MAAM,QAAQ,GAAG,cAAc,CAAC,GAAG,CAAC,CAAC;QAErC,0EAA0E;QAC1E,MAAM,YAAY,GAAG,GAAG,CAAC,gBAAgB,QAAQ,GAAG,CAAC,CAAC,KAAK,EAAE,CAAC;QAC9D,IAAI,GAA+C,CAAC;QACpD,IAAI,CAAC;YACH,GAAG,GAAG,MAAM,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;YACnD,YAAY,CAAC,IAAI,EAAE,CAAC;QACtB,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,YAAY,CAAC,IAAI,EAAE,CAAC;YACpB,MAAM,GAAG,CAAC;QACZ,CAAC;QAED,0EAA0E;QAC1E,IAAI,GAAG,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;YAC3B,MAAM,CAAC,IAAI,CAAC,OAAO,QAAQ,qBAAqB,CAAC,CAAC;YAClD,MAAM,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,GAAG,EAAE,CAAC,CAAC;YAC7B,OAAO;QACT,CAAC;QACD,IAAI,GAAG,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;YAC3B,MAAM,IAAI,SAAS,CACjB,OAAO,QAAQ,mDAAmD,EAClE,EAAE,QAAQ,EAAE,CAAC,EAAE,CAChB,CAAC;QACJ,CAAC;QAED,0EAA0E;QAC1E,MAAM,WAAW,GAA6C;YAC5D,MAAM,EAAE,kBAAkB;YAC1B,KAAK,EAAG,uBAAuB;YAC/B,MAAM,EAAE,kBAAkB;SAC3B,CAAC;QAEF,MAAM,CAAC,IAAI,CAAC,QAAQ,GAAG,CAAC,MAAM,KAAK,GAAG,CAAC,KAAK,EAAE,CAAC,CAAC;QAChD,MAAM,CAAC,IAAI,CAAC,cAAc,GAAG,CAAC,IAAI,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC;QACpD,MAAM,CAAC,IAAI,CAAC,cAAc,GAAG,CAAC,MAAM,EAAE,CAAC,CAAC;QACxC,MAAM,CAAC,IAAI,CAAC,cAAc,WAAW,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QACjD,MAAM,CAAC,IAAI,CAAC,cAAc,GAAG,CAAC,GAAG,IAAI,CAAC,CAAC;QAEvC,0EAA0E;QAC1E,IAAI,CAAC,IAAI,CAAC,KAAK,EAAE,CAAC;YAChB,MAAM,EAAE,SAAS,EAAE,GAAG,MAAM,QAAQ,CAAC,MAAM,CAAyB;gBAClE;oBACE,IAAI,EAAE,SAAS;oBACf,IAAI,EAAE,WAAW;oBACjB,OAAO,EAAE,aAAa,QAAQ,GAAG,IAAI,CAAC,YAAY,CAAC,CAAC,CAAC,2BAA2B,CAAC,CAAC,CAAC,EAAE,GAAG;oBACxF,OAAO,EAAE,IAAI;iBACd;aACF,CAAC,CAAC;YACH,IAAI,CAAC,SAAS,EAAE,CAAC;gBACf,MAAM,CAAC,IAAI,CAAC,0BAA0B,CAAC,CAAC;gBACxC,OAAO;YACT,CAAC;QACH,CAAC;QAED,0EAA0E;QAC1E,MAAM,YAAY,GAAG,GAAG,CAAC,eAAe,QAAQ,KAAK,MAAM,IAAI,CAAC,CAAC,KAAK,EAAE,CAAC;QACzE,IAAI,CAAC;YACH,MAAM,QAAQ,CAAC,OAAO,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,EAAE;gBAC7C,MAAM;gBACN,WAAW,EAAE,GAAG,CAAC,KAAK;gBACtB,kBAAkB,EAAE,IAAI,CAAC,YAAY;aACtC,CAAC,CAAC;QACL,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,YAAY,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;YACnC,MAAM,GAAG,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;YAE7D,wEAAwE;YACxE,MAAM,UAAU,GACd,GAAG,CAAC,QAAQ,CAAC,gBAAgB,CAAC;gBAC9B,GAAG,CAAC,QAAQ,CAAC,KAAK,CAAC;gBACnB,GAAG,CAAC,QAAQ,CAAC,eAAe,CAAC;gBAC7B,GAAG,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC;YAEzC,IAAI,UAAU,EAAE,CAAC;gBACf,MAAM,CAAC,KAAK,CAAC,WAAW,QAAQ,yBAAyB,CAAC,CAAC;gBAC3D,MAAM,CAAC,IAAI,CAAC,iBAAiB,GAAG,CAAC,IAAI,0BAA0B,GAAG,CAAC,IAAI,IAAI,CAAC,CAAC;gBAC7E,MAAM,CAAC,IAAI,CAAC,6CAA6C,CAAC,CAAC;gBAC3D,MAAM,CAAC,IAAI,CAAC,4CAA4C,GAAG,CAAC,IAAI,WAAW,CAAC,CAAC;gBAC7E,MAAM,CAAC,IAAI,CAAC,qBAAqB,QAAQ,0BAA0B,CAAC,CAAC;gBACrE,MAAM,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC;gBACtC,MAAM,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAC;gBACpC,MAAM,CAAC,IAAI,CAAC,yBAAyB,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC;gBACjD,MAAM,CAAC,IAAI,CAAC,sCAAsC,CAAC,CAAC;gBACpD,MAAM,CAAC,IAAI,CAAC,wCAAwC,CAAC,CAAC;gBACtD,MAAM,CAAC,IAAI,CAAC,iCAAiC,CAAC,CAAC;gBAC/C,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;gBACrB,OAAO;YACT,CAAC;YAED,MAAM,GAAG,CAAC,CAAC,+BAA+B;QAC5C,CAAC;QACD,YAAY,CAAC,OAAO,CAAC,OAAO,QAAQ,WAAW,CAAC,CAAC;QAEjD,0EAA0E;QAC1E,IAAI,IAAI,CAAC,YAAY,EAAE,CAAC;YACtB,MAAM,aAAa,GAAG,MAAM,gBAAgB,CAAC,GAAG,CAAC,CAAC;YAElD,sDAAsD;YACtD,IAAI,aAAa,KAAK,GAAG,CAAC,IAAI,EAAE,CAAC;gBAC/B,MAAM,aAAa,GAAG,GAAG,CAAC,gBAAgB,GAAG,CAAC,IAAI,GAAG,CAAC,CAAC,KAAK,EAAE,CAAC;gBAC/D,IAAI,CAAC;oBACH,MAAM,QAAQ,CAAC,CAAC,UAAU,EAAE,GAAG,CAAC,IAAI,CAAC,EAAE,GAAG,CAAC,CAAC;oBAC5C,MAAM,QAAQ,CAAC,CAAC,MAAM,EAAE,WAAW,CAAC,EAAE,GAAG,CAAC,CAAC;oBAC3C,aAAa,CAAC,OAAO,CAAC,eAAe,GAAG,CAAC,IAAI,qBAAqB,CAAC,CAAC;gBACtE,CAAC;gBAAC,MAAM,CAAC;oBACP,aAAa,CAAC,IAAI,CAAC,uBAAuB,GAAG,CAAC,IAAI,qCAAqC,CAAC,CAAC;gBAC3F,CAAC;YACH,CAAC;YAED,sBAAsB;YACtB,IAAI,CAAC;gBACH,MAAM,QAAQ,CAAC,CAAC,QAAQ,EAAE,IAAI,EAAE,GAAG,CAAC,IAAI,CAAC,EAAE,GAAG,CAAC,CAAC;gBAChD,MAAM,CAAC,OAAO,CAAC,4BAA4B,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC;YACzD,CAAC;YAAC,MAAM,CAAC;gBACP,2DAA2D;gBAC3D,IAAI,CAAC;oBACH,MAAM,QAAQ,CAAC,CAAC,QAAQ,EAAE,IAAI,EAAE,GAAG,CAAC,IAAI,CAAC,EAAE,GAAG,CAAC,CAAC;oBAChD,MAAM,CAAC,OAAO,CAAC,kCAAkC,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC;gBAC/D,CAAC;gBAAC,MAAM,CAAC;oBACP,MAAM,CAAC,IAAI,CAAC,qCAAqC,GAAG,CAAC,IAAI,yBAAyB,CAAC,CAAC;gBACtF,CAAC;YACH,CAAC;YAED,uBAAuB;YACvB,IAAI,CAAC;gBACH,MAAM,QAAQ,CAAC,CAAC,MAAM,EAAE,QAAQ,EAAE,UAAU,EAAE,GAAG,CAAC,IAAI,CAAC,EAAE,GAAG,CAAC,CAAC;gBAC9D,MAAM,CAAC,OAAO,CAAC,oCAAoC,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC;YACjE,CAAC;YAAC,MAAM,CAAC;gBACP,MAAM,CAAC,IAAI,CAAC,6CAA6C,GAAG,CAAC,IAAI,6BAA6B,CAAC,CAAC;YAClG,CAAC;QACH,CAAC;aAAM,CAAC;YACN,kEAAkE;YAClE,MAAM,aAAa,GAAG,MAAM,gBAAgB,CAAC,GAAG,CAAC,CAAC;YAClD,IAAI,aAAa,KAAK,GAAG,CAAC,IAAI,EAAE,CAAC;gBAC/B,IAAI,CAAC;oBACH,MAAM,QAAQ,CAAC,CAAC,MAAM,EAAE,WAAW,CAAC,EAAE,GAAG,CAAC,CAAC;oBAC3C,MAAM,CAAC,OAAO,CAAC,oBAAoB,GAAG,CAAC,IAAI,GAAG,CAAC,CAAC;gBAClD,CAAC;gBAAC,MAAM,CAAC;oBACP,YAAY;gBACd,CAAC;YACH,CAAC;QACH,CAAC;QAED,MAAM,CAAC,OAAO,CAAC,aAAa,GAAG,CAAC,GAAG,EAAE,CAAC,CAAC;IACzC,CAAC,CAAC,CAAC;AACP,CAAC","sourcesContent":["/**\n * gitx pr merge <number>\n *\n * Merge a pull request via the provider API.\n *\n * Flow:\n *   1. Fetch PR details and show a summary\n *   2. Guard: PR must be open (not already merged / closed)\n *   3. Confirm (or skip with --force)\n *   4. Merge via API using the chosen strategy\n *   5. Optionally delete the source branch (local + remote)\n *   6. Optionally checkout the base branch and pull\n *\n * Usage:\n *   gitx pr merge 42                    # squash-merge (default), asks to confirm\n *   gitx pr merge 42 --method merge     # regular merge commit\n *   gitx pr merge 42 --method rebase    # rebase and merge\n *   gitx pr merge 42 --delete-branch    # delete source branch after merging\n *   gitx pr merge 42 --force            # skip confirmation prompt\n */\n\nimport type { Command } from \"commander\";\nimport inquirer from \"inquirer\";\nimport ora from \"ora\";\nimport { execFile } from \"node:child_process\";\nimport { promisify } from \"node:util\";\nimport { logger } from \"../../../logger/logger.js\";\nimport { Gitx } from \"../../../core/gitx.js\";\nimport { createProvider } from \"../../../providers/factory.js\";\nimport type { MergePrOptions } from \"../../../providers/base.js\";\nimport { GitxError } from \"../../../utils/errors.js\";\nimport { getCurrentBranch } from \"../../../utils/gitOps.js\";\n\nconst execFileAsync = promisify(execFile);\n\nasync function gitLocal(args: string[], cwd: string): Promise<string> {\n  const { stdout } = await execFileAsync(\"git\", args, { cwd });\n  return stdout.trim();\n}\n\nexport function registerPrMergeCommand(pr: Command): void {\n  pr.command(\"merge <number>\")\n    .description(\"🔀 Merge a pull request\")\n    .option(\n      \"--method <method>\",\n      \"Merge strategy: squash | merge | rebase (default: squash)\",\n      \"squash\"\n    )\n    .option(\"--delete-branch\", \"Delete the source branch after merging\", false)\n    .option(\"-f, --force\", \"Skip confirmation prompt\", false)\n    .action(async (numberArg: string, opts: {\n      method: string;\n      deleteBranch: boolean;\n      force: boolean;\n    }) => {\n      const prNumber = parseInt(numberArg, 10);\n      if (isNaN(prNumber) || prNumber <= 0) {\n        throw new GitxError(`Invalid PR number: \"${numberArg}\"`, { exitCode: 2 });\n      }\n\n      const validMethods = [\"squash\", \"merge\", \"rebase\"];\n      if (!validMethods.includes(opts.method)) {\n        throw new GitxError(\n          `Invalid merge method \"${opts.method}\". Choose: squash | merge | rebase`,\n          { exitCode: 2 }\n        );\n      }\n      const method = opts.method as MergePrOptions[\"method\"];\n\n      const cwd = process.cwd();\n      const gitx = await Gitx.fromCwd(cwd);\n      const ctx = await gitx.getRepoContext();\n      const provider = createProvider(ctx);\n\n      // ── Fetch PR details ───────────────────────────────────────────────────\n      const fetchSpinner = ora(`Fetching PR #${prNumber}…`).start();\n      let pr_: Awaited<ReturnType<typeof provider.getPR>>;\n      try {\n        pr_ = await provider.getPR(ctx.repoSlug, prNumber);\n        fetchSpinner.stop();\n      } catch (err) {\n        fetchSpinner.fail();\n        throw err;\n      }\n\n      // ── Guards ─────────────────────────────────────────────────────────────\n      if (pr_.state === \"merged\") {\n        logger.warn(`PR #${prNumber} is already merged.`);\n        logger.info(`   ${pr_.url}`);\n        return;\n      }\n      if (pr_.state === \"closed\") {\n        throw new GitxError(\n          `PR #${prNumber} is closed and cannot be merged. Reopen it first.`,\n          { exitCode: 1 }\n        );\n      }\n\n      // ── Show PR summary ────────────────────────────────────────────────────\n      const methodLabel: Record<MergePrOptions[\"method\"], string> = {\n        squash: \"Squash and merge\",\n        merge:  \"Create a merge commit\",\n        rebase: \"Rebase and merge\",\n      };\n\n      logger.info(`\\n  #${pr_.number}  ${pr_.title}`);\n      logger.info(`  Branch:  ${pr_.head} → ${pr_.base}`);\n      logger.info(`  Author:  ${pr_.author}`);\n      logger.info(`  Method:  ${methodLabel[method]}`);\n      logger.info(`  URL:     ${pr_.url}\\n`);\n\n      // ── Confirm ────────────────────────────────────────────────────────────\n      if (!opts.force) {\n        const { confirmed } = await inquirer.prompt<{ confirmed: boolean }>([\n          {\n            type: \"confirm\",\n            name: \"confirmed\",\n            message: `Merge PR #${prNumber}${opts.deleteBranch ? \" and delete source branch\" : \"\"}?`,\n            default: true,\n          },\n        ]);\n        if (!confirmed) {\n          logger.info(\"Aborted — PR not merged.\");\n          return;\n        }\n      }\n\n      // ── Merge via API ──────────────────────────────────────────────────────\n      const mergeSpinner = ora(`Merging PR #${prNumber} (${method})…`).start();\n      try {\n        await provider.mergePR(ctx.repoSlug, prNumber, {\n          method,\n          commitTitle: pr_.title,\n          deleteSourceBranch: opts.deleteBranch,\n        });\n      } catch (err) {\n        mergeSpinner.fail(\"Merge failed.\");\n        const msg = err instanceof Error ? err.message : String(err);\n\n        // GitHub 405 / GitLab 405/406 = merge conflicts or branch not mergeable\n        const isConflict =\n          msg.includes(\"merge conflict\") ||\n          msg.includes(\"405\") ||\n          msg.includes(\"not mergeable\") ||\n          msg.toLowerCase().includes(\"conflict\");\n\n        if (isConflict) {\n          logger.error(`\\n❌ PR #${prNumber} has merge conflicts.\\n`);\n          logger.info(`  The branch \"${pr_.head}\" is out of sync with \"${pr_.base}\".`);\n          logger.info(`  Sync and resolve conflicts, then retry:\\n`);\n          logger.info(`    gitx sync              ← rebase onto ${pr_.base} and push`);\n          logger.info(`    gitx pr merge ${prNumber}    ← retry after sync\\n`);\n          logger.info(`  Or resolve manually:`);\n          logger.info(`    git fetch origin`);\n          logger.info(`    git rebase origin/${pr_.base}`);\n          logger.info(`    # fix conflicts in editor, then:`);\n          logger.info(`    git add . && git rebase --continue`);\n          logger.info(`    git push --force-with-lease`);\n          process.exitCode = 1;\n          return;\n        }\n\n        throw err; // re-throw non-conflict errors\n      }\n      mergeSpinner.succeed(`PR #${prNumber} merged ✓`);\n\n      // ── Delete source branch locally if requested ──────────────────────────\n      if (opts.deleteBranch) {\n        const currentBranch = await getCurrentBranch(cwd);\n\n        // If we're ON the merged branch, switch to base first\n        if (currentBranch === pr_.head) {\n          const switchSpinner = ora(`Switching to ${pr_.base}…`).start();\n          try {\n            await gitLocal([\"checkout\", pr_.base], cwd);\n            await gitLocal([\"pull\", \"--ff-only\"], cwd);\n            switchSpinner.succeed(`Switched to ${pr_.base} and pulled latest.`);\n          } catch {\n            switchSpinner.fail(`Could not switch to ${pr_.base} — switch manually before deleting.`);\n          }\n        }\n\n        // Delete local branch\n        try {\n          await gitLocal([\"branch\", \"-d\", pr_.head], cwd);\n          logger.success(`   Deleted local branch: ${pr_.head}`);\n        } catch {\n          // -d refuses to delete if not fully merged locally; use -D\n          try {\n            await gitLocal([\"branch\", \"-D\", pr_.head], cwd);\n            logger.success(`   Force-deleted local branch: ${pr_.head}`);\n          } catch {\n            logger.warn(`   Could not delete local branch \"${pr_.head}\" — delete it manually.`);\n          }\n        }\n\n        // Delete remote branch\n        try {\n          await gitLocal([\"push\", \"origin\", \"--delete\", pr_.head], cwd);\n          logger.success(`   Deleted remote branch: origin/${pr_.head}`);\n        } catch {\n          logger.warn(`   Could not delete remote branch \"origin/${pr_.head}\" — it may already be gone.`);\n        }\n      } else {\n        // Even without --delete-branch, offer to pull base if we're on it\n        const currentBranch = await getCurrentBranch(cwd);\n        if (currentBranch === pr_.base) {\n          try {\n            await gitLocal([\"pull\", \"--ff-only\"], cwd);\n            logger.success(`   Pulled latest ${pr_.base}.`);\n          } catch {\n            // Non-fatal\n          }\n        }\n      }\n\n      logger.success(`\\n✅ Done! ${pr_.url}`);\n    });\n}\n"]}

package/dist/cli/commands/pr/resolve.d.ts

@@ -0,0 +1,3 @@
+import type { Command } from "commander";
+export declare function registerPrResolveCommand(pr: Command): void;
+//# sourceMappingURL=resolve.d.ts.map

package/dist/cli/commands/pr/resolve.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"resolve.d.ts","sourceRoot":"","sources":["../../../../src/cli/commands/pr/resolve.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAOzC,wBAAgB,wBAAwB,CAAC,EAAE,EAAE,OAAO,GAAG,IAAI,CA+F1D"}

package/dist/cli/commands/pr/resolve.js

@@ -0,0 +1,92 @@
+import inquirer from "inquirer";
+import { logger } from "../../../logger/logger.js";
+import { Gitx } from "../../../core/gitx.js";
+import { runFixCommentsWorkflow } from "../../../workflows/pr.js";
+import { pushBranch, getCurrentBranch, isWorkingTreeDirty } from "../../../utils/gitOps.js";
+export function registerPrResolveCommand(pr) {
+    pr.command("resolve")
+        .description("🔧 AI-resolve review comments: applies code fixes, commits, and pushes")
+        .argument("<id>", "Pull request number")
+        .option("--dry-run", "Preview fixes without applying or committing", false)
+        .option("--no-commit", "Apply fixes to working tree only — skip commit and push", false)
+        .option("--no-push", "Apply & commit locally but skip push", false)
+        .action(async (id, options) => {
+        const prNumber = parseInt(id, 10);
+        if (isNaN(prNumber) || prNumber <= 0) {
+            logger.error(`Invalid PR number: ${id}`);
+            process.exit(1);
+        }
+        // Commander sets options.commit = false when --no-commit is passed
+        const noCommit = options.commit === false;
+        const gitx = await Gitx.fromCwd();
+        const ctx = await gitx.getRepoContext();
+        logger.info(`🔧 Resolving review comments on PR #${prNumber} (${ctx.repoSlug})…\n`);
+        // ── AI availability check ─────────────────────────────────────────────
+        if (!await Gitx.isAiAvailable(gitx.config)) {
+            logger.warn("⚠️  No AI provider configured — cannot generate fixes.\n" +
+                "   Run `gitx config` to set up an AI provider (Claude, OpenAI, or claude-cli).");
+            return;
+        }
+        // ── Confirmation prompt ───────────────────────────────────────────────
+        if (!options.dryRun) {
+            const modeDesc = noCommit
+                ? "apply fixes to your working tree only (no commit or push)"
+                : options.push === false
+                    ? "apply fixes and commit locally (no push)"
+                    : "apply fixes, commit, and push";
+            const { proceed } = await inquirer.prompt([
+                {
+                    type: "confirm",
+                    name: "proceed",
+                    message: `This will ${modeDesc}. Continue?`,
+                    default: false,
+                },
+            ]);
+            if (!proceed) {
+                logger.warn("Cancelled.");
+                return;
+            }
+        }
+        // ── Guard: warn about uncommitted changes ─────────────────────────────
+        if (!options.dryRun) {
+            const dirty = await isWorkingTreeDirty(gitx.cwd);
+            if (dirty) {
+                logger.warn("⚠️  You have uncommitted changes. They may conflict with applied fixes.");
+                const { cont } = await inquirer.prompt([
+                    { type: "confirm", name: "cont", message: "Continue anyway?", default: false },
+                ]);
+                if (!cont) {
+                    logger.warn("Cancelled.");
+                    return;
+                }
+            }
+        }
+        const result = await runFixCommentsWorkflow(gitx, prNumber, options.dryRun, noCommit);
+        logger.info(`\n📋 PR: ${result.pr.title}`);
+        if (result.appliedFixes.length === 0 && result.skippedFixes.length === 0) {
+            logger.info("No actionable review comments found.");
+            return;
+        }
+        if (result.appliedFixes.length > 0) {
+            logger.success(`\n✅ Applied ${result.appliedFixes.length} fix(es):`);
+            result.appliedFixes.forEach((f) => logger.info(`  • ${f.path} — ${f.rationale}`));
+        }
+        if (result.skippedFixes.length > 0) {
+            logger.warn(`\n⚠️  Skipped ${result.skippedFixes.length} fix(es):`);
+            result.skippedFixes.forEach((f) => logger.warn(`  • ${f.path}: ${f.reason}`));
+        }
+        // ── Push if applicable ────────────────────────────────────────────────
+        if (!options.dryRun && !noCommit && options.push !== false && result.appliedFixes.length > 0) {
+            const branch = await getCurrentBranch(gitx.cwd);
+            logger.info(`\n🚀 Pushing ${branch}…`);
+            try {
+                await pushBranch(branch, gitx.cwd);
+                logger.success("Branch pushed.");
+            }
+            catch (err) {
+                logger.warn(`Push failed: ${String(err.message ?? err)}`);
+            }
+        }
+    });
+}
+//# sourceMappingURL=resolve.js.map

package/dist/cli/commands/pr/resolve.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"resolve.js","sourceRoot":"","sources":["../../../../src/cli/commands/pr/resolve.ts"],"names":[],"mappings":"AACA,OAAO,QAAQ,MAAM,UAAU,CAAC;AAChC,OAAO,EAAE,MAAM,EAAE,MAAM,2BAA2B,CAAC;AACnD,OAAO,EAAE,IAAI,EAAE,MAAM,uBAAuB,CAAC;AAC7C,OAAO,EAAE,sBAAsB,EAAE,MAAM,0BAA0B,CAAC;AAClE,OAAO,EAAE,UAAU,EAAE,gBAAgB,EAAE,kBAAkB,EAAE,MAAM,0BAA0B,CAAC;AAE5F,MAAM,UAAU,wBAAwB,CAAC,EAAW;IAClD,EAAE,CAAC,OAAO,CAAC,SAAS,CAAC;SAClB,WAAW,CAAC,wEAAwE,CAAC;SACrF,QAAQ,CAAC,MAAM,EAAE,qBAAqB,CAAC;SACvC,MAAM,CAAC,WAAW,EAAE,8CAA8C,EAAE,KAAK,CAAC;SAC1E,MAAM,CAAC,aAAa,EAAE,yDAAyD,EAAE,KAAK,CAAC;SACvF,MAAM,CAAC,WAAW,EAAE,sCAAsC,EAAE,KAAK,CAAC;SAClE,MAAM,CAAC,KAAK,EAAE,EAAU,EAAE,OAA4D,EAAE,EAAE;QACzF,MAAM,QAAQ,GAAG,QAAQ,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC;QAClC,IAAI,KAAK,CAAC,QAAQ,CAAC,IAAI,QAAQ,IAAI,CAAC,EAAE,CAAC;YACrC,MAAM,CAAC,KAAK,CAAC,sBAAsB,EAAE,EAAE,CAAC,CAAC;YACzC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAClB,CAAC;QAED,mEAAmE;QACnE,MAAM,QAAQ,GAAG,OAAO,CAAC,MAAM,KAAK,KAAK,CAAC;QAE1C,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,OAAO,EAAE,CAAC;QAClC,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,cAAc,EAAE,CAAC;QACxC,MAAM,CAAC,IAAI,CAAC,uCAAuC,QAAQ,KAAK,GAAG,CAAC,QAAQ,MAAM,CAAC,CAAC;QAEpF,yEAAyE;QACzE,IAAI,CAAC,MAAM,IAAI,CAAC,aAAa,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC;YAC3C,MAAM,CAAC,IAAI,CACT,0DAA0D;gBAC1D,gFAAgF,CACjF,CAAC;YACF,OAAO;QACT,CAAC;QAED,yEAAyE;QACzE,IAAI,CAAC,OAAO,CAAC,MAAM,EAAE,CAAC;YACpB,MAAM,QAAQ,GAAG,QAAQ;gBACvB,CAAC,CAAC,2DAA2D;gBAC7D,CAAC,CAAC,OAAO,CAAC,IAAI,KAAK,KAAK;oBACtB,CAAC,CAAC,0CAA0C;oBAC5C,CAAC,CAAC,+BAA+B,CAAC;YAEtC,MAAM,EAAE,OAAO,EAAE,GAAG,MAAM,QAAQ,CAAC,MAAM,CAAuB;gBAC9D;oBACE,IAAI,EAAE,SAAS;oBACf,IAAI,EAAE,SAAS;oBACf,OAAO,EAAE,aAAa,QAAQ,aAAa;oBAC3C,OAAO,EAAE,KAAK;iBACf;aACF,CAAC,CAAC;YACH,IAAI,CAAC,OAAO,EAAE,CAAC;gBACb,MAAM,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;gBAC1B,OAAO;YACT,CAAC;QACH,CAAC;QAED,yEAAyE;QACzE,IAAI,CAAC,OAAO,CAAC,MAAM,EAAE,CAAC;YACpB,MAAM,KAAK,GAAG,MAAM,kBAAkB,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACjD,IAAI,KAAK,EAAE,CAAC;gBACV,MAAM,CAAC,IAAI,CAAC,yEAAyE,CAAC,CAAC;gBACvF,MAAM,EAAE,IAAI,EAAE,GAAG,MAAM,QAAQ,CAAC,MAAM,CAAoB;oBACxD,EAAE,IAAI,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,kBAAkB,EAAE,OAAO,EAAE,KAAK,EAAE;iBAC/E,CAAC,CAAC;gBACH,IAAI,CAAC,IAAI,EAAE,CAAC;oBAAC,MAAM,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;oBAAC,OAAO;gBAAC,CAAC;YACnD,CAAC;QACH,CAAC;QAED,MAAM,MAAM,GAAG,MAAM,sBAAsB,CAAC,IAAI,EAAE,QAAQ,EAAE,OAAO,CAAC,MAAM,EAAE,QAAQ,CAAC,CAAC;QAEtF,MAAM,CAAC,IAAI,CAAC,YAAY,MAAM,CAAC,EAAE,CAAC,KAAK,EAAE,CAAC,CAAC;QAE3C,IAAI,MAAM,CAAC,YAAY,CAAC,MAAM,KAAK,CAAC,IAAI,MAAM,CAAC,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACzE,MAAM,CAAC,IAAI,CAAC,sCAAsC,CAAC,CAAC;YACpD,OAAO;QACT,CAAC;QAED,IAAI,MAAM,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACnC,MAAM,CAAC,OAAO,CAAC,eAAe,MAAM,CAAC,YAAY,CAAC,MAAM,WAAW,CAAC,CAAC;YACrE,MAAM,CAAC,YAAY,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,MAAM,CAAC,CAAC,SAAS,EAAE,CAAC,CAAC,CAAC;QACpF,CAAC;QAED,IAAI,MAAM,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACnC,MAAM,CAAC,IAAI,CAAC,iBAAiB,MAAM,CAAC,YAAY,CAAC,MAAM,WAAW,CAAC,CAAC;YACpE,MAAM,CAAC,YAAY,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,KAAK,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QAChF,CAAC;QAED,yEAAyE;QACzE,IAAI,CAAC,OAAO,CAAC,MAAM,IAAI,CAAC,QAAQ,IAAI,OAAO,CAAC,IAAI,KAAK,KAAK,IAAI,MAAM,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC7F,MAAM,MAAM,GAAG,MAAM,gBAAgB,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YAChD,MAAM,CAAC,IAAI,CAAC,gBAAgB,MAAM,GAAG,CAAC,CAAC;YACvC,IAAI,CAAC;gBACH,MAAM,UAAU,CAAC,MAAM,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC;gBACnC,MAAM,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAC;YACnC,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,MAAM,CAAC,IAAI,CAAC,gBAAgB,MAAM,CAAE,GAAa,CAAC,OAAO,IAAI,GAAG,CAAC,EAAE,CAAC,CAAC;YACvE,CAAC;QACH,CAAC;IACH,CAAC,CAAC,CAAC;AACP,CAAC","sourcesContent":["import type { Command } from \"commander\";\nimport inquirer from \"inquirer\";\nimport { logger } from \"../../../logger/logger.js\";\nimport { Gitx } from \"../../../core/gitx.js\";\nimport { runFixCommentsWorkflow } from \"../../../workflows/pr.js\";\nimport { pushBranch, getCurrentBranch, isWorkingTreeDirty } from \"../../../utils/gitOps.js\";\n\nexport function registerPrResolveCommand(pr: Command): void {\n  pr.command(\"resolve\")\n    .description(\"🔧 AI-resolve review comments: applies code fixes, commits, and pushes\")\n    .argument(\"<id>\", \"Pull request number\")\n    .option(\"--dry-run\", \"Preview fixes without applying or committing\", false)\n    .option(\"--no-commit\", \"Apply fixes to working tree only — skip commit and push\", false)\n    .option(\"--no-push\", \"Apply & commit locally but skip push\", false)\n    .action(async (id: string, options: { dryRun: boolean; commit: boolean; push: boolean }) => {\n      const prNumber = parseInt(id, 10);\n      if (isNaN(prNumber) || prNumber <= 0) {\n        logger.error(`Invalid PR number: ${id}`);\n        process.exit(1);\n      }\n\n      // Commander sets options.commit = false when --no-commit is passed\n      const noCommit = options.commit === false;\n\n      const gitx = await Gitx.fromCwd();\n      const ctx = await gitx.getRepoContext();\n      logger.info(`🔧 Resolving review comments on PR #${prNumber} (${ctx.repoSlug})…\\n`);\n\n      // ── AI availability check ─────────────────────────────────────────────\n      if (!await Gitx.isAiAvailable(gitx.config)) {\n        logger.warn(\n          \"⚠️  No AI provider configured — cannot generate fixes.\\n\" +\n          \"   Run `gitx config` to set up an AI provider (Claude, OpenAI, or claude-cli).\"\n        );\n        return;\n      }\n\n      // ── Confirmation prompt ───────────────────────────────────────────────\n      if (!options.dryRun) {\n        const modeDesc = noCommit\n          ? \"apply fixes to your working tree only (no commit or push)\"\n          : options.push === false\n            ? \"apply fixes and commit locally (no push)\"\n            : \"apply fixes, commit, and push\";\n\n        const { proceed } = await inquirer.prompt<{ proceed: boolean }>([\n          {\n            type: \"confirm\",\n            name: \"proceed\",\n            message: `This will ${modeDesc}. Continue?`,\n            default: false,\n          },\n        ]);\n        if (!proceed) {\n          logger.warn(\"Cancelled.\");\n          return;\n        }\n      }\n\n      // ── Guard: warn about uncommitted changes ─────────────────────────────\n      if (!options.dryRun) {\n        const dirty = await isWorkingTreeDirty(gitx.cwd);\n        if (dirty) {\n          logger.warn(\"⚠️  You have uncommitted changes. They may conflict with applied fixes.\");\n          const { cont } = await inquirer.prompt<{ cont: boolean }>([\n            { type: \"confirm\", name: \"cont\", message: \"Continue anyway?\", default: false },\n          ]);\n          if (!cont) { logger.warn(\"Cancelled.\"); return; }\n        }\n      }\n\n      const result = await runFixCommentsWorkflow(gitx, prNumber, options.dryRun, noCommit);\n\n      logger.info(`\\n📋 PR: ${result.pr.title}`);\n\n      if (result.appliedFixes.length === 0 && result.skippedFixes.length === 0) {\n        logger.info(\"No actionable review comments found.\");\n        return;\n      }\n\n      if (result.appliedFixes.length > 0) {\n        logger.success(`\\n✅ Applied ${result.appliedFixes.length} fix(es):`);\n        result.appliedFixes.forEach((f) => logger.info(`  • ${f.path} — ${f.rationale}`));\n      }\n\n      if (result.skippedFixes.length > 0) {\n        logger.warn(`\\n⚠️  Skipped ${result.skippedFixes.length} fix(es):`);\n        result.skippedFixes.forEach((f) => logger.warn(`  • ${f.path}: ${f.reason}`));\n      }\n\n      // ── Push if applicable ────────────────────────────────────────────────\n      if (!options.dryRun && !noCommit && options.push !== false && result.appliedFixes.length > 0) {\n        const branch = await getCurrentBranch(gitx.cwd);\n        logger.info(`\\n🚀 Pushing ${branch}…`);\n        try {\n          await pushBranch(branch, gitx.cwd);\n          logger.success(\"Branch pushed.\");\n        } catch (err) {\n          logger.warn(`Push failed: ${String((err as Error).message ?? err)}`);\n        }\n      }\n    });\n}\n"]}

package/dist/cli/commands/pr/review.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"review.d.ts","sourceRoot":"","sources":["../../../../src/cli/commands/pr/review.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAIzC,wBAAgB,uBAAuB,CAAC,EAAE,EAAE,OAAO,GAAG,IAAI,CAUzD"}
+{"version":3,"file":"review.d.ts","sourceRoot":"","sources":["../../../../src/cli/commands/pr/review.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAKzC,wBAAgB,uBAAuB,CAAC,EAAE,EAAE,OAAO,GAAG,IAAI,CAqJzD"}

package/dist/cli/commands/pr/review.js

@@ -1,14 +1,129 @@
 import { logger } from "../../../logger/logger.js";
 import { Gitx } from "../../../core/gitx.js";
+import { runReviewWorkflow } from "../../../workflows/pr.js";
 export function registerPrReviewCommand(pr) {
     pr.command("review")
-        .description("🧐 Review a pull request")
-        .argument("<id>", "Pull request id/number")
-        .action(async (id) => {
+        .description("🧐 Senior-dev AI review: inline comments, checklist, verdict")
+        .argument("<id>", "Pull request number")
+        .option("--no-comment", "Show review locally only — do not post to PR")
+        .option("--no-inline", "Post overall review but skip inline file comments")
+        .action(async (id, options) => {
+        const prNumber = parseInt(id, 10);
+        if (isNaN(prNumber) || prNumber <= 0) {
+            logger.error(`Invalid PR number: ${id}`);
+            process.exit(1);
+        }
         const gitx = await Gitx.fromCwd();
-        await gitx.getRepoContext();
-        logger.warn("Not implemented yet (provider system next).");
-        logger.info(`PR: ${id}`);
+        if (!await Gitx.isAiAvailable(gitx.config)) {
+            logger.warn("⚠️  No AI provider configured — review will not be meaningful.\n" +
+                "   Run: gitx config set anthropic   (or openai / claude-cli)");
+            return;
+        }
+        const postComment = options.comment !== false;
+        logger.info(`\n🧐 Senior-dev AI review — PR #${prNumber}\n`);
+        let result;
+        try {
+            result = await runReviewWorkflow(gitx, prNumber, postComment);
+        }
+        catch (err) {
+            logger.error(`Review failed: ${String(err.message ?? err)}`);
+            process.exitCode = 1;
+            return;
+        }
+        const { pr: pullReq, review, comments } = result;
+        // ── PR header ─────────────────────────────────────────────────────────
+        logger.info(`\n📋  ${pullReq.title}`);
+        logger.info(`    ${pullReq.head} → ${pullReq.base}  ·  by ${pullReq.author}  ·  ${pullReq.state}`);
+        logger.info(`    ${pullReq.url}\n`);
+        if (!review) {
+            logger.info(`🤖 Review:\n${result.aiSummary}`);
+            return;
+        }
+        // ── Verdict ───────────────────────────────────────────────────────────
+        const verdictIcon = review.verdict === "approve"
+            ? "✅  APPROVE"
+            : review.verdict === "request_changes"
+                ? "🔴  REQUEST CHANGES"
+                : "💬  COMMENT";
+        logger.info(`🤖 Verdict: ${verdictIcon}\n`);
+        // ── Summary ───────────────────────────────────────────────────────────
+        logger.info(`📝 Summary:\n   ${review.summary.replace(/\n/g, "\n   ")}\n`);
+        // ── Checklist ─────────────────────────────────────────────────────────
+        if (review.checklist.length > 0) {
+            logger.info("✅ Review Checklist:");
+            for (const item of review.checklist) {
+                const icon = item.status === "pass" ? "✅" : item.status === "warn" ? "⚠️ " : "❌";
+                logger.info(`   ${icon}  ${item.area.padEnd(20)} ${item.note}`);
+            }
+            logger.info("");
+        }
+        // ── Issues ────────────────────────────────────────────────────────────
+        const criticals = review.issues.filter((i) => i.severity === "critical");
+        const warnings = review.issues.filter((i) => i.severity === "warning");
+        const suggestions = review.issues.filter((i) => i.severity === "suggestion");
+        if (review.issues.length > 0) {
+            logger.info(`🔎 Issues (${review.issues.length}):`);
+            for (const issue of [...criticals, ...warnings, ...suggestions]) {
+                const icon = issue.severity === "critical" ? "🔴" : issue.severity === "warning" ? "🟡" : "💡";
+                const loc = issue.file ? `  [${issue.file}${issue.line ? `:${issue.line}` : ""}]` : "";
+                logger.info(`   ${icon} ${issue.description}${loc}`);
+            }
+            logger.info("");
+        }
+        // ── Inline comments ───────────────────────────────────────────────────
+        if (review.inlineComments.length > 0) {
+            logger.info(`💬 Inline Comments (${review.inlineComments.length}):`);
+            for (const c of review.inlineComments) {
+                const icon = c.severity === "critical" ? "🔴" : c.severity === "warning" ? "🟡" : "💡";
+                logger.info(`\n   ${icon} ${c.path}:${c.line}`);
+                logger.info(`      ${c.body.replace(/\n/g, "\n      ")}`);
+                if (c.suggestion) {
+                    logger.info(`      📌 Suggestion: ${c.suggestion.split("\n")[0]}…`);
+                }
+            }
+            logger.info("");
+        }
+        // ── Positives ─────────────────────────────────────────────────────────
+        if (review.positives.length > 0) {
+            logger.info("👍 Positives:");
+            for (const p of review.positives)
+                logger.info(`   ✔  ${p}`);
+            logger.info("");
+        }
+        // ── Testing notes ─────────────────────────────────────────────────────
+        if (review.testingNotes) {
+            logger.info(`🧪 How to test:\n   ${review.testingNotes.replace(/\n/g, "\n   ")}\n`);
+        }
+        // ── Existing PR comments ──────────────────────────────────────────────
+        if (comments.length > 0) {
+            logger.info(`💬 Existing comments on PR (${comments.length}):`);
+            for (const c of comments.slice(0, 4)) {
+                const loc = c.path ? ` @ ${c.path}${c.line ? `:${c.line}` : ""}` : "";
+                logger.info(`   [${c.author}${loc}]: ${c.body.slice(0, 120)}${c.body.length > 120 ? "…" : ""}`);
+            }
+            if (comments.length > 4)
+                logger.info(`   … and ${comments.length - 4} more.`);
+            logger.info("");
+        }
+        // ── Status ────────────────────────────────────────────────────────────
+        if (!postComment) {
+            logger.info("ℹ️  Review shown locally only (--no-comment).");
+        }
+        else if (result.reviewPosted) {
+            const inlineCount = review.inlineComments.length;
+            if (inlineCount > 0) {
+                logger.success(`✅ Review posted to PR. ${inlineCount} inline comment(s) — check the PR to see delivery method.`);
+                logger.info(`   (If lines were outside the diff, comments were posted as plain PR comments instead.)`);
+            }
+            else {
+                logger.success("✅ Review posted to PR.");
+            }
+        }
+        else {
+            logger.warn("⚠️  Review could not be posted to the PR (see error above).\n" +
+                "   The full review is shown above — you can copy it manually.");
+        }
+        // To resolve the review comments, run: gitx pr resolve <PR_number>
     });
 }
 //# sourceMappingURL=review.js.map

package/dist/cli/commands/pr/review.js.map

@@ -1 +1 @@
-{"version":3,"file":"review.js","sourceRoot":"","sources":["../../../../src/cli/commands/pr/review.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,MAAM,EAAE,MAAM,2BAA2B,CAAC;AACnD,OAAO,EAAE,IAAI,EAAE,MAAM,uBAAuB,CAAC;AAE7C,MAAM,UAAU,uBAAuB,CAAC,EAAW;IACjD,EAAE,CAAC,OAAO,CAAC,QAAQ,CAAC;SACjB,WAAW,CAAC,0BAA0B,CAAC;SACvC,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;SAC1C,MAAM,CAAC,KAAK,EAAE,EAAU,EAAE,EAAE;QAC3B,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,OAAO,EAAE,CAAC;QAClC,MAAM,IAAI,CAAC,cAAc,EAAE,CAAC;QAC5B,MAAM,CAAC,IAAI,CAAC,6CAA6C,CAAC,CAAC;QAC3D,MAAM,CAAC,IAAI,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC;IAC3B,CAAC,CAAC,CAAC;AACP,CAAC","sourcesContent":["import type { Command } from \"commander\";\nimport { logger } from \"../../../logger/logger.js\";\nimport { Gitx } from \"../../../core/gitx.js\";\n\nexport function registerPrReviewCommand(pr: Command): void {\n  pr.command(\"review\")\n    .description(\"🧐 Review a pull request\")\n    .argument(\"<id>\", \"Pull request id/number\")\n    .action(async (id: string) => {\n      const gitx = await Gitx.fromCwd();\n      await gitx.getRepoContext();\n      logger.warn(\"Not implemented yet (provider system next).\");\n      logger.info(`PR: ${id}`);\n    });\n}\n"]}
+{"version":3,"file":"review.js","sourceRoot":"","sources":["../../../../src/cli/commands/pr/review.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,MAAM,EAAE,MAAM,2BAA2B,CAAC;AACnD,OAAO,EAAE,IAAI,EAAE,MAAM,uBAAuB,CAAC;AAC7C,OAAO,EAAE,iBAAiB,EAAE,MAAM,0BAA0B,CAAC;AAE7D,MAAM,UAAU,uBAAuB,CAAC,EAAW;IACjD,EAAE,CAAC,OAAO,CAAC,QAAQ,CAAC;SACjB,WAAW,CAAC,8DAA8D,CAAC;SAC3E,QAAQ,CAAC,MAAM,EAAE,qBAAqB,CAAC;SACvC,MAAM,CAAC,cAAc,EAAE,8CAA8C,CAAC;SACtE,MAAM,CAAC,aAAa,EAAE,mDAAmD,CAAC;SAC1E,MAAM,CAAC,KAAK,EAAE,EAAU,EAAE,OAA8C,EAAE,EAAE;QAC3E,MAAM,QAAQ,GAAG,QAAQ,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC;QAClC,IAAI,KAAK,CAAC,QAAQ,CAAC,IAAI,QAAQ,IAAI,CAAC,EAAE,CAAC;YACrC,MAAM,CAAC,KAAK,CAAC,sBAAsB,EAAE,EAAE,CAAC,CAAC;YACzC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAClB,CAAC;QAED,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,OAAO,EAAE,CAAC;QAElC,IAAI,CAAC,MAAM,IAAI,CAAC,aAAa,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC;YAC3C,MAAM,CAAC,IAAI,CACT,kEAAkE;gBAClE,8DAA8D,CAC/D,CAAC;YACF,OAAO;QACT,CAAC;QAED,MAAM,WAAW,GAAG,OAAO,CAAC,OAAO,KAAK,KAAK,CAAC;QAE9C,MAAM,CAAC,IAAI,CAAC,mCAAmC,QAAQ,IAAI,CAAC,CAAC;QAE7D,IAAI,MAAqD,CAAC;QAC1D,IAAI,CAAC;YACH,MAAM,GAAG,MAAM,iBAAiB,CAAC,IAAI,EAAE,QAAQ,EAAE,WAAW,CAAC,CAAC;QAChE,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,MAAM,CAAC,KAAK,CAAC,kBAAkB,MAAM,CAAE,GAAa,CAAC,OAAO,IAAI,GAAG,CAAC,EAAE,CAAC,CAAC;YACxE,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;YACrB,OAAO;QACT,CAAC;QAED,MAAM,EAAE,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,GAAG,MAAM,CAAC;QAEjD,yEAAyE;QACzE,MAAM,CAAC,IAAI,CAAC,SAAS,OAAO,CAAC,KAAK,EAAE,CAAC,CAAC;QACtC,MAAM,CAAC,IAAI,CAAC,OAAO,OAAO,CAAC,IAAI,MAAM,OAAO,CAAC,IAAI,WAAW,OAAO,CAAC,MAAM,QAAQ,OAAO,CAAC,KAAK,EAAE,CAAC,CAAC;QACnG,MAAM,CAAC,IAAI,CAAC,OAAO,OAAO,CAAC,GAAG,IAAI,CAAC,CAAC;QAEpC,IAAI,CAAC,MAAM,EAAE,CAAC;YACZ,MAAM,CAAC,IAAI,CAAC,eAAe,MAAM,CAAC,SAAS,EAAE,CAAC,CAAC;YAC/C,OAAO;QACT,CAAC;QAED,yEAAyE;QACzE,MAAM,WAAW,GACf,MAAM,CAAC,OAAO,KAAK,SAAS;YAC1B,CAAC,CAAC,YAAY;YACd,CAAC,CAAC,MAAM,CAAC,OAAO,KAAK,iBAAiB;gBACtC,CAAC,CAAC,qBAAqB;gBACvB,CAAC,CAAC,aAAa,CAAC;QAEpB,MAAM,CAAC,IAAI,CAAC,eAAe,WAAW,IAAI,CAAC,CAAC;QAE5C,yEAAyE;QACzE,MAAM,CAAC,IAAI,CAAC,mBAAmB,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,KAAK,EAAE,OAAO,CAAC,IAAI,CAAC,CAAC;QAE3E,yEAAyE;QACzE,IAAI,MAAM,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAChC,MAAM,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;YACnC,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,SAAS,EAAE,CAAC;gBACpC,MAAM,IAAI,GAAG,IAAI,CAAC,MAAM,KAAK,MAAM,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,KAAK,MAAM,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC;gBACjF,MAAM,CAAC,IAAI,CAAC,MAAM,IAAI,KAAK,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC;YAClE,CAAC;YACD,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAClB,CAAC;QAED,yEAAyE;QACzE,MAAM,SAAS,GAAM,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,UAAU,CAAC,CAAC;QAC5E,MAAM,QAAQ,GAAO,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,SAAS,CAAC,CAAC;QAC3E,MAAM,WAAW,GAAI,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,YAAY,CAAC,CAAC;QAE9E,IAAI,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC7B,MAAM,CAAC,IAAI,CAAC,cAAc,MAAM,CAAC,MAAM,CAAC,MAAM,IAAI,CAAC,CAAC;YACpD,KAAK,MAAM,KAAK,IAAI,CAAC,GAAG,SAAS,EAAE,GAAG,QAAQ,EAAE,GAAG,WAAW,CAAC,EAAE,CAAC;gBAChE,MAAM,IAAI,GAAG,KAAK,CAAC,QAAQ,KAAK,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,KAAK,CAAC,QAAQ,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC;gBAC/F,MAAM,GAAG,GAAI,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,KAAK,CAAC,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;gBACxF,MAAM,CAAC,IAAI,CAAC,MAAM,IAAI,IAAI,KAAK,CAAC,WAAW,GAAG,GAAG,EAAE,CAAC,CAAC;YACvD,CAAC;YACD,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAClB,CAAC;QAED,yEAAyE;QACzE,IAAI,MAAM,CAAC,cAAc,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACrC,MAAM,CAAC,IAAI,CAAC,uBAAuB,MAAM,CAAC,cAAc,CAAC,MAAM,IAAI,CAAC,CAAC;YACrE,KAAK,MAAM,CAAC,IAAI,MAAM,CAAC,cAAc,EAAE,CAAC;gBACtC,MAAM,IAAI,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;gBACvF,MAAM,CAAC,IAAI,CAAC,QAAQ,IAAI,IAAI,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;gBAChD,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,UAAU,CAAC,EAAE,CAAC,CAAC;gBAC1D,IAAI,CAAC,CAAC,UAAU,EAAE,CAAC;oBACjB,MAAM,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC,UAAU,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC;gBACtE,CAAC;YACH,CAAC;YACD,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAClB,CAAC;QAED,yEAAyE;QACzE,IAAI,MAAM,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAChC,MAAM,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;YAC7B,KAAK,MAAM,CAAC,IAAI,MAAM,CAAC,SAAS;gBAAE,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC,CAAC;YAC5D,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAClB,CAAC;QAED,yEAAyE;QACzE,IAAI,MAAM,CAAC,YAAY,EAAE,CAAC;YACxB,MAAM,CAAC,IAAI,CAAC,uBAAuB,MAAM,CAAC,YAAY,CAAC,OAAO,CAAC,KAAK,EAAE,OAAO,CAAC,IAAI,CAAC,CAAC;QACtF,CAAC;QAED,yEAAyE;QACzE,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACxB,MAAM,CAAC,IAAI,CAAC,+BAA+B,QAAQ,CAAC,MAAM,IAAI,CAAC,CAAC;YAChE,KAAK,MAAM,CAAC,IAAI,QAAQ,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,CAAC;gBACrC,MAAM,GAAG,GAAG,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;gBACtE,MAAM,CAAC,IAAI,CACT,OAAO,CAAC,CAAC,MAAM,GAAG,GAAG,MAAM,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,MAAM,GAAG,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CACnF,CAAC;YACJ,CAAC;YACD,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC;gBAAE,MAAM,CAAC,IAAI,CAAC,YAAY,QAAQ,CAAC,MAAM,GAAG,CAAC,QAAQ,CAAC,CAAC;YAC9E,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAClB,CAAC;QAED,yEAAyE;QACzE,IAAI,CAAC,WAAW,EAAE,CAAC;YACjB,MAAM,CAAC,IAAI,CAAC,+CAA+C,CAAC,CAAC;QAC/D,CAAC;aAAM,IAAI,MAAM,CAAC,YAAY,EAAE,CAAC;YAC/B,MAAM,WAAW,GAAG,MAAM,CAAC,cAAc,CAAC,MAAM,CAAC;YACjD,IAAI,WAAW,GAAG,CAAC,EAAE,CAAC;gBACpB,MAAM,CAAC,OAAO,CACZ,0BAA0B,WAAW,2DAA2D,CACjG,CAAC;gBACF,MAAM,CAAC,IAAI,CACT,yFAAyF,CAC1F,CAAC;YACJ,CAAC;iBAAM,CAAC;gBACN,MAAM,CAAC,OAAO,CAAC,wBAAwB,CAAC,CAAC;YAC3C,CAAC;QACH,CAAC;aAAM,CAAC;YACN,MAAM,CAAC,IAAI,CACT,+DAA+D;gBAC/D,+DAA+D,CAChE,CAAC;QACJ,CAAC;QAED,mEAAmE;IACrE,CAAC,CAAC,CAAC;AACP,CAAC","sourcesContent":["import type { Command } from \"commander\";\nimport { logger } from \"../../../logger/logger.js\";\nimport { Gitx } from \"../../../core/gitx.js\";\nimport { runReviewWorkflow } from \"../../../workflows/pr.js\";\n\nexport function registerPrReviewCommand(pr: Command): void {\n  pr.command(\"review\")\n    .description(\"🧐 Senior-dev AI review: inline comments, checklist, verdict\")\n    .argument(\"<id>\", \"Pull request number\")\n    .option(\"--no-comment\", \"Show review locally only — do not post to PR\")\n    .option(\"--no-inline\", \"Post overall review but skip inline file comments\")\n    .action(async (id: string, options: { comment: boolean; inline: boolean }) => {\n      const prNumber = parseInt(id, 10);\n      if (isNaN(prNumber) || prNumber <= 0) {\n        logger.error(`Invalid PR number: ${id}`);\n        process.exit(1);\n      }\n\n      const gitx = await Gitx.fromCwd();\n\n      if (!await Gitx.isAiAvailable(gitx.config)) {\n        logger.warn(\n          \"⚠️  No AI provider configured — review will not be meaningful.\\n\" +\n          \"   Run: gitx config set anthropic   (or openai / claude-cli)\"\n        );\n        return;\n      }\n\n      const postComment = options.comment !== false;\n\n      logger.info(`\\n🧐 Senior-dev AI review — PR #${prNumber}\\n`);\n\n      let result: Awaited<ReturnType<typeof runReviewWorkflow>>;\n      try {\n        result = await runReviewWorkflow(gitx, prNumber, postComment);\n      } catch (err) {\n        logger.error(`Review failed: ${String((err as Error).message ?? err)}`);\n        process.exitCode = 1;\n        return;\n      }\n\n      const { pr: pullReq, review, comments } = result;\n\n      // ── PR header ─────────────────────────────────────────────────────────\n      logger.info(`\\n📋  ${pullReq.title}`);\n      logger.info(`    ${pullReq.head} → ${pullReq.base}  ·  by ${pullReq.author}  ·  ${pullReq.state}`);\n      logger.info(`    ${pullReq.url}\\n`);\n\n      if (!review) {\n        logger.info(`🤖 Review:\\n${result.aiSummary}`);\n        return;\n      }\n\n      // ── Verdict ───────────────────────────────────────────────────────────\n      const verdictIcon =\n        review.verdict === \"approve\"\n          ? \"✅  APPROVE\"\n          : review.verdict === \"request_changes\"\n          ? \"🔴  REQUEST CHANGES\"\n          : \"💬  COMMENT\";\n\n      logger.info(`🤖 Verdict: ${verdictIcon}\\n`);\n\n      // ── Summary ───────────────────────────────────────────────────────────\n      logger.info(`📝 Summary:\\n   ${review.summary.replace(/\\n/g, \"\\n   \")}\\n`);\n\n      // ── Checklist ─────────────────────────────────────────────────────────\n      if (review.checklist.length > 0) {\n        logger.info(\"✅ Review Checklist:\");\n        for (const item of review.checklist) {\n          const icon = item.status === \"pass\" ? \"✅\" : item.status === \"warn\" ? \"⚠️ \" : \"❌\";\n          logger.info(`   ${icon}  ${item.area.padEnd(20)} ${item.note}`);\n        }\n        logger.info(\"\");\n      }\n\n      // ── Issues ────────────────────────────────────────────────────────────\n      const criticals    = review.issues.filter((i) => i.severity === \"critical\");\n      const warnings     = review.issues.filter((i) => i.severity === \"warning\");\n      const suggestions  = review.issues.filter((i) => i.severity === \"suggestion\");\n\n      if (review.issues.length > 0) {\n        logger.info(`🔎 Issues (${review.issues.length}):`);\n        for (const issue of [...criticals, ...warnings, ...suggestions]) {\n          const icon = issue.severity === \"critical\" ? \"🔴\" : issue.severity === \"warning\" ? \"🟡\" : \"💡\";\n          const loc  = issue.file ? `  [${issue.file}${issue.line ? `:${issue.line}` : \"\"}]` : \"\";\n          logger.info(`   ${icon} ${issue.description}${loc}`);\n        }\n        logger.info(\"\");\n      }\n\n      // ── Inline comments ───────────────────────────────────────────────────\n      if (review.inlineComments.length > 0) {\n        logger.info(`💬 Inline Comments (${review.inlineComments.length}):`);\n        for (const c of review.inlineComments) {\n          const icon = c.severity === \"critical\" ? \"🔴\" : c.severity === \"warning\" ? \"🟡\" : \"💡\";\n          logger.info(`\\n   ${icon} ${c.path}:${c.line}`);\n          logger.info(`      ${c.body.replace(/\\n/g, \"\\n      \")}`);\n          if (c.suggestion) {\n            logger.info(`      📌 Suggestion: ${c.suggestion.split(\"\\n\")[0]}…`);\n          }\n        }\n        logger.info(\"\");\n      }\n\n      // ── Positives ─────────────────────────────────────────────────────────\n      if (review.positives.length > 0) {\n        logger.info(\"👍 Positives:\");\n        for (const p of review.positives) logger.info(`   ✔  ${p}`);\n        logger.info(\"\");\n      }\n\n      // ── Testing notes ─────────────────────────────────────────────────────\n      if (review.testingNotes) {\n        logger.info(`🧪 How to test:\\n   ${review.testingNotes.replace(/\\n/g, \"\\n   \")}\\n`);\n      }\n\n      // ── Existing PR comments ──────────────────────────────────────────────\n      if (comments.length > 0) {\n        logger.info(`💬 Existing comments on PR (${comments.length}):`);\n        for (const c of comments.slice(0, 4)) {\n          const loc = c.path ? ` @ ${c.path}${c.line ? `:${c.line}` : \"\"}` : \"\";\n          logger.info(\n            `   [${c.author}${loc}]: ${c.body.slice(0, 120)}${c.body.length > 120 ? \"…\" : \"\"}`\n          );\n        }\n        if (comments.length > 4) logger.info(`   … and ${comments.length - 4} more.`);\n        logger.info(\"\");\n      }\n\n      // ── Status ────────────────────────────────────────────────────────────\n      if (!postComment) {\n        logger.info(\"ℹ️  Review shown locally only (--no-comment).\");\n      } else if (result.reviewPosted) {\n        const inlineCount = review.inlineComments.length;\n        if (inlineCount > 0) {\n          logger.success(\n            `✅ Review posted to PR. ${inlineCount} inline comment(s) — check the PR to see delivery method.`\n          );\n          logger.info(\n            `   (If lines were outside the diff, comments were posted as plain PR comments instead.)`\n          );\n        } else {\n          logger.success(\"✅ Review posted to PR.\");\n        }\n      } else {\n        logger.warn(\n          \"⚠️  Review could not be posted to the PR (see error above).\\n\" +\n          \"   The full review is shown above — you can copy it manually.\"\n        );\n      }\n\n      // To resolve the review comments, run: gitx pr resolve <PR_number>\n    });\n}\n\n"]}

package/dist/cli/commands/push.d.ts

@@ -0,0 +1,16 @@
+/**
+ * gitx push
+ *
+ * The single-command workflow: stage everything → AI-generate a conventional
+ * commit message → commit → push to origin.  No flags needed.
+ *
+ * Usage:
+ *   gitx push                  # full auto: stage + AI commit + push
+ *   gitx push --staged         # commit only already-staged changes, leave the rest untouched
+ *   gitx push -m "fix: typo"   # use a custom message, skip AI
+ *   gitx push --dry-run        # preview message without committing or pushing
+ *   gitx push --branch feat    # push to a specific branch instead of current
+ */
+import type { Command } from "commander";
+export declare function registerPushCommand(program: Command): void;
+//# sourceMappingURL=push.d.ts.map