@bobbyg603/mog 1.2.0 → 1.4.0

package/README.md

@@ -19,7 +19,7 @@ That's it. `mog` will:
 ## Prerequisites
 
 - **macOS or Windows** (Docker sandbox microVMs require Docker Desktop)
-- **Docker Desktop** — running and up to date (must support `docker sandbox`)
+- **Docker Desktop 4.40+** — running and up to date. Docker sandbox support (required by mog) was introduced in Docker Desktop 4.40. Verify with `docker sandbox ls`.
 - **Bun** — install from [bun.sh](https://bun.sh)
 - **GitHub CLI** (`gh`) — authenticated via `gh auth login`
 - **Git** with push access to your target repos
@@ -33,6 +33,9 @@ bun install -g @bobbyg603/mog
 ## Quick start
 
 ```bash
+# 0. Verify Docker sandbox support is available
+docker sandbox ls
+
 # 1. One-time setup: create sandbox & authenticate
 mog init
 # This launches Claude Code — use /login to authenticate with your Max subscription
@@ -131,6 +134,10 @@ git worktree remove ../repo-worktrees/123-fix-broken-login
 
 **"No changes detected"** — Claude may have struggled with the issue. Check the worktree manually, or re-run with a more detailed issue description.
 
+**"Docker sandbox state is stale"** — Restart Docker Desktop, or remove and recreate the sandbox: `docker sandbox rm mog && mog init`.
+
+**"docker: 'sandbox' is not a docker command"** — Your Docker Desktop version doesn't support sandboxes. Update Docker Desktop to **4.40 or later**, then verify with `docker sandbox ls`.
+
 **"Failed to push"** — Ensure `gh` is authenticated with push access. Try `gh auth login` and select HTTPS.
 
 ## Managing the sandbox

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobbyg603/mog",
-  "version": "1.2.0",
+  "version": "1.4.0",
   "description": "One command to go from GitHub issue to pull request, powered by Claude Code in a Docker sandbox",
   "module": "src/index.ts",
   "type": "module",

package/src/github.ts

@@ -4,6 +4,7 @@ export interface Issue {
   title: string;
   body: string;
   labels: string;
+  comments: string;
 }
 
 export function fetchIssue(repo: string, issueNum: string): Issue {
@@ -12,7 +13,7 @@ export function fetchIssue(repo: string, issueNum: string): Issue {
   const proc = Bun.spawnSync([
     "gh", "issue", "view", issueNum,
     "--repo", repo,
-    "--json", "title,body,labels",
+    "--json", "title,body,labels,comments",
   ]);
 
   if (proc.exitCode !== 0) {
@@ -21,13 +22,60 @@ export function fetchIssue(repo: string, issueNum: string): Issue {
 
   const json = JSON.parse(proc.stdout.toString());
 
+  const comments = (json.comments || [])
+    .map((c: { author: { login: string }; body: string }) => `**@${c.author.login}:** ${c.body}`)
+    .join("\n\n");
+
   return {
     title: json.title,
     body: json.body || "No description provided.",
     labels: json.labels?.map((l: { name: string }) => l.name).join(", ") || "none",
+    comments,
   };
 }
 
+export function listIssues(repo: string, verbose: boolean): void {
+  log.info(`Fetching open issues for ${repo}...`);
+
+  const fields = verbose
+    ? "number,title,body,labels,assignees"
+    : "number,title";
+
+  const proc = Bun.spawnSync([
+    "gh", "issue", "list",
+    "--repo", repo,
+    "--state", "open",
+    "--json", fields,
+  ]);
+
+  if (proc.exitCode !== 0) {
+    log.die(`Failed to fetch issues for ${repo}. Check the repo name.`);
+  }
+
+  const issues = JSON.parse(proc.stdout.toString());
+
+  if (issues.length === 0) {
+    log.info("No open issues found.");
+    return;
+  }
+
+  log.ok(`${issues.length} open issue(s):\n`);
+
+  for (const issue of issues) {
+    if (verbose) {
+      const labels = issue.labels?.map((l: { name: string }) => l.name).join(", ") || "none";
+      const assignees = issue.assignees?.map((a: { login: string }) => a.login).join(", ") || "unassigned";
+      console.log(`  #${issue.number}  ${issue.title}`);
+      console.log(`         Labels: ${labels}`);
+      console.log(`         Assignees: ${assignees}`);
+      console.log(`         ${(issue.body || "No description.").split("\n")[0]}`);
+      console.log();
+    } else {
+      console.log(`  #${issue.number}  ${issue.title}`);
+    }
+  }
+}
+
 export function pushAndCreatePR(
   repo: string,
   worktreeDir: string,
@@ -65,6 +113,22 @@ export function pushAndCreatePR(
     }
   }
 
