@ganakailabs/cloudeval-cli 0.27.1 → 0.27.2

package/dist/{App-XHSK5SJA.js → App-VSOTHLOD.js}

@@ -38,10 +38,10 @@ import {
 } from "./chunk-USSCB2ZU.js";
 import {
   Banner
-} from "./chunk-LWHSK5SO.js";
+} from "./chunk-3DBCT463.js";
 import {
   CLI_VERSION
-} from "./chunk-FJDUSS2S.js";
+} from "./chunk-SLRYBS3P.js";
 import {
   raisedButtonStyle,
   terminalTheme

package/dist/{Banner-GAOI3CZR.js → Banner-7XUOJYL5.js}

@@ -3,8 +3,8 @@ import {
   bannerMetaColor,
   bannerSegmentColor,
   splitBannerLineSegments
-} from "./chunk-LWHSK5SO.js";
-import "./chunk-FJDUSS2S.js";
+} from "./chunk-3DBCT463.js";
+import "./chunk-SLRYBS3P.js";
 import "./chunk-ZDKRIOMB.js";
 export {
   Banner,

package/dist/{chunk-LWHSK5SO.js → chunk-3DBCT463.js}

@@ -1,6 +1,6 @@
 import {
   CLI_VERSION
-} from "./chunk-FJDUSS2S.js";
+} from "./chunk-SLRYBS3P.js";
 import {
   shouldUseColor,
   terminalTheme

package/dist/{chunk-FJDUSS2S.js → chunk-SLRYBS3P.js}

@@ -1,5 +1,5 @@
 // src/version.ts
-var CLI_VERSION = "0.27.1";
+var CLI_VERSION = "0.27.2";
 
 export {
   CLI_VERSION

package/dist/cli.js

@@ -39,7 +39,7 @@ import {
 } from "./chunk-USSCB2ZU.js";
 import {
   CLI_VERSION
-} from "./chunk-FJDUSS2S.js";
+} from "./chunk-SLRYBS3P.js";
 
 // src/runtime/prepareInk.ts
 import fs from "fs";
@@ -2506,6 +2506,14 @@ var firstRecord = (...values) => {
   }
   return void 0;
 };
