@iloom/cli 0.4.1 → 0.5.1

package/dist/cli.js

@@ -1,98 +1,107 @@
 #!/usr/bin/env node
 import {
   SessionSummaryService
-} from "./chunk-TMZAVPGF.js";
+} from "./chunk-HABINPX2.js";
+import {
+  FirstRunManager,
+  IssueTrackerFactory,
+  generateIssueManagementMcpConfig
+} from "./chunk-TSLKDFAF.js";
+import "./chunk-QHA67Q7A.js";
 import {
   CLIIsolationManager,
   DatabaseManager,
   EnvironmentManager,
   LoomManager,
   ResourceCleanup
-} from "./chunk-IARWMDAX.js";
+} from "./chunk-6KB7R22U.js";
+import {
+  detectPackageManager,
+  installDependencies,
+  runScript
+} from "./chunk-VBFDVGAE.js";
+import {
+  ProcessManager
+} from "./chunk-VU3QMIP2.js";
+import {
+  IdentifierParser
+} from "./chunk-AS2IRKLU.js";
+import {
+  createNeonProviderFromSettings
+} from "./chunk-UNXRACJ7.js";
 import {
   InitCommand,
   ShellCompletion
-} from "./chunk-G6CIIJLT.js";
+} from "./chunk-4YTILIIH.js";
 import "./chunk-UYWAESOT.js";
 import {
   IssueEnhancementService,
   capitalizeFirstLetter
-} from "./chunk-VTXCGKV5.js";
+} from "./chunk-BVIK2P6P.js";
 import {
-  MergeManager
-} from "./chunk-BIIQHEXJ.js";
+  AgentManager
+} from "./chunk-RNZMHJK7.js";
 import {
-  FirstRunManager,
-  IssueTrackerFactory,
-  generateIssueManagementMcpConfig
-} from "./chunk-HD5SUKI2.js";
-import "./chunk-QHA67Q7A.js";
+  ProjectCapabilityDetector
+} from "./chunk-EBISESAP.js";
 import {
-  AgentManager
-} from "./chunk-O7WHXLCB.js";
+  hasScript,
+  readPackageJson
+} from "./chunk-2ZPFJQ3B.js";
 import {
-  IdentifierParser
-} from "./chunk-55TB3FSG.js";
+  MergeManager
+} from "./chunk-GVRO4PWE.js";
 import {
-  ProcessManager
-} from "./chunk-VU3QMIP2.js";
+  GitWorktreeManager
+} from "./chunk-3CMGCRB5.js";
+import {
+  PRManager
+} from "./chunk-P2ZQ5LKB.js";
 import {
   openBrowser
 } from "./chunk-YETJNRQM.js";
 import {
-  GitWorktreeManager
-} from "./chunk-JC5HXN75.js";
+  getConfiguredRepoFromSettings,
+  hasMultipleRemotes
+} from "./chunk-PSFVTBM7.js";
 import {
-  detectPackageManager,
-  installDependencies,
-  runScript
-} from "./chunk-VBFDVGAE.js";
+  getIdeConfig,
+  getInstallHint,
+  isIdeAvailable
+} from "./chunk-CDF7ZX2B.js";
 import {
   ClaudeContextManager
-} from "./chunk-UPUAQYAW.js";
-import "./chunk-POI7KLBH.js";
+} from "./chunk-S65T4O6I.js";
+import "./chunk-CDQEK2WD.js";
+import "./chunk-DKQ4SUII.js";
 import {
   extractSettingsOverrides
 } from "./chunk-GYCR2LOU.js";
 import {
   DefaultBranchNamingService
 } from "./chunk-QIUJPPJQ.js";
-import {
-  ProjectCapabilityDetector
-} from "./chunk-EBISESAP.js";
-import {
-  hasScript,
-  readPackageJson
-} from "./chunk-2ZPFJQ3B.js";
-import {
-  createNeonProviderFromSettings
-} from "./chunk-UNXRACJ7.js";
-import {
-  getConfiguredRepoFromSettings,
-  getEffectivePRTargetRemote,
-  hasMultipleRemotes,
-  parseGitRemotes
-} from "./chunk-PSFVTBM7.js";
 import {
   executeGitCommand,
   extractIssueNumber,
   findMainWorktreePathWithSettings,
+  findPlaceholderCommitSha,
   getMergeTargetBranch,
   getRepoRoot,
-  pushBranchToRemote
-} from "./chunk-2W2FBL5G.js";
-import {
-  MetadataManager
-} from "./chunk-IJ7IGJT3.js";
+  isPlaceholderCommit,
+  pushBranchToRemote,
+  removePlaceholderCommitFromHead,
+  removePlaceholderCommitFromHistory
+} from "./chunk-LN4H3A6A.js";
 import {
   SettingsManager
-} from "./chunk-VWNS6DH5.js";
+} from "./chunk-OOU3DKNT.js";
+import {
+  MetadataManager
+} from "./chunk-YZTDGPFB.js";
 import {
   GitHubService
 } from "./chunk-OEGECBFS.js";
