@pagopa/dx-savemoney 0.5.0 → 0.6.2

package/src/azure/pricing/pricing-service.ts

@@ -116,9 +116,12 @@ export class PricingService {
     try {
       return await factory();
     } catch (error) {
-      logger.warn(
-        `Pricing lookup failed: ${error instanceof Error ? error.message : String(error)}`,
-      );
+      // Passed as a template property, not string-interpolated: logtape parses
+      // `{...}` in the message itself as placeholders (e.g. a ZodError's
+      // JSON-formatted issues would otherwise get swallowed as `undefined`).
+      logger.warn("Pricing lookup failed: {message}", {
+        message: error instanceof Error ? error.message : String(error),
+      });
       return undefined;
     }
   }

package/src/azure/pricing/schema.ts

@@ -12,6 +12,14 @@
 import { z } from "zod";
 
 const NonEmptyStringSchema = z.string().min(1);
+/**
+ * Optional metadata fields. The Retail Prices API returns `""` (not the
+ * field omitted) for meters where a given attribute doesn't apply (e.g.
+ * `armSkuName` for Bandwidth/Support meters), so these must tolerate empty
+ * strings — resolvers already treat empty and missing values the same way
+ * (`?.toLowerCase() ?? ""`).
+ */
+const OptionalStringSchema = z.string().optional();
 const PriceTypeSchema = z.enum([
   "Consumption",
   "DevTestConsumption",
@@ -31,23 +39,23 @@ const PriceValueSchema = z.number().finite().nonnegative();
  *   want on-demand pricing must filter by `type === "Consumption"`.
  */
 export const PriceItemSchema = z.object({
-  armRegionName: NonEmptyStringSchema.optional(),
-  armSkuName: NonEmptyStringSchema.optional(),
+  armRegionName: OptionalStringSchema,
+  armSkuName: OptionalStringSchema,
   currencyCode: z.string().length(3),
-  effectiveStartDate: NonEmptyStringSchema.optional(),
+  effectiveStartDate: OptionalStringSchema,
   isPrimaryMeterRegion: z.boolean().optional(),
-  location: NonEmptyStringSchema.optional(),
-  meterId: NonEmptyStringSchema.optional(),
-  meterName: NonEmptyStringSchema.optional(),
-  productId: NonEmptyStringSchema.optional(),
-  productName: NonEmptyStringSchema.optional(),
-  reservationTerm: NonEmptyStringSchema.optional(),
+  location: OptionalStringSchema,
+  meterId: OptionalStringSchema,
+  meterName: OptionalStringSchema,
+  productId: OptionalStringSchema,
+  productName: OptionalStringSchema,
+  reservationTerm: OptionalStringSchema,
   retailPrice: PriceValueSchema,
-  serviceFamily: NonEmptyStringSchema.optional(),
-  serviceId: NonEmptyStringSchema.optional(),
-  serviceName: NonEmptyStringSchema.optional(),
-  skuId: NonEmptyStringSchema.optional(),
-  skuName: NonEmptyStringSchema.optional(),
+  serviceFamily: OptionalStringSchema,
+  serviceId: OptionalStringSchema,
+  serviceName: OptionalStringSchema,
+  skuId: OptionalStringSchema,
+  skuName: OptionalStringSchema,
   tierMinimumUnits: PriceValueSchema.optional(),
   /** "Consumption" | "Reservation" | "DevTestConsumption". */
   type: PriceTypeSchema,

package/src/azure/resources/app-service.ts

@@ -139,7 +139,8 @@ export async function analyzeAppServicePlan(
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
     logger.warn(
-      `Failed to get App Service Plan details for ${planName}: ${error instanceof Error ? error.message : error}`,
+      `Failed to get App Service Plan details for ${planName}: {message}`,
+      { message: error instanceof Error ? error.message : error },
     );
     reason += "Could not retrieve detailed App Service Plan information. ";
   }
@@ -184,7 +185,8 @@ async function enrichWithPricing(
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
     logger.warn(
-      `Failed to enrich App Service Plan ${resource.name ?? "unknown"} with pricing: ${error instanceof Error ? error.message : error}`,
+      `Failed to enrich App Service Plan ${resource.name ?? "unknown"} with pricing: {message}`,
+      { message: error instanceof Error ? error.message : error },
     );
     return result;
   }

package/src/azure/resources/container-app.ts

@@ -100,7 +100,8 @@ export async function analyzeContainerApp(
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
     logger.warn(
-      `Failed to get Container App details for ${containerAppName}: ${error instanceof Error ? error.message : error}`,
+      `Failed to get Container App details for ${containerAppName}: {message}`,
+      { message: error instanceof Error ? error.message : error },
     );
     reason += "Could not retrieve detailed Container App information. ";
   }

package/src/azure/resources/disk.ts

@@ -81,9 +81,9 @@ export async function analyzeDisk(
     }
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
-    logger.warn(
-      `Failed to get disk details for ${diskName}: ${error instanceof Error ? error.message : error}`,
-    );
+    logger.warn(`Failed to get disk details for ${diskName}: {message}`, {
+      message: error instanceof Error ? error.message : error,
+    });
     // Fallback to checking properties if API call fails
     // Note: GenericResource doesn't have diskState property
     // Without API access, we can't determine if disk is unattached

package/src/azure/resources/nic.ts

@@ -77,9 +77,9 @@ export async function analyzeNic(
     // The primary checks (attachment to VM and public IP assignment) are sufficient
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
-    logger.warn(
-      `Failed to get NIC details for ${nicName}: ${error instanceof Error ? error.message : error}`,
-    );
+    logger.warn(`Failed to get NIC details for ${nicName}: {message}`, {
+      message: error instanceof Error ? error.message : error,
+    });
     reason += "Could not retrieve detailed NIC information. ";
   }
 

package/src/azure/resources/private-endpoint.ts

@@ -100,7 +100,8 @@ export async function analyzePrivateEndpoint(
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
     logger.warn(
-      `Failed to get Private Endpoint details for ${privateEndpointName}: ${error instanceof Error ? error.message : error}`,
+      `Failed to get Private Endpoint details for ${privateEndpointName}: {message}`,
+      { message: error instanceof Error ? error.message : error },
     );
     reason += "Could not retrieve detailed Private Endpoint information. ";
   }

package/src/azure/resources/public-ip.ts

@@ -66,8 +66,7 @@ export async function analyzePublicIp(
   // Capture details outside the try so the pricing enrichment below can
   // reuse them without an extra API round-trip.
   let publicIpDetails:
-    | Awaited<ReturnType<typeof networkClient.publicIPAddresses.get>>
-    | undefined;
+    Awaited<ReturnType<typeof networkClient.publicIPAddresses.get>> | undefined;
 
   try {
     // Get detailed Public IP information
@@ -107,7 +106,8 @@ export async function analyzePublicIp(
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
     logger.warn(
-      `Failed to get Public IP details for ${publicIpName}: ${error instanceof Error ? error.message : error}`,
+      `Failed to get Public IP details for ${publicIpName}: {message}`,
+      { message: error instanceof Error ? error.message : error },
     );
     reason += "Could not retrieve detailed Public IP information. ";
   }
@@ -161,9 +161,9 @@ async function enrichWithPricing(
     return { ...result, estimatedMonthlySavings };
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
-    logger.warn(
-      `Failed to enrich Public IP with pricing: ${error instanceof Error ? error.message : error}`,
-    );
+    logger.warn("Failed to enrich Public IP with pricing: {message}", {
+      message: error instanceof Error ? error.message : error,
+    });
     return result;
   }
 }

package/src/azure/resources/static-web-app.ts

@@ -101,7 +101,8 @@ export async function analyzeStaticSite(
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
     logger.warn(
-      `Failed to get metrics for Static Web App ${resource.name}: ${error instanceof Error ? error.message : error}`,
+      `Failed to get metrics for Static Web App ${resource.name}: {message}`,
+      { message: error instanceof Error ? error.message : error },
     );
   }
 

package/src/azure/resources/vm.ts

@@ -114,9 +114,9 @@ export async function analyzeVM(
     }
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
-    logger.warn(
-      `Failed to get VM instance view for ${vmName}: ${error instanceof Error ? error.message : error}`,
-    );
+    logger.warn(`Failed to get VM instance view for ${vmName}: {message}`, {
+      message: error instanceof Error ? error.message : error,
+    });
     // Continue with metric analysis if instance view fails
   }
 
@@ -203,9 +203,9 @@ async function enrichWithPricing(
     return { ...result, estimatedMonthlySavings };
   } catch (error) {
     const logger = getLogger(["savemoney", "azure"]);
-    logger.warn(
-      `Failed to enrich VM ${vmName} with pricing: ${error instanceof Error ? error.message : error}`,
-    );
+    logger.warn(`Failed to enrich VM ${vmName} with pricing: {message}`, {
+      message: error instanceof Error ? error.message : error,
+    });
     return result;
   }
 }

package/src/azure/types.ts

@@ -17,6 +17,14 @@ import type {
  * Azure configuration extending base config
  */
 export type AzureConfig = BaseConfig & {
+  /**
+   * Optional path to an AZQR (`azqr scan --json`) report. When set, the
+   * orchestrator ingests its `impacted` resources, filters them down to
+   * FinOps-relevant opportunities and merges them into the run output as
+   * findings with `source: "azqr"`. Can be set via YAML (`azure.azqrReportPath`)
+   * or the CLI `--azqr-report` flag, which takes precedence.
+   */
+  azqrReportPath?: string;
   /**
    * Maximum number of resources analyzed in parallel within a single
    * subscription. Defaults to 8 when not provided. Set to 1 for a fully

package/src/azure/utils.ts

@@ -219,7 +219,11 @@ export function verboseLog(
   if (verbose) {
     const logger = getLogger(["savemoney", "azure", "verbose"]);
     if (object !== undefined) {
-      logger.debug(`${message} ${JSON.stringify(object, null, 2)}`);
+      // Passed as a template property (see pricing-service.ts for why: avoids
+      // logtape misparsing literal `{`/`}` in the JSON as placeholders).
+      logger.debug(`${message} {object}`, {
+        object: JSON.stringify(object, null, 2),
+      });
     } else {
       logger.debug(message);
     }
@@ -308,8 +312,10 @@ async function fetchMetric(
     return aggregatedValue;
   } catch (error) {
     const logger = getLogger(["savemoney", "azure", "metrics"]);
+    // Passed as a template property (see pricing-service.ts for why).
     logger.error(
-      `Failed to fetch metric ${metricName} for resource ${resourceId}: ${error instanceof Error ? error.message : String(error)}`,
+      `Failed to fetch metric ${metricName} for resource ${resourceId}: {message}`,
+      { message: error instanceof Error ? error.message : String(error) },
     );
     return null;
   }

package/src/finding.ts

@@ -67,20 +67,17 @@ export type Finding = {
  * model is open to future Advisor categories.
  */
 export type FindingCategory =
-  | "cost"
-  | "operationalExcellence"
-  | "performance"
-  | "reliability"
-  | "security";
+  "cost" | "operationalExcellence" | "performance" | "reliability" | "security";
 
 /**
  * Where the finding originated from.
  *
  * - `custom`  → emitted by a savemoney analyzer plugin
  * - `advisor` → fetched from Azure Advisor recommendations
+ * - `azqr`    → ingested from an AZQR (`azqr scan --json`) report
  * - `aws`     → reserved for future AWS Trusted Advisor / Compute Optimizer
  */
-export type FindingSource = "advisor" | "aws" | "custom";
+export type FindingSource = "advisor" | "aws" | "azqr" | "custom";
 
 /**
  * Monetary value associated with a finding, when known.

package/src/schema.ts

@@ -109,6 +109,12 @@ const AzureSourceSchema = z.enum(AZURE_SOURCE_VALUES);
 
 const AzureSectionSchema = z
   .object({
+    /**
+     * Optional path to an AZQR (`azqr scan --json`) report. When set, its
+     * FinOps-relevant impacted resources are merged into the analysis. The
+     * `--azqr-report` CLI flag, when provided, overrides this value.
+     */
+    azqrReportPath: z.string().optional(),
     /**
      * Maximum number of resources analyzed in parallel within a single
      * subscription. Defaults to 8 when not provided.

package/dist/__tests__/finding.test.d.ts

@@ -1,17 +0,0 @@
-/**
- * Tests for findingsFromAnalysisResult — verifies the adapter that converts
- * a legacy `AnalysisResult.reason` string into a structured `Finding[]`.
- *
- * Behaviours covered:
- * 1. Empty / whitespace-only reason → empty array.
- * 2. Single sentence (with or without trailing period) → one finding.
- * 3. Multi-sentence reason (". " separator) → one finding per sentence.
- * 4. Trailing period on the whole string → not duplicated.
- * 5. Custom `code` → propagated to every finding.
- * 6. Omitted `code` → defaults to "custom.unknown".
- * 7. Custom `source` → propagated.
- * 8. Omitted `source` → defaults to "custom".
- * 9. Every finding carries the correct `resourceId`, `severity`, `category`.
- */
-export {};
-//# sourceMappingURL=finding.test.d.ts.map

package/dist/__tests__/finding.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"finding.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/finding.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG"}

package/dist/__tests__/finding.test.js

@@ -1,146 +0,0 @@
-/**
- * Tests for findingsFromAnalysisResult — verifies the adapter that converts
- * a legacy `AnalysisResult.reason` string into a structured `Finding[]`.
- *
- * Behaviours covered:
- * 1. Empty / whitespace-only reason → empty array.
- * 2. Single sentence (with or without trailing period) → one finding.
- * 3. Multi-sentence reason (". " separator) → one finding per sentence.
- * 4. Trailing period on the whole string → not duplicated.
- * 5. Custom `code` → propagated to every finding.
- * 6. Omitted `code` → defaults to "custom.unknown".
- * 7. Custom `source` → propagated.
- * 8. Omitted `source` → defaults to "custom".
- * 9. Every finding carries the correct `resourceId`, `severity`, `category`.
- */
-import { describe, expect, it } from "vitest";
-import { findingsFromAnalysisResult } from "../finding.js";
-const BASE_ARGS = {
-    reason: "Low CPU usage.",
-    resourceId: "/subscriptions/sub1/resourceGroups/rg/providers/Microsoft.Compute/virtualMachines/vm",
-    severity: "high",
-};
-describe("findingsFromAnalysisResult", () => {
-    describe("empty / whitespace input", () => {
-        it("returns [] for an empty string", () => {
-            expect(findingsFromAnalysisResult({ ...BASE_ARGS, reason: "" })).toEqual([]);
-        });
-        it("returns [] for a whitespace-only string", () => {
-            expect(findingsFromAnalysisResult({ ...BASE_ARGS, reason: "   " })).toEqual([]);
-        });
-        it("returns [] for a bare period", () => {
-            expect(findingsFromAnalysisResult({ ...BASE_ARGS, reason: "." })).toEqual([]);
-        });
-    });
-    describe("single sentence", () => {
-        it("returns one finding with a trailing period", () => {
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                reason: "VM is deallocated.",
-            });
-            expect(result).toHaveLength(1);
-            expect(result[0].reason).toBe("VM is deallocated.");
-        });
-        it("appends a trailing period when missing", () => {
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                reason: "VM is deallocated",
-            });
-            expect(result).toHaveLength(1);
-            expect(result[0].reason).toBe("VM is deallocated.");
-        });
-    });
-    describe("multi-sentence reason", () => {
-        it("splits on '. ' into separate findings", () => {
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                reason: "VM is deallocated. No tags found. Low CPU usage.",
-            });
-            expect(result).toHaveLength(3);
-            expect(result[0].reason).toBe("VM is deallocated.");
-            expect(result[1].reason).toBe("No tags found.");
-            expect(result[2].reason).toBe("Low CPU usage.");
-        });
-        it("does not create an extra empty finding for a trailing period", () => {
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                reason: "Sentence one. Sentence two.",
-            });
-            expect(result).toHaveLength(2);
-        });
-    });
-    describe("code field", () => {
-        it("uses the provided code for every finding", () => {
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                code: "vm.deallocated",
-                reason: "A. B.",
-            });
-            expect(result.every((f) => f.code === "vm.deallocated")).toBe(true);
-        });
-        it("defaults code to 'custom.unknown' when omitted", () => {
-            const result = findingsFromAnalysisResult(BASE_ARGS);
-            expect(result[0].code).toBe("custom.unknown");
-        });
-    });
-    describe("source field", () => {
-        it("uses the provided source for every finding", () => {
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                source: "advisor",
-            });
-            expect(result.every((f) => f.source === "advisor")).toBe(true);
-        });
-        it("defaults source to 'custom' when omitted", () => {
-            const result = findingsFromAnalysisResult(BASE_ARGS);
-            expect(result[0].source).toBe("custom");
-        });
-    });
-    describe("static fields on every finding", () => {
-        it("propagates resourceId to every finding", () => {
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                reason: "A. B.",
-            });
-            expect(result.every((f) => f.resourceId === BASE_ARGS.resourceId)).toBe(true);
-        });
-        it("propagates severity to every finding", () => {
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                reason: "A. B.",
-                severity: "medium",
-            });
-            expect(result.every((f) => f.severity === "medium")).toBe(true);
-        });
-        it("sets category to 'cost' on every finding", () => {
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                reason: "A. B.",
-            });
-            expect(result.every((f) => f.category === "cost")).toBe(true);
-        });
-        it("never attaches estimatedMonthlySavings to custom findings", () => {
-            // Custom analyzers carry the resource's monthly cost on
-            // `AnalysisResult.estimatedMonthlySavings` and the report layer
-            // renders it at the resource level. Propagating it down to a
-            // single finding sentence (e.g. "No tags found") would falsely
-            // suggest that fixing that sentence yields the full saving.
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                reason: "Low CPU. Low network. Idle.",
-            });
-            expect(result).toHaveLength(3);
-            for (const f of result) {
-                expect(f.estimatedMonthlySavings).toBeUndefined();
-            }
-        });
-        it("omits estimatedMonthlySavings entirely", () => {
-            const result = findingsFromAnalysisResult({
-                ...BASE_ARGS,
-                reason: "Low CPU.",
-            });
-            expect(result[0].estimatedMonthlySavings).toBeUndefined();
-        });
-    });
-});
-//# sourceMappingURL=finding.test.js.map

