fix: added chaining operator for 'conditions' property (#137)

Author: lucasgoral

src/components/ControlPlane/FluxList.tsx

@@ -153,10 +153,11 @@ export default function FluxList() {
       return {
         name: item.metadata.name,
         isReady:
-          item.status.conditions.find((x) => x.type === 'Ready')?.status ===
+          item?.status?.conditions?.find((x) => x.type === 'Ready')?.status ===
           'True',
-        statusUpdateTime: item.status.conditions.find((x) => x.type === 'Ready')
-          ?.lastTransitionTime,
+        statusUpdateTime: item.status?.conditions?.find(
+          (x) => x.type === 'Ready',
+        )?.lastTransitionTime,
         revision: shortenCommitHash(item.status.artifact?.revision ?? '-'),
         created: timeAgo.format(new Date(item.metadata.creationTimestamp)),
         item: item,
@@ -168,10 +169,11 @@ export default function FluxList() {
       return {
         name: item.metadata.name,
         isReady:
-          item.status.conditions.find((x) => x.type === 'Ready')?.status ===
+          item.status?.conditions?.find((x) => x.type === 'Ready')?.status ===
           'True',
-        statusUpdateTime: item.status.conditions.find((x) => x.type === 'Ready')
-          ?.lastTransitionTime,
+        statusUpdateTime: item.status?.conditions?.find(
+          (x) => x.type === 'Ready',
+        )?.lastTransitionTime,
         created: timeAgo.format(new Date(item.metadata.creationTimestamp)),
         item: item,
       };

src/components/ControlPlane/MCPHealthPopoverButton.tsx

@@ -63,7 +63,7 @@ export default function MCPHealthPopoverButton({
 
     const statusDetails = mcpStatus?.conditions
       ? `${t('MCPHealthPopoverButton.statusDetailsLabel')}: ${mcpStatus.status}\n\n${t('MCPHealthPopoverButton.detailsLabel')}\n` +
-        mcpStatus.conditions
+        mcpStatus?.conditions
           .map((condition) => {
             let text = `- ${condition.type}: ${condition.status}\n`;
             if (condition.reason)
@@ -160,7 +160,7 @@ function StatusTable({
         scaleWidthMode="Smart"
         columns={tableColumns}
         data={
-          status?.conditions.sort((a, b) => {
+          status?.conditions?.sort((a, b) => {
             return a.type < b.type ? -1 : 1;
           }) ?? []
         }

src/components/ControlPlane/ManagedResources.tsx

@@ -106,10 +106,10 @@ export function ManagedResources() {
       ?.filter((managedResource) => managedResource.items)
       .flatMap((managedResource) =>
         managedResource.items?.map((item) => {
-          const conditionSynced = item.status.conditions?.find(
+          const conditionSynced = item.status?.conditions?.find(
             (condition) => condition.type === 'Synced',
           );
-          const conditionReady = item.status.conditions?.find(
+          const conditionReady = item.status?.conditions?.find(
             (condition) => condition.type === 'Ready',
           );
 

src/components/ControlPlane/Providers.tsx

@@ -106,10 +106,10 @@ export function Providers() {
 
   const rows: ProvidersRow[] =
     providers?.items?.map((item) => {
-      const installed = item.status.conditions?.find(
+      const installed = item.status?.conditions?.find(
         (condition) => condition.type === 'Installed',
       );
-      const healthy = item.status.conditions?.find(
+      const healthy = item.status?.conditions?.find(
         (condition) => condition.type === 'Healthy',
       );
       return {

src/lib/api/types/crossplane/listManagedResources.ts

@@ -9,8 +9,8 @@ export type ManagedResourcesResponse = [
           name: string;
           creationTimestamp: string;
         };
-        status: {
-          conditions: [
+        status?: {
+          conditions?: [
             {
               type: 'Ready' | 'Synced' | unknown;
               status: 'True' | 'False';

src/lib/api/types/crossplane/listProviders.ts

@@ -11,8 +11,8 @@ export type ProvidersListResponse = {
         name: string;
         creationTimestamp: string;
       };
-      status: {
-        conditions: [
+      status?: {
+        conditions?: [
           {
             type: 'Healthy' | 'Installed' | unknown;
             status: 'True' | 'False';

src/lib/api/types/flux/listGitRepo.ts

@@ -15,7 +15,7 @@ export type GitReposResponse = {
         artifact: {
           revision: string;
         };
-        conditions: [
+        conditions?: [
           {
             status: string;
             type: string;