address copilot comments

Signed-off-by: Adhitya Mamallan <[email protected]>

Author: adhityamamallan

src/views/workflow-history-v2/config/workflow-history-details-row-parsers.config.ts

@@ -33,7 +33,7 @@ const workflowHistoryDetailsRowParsersConfig: Array<DetailsRowItemParser> = [
     icon: MdHourglassBottom,
   },
   {
-    name: '"attempt" greater than 1, as "retries"',
+    name: '"attempt" greater than 0, as "retries"',
     matcher: (name) => name === 'attempt',
     hide: (_, value) => typeof value === 'number' && value <= 0,
     icon: MdReplay,

src/views/workflow-history-v2/workflow-history-details-row/workflow-history-details-row.tsx

@@ -18,14 +18,22 @@ export default function WorkflowHistoryDetailsRow({
     return getParsedDetailsRowItems(detailsEntries);
   }, [detailsEntries]);
 
+  const negativePathsSet = useMemo(
+    () =>
+      new Set(
+        detailsEntries
+          .filter((entry) => entry.isNegative)
+          .map((entry) => entry.path)
+      ),
+    [detailsEntries]
+  );
+
   if (rowItems.length === 0) return <div />;
 
   return (
     <styled.DetailsRowContainer>
       {rowItems.map((item) => {
-        const isNegative = detailsEntries.some(
-          (detail) => detail.path === item.path && detail.isNegative
-        );
+        const isNegative = negativePathsSet.has(item.path);
 
         return (
           <StatefulTooltip

src/views/workflow-history-v2/workflow-history-event-group/workflow-history-event-group.tsx

@@ -75,7 +75,9 @@ export default function WorkflowHistoryEventGroup({
 
   const groupSummaryDetails = useMemo(
     () =>
-      summaryDetailsEntries.flatMap(([_, { eventDetails }]) => eventDetails),
+      summaryDetailsEntries.flatMap(
+        ([_eventId, { eventDetails }]) => eventDetails
+      ),
     [summaryDetailsEntries]
   );