automata-cli 0.1.0-feature-021-execute-command.125 → 0.1.0-feature-021-execute-command.127

package/dist/index.js

@@ -557,6 +557,14 @@ function extractSonarProjectKey(url) {
     return null;
   }
 }
+function buildSonarPullRequestUrl(projectKey, prNumber) {
+  return `https://sonarcloud.io/summary/new_code?id=${encodeURIComponent(projectKey)}&pullRequest=${String(prNumber)}`;
+}
+function resolveSonarPullRequestUrl(url, prNumber) {
+  const projectKey = extractSonarProjectKey(url);
+  if (!projectKey) return url;
+  return buildSonarPullRequestUrl(projectKey, prNumber);
+}
 function isSonarUrl(url) {
   try {
     const hostname = new URL(url).hostname;
@@ -671,13 +679,28 @@ async function fetchSonarJson(apiUrl) {
     clearTimeout(timeoutId);
   }
 }
-async function fetchSonarNewIssues(projectKey, prNumber) {
+async function fetchSonarNewIssues(projectKey, prNumber, sonarcloudUrl) {
   const apiUrl = `https://sonarcloud.io/api/issues/search?componentKeys=${encodeURIComponent(projectKey)}&pullRequest=${String(prNumber)}&resolved=false&ps=1`;
   const response = await fetchSonarJson(apiUrl);
   if (!response.ok) {
-    return null;
+    if (response.status === 401) {
+      return {
+        total: null,
+        note: `SonarCloud project is private. Open the Sonar URL in an authenticated browser: ${sonarcloudUrl}`
+      };
+    }
+    return {
+      total: null,
+      note: "SonarCloud new-issue count is unavailable right now."
+    };
+  }
+  if (typeof response.data.paging?.total === "number") {
+    return { total: response.data.paging.total };
   }
-  return response.data.paging?.total ?? null;
+  return {
+    total: null,
+    note: "SonarCloud did not return a new-issue count for this pull request."
+  };
 }
 async function fetchSonarGateViolations(projectKey, prNumber) {
   const apiUrl = `https://sonarcloud.io/api/qualitygates/project_status?projectKey=${encodeURIComponent(projectKey)}&pullRequest=${String(prNumber)}`;
@@ -793,19 +816,22 @@ async function fetchSonarFailureSummary(projectKey, prNumber, sonarcloudUrl) {
   if (!gateResult.ok && gateResult.status === 401) {
     return {
       summary: sonarPrivateFailureSummary(sonarcloudUrl),
-      issueTotal: null
+      issueTotal: null,
+      issueNote: `SonarCloud project is private. Open the Sonar URL in an authenticated browser: ${sonarcloudUrl}`
     };
   }
   if (!issuesResult.ok && issuesResult.status === 401) {
     return {
       summary: sonarPrivateFailureSummary(sonarcloudUrl),
-      issueTotal: null
+      issueTotal: null,
+      issueNote: `SonarCloud project is private. Open the Sonar URL in an authenticated browser: ${sonarcloudUrl}`
     };
   }
   if (!hotspotsResult.ok && hotspotsResult.status === 401) {
     return {
       summary: sonarPrivateFailureSummary(sonarcloudUrl),
-      issueTotal: null
+      issueTotal: null,
+      issueNote: `SonarCloud project is private. Open the Sonar URL in an authenticated browser: ${sonarcloudUrl}`
     };
   }
   const gateViolations = gateResult.ok ? gateResult.data.gateViolations : [];
@@ -813,6 +839,7 @@ async function fetchSonarFailureSummary(projectKey, prNumber, sonarcloudUrl) {
   const securityHotspots = hotspotsResult.ok ? hotspotsResult.data : [];
   const qualityGateStatus = gateResult.ok ? gateResult.data.qualityGateStatus : void 0;
   const issueTotal = issuesResult.ok ? issuesResult.data.total : null;
+  const issueNote = issuesResult.ok ? void 0 : "SonarCloud new-issue count is unavailable right now.";
   if (gateViolations.length === 0 && issues.length === 0 && securityHotspots.length === 0 && !qualityGateStatus) {
     return {
       summary: {
@@ -822,7 +849,8 @@ async function fetchSonarFailureSummary(projectKey, prNumber, sonarcloudUrl) {
         securityHotspots: [],
         unavailableMessage: "SonarCloud failure details are unavailable right now."
       },
-      issueTotal
+      issueTotal,
+      issueNote
     };
   }
   return {
@@ -833,7 +861,8 @@ async function fetchSonarFailureSummary(projectKey, prNumber, sonarcloudUrl) {
       issues,
       securityHotspots
     },
-    issueTotal
+    issueTotal,
+    issueNote
   };
 }
 async function getPrInfoGh(branch) {
@@ -851,12 +880,16 @@ async function getPrInfoGh(branch) {
     throw new Error(stderr.trim() || "Failed to query GitHub. Is `gh` installed and authenticated?");
   }
   const raw = JSON.parse(stdout);
-  const failedChecks = (raw.statusCheckRollup ?? []).filter(
+  const statusChecks = raw.statusCheckRollup ?? [];
+  const failedChecks = statusChecks.filter(
     (c) => c.conclusion !== null && ["FAILURE", "TIMED_OUT", "ACTION_REQUIRED", "CANCELLED"].includes(c.conclusion)
   );
-  const ownerRepo = failedChecks.length > 0 ? parseOwnerRepo() : null;
+  const sonarNeedsCheckRunOutput = statusChecks.some(
+    (c) => (isSonarUrl(c.detailsUrl) || c.name.toLowerCase().includes("sonar")) && !extractSonarProjectKey(c.detailsUrl)
+  );
+  const ownerRepo = failedChecks.length > 0 || sonarNeedsCheckRunOutput ? parseOwnerRepo() : null;
   const checkOutputs = ownerRepo ? fetchCheckRunOutputs(ownerRepo, raw.headRefOid) : /* @__PURE__ */ new Map();
-  const checks = (raw.statusCheckRollup ?? []).map((c) => {
+  const checks = statusChecks.map((c) => {
     const enriched = checkOutputs.get(c.name);
     return {
       name: c.name,
@@ -869,20 +902,25 @@ async function getPrInfoGh(branch) {
   const sonarCheck = checks.find((c) => isSonarUrl(c.detailsUrl));
   let sonarcloudUrl;
   let sonarNewIssues;
+  let sonarNewIssuesNote;
   let sonarFailures;
   if (sonarCheck) {
-    sonarcloudUrl = sonarCheck.detailsUrl;
-    const projectKey = extractSonarProjectKey(sonarCheck.detailsUrl);
+    sonarcloudUrl = resolveSonarPullRequestUrl(sonarCheck.detailsUrl, raw.number);
+    const projectKey = extractSonarProjectKey(sonarcloudUrl);
     if (projectKey) {
       if (sonarCheck.conclusion !== null && SONAR_FAIL_CONCLUSIONS.has(sonarCheck.conclusion)) {
         const sonarSummary = await fetchSonarFailureSummary(projectKey, raw.number, sonarcloudUrl);
         sonarFailures = sonarSummary.summary;
         sonarNewIssues = sonarSummary.issueTotal;
+        sonarNewIssuesNote = sonarSummary.issueNote;
       } else {
-        sonarNewIssues = await fetchSonarNewIssues(projectKey, raw.number);
+        const sonarNewIssuesResult = await fetchSonarNewIssues(projectKey, raw.number, sonarcloudUrl);
+        sonarNewIssues = sonarNewIssuesResult.total;
+        sonarNewIssuesNote = sonarNewIssuesResult.note;
       }
     } else {
       sonarNewIssues = null;
+      sonarNewIssuesNote = "Could not determine the SonarCloud project key from the check URL.";
     }
   }
   return {
@@ -891,7 +929,7 @@ async function getPrInfoGh(branch) {
     state: raw.state,
     url: raw.url,
     checks,
-    ...sonarcloudUrl === void 0 ? {} : { sonarcloudUrl, sonarNewIssues },
+    ...sonarcloudUrl === void 0 ? {} : { sonarcloudUrl, sonarNewIssues, sonarNewIssuesNote },
     ...sonarFailures === void 0 ? {} : { sonarFailures }
   };
 }
@@ -1221,6 +1259,12 @@ function formatSonarFailures(sonarFailures, sonarcloudUrl) {
   }
   return lines.join("\n") + "\n";
 }
+function sonarFailureNote(sonarFailures) {
+  if (!sonarFailures) return void 0;
+  if (sonarFailures.status === "private") return sonarFailures.privateMessage;
+  if (sonarFailures.status === "unavailable") return sonarFailures.unavailableMessage;
+  return void 0;
+}
 var POLL_INTERVAL_MS = 1e4;
 var getPrInfoCmd = new Command2("get-pr-info").description("Show pull request info for the current branch").option("--json", "Output as JSON").option("--wait-finish-checks", "Poll until all checks complete, then print the normal get-pr-info output").addHelpText(
   "after",
@@ -1290,6 +1334,11 @@ URL:   ${pr.url}
       const issueStr = pr.sonarNewIssues === null || pr.sonarNewIssues === void 0 ? "unavailable" : String(pr.sonarNewIssues);
       process.stdout.write(`Sonar New Issues: ${issueStr}
 `);
+      const failureNote = sonarFailureNote(pr.sonarFailures);
+      if (pr.sonarNewIssuesNote && pr.sonarNewIssuesNote !== failureNote) {
+        process.stdout.write(`Sonar Note: ${sanitizeText(pr.sonarNewIssuesNote)}
+`);
+      }
     }
     process.stdout.write(formatCheckSummary(pr.checks));
     process.stdout.write(formatChecks(pr.checks));
@@ -1889,16 +1938,15 @@ function readStdin() {
   return new Promise((resolve3, reject) => {
     const chunks = [];
     process.stdin.on("data", (chunk) => chunks.push(chunk));
-    process.stdin.on("end", () => resolve3(Buffer.concat(chunks).toString("utf8").trim()));
+    process.stdin.on("end", () => resolve3(Buffer.concat(chunks).toString("utf8")));
     process.stdin.on("error", reject);
   });
 }
 async function resolvePrompt(options) {
   const hasCli = options.prompt !== void 0;
   const hasFile = options.filePrompt !== void 0;
-  const hasStdin = !process.stdin.isTTY;
-  if (hasCli && (hasFile || hasStdin)) {
-    process.stderr.write("Error: --prompt is mutually exclusive with --file-prompt and stdin.\n");
+  if (hasCli && hasFile) {
+    process.stderr.write("Error: --prompt is mutually exclusive with --file-prompt.\n");
     process.exit(1);
   }
   if (hasCli) {
@@ -1914,9 +1962,9 @@ async function resolvePrompt(options) {
       process.exit(1);
     }
   }
-  if (hasStdin) {
+  if (!process.stdin.isTTY) {
     const content = await readStdin();
-    if (content.length === 0) {
+    if (content.trim().length === 0) {
       process.stderr.write("Error: No prompt provided. Use --prompt, --file-prompt, or pipe via stdin.\n");
       process.exit(1);
     }
@@ -1925,7 +1973,7 @@ async function resolvePrompt(options) {
   process.stderr.write("Error: No prompt provided. Use --prompt, --file-prompt, or pipe via stdin.\n");
   process.exit(1);
 }
-var executeCommand = new Command4("execute").description("Delegate work to an AI executor").requiredOption("--with <executor>", "Executor to use: claude or codex").option("--prompt <string>", "Prompt to send to the executor").option("--file-prompt <path>", "Path to a file whose content is used as the prompt").option("--silent", "Suppress step-by-step output; show only the final summary").option("--model <string>", "Model identifier to pass to the executor").action(async (options) => {
+var executeCommand = new Command4("execute").description("Delegate work to an AI executor").requiredOption("--with <executor>", "Executor to use: claude or codex").option("--prompt <string>", "Prompt to send to the executor").option("--file-prompt <path>", "Path to a file whose content is used as the prompt").option("--silent", "Suppress step-by-step Claude output; show only the final summary").option("--model <string>", "Model identifier to pass to the executor").action(async (options) => {
   const executor = options.with.toLowerCase();
   if (executor !== "claude" && executor !== "codex") {
     process.stderr.write(`Error: --with must be 'claude' or 'codex', got '${options.with}'.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "automata-cli",
-  "version": "0.1.0-feature-021-execute-command.125",
+  "version": "0.1.0-feature-021-execute-command.127",
   "description": "Automata CLI tool",
   "type": "module",
   "bin": {