@f5xc-salesdemos/xcsh 18.69.0 → 18.72.0

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5xc-salesdemos/xcsh",
-	"version": "18.69.0",
+	"version": "18.72.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/f5xc-salesdemos/xcsh",
 	"author": "Can Boluk",
@@ -48,12 +48,12 @@
 	"dependencies": {
 		"@agentclientprotocol/sdk": "0.16.1",
 		"@mozilla/readability": "^0.6",
-		"@f5xc-salesdemos/xcsh-stats": "18.69.0",
-		"@f5xc-salesdemos/pi-agent-core": "18.69.0",
-		"@f5xc-salesdemos/pi-ai": "18.69.0",
-		"@f5xc-salesdemos/pi-natives": "18.69.0",
-		"@f5xc-salesdemos/pi-tui": "18.69.0",
-		"@f5xc-salesdemos/pi-utils": "18.69.0",
+		"@f5xc-salesdemos/xcsh-stats": "18.72.0",
+		"@f5xc-salesdemos/pi-agent-core": "18.72.0",
+		"@f5xc-salesdemos/pi-ai": "18.72.0",
+		"@f5xc-salesdemos/pi-natives": "18.72.0",
+		"@f5xc-salesdemos/pi-tui": "18.72.0",
+		"@f5xc-salesdemos/pi-utils": "18.72.0",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/internal-urls/build-info.generated.ts

@@ -17,17 +17,17 @@ export interface BuildInfo {
 }
 
 export const BUILD_INFO: BuildInfo = {
-	"version": "18.69.0",
-	"commit": "309ca5b1b28a5530f01b83381e69f8a6c94f83c8",
-	"shortCommit": "309ca5b",
+	"version": "18.72.0",
+	"commit": "ee865e583dfa21db5c5f5bf07001570c5f7f9086",
+	"shortCommit": "ee865e5",
 	"branch": "main",
-	"tag": "v18.69.0",
-	"commitDate": "2026-05-18T23:00:56Z",
-	"buildDate": "2026-05-18T23:27:04.429Z",
+	"tag": "v18.72.0",
+	"commitDate": "2026-05-19T19:45:52Z",
+	"buildDate": "2026-05-19T20:24:28.090Z",
 	"dirty": false,
 	"prNumber": "",
 	"repoUrl": "https://github.com/f5xc-salesdemos/xcsh",
 	"repoSlug": "f5xc-salesdemos/xcsh",
-	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/309ca5b1b28a5530f01b83381e69f8a6c94f83c8",
-	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.69.0"
+	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/ee865e583dfa21db5c5f5bf07001570c5f7f9086",
+	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.72.0"
 };

package/src/pipeline-report/generator.ts

@@ -2,9 +2,12 @@ import { logger } from "@f5xc-salesdemos/pi-utils";
 import { $ } from "bun";
 import type {
 	AccountRow,
+	CloseMonthBucket,
 	DataAnomaly,
+	DealSummary,
 	ForecastSummary,
 	LineItemRecord,
+	PipelineChange,
 	PipelineReportData,
 	PipelineReportOptions,
 	SectionData,
@@ -67,6 +70,12 @@ function parseLineItem(record: Record<string, unknown>, rules: SkuClassification
 		skuName,
 		fyb: (record.FYB_Total_Price__c as number) ?? 0,
 		category: classifySku(skuName, rules),
+		closeDate: (opp.CloseDate as string) ?? undefined,
+		oppName: (opp.Name as string) ?? undefined,
+		stage: (opp.StageName as string) ?? undefined,
+		lastActivityDate: (opp.LastActivityDate as string) ?? undefined,
+		ownerName: ((opp.Owner as Record<string, unknown> | undefined)?.Name as string) ?? undefined,
+		nextStep: (opp.NextStep as string) ?? undefined,
 	};
 }
 
@@ -139,6 +148,116 @@ function buildForecast(items: LineItemRecord[]): ForecastSummary {
 	return { commit: buckets.Commit, bestCase: buckets["Best Case"], pipeline: buckets.Pipeline };
 }
 
+/** Aggregate net new line items by opportunity and return the top N by quota-eligible amount. */
+function buildTopDeals(items: LineItemRecord[], limit = 5): DealSummary[] {
+	const oppMap = new Map<
+		string,
+		{
+			name: string;
+			accountName: string;
+			stage: string;
+			closeDate: string;
+			forecast: string;
+			amount: number;
+			ownerName: string;
+			nextStep: string;
+		}
+	>();
+	for (const item of items) {
+		if (item.category === "other" || item.fyb <= 0) continue;
+		const existing = oppMap.get(item.opportunityId);
+		if (existing) {
+			existing.amount += item.fyb;
+		} else {
+			oppMap.set(item.opportunityId, {
+				name: item.oppName ?? item.accountName,
+				accountName: item.accountName,
+				stage: item.stage ?? "",
+				closeDate: item.closeDate ?? "",
+				forecast: item.forecast,
+				amount: item.fyb,
+				ownerName: item.ownerName ?? "",
+				nextStep: item.nextStep ?? "",
+			});
+		}
+	}
+	return [...oppMap.entries()]
+		.sort(([, a], [, b]) => b.amount - a.amount)
+		.slice(0, limit)
+		.map(([id, d]) => ({
+			oppId: id,
+			name: d.name,
+			accountName: d.accountName,
+			stage: d.stage,
+			closeDate: d.closeDate,
+			forecast: d.forecast,
+			amount: d.amount,
+			ownerName: d.ownerName,
+			nextStep: d.nextStep || undefined,
+		}));
+}
+
+/** Build close-date distribution buckets from net new line items (quota-eligible only). */
+function buildCloseDistribution(items: LineItemRecord[]): CloseMonthBucket[] {
+	const buckets = new Map<
+		string,
+		{ amount: number; commit: number; bestCase: number; pipeline: number; oppIds: Set<string> }
+	>();
+	for (const item of items) {
+		if (item.category === "other" || item.fyb <= 0 || !item.closeDate) continue;
+		const ym = item.closeDate.slice(0, 7); // YYYY-MM
+		const existing = buckets.get(ym) ?? {
+			amount: 0,
+			commit: 0,
+			bestCase: 0,
+			pipeline: 0,
+			oppIds: new Set<string>(),
+		};
+		existing.amount += item.fyb;
+		existing.oppIds.add(item.opportunityId);
+		if (item.forecast === "Commit") existing.commit += item.fyb;
+		else if (item.forecast === "Best Case") existing.bestCase += item.fyb;
+		else if (item.forecast === "Pipeline") existing.pipeline += item.fyb;
+		buckets.set(ym, existing);
+	}
+	return [...buckets.entries()]
+		.sort(([a], [b]) => a.localeCompare(b))
+		.map(([ym, data]) => {
+			const d = new Date(`${ym}-01T00:00:00`);
+			const label = d.toLocaleDateString("en-US", { month: "long", year: "numeric" });
+			return {
+				label,
+				yearMonth: ym,
+				amount: data.amount,
+				commit: data.commit,
+				bestCase: data.bestCase,
+				pipeline: data.pipeline,
+				oppCount: data.oppIds.size,
+			};
+		});
+}
+
+/** Parse OpportunityFieldHistory records into PipelineChange entries. */
+function parseHistoryRecords(records: Record<string, unknown>[]): PipelineChange[] {
+	return records
+		.map((r): PipelineChange | null => {
+			const opp = (r.Opportunity ?? {}) as Record<string, unknown>;
+			const acct = (opp.Account ?? {}) as Record<string, unknown>;
+			const field = r.Field as string;
+			if (field !== "Amount" && field !== "ForecastCategoryName" && field !== "StageName") return null;
+			return {
+				oppId: (r.OpportunityId as string) ?? "",
+				dealName: ((opp.Name as string) ?? "").trim(),
+				accountName: ((acct.Name as string) ?? "").trim(),
+				field,
+				oldValue: String(r.OldValue ?? "\u2014"),
+				newValue: String(r.NewValue ?? "\u2014"),
+				date: ((r.CreatedDate as string) ?? "").slice(0, 10),
+			};
+		})
+		.filter((c): c is PipelineChange => c !== null);
+}
+
 // ---------------------------------------------------------------------------
 // Anomaly detection
 // ---------------------------------------------------------------------------
@@ -146,6 +265,7 @@ function buildForecast(items: LineItemRecord[]): ForecastSummary {
 function detectAnomalies(
 	netNew: LineItemRecord[],
 	booked: LineItemRecord[],
+	renewals: LineItemRecord[],
 	classification: SkuClassification,
 ): DataAnomaly[] {
 	const anomalies: DataAnomaly[] = [];
@@ -210,6 +330,93 @@ function detectAnomalies(
 		});
 	}
 
+	// 5. Open deals with close dates that have slipped past TODAY (requires CloseDate in data)
+	const today = new Date().toISOString().split("T")[0]!;
+	const slippedAccounts = new Map<string, string>(); // accountName → earliest slipped closeDate
+	for (const item of netNew) {
+		if (item.closeDate && item.closeDate < today) {
+			const existing = slippedAccounts.get(item.accountName);
+			if (!existing || item.closeDate < existing) {
+				slippedAccounts.set(item.accountName, item.closeDate);
+			}
+		}
+	}
+	if (slippedAccounts.size > 0) {
+		const details = [...slippedAccounts.entries()]
+			.sort(([, a], [, b]) => a.localeCompare(b))
+			.map(([name, date]) => `${name} (${date})`)
+			.join(", ");
+		anomalies.push({
+			severity: "warning",
+			category: "slipped-close-date",
+			message: `${slippedAccounts.size} account(s) have open pipeline with close dates in the past`,
+			details,
+		});
+	}
+
+	// 6. Stalled deals — open pipeline with no activity in >30 days
+	// Use the MOST RECENT activity date per account — if any deal has recent activity,
+	// the account is not stalled.
+	const staleThreshold = new Date();
+	staleThreshold.setDate(staleThreshold.getDate() - 30);
+	const staleStr = staleThreshold.toISOString().split("T")[0]!;
+	const acctLastActivity = new Map<string, string | null>(); // accountName → best lastActivityDate
+	for (const item of netNew) {
+		const existing = acctLastActivity.get(item.accountName);
+		const lad = item.lastActivityDate ?? null;
+		if (existing === undefined) {
+			acctLastActivity.set(item.accountName, lad);
+		} else if (lad && (!existing || lad > existing)) {
+			acctLastActivity.set(item.accountName, lad);
+		}
+	}
+	const stalledAccounts = new Map<string, string>();
+	for (const [name, lad] of acctLastActivity) {
+		if (!lad) {
+			stalledAccounts.set(name, "(no activity recorded)");
+		} else if (lad < staleStr) {
+			stalledAccounts.set(name, lad);
+		}
+	}
+	if (stalledAccounts.size > 0) {
+		const details = [...stalledAccounts.entries()]
+			.sort(([a], [b]) => a.localeCompare(b))
+			.map(([name, date]) => `${name} (${date})`)
+			.join(", ");
+		anomalies.push({
+			severity: "warning",
+			category: "stalled-deals",
+			message: `${stalledAccounts.size} account(s) have open pipeline with no activity in >30 days`,
+			details,
+		});
+	}
+
+	// 7. Urgent renewals — closing within 30 days
+	const urgentThreshold = new Date();
+	urgentThreshold.setDate(urgentThreshold.getDate() + 30);
+	const urgentStr = urgentThreshold.toISOString().split("T")[0]!;
+	const urgentRenewals = new Map<string, string>(); // accountName → closeDate
+	for (const item of renewals) {
+		if (item.closeDate && item.closeDate <= urgentStr) {
+			const existing = urgentRenewals.get(item.accountName);
+			if (!existing || item.closeDate < existing) {
+				urgentRenewals.set(item.accountName, item.closeDate);
+			}
+		}
+	}
+	if (urgentRenewals.size > 0) {
+		const details = [...urgentRenewals.entries()]
+			.sort(([, a], [, b]) => a.localeCompare(b))
+			.map(([name, date]) => `${name} (${date})`)
+			.join(", ");
+		anomalies.push({
+			severity: "warning",
+			category: "urgent-renewals",
+			message: `${urgentRenewals.size} account(s) have renewals closing within 30 days`,
+			details,
+		});
+	}
+
 	return anomalies;
 }
 
