Skip to content

Add actor and event-type filters for workflow timeline#149

Closed
akgohain wants to merge 1 commit intomainfrom
codex/add-timeline-filters-for-debugging
Closed

Add actor and event-type filters for workflow timeline#149
akgohain wants to merge 1 commit intomainfrom
codex/add-timeline-filters-for-debugging

Conversation

@akgohain
Copy link
Copy Markdown
Collaborator

Motivation

  • Improve debugging and study review by adding timeline filters for event actor and event type while preserving the default full timeline view.
  • Keep rendering performant for long timelines by avoiding unnecessary work when no filters are active.

Description

  • Add a reusable filtering utility timelineFilters.js that exports TIMELINE_ACTOR_OPTIONS, DEFAULT_TIMELINE_FILTERS, normalizeTimelineFilters, eventMatchesTimelineFilters, and filterTimelineEvents with a fast-path when no filters are set.
  • Add WorkflowTimelineFilters UI component providing an actor dropdown, an event-type text input, and a clear/reset button that calls the provided callbacks.
  • Add WorkflowTimeline container component that renders the filter controls and memoizes filtered events using useMemo so filtering recomputes only when events or filters change.
  • Add unit tests workflowTimelineFilters.test.js that cover default full-timeline behavior, actor/text filter combinations, and clear/reset interactions.

Testing

  • npm --prefix client test -- --watchAll=false --runInBand workflowTimelineFilters failed because this repository's client/package.json does not define a test script (expected fail).
  • cd client && npx react-scripts test --watchAll=false --runInBand workflowTimelineFilters passed and reported 3 tests passing for the new suite.
  • npm --prefix client run build completed successfully and produced a production build with warnings only.

Codex Task

@akgohain
Copy link
Copy Markdown
Collaborator Author

Consolidated into draft PR #165 (feat/codex-wave-integration). Closing this source draft to keep review in one place.

@akgohain akgohain closed this Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant