Add single-line summary for history events

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

Author: adhityamamallan

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

@@ -0,0 +1,44 @@
+import {
+  MdHourglassBottom,
+  MdOutlineMonitorHeart,
+  MdReplay,
+} from 'react-icons/md';
+
+import { type DetailsRowItemParser } from '../workflow-history-details-row/workflow-history-details-row.types';
+import WorkflowHistoryDetailsRowJson from '../workflow-history-details-row-json/workflow-history-details-row-json';
+import WorkflowHistoryDetailsRowTooltipJson from '../workflow-history-details-row-tooltip-json/workflow-history-details-row-tooltip-json';
+
+const workflowHistoryDetailsRowParsersConfig: Array<DetailsRowItemParser> = [
+  {
+    name: 'Heartbeat time',
+    matcher: (name) => name === 'lastHeartbeatTime',
+    icon: MdOutlineMonitorHeart,
+  },
+  {
+    name: 'Json as PrettyJson',
+    matcher: (name, value) =>
+      value !== null &&
+      new RegExp(
+        '(input|result|details|failureDetails|Error|lastCompletionResult|heartbeatDetails|lastFailureDetails)$'
+      ).test(name),
+    icon: null,
+    customRenderValue: WorkflowHistoryDetailsRowJson,
+    customTooltipContent: WorkflowHistoryDetailsRowTooltipJson,
+    invertTooltipColors: true,
+  },
+  {
+    name: 'Timeouts with timer icon',
+    matcher: (name) =>
+      new RegExp('(TimeoutSeconds|BackoffSeconds|InSeconds)$').test(name),
+    icon: MdHourglassBottom,
+  },
+  {
+    name: '"attempt" greater than 1, as "retries"',
+    matcher: (name) => name === 'attempt',
+    hide: (_, value) => typeof value === 'number' && value <= 0,
+    icon: MdReplay,
+    customTooltipContent: () => 'retries',
+  },
+];
+
+export default workflowHistoryDetailsRowParsersConfig;

src/views/workflow-history-v2/workflow-history-details-row-json/workflow-history-details-row-json.styles.ts

@@ -0,0 +1,15 @@
+import { styled as createStyled } from 'baseui';
+
+export const styled = {
+  JsonViewContainer: createStyled<'div', { $isNegative: boolean }>(
+    'div',
+    ({ $theme, $isNegative }) => ({
+      color: $isNegative ? $theme.colors.contentNegative : '#A964F7',
+      maxWidth: '360px',
+      overflow: 'hidden',
+      whiteSpace: 'nowrap',
+      textOverflow: 'ellipsis',
+      ...$theme.typography.MonoParagraphXSmall,
+    })
+  ),
+};

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

@@ -0,0 +1,16 @@
+import losslessJsonStringify from '@/utils/lossless-json-stringify';
+
+import { type DetailsRowValueComponentProps } from '../workflow-history-details-row/workflow-history-details-row.types';
+
+import { styled } from './workflow-history-details-row-json.styles';
+
+export default function WorkflowHistoryDetailsRowJson({
+  value,
+  isNegative,
+}: DetailsRowValueComponentProps) {
+  return (
+    <styled.JsonViewContainer $isNegative={isNegative ?? false}>
+      {losslessJsonStringify(value)}
+    </styled.JsonViewContainer>
+  );
+}

src/views/workflow-history-v2/workflow-history-details-row-json/workflow-history-details-row-json.types.ts

@@ -0,0 +1,3 @@
+import { type DetailsRowValueComponentProps } from '../workflow-history-details-row/workflow-history-details-row.types';
+
+export type Props = DetailsRowValueComponentProps;

src/views/workflow-history-v2/workflow-history-details-row-tooltip-json/workflow-history-details-row-tooltip-json.styles.ts

@@ -0,0 +1,13 @@
+import { styled as createStyled } from 'baseui';
+
+export const styled = {
+  JsonPreviewContainer: createStyled('div', ({ $theme }) => ({
+    display: 'flex',
+    flexDirection: 'column',
+    alignItems: 'flex-start',
+    gap: $theme.sizing.scale200,
+  })),
+  JsonPreviewLabel: createStyled('div', ({ $theme }) => ({
+    ...$theme.typography.LabelXSmall,
+  })),
+};

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

@@ -0,0 +1,21 @@
+import WorkflowHistoryEventDetailsJson from '@/views/workflow-history/workflow-history-event-details-json/workflow-history-event-details-json';
+
+import { type DetailsRowValueComponentProps } from '../workflow-history-details-row/workflow-history-details-row.types';
+
+import { styled } from './workflow-history-details-row-tooltip-json.styles';
+
+export default function WorkflowHistoryDetailsRowTooltipJson({
+  value,
+  label,
+  isNegative,
+}: DetailsRowValueComponentProps) {
+  return (
+    <styled.JsonPreviewContainer>
+      <styled.JsonPreviewLabel>{label}</styled.JsonPreviewLabel>
+      <WorkflowHistoryEventDetailsJson
+        entryValue={value}
+        isNegative={isNegative}
+      />
+    </styled.JsonPreviewContainer>
+  );
+}

