From 306cdd5f5381002f10a3f26dc04d5759cd9e5994 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 30 Oct 2025 05:31:08 +0000 Subject: [PATCH] Fix inconsistent comment phrasing in isOrganizationReportSummary MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Update the comment for the newApplications field validation to accurately reflect that the field IS in the type definition (as optional), but is optional due to legacy encrypted data. Changed from: "not in type definition" Changed to: "legacy encrypted data predates this field" This aligns with the explanation used elsewhere in the codebase and accurately describes why the field must remain optional. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- .../risk-insights/services/domain/risk-insights-type-guards.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bitwarden_license/bit-common/src/dirt/reports/risk-insights/services/domain/risk-insights-type-guards.ts b/bitwarden_license/bit-common/src/dirt/reports/risk-insights/services/domain/risk-insights-type-guards.ts index bcd45e4719d..b44f64d4021 100644 --- a/bitwarden_license/bit-common/src/dirt/reports/risk-insights/services/domain/risk-insights-type-guards.ts +++ b/bitwarden_license/bit-common/src/dirt/reports/risk-insights/services/domain/risk-insights-type-guards.ts @@ -230,7 +230,7 @@ export function isOrganizationReportSummary(obj: any): obj is OrganizationReport Number.isSafeInteger(obj.totalCriticalAtRiskApplicationCount) && obj.totalCriticalAtRiskApplicationCount >= 0 && obj.totalCriticalAtRiskApplicationCount <= MAX_COUNT && - // newApplications is optional (backward compatibility - not in type definition) + // newApplications is optional (backward compatibility - legacy encrypted data predates this field) (obj.newApplications === undefined || (Array.isArray(obj.newApplications) && obj.newApplications.length <= MAX_ARRAY_LENGTH &&