package/dist/__tests__/finding.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"finding.test.js","sourceRoot":"","sources":["../../src/__tests__/finding.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE9C,OAAO,EAAE,0BAA0B,EAAE,MAAM,eAAe,CAAC;AAE3D,MAAM,SAAS,GAAG;IAChB,MAAM,EAAE,gBAAgB;IACxB,UAAU,EACR,sFAAsF;IACxF,QAAQ,EAAE,MAAe;CAC1B,CAAC;AAEF,QAAQ,CAAC,4BAA4B,EAAE,GAAG,EAAE;IAC1C,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;QACxC,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;YACxC,MAAM,CAAC,0BAA0B,CAAC,EAAE,GAAG,SAAS,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CACtE,EAAE,CACH,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;YACjD,MAAM,CACJ,0BAA0B,CAAC,EAAE,GAAG,SAAS,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAC5D,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAChB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,8BAA8B,EAAE,GAAG,EAAE;YACtC,MAAM,CAAC,0BAA0B,CAAC,EAAE,GAAG,SAAS,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC,OAAO,CACvE,EAAE,CACH,CAAC;QACJ,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;QAC/B,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;YACpD,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,MAAM,EAAE,oBAAoB;aAC7B,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,oBAAoB,CAAC,CAAC;QACtD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,MAAM,EAAE,mBAAmB;aAC5B,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,oBAAoB,CAAC,CAAC;QACtD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;QACrC,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,MAAM,EAAE,kDAAkD;aAC3D,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,oBAAoB,CAAC,CAAC;YACpD,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;QAClD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;YACtE,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,MAAM,EAAE,6BAA6B;aACtC,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACjC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,YAAY,EAAE,GAAG,EAAE;QAC1B,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,IAAI,EAAE,gBAAgB;gBACtB,MAAM,EAAE,OAAO;aAChB,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,gBAAgB,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACtE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;YACxD,MAAM,MAAM,GAAG,0BAA0B,CAAC,SAAS,CAAC,CAAC;YACrD,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;QAChD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,cAAc,EAAE,GAAG,EAAE;QAC5B,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;YACpD,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,MAAM,EAAE,SAAS;aAClB,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACjE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,MAAM,GAAG,0BAA0B,CAAC,SAAS,CAAC,CAAC;YACrD,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAC1C,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,gCAAgC,EAAE,GAAG,EAAE;QAC9C,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,MAAM,EAAE,OAAO;aAChB,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,KAAK,SAAS,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CACrE,IAAI,CACL,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;YAC9C,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,MAAM,EAAE,OAAO;gBACf,QAAQ,EAAE,QAAQ;aACnB,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAClE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,MAAM,EAAE,OAAO;aAChB,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAChE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;YACnE,wDAAwD;YACxD,gEAAgE;YAChE,6DAA6D;YAC7D,+DAA+D;YAC/D,4DAA4D;YAC5D,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,MAAM,EAAE,6BAA6B;aACtC,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC/B,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;gBACvB,MAAM,CAAC,CAAC,CAAC,uBAAuB,CAAC,CAAC,aAAa,EAAE,CAAC;YACpD,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,MAAM,GAAG,0BAA0B,CAAC;gBACxC,GAAG,SAAS;gBACZ,MAAM,EAAE,UAAU;aACnB,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,uBAAuB,CAAC,CAAC,aAAa,EAAE,CAAC;QAC5D,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/azure/__tests__/analyzer-tags.test.d.ts

@@ -1,8 +0,0 @@
-/**
- * Unit tests for tag-filter behavior on Advisor findings.
- *
- * These tests exercise the pure helper used by the Azure analyzer
- * orchestrator to keep filtering semantics explicit and stable.
- */
-export {};
-//# sourceMappingURL=analyzer-tags.test.d.ts.map

package/dist/azure/__tests__/analyzer-tags.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"analyzer-tags.test.d.ts","sourceRoot":"","sources":["../../../src/azure/__tests__/analyzer-tags.test.ts"],"names":[],"mappings":"AAAA;;;;;GAKG"}