@@ -231,42 +438,38 @@ export async function generatePipelineReport(options: PipelineReportOptions): Pr
 	const staleCutoff = options.staleCutoff;
 
 	const userFilter = buildUserFilter(userIds);
-	const ownerFilter =
-		userIds.length === 1 ? `OwnerId = '${userIds[0]}'` : `OwnerId IN (${userIds.map(id => `'${id}'`).join(",")})`;
+	// SOQL does not allow OR between a semi-join subselect and other fields inside
+	// parentheses ("Semi join sub-selects are only allowed at the top level").
+	// The OpportunityTeamMember subselect already covers both SE and AE via userIds.
 	const skuFilter = buildSkuFilter(skuPrefixes);
 
 	const fields = [
 		"Product2.Name",
 		"FYB_Total_Price__c",
 		"Opportunity.Id",
+		"Opportunity.Name",
 		"Opportunity.Account.Name",
 		"Opportunity.Territory_Credited_District_del__c",
 		"Opportunity.ForecastCategoryName",
+		"Opportunity.StageName",
+		"Opportunity.IsClosed",
+		"Opportunity.IsWon",
+		"Opportunity.CloseDate",
+		"Opportunity.LastActivityDate",
+		"Opportunity.Owner.Name",
+		"Opportunity.NextStep",
 	].join(", ");
 	const commonFilters = `Subscription_Renewal__c = false AND Opportunity.ForecastCategoryName != 'Omitted' AND FYB_Total_Price__c > 0 AND (${skuFilter})`;
 	const quarterDateFilter = `Opportunity.CloseDate >= ${quarterStart} AND Opportunity.CloseDate <= ${quarterEnd}`;
 	const inPlayDateFilter = staleCutoff ? `Opportunity.CloseDate >= ${staleCutoff}` : quarterDateFilter;
 
