Address PR review feedback

- Use ?? instead of || for count/length in StatusCard computed
- Add .filter((c) => c) to workload cards getter for null safety
- Initialize summaries.pods = null explicitly when no pod selector
- Use normalizedType in useResourceSummary dispatch for consistency

Author: codyrancher

shell/components/Resource/Detail/Card/StateCard/composables.ts

@@ -293,7 +293,7 @@ export function useResourceSummary(type: string, opt: ResourceSummaryOpt) {
     const id = ++fetchId;
 
     try {
-      const result = await store.dispatch('cluster/fetchResourceSummary', { type, opt });
+      const result = await store.dispatch('cluster/fetchResourceSummary', { type: normalizedType, opt });
 
       if (id !== fetchId) {
         return;

shell/components/Resource/Detail/Card/StatusCard/index.vue

@@ -106,10 +106,10 @@ const percent = (count: number, total: number) => {
 
 const count = computed(() => {
   if (summaryStateCounts.value) {
-    return props.summaryData?.count || 0;
+    return props.summaryData?.count ?? 0;
   }
 
-  return props.resources?.length || 0;
+  return props.resources?.length ?? 0;
 });
 
 const segmentColors = computed(() => Object.keys(segmentAccumulator.value) as StateColor[]);

shell/models/workload.js

@@ -595,6 +595,8 @@ export default class Workload extends WorkloadService {
 
     const podSelectorStr = this.podSelector;
 
+    summaries.pods = null;
+
     if (podSelectorStr) {
       const filters = [nsFilter];
 
@@ -931,6 +933,6 @@ export default class Workload extends WorkloadService {
       this.resourcesCard,
       this.insightCard,
       ...this._cards
-    ];
+    ].filter((c) => c);
   }
 }