[web-shared][web] Improve o11y UX with decoupled pagination, stream virtualization, and decrypt actions

[karthikscale3]

Changes

• Decoupled trace-viewer and events-tab data loading by introducing a dedicated events data hook (use-events-list-data) and related wiring in run detail views, so each panel can fetch independently and avoid blocking each other.
• Added incremental pagination UX (Load more + Spinner) for events/streams, so large runs load progressively instead of rendering everything at once.
• Virtualized the streams tab rendering path, so long stream lists perform better (lower initial render cost, smoother scrolling/interactions).
• Added reusable Decrypt actions in both event and stream views (decrypt-button + integrations), so encrypted payloads can be inspected directly in the UI when needed.
• Updated shared component exports and sidebar/detail integrations to support the new controls and data flow, keeping observability UI behavior consistent across panels.

[VaguelySerious]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

[karthikscale3]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

It's fixed. Passing an enabled boolean flag here and top level gates it with enabled: activeTab === 'events'

Addressed the comment. Github is not showing me the option to resolve the comment for some reason. So dismissing it instead.

[karthikscale3]

Changes

• Decoupled trace-viewer and events-tab data loading by introducing a dedicated events data hook (use-events-list-data) and related wiring in run detail views, so each panel can fetch independently and avoid blocking each other.
• Added incremental pagination UX (Load more + Spinner) for events/streams, so large runs load progressively instead of rendering everything at once.
• Virtualized the streams tab rendering path, so long stream lists perform better (lower initial render cost, smoother scrolling/interactions).
• Added reusable Decrypt actions in both event and stream views (decrypt-button + integrations), so encrypted payloads can be inspected directly in the UI when needed.
• Updated shared component exports and sidebar/detail integrations to support the new controls and data flow, keeping observability UI behavior consistent across panels.

[VaguelySerious]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

[karthikscale3]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

It's fixed. Passing an enabled boolean flag here and top level gates it with enabled: activeTab === 'events'

Addressed the comment. Github is not showing me the option to resolve the comment for some reason. So dismissing it instead.

[karthikscale3]

Changes

• Decoupled trace-viewer and events-tab data loading by introducing a dedicated events data hook (use-events-list-data) and related wiring in run detail views, so each panel can fetch independently and avoid blocking each other.
• Added incremental pagination UX (Load more + Spinner) for events/streams, so large runs load progressively instead of rendering everything at once.
• Virtualized the streams tab rendering path, so long stream lists perform better (lower initial render cost, smoother scrolling/interactions).
• Added reusable Decrypt actions in both event and stream views (decrypt-button + integrations), so encrypted payloads can be inspected directly in the UI when needed.
• Updated shared component exports and sidebar/detail integrations to support the new controls and data flow, keeping observability UI behavior consistent across panels.

[VaguelySerious]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

[karthikscale3]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

It's fixed. Passing an enabled boolean flag here and top level gates it with enabled: activeTab === 'events'

Addressed the comment. Github is not showing me the option to resolve the comment for some reason. So dismissing it instead.

[karthikscale3]

Changes

• Decoupled trace-viewer and events-tab data loading by introducing a dedicated events data hook (use-events-list-data) and related wiring in run detail views, so each panel can fetch independently and avoid blocking each other.
• Added incremental pagination UX (Load more + Spinner) for events/streams, so large runs load progressively instead of rendering everything at once.
• Virtualized the streams tab rendering path, so long stream lists perform better (lower initial render cost, smoother scrolling/interactions).
• Added reusable Decrypt actions in both event and stream views (decrypt-button + integrations), so encrypted payloads can be inspected directly in the UI when needed.
• Updated shared component exports and sidebar/detail integrations to support the new controls and data flow, keeping observability UI behavior consistent across panels.

[VaguelySerious]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

[karthikscale3]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

It's fixed. Passing an enabled boolean flag here and top level gates it with enabled: activeTab === 'events'

Addressed the comment. Github is not showing me the option to resolve the comment for some reason. So dismissing it instead.

[karthikscale3]

Changes