-	const teamScope = `(OpportunityId IN (SELECT OpportunityId FROM OpportunityTeamMember WHERE ${userFilter}) OR ${ownerFilter})`;
+	const oliTeamScope = `OpportunityId IN (SELECT OpportunityId FROM OpportunityTeamMember WHERE ${userFilter})`;
 
-	// In-play: uses staleCutoff if set, otherwise quarter dates
-	// Booked: always quarter dates
-	const [netNewRecords, bookedRecords] = await Promise.all([
-		runSfQuery(
-			`SELECT ${fields} FROM OpportunityLineItem WHERE ${teamScope} AND Opportunity.IsClosed = false AND ${inPlayDateFilter} AND ${commonFilters}`,
-			orgAlias,
-		),
-		runSfQuery(
-			`SELECT ${fields} FROM OpportunityLineItem WHERE ${teamScope} AND Opportunity.IsWon = true AND ${quarterDateFilter} AND ${commonFilters}`,
-			orgAlias,
-		),
-	]);
-
-	// Parse line items
-	const netNewItems = netNewRecords.map(r => parseLineItem(r, skuClassification));
-	const bookedItems = bookedRecords.map(r => parseLineItem(r, skuClassification));
+	// Combined OLI query: net new (open, in-play dates) + booked (won, quarter dates) in one SOQL.
+	// The OR between regular conditions is allowed (SOQL only restricts OR with semi-join subselects).
+	const combinedDateFilter = `((Opportunity.IsClosed = false AND ${inPlayDateFilter}) OR (Opportunity.IsWon = true AND ${quarterDateFilter}))`;
 