+  // Squash all commits into one
+  const commitCount = Bun.spawnSync(["git", "rev-list", "--count", `${defaultBranch}..HEAD`], { cwd: worktreeDir });
+  const count = parseInt(commitCount.stdout.toString().trim(), 10) || 0;
+  if (count > 1) {
+    log.info(`Squashing ${count} commits into one...`);
+    const prefix = issue.labels.includes("enhancement") || issue.labels.includes("feature") ? "feat" : "fix";
+    const squash = Bun.spawnSync(["git", "reset", "--soft", defaultBranch], { cwd: worktreeDir });
+    if (squash.exitCode === 0) {
+      const msg = `${prefix}: ${issue.title.toLowerCase()} (#${issueNum})`;
+      Bun.spawnSync(["git", "commit", "-m", msg], { cwd: worktreeDir });
+      log.ok("Commits squashed.");
+    } else {
+      log.warn("Failed to squash — pushing individual commits instead.");
+    }
+  }
+
   // Push
   log.info(`Pushing branch '${branchName}' to origin...`);
   const push = Bun.spawnSync(["git", "push", "-u", "origin", branchName], { cwd: worktreeDir });

package/src/index.ts

@@ -1,7 +1,9 @@
 #!/usr/bin/env bun
 
-import { fetchIssue } from "./github";
-import { ensureRepo, createWorktree } from "./worktree";
+import fs from "fs";
+import path from "path";
+import { fetchIssue, listIssues } from "./github";
+import { detectRepo, ensureRepo, createWorktree } from "./worktree";
 import { runClaude } from "./sandbox";
 import { pushAndCreatePR } from "./github";
 import { log } from "./log";
@@ -54,7 +56,7 @@ async function init() {
     log.ok("Snapshot saved.");
   }
 
-  log.ok("mog is ready. Run: mog <owner/repo> <issue_number>");
+  log.ok("mog is ready. Run: mog <issue_number> (from a git repo) or mog <owner/repo> <issue_number>");
 }
 
 async function main() {
@@ -84,22 +86,99 @@ async function main() {
     return;
   }
 
-  if (args.length < 2) {
+  // mog list [--verbose]  or  mog <owner/repo> list [--verbose]
+  if (args[0] === "list" || args[1] === "list") {
+    let repo: string;
+    const verbose = args.includes("--verbose");
+
+    if (args[0] === "list") {
+      const detected = detectRepo();
+      if (!detected) {
+        log.die("Could not detect repo from git remote. Run from inside a git repo or use: mog <owner/repo> list");
+      }
+      repo = detected;
+    } else {
+      repo = args[0];
+    }
+
+    listIssues(repo, verbose);
+    return;
+  }
+
+  if (args.length < 1) {
     console.log("Usage:");
     console.log("  mog init                      — one-time setup (create sandbox & login)");
+    console.log("  mog <issue_num>               — auto-detect repo from git remote");
     console.log("  mog <owner/repo> <issue_num>  — fetch issue, run Claude, open PR");
+    console.log("  mog list [--verbose]           — list open issues (auto-detect repo)");
+    console.log("  mog <owner/repo> list [--verbose] — list open issues for a repo");
+    console.log();
+    console.log("Options:");
+    console.log("  --include <file>              — copy a file into the worktree (repeatable)");
     console.log();
     console.log("Example:");
     console.log("  mog init");
+    console.log("  mog 123");
+    console.log("  mog 123 --include .env");
     console.log("  mog workingdevshero/automate-it 123");
+    console.log("  mog list");
+    console.log("  mog list --verbose");
     return;
   }
 
-  const repo = args[0] as string;
-  const issueNum = args[1] as string;
+  // Parse --include flags
+  const includeFiles: string[] = [];
+  const filteredArgs: string[] = [];
+  for (let i = 0; i < args.length; i++) {
+    if (args[i] === "--include" && i + 1 < args.length) {
+      const filePath = path.resolve(args[i + 1]!);
+      if (!fs.existsSync(filePath)) {
+        log.die(`Include file not found: ${args[i + 1]}`);
+      }
+      includeFiles.push(filePath);
+      i++; // skip the path argument
+    } else {
+      filteredArgs.push(args[i]!);
+    }
+  }
 
-  if (!repo || !issueNum || !/^\d+$/.test(issueNum)) {
-    log.die(`Invalid issue number: '${issueNum}'. Must be a positive integer.`);
+  let repo: string;
+  let issueNum: string;
+
+  if (/^\d+$/.test(filteredArgs[0])) {
+    // mog <issue_number> — auto-detect repo
+    const detected = detectRepo();
+    if (!detected) {
+      log.die("Could not detect repo from git remote. Run from inside a git repo or use: mog <owner/repo> <issue_num>");
+    }
+    repo = detected;
+    issueNum = filteredArgs[0];
+  } else if (filteredArgs.length >= 2) {
+    // mog <owner/repo> <issue_number>
+    repo = filteredArgs[0];
+    issueNum = filteredArgs[1];
+    if (!/^\d+$/.test(issueNum)) {
+      log.die(`Invalid issue number: '${issueNum}'. Must be a positive integer.`);
+    }
+  } else {
+    console.log("Usage:");
+    console.log("  mog init                      — one-time setup (create sandbox & login)");
+    console.log("  mog <issue_num>               — auto-detect repo from git remote");
+    console.log("  mog <owner/repo> <issue_num>  — fetch issue, run Claude, open PR");
+    console.log("  mog list [--verbose]           — list open issues (auto-detect repo)");
+    console.log("  mog <owner/repo> list [--verbose] — list open issues for a repo");
+    console.log();
+    console.log("Options:");
+    console.log("  --include <file>              — copy a file into the worktree (repeatable)");
+    console.log();
+    console.log("Example:");
+    console.log("  mog init");
+    console.log("  mog 123");
+    console.log("  mog 123 --include .env");
+    console.log("  mog workingdevshero/automate-it 123");
+    console.log("  mog list");
+    console.log("  mog list --verbose");
+    return;
   }
 
   const parts = repo.split("/");
@@ -135,10 +214,21 @@ async function main() {
     reposDir, owner, repoName, defaultBranch, issueNum, issue.title
   );
 
+  // Copy included files into worktree
+  const copiedFiles: string[] = [];
+  for (const filePath of includeFiles) {
+    const basename = path.basename(filePath);
+    const dest = path.join(worktreeDir, basename);
+    fs.copyFileSync(filePath, dest);
+    copiedFiles.push(dest);
+    log.ok(`Included: ${basename}`);
+  }
+
   // Build prompts
   const planningPrompt = buildPlanningPrompt(repo, issueNum, issue);
   const buildingPromptFn = (remaining: string[], plan: string) =>
     buildBuildingPrompt(repo, issueNum, issue, remaining, plan);
+  const reviewPrompt = buildReviewPrompt(repo, issueNum, issue);
 
   // Run Claude in sandbox
   log.info("Launching Claude Code in sandbox...");
@@ -146,7 +236,18 @@ async function main() {
   log.info(`Worktree: ${worktreeDir}`);
   console.log();
 
-  const summary = await runClaude(SANDBOX_NAME, worktreeDir, planningPrompt, buildingPromptFn);
+  const summary = await runClaude(SANDBOX_NAME, worktreeDir, planningPrompt, buildingPromptFn, reviewPrompt);
+
+  // Remove included files so they don't end up in the PR
+  for (const filePath of copiedFiles) {
+    try {
+      fs.unlinkSync(filePath);
+      // Unstage if Claude happened to git add it
+      Bun.spawnSync(["git", "rm", "--cached", "--ignore-unmatch", path.basename(filePath)], { cwd: worktreeDir });
+    } catch {
+      // File may already be gone
+    }
+  }
 
   // Push and create PR
   pushAndCreatePR(repo, worktreeDir, branchName, defaultBranch, issueNum, issue, summary);
@@ -201,29 +302,45 @@ function tryRecoverSandbox(reposDir: string): boolean {
   return true;
 }
 
-function buildPlanningPrompt(repo: string, issueNum: string, issue: { title: string; body: string; labels: string }): string {
-  return `You are working on GitHub issue #${issueNum} for the repository ${repo}.
-
-## Issue: ${issue.title}
+function formatIssueContext(issueNum: string, issue: { title: string; body: string; labels: string; comments: string }): string {
+  let context = `## Issue: ${issue.title}
 
 ### Description
 ${issue.body}
 
 ### Labels
-${issue.labels}
+${issue.labels}`;
+
+  if (issue.comments) {
+    context += `
+
+### Comments
+${issue.comments}`;
+  }
+
+  return context;
+}
+
+function buildPlanningPrompt(repo: string, issueNum: string, issue: { title: string; body: string; labels: string; comments: string }): string {
+  return `You are working on GitHub issue #${issueNum} for the repository ${repo}.
+
+${formatIssueContext(issueNum, issue)}
 
 ## Instructions
 
 Your job in this step is to **plan only** — do NOT implement anything and do NOT commit.
 
 1. Read and understand the codebase structure thoroughly.
-2. Analyze the issue and break it down into small, atomic implementation tasks.
-3. Create a file called \`IMPLEMENTATION_PLAN.md\` in the root of the repository with a checklist of tasks.
+2. **Search the entire codebase** for code related to the issue — look for similar patterns, duplicate logic, and any modules that handle the same concern. Use Grep and Glob liberally to find all relevant locations, not just the most obvious one.
+3. Analyze the issue and break it down into small, atomic implementation tasks.
+4. Create a file called \`IMPLEMENTATION_PLAN.md\` in the root of the repository with a checklist of tasks.
 
 The plan should:
 - Have 3-8 tasks (fewer for simple issues, more for complex ones)
 - Order tasks by dependency (implement foundations first)
 - Each task should be a single, atomic unit of work that results in one commit
+- **Include tasks to update ALL locations** where the same pattern or concern exists — not just the most obvious one. If the same logic appears in multiple modules, the plan must cover all of them.
+- If you find duplicate or near-duplicate logic across modules, include a task to consolidate it into a shared utility or function.
 - Use markdown checklist format: \`- [ ] Task description\`
 
 Example format:
@@ -242,7 +359,7 @@ Do NOT implement any code changes. Do NOT make any commits. Only create the plan
 function buildBuildingPrompt(
   repo: string,
   issueNum: string,
-  issue: { title: string; body: string; labels: string },
+  issue: { title: string; body: string; labels: string; comments: string },
   remainingItems: string[],
   planContent: string,
 ): string {
@@ -250,19 +367,13 @@ function buildBuildingPrompt(
   if (remainingItems.length === 0 && !planContent) {
     return `You are working on GitHub issue #${issueNum} for the repository ${repo}.
 
-## Issue: ${issue.title}
-
-### Description
-${issue.body}
-
-### Labels
-${issue.labels}
+${formatIssueContext(issueNum, issue)}
 
 ## Instructions
 1. Read and understand the codebase structure first.
 2. Implement the changes described in the issue above.
 3. Write clean, well-documented code that follows the existing project conventions.
-4. Add or update tests if applicable.
+4. If the project has an existing test suite, add or update tests to cover the changes.
 5. Make sure the code builds/lints without errors if there's a build system.
 6. Commit your changes with a clear commit message referencing issue #${issueNum}.
 
@@ -274,13 +385,7 @@ a message like: "fix: <short description> (#${issueNum})"`;
 
   return `You are working on GitHub issue #${issueNum} for the repository ${repo}.
 
-## Issue: ${issue.title}
-
-### Description
-${issue.body}
-
-### Labels
-${issue.labels}
+${formatIssueContext(issueNum, issue)}
 
 ## Current Implementation Plan
 
@@ -299,6 +404,31 @@ Rules:
 5. Do NOT work on any other tasks after committing.`;
 }
 
+function buildReviewPrompt(
+  repo: string,
+  issueNum: string,
+  issue: { title: string; body: string; labels: string; comments: string },
+): string {
+  return `You are reviewing changes made for GitHub issue #${issueNum} in the repository ${repo}.
+
+${formatIssueContext(issueNum, issue)}
+
+## Instructions
+
+All implementation tasks are complete. Your job is to **review the entire branch** for quality and completeness.
+
+Run \`git diff main...HEAD\` (or the equivalent for the default branch) to see all changes made.
+
+Check for:
+1. **Missed locations**: Search the codebase for similar patterns, logic, or code that handles the same concern as the changes. If the fix or feature was applied in one place but a similar pattern exists elsewhere, apply it there too.
+2. **Code duplication**: If the changes introduced logic that duplicates existing code (or if pre-existing duplication was missed), consolidate it into a shared function or utility.
+3. **Quality issues**: Look for missing edge cases, error handling gaps, or inconsistencies with the rest of the codebase.
+4. **Tests**: If the project has an existing test suite, verify that tests were added or updated to cover the changes. If not, add them. Do not create a test framework or test infrastructure from scratch.
+
+If you find issues, fix them and commit each fix separately with a clear commit message referencing #${issueNum}.
+If everything looks good, do nothing.`;
+}
+
 main().catch((err) => {
   log.die(err.message);
 });

package/src/sandbox.ts

@@ -76,6 +76,7 @@ export async function runClaude(
   worktreeDir: string,
   planningPrompt: string,
   buildingPromptFn: (remainingItems: string[], planContent: string) => string,
+  reviewPrompt?: string,
 ): Promise<string> {
   let lastResult = "";
 
@@ -152,7 +153,14 @@ export async function runClaude(
     }
   }
 
-  // Phase 3 — Cleanup
+  // Phase 3 — Review
+  if (reviewPrompt) {
+    log.info("Phase 3: Reviewing changes for quality and completeness...");
+    const reviewResult = await execClaude(sandboxName, worktreeDir, ["-p", reviewPrompt]);
+    if (reviewResult) lastResult = reviewResult;
+  }
+
+  // Phase 4 — Cleanup
   cleanupPlanFile(sandboxName, worktreeDir);
 
   const finalPlan = readPlanFile(worktreeDir);

package/src/worktree.ts

@@ -1,6 +1,29 @@
 import fs from "fs";
 import { log } from "./log";
 
+export function detectRepo(): string | null {
+  const result = Bun.spawnSync(["git", "remote", "get-url", "origin"]);
+  if (result.exitCode !== 0) {
+    return null;
+  }
+
+  const url = result.stdout.toString().trim();
+
+  // SSH: git@github.com:owner/repo.git
+  const sshMatch = url.match(/^git@[^:]+:([^/]+)\/([^/]+?)(?:\.git)?$/);
+  if (sshMatch) {
+    return `${sshMatch[1]}/${sshMatch[2]}`;
+  }
+
+  // HTTPS: https://github.com/owner/repo.git
+  const httpsMatch = url.match(/^https?:\/\/[^/]+\/([^/]+)\/([^/]+?)(?:\.git)?$/);
+  if (httpsMatch) {
+    return `${httpsMatch[1]}/${httpsMatch[2]}`;
+  }
+
+  return null;
+}
+
 export function ensureRepo(
   repo: string,
   owner: string,