• Decoupled trace-viewer and events-tab data loading by introducing a dedicated events data hook (use-events-list-data) and related wiring in run detail views, so each panel can fetch independently and avoid blocking each other.
• Added incremental pagination UX (Load more + Spinner) for events/streams, so large runs load progressively instead of rendering everything at once.
• Virtualized the streams tab rendering path, so long stream lists perform better (lower initial render cost, smoother scrolling/interactions).
• Added reusable Decrypt actions in both event and stream views (decrypt-button + integrations), so encrypted payloads can be inspected directly in the UI when needed.
• Updated shared component exports and sidebar/detail integrations to support the new controls and data flow, keeping observability UI behavior consistent across panels.

[VaguelySerious]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

[karthikscale3]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

It's fixed. Passing an enabled boolean flag here and top level gates it with enabled: activeTab === 'events'

Addressed the comment. Github is not showing me the option to resolve the comment for some reason. So dismissing it instead.

[karthikscale3]

Changes

• Decoupled trace-viewer and events-tab data loading by introducing a dedicated events data hook (use-events-list-data) and related wiring in run detail views, so each panel can fetch independently and avoid blocking each other.
• Added incremental pagination UX (Load more + Spinner) for events/streams, so large runs load progressively instead of rendering everything at once.
• Virtualized the streams tab rendering path, so long stream lists perform better (lower initial render cost, smoother scrolling/interactions).
• Added reusable Decrypt actions in both event and stream views (decrypt-button + integrations), so encrypted payloads can be inspected directly in the UI when needed.
• Updated shared component exports and sidebar/detail integrations to support the new controls and data flow, keeping observability UI behavior consistent across panels.

[VaguelySerious]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

[karthikscale3]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

It's fixed. Passing an enabled boolean flag here and top level gates it with enabled: activeTab === 'events'

Addressed the comment. Github is not showing me the option to resolve the comment for some reason. So dismissing it instead.

[karthikscale3]

Changes

• Decoupled trace-viewer and events-tab data loading by introducing a dedicated events data hook (use-events-list-data) and related wiring in run detail views, so each panel can fetch independently and avoid blocking each other.
• Added incremental pagination UX (Load more + Spinner) for events/streams, so large runs load progressively instead of rendering everything at once.
• Virtualized the streams tab rendering path, so long stream lists perform better (lower initial render cost, smoother scrolling/interactions).
• Added reusable Decrypt actions in both event and stream views (decrypt-button + integrations), so encrypted payloads can be inspected directly in the UI when needed.
• Updated shared component exports and sidebar/detail integrations to support the new controls and data flow, keeping observability UI behavior consistent across panels.

[VaguelySerious]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

[karthikscale3]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

It's fixed. Passing an enabled boolean flag here and top level gates it with enabled: activeTab === 'events'

Addressed the comment. Github is not showing me the option to resolve the comment for some reason. So dismissing it instead.

[karthikscale3]

Changes

• Decoupled trace-viewer and events-tab data loading by introducing a dedicated events data hook (use-events-list-data) and related wiring in run detail views, so each panel can fetch independently and avoid blocking each other.
• Added incremental pagination UX (Load more + Spinner) for events/streams, so large runs load progressively instead of rendering everything at once.
• Virtualized the streams tab rendering path, so long stream lists perform better (lower initial render cost, smoother scrolling/interactions).
• Added reusable Decrypt actions in both event and stream views (decrypt-button + integrations), so encrypted payloads can be inspected directly in the UI when needed.
• Updated shared component exports and sidebar/detail integrations to support the new controls and data flow, keeping observability UI behavior consistent across panels.

[VaguelySerious]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

[karthikscale3]

LGTM under assumption we'll get rid of withData soon for event listing, and we'll also bypass useEventsListData entirely in the special case where the run is terminal and the trace viewer already fetched all events

Never call useEventsListData top-level unless the user navigates to the event list tab

I think this is still missing in this PR? You mentioned you fixed it, but I'm not seeing it

It's fixed. Passing an enabled boolean flag here and top level gates it with enabled: activeTab === 'events'

Addressed the comment. Github is not showing me the option to resolve the comment for some reason. So dismissing it instead.