Skip to content

Conversation

aaronvg
Copy link
Contributor

@aaronvg aaronvg commented Oct 6, 2025

  • type fixes
  • tweaks to blob storage hash
  • play with baml viz
  • graph view

Note

Adds a new Workflow visualization tab (ReactFlow + d3) and changes blob hashing to BLAKE3, with new UI blob/source endpoints and stricter TS typings.

  • Frontend (viz):
    • Workflow View: Adds WorkflowView with ReactFlow + d3-hierarchy, terminal/logs, and details panel; new Workflow tab (default) in prompt preview; temporarily hides TestPanel.
    • Layout: Ensures full-height containers and minor positioning tweaks.
    • Dependencies: Adds d3-hierarchy and @types/d3-hierarchy.
  • Backend/Runtime:
    • Blob Hashing: Switches blob hashing from SHA-256 to blake3 in baml-runtime and updates deps.
  • API/Types:
    • New UI Modules: Adds ui_blobs with GetBlobRequest/GetBlobResponse and BlobFormat; adds GetBamlSrcBundle endpoint for source bundles.
    • TypeScript Exports: Tightens TS types (anyunknown, refine Record<...> in ui_types and ui_baml_src).

Written by Cursor Bugbot for commit 3cf53c7. This will update automatically on new commits. Configure here.

Copy link

vercel bot commented Oct 6, 2025

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

Project Deployment Preview Comments Updated (UTC)
promptfiddle Ready Ready Preview Comment Oct 6, 2025 10:19pm


if (!mockTraceRuns || mockTraceRuns.length === 0) {
return <div className="p-4 text-red-500">No mock trace runs available</div>;
}
Copy link

Choose a reason for hiding this comment

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

Bug: Unsafe Access Before Null Check

The component accesses mockTraceRuns[0] with a non-null assertion before checking if the mockTraceRuns array is empty. This can cause a runtime error if no trace runs are available.

Additional Locations (1)

Fix in Cursor Fix in Web

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