Skip to content

Conversation

@knjiang
Copy link

@knjiang knjiang commented Nov 21, 2025

This PR enables braintrust as a storage option.

@changeset-bot
Copy link

changeset-bot bot commented Nov 21, 2025

🦋 Changeset detected

Latest commit: 43f8332

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

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

@vercel
Copy link

vercel bot commented Nov 21, 2025

@knjiang is attempting to deploy a commit to the Skill Recordings Team on Vercel.

A member of the Team first needs to authorize it.

@mattpocock mattpocock modified the milestones: Post-v1, v1 Dec 3, 2025
@@ -0,0 +1,332 @@
import type { Evalite } from "./types.js";
Copy link
Author

@knjiang knjiang Dec 15, 2025

Choose a reason for hiding this comment

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

this mostly wraps in-memory.ts and "syncs" writes to braintrust.

}

// If using Braintrust storage, output the experiment URL
// Use duck typing to check for getExperimentUrl method
Copy link
Author

Choose a reason for hiding this comment

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

this powers the link below:

Image

feel a bit iffy on the ducktyping but i didn't want to pull in braintrust so lmk if there's any other suggestions!

@knjiang knjiang marked this pull request as ready for review December 15, 2025 22:35
@tuntisz
Copy link

tuntisz commented Dec 24, 2025

Love to see this! Following along...

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.

3 participants