-import {
-  executeGhCommand
-} from "./chunk-KO2FOMHL.js";
+import "./chunk-KO2FOMHL.js";
 import {
   promptCommitAction,
   promptConfirmation,
@@ -111,7 +120,6 @@ import {
   getLogger,
   withLogger
 } from "./chunk-6UIGZD2N.js";
-import "./chunk-74VMN2KC.js";
 import {
   createStderrLogger,
   logger
@@ -605,7 +613,11 @@ var AddIssueCommand = class {
         getLogger().info(`Using GitHub repository: ${repo}`);
       }
     }
-    if (!description || !this.enhancementService.validateDescription(description)) {
+    const hasBody = !!body;
+    if (!description || !this.enhancementService.validateDescription(description, hasBody)) {
+      if (hasBody) {
+        throw new Error("Description is required and cannot be empty");
+      }
       throw new Error("Description is required and must be more than 30 characters with at least 3 words");
     }
     const issueBody = body ?? await this.enhancementService.enhanceDescription(description);
@@ -1664,267 +1676,6 @@ Run '${runCommand}' to see detailed errors.`
   }
 };
 
-// src/lib/PRManager.ts
-var PRManager = class {
-  constructor(settings) {
-    this.settings = settings;
-  }
-  /**
-   * Check if a PR already exists for the given branch
-   * @param branchName - Branch to check
-   * @param cwd - Working directory
-   * @returns Existing PR info or null if none found
-   */
-  async checkForExistingPR(branchName, cwd) {
-    try {
-      const prList = await executeGhCommand(
-        ["pr", "list", "--head", branchName, "--state", "open", "--json", "number,url"],
-        cwd ? { cwd } : void 0
-      );
-      if (prList.length > 0) {
-        return prList[0] ?? null;
-      }
-      return null;
-    } catch (error) {
-      getLogger().debug("Error checking for existing PR", { error });
-      return null;
-    }
-  }
-  /**
-   * Generate PR body using Claude if available, otherwise use simple template
-   * @param issueNumber - Issue number to include in body
-   * @param worktreePath - Path to worktree for context
-   * @returns PR body markdown
-   */
-  async generatePRBody(issueNumber, worktreePath) {
-    const hasClaudeCli = await detectClaudeCli();
-    if (hasClaudeCli) {
-      try {
-        const prompt = this.buildPRBodyPrompt(issueNumber);
-        const body2 = await launchClaude(prompt, {
-          headless: true,
-          addDir: worktreePath,
-          timeout: 3e4
-        });
-        if (body2 && typeof body2 === "string" && body2.trim()) {
-          const sanitized = this.sanitizeClaudeOutput(body2);
-          if (sanitized) {
-            return sanitized;
-          }
-        }
-      } catch (error) {
-        getLogger().debug("Claude PR body generation failed, using template", { error });
-      }
-    }
-    let body = "This PR contains changes from the iloom workflow.\n\n";
-    if (issueNumber) {
-      body += `Fixes #${issueNumber}`;
-    }
-    return body;
-  }
-  /**
-   * Build structured XML prompt for PR body generation
-   * Uses XML format for clear task definition and output expectations
-   */
-  buildPRBodyPrompt(issueNumber) {
-    const issueContext = issueNumber ? `
-<IssueContext>
-This PR is associated with GitHub issue #${issueNumber}.
-Include "Fixes #${issueNumber}" at the end of the body on its own line.
-</IssueContext>` : "";
-    return `<Task>
-You are a software engineer writing a pull request body for this repository.
-Examine the changes in the git repository and generate a concise, professional PR description.
-</Task>
-
-<Requirements>
-<Format>Write 2-3 sentences summarizing what was changed and why.${issueNumber ? `
-
-End with "Fixes #${issueNumber}" on its own line.` : ""}</Format>
-<Tone>Professional and concise</Tone>
-<Focus>Summarize the changes and their purpose</Focus>
-<NoMeta>CRITICAL: Do NOT include ANY explanatory text, analysis, or meta-commentary. Output ONLY the raw PR body text.</NoMeta>
-<Examples>
-Good: "Add user authentication with JWT tokens to secure the API endpoints. This includes login and registration endpoints with proper password hashing.
-
-Fixes #42"
-Good: "Fix navigation bug in sidebar menu that caused incorrect highlighting on nested routes."
-Bad: "Here's the PR body:
-
----
-
-Add user authentication..."
-Bad: "Based on the changes, I'll write: Fix navigation bug..."
-</Examples>
-${issueContext}
-</Requirements>
-
-<Output>
-IMPORTANT: Your entire response will be used directly as the GitHub pull request body.
-Do not include any explanatory text, headers, or separators before or after the body.
-Start your response immediately with the PR body text.
-</Output>`;
-  }
-  /**
-   * Sanitize Claude output to remove meta-commentary and clean formatting
-   * Handles cases where Claude includes explanatory text despite instructions
-   */
-  sanitizeClaudeOutput(rawOutput) {
-    let cleaned = rawOutput.trim();
-    const metaPatterns = [
-      /^.*?based on.*?changes.*?:/i,
-      /^.*?looking at.*?files.*?:/i,
-      /^.*?examining.*?:/i,
-      /^.*?analyzing.*?:/i,
-      /^.*?i'll.*?generate.*?:/i,
-      /^.*?let me.*?:/i,
-      /^.*?here.*?is.*?(?:the\s+)?(?:pr|pull request).*?body.*?:/i,
-      /^.*?here's.*?(?:the\s+)?(?:pr|pull request).*?body.*?:/i
-    ];
-    for (const pattern of metaPatterns) {
-      cleaned = cleaned.replace(pattern, "").trim();
-    }
-    cleaned = cleaned.replace(/^[-=]{3,}\s*/m, "").trim();
-    if (cleaned.includes(":")) {
-      const colonIndex = cleaned.indexOf(":");
-      const beforeColon = cleaned.substring(0, colonIndex).trim().toLowerCase();
-      const metaIndicators = [
-        "here is the pr body",
-        "here is the pull request body",
-        "pr body",
-        "pull request body",
-        "here is",
-        "here's",
-        "the body should be",
-        "i suggest",
-        "my suggestion"
-      ];
-      const isMetaCommentary = metaIndicators.some((indicator) => beforeColon.includes(indicator));
-      if (isMetaCommentary) {
-        const afterColon = cleaned.substring(colonIndex + 1).trim();
-        const afterSeparator = afterColon.replace(/^[-=]{3,}\s*/m, "").trim();
-        if (afterSeparator && afterSeparator.length > 10) {
-          cleaned = afterSeparator;
-        }
-      }
-    }
-    if (cleaned.startsWith('"') && cleaned.endsWith('"') || cleaned.startsWith("'") && cleaned.endsWith("'")) {
-      cleaned = cleaned.slice(1, -1).trim();
-    }
-    return cleaned;
-  }
-  /**
-   * Create a GitHub PR for the branch
-   * @param branchName - Branch to create PR from (used as --head)
-   * @param title - PR title
-   * @param body - PR body
-   * @param baseBranch - Base branch to target (usually main/master)
-   * @param cwd - Working directory
-   * @returns PR URL
-   */
-  async createPR(branchName, title, body, baseBranch, cwd) {
-    try {
-      const targetRemote = await getEffectivePRTargetRemote(this.settings, cwd);
-      let headValue = branchName;
-      if (targetRemote !== "origin") {
-        const remotes = await parseGitRemotes(cwd);
-        const originRemote = remotes.find((r) => r.name === "origin");
-        if (originRemote) {
-          headValue = `${originRemote.owner}:${branchName}`;
-          getLogger().debug(`Fork workflow detected, using head: ${headValue}`);
-        }
-      }
-      const args = ["pr", "create", "--head", headValue, "--title", title, "--body", body, "--base", baseBranch];
-      if (targetRemote !== "origin") {
-        const repo = await getConfiguredRepoFromSettings(this.settings, cwd);
-        args.push("--repo", repo);
-      }
-      const result = await executeGhCommand(args, cwd ? { cwd } : void 0);
-      const url = typeof result === "string" ? result.trim() : String(result).trim();
-      if (!url.includes("github.com") || !url.includes("/pull/")) {
-        throw new Error(`Unexpected response from gh pr create: ${url}`);
-      }
-      return url;
-    } catch (error) {
-      const errorMessage = error instanceof Error ? error.message : String(error);
-      if (errorMessage.includes("Head sha can't be blank") || errorMessage.includes("No commits between")) {
-        throw new Error(
-          `Failed to create pull request: ${errorMessage}
-
-This error typically occurs when:
-  - The branch was not fully pushed to the remote
-  - There's a race condition between push and PR creation
-  - The branch has no commits ahead of the base branch
-
-Try running: git push -u origin ${branchName}
-Then retry: il finish`
-        );
-      }
-      throw new Error(`Failed to create pull request: ${errorMessage}`);
-    }
-  }
-  /**
-   * Open PR URL in browser
-   * @param url - PR URL to open
-   */
-  async openPRInBrowser(url) {
-    try {
-      await openBrowser(url);
-      getLogger().debug("Opened PR in browser", { url });
-    } catch (error) {
-      getLogger().warn("Failed to open PR in browser", { error });
-    }
-  }
-  /**
-   * Complete PR workflow: check for existing, create if needed, optionally open in browser
-   * @param branchName - Branch to create PR from
-   * @param title - PR title
-   * @param issueNumber - Optional issue number for body generation
-   * @param baseBranch - Base branch to target
-   * @param worktreePath - Path to worktree
-   * @param openInBrowser - Whether to open PR in browser
-   * @returns PR creation result
-   */
-  async createOrOpenPR(branchName, title, issueNumber, baseBranch, worktreePath, openInBrowser) {
-    const existingPR = await this.checkForExistingPR(branchName, worktreePath);
-    if (existingPR) {
-      getLogger().info(`Pull request already exists: ${existingPR.url}`);
-      if (openInBrowser) {
-        await this.openPRInBrowser(existingPR.url);
-      }
-      return {
-        url: existingPR.url,
-        number: existingPR.number,
-        wasExisting: true
-      };
-    }
-    const body = await this.generatePRBody(issueNumber, worktreePath);
-    getLogger().info("Creating pull request...");
-    const url = await this.createPR(branchName, title, body, baseBranch, worktreePath);
-    const prNumber = this.extractPRNumberFromUrl(url);
-    if (openInBrowser) {
-      await this.openPRInBrowser(url);
-    }
-    return {
-      url,
-      number: prNumber,
-      wasExisting: false
-    };
-  }
-  /**
-   * Extract PR number from GitHub PR URL
-   * @param url - PR URL (e.g., https://github.com/owner/repo/pull/123)
-   * @returns PR number
-   */
-  extractPRNumberFromUrl(url) {
-    const match = url.match(/\/pull\/(\d+)/);
-    if (match == null ? void 0 : match[1]) {
-      return parseInt(match[1], 10);
-    }
-    throw new Error(`Could not extract PR number from URL: ${url}`);
-  }
-};
-
 // src/commands/finish.ts
 import path3 from "path";
 var FinishCommand = class {
@@ -2302,10 +2053,22 @@ var FinishCommand = class {
   }
   /**
    * Execute workflow for issues and branches (merge into main)
-   * This is the traditional workflow: validate → commit → rebase → merge → cleanup
+   * This is the workflow: rebase → validate → commit → merge → cleanup
    */
   async executeIssueWorkflow(parsed, options, worktree, result) {
-    var _a, _b;
+    var _a, _b, _c;
+    getLogger().info("Rebasing branch on main...");
+    const mergeOptions = {
+      dryRun: options.dryRun ?? false,
+      force: options.force ?? false
+    };
+    await this.mergeManager.rebaseOnMain(worktree.path, mergeOptions);
+    getLogger().success("Branch rebased successfully");
+    result.operations.push({
+      type: "rebase",
+      message: "Branch rebased on main",
+      success: true
+    });
     if (!options.dryRun) {
       getLogger().info("Running pre-merge validations...");
       await this.validationRunner.runValidations(worktree.path, {
@@ -2380,18 +2143,92 @@ var FinishCommand = class {
       await this.executeGitHubPRWorkflow(parsed, options, worktree, settings, result);
       return;
     }
-    getLogger().info("Rebasing branch on main...");
-    const mergeOptions = {
-      dryRun: options.dryRun ?? false,
-      force: options.force ?? false
-    };
-    await this.mergeManager.rebaseOnMain(worktree.path, mergeOptions);
-    getLogger().success("Branch rebased successfully");
-    result.operations.push({
-      type: "rebase",
-      message: "Branch rebased on main",
-      success: true
-    });
+    if (mergeBehavior.mode === "github-draft-pr") {
+      if (!this.issueTracker.supportsPullRequests) {
+        throw new Error(
+          `The 'github-draft-pr' merge mode requires a GitHub-compatible issue tracker. Your provider (${this.issueTracker.providerName}) does not support pull requests.`
+        );
+      }
+      const { MetadataManager: MetadataManager2 } = await import("./MetadataManager-WXUVXKUS.js");
+      const metadataManager = new MetadataManager2();
+      const metadata = await metadataManager.readMetadata(worktree.path);
+      getLogger().debug(`Draft PR mode: worktree=${worktree.path}, draftPrNumber=${(metadata == null ? void 0 : metadata.draftPrNumber) ?? "none"}`);
+      if (!(metadata == null ? void 0 : metadata.draftPrNumber)) {
+        getLogger().warn("No draft PR found in metadata, creating new PR...");
+        await this.executeGitHubPRWorkflow(parsed, options, worktree, settings, result);
+        return;
+      }
+      const isHeadPlaceholder = await isPlaceholderCommit(worktree.path);
+      const placeholderSha = await findPlaceholderCommitSha(worktree.path);
+      getLogger().debug(`Placeholder detection: isHead=${isHeadPlaceholder}, sha=${placeholderSha ?? "none"}`);
+      if (isHeadPlaceholder) {
+        const commitCount = await executeGitCommand(
+          ["rev-list", "--count", "HEAD"],
+          { cwd: worktree.path }
+        );
+        if (parseInt(commitCount.trim(), 10) <= 1) {
+          throw new Error(
+            "Cannot finish draft PR: no changes have been committed.\nPlease make at least one commit before finishing."
+          );
+        }
+        if (!options.dryRun) {
+          getLogger().info("Removing placeholder commit from HEAD...");
+          await removePlaceholderCommitFromHead(worktree.path);
+        } else {
+          getLogger().info("[DRY RUN] Would remove placeholder commit from HEAD");
+        }
+      } else if (placeholderSha) {
+        const commitsAfterPlaceholder = await executeGitCommand(
+          ["rev-list", "--count", `${placeholderSha}..HEAD`],
+          { cwd: worktree.path }
+        );
+        if (parseInt(commitsAfterPlaceholder.trim(), 10) === 0) {
+          throw new Error(
+            "Cannot finish draft PR: no changes have been committed after the placeholder.\nPlease make at least one commit before finishing."
+          );
+        }
+        if (!options.dryRun) {
+          getLogger().info("Removing placeholder commit from history...");
+          await removePlaceholderCommitFromHistory(worktree.path, placeholderSha);
+        } else {
+          getLogger().info("[DRY RUN] Would remove placeholder commit from history");
+        }
+      }
+      const needsForceWithLease = isHeadPlaceholder || placeholderSha;
+      if (!options.dryRun) {
+        getLogger().info("Pushing final commits to remote...");
+        if (needsForceWithLease) {
+          await executeGitCommand(["push", "--force-with-lease", "origin", worktree.branch], { cwd: worktree.path });
+        } else {
+          await pushBranchToRemote(worktree.branch, worktree.path, { dryRun: false });
+        }
+      } else {
+        if (needsForceWithLease) {
+          getLogger().info("[DRY RUN] Would force push final commits to remote (history rewritten)");
+        } else {
+          getLogger().info("[DRY RUN] Would push final commits to remote");
+        }
+      }
+      const prManager = new PRManager(settings);
+      if (!options.dryRun) {
+        await prManager.markPRReady(metadata.draftPrNumber, worktree.path);
+        getLogger().success(`PR #${metadata.draftPrNumber} marked as ready for review`);
+      } else {
+        getLogger().info(`[DRY RUN] Would mark PR #${metadata.draftPrNumber} as ready for review`);
+      }
+      const prUrl = (_c = metadata.prUrls) == null ? void 0 : _c[String(metadata.draftPrNumber)];
+      if (prUrl) {
+        result.prUrl = prUrl;
+      }
+      result.operations.push({
+        type: "pr-ready",
+        message: `PR #${metadata.draftPrNumber} marked as ready for review`,
+        success: true
+      });
+      await this.generateSessionSummaryIfConfigured(parsed, worktree, options, metadata.draftPrNumber);
+      await this.handlePRCleanupPrompt(parsed, options, worktree, result);
+      return;
+    }
     getLogger().info("Performing fast-forward merge...");
     await this.mergeManager.performFastForwardMerge(worktree.branch, worktree.path, mergeOptions);
     getLogger().success("Fast-forward merge completed successfully");
@@ -2555,7 +2392,7 @@ var FinishCommand = class {
         });
       }
       finishResult.prUrl = prResult.url;
-      await this.generateSessionSummaryIfConfigured(parsed, worktree, options);
+      await this.generateSessionSummaryIfConfigured(parsed, worktree, options, prResult.number);
       await this.handlePRCleanupPrompt(parsed, options, worktree, finishResult);
     }
   }
@@ -2578,8 +2415,8 @@ var FinishCommand = class {
       getLogger().info("");
       const shouldCleanup = await promptConfirmation(
         "Clean up worktree now?",
-        false
-        // Default to keeping worktree (safer option)
+        true
+        // Default to keeping worktree - won't delete if unmerged changes
       );
       if (shouldCleanup) {
         await this.performWorktreeCleanup(parsed, options, worktree, finishResult);
@@ -2720,8 +2557,13 @@ var FinishCommand = class {
    * This ensures the finish workflow continues even if summary generation fails
    *
    * In dry-run mode: generates summary and shows preview, but doesn't post
+   *
+   * @param parsed - The parsed input identifying the issue/PR being finished
+   * @param worktree - The worktree being finished
+   * @param options - Finish options (including dryRun flag)
+   * @param prNumber - Optional PR number - when provided, summary is posted to the PR instead of the issue
    */
-  async generateSessionSummaryIfConfigured(parsed, worktree, options) {
+  async generateSessionSummaryIfConfigured(parsed, worktree, options, prNumber) {
     if (parsed.type === "branch") {
       return;
     }
@@ -2752,7 +2594,8 @@ var FinishCommand = class {
       worktreePath: worktree.path,
       issueNumber: parsed.number ?? 0,
       branchName: worktree.branch,
-      loomType: parsed.type
+      loomType: parsed.type,
+      ...prNumber !== void 0 && { prNumber }
     });
   }
   /**
@@ -2992,7 +2835,9 @@ function formatLoomForJson(worktree, mainWorktreePath, metadata) {
     created_at: (metadata == null ? void 0 : metadata.created_at) ?? null,
     issueTracker: (metadata == null ? void 0 : metadata.issueTracker) ?? null,
     colorHex: (metadata == null ? void 0 : metadata.colorHex) ?? null,
-    projectPath: (metadata == null ? void 0 : metadata.projectPath) ?? null
+    projectPath: (metadata == null ? void 0 : metadata.projectPath) ?? null,
+    issueUrls: (metadata == null ? void 0 : metadata.issueUrls) ?? {},
+    prUrls: (metadata == null ? void 0 : metadata.prUrls) ?? {}
   };
 }
 function formatLoomsForJson(worktrees, mainWorktreePath, metadata) {
@@ -3006,7 +2851,7 @@ function parseIssueIdentifier(value) {
   const parsed = parseInt(value, 10);
   return !isNaN(parsed) && String(parsed) === value ? parsed : value;
 }
-program.name("iloom").description(packageJson.description).version(packageJson.version).option("--debug", "Enable debug output (default: based on ILOOM_DEBUG env var)").option("--completion", "Output shell completion script for current shell").option("--set <key=value>", "Override any setting using dot notation (repeatable, e.g., --set workflows.issue.startIde=false)").allowUnknownOption().hook("preAction", async (thisCommand) => {
+program.name("iloom").description(packageJson.description).version(packageJson.version).option("--debug", "Enable debug output (default: based on ILOOM_DEBUG env var)").option("--completion", "Output shell completion script for current shell").option("--set <key=value>", "Override any setting using dot notation (repeatable, e.g., --set workflows.issue.startIde=false)").allowUnknownOption().hook("preAction", async (thisCommand, actionCommand) => {
   const options = thisCommand.opts();
   const envDebug = process.env.ILOOM_DEBUG === "true";
   const debugEnabled = options.debug !== void 0 ? options.debug : envDebug;
@@ -3030,16 +2875,18 @@ program.name("iloom").description(packageJson.description).version(packageJson.v
   } catch (error) {
     logger.debug(`Settings migration failed: ${error instanceof Error ? error.message : "Unknown"}`);
   }
-  await validateSettingsForCommand(thisCommand);
-  await validateGhCliForCommand(thisCommand);
+  await validateSettingsForCommand(actionCommand);
+  await validateGhCliForCommand(actionCommand);
+  await validateIdeForStartCommand(thisCommand);
 });
 async function validateSettingsForCommand(command) {
   var _a, _b;
-  const commandName = command.args[0] ?? "";
+  const commandName = command.name();
   const bypassCommands = ["help", "init", "update", "contribute"];
   if (bypassCommands.includes(commandName)) {
     return;
   }
+  const warnOnlyCommands = ["list", "projects"];
   try {
     const settingsManager = new SettingsManager();
     const settings = await settingsManager.loadSettings();
@@ -3049,6 +2896,10 @@ async function validateSettingsForCommand(command) {
       return;
     }
   } catch (error) {
+    if (warnOnlyCommands.includes(commandName)) {
+      logger.warn(`Configuration warning: ${error instanceof Error ? error.message : "Unknown error"}`);
+      return;
+    }
     logger.error(`Configuration error: ${error instanceof Error ? error.message : "Unknown error"}`);
     logger.info("Please fix your .iloom/settings.json file and try again.");
     process.exit(1);
@@ -3056,7 +2907,7 @@ async function validateSettingsForCommand(command) {
 }
 async function validateGhCliForCommand(command) {
   var _a, _b;
-  const commandName = command.args[0] ?? "";
+  const commandName = command.name();
   const alwaysRequireGh = ["feedback", "contribute"];
   const conditionallyRequireGh = ["start", "finish", "enhance", "add-issue", "ignite", "spin"];
   const warnOnly = ["init", "list", "rebase", "cleanup", "run", "update", "open"];
@@ -3107,6 +2958,41 @@ async function validateGhCliForCommand(command) {
     }
   }
 }
+async function validateIdeForStartCommand(command) {
+  var _a, _b;
+  const commandName = command.args[0] ?? "";
+  if (commandName !== "start") {
+    return;
+  }
+  const codeOption = command.opts()["code"];
+  if (codeOption === false) {
+    return;
+  }
+  const settingsManager = new SettingsManager();
+  let settings;
+  try {
+    settings = await settingsManager.loadSettings();
+  } catch {
+    return;
+  }
+  const workflowConfig = (_a = settings.workflows) == null ? void 0 : _a.issue;
+  if ((workflowConfig == null ? void 0 : workflowConfig.startIde) === false && codeOption !== true) {
+    return;
+  }
+  const ideConfig = getIdeConfig(settings.ide);
+  const available = await isIdeAvailable(ideConfig.command);
+  if (!available) {
+    const hint = getInstallHint(((_b = settings.ide) == null ? void 0 : _b.type) ?? "vscode");
+    logger.error(
+      `${ideConfig.name} is configured as your IDE but "${ideConfig.command}" command was not found.`
+    );
+    logger.info("");
+    logger.info(hint);
+    logger.info("");
+    logger.info("Alternatively, use --no-code to skip IDE launch or configure a different IDE in settings.");
+    process.exit(1);
+  }
+}
 async function autoLaunchInitForMultipleRemotes() {
   var _a, _b;
   logger.info("Multiple git remotes detected, but no GitHub remote is configured.");
@@ -3118,14 +3004,14 @@ async function autoLaunchInitForMultipleRemotes() {
   await waitForKeypress2("Press any key to start configuration...");
   logger.info("");
   try {
-    const { InitCommand: InitCommand2 } = await import("./init-LBA6NUK2.js");
+    const { InitCommand: InitCommand2 } = await import("./init-F6PFMSU5.js");
     const initCommand = new InitCommand2();
     const customInitialMessage = "Help me configure which git remote iloom should use for GitHub operations. I have multiple remotes and need to select the correct one.";
     await initCommand.execute(customInitialMessage);
     logger.info("");
     logger.info("Configuration complete! Continuing with your original command...");
     logger.info("");
-    const { SettingsManager: SettingsManager2 } = await import("./SettingsManager-FJFU6JJD.js");
+    const { SettingsManager: SettingsManager2 } = await import("./SettingsManager-XPR4TEQL.js");
     const settingsManager = new SettingsManager2();
     const settings = await settingsManager.loadSettings();
     const { hasMultipleRemotes: hasMultipleRemotes2 } = await import("./remote-73TZ2ADI.js");
@@ -3183,7 +3069,7 @@ program.command("start").alias("new").alias("create").alias("up").description("C
     await executeAction();
   }
 });
-program.command("add-issue").alias("a").description("Create and enhance GitHub issue without starting workspace").argument("<description>", "Natural language description of the issue (>50 chars, >2 spaces)").option("--body <text>", "Body text for issue (skips AI enhancement)").option("--json", "Output result as JSON").action(async (description, options) => {
+program.command("add-issue").alias("a").description("Create and enhance GitHub issue without starting workspace").argument("<description>", "Issue title (>30 chars, >2 spaces; or any non-empty text when --body provided)").option("--body <text>", "Body text for issue (skips AI enhancement)").option("--json", "Output result as JSON").action(async (description, options) => {
   const executeAction = async () => {
     try {
       const settingsManager = new SettingsManager();
@@ -3217,9 +3103,9 @@ program.command("add-issue").alias("a").description("Create and enhance GitHub i
     await executeAction();
   }
 });
-program.command("feedback").alias("f").description("Submit feedback/bug report to iloom-cli repository").argument("<description>", "Natural language description of feedback (>50 chars, >2 spaces)").option("--body <text>", "Body text for feedback (added after diagnostics)").action(async (description, options) => {
+program.command("feedback").alias("f").description("Submit feedback/bug report to iloom-cli repository").argument("<description>", "Feedback title (>30 chars, >2 spaces; or any non-empty text when --body provided)").option("--body <text>", "Body text for feedback (added after diagnostics)").action(async (description, options) => {
   try {
-    const { FeedbackCommand } = await import("./feedback-7PVBQNLJ.js");
+    const { FeedbackCommand } = await import("./feedback-OFVW22UW.js");
     const command = new FeedbackCommand();
     const feedbackOptions = {};
     if (options.body !== void 0) {
@@ -3299,7 +3185,7 @@ program.command("finish").alias("dn").description("Merge work and cleanup worksp
 });
 program.command("rebase").description("Rebase current branch on main with Claude-assisted conflict resolution").option("-f, --force", "Skip confirmation prompts").option("-n, --dry-run", "Preview actions without executing").action(async (options) => {
   try {
-    const { RebaseCommand } = await import("./rebase-4T5FQHNH.js");
+    const { RebaseCommand } = await import("./rebase-WZHHE5LU.js");
     const command = new RebaseCommand();
     await command.execute(options);
   } catch (error) {
@@ -3311,7 +3197,7 @@ program.command("spin").alias("ignite").description("Launch Claude with auto-det
   new Option("--one-shot <mode>", "One-shot automation mode").choices(["default", "noReview", "bypassPermissions"]).default("default")
 ).action(async (options) => {
   try {
-    const { IgniteCommand } = await import("./ignite-3B264M7K.js");
+    const { IgniteCommand } = await import("./ignite-NREQ3JRM.js");
     const command = new IgniteCommand();
     await command.execute(options.oneShot ?? "default");
   } catch (error) {
@@ -3322,7 +3208,7 @@ program.command("spin").alias("ignite").description("Launch Claude with auto-det
 program.command("open").description("Open workspace in browser or run CLI tool").argument("[identifier]", "Issue number, PR number, or branch name (auto-detected if omitted)").allowUnknownOption().action(async (identifier, _options, command) => {
   try {
     const args = (command == null ? void 0 : command.args) ? command.args.slice(identifier ? 1 : 0) : [];
-    const { OpenCommand } = await import("./open-OGCV32Z4.js");
+    const { OpenCommand } = await import("./open-KW4NTLXH.js");
     const cmd = new OpenCommand();
     const input = identifier ? { identifier, args } : { args };
     await cmd.execute(input);
@@ -3334,7 +3220,7 @@ program.command("open").description("Open workspace in browser or run CLI tool")
 program.command("run").description("Run CLI tool or open workspace in browser").argument("[identifier]", "Issue number, PR number, or branch name (auto-detected if omitted)").allowUnknownOption().action(async (identifier, _options, command) => {
   try {
     const args = (command == null ? void 0 : command.args) ? command.args.slice(identifier ? 1 : 0) : [];
-    const { RunCommand } = await import("./run-HNOP6WE2.js");
+    const { RunCommand } = await import("./run-HRYQ7TR7.js");
     const cmd = new RunCommand();
     const input = identifier ? { identifier, args } : { args };
     await cmd.execute(input);
@@ -3345,7 +3231,7 @@ program.command("run").description("Run CLI tool or open workspace in browser").
 });
 program.command("dev-server").alias("dev").description("Start dev server for workspace (foreground)").argument("[identifier]", "Issue number, PR number, or branch name (auto-detected if omitted)").option("--json", "Output as JSON").action(async (identifier, options) => {
   try {
-    const { DevServerCommand } = await import("./dev-server-JCJGQ3PV.js");
+    const { DevServerCommand } = await import("./dev-server-ASH7HJVI.js");
     const cmd = new DevServerCommand();
     await cmd.execute({ identifier, json: options == null ? void 0 : options.json });
   } catch (error) {
@@ -3355,7 +3241,7 @@ program.command("dev-server").alias("dev").description("Start dev server for wor
 });
 program.command("shell").alias("terminal").description("Open interactive shell with workspace environment").argument("[identifier]", "Issue number, PR number, or branch name (auto-detected if omitted)").action(async (identifier) => {
   try {
-    const { ShellCommand } = await import("./shell-DE3HKJSM.js");
+    const { ShellCommand } = await import("./shell-JMU5XTHW.js");
     const cmd = new ShellCommand();
     await cmd.execute({ identifier });
   } catch (error) {
@@ -3366,7 +3252,7 @@ program.command("shell").alias("terminal").description("Open interactive shell w
 program.command("cleanup").alias("remove").alias("clean").description("Remove workspaces").argument("[identifier]", "Branch name or issue number to cleanup (auto-detected)").option("-l, --list", "List all worktrees").option("-a, --all", "Remove all worktrees (interactive confirmation)").option("-i, --issue <number>", "Cleanup by issue number", parseInt).option("-f, --force", "Skip confirmations and force removal").option("--dry-run", "Show what would be done without doing it").option("--json", "Output result as JSON").action(async (identifier, options) => {
   const executeAction = async () => {
     try {
-      const { CleanupCommand } = await import("./cleanup-KDLVTT7M.js");
+      const { CleanupCommand } = await import("./cleanup-LU6NU2NZ.js");
       const command = new CleanupCommand();
       const input = {
         options: options ?? {}
@@ -3406,7 +3292,11 @@ program.command("list").description("Show active workspaces").option("--json", "
       metadata.set(worktree.path, loomMetadata);
     }
     if (options.json) {
-      const mainWorktreePath = await findMainWorktreePathWithSettings();
+      let mainWorktreePath;
+      try {
+        mainWorktreePath = await findMainWorktreePathWithSettings();
+      } catch {
+      }
       console.log(JSON.stringify(formatLoomsForJson(worktrees, mainWorktreePath, metadata), null, 2));
       return;
     }
@@ -3440,7 +3330,7 @@ program.command("list").description("Show active workspaces").option("--json", "
 });
 program.command("projects").description("List configured iloom projects").option("--json", "Output as JSON (default behavior)").action(async (options) => {
   try {
-    const { ProjectsCommand } = await import("./projects-P55273AB.js");
+    const { ProjectsCommand } = await import("./projects-QEAEBAT2.js");
     const command = new ProjectsCommand();
     const result = await command.execute(options);
     console.log(JSON.stringify(result, null, 2));
@@ -3449,9 +3339,9 @@ program.command("projects").description("List configured iloom projects").option
     process.exit(1);
   }
 });
-program.command("init").alias("config").description("Initialize iloom configuration and setup shell autocomplete").argument("[prompt]", 'Custom initial message to send to Claude (defaults to "Help me configure iloom settings.")').action(async (prompt) => {
+program.command("init").alias("config").description("Initialize iloom configuration").argument("[prompt]", 'Custom initial message to send to Claude (defaults to "Help me configure iloom settings.")').action(async (prompt) => {
   try {
-    const { InitCommand: InitCommand2 } = await import("./init-LBA6NUK2.js");
+    const { InitCommand: InitCommand2 } = await import("./init-F6PFMSU5.js");
     const command = new InitCommand2();
     const trimmedPrompt = prompt == null ? void 0 : prompt.trim();
     const customPrompt = trimmedPrompt && trimmedPrompt.length > 0 ? trimmedPrompt : void 0;
@@ -3463,7 +3353,7 @@ program.command("init").alias("config").description("Initialize iloom configurat
 });
 program.command("contribute").description("Set up local development environment for contributing to iloom").action(async () => {
   try {
-    const { ContributeCommand } = await import("./contribute-HY372S6F.js");
+    const { ContributeCommand } = await import("./contribute-RS3DO3WP.js");
     const command = new ContributeCommand();
     await command.execute();
   } catch (error) {
@@ -3543,9 +3433,9 @@ program.command("test-github").description("Test GitHub integration (Issue #3)")
 program.command("test-claude").description("Test Claude integration (Issue #10)").option("--detect", "Test Claude CLI detection").option("--version", "Get Claude CLI version").option("--branch <title>", "Test branch name generation with given title").option("--issue <number>", "Issue number for branch generation", "123").option("--launch <prompt>", "Launch Claude with a prompt (headless)").option("--interactive", "Launch Claude interactively (requires --launch)").option("--template <name>", "Test template loading").action(async (options) => {
   try {
     const { detectClaudeCli: detectClaudeCli2, getClaudeVersion, generateBranchName, launchClaude: launchClaude2 } = await import("./claude-ACVXNB6N.js");
-    const { PromptTemplateManager } = await import("./PromptTemplateManager-2TDZAUC6.js");
-    const { ClaudeService } = await import("./ClaudeService-W3SA7HVG.js");
-    const { ClaudeContextManager: ClaudeContextManager2 } = await import("./ClaudeContextManager-DK77227F.js");
+    const { PromptTemplateManager } = await import("./PromptTemplateManager-72FEOGT6.js");
+    const { ClaudeService } = await import("./ClaudeService-CJS32WG2.js");
+    const { ClaudeContextManager: ClaudeContextManager2 } = await import("./ClaudeContextManager-DQFKIMEP.js");
     logger.info("Testing Claude Integration\n");
     if (options.detect) {
       logger.info("Detecting Claude CLI...");
@@ -3693,7 +3583,7 @@ program.command("test-webserver").description("Test if a web server is running o
 });
 program.command("test-git").description("Test Git integration - findMainWorktreePath() function (reads .iloom/settings.json)").action(async () => {
   try {
-    const { TestGitCommand } = await import("./test-git-YMAE57UP.js");
+    const { TestGitCommand } = await import("./test-git-6SAIRBUD.js");
     const command = new TestGitCommand();
     await command.execute();
   } catch (error) {
@@ -3719,7 +3609,7 @@ program.command("test-tabs").description("Test iTerm2 dual tab functionality - o
 });
 program.command("test-prefix").description("Test worktree prefix configuration - preview worktree paths (reads .iloom/settings.json)").action(async () => {
   try {
-    const { TestPrefixCommand } = await import("./test-prefix-YCKL6CMT.js");
+    const { TestPrefixCommand } = await import("./test-prefix-RLVRK5ZD.js");
     const command = new TestPrefixCommand();
     await command.execute();
   } catch (error) {
@@ -3733,7 +3623,7 @@ program.command("test-prefix").description("Test worktree prefix configuration -
 program.command("summary").description("Generate Claude session summary for a loom").argument("[identifier]", "Issue number, PR number (pr/123), or branch name (auto-detected if omitted)").option("--with-comment", "Post summary as a comment to the issue/PR").option("--json", "Output result as JSON").action(async (identifier, options) => {
   const executeAction = async () => {
     try {
-      const { SummaryCommand } = await import("./summary-GDT7DTRI.js");
+      const { SummaryCommand } = await import("./summary-4SSGGH7N.js");
       const command = new SummaryCommand();
       const result = await command.execute({ identifier, options });
       if (options.json && result) {
@@ -3759,10 +3649,39 @@ program.command("summary").description("Generate Claude session summary for a lo
     await executeAction();
   }
 });
+program.command("recap").description("Get recap for a loom (defaults to current directory)").argument("[identifier]", "Issue number, PR number (pr/123), or branch name (auto-detected if omitted)").option("--json", "Output as JSON with filePath for file watching").action(async (identifier, options) => {
+  const executeAction = async () => {
+    try {
+      const { RecapCommand } = await import("./recap-33NPZ3ZO.js");
+      const command = new RecapCommand();
+      const result = await command.execute({ identifier, json: options.json });
+      if (options.json && result) {
+        console.log(JSON.stringify(result, null, 2));
+      }
+      process.exit(0);
+    } catch (error) {
+      if (options.json) {
+        console.log(JSON.stringify({ error: error instanceof Error ? error.message : "Unknown error" }, null, 2));
+      } else {
+        logger.error(`Recap failed: ${error instanceof Error ? error.message : "Unknown error"}`);
+        if (error instanceof Error && error.stack) {
+          logger.debug(error.stack);
+        }
+      }
+      process.exit(1);
+    }
+  };
+  if (options.json) {
+    const jsonLogger = createStderrLogger();
+    await withLogger(jsonLogger, executeAction);
+  } else {
+    await executeAction();
+  }
+});
 program.command("test-neon").description("Test Neon integration and debug configuration").action(async () => {
   var _a;
   try {
-    const { SettingsManager: SettingsManager2 } = await import("./SettingsManager-FJFU6JJD.js");
+    const { SettingsManager: SettingsManager2 } = await import("./SettingsManager-XPR4TEQL.js");
     const { createNeonProviderFromSettings: createNeonProviderFromSettings2 } = await import("./neon-helpers-L5CXQ5CT.js");
     logger.info("Testing Neon Integration\n");
     logger.info("1. Settings Configuration:");
@@ -3861,6 +3780,7 @@ if (isRunDirectly) {
   }
 }
 export {
-  validateGhCliForCommand
+  validateGhCliForCommand,
+  validateIdeForStartCommand
 };
 //# sourceMappingURL=cli.js.map