Skip to content

Conversation

@simeonlee
Copy link
Member

Status: WIP - NOT READY FOR REVIEW

This is part of the error boundaries work, split into separate PRs for review.

Stacked on: #5588

Note: InferenceDetailContent.tsx has overlapping changes with PR 2a (#5597). These PRs need to be coordinated when merging.

Summary

Transforms inference detail page to use streaming data patterns:

  • Primary data (inference) is awaited for immediate render
  • Secondary data (model inferences, feedback) is deferred for streaming
  • Uses Await/Suspense for progressive loading
  • Adds skeleton loading states for deferred sections
  • Includes TryWithButton variant response functionality

This improves perceived performance by showing the main inference data immediately while loading secondary data in the background.

Test plan

  • Typecheck passes
  • Lint passes
  • Unit tests pass
  • Manual verification: Page loads instantly with skeletons, secondary data streams in

Transforms inference detail page to use streaming data patterns:
- Primary data (inference) is awaited for immediate render
- Secondary data (model inferences, feedback) is deferred for streaming
- Uses Await/Suspense for progressive loading
- Adds skeleton loading states for deferred sections
- Includes TryWithButton variant response functionality

Note: InferenceDetailContent.tsx has overlapping changes with PR 2a.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants