fix: handle null and undefined cases in flattenObjectProperty and ensure safe summation in AppStats

Author: itstauq

justfile

@@ -458,7 +458,7 @@ bump-version level="patch":
 
     level = "{{ level }}"
     valid_levels = ["major", "minor", "patch"]
-    
+
     if level not in valid_levels:
         print(f"Error: Invalid level. Use 'major', 'minor', or 'patch'.", file=sys.stderr)
         sys.exit(1)
@@ -470,16 +470,16 @@ bump-version level="patch":
             "--format", "json",
             "--increment", level
         ], capture_output=True, text=True, check=True)
-        
+
         version_info = json.loads(result.stdout)
         current_version = version_info["current_version"]
         new_version = version_info["new_version"]
-        
+
         print(f"Bumping version from {current_version} to {new_version}")
-        
+
         # Bump version
         subprocess.run(["uvx", "bump-my-version", "bump", level], check=True)
-        
+
     except subprocess.CalledProcessError as e:
         print(f"Error running bump-my-version: {e}", file=sys.stderr)
         sys.exit(e.returncode)

src-frontend/components/apps/app-stats.tsx

@@ -79,10 +79,15 @@ function formatType(type: number): string {
 }
 
 const flattenObjectProperty = <T extends object>(
-  obj: T,
+  obj: T | null | undefined,
   property: string,
   childrenKey: keyof T,
 ): Array<unknown> => {
+  // Handle null or undefined objects
+  if (!obj) {
+    return [];
+  }
+
   const getNestedValue = (obj: object, path: string): unknown => {
     return path
       .split(".")
@@ -312,13 +317,13 @@ export function AppStats({ appId }: { appId: string }) {
     stats,
     "cpuPercent",
     "children",
-  ).reduce((acc, curr) => acc + (curr as number), 0);
+  ).reduce((acc, curr) => acc + ((curr as number) || 0), 0);
 
   const cumulativeNumThreads = flattenObjectProperty(
     stats,
     "numThreads",
     "children",
-  ).reduce((acc, curr) => acc + (curr as number), 0);
+  ).reduce((acc, curr) => acc + ((curr as number) || 0), 0);
 
   const totalMemory =
     ((stats.memoryInfo?.rss ?? 0) / stats.memoryPercent) * 100;
@@ -327,31 +332,33 @@ export function AppStats({ appId }: { appId: string }) {
     stats,
     "memoryPercent",
     "children",
-  ).reduce((acc, curr) => acc + (curr as number), 0);
+  ).reduce((acc, curr) => acc + ((curr as number) || 0), 0);
 
   const cumulativeMemoryRSS = flattenObjectProperty(
     stats,
     "memoryInfo.rss",
     "children",
-  ).reduce((acc, curr) => acc + (curr as number), 0);
+  ).reduce((acc, curr) => acc + ((curr as number) || 0), 0);
 
   const cumulativeMemoryVMS =
     flattenObjectProperty(stats, "memoryInfo.vms", "children").reduce(
-      (acc, curr) => acc + (curr as number),
+      (acc, curr) => acc + ((curr as number) || 0),
       0,
     ) / 1000;
 
   const cumulativeMemorySwap = flattenObjectProperty(
     stats,
     "memoryInfo.swap",
     "children",
-  ).reduce((acc, curr) => acc + (curr as number), 0);
+  ).reduce((acc, curr) => acc + ((curr as number) || 0), 0);
 
   const allConnections: ConnectionStat[] = flattenObjectProperty(
     stats,
     "connections",
     "children",
-  ).flat() as ConnectionStat[];
+  )
+    .flat()
+    .filter(Boolean) as ConnectionStat[];
 
   return (
     <div className="flex h-full flex-col gap-2">