-	// --- Renewals: opportunity-level True_ACV (FYB returns $0 for renewals) ---
+	// Build renewals query alongside OLI query (no dependency between them)
 	const renewalFields = [
 		"Id",
 		"Account.Name",
@@ -277,17 +480,133 @@ export async function generatePipelineReport(options: PipelineReportOptions): Pr
 		"Use_Case_Category__c",
 		"ForecastCategoryName",
 		"Territory_Credited_District_del__c",
+		"CloseDate",
+		"Name",
 	].join(", ");
-
 	const renewalDateFilter = staleCutoff
 		? `CloseDate >= ${staleCutoff}`
 		: `CloseDate >= ${quarterStart} AND CloseDate <= ${quarterEnd}`;
-	const renewalWhere = `(Id IN (SELECT OpportunityId FROM OpportunityTeamMember WHERE ${userFilter}) OR ${ownerFilter}) AND IsClosed = false AND Renewal__c = true AND ${renewalDateFilter} AND ForecastCategoryName != 'Omitted' AND (True_ACV__c > 1 OR Upsell_ACV__c > 1 OR Amount > 1)`;
+	const renewalWhere = `Id IN (SELECT OpportunityId FROM OpportunityTeamMember WHERE ${userFilter}) AND IsClosed = false AND Renewal__c = true AND ${renewalDateFilter} AND ForecastCategoryName != 'Omitted' AND (True_ACV__c > 1 OR Upsell_ACV__c > 1 OR Amount > 1)`;
+	// F5 fiscal year starts November 1. Derive from the report's quarter dates, not wall-clock,
+	// so the FY context matches the requested report period.
+	const qStartDate = new Date(quarterStart + "T00:00:00");
+	const qMonth = qStartDate.getMonth();
+	const qYear = qStartDate.getFullYear();
+	const fyStartYear = qMonth >= 10 ? qYear : qYear - 1;
+	const fyStart = `${fyStartYear}-11-01`;
+	const fyLabel = `FY${(fyStartYear + 1) % 100}`;
+	const todayStr = new Date().toISOString().split("T")[0]!;
+	const oppTeamScope = `Id IN (SELECT OpportunityId FROM OpportunityTeamMember WHERE ${userFilter})`;
+	const fyBookedQuery = `SELECT SUM(Amount) total FROM Opportunity WHERE ${oppTeamScope} AND IsWon = true AND CloseDate >= ${fyStart} AND CloseDate <= ${todayStr}`;
+
+	// Three parallel queries first: combined OLI + renewals + FY booked
+	const [oliRecords, renewalRecords, fyBookedResult] = await Promise.all([
+		runSfQuery(
+			`SELECT ${fields} FROM OpportunityLineItem WHERE ${oliTeamScope} AND ${combinedDateFilter} AND ${commonFilters}`,
+			orgAlias,
+		),
+		runSfQuery(
+			`SELECT ${renewalFields} FROM Opportunity WHERE ${renewalWhere} ORDER BY True_ACV__c DESC NULLS LAST`,
+			orgAlias,
+		),
+		runSfQuery(fyBookedQuery, orgAlias),
+	]);
+	const fyBookedTotal = (fyBookedResult[0]?.total as number) ?? 0;
+
+	// --- Split combined OLI records + fallback when empty ---
+	let netNewItems: LineItemRecord[];
+	let bookedItems: LineItemRecord[];
+
+	if (oliRecords.length > 0) {
+		netNewItems = [];
+		bookedItems = [];
+		for (const r of oliRecords) {
+			const item = parseLineItem(r, skuClassification);
+			const opp = (r.Opportunity ?? {}) as Record<string, unknown>;
+			if (opp.IsWon as boolean) {
+				bookedItems.push(item);
+			} else if (!(opp.IsClosed as boolean)) {
+				netNewItems.push(item);
+			}
+		}
+	} else {
+		// Opportunity-level fallback when OLI queries return no data.
+		// Collects renewal IDs first so we can exclude them — renewals have their own query
+		// and would double-count if included in net new/booked.
+		const renewalIds = new Set<string>();
+		for (const r of renewalRecords) {
+			const id = (r.Id ?? "") as string;
+			if (id) renewalIds.add(id);
+		}
+		const oppFields =
+			"Id, Name, Account.Name, Amount, ForecastCategoryName, StageName, CloseDate, LastActivityDate, Owner.Name, IsClosed, IsWon";
+		const oppTeamScope = `Id IN (SELECT OpportunityId FROM OpportunityTeamMember WHERE ${userFilter})`;
+		const oppDateFilter = staleCutoff
+			? `CloseDate >= ${staleCutoff}`
+			: `CloseDate >= ${quarterStart} AND CloseDate <= ${quarterEnd}`;
+		const oppWhere = `${oppTeamScope} AND ForecastCategoryName != 'Omitted' AND Amount > 0 AND IsClosed = false AND ${oppDateFilter}`;
+		const bookedWhere = `${oppTeamScope} AND ForecastCategoryName != 'Omitted' AND Amount > 0 AND IsWon = true AND CloseDate >= ${quarterStart} AND CloseDate <= ${quarterEnd}`;
+
+		const [fallbackNetNew, fallbackBooked] = await Promise.all([
+			runSfQuery(`SELECT ${oppFields} FROM Opportunity WHERE ${oppWhere} ORDER BY Amount DESC NULLS LAST`, orgAlias),
+			runSfQuery(
+				`SELECT ${oppFields} FROM Opportunity WHERE ${bookedWhere} ORDER BY Amount DESC NULLS LAST`,
+				orgAlias,
+			),
+		]);
+
+		function parseOppFallback(records: Record<string, unknown>[]): LineItemRecord[] {
+			const items: LineItemRecord[] = [];
+			const seen = new Set<string>();
+			for (const r of records) {
+				const id = (r.Id ?? "") as string;
+				if (seen.has(id) || renewalIds.has(id)) continue;
+				seen.add(id);
+				const acct = ((r.Account as Record<string, unknown> | undefined)?.Name ?? "") as string;
+				const fc = (r.ForecastCategoryName ?? "") as string;
+				const amt = (r.Amount as number) ?? 0;
+				if (amt <= 0) continue;
+				const owner = (r.Owner as Record<string, unknown> | undefined)?.Name;
+				items.push({
+					opportunityId: id,
+					accountName: acct,
+					territory: "",
+					forecast: fc,
+					skuName: "Opportunity-level",
+					fyb: amt,
+					category: "platform",
+					closeDate: (r.CloseDate as string) ?? undefined,
+					oppName: (r.Name as string) ?? undefined,
+					stage: (r.StageName as string) ?? undefined,
+					lastActivityDate: (r.LastActivityDate as string) ?? undefined,
+					ownerName: (owner as string) ?? undefined,
+				});
+			}
+			return items;
+		}
 
-	const renewalRecords = await runSfQuery(
-		`SELECT ${renewalFields} FROM Opportunity WHERE ${renewalWhere} ORDER BY True_ACV__c DESC NULLS LAST`,
-		orgAlias,
-	);
+		netNewItems = parseOppFallback(fallbackNetNew);
+		bookedItems = parseOppFallback(fallbackBooked);
+	}
+
+	// History query: run AFTER OLI to use specific opp IDs (avoids slow semi-join on FieldHistory).
+	// Only runs in the normal OLI path — in the fallback path we've already spent 5 queries
+	// (3 parallel + 2 fallback) and would exceed the 5-query constraint.
+	const allOppIds = new Set<string>();
+	for (const item of [...netNewItems, ...bookedItems]) {
+		if (item.opportunityId) allOppIds.add(item.opportunityId);
+	}
+	let historyRecords: Record<string, unknown>[] = [];
+	if (oliRecords.length > 0 && allOppIds.size > 0) {
+		const idList = [...allOppIds].map(id => `'${id}'`).join(",");
+		const historyFields =
+			"OpportunityId, Opportunity.Name, Opportunity.Account.Name, Field, OldValue, NewValue, CreatedDate";
+		const historyWhere = `OpportunityId IN (${idList}) AND Field IN ('Amount','ForecastCategoryName','StageName') AND CreatedDate = LAST_N_DAYS:7`;
+		historyRecords = await runSfQuery(
+			`SELECT ${historyFields} FROM OpportunityFieldHistory WHERE ${historyWhere} ORDER BY CreatedDate DESC LIMIT 50`,
+			orgAlias,
+		);
+	}
 
 	// Parse and classify renewal opps, dedup by Id
 	const seenIds = new Set<string>();