+var firstArray = (...values) => {
+  for (const value of values) {
+    if (Array.isArray(value)) {
+      return value;
+    }
+  }
+  return void 0;
+};
 var entriesAsNamedRecords = (value, amountKey = "amount") => {
   const record = firstRecord(value);
   if (!record) {
@@ -2660,21 +2668,6 @@ var statusIcon = (status) => {
       return "\u26AA";
   }
 };
-var formatValidation = (validation) => {
-  const policyFailed = numberFrom(validation?.policyChecks?.failed);
-  const unitFailed = numberFrom(validation?.unitTests?.failed);
-  if (policyFailed === 0 && unitFailed === 0) {
-    return "Validation clean";
-  }
-  const parts = [];
-  if (unitFailed !== void 0) {
-    parts.push(unitFailed === 0 ? "unit tests clean" : `${unitFailed} unit tests failed`);
-  }
-  if (policyFailed !== void 0) {
-    parts.push(policyFailed === 0 ? "policy checks clean" : `${policyFailed} policy checks failed`);
-  }
-  return parts.length ? parts.join(", ") : "Validation not available";
-};
 var validationSummaryLine = (validation) => {
   const unitFailed = numberFrom(validation?.unitTests?.failed);
   if (unitFailed === void 0) {
@@ -2716,6 +2709,135 @@ var validationDetailLines = (validation) => {
   }
   return lines.length ? lines : ["- Validation data was not available."];
 };
+var failedValidationRecords = (records) => (Array.isArray(records) ? records : []).map(asRecord).filter((record) => {
+  const status = String(
+    record.status ?? record.outcome ?? record.result ?? record.state ?? ""
+  ).toLowerCase();
+  if (record.passed === false || record.success === false) {
+    return true;
+  }
+  return ["fail", "failed", "error", "critical"].includes(status);
+});
+var severityIcon = (severity) => {
+  switch (String(severity ?? "").toLowerCase()) {
+    case "critical":
+    case "error":
+    case "high":
+    case "fail":
+    case "failed":
+      return "\u{1F534}";
+    case "warning":
+    case "warn":
+    case "medium":
+      return "\u{1F7E1}";
+    case "info":
+    case "low":
+      return "\u{1F535}";
+    default:
+      return "\u26AA";
+  }
+};
+var compactMarkdownCell = (value, fallback = "not available") => {
+  const text = String(value ?? "").replace(/\s+/g, " ").replace(/\|/g, "\\|").trim();
+  return text || fallback;
+};
+var failureName = (record, fallback) => compactMarkdownCell(
+  record.test_name ?? record.testName ?? record.rule_name ?? record.ruleName ?? record.name ?? record.title ?? record.id ?? fallback
+);
+var failureLocation = (record) => compactMarkdownCell(
+  record.file_path ?? record.filePath ?? record.path ?? record.resource_id ?? record.resourceId ?? record.resource ?? record.target,
+  "-"
+);
+var failureWhy = (record) => {
+  const message = compactMarkdownCell(
+    record.message ?? record.reason ?? record.description ?? record.details,
+    ""
+  );
+  const recommendation = compactMarkdownCell(
+    record.recommendation ?? record.remediation ?? record.fix ?? record.next_step,
+    ""
+  );
+  if (message && recommendation && message !== recommendation) {
+    return `${message} ${recommendation}`;
+  }
+  return message || recommendation || "No failure reason was included in the report payload.";
+};
+var validationFailureRows = (validation) => {
+  const rows = [];
+  const unitFailures = (Array.isArray(validation?.unitTests?.failures) ? validation?.unitTests?.failures : []).map(asRecord).slice(0, 5);
+  const policyFailures = (Array.isArray(validation?.policyChecks?.failures) ? validation?.policyChecks?.failures : []).map(asRecord).slice(0, 5);
+  const groupedFailures = [
+    ["Unit test", unitFailures],
+    ["Policy check", policyFailures]
+  ];
+  for (const [kind, failures] of groupedFailures) {
+    failures.forEach((failure, index) => {
+      const severity = compactMarkdownCell(
+        failure.severity ?? failure.status ?? failure.outcome,
+        "failed"
+      );
+      rows.push(
+        `| ${kind} | ${failureName(failure, `${kind} ${index + 1}`)} | \`${failureLocation(failure)}\` | ${severityIcon(severity)} ${severity} | ${failureWhy(failure)} |`
+      );
+    });
+  }
+  return rows;
+};
+var architectureSignalLines = ({
+  architecture,
+  costServices,
+  costCurrency,
+  highRiskFindings,
+  pillars
+}) => {
+  const lines = [];
+  const resourceCount = numberFrom(architecture?.resources);
+  const resourceTypeCount = numberFrom(architecture?.resourceTypes);
+  const relationshipCount = numberFrom(architecture?.relationships);
+  const density = numberFrom(architecture?.relationshipDensity);
+  if (resourceCount !== void 0 && resourceTypeCount !== void 0) {
+    lines.push(
+      `- Scale: **${displayNumber(resourceCount)} resources** across **${displayNumber(resourceTypeCount)} resource types**`
+    );
+  } else if (resourceCount !== void 0) {
+    lines.push(`- Scale: **${displayNumber(resourceCount)} resources**`);
+  }
+  if (relationshipCount !== void 0) {
+    const densityText = density !== void 0 ? ` (**${trimNumber(density, 2)} per resource**)` : "";
+    const shape = density !== void 0 && density < 0.5 ? "; sparse dependency graph, review isolated resources and missing links" : density !== void 0 && density > 2 ? "; dense dependency graph, review blast radius before changes" : "";
+    lines.push(
+      `- Dependency shape: **${displayNumber(relationshipCount)} relationships**${densityText}${shape}`
+    );
+  }
+  const highRisk = numberFrom(highRiskFindings);
+  const weakestPillar = pillars.map((pillar) => ({
+    label: String(pillar.label ?? pillar.id ?? "pillar"),
+    score: numberFrom(pillar.score)
+  })).filter((pillar) => pillar.score !== void 0).sort((left, right) => left.score - right.score)[0];
+  if (highRisk !== void 0 || weakestPillar) {
+    const parts = [];
+    if (highRisk !== void 0) {
+      parts.push(`**${displayNumber(highRisk)} high-risk findings**`);
+    }
+    if (weakestPillar) {
+      const rating = scoreRating(weakestPillar.score);
+      parts.push(
+        `weakest pillar **${weakestPillar.label} ${formatScore(weakestPillar.score)} (${rating ?? "UNKNOWN"})**`
+      );
+    }
+    lines.push(`- Risk concentration: ${parts.join("; ")}`);
+  }
+  const topCostDrivers = costServices.filter((service) => service.amount > 0).sort((left, right) => right.amount - left.amount).slice(0, 2);
+  if (topCostDrivers.length) {
+    const total = topCostDrivers.reduce((sum, service) => sum + service.amount, 0);
+    lines.push(
+      `- Cost drivers: ${joinReadableList(
+        topCostDrivers.map((service) => `**${service.name}**`)
+      )} account for **${formatMonthlyMoney(total, costCurrency ?? topCostDrivers[0]?.currency)}**`
+    );
+  }
+  return lines;
+};
 var namedAmount = (record) => numberFrom(
   record.amount,
   record.monthly_cost,
@@ -2725,6 +2847,15 @@ var namedAmount = (record) => numberFrom(
 );
 var namedLabel = (record, fallback) => String(record.name ?? record.service ?? record.label ?? record.category ?? fallback);
 var mermaidLabel = (value) => value.replace(/"/g, "'");
+var joinReadableList = (values) => {
+  if (values.length <= 1) {
+    return values[0] ?? "";
+  }
+  if (values.length === 2) {
+    return `${values[0]} and ${values[1]}`;
+  }
+  return `${values.slice(0, -1).join(", ")}, and ${values[values.length - 1]}`;
+};
 var costServiceRows = (services, currency) => (Array.isArray(services) ? services : []).map((service, index) => {
   const record = asRecord(service);
   const amount = namedAmount(record);
@@ -2818,19 +2949,41 @@ var extractValidation = ({
     preload?.reports?.unit_tests?.metrics,
     project?.status?.unit_tests
   );
+  const unitFailures = failedValidationRecords(
+    firstArray(
+      preload?.latest_payloads?.unit_tests?.test_results,
+      preload?.latest_payloads?.unit_tests?.results,
+      preload?.reports?.unit_tests?.metrics?.test_results,
+      preload?.reports?.unit_tests?.metrics?.results,
+      project?.status?.unit_tests?.test_results,
+      project?.status?.unit_tests?.results
+    )
+  );
+  const policyFailures = failedValidationRecords(
+    firstArray(
+      policySummary?.results,
+      policySummary?.checks,
+      waf?.parsed?.validation_results,
+      waf?.parsed?.rules,
+      waf?.raw?.rules,
+      failedRules
+    )
+  );
   return {
     policyChecks: {
       total: numberFrom(policySummary?.total_rules, policySummary?.totalRules, rules.length),
       passed: numberFrom(policySummary?.passed_rules, policySummary?.passedRules),
       failed: numberFrom(policySummary?.failed_rules, policySummary?.failedRules, failedRules.length),
-      errors: numberFrom(policySummary?.error_rules, policySummary?.errorRules)
+      errors: numberFrom(policySummary?.error_rules, policySummary?.errorRules),
+      failures: policyFailures.slice(0, 5)
     },
     unitTests: {
       status: unitSummary?.status,
       total: numberFrom(unitSummary?.total_tests, unitSummary?.totalTests),
       passed: numberFrom(unitSummary?.passed_tests, unitSummary?.passedTests),
       failed: numberFrom(unitSummary?.failed_tests, unitSummary?.failedTests),
-      skipped: numberFrom(unitSummary?.skipped_tests, unitSummary?.skippedTests)
+      skipped: numberFrom(unitSummary?.skipped_tests, unitSummary?.skippedTests),
+      failures: unitFailures.slice(0, 5)
     }
   };
 };
@@ -3185,81 +3338,8 @@ var waitForReviewReports = async ({
   }
   return latest;
 };
-var buildAiSummaryPrompt = (data) => [
-  "Write a concise CloudEval pull request review summary in Markdown.",
-  "Return exactly two sections: Short Summary and Details.",
-  "Short Summary: one dense paragraph under 45 words with gate status, Well-Architected posture, validation, and cost.",
-  "Details: short bullets with bold labels only when useful, such as **Key risks:**, **Cost posture:**, and **Recommended next step:**.",
-  "Keep the full response under 180 words. Do not invent facts not present below.",
-  "Do not include citations, source markers, hidden tool ids, or HTML comments.",
-  "",
-  `Project: ${data.projectId}`,
-  `Repository: ${data.repo ?? "unknown"}`,
-  `Ref: ${data.ref ?? "unknown"}`,
-  `Commit: ${data.commitSha ?? "unknown"}`,
-  `Gate: ${String(data.gate?.status ?? "unknown").toUpperCase()}`,
-  `Well-Architected score: ${data.gate?.wellArchitected?.overall?.score ?? data.gate?.overallScore ?? "unknown"}`,
-  `High-risk findings: ${data.gate?.wellArchitected?.risks?.high ?? data.gate?.highRisk ?? "unknown"}`,
-  `Monthly cost: ${formatMoney(data.gate?.cost?.monthly?.amount ?? data.gate?.monthlyCost, data.gate?.cost?.monthly?.currency)}`,
-  `Validation: ${formatValidation(data.gate?.validation)}`,
-  Array.isArray(data.gate?.failures) && data.gate.failures.length ? `Gate failures: ${data.gate.failures.join("; ")}` : "Gate failures: none reported"
-].join("\n");
-var isTransientAiSummaryText = (text) => {
-  if (!text?.trim()) {
-    return false;
-  }
-  return /too many requests|rate[- ]?limit|try again in a moment|temporarily unavailable|something went wrong while processing|please try again or ask a different question|did not complete within/i.test(
-    text
-  );
-};
-var normalizeAiSummaryMarkdown = (text) => {
-  return normalizeAiSummarySections(text).markdown;
-};
-var normalizeAiSummarySections = (text) => {
-  const trimmed = String(text ?? "").trim();
-  if (!trimmed) {
-    return { shortSummary: "", detailsMarkdown: "", markdown: "" };
-  }
-  const withNewlines = sanitizeAiSummaryMarkdown(trimmed.replace(/\\n/g, "\n"));
-  const requestMatch = /\bRequest:\s*/i.exec(withNewlines);
-  const sourceText = requestMatch ? withNewlines.slice(requestMatch.index + requestMatch[0].length).split(/\n\s*\nThe request reached\b/i)[0].trim() || withNewlines : withNewlines;
-  const sections = splitAiSummarySections(sourceText);
-  return {
-    ...sections,
-    markdown: renderAiSummarySections(sections.shortSummary, sections.detailsMarkdown)
-  };
-};
-var sanitizeAiSummaryMarkdown = (text) => text.replace(/<!--[\s\S]*?-->/g, "").replace(/\s*\[S_[A-Za-z0-9_:-]+\]/g, "").replace(/\n{3,}/g, "\n\n").trim();
-var stripAiSectionLabel = (text, label) => text.replace(new RegExp(`^\\s*#{0,6}\\s*\\*\\*?${label}\\*\\*?\\s*:?\\s*`, "i"), "").replace(new RegExp(`^\\s*${label}\\s*:?\\s*`, "i"), "").trim();
-var normalizeAiDetailHeadings = (text) => text.split("\n").map((line) => {
-  const cleaned = line.replace(/^\s*#{1,6}\s*/, "").replace(/^\s*[-*]\s+/, "").trimEnd();
-  const labelMatch = cleaned.match(/^\*\*(Key risks|Cost posture|Recommended next step|Recommendation|Validation|Impact):\*\*\s*(.*)$/i) ?? cleaned.match(/^\*\*(Key risks|Cost posture|Recommended next step|Recommendation|Validation|Impact)\*\*\s*:\s*(.*)$/i) ?? cleaned.match(/^(Key risks|Cost posture|Recommended next step|Recommendation|Validation|Impact)\s*:\s*(.*)$/i);
-  if (!labelMatch) {
-    return cleaned;
-  }
-  return `- **${labelMatch[1]}:** ${labelMatch[2]}`.trimEnd();
-}).join("\n").trim();
-var splitAiSummarySections = (text) => {
-  const cleaned = sanitizeAiSummaryMarkdown(text).replace(/^\s*#{1,6}\s*AI summary\s*$/gim, "").trim();
-  const shortMatch = /\bShort Summary\s*:\s*/i.exec(cleaned);
-  const detailsMatch = /\bDetails\s*:\s*/i.exec(cleaned);
-  if (shortMatch && detailsMatch && shortMatch.index < detailsMatch.index) {
-    const shortSummary2 = stripAiSectionLabel(
-      cleaned.slice(shortMatch.index, detailsMatch.index).trim(),
-      "Short Summary"
-    );
-    const detailsMarkdown2 = normalizeAiDetailHeadings(
-      stripAiSectionLabel(cleaned.slice(detailsMatch.index).trim(), "Details")
-    );
-    return { shortSummary: shortSummary2, detailsMarkdown: detailsMarkdown2 };
-  }
-  const paragraphs = cleaned.split(/\n\s*\n/).map((paragraph) => paragraph.trim()).filter(Boolean);
-  const shortSummary = stripAiSectionLabel(paragraphs[0] ?? cleaned, "Short Summary");
-  const detailsMarkdown = normalizeAiDetailHeadings(paragraphs.slice(1).join("\n\n"));
-  return { shortSummary, detailsMarkdown };
-};
 var renderAiSummarySections = (shortSummary, detailsMarkdown) => {
-  const lines = [`**Short summary:** ${shortSummary.trim()}`];
+  const lines = [shortSummary.trim()];
   if (detailsMarkdown.trim()) {
     lines.push(
       "",
@@ -3273,195 +3353,108 @@ var renderAiSummarySections = (shortSummary, detailsMarkdown) => {
   }
   return lines.join("\n");
 };
-var aiSummaryAttemptTimeoutMs = () => {
-  const raw = process.env.CLOUDEVAL_REVIEW_AI_ATTEMPT_TIMEOUT_MS;
-  if (raw?.trim()) {
-    const parsed = Number.parseInt(raw.trim(), 10);
-    if (Number.isFinite(parsed) && parsed > 0) {
-      return parsed;
-    }
-  }
-  return 18e4;
-};
-var aiSummaryRetryDelaysMs = () => {
-  const raw = process.env.CLOUDEVAL_REVIEW_AI_RETRY_DELAYS_MS;
-  if (raw?.trim()) {
-    return raw.split(",").map((value) => Number.parseInt(value.trim(), 10)).filter((value) => Number.isFinite(value) && value >= 0);
+var githubWorkflowRunUrl = () => {
+  const server = process.env.GITHUB_SERVER_URL;
+  const repo = process.env.GITHUB_REPOSITORY;
+  const runId = process.env.GITHUB_RUN_ID;
+  if (server && repo && runId) {
+    return `${server.replace(/\/$/, "")}/${repo}/actions/runs/${runId}`;
   }
-  return [5e3, 15e3];
+  return void 0;
 };
-var generateAiSummaryAttempt = async ({
-  baseUrl,
-  token,
-  user,
-  project,
-  model,
-  mode,
-  agentProfileId,
-  data
-}) => {
-  const core = await import("./dist-PEYJDO7A.js");
-  const threadId = `review-${data.projectId}-${Date.now()}`;
-  let markdown = "";
-  let chatState = { ...core.initialChatState, threadId };
-  const attemptTimeoutMs = aiSummaryAttemptTimeoutMs();
-  let timedOut = false;
-  let timeoutId;
-  const timeoutPromise = new Promise((_, reject) => {
-    timeoutId = setTimeout(() => {
-      timedOut = true;
-      reject(
-        new Error(`AI summary did not complete within ${attemptTimeoutMs}ms.`)
-      );
-    }, attemptTimeoutMs);
-    timeoutId.unref?.();
-  });
-  const iterator = core.streamChat({
-    baseUrl,
-    authToken: token,
-    message: buildAiSummaryPrompt(data),
-    threadId,
-    user: {
-      id: String(project?.user_id ?? user?.id ?? "cli-user"),
-      name: String(user?.full_name ?? user?.name ?? user?.email ?? "CloudEval CI")
-    },
-    project: project ? {
-      id: String(project.id ?? data.projectId),
-      name: String(project.name ?? data.projectId),
-      user_id: typeof project.user_id === "string" ? project.user_id : void 0,
-      cloud_provider: typeof project.cloud_provider === "string" ? project.cloud_provider : void 0,
-      type: typeof project.type === "string" ? project.type : void 0,
-      connection_ids: Array.isArray(project.connection_ids) ? project.connection_ids : void 0
-    } : {
-      id: String(data.projectId),
-      name: String(data.projectId)
-    },
-    settings: {
-      mode,
-      ...model ? { model } : {}
-    },
-    ...mode === "agent" ? { agentProfileId: agentProfileId ?? "architecture" } : {},
-    completeAfterResponse: true,
-    responseCompletionGraceMs: 250,
-    streamIdleTimeoutMs: 12e4
-  });
-  try {
-    while (true) {
-      const next = await Promise.race([iterator.next(), timeoutPromise]);
-      if (next.done) {
-        break;
-      }
-      const chunk = next.value;
-      chatState = core.reduceChunk(chatState, chunk);
-      const latestMessage = [...chatState.messages ?? []].reverse().find((message) => message.role === "assistant");
-      const chunkAssistantMessage = Array.isArray(chunk?.messages) ? [...chunk.messages].reverse().find((message) => message?.role === "assistant" && typeof message?.content === "string")?.content : void 0;
-      const content = chunk?.content;
-      if (typeof chunkAssistantMessage === "string" && chunkAssistantMessage.trim()) {
-        markdown = chunkAssistantMessage;
-      }
-      if (chunk.type === "responding" && typeof content === "string") {
-        markdown = latestMessage?.content || `${markdown}${content}`;
-      }
-    }
-  } finally {
-    if (timeoutId) {
-      clearTimeout(timeoutId);
-    }
-    if (timedOut) {
-      await iterator.return?.(void 0).catch(() => void 0);
-    }
-  }
-  const finalMessage = [...chatState.messages ?? []].reverse().find((message) => message.role === "assistant");
-  const rawMarkdown = String(finalMessage?.content || markdown).trim();
-  const normalized = normalizeAiSummarySections(rawMarkdown);
+var reviewSurface = () => {
+  const event = String(process.env.GITHUB_EVENT_NAME ?? "").toLowerCase();
+  const ref = String(process.env.GITHUB_REF ?? "").toLowerCase();
+  return event.startsWith("pull_request") || ref.startsWith("refs/pull/") ? "pull_request" : "local_review";
+};
+var buildReviewSummaryPayload = (data) => ({
+  source: process.env.GITHUB_ACTIONS === "true" ? "github_action" : "cli",
+  surface: reviewSurface(),
+  project: data.project ?? { id: data.projectId },
+  repository: { full_name: data.repo },
+  ref: data.ref,
+  commit_sha: data.commitSha,
+  workflow_run_url: githubWorkflowRunUrl(),
+  gate_result: {
+    status: data.gate?.status,
+    failures: data.gate?.failures ?? [],
+    thresholds: data.gate?.thresholds ?? {},
+    enforcement: data.gate?.enforcement
+  },
+  well_architected: data.gate?.wellArchitected ?? {},
+  cost: data.gate?.cost ?? {},
+  validation: data.gate?.validation ?? {},
+  policy: data.gate?.validation?.policy ?? data.gate?.policy ?? {},
+  architecture_signals: data.gate?.architecture ?? {},
+  changed_files: data.changedFiles ?? []
+});
+var deterministicAiSummary = (data, error) => {
+  const score = data.gate?.wellArchitected?.overall?.score ?? data.gate?.overallScore;
+  const rating = scoreRating(score) ?? "UNKNOWN";
+  const validation = data.gate?.validation ?? {};
+  const cost = data.gate?.cost?.monthly ?? {};
+  const failedTests = numberFrom(validation?.unitTests?.failed) ?? numberFrom(validation?.unit_tests?.failed) ?? numberFrom(validation?.failedUnitTests) ?? numberFrom(validation?.failed_tests) ?? 0;
+  const policyFailed = numberFrom(validation?.policyChecks?.failed) ?? numberFrom(validation?.policy_checks?.failed) ?? numberFrom(data.gate?.policy?.failed) ?? 0;
+  const policyStatus = policyFailed > 0 ? "has failed checks" : "GOOD";
+  const summary = [
+    `CloudEval review completed with ${String(data.gate?.status ?? "UNKNOWN").toUpperCase()}.`,
+    `Well-Architected posture is ${formatScore(score)} (${rating}), validation has ${displayNumber(failedTests)} failed unit tests, policy checks are ${policyStatus}, and monthly cost is ${formatMonthlyMoney(cost?.amount, cost?.currency)}.`,
+    "Prioritize failed validation checks and the weakest Well-Architected pillar first."
+  ].join(" ");
   return {
     enabled: true,
-    status: "ok",
-    mode,
-    ...mode === "agent" ? { agentProfileId: agentProfileId ?? "architecture" } : {},
-    ...model ? { model } : {},
-    ...normalized,
-    threadId
+    status: "fallback",
+    fallbackUsed: true,
+    warnings: error ? [`Review summary endpoint failed: ${error}`] : [],
+    shortSummary: summary,
+    detailsMarkdown: [
+      "**Main risk**\nCloudEval could not produce an AI-written review summary, so use the deterministic gate evidence.",
+      "**Why it matters**\nFailed validation and weak architecture pillars are the highest-signal remediation inputs.",
+      "**Recommended actions**\nFix failed validation checks, address the weakest pillar, rerun CloudEval review, and compare the updated gate.",
+      "**Evidence used**\nGate status, Well-Architected score, validation totals, policy totals, and monthly cost."
+    ].join("\n\n"),
+    markdown: renderAiSummarySections(
+      summary,
+      [
+        "**Main risk**\nCloudEval could not produce an AI-written review summary, so use the deterministic gate evidence.",
+        "**Why it matters**\nFailed validation and weak architecture pillars are the highest-signal remediation inputs.",
+        "**Recommended actions**\nFix failed validation checks, address the weakest pillar, rerun CloudEval review, and compare the updated gate.",
+        "**Evidence used**\nGate status, Well-Architected score, validation totals, policy totals, and monthly cost."
+      ].join("\n\n")
+    )
   };
 };
 var generateAiSummary = async (input) => {
-  const retryDelays = aiSummaryRetryDelaysMs();
-  let lastResult;
-  let activeInput = input;
-  let fallbackFromMode;
-  for (let attemptIndex = 0; attemptIndex <= retryDelays.length; attemptIndex += 1) {
-    let result;
-    try {
-      result = await generateAiSummaryAttempt(activeInput);
-    } catch (error) {
-      const message = error?.message ?? "AI summary failed";
-      if (!isTransientAiSummaryText(message)) {
-        throw error;
-      }
-      result = {
-        enabled: true,
-        status: "transient_error",
-        mode: activeInput.mode,
-        ...activeInput.mode === "agent" ? { agentProfileId: activeInput.agentProfileId ?? "architecture" } : {},
-        ...activeInput.model ? { model: activeInput.model } : {},
-        markdown: message,
-        error: message
-      };
-    }
-    result.attempts = attemptIndex + 1;
-    if (fallbackFromMode) {
-      result.fallbackFromMode = fallbackFromMode;
-    }
-    const normalizedSummary = typeof result.shortSummary === "string" || typeof result.detailsMarkdown === "string" ? {
-      shortSummary: String(result.shortSummary ?? "").trim(),
-      detailsMarkdown: String(result.detailsMarkdown ?? "").trim(),
-      markdown: renderAiSummarySections(
-        String(result.shortSummary ?? "").trim(),
-        String(result.detailsMarkdown ?? "").trim()
-      )
-    } : normalizeAiSummarySections(result.markdown);
-    result = { ...result, ...normalizedSummary };
-    if (isTransientAiSummaryText(result.markdown)) {
-      const normalizedError = normalizeAiSummaryMarkdown(result.error);
-      if (normalizedError && !isTransientAiSummaryText(normalizedError)) {
-        result = {
-          ...result,
-          ...normalizeAiSummarySections(normalizedError)
-        };
-        result.status = "ok";
-        delete result.error;
-      }
-    }
-    lastResult = result;
-    if (!isTransientAiSummaryText(result.markdown)) {
-      return result;
-    }
-    const retryDelay = retryDelays[attemptIndex];
-    if (retryDelay === void 0) {
-      break;
-    }
-    await sleep2(retryDelay);
-    if (activeInput.mode === "agent") {
-      fallbackFromMode = "agent";
-      activeInput = {
-        ...input,
-        mode: "ask",
-        agentProfileId: void 0
-      };
+  try {
+    const payload = buildReviewSummaryPayload(input.data);
+    const response = await fetchCloudEvalJson({
+      baseUrl: input.baseUrl,
+      authToken: input.token,
+      path: `/projects/${encodeURIComponent(String(input.data.projectId))}/review/summary`,
+      method: "POST",
+      body: payload,
+      idempotencyKey: `cloudeval-review-summary-${input.data.projectId}-${input.data.commitSha ?? "head"}`
+    });
+    const shortSummary = String(response.summary ?? "").trim();
+    const detailsMarkdown = String(response.details ?? "").trim();
+    if (!shortSummary) {
+      return deterministicAiSummary(input.data, "Review summary endpoint returned no summary.");
     }
+    return {
+      enabled: true,
+      status: response.fallback_used ? "fallback" : "ok",
+      fallbackUsed: Boolean(response.fallback_used),
+      warnings: Array.isArray(response.warnings) ? response.warnings : [],
+      riskHighlights: Array.isArray(response.risk_highlights) ? response.risk_highlights : [],
+      recommendedActions: Array.isArray(response.recommended_actions) ? response.recommended_actions : [],
+      evidenceUsed: Array.isArray(response.evidence_used) ? response.evidence_used : [],
+      shortSummary,
+      detailsMarkdown,
+      markdown: renderAiSummarySections(shortSummary, detailsMarkdown)
+    };
+  } catch (error) {
+    return deterministicAiSummary(input.data, error?.message ?? "request failed");
   }
-  return {
-    ...lastResult ?? {},
-    enabled: true,
-    status: "unavailable",
-    mode: activeInput.mode,
-    ...activeInput.mode === "agent" ? { agentProfileId: activeInput.agentProfileId ?? "architecture" } : {},
-    ...input.model ? { model: input.model } : {},
-    ...fallbackFromMode ? { fallbackFromMode } : {},
-    attempts: lastResult?.attempts ?? retryDelays.length + 1,
-    error: lastResult?.markdown || "AI summary unavailable",
-    markdown: "AI summary unavailable: CloudEval AI was rate-limited. Retry the workflow or rerun `cloudeval review`."
-  };
 };
 var buildMarkdownSummary = (data) => {
   const gateStatus = String(data.gate?.status ?? "unknown").toUpperCase();
@@ -3471,7 +3464,8 @@ var buildMarkdownSummary = (data) => {
   const architecture = data.gate?.architecture;
   const projectLabel = String(data.project?.name ?? data.projectName ?? data.projectId);
   const projectDisplay = markdownLink(projectLabel, data.project?.url ?? data.projectUrl);
-  const source = data.repo ? `${data.repo}${data.ref ? ` @ ${data.ref}` : ""}` : data.ref ?? "unknown source";
+  const repository = String(data.repo ?? "unknown repository");
+  const ref = String(data.ref ?? "unknown ref");
   const commit = String(data.commitSha ?? "unknown").slice(0, 12);
   const pillarLines = Array.isArray(data.gate?.wellArchitected?.pillars) ? data.gate.wellArchitected.pillars.map((pillar) => {
     const rating = scoreRating(pillar.score);
@@ -3488,24 +3482,14 @@ var buildMarkdownSummary = (data) => {
     cost?.currency
   );
   const positiveCostServices = costServices.filter((service) => service.amount > 0);
-  const architectureLines = [
-    ["Resources", architecture?.resources],
-    ["Relationships", architecture?.relationships],
-    ["Resource types", architecture?.resourceTypes]
-  ].filter(([, value]) => numberFrom(value) !== void 0).map(([label, value]) => `- ${label}: **${displayNumber(value)}**`);
-  const density = numberFrom(architecture?.relationshipDensity);
-  if (density !== void 0) {
-    architectureLines.push(
-      `- Graph connectivity: **${trimNumber(density, 2)} relationships per resource**`
-    );
-  }
-  const resourceCount = numberFrom(architecture?.resources);
-  const resourceTypeCount = numberFrom(architecture?.resourceTypes);
-  if (resourceCount !== void 0 && resourceTypeCount !== void 0) {
-    architectureLines.push(
-      `- Resource diversity: **${displayNumber(resourceTypeCount)} types across ${displayNumber(resourceCount)} resources**`
-    );
-  }
+  const architectureLines = architectureSignalLines({
+    architecture,
+    costServices,
+    costCurrency: cost?.currency,
+    highRiskFindings: data.gate?.wellArchitected?.risks?.high,
+    pillars: Array.isArray(data.gate?.wellArchitected?.pillars) ? data.gate.wellArchitected.pillars : []
+  });
+  const validationRows = validationFailureRows(validation);
   const overallRating = scoreRating(score);
   const lines = [
     `${statusIcon(data.gate?.status)} **Overall** : ${gateStatus}`,
@@ -3513,9 +3497,13 @@ var buildMarkdownSummary = (data) => {
     validationSummaryLine(validation),
     policySummaryLine(validation),
     costSummaryLine(cost),
-    `**Cloudeval Project**: ${projectDisplay}`,
     "",
-    `Source: \`${source}\` \xB7 commit \`${commit}\``
+    "#### Source",
+    "",
+    `- **CloudEval project**: ${projectDisplay}`,
+    `- **Repository**: \`${repository}\``,
+    `- **Ref**: \`${ref}\``,
+    `- **Commit**: \`${commit}\``
   ];
   if (data.aiSummary?.markdown) {
     lines.push("", "#### AI summary", "", data.aiSummary.markdown);
@@ -3582,9 +3570,15 @@ var buildMarkdownSummary = (data) => {
     lines.push(
       "",
       "<details>",
-      "<summary>Validation details</summary>",
+      `<summary>${validationRows.length ? "Validation failures" : "Validation details"}</summary>`,
       "",
       ...validationDetailLines(validation),
+      ...validationRows.length ? [
+        "",
+        "| Type | Name | Location | Severity | Why / next step |",
+        "| --- | --- | --- | --- | --- |",
+        ...validationRows
+      ] : [],
       "",
       "</details>"
     );
@@ -3593,7 +3587,7 @@ var buildMarkdownSummary = (data) => {
     lines.push(
       "",
       "<details>",
-      "<summary>Architecture insights</summary>",
+      "<summary>Architecture signals</summary>",
       "",
       ...architectureLines,
       "",
@@ -15632,7 +15626,7 @@ program.command("tui").description("Open the CloudEval Terminal UI").option(
   const { assertSecureBaseUrl } = await import("./dist-PEYJDO7A.js");
   const [{ render }, { App }] = await Promise.all([
     import("ink"),
-    import("./App-XHSK5SJA.js")
+    import("./App-VSOTHLOD.js")
   ]);
   const baseUrl = await resolveBaseUrl(options, command);
   assertSecureBaseUrl(baseUrl);
@@ -15690,7 +15684,7 @@ program.command("chat").description("Start an interactive chat session").option(
   const { assertSecureBaseUrl } = await import("./dist-PEYJDO7A.js");
   const [{ render }, { App }] = await Promise.all([
     import("ink"),
-    import("./App-XHSK5SJA.js")
+    import("./App-VSOTHLOD.js")
   ]);
   const baseUrl = await resolveBaseUrl(options, command);
   assertSecureBaseUrl(baseUrl);
@@ -16444,7 +16438,7 @@ Error: ${errorMsg}
 program.command("banner").description("Preview the startup banner and terminal capabilities").action(async () => {
   const { render } = await import("ink");
   const BannerPreview = React.lazy(async () => ({
-    default: (await import("./Banner-GAOI3CZR.js")).Banner
+    default: (await import("./Banner-7XUOJYL5.js")).Banner
   }));
   render(
     /* @__PURE__ */ jsx(React.Suspense, { fallback: null, children: /* @__PURE__ */ jsx(BannerPreview, { disable: false }) })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ganakailabs/cloudeval-cli",
-  "version": "0.27.1",
+  "version": "0.27.2",
   "license": "LicenseRef-CloudEval-CLI",
   "type": "module",
   "description": "CloudEval CLI for cloud architecture, cost, report, automation, and MCP workflows.",

package/sbom.spdx.json

@@ -14,7 +14,7 @@
     {
       "SPDXID": "SPDXRef-Package-CloudEval-CLI",
       "name": "CloudEval CLI",
-      "versionInfo": "0.27.1",
+      "versionInfo": "0.27.2",
       "downloadLocation": "https://github.com/ganakailabs/cloudeval-cli",
       "filesAnalyzed": false,
       "licenseConcluded": "LicenseRef-CloudEval-CLI",