@ganakailabs/cloudeval-cli 0.27.2 → 0.28.0

package/THIRD_PARTY_NOTICES.md

@@ -235,7 +235,7 @@ This notice is not a substitute for legal review before public or enterprise dis
 | safer-buffer | 2.1.2 | MIT | Nikita Skovoroda | https://github.com/ChALkeR/safer-buffer#readme |
 | scheduler | 0.23.2 | MIT | NOASSERTION | https://reactjs.org/ |
 | semver | 7.7.3 | ISC | GitHub Inc. | https://github.com/npm/node-semver#readme |
-| shell-quote | 1.8.3 | MIT | James Halliday | https://github.com/ljharb/shell-quote |
+| shell-quote | 1.8.4 | MIT | James Halliday | https://github.com/ljharb/shell-quote |
 | signal-exit | 3.0.7 | ISC | Ben Coe | https://github.com/tapjs/signal-exit |
 | skin-tone | 2.0.0 | MIT | Sindre Sorhus | https://github.com/sindresorhus/skin-tone#readme |
 | slice-ansi | 5.0.0 | MIT | NOASSERTION | https://github.com/chalk/slice-ansi#readme |

package/dist/{App-VSOTHLOD.js → App-3RDW53BW.js}

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

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

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

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

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

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

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

package/dist/cli.js

@@ -39,7 +39,7 @@ import {
 } from "./chunk-USSCB2ZU.js";
 import {
   CLI_VERSION
-} from "./chunk-SLRYBS3P.js";
+} from "./chunk-JFJQZGZH.js";
 
 // src/runtime/prepareInk.ts
 import fs from "fs";
@@ -2842,10 +2842,14 @@ var namedAmount = (record) => numberFrom(
   record.amount,
   record.monthly_cost,
   record.monthlyCost,
+  record.monthly_cost_estimate,
+  record.monthlyCostEstimate,
   record.cost,
   record.value
 );
