ira-review 3.1.4 → 3.1.6

package/README.github.md

@@ -137,10 +137,6 @@ This is the feature that catches "does this actually match the ticket?" before a
 
 When a JIRA ticket has **no acceptance criteria at all**, IRA generates suggested ACs from the diff and (by default) posts them as a comment on the JIRA ticket for the Product Owner to review and refine. If your CI environment cannot or should not write back to JIRA — for example, the JIRA service account is read-only, or org policy forbids automated JIRA writes — pass `--no-post-acs-to-jira` (or set `IRA_POST_ACS_TO_JIRA=false`). The suggestions still render in the PR summary under **📝 Suggested Acceptance Criteria**, so reviewers see them either way; only the JIRA write is suppressed.
 
-### "All Clear" PR Summary
-
-When IRA finishes a review with **zero issues** found and **JIRA acceptance criteria are 100% covered** (or the ticket needs no ACs), the PR summary leads with a celebratory ✅ banner: a confident, specific signal that automated review passed, alongside an explicit reminder that **human reviewer approval is still required before merge**. The banner is suppressed when an AC gap exists so the summary never reads "safe to approve" while a requirements gap is still visible — IRA augments your code review process, it doesn't replace it.
-
 ### Inline AI Comments
 
 Each issue is posted as an inline comment on the exact line in the PR, containing:

package/README.md

@@ -42,7 +42,6 @@ Each issue is posted as an inline comment on the exact PR line with explanation,
 - Two-pass critical review (`--ai-model-critical`) — bulk pass uses your everyday model; only `CRITICAL`/`BLOCKER` findings are re-run against a stronger model, keeping premium-request cost low while preserving deep analysis on what matters
 - JIRA acceptance criteria validation with per-criterion pass/fail and edge case detection
 - JIRA AC auto-detection — finds AC from custom field or description automatically
-- "All Clear" PR summary block — celebratory ✅ banner when zero issues are found and JIRA AC coverage is 100%, with a clear "human reviewer approval is still required before merge" reminder. Suppressed automatically if any AC gap exists, so the summary never claims "safe to approve" while requirements are unmet.
 - Custom team review rules via `.ira-rules.json` (see below)
 - Test case generation from JIRA tickets (Jest, Vitest, Playwright, etc.)
 - Comment deduplication across re-runs

package/README.npm.md

@@ -42,7 +42,6 @@ Each issue is posted as an inline comment on the exact PR line with explanation,
 - Two-pass critical review (`--ai-model-critical`) — bulk pass uses your everyday model; only `CRITICAL`/`BLOCKER` findings are re-run against a stronger model, keeping premium-request cost low while preserving deep analysis on what matters
 - JIRA acceptance criteria validation with per-criterion pass/fail and edge case detection
 - JIRA AC auto-detection — finds AC from custom field or description automatically
-- "All Clear" PR summary block — celebratory ✅ banner when zero issues are found and JIRA AC coverage is 100%, with a clear "human reviewer approval is still required before merge" reminder. Suppressed automatically if any AC gap exists, so the summary never claims "safe to approve" while requirements are unmet.
 - Custom team review rules via `.ira-rules.json` (see below)
 - Test case generation from JIRA tickets (Jest, Vitest, Playwright, etc.)
 - Comment deduplication across re-runs

package/dist/cli.js

