Skip to content

Add workflow timeline actor and event-type filters with tests#150

Closed
akgohain wants to merge 1 commit into
mainfrom
codex/add-timeline-filters-for-debugging-7zgxss
Closed

Add workflow timeline actor and event-type filters with tests#150
akgohain wants to merge 1 commit into
mainfrom
codex/add-timeline-filters-for-debugging-7zgxss

Conversation

@akgohain
Copy link
Copy Markdown
Collaborator

Motivation

  • Provide timeline filtering controls (actor + event-type text) to aid debugging and study review while keeping the default full timeline visible when no filters are applied.
  • Centralize filter state so multiple components can reuse or extend filtering behavior.

Description

  • Added a WorkflowTimelineFilterContext provider exposing filters, setActorFilter, setEventTypeFilter, and clearFilters to centralize filter state.
  • Implemented WorkflowTimelineFilters UI with actor options (all/user/agent/system), an event-type text input, and a clear/reset button, using a simple native select to avoid test environment issues.
  • Implemented WorkflowTimeline which applies filters with useMemo for efficient filtering of long event lists and preserves the full timeline view when filters are empty.
  • Added unit tests covering default full timeline, combined actor + event-type filtering, and clear/reset behavior, and added a test script to client/package.json to run the suite.

Testing

  • Ran npm --prefix client test -- --watchAll=false --runInBand workflowTimelineFilters and the new test suite passed (3/3 tests).
  • Ran npm --prefix client run build and the production build completed successfully (build succeeded with unrelated ESLint warnings reported).

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