package/dist/azure/__tests__/analyzer-tags.test.js

@@ -1,70 +0,0 @@
-/**
- * Unit tests for tag-filter behavior on Advisor findings.
- *
- * These tests exercise the pure helper used by the Azure analyzer
- * orchestrator to keep filtering semantics explicit and stable.
- */
-import { describe, expect, it } from "vitest";
-import { mergeFinding, shouldIncludeAdvisorFindingForTags, } from "../analyzer.js";
-function mkFinding(resourceId, source) {
-    return {
-        category: "cost",
-        code: `${source}.test`,
-        reason: "Test finding.",
-        resourceId,
-        severity: "low",
-        source,
-    };
-}
-describe("shouldIncludeAdvisorFindingForTags", () => {
-    it("includes all findings when no tag filter is active", () => {
-        const finding = mkFinding("/subscriptions/sub1/resourceGroups/rg1/providers/Microsoft.Compute/virtualMachines/vm1", "advisor");
-        expect(shouldIncludeAdvisorFindingForTags(finding, new Set(), false)).toBe(true);
-    });
-    it("keeps subscription-level Advisor findings global even with tag filters", () => {
-        const finding = mkFinding("/subscriptions/sub1", "advisor");
-        expect(shouldIncludeAdvisorFindingForTags(finding, new Set(), true)).toBe(true);
-    });
-    it("includes resource-level Advisor findings only when resource id is tag-matched", () => {
-        const finding = mkFinding("/subscriptions/sub1/resourceGroups/rg1/providers/Microsoft.Compute/virtualMachines/vm1", "advisor");
-        const taggedResourceIds = new Set([
-            "/subscriptions/sub1/resourcegroups/rg1/providers/microsoft.compute/virtualmachines/vm1",
-        ]);
-        expect(shouldIncludeAdvisorFindingForTags(finding, taggedResourceIds, true)).toBe(true);
-    });
-    it("excludes resource-level Advisor findings when resource id is not tag-matched", () => {
-        const finding = mkFinding("/subscriptions/sub1/resourceGroups/rg1/providers/Microsoft.Compute/virtualMachines/vm2", "advisor");
-        const taggedResourceIds = new Set([
-            "/subscriptions/sub1/resourcegroups/rg1/providers/microsoft.compute/virtualmachines/vm1",
-        ]);
-        expect(shouldIncludeAdvisorFindingForTags(finding, taggedResourceIds, true)).toBe(false);
-    });
-});
-describe("mergeFinding", () => {
-    it("preserves custom cost at risk when merging an Advisor finding", () => {
-        const resourceId = "/subscriptions/sub1/resourceGroups/rg1/providers/Microsoft.Compute/virtualMachines/vm1";
-        const existing = {
-            analysis: {
-                costRisk: "high",
-                estimatedMonthlySavings: { amount: 29.94, currency: "EUR" },
-                reason: "VM is deallocated.",
-                suspectedUnused: true,
-            },
-            findings: [mkFinding(resourceId, "custom")],
-            resource: {
-                id: resourceId,
-                name: "vm1",
-                type: "Microsoft.Compute/virtualMachines",
-            },
-        };
-        const reports = [existing];
-        const reportsById = new Map([[resourceId.toLowerCase(), existing]]);
-        mergeFinding(mkFinding(resourceId, "advisor"), reports, reportsById);
-        expect(existing.analysis.estimatedMonthlySavings).toEqual({
-            amount: 29.94,
-            currency: "EUR",
-        });
-        expect(existing.findings).toHaveLength(2);
-    });
-});
-//# sourceMappingURL=analyzer-tags.test.js.map