-var namedLabel = (record, fallback) => String(record.name ?? record.service ?? record.label ?? record.category ?? fallback);
+var namedLabel = (record, fallback) => String(
+  record.name ?? record.resource_name ?? record.resourceName ?? record.service ?? record.label ?? record.category ?? fallback
+);
 var mermaidLabel = (value) => value.replace(/"/g, "'");
 var joinReadableList = (values) => {
   if (values.length <= 1) {
@@ -2869,6 +2873,7 @@ var costServiceRows = (services, currency) => (Array.isArray(services) ? service
     ...rowCurrency ? { currency: rowCurrency } : {}
   };
 }).filter((service) => service !== void 0);
+var sortedPositiveCostRows = (rows) => rows.filter((row) => row.amount > 0).sort((left, right) => right.amount - left.amount);
 var reconcileCostServiceRows = (services, totalAmount, currency) => {
   const total = numberFrom(totalAmount);
   if (total === void 0 || services.length === 0) {
@@ -2897,7 +2902,80 @@ var reconcileCostServiceRows = (services, totalAmount, currency) => {
   }
   return services;
 };
+var compactCostRowsForChart = (rows, totalAmount, currency, options = {}) => {
+  const maxRows = options.maxRows ?? 5;
+  const remainderLabel = options.remainderLabel ?? "Unallocated";
+  const total = numberFrom(totalAmount);
+  const positiveRows = sortedPositiveCostRows(rows);
+  const selected = positiveRows.slice(0, maxRows);
+  const collapsed = positiveRows.slice(maxRows);
+  const collapsedSum = Number(
+    collapsed.reduce((sum, row) => sum + row.amount, 0).toFixed(3)
+  );
+  const result = [...selected];
+  if (collapsedSum > 1e-3) {
+    result.push({
+      name: "Other",
+      amount: collapsedSum,
+      ...currency ? { currency } : {}
+    });
+  }
+  const represented = result.reduce((sum, row) => sum + row.amount, 0);
+  if (total !== void 0) {
+    const delta = Number((total - represented).toFixed(3));
+    if (delta > 1e-3) {
+      result.push({
+        name: remainderLabel,
+        amount: delta,
+        ...currency ? { currency } : {}
+      });
+    }
+  }
+  return result;
+};
 var markdownLink = (label, url) => url ? `[${label}](${url})` : label;
+var openInCloudEvalLines = (links) => {
+  if (!links) {
+    return [];
+  }
+  const reports = asRecord(links.reports);
+  const downloads = asRecord(links.downloads);
+  const entries = [
+    ["Project preview", links.project],
+    ["Architecture report", reports.architecture],
+    ["Cost report", reports.cost],
+    ["Validation details", reports.validation],
+    ["Download PDF", downloads.pdf],
+    ["Workflow run", links.workflowRun],
+    ["Download review artifacts", downloads.reviewArtifacts]
+  ].filter((entry) => typeof entry[1] === "string" && entry[1].length > 0);
+  return entries.map(([label, url]) => `- ${markdownLink(label, url)}`);
+};
+var monthlyCostImpactLines = (currentAmount, savingsAmount, currency) => {
+  const current = numberFrom(currentAmount);
+  const savings = numberFrom(savingsAmount);
+  if (current === void 0 || current <= 0 || savings === void 0 || savings <= 0) {
+    return [];
+  }
+  const optimized = Math.max(current - savings, 0);
+  const savingsPercent = current > 0 ? savings / current * 100 : void 0;
+  const yMax = Math.max(current, optimized, savings);
+  return [
+    "| Metric | Amount |",
+    "| --- | ---: |",
+    `| Current monthly cost | **${formatMonthlyMoney(current, currency)}** |`,
+    `| Potential savings | **${formatMonthlyMoney(savings, currency)}${savingsPercent !== void 0 ? ` (${trimNumber(savingsPercent, 1)}%)` : ""}** |`,
+    `| Optimized monthly cost | **${formatMonthlyMoney(optimized, currency)}** |`,
+    "",
+    "```mermaid",
+    "xychart-beta",
+    '  title "Monthly cost impact"',
+    '  x-axis ["Current", "Optimized"]',
+    `  y-axis "${currency ? `${currency}/mo` : "monthly cost"}" 0 --> ${trimNumber(yMax, 3)}`,
+    `  bar [${trimNumber(current, 3)}, ${trimNumber(optimized, 3)}]`,
+    "```"
+  ];
+};
 var extractPillars = (waf) => {
   const fromArray = waf?.parsed?.score?.pillars;
   if (Array.isArray(fromArray)) {
@@ -3125,6 +3203,16 @@ var evaluateGate = ({
       currency
     },
     topServices: (Array.isArray(cost?.parsed?.serviceGroups) ? cost.parsed.serviceGroups : entriesAsNamedRecords(cost?.report?.processed?.cost_by_service_family)).slice(0, 5),
+    topResources: (firstArray(
+      cost?.report?.raw?.resource_estimates,
+      cost?.report?.raw?.resourceEstimates,
+      cost?.raw?.resource_estimates,
+      cost?.raw?.resourceEstimates,
+      cost?.parsed?.resourceEstimates,
+      cost?.parsed?.resource_estimates,
+      preload?.latest_payloads?.cost?.raw?.resource_estimates,
+      preload?.latest_payloads?.cost?.raw?.resourceEstimates
+    ) ?? []).slice(0, 20),
     recommendations: (Array.isArray(cost?.parsed?.recommendations) ? cost.parsed.recommendations : Array.isArray(cost?.report?.processed?.optimization_recommendations) ? cost.report.processed.optimization_recommendations : []).slice(0, 5)
   };
   if (!gateConfig) {
@@ -3397,9 +3485,9 @@ var deterministicAiSummary = (data, error) => {
   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."
+    `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,
@@ -3409,17 +3497,17 @@ var deterministicAiSummary = (data, 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."
+      "**Why it matters**\n**Failed 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**\n**Gate 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."
+        "**Why it matters**\n**Failed 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**\n**Gate status**, **Well-Architected score**, **validation totals**, **policy totals**, and **monthly cost**."
       ].join("\n\n")
     )
   };
@@ -3481,7 +3569,14 @@ var buildMarkdownSummary = (data) => {
     cost?.amount,
     cost?.currency
   );
-  const positiveCostServices = costServices.filter((service) => service.amount > 0);
+  const resourceCostRows = compactCostRowsForChart(
+    costServiceRows(data.gate?.cost?.topResources, cost?.currency),
+    cost?.amount,
+    cost?.currency,
+    { maxRows: 5, remainderLabel: "Unallocated" }
+  );
+  const positiveResourceCosts = resourceCostRows.filter((resource) => resource.amount > 0);
+  const openLinks = openInCloudEvalLines(data.links);
   const architectureLines = architectureSignalLines({
     architecture,
     costServices,
@@ -3505,6 +3600,9 @@ var buildMarkdownSummary = (data) => {
     `- **Ref**: \`${ref}\``,
     `- **Commit**: \`${commit}\``
   ];
+  if (openLinks.length) {
+    lines.push("", "#### Open in CloudEval", "", ...openLinks);
+  }
   if (data.aiSummary?.markdown) {
     lines.push("", "#### AI summary", "", data.aiSummary.markdown);
   }
@@ -3528,11 +3626,29 @@ var buildMarkdownSummary = (data) => {
   }
   if (cost?.amount !== void 0 || cost?.threshold !== void 0) {
     const costLines = [];
-    if (data.gate?.cost?.estimatedSavings?.amount !== void 0) {
+    const impactLines = monthlyCostImpactLines(
+      cost?.amount,
+      data.gate?.cost?.estimatedSavings?.amount,
+      cost?.currency ?? data.gate?.cost?.estimatedSavings?.currency
+    );
+    if (impactLines.length) {
+      costLines.push(...impactLines);
+    } else if (data.gate?.cost?.estimatedSavings?.amount !== void 0) {
       costLines.push(
         `- Estimated savings: **${formatMonthlyMoney(data.gate.cost.estimatedSavings.amount, data.gate.cost.estimatedSavings.currency)}**`
       );
     }
+    if (positiveResourceCosts.length) {
+      costLines.push(
+        "",
+        "```mermaid",
+        "pie title Monthly cost by resource",
+        ...positiveResourceCosts.map(
+          (resource) => `  "${mermaidLabel(resource.name)}" : ${trimNumber(resource.amount, 3)}`
+        ),
+        "```"
+      );
+    }
     if (costServices.length) {
       costLines.push(
         "",
@@ -3543,17 +3659,6 @@ var buildMarkdownSummary = (data) => {
         )
       );
     }
-    if (positiveCostServices.length) {
-      costLines.push(
-        "",
-        "```mermaid",
-        "pie title Monthly cost by service",
-        ...positiveCostServices.map(
-          (service) => `  "${mermaidLabel(service.name)}" : ${trimNumber(service.amount, 3)}`
-        ),
-        "```"
-      );
-    }
     if (costLines.length) {
       lines.push(
         "",
@@ -3673,18 +3778,58 @@ var registerReviewCommand = (program2, deps) => {
         }),
         readConfigText(cwd, options)
       ]);
+      const frontendBaseUrl = resolveFrontendBaseUrl({ apiBaseUrl: context.baseUrl });
+      const projectUrl = buildFrontendUrl({
+        baseUrl: frontendBaseUrl,
+        target: "project",
+        projectId,
+        view: "preview",
+        layout: "architecture"
+      });
+      const workflowRunUrl = githubWorkflowRunUrl();
       const data = {
         projectId,
         project: {
           id: projectId,
           name: String(project?.name ?? projectId),
-          url: buildFrontendUrl({
-            baseUrl: resolveFrontendBaseUrl({ apiBaseUrl: context.baseUrl }),
-            target: "project",
-            projectId,
-            view: "preview",
-            layout: "architecture"
-          })
+          url: projectUrl
+        },
+        links: {
+          project: projectUrl,
+          reports: {
+            architecture: buildFrontendUrl({
+              baseUrl: frontendBaseUrl,
+              target: "reports",
+              projectId,
+              tab: "architecture",
+              reportType: "architecture"
+            }),
+            cost: buildFrontendUrl({
+              baseUrl: frontendBaseUrl,
+              target: "reports",
+              projectId,
+              tab: "cost",
+              reportType: "cost"
+            }),
+            validation: buildFrontendUrl({
+              baseUrl: frontendBaseUrl,
+              target: "reports",
+              projectId,
+              tab: "validation",
+              reportType: "unit_tests"
+            })
+          },
+          downloads: {
+            pdf: buildFrontendUrl({
+              baseUrl: frontendBaseUrl,
+              target: "reports",
+              projectId,
+              downloadPdf: true,
+              pdfVerbosity: "full"
+            }),
+            ...workflowRunUrl ? { reviewArtifacts: workflowRunUrl } : {}
+          },
+          ...workflowRunUrl ? { workflowRun: workflowRunUrl } : {}
         },
         repo,
         ref,
@@ -15626,7 +15771,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-VSOTHLOD.js")
+    import("./App-3RDW53BW.js")
   ]);
   const baseUrl = await resolveBaseUrl(options, command);
   assertSecureBaseUrl(baseUrl);
@@ -15684,7 +15829,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-VSOTHLOD.js")
+    import("./App-3RDW53BW.js")
   ]);
   const baseUrl = await resolveBaseUrl(options, command);
   assertSecureBaseUrl(baseUrl);
@@ -16438,7 +16583,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-7XUOJYL5.js")).Banner
+    default: (await import("./Banner-HGG5NHXF.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.2",
+  "version": "0.28.0",
   "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.2",
+      "versionInfo": "0.28.0",
       "downloadLocation": "https://github.com/ganakailabs/cloudeval-cli",
       "filesAnalyzed": false,
       "licenseConcluded": "LicenseRef-CloudEval-CLI",
@@ -2354,9 +2354,9 @@
       "summary": "The semantic version parser used by npm."
     },
     {
-      "SPDXID": "SPDXRef-Package-shell-quote-1.8.3",
+      "SPDXID": "SPDXRef-Package-shell-quote-1.8.4",
       "name": "shell-quote",
-      "versionInfo": "1.8.3",
+      "versionInfo": "1.8.4",
       "downloadLocation": "https://github.com/ljharb/shell-quote",
       "filesAnalyzed": false,
       "licenseConcluded": "MIT",
@@ -3847,7 +3847,7 @@
     {
       "spdxElementId": "SPDXRef-Package-CloudEval-CLI",
       "relationshipType": "DEPENDS_ON",
-      "relatedSpdxElement": "SPDXRef-Package-shell-quote-1.8.3"
+      "relatedSpdxElement": "SPDXRef-Package-shell-quote-1.8.4"
     },
     {
       "spdxElementId": "SPDXRef-Package-CloudEval-CLI",