From 0c14c8812f95d4ad09b7ef4cddaf637cfe6c2ce7 Mon Sep 17 00:00:00 2001 From: Ugo Palatucci Date: Mon, 16 Mar 2026 18:00:50 +0100 Subject: [PATCH] fix null safety guards to prevent runtime crashes Made-with: Cursor --- .../PolicyWizard/utils/hooks/useNodeInterfaces/utils/utils.ts | 2 +- src/utils/resources/enactments/utils.ts | 2 +- src/utils/resources/policies/utils.ts | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/utils/components/PolicyForm/PolicyWizard/utils/hooks/useNodeInterfaces/utils/utils.ts b/src/utils/components/PolicyForm/PolicyWizard/utils/hooks/useNodeInterfaces/utils/utils.ts index 061567f1..c462378e 100644 --- a/src/utils/components/PolicyForm/PolicyWizard/utils/hooks/useNodeInterfaces/utils/utils.ts +++ b/src/utils/components/PolicyForm/PolicyWizard/utils/hooks/useNodeInterfaces/utils/utils.ts @@ -23,7 +23,7 @@ const getUsedPortNamesForNode = (nns: V1beta1NodeNetworkState) => { const interfaces = getInterfaces(nns); return interfaces.reduce((acc, iface) => { if (bridgeTypes.includes(iface?.type)) { - const ports = iface?.bridge?.port?.map((port) => port?.name); + const ports = iface?.bridge?.port?.map((port) => port?.name) || []; acc = [...acc, ...ports]; } diff --git a/src/utils/resources/enactments/utils.ts b/src/utils/resources/enactments/utils.ts index c29509d2..7ecc4d43 100644 --- a/src/utils/resources/enactments/utils.ts +++ b/src/utils/resources/enactments/utils.ts @@ -1,7 +1,7 @@ import { V1beta1NodeNetworkConfigurationEnactment } from '@kubevirt-ui/kubevirt-api/nmstate'; export const getEnactmentStatus = (enactment: V1beta1NodeNetworkConfigurationEnactment): string => - enactment?.status?.conditions?.find((condition) => condition.status === 'True').type; + enactment?.status?.conditions?.find((condition) => condition.status === 'True')?.type; export const categorizeEnactments = (enactments: V1beta1NodeNetworkConfigurationEnactment[]) => { return (enactments || []).reduce( diff --git a/src/utils/resources/policies/utils.ts b/src/utils/resources/policies/utils.ts index d0c020da..9d99cfa8 100644 --- a/src/utils/resources/policies/utils.ts +++ b/src/utils/resources/policies/utils.ts @@ -33,7 +33,7 @@ export const isPolicyAppliedInNode = ( export const filterPolicyAppliedNodes = ( nodes: IoK8sApiCoreV1Node[], policy: V1NodeNetworkConfigurationPolicy, -) => nodes.filter((node) => isPolicyAppliedInNode(policy, node)); +) => (nodes || []).filter((node) => isPolicyAppliedInNode(policy, node)); export const getPolicyInterfaces = ( policy: V1NodeNetworkConfigurationPolicy,