chore: separate timeline components to own modules, fix lint issues

[karl-power]

Closes HDX-3518

Changes

• Separates the different Timeline related components into their own modules inside packages/app/src/components/TimelineChart.
• Fixes lint and TS errors / warnings.

Testing

• Trace waterfall view should work just like before.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
hyperdx-oss	Ready	Preview, Comment	Mar 4, 2026 8:21pm
hyperdx-v2-oss-app	Ready	Preview, Comment	Mar 4, 2026 8:21pm

[changeset-bot]

🦋 Changeset detected

Latest commit: ced77a3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
@hyperdx/app	Patch
@hyperdx/api	Patch
@hyperdx/otel-collector	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

E2E Test Results

✅ All tests passed • 75 passed • 4 skipped • 945s

Status	Count
✅ Passed	75
❌ Failed	0
⚠️ Flaky	3
⏭️ Skipped	4

Tests ran across 4 shards in parallel.

View full report →

[github-actions]

Claude finished @karl-power's task in 1m 17s —— View job

---

PR Review

✅ Clean refactoring overall — components are properly split and existing functionality is preserved. A few minor items to address:

• ⚠️ Dead prop in TimelineChartRowEvents.tsx:31 → onEventClick?: (event: TTimelineEvent) => void is defined in the type but never destructured or used in the component. Remove it or wire it up. Fix this →

• ⚠️ Self-referential null check in TimelineMouseCursor.tsx:39 → if (onMouseMove != null) checks the locally-defined onMouseMove function (always truthy). Remove the conditional, just call setXPerc(xPerc) directly. Fix this →

• ⚠️ Unused variable in TimelineMouseCursor.tsx:33 → const y = e.clientY - rect.top; is computed but never used. Remove it. Fix this →

• ℹ️ index.ts only re-exports TimelineChart — if consumers need TTimelineEvent or TTimelineSpanEventMarker types externally in the future, they'd need to import from internal paths. No issue now since no external consumers need them.

[github-actions]

Claude finished @karl-power's task in 34s —— View job

---

PR Review

The three issues flagged in the previous review are still unresolved:

• ⚠️ Dead prop TimelineChartRowEvents.tsx:31 → onEventClick? is declared in TimelineChartRowProps but never destructured or used in the component body. Remove it or wire it up. Fix this →

• ⚠️ Self-referential null check TimelineMouseCursor.tsx:39 → if (onMouseMove != null) always evaluates to true (it checks the locally-defined onMouseMove function, not a prop). Remove the if wrapper and just call setXPerc(xPerc) directly. Fix this →

• ⚠️ Unused variable TimelineMouseCursor.tsx:33 → const y = e.clientY - rect.top; is computed but never referenced. Remove it. Fix this →

Otherwise the refactor is clean — components are properly modularized and functionality is preserved.