package/dist/azure/__tests__/analyzer-tags.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"analyzer-tags.test.js","sourceRoot":"","sources":["../../../src/azure/__tests__/analyzer-tags.test.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAK9C,OAAO,EACL,YAAY,EACZ,kCAAkC,GACnC,MAAM,gBAAgB,CAAC;AAExB,SAAS,SAAS,CAAC,UAAkB,EAAE,MAAyB;IAC9D,OAAO;QACL,QAAQ,EAAE,MAAM;QAChB,IAAI,EAAE,GAAG,MAAM,OAAO;QACtB,MAAM,EAAE,eAAe;QACvB,UAAU;QACV,QAAQ,EAAE,KAAK;QACf,MAAM;KACP,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,oCAAoC,EAAE,GAAG,EAAE;IAClD,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAC5D,MAAM,OAAO,GAAG,SAAS,CACvB,wFAAwF,EACxF,SAAS,CACV,CAAC;QAEF,MAAM,CACJ,kCAAkC,CAAC,OAAO,EAAE,IAAI,GAAG,EAAU,EAAE,KAAK,CAAC,CACtE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACf,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wEAAwE,EAAE,GAAG,EAAE;QAChF,MAAM,OAAO,GAAG,SAAS,CAAC,qBAAqB,EAAE,SAAS,CAAC,CAAC;QAE5D,MAAM,CACJ,kCAAkC,CAAC,OAAO,EAAE,IAAI,GAAG,EAAU,EAAE,IAAI,CAAC,CACrE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACf,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+EAA+E,EAAE,GAAG,EAAE;QACvF,MAAM,OAAO,GAAG,SAAS,CACvB,wFAAwF,EACxF,SAAS,CACV,CAAC;QAEF,MAAM,iBAAiB,GAAG,IAAI,GAAG,CAAS;YACxC,wFAAwF;SACzF,CAAC,CAAC;QAEH,MAAM,CACJ,kCAAkC,CAAC,OAAO,EAAE,iBAAiB,EAAE,IAAI,CAAC,CACrE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACf,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8EAA8E,EAAE,GAAG,EAAE;QACtF,MAAM,OAAO,GAAG,SAAS,CACvB,wFAAwF,EACxF,SAAS,CACV,CAAC;QAEF,MAAM,iBAAiB,GAAG,IAAI,GAAG,CAAS;YACxC,wFAAwF;SACzF,CAAC,CAAC;QAEH,MAAM,CACJ,kCAAkC,CAAC,OAAO,EAAE,iBAAiB,EAAE,IAAI,CAAC,CACrE,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAChB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,cAAc,EAAE,GAAG,EAAE;IAC5B,EAAE,CAAC,+DAA+D,EAAE,GAAG,EAAE;QACvE,MAAM,UAAU,GACd,wFAAwF,CAAC;QAC3F,MAAM,QAAQ,GAAgC;YAC5C,QAAQ,EAAE;gBACR,QAAQ,EAAE,MAAM;gBAChB,uBAAuB,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,KAAK,EAAE;gBAC3D,MAAM,EAAE,oBAAoB;gBAC5B,eAAe,EAAE,IAAI;aACtB;YACD,QAAQ,EAAE,CAAC,SAAS,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAC;YAC3C,QAAQ,EAAE;gBACR,EAAE,EAAE,UAAU;gBACd,IAAI,EAAE,KAAK;gBACX,IAAI,EAAE,mCAAmC;aAC1C;SACF,CAAC;QACF,MAAM,OAAO,GAAG,CAAC,QAAQ,CAAC,CAAC;QAC3B,MAAM,WAAW,GAAG,IAAI,GAAG,CAAC,CAAC,CAAC,UAAU,CAAC,WAAW,EAAE,EAAE,QAAQ,CAAC,CAAC,CAAC,CAAC;QAEpE,YAAY,CAAC,SAAS,CAAC,UAAU,EAAE,SAAS,CAAC,EAAE,OAAO,EAAE,WAAW,CAAC,CAAC;QAErE,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAC,uBAAuB,CAAC,CAAC,OAAO,CAAC;YACxD,MAAM,EAAE,KAAK;YACb,QAAQ,EAAE,KAAK;SAChB,CAAC,CAAC;QACH,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/azure/__tests__/config.test.d.ts

@@ -1,9 +0,0 @@
-/**
- * Tests for loadConfig() — verifies that:
- * 1. Returns default thresholds and prompts for subscriptionIds when no file is given.
- * 2. Loads subscriptionIds, location, timespanDays and thresholds from a YAML file.
- * 3. Partial threshold overrides keep defaults for non-overridden fields.
- * 4. Throws a clear error for a non-existent explicit path.
- */
-export {};
-//# sourceMappingURL=config.test.d.ts.map

package/dist/azure/__tests__/config.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"config.test.d.ts","sourceRoot":"","sources":["../../../src/azure/__tests__/config.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG"}