@@ -14,10 +14,9 @@ import {
 
 // src/cli.ts
 import { Command } from "commander";
-import { existsSync as existsSync9, readFileSync as readFileSync7 } from "fs";
+import { existsSync as existsSync9 } from "fs";
 import { readFile } from "fs/promises";
-import { resolve as resolve3, join as join7, dirname } from "path";
-import { fileURLToPath } from "url";
+import { resolve as resolve4, join as join7 } from "path";
 import { execSync as execSync5 } from "child_process";
 
 // src/core/sonarClient.ts
@@ -914,7 +913,7 @@ var AmpCliProvider = class {
     return parseAIResponse(cleaned);
   }
   rawReview(prompt) {
-    return new Promise((resolve4, reject) => {
+    return new Promise((resolve5, reject) => {
       const env2 = { ...process.env };
       const home = homedir();
       const isWin = process.platform === "win32";
@@ -1005,13 +1004,13 @@ var AmpCliProvider = class {
           }
         }
         if (result) {
-          resolve4(result);
+          resolve5(result);
         } else if (errorOutput) {
           reject(new Error(`AMP CLI failed: ${errorOutput.trim()}`));
         } else if (code !== 0) {
           reject(new Error(`AMP CLI exited with code ${code}`));
         } else {
-          resolve4("");
+          resolve5("");
         }
       });
     });
@@ -1028,11 +1027,9 @@ var CopilotCliProvider = class {
     return parseAIResponse(cleaned);
   }
   rawReview(prompt) {
-    return new Promise((resolve4, reject) => {
+    return new Promise((resolve5, reject) => {
       const env2 = { ...process.env };
       const args = [
-        "-p",
-        prompt,
         "-s",
         // silent — only the response, no stats lines
         "--allow-all-tools",
@@ -1043,10 +1040,12 @@ var CopilotCliProvider = class {
       ];
       const useShell = process.platform === "win32";
       const child = spawn("copilot", args, {
-        stdio: ["ignore", "pipe", "pipe"],
+        stdio: ["pipe", "pipe", "pipe"],
         env: env2,
         shell: useShell
       });
+      child.stdin.write(prompt);
+      child.stdin.end();
       let stdout = "";
       let stderr = "";
       child.stdout.on("data", (chunk) => {
@@ -1061,7 +1060,7 @@ var CopilotCliProvider = class {
       });
       child.on("close", (code) => {
         if (code === 0) {
-          resolve4(stdout.trim());
+          resolve5(stdout.trim());
           return;
         }
         const detail = stderr.trim() || stdout.trim() || `exited with code ${code}`;
@@ -2612,73 +2611,36 @@ function validateCoverage(value) {
 }
 
 // src/core/summaryBuilder.ts
-function buildSummary2(result) {
+function buildSummary2(result, meta = {}) {
   const lines = [];
-  lines.push("# \u{1F50D} IRA Review Summary");
+  const dot = computeStatusDot(result);
+  const riskLevel = result.risk?.level ?? "LOW";
+  const riskScore = result.risk?.score ?? 0;
+  const riskMax = result.risk?.maxScore ?? 100;
+  lines.push(`### ${dot} IRA Review \xB7 ${riskLevel} risk (${riskScore}/${riskMax})`);
   lines.push("");
-  if (result.risk) {
-    const emoji = result.risk.level === "CRITICAL" ? "\u{1F534}" : result.risk.level === "HIGH" ? "\u{1F7E0}" : result.risk.level === "MEDIUM" ? "\u{1F7E1}" : "\u{1F7E2}";
-    lines.push(
-      `## ${emoji} Risk: ${result.risk.level} (${result.risk.score}/${result.risk.maxScore})`
-    );
-    lines.push("");
-    if (result.reviewMode === "sonar") {
-      lines.push("| Factor | Score | Detail |");
-      lines.push("|---|---|---|");
-      for (const f of result.risk.factors) {
-        lines.push(`| ${f.name} | ${f.score}/${f.maxScore} | ${f.detail} |`);
-      }
-      lines.push("");
-    }
-  }
-  const acGapExists = result.requirementCompletion && result.requirementCompletion.completionPercentage < 100 || result.acceptanceValidation && !result.acceptanceValidation.overallPass;
-  if (result.comments.length === 0 && !acGapExists) {
-    const fw = result.framework ?? "your stack";
-    const acLine = result.requirementCompletion ? `Acceptance criteria for **${result.requirementCompletion.jiraKey}**: **${result.requirementCompletion.completionPercentage}% covered** (${result.requirementCompletion.metCriteria}/${result.requirementCompletion.totalCriteria}).` : result.acceptanceValidation ? `Acceptance criteria for **${result.acceptanceValidation.jiraKey}**: ${result.acceptanceValidation.overallPass ? "**all met** \u2705" : "**partially met** \u2014 see the JIRA section above"}.` : result.acGeneration && result.acGeneration.criteria.length > 0 ? result.acGeneration.postedToJira ? `\u{1F4DD} No acceptance criteria found on **${result.acGeneration.jiraKey}** \u2014 IRA generated **${result.acGeneration.totalCriteria} suggested AC${result.acGeneration.totalCriteria === 1 ? "" : "s"}** and posted them as a comment on the JIRA ticket for the Product Owner / requirement author to review and refine.` : `\u{1F4DD} No acceptance criteria found on **${result.acGeneration.jiraKey}** \u2014 IRA generated **${result.acGeneration.totalCriteria} suggested AC${result.acGeneration.totalCriteria === 1 ? "" : "s"}** (see the Suggested Acceptance Criteria section below).` : null;
-    lines.push("## \u2705 All Clear \u2014 No Issues Found");
-    lines.push("");
-    lines.push(`> \u{1F389} **Nice work on PR #${result.pullRequestId}!**`);
-    lines.push(`>`);
-    lines.push(`> IRA scanned every changed file across **${fw}** and didn't surface a single concern.`);
-    if (acLine) {
-      lines.push(`>`);
-      lines.push(`> ${acLine}`);
-    }
-    if (result.risk) {
-      lines.push(`>`);
-      lines.push(`> Risk score: **${result.risk.score}/${result.risk.maxScore}** (${result.risk.level}).`);
-    }
-    lines.push(`>`);
-    lines.push(`> \u2705 **Safe to approve from an automated-review standpoint.**`);
-    lines.push(`>`);
-    lines.push(`> \u{1F465} **Human reviewer approval is still required before merge.** IRA augments your code review process \u2014 it doesn't replace it. Please ensure your team's review and approval requirements have been met before merging.`);
-    lines.push("");
-  }
-  lines.push("## Overview");
-  lines.push("");
-  lines.push(`| Metric | Value |`);
-  lines.push(`|---|---|`);
-  lines.push(`| Review mode | ${result.reviewMode === "standalone" ? "AI-only" : "Sonar + AI"} |`);
-  lines.push(`| Total issues | ${result.totalIssues} |`);
-  lines.push(`| Reviewed (AI) | ${result.reviewedIssues} |`);
-  lines.push(
-    `| Framework | ${result.framework ?? "not detected"} |`
-  );
+  const metrics = [];
+  if (typeof result.filesReviewed === "number") {
+    metrics.push(`${result.filesReviewed} file${result.filesReviewed === 1 ? "" : "s"} reviewed`);
+  }
+  metrics.push(`${result.comments.length} finding${result.comments.length === 1 ? "" : "s"}`);
+  const acMetric = formatAcMetric(result);
+  if (acMetric) metrics.push(acMetric);
+  lines.push(metrics.join(" \xB7 "));
   lines.push("");
-  if (result.complexity && result.complexity.hotspots.length > 0) {
-    lines.push("## \u{1F9E0} Complexity Hotspots");
+  if (result.comments.length > 0) {
+    lines.push("## Findings");
     lines.push("");
-    lines.push("| File | Complexity | Cognitive |");
-    lines.push("|---|---|---|");
-    for (const h of result.complexity.hotspots.slice(0, 5)) {
-      lines.push(`| ${h.filePath} | ${h.complexity} | ${h.cognitiveComplexity} |`);
+    lines.push("| File | Line | Rule | Severity |");
+    lines.push("|---|---|---|---|");
+    for (const c of result.comments) {
+      lines.push(`| ${c.filePath} | ${c.line} | \`${c.rule}\` | ${c.severity} |`);
     }
     lines.push("");
   }
   if (result.requirementCompletion) {
     const rc = result.requirementCompletion;
-    const pctIcon = rc.completionPercentage === 100 ? "\u2705" : rc.completionPercentage >= 50 ? "\u{1F7E1}" : "\u{1F534}";
-    lines.push(`## ${pctIcon} Requirements: ${rc.jiraKey} - ${rc.completionPercentage}% Complete (${rc.metCriteria}/${rc.totalCriteria})`);
+    lines.push(`## Acceptance Criteria \u2014 ${rc.jiraKey} (${rc.completionPercentage}% covered, ${rc.metCriteria}/${rc.totalCriteria})`);
     lines.push("");
     for (const r of rc.requirements) {
       const icon = r.coverage === "full" ? "\u2705" : r.coverage === "partial" ? "\u{1F7E1}" : "\u274C";
@@ -2689,42 +2651,65 @@ function buildSummary2(result) {
     }
     if (rc.edgeCases.length > 0) {
       lines.push("");
-      lines.push("### \u26A0\uFE0F Edge Cases Not Covered");
+      lines.push("### Edge cases not covered");
       for (const e of rc.edgeCases) {
         lines.push(`- ${e}`);
       }
     }
     if (rc.parseWarning) {
       lines.push("");
-      lines.push(`> \u26A0\uFE0F **Warning:** ${rc.parseWarning}`);
+      lines.push(`> \u26A0\uFE0F ${rc.parseWarning}`);
     }
     lines.push("");
   } else if (result.acceptanceValidation) {
     const av = result.acceptanceValidation;
-    const icon = av.overallPass ? "\u2705" : "\u274C";
-    lines.push(`## ${icon} JIRA: ${av.jiraKey} - ${av.summary}`);
+    const status = av.overallPass ? "all met" : "gaps found";
+    lines.push(`## Acceptance Criteria \u2014 ${av.jiraKey} (${status})`);
+    lines.push("");
+    lines.push(`_${av.summary}_`);
     lines.push("");
     for (const c of av.criteria) {
       lines.push(`- ${c.met ? "\u2705" : "\u274C"} ${c.description}`);
     }
     lines.push("");
   }
+  if (result.risk && result.reviewMode === "sonar" && result.risk.factors.length > 0) {
+    lines.push("## Risk Factors");
+    lines.push("");
+    lines.push("| Factor | Score | Detail |");
+    lines.push("|---|---|---|");
+    for (const f of result.risk.factors) {
+      lines.push(`| ${f.name} | ${f.score}/${f.maxScore} | ${f.detail} |`);
+    }
+    lines.push("");
+  }
+  if (result.complexity && result.complexity.hotspots.length > 0) {
+    lines.push("## Complexity Hotspots");
+    lines.push("");
+    lines.push("| File | Complexity | Cognitive |");
+    lines.push("|---|---|---|");
+    for (const h of result.complexity.hotspots.slice(0, 5)) {
+      lines.push(`| ${h.filePath} | ${h.complexity} | ${h.cognitiveComplexity} |`);
+    }
+    lines.push("");
+  }
   if (result.acGeneration && result.acGeneration.criteria.length > 0) {
     const ag = result.acGeneration;
-    lines.push(`## \u{1F4DD} Suggested Acceptance Criteria (${ag.totalCriteria} generated)`);
+    lines.push(`## Suggested Acceptance Criteria \u2014 ${ag.jiraKey} (${ag.totalCriteria} generated)`);
     lines.push("");
-    lines.push(`> No acceptance criteria found in ${ag.jiraKey}. IRA inferred the following from: ${ag.sources.join(", ")}.`);
+    const postedNote = ag.postedToJira ? `Posted to JIRA as a comment for the Product Owner to review.` : `Not posted to JIRA (suggestions stay in this summary only).`;
+    lines.push(`> No acceptance criteria found on **${ag.jiraKey}**. IRA inferred the following from: ${ag.sources.join(", ")}. ${postedNote}`);
     lines.push("");
     for (const ac of ag.criteria) {
-      lines.push(`**${ac.id}:**`);
+      lines.push(`**${ac.id}**`);
       lines.push(`- **Given** ${ac.given}`);
       lines.push(`- **When** ${ac.when}`);
       lines.push(`- **Then** ${ac.then}`);
       lines.push("");
     }
     if (ag.reviewHints && ag.reviewHints.length > 0) {
-      lines.push(`### \u2753 Questions for PO`);
-      lines.push(`> IRA could not determine the following from the code. Answering these will strengthen the ACs above:`);
+      lines.push(`### Questions for PO`);
+      lines.push(`> IRA could not determine the following from the code. Answering these will strengthen the ACs above.`);
       lines.push("");
       for (const hint of ag.reviewHints) {
         lines.push(`- ${hint}`);
@@ -2732,7 +2717,7 @@ function buildSummary2(result) {
       lines.push("");
     }
     if (ag.parseWarning) {
-      lines.push(`> \u26A0\uFE0F **Warning:** ${ag.parseWarning}`);
+      lines.push(`> \u26A0\uFE0F ${ag.parseWarning}`);
       lines.push("");
     }
   }
@@ -2743,7 +2728,7 @@ function buildSummary2(result) {
     const headerParts = [`${testableCount} test${testableCount !== 1 ? "s" : ""}`];
     if (tg.edgeCases > 0) headerParts.push(`${tg.edgeCases} advanced cases`);
     if (notTestableCount > 0) headerParts.push(`${notTestableCount} not-testable`);
-    lines.push(`## \u{1F9EA} Generated Test Cases (${headerParts.join(", ")})`);
+    lines.push(`## Generated Test Cases (${headerParts.join(", ")})`);
     lines.push("");
     const byCriterion = /* @__PURE__ */ new Map();
     for (const tc of tg.testCases) {
@@ -2772,25 +2757,62 @@ function buildSummary2(result) {
     }
   }
   if (result.testGeneration?.parseWarning) {
-    lines.push(`> \u26A0\uFE0F **Warning:** ${result.testGeneration.parseWarning}`);
-    lines.push("");
-  }
-  if (result.comments.length > 0) {
-    lines.push("## Issues Reviewed");
-    lines.push("");
-    lines.push("| File | Line | Rule | Severity |");
-    lines.push("|---|---|---|---|");
-    for (const c of result.comments) {
-      lines.push(
-        `| ${c.filePath} | ${c.line} | \`${c.rule}\` | ${c.severity} |`
-      );
-    }
+    lines.push(`> \u26A0\uFE0F ${result.testGeneration.parseWarning}`);
     lines.push("");
   }
   lines.push("---");
-  lines.push("*Generated by [ira-review](https://www.npmjs.com/package/ira-review)*");
+  lines.push(`_${formatFooter(meta)}_`);
+  lines.push("");
+  lines.push("_Human reviewer approval is still required before merge. IRA augments code review; it does not replace it._");
   return lines.join("\n");
 }
+function computeStatusDot(result) {
+  const riskRank = riskRankFor(result.risk?.level ?? "LOW");
+  const acRank = acGapRank(result);
+  const blockerRank = result.comments.some((c) => c.severity === "BLOCKER") ? 3 : 0;
+  const worst = Math.max(riskRank, acRank, blockerRank);
+  return ["\u{1F7E2}", "\u{1F7E1}", "\u{1F7E0}", "\u{1F534}"][worst] ?? "\u{1F7E2}";
+}
+function riskRankFor(level) {
+  switch (level) {
+    case "CRITICAL":
+      return 3;
+    case "HIGH":
+      return 2;
+    case "MEDIUM":
+      return 1;
+    default:
+      return 0;
+  }
+}
+function acGapRank(result) {
+  if (result.requirementCompletion && result.requirementCompletion.completionPercentage < 100) return 1;
+  if (result.acceptanceValidation && !result.acceptanceValidation.overallPass) return 1;
+  return 0;
+}
+function formatAcMetric(result) {
+  if (result.requirementCompletion) {
+    const rc = result.requirementCompletion;
+    return `${rc.jiraKey}: ${rc.metCriteria}/${rc.totalCriteria} ACs met`;
+  }
+  if (result.acceptanceValidation) {
+    const av = result.acceptanceValidation;
+    return `${av.jiraKey}: ${av.overallPass ? "ACs met" : "AC gaps"}`;
+  }
+  if (result.acGeneration && result.acGeneration.criteria.length > 0) {
+    const ag = result.acGeneration;
+    return `${ag.jiraKey}: ${ag.totalCriteria} AC${ag.totalCriteria === 1 ? "" : "s"} suggested`;
+  }
+  return null;
+}
+function formatFooter(meta) {
+  const parts = [];
+  parts.push(`ira-review${meta.version ? ` ${meta.version}` : ""}`);
+  if (meta.aiProvider || meta.aiModel) {
+    parts.push([meta.aiProvider, meta.aiModel].filter(Boolean).join("/"));
+  }
+  return parts.join(" \xB7 ");
+}
 
 // src/integrations/notifier.ts
 var Notifier = class {
@@ -2910,6 +2932,31 @@ function resolveGitRoot() {
   }
 }
 
+// src/utils/packageInfo.ts
+import { readFileSync as readFileSync6 } from "fs";
+import { dirname, resolve as resolve2 } from "path";
+import { fileURLToPath } from "url";
+function readPackageVersion() {
+  try {
+    const here = dirname(fileURLToPath(import.meta.url));
+    const candidates = [
+      resolve2(here, "..", "package.json"),
+      // dist/utils/* → ../package.json
+      resolve2(here, "..", "..", "package.json")
+      // src/utils/*  → ../../package.json
+    ];
+    for (const pkgPath of candidates) {
+      try {
+        const pkg = JSON.parse(readFileSync6(pkgPath, "utf-8"));
+        if (pkg.name === "ira-review" && pkg.version) return pkg.version;
+      } catch {
+      }
+    }
+  } catch {
+  }
+  return "unknown";
+}
+
 // src/core/acGenerator.ts
 function escapeSentinels5(text) {
   return text.replace(/<\/(ticket_context|diff_context|commit_context|epic_context)>/gi, "<\\/$1>");
@@ -3514,6 +3561,7 @@ Cause: ${warnings[warnings.length - 1]}` : "";
         (c) => !existing.has(deduplicateKey(c.filePath, c.line, c.rule))
       );
     }
+    const filesReviewed = reviewMode === "sonar" ? grouped.length : diffByFile.size;
     const result = {
       pullRequestId,
       framework,
@@ -3528,9 +3576,14 @@ Cause: ${warnings[warnings.length - 1]}` : "";
       testGeneration,
       requirementCompletion,
       acGeneration,
-      warnings
+      warnings,
+      filesReviewed
     };
-    const summary = buildSummary2(result);
+    const summary = buildSummary2(result, {
+      version: readPackageVersion(),
+      aiProvider: this.config.ai.provider,
+      aiModel: this.config.ai.model
+    });
     if (this.config.dryRun) {
       console.log(summary);
       for (const comment of comments) {
@@ -3850,19 +3903,19 @@ function optionalEnv(key) {
 }
 
 // src/utils/configFile.ts
-import { readFileSync as readFileSync6, existsSync as existsSync8 } from "fs";
-import { resolve as resolve2 } from "path";
+import { readFileSync as readFileSync7, existsSync as existsSync8 } from "fs";
+import { resolve as resolve3 } from "path";
 var CONFIG_FILENAMES = [".irarc.json", "ira.config.json"];
 function loadConfigFile(explicitPath, cwd = process.cwd()) {
   if (explicitPath) {
-    const filePath = resolve2(cwd, explicitPath);
+    const filePath = resolve3(cwd, explicitPath);
     if (!existsSync8(filePath)) {
       throw new Error(`Config file not found: ${filePath}`);
     }
     return parseConfigFile(filePath);
   }
   for (const filename of CONFIG_FILENAMES) {
-    const filePath = resolve2(cwd, filename);
+    const filePath = resolve3(cwd, filename);
     if (existsSync8(filePath)) {
       return parseConfigFile(filePath);
     }
@@ -3871,7 +3924,7 @@ function loadConfigFile(explicitPath, cwd = process.cwd()) {
 }
 function parseConfigFile(filePath) {
   try {
-    const raw = readFileSync6(filePath, "utf-8");
+    const raw = readFileSync7(filePath, "utf-8");
     const parsed = JSON.parse(raw);
     return mapConfigToFlat(parsed);
   } catch (error) {
@@ -4059,7 +4112,7 @@ function isCiEnvironment() {
 }
 function isFirstRun() {
   if (isCiEnvironment()) return false;
-  return !existsSync9(resolve3(process.cwd(), ".irarc.json")) && !existsSync9(resolve3(process.cwd(), "ira.config.json"));
+  return !existsSync9(resolve4(process.cwd(), ".irarc.json")) && !existsSync9(resolve4(process.cwd(), "ira.config.json"));
 }
 function logEnvironmentInfo(config) {
   const proxy = process.env.HTTPS_PROXY ?? process.env.HTTP_PROXY ?? process.env.https_proxy ?? process.env.http_proxy;
@@ -4085,16 +4138,6 @@ function logEnvironmentInfo(config) {
     step("\u{1F510}", `SSL_CERT_FILE:     ${sslCert}`);
   }
 }
-function readPackageVersion() {
-  try {
-    const here = dirname(fileURLToPath(import.meta.url));
-    const pkgPath = resolve3(here, "..", "package.json");
-    const pkg = JSON.parse(readFileSync7(pkgPath, "utf-8"));
-    if (pkg.version) return pkg.version;
-  } catch {
-  }
-  return "unknown";
-}
 var program = new Command();
 program.name("ira-review").description("AI-powered PR review tool with SonarQube + GitHub/Bitbucket integration").version(readPackageVersion());
 program.command("review").description("Run AI-powered review on a pull request").option("--sonar-url <url>", "SonarQube/SonarCloud base URL (or IRA_SONAR_URL)").option("--sonar-token <token>", "SonarQube API token (or IRA_SONAR_TOKEN)").option("--project-key <key>", "Sonar project key (or IRA_PROJECT_KEY)").option("--pr <id>", "Pull request ID (or IRA_PR)").option("--scm-provider <provider>", "SCM provider: bitbucket or github (or IRA_SCM_PROVIDER)").option("--bitbucket-token <token>", "Bitbucket API token (or IRA_BITBUCKET_TOKEN)").option("--repo <repo>", "Bitbucket workspace/repo-slug (Cloud) or PROJECT/repo-slug (Server) (or IRA_REPO)").option("--bitbucket-url <url>", "Bitbucket base URL (or IRA_BITBUCKET_URL)").option("--bitbucket-type <type>", "Bitbucket type: cloud or server (auto-detects from URL if omitted; or IRA_BITBUCKET_TYPE)").option("--github-token <token>", "GitHub API token (or IRA_GITHUB_TOKEN)").option("--github-repo <repo>", "GitHub owner/repo (or IRA_GITHUB_REPO)").option("--github-url <url>", "GitHub Enterprise URL (or IRA_GITHUB_URL)").option("--ai-provider <provider>", "AI provider").option("--ai-model <model>", "AI model to use").option("--dry-run", "Print comments to stdout instead of posting to SCM").option("--min-severity <level>", "Minimum severity to review (BLOCKER|CRITICAL|MAJOR|MINOR|INFO)").option("--jira-url <url>", "JIRA base URL (or IRA_JIRA_URL)").option("--jira-email <email>", "JIRA email (or IRA_JIRA_EMAIL)").option("--jira-token <token>", "JIRA API token (or IRA_JIRA_TOKEN)").option("--jira-type <type>", "JIRA type: cloud or server (auto-detects from URL if omitted)").option("--jira-ticket <key>", "JIRA ticket key (e.g. PROJ-123)").option("--jira-ac-field <field>", "Custom field ID for acceptance criteria").option("--jira-ac-source <source>", "Where to look for AC: customField, description, or both (default: customField)").option("--no-post-acs-to-jira", "Don't post AI-generated acceptance criteria back to the JIRA ticket when none exist; keep them in the PR summary only (or IRA_POST_ACS_TO_JIRA=false)").option("--slack-webhook <url>", "Slack webhook URL for notifications").option("--teams-webhook <url>", "Teams webhook URL for notifications").option("--notify-min-risk <level>", "Only notify when risk is at or above this level: low, medium, high, critical").option("--notify-on-ac-fail", "Send notification when JIRA acceptance criteria validation fails").option("--ai-base-url <url>", "AI provider base URL (Azure endpoint, Ollama URL)").option("--ai-api-key <key>", "AI API key (or IRA_AI_API_KEY / OPENAI_API_KEY)").option("--ai-api-version <version>", "Azure OpenAI API version").option("--ai-deployment <name>", "Azure OpenAI deployment name").option("--ai-model-critical <model>", "Stronger AI model for BLOCKER/CRITICAL issues").option("--generate-tests", "Generate test cases from JIRA acceptance criteria").option("--test-framework <framework>", "Test framework: jest, vitest, mocha, playwright, cypress, gherkin, pytest, junit (default: jest)").option("--config <path>", "Path to config file (default: auto-detect .irarc.json / ira.config.json)").option("--no-config-file", "Disable auto-loading config file from repo").option("--rules-url <url>", "Fetch .ira-rules.json from URL (raw HTTP) \u2014 useful when CI has no full checkout (or IRA_RULES_URL)").option("--comment-style <style>", "Comment formatter style: compact (default) or detailed (or IRA_COMMENT_STYLE)").action(async (opts) => {
@@ -4406,7 +4449,7 @@ program.command("init").description("Interactive setup: detect config and write
   const ai = detectAiProvider();
   step("\u2713", `SCM: ${scm ?? "not detected"} | AI: ${ai ? ai.provider : "not detected"}`);
   step("\u23F3", "Writing project config\u2026");
-  const rcPath = resolve3(process.cwd(), ".irarc.json");
+  const rcPath = resolve4(process.cwd(), ".irarc.json");
   let rcConfig = {};
   if (existsSync9(rcPath)) {
     try {