@@ -326,6 +645,8 @@ export async function generatePipelineReport(options: PipelineReportOptions): Pr
 			skuName: (r.Product_Segmentation__c ?? "") as string,
 			fyb: val,
 			category,
+			closeDate: (r.CloseDate as string) ?? undefined,
+			oppName: (r.Name as string) ?? undefined,
 		});
 	}
 
@@ -335,7 +656,7 @@ export async function generatePipelineReport(options: PipelineReportOptions): Pr
 		if (item.skuName) allSkus.add(item.skuName);
 	}
 	// Detect anomalies
-	const anomalies = detectAnomalies(netNewItems, bookedItems, skuClassification);
+	const anomalies = detectAnomalies(netNewItems, bookedItems, renewalItems, skuClassification);
 
 	return {
 		generated: new Date().toISOString(),
@@ -349,5 +670,12 @@ export async function generatePipelineReport(options: PipelineReportOptions): Pr
 		lineItemCount: netNewItems.length + bookedItems.length + renewalItems.length,
 		skusFound: [...allSkus].sort(),
 		anomalies,
+		topDeals: buildTopDeals(netNewItems),
+		topRenewals: renewalItems.length > 0 ? buildTopDeals(renewalItems) : undefined,
+		closeDistribution: buildCloseDistribution(netNewItems),
+		renewalDistribution: renewalItems.length > 0 ? buildCloseDistribution(renewalItems) : undefined,
+		fyBookedTotal: fyBookedTotal > 0 ? fyBookedTotal : undefined,
+		fyLabel,
+		recentChanges: parseHistoryRecords(historyRecords),
 	};
 }