src/views/workflow-history-v2/workflow-history-details-row-tooltip-json/workflow-history-details-row-tooltip-json.types.ts

@@ -0,0 +1,3 @@
+import { type DetailsRowValueComponentProps } from '../workflow-history-details-row/workflow-history-details-row.types';
+
+export type Props = DetailsRowValueComponentProps;

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

@@ -0,0 +1,129 @@
+import { render, screen, userEvent } from '@/test-utils/rtl';
+
+import type { WorkflowPageParams } from '@/views/workflow-page/workflow-page.types';
+
+import { type EventDetailsEntries } from '../../workflow-history-event-details/workflow-history-event-details.types';
+import WorkflowHistoryDetailsRow from '../workflow-history-details-row';
+
+jest.mock('../helpers/get-parsed-details-row-items', () =>
+  jest.fn((detailsEntries: EventDetailsEntries) =>
+    detailsEntries
+      .filter((entry) => !entry.isGroup)
+      .map((entry, index) => ({
+        path: entry.path,
+        label: entry.path,
+        value: entry.value,
+        icon: ({ size }: any) => (
+          <span data-testid={`icon-${entry.path}`} data-size={size} />
+        ),
+        renderValue: ({ value, isNegative }: any) => (
+          <span data-testid={`field-${entry.path}`} data-negative={isNegative}>
+            {value}
+          </span>
+        ),
+        renderTooltip: ({ label }: any) => (
+          <span data-testid={`tooltip-${entry.path}`}>{label}</span>
+        ),
+        invertTooltipColors: index === 1, // Second item has inverted tooltip
+        omitWrapping: index === 2, // Third item omits wrapping
+      }))
+  )
+);
+
+const mockWorkflowPageParams: WorkflowPageParams = {
+  cluster: 'test-cluster',
+  domain: 'test-domain',
+  workflowId: 'test-workflow',
+  runId: 'test-run',
+};
+
+const mockDetailsEntries: EventDetailsEntries = [
+  {
+    key: 'field1',
+    path: 'field1',
+    isGroup: false,
+    value: 'value1',
+    isNegative: false,
+    renderConfig: null,
+  },
+  {
+    key: 'field2',
+    path: 'field2',
+    isGroup: false,
+    value: 'value2',
+    isNegative: true,
+    renderConfig: null,
+  },
+  {
+    key: 'field3',
+    path: 'field3',
+    isGroup: false,
+    value: 'value3',
+    isNegative: false,
+    renderConfig: null,
+  },
+];
+
+describe(WorkflowHistoryDetailsRow.name, () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it('should render details row items when detailsEntries has items', () => {
+    setup();
+
+    expect(screen.getByTestId('field-field1')).toBeInTheDocument();
+    expect(screen.getByTestId('field-field2')).toBeInTheDocument();
+    expect(screen.getByTestId('field-field3')).toBeInTheDocument();
+    expect(screen.getByText('value1')).toBeInTheDocument();
+    expect(screen.getByText('value2')).toBeInTheDocument();
+    expect(screen.getByText('value3')).toBeInTheDocument();
+  });
+
+  it('should mark negative fields correctly', () => {
+    setup();
+
+    const negativeField = screen.getByTestId('field-field2');
+    expect(negativeField).toHaveAttribute('data-negative', 'true');
+
+    const positiveField = screen.getByTestId('field-field1');
+    expect(positiveField).toHaveAttribute('data-negative', 'false');
+  });
+
+  it('should render icons when provided in item config', () => {
+    setup();
+
+    expect(screen.getByTestId('icon-field1')).toBeInTheDocument();
+    expect(screen.getByTestId('icon-field2')).toBeInTheDocument();
+    expect(screen.getByTestId('icon-field3')).toBeInTheDocument();
+  });
+
+  it('should render tooltip content on hover', async () => {
+    const { user } = setup();
+
+    const field1 = screen.getByTestId('field-field1');
+    await user.hover(field1);
+
+    expect(await screen.findByTestId('tooltip-field1')).toBeInTheDocument();
+    expect(screen.getByText('field1')).toBeInTheDocument();
+  });
+});
+
+function setup({
+  detailsEntries = mockDetailsEntries,
+  workflowPageParams = mockWorkflowPageParams,
+}: {
+  detailsEntries?: EventDetailsEntries;
+  workflowPageParams?: WorkflowPageParams;
+} = {}) {
+  const user = userEvent.setup();
+
+  const renderResult = render(
+    <WorkflowHistoryDetailsRow
+      detailsEntries={detailsEntries}
+      {...workflowPageParams}
+    />
+  );
+
+  return { user, ...renderResult };
+}