Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show last computed prompt when runtime is loading, and be more resilient to exception from rust #1325

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

aaronvg
Copy link
Contributor

@aaronvg aaronvg commented Jan 14, 2025

Important

Enhance prompt preview loading and error handling in the playground panel, with UI adjustments in ClientGraphView.tsx and ErrorFallback.tsx.

  • Behavior:
    • In prompt-preview-content.tsx, show last known prompt if runtime is loading and no new preview is available.
    • In atoms.ts, wrap runtime computation in a try-catch to handle exceptions from Rust gracefully.
  • UI Changes:
    • Adjust ClientGraphView.tsx height from 400px to 350px.
    • Update ErrorFallback.tsx to reload the page on error reset.
  • Misc:
    • Minor CSS class reordering in ErrorFallback.tsx.

This description was created by Ellipsis for 63e376d. It will automatically update as commits are pushed.

Copy link

vercel bot commented Jan 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
baml ❌ Failed (Inspect) Jan 14, 2025 0:37am

@aaronvg aaronvg enabled auto-merge January 14, 2025 00:33
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me! Reviewed everything up to 63e376d in 27 seconds

More details
  • Looked at 125 lines of code in 4 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. typescript/playground-common/src/shared/baml-project-panel/atoms.ts:78
  • Draft comment:
    WasmDiagnosticError is not defined in this scope. Ensure it is imported or defined before use.
  • Reason this comment was not posted:
    Comment was not on a valid diff hunk.

Workflow ID: wflow_hUWlJco5LQqGvDIq


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@aaronvg aaronvg added this pull request to the merge queue Jan 14, 2025
Merged via the queue into canary with commit 3b116fa Jan 14, 2025
10 of 11 checks passed
@aaronvg aaronvg deleted the playground-fixes3 branch January 14, 2025 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant