Skip to content

convert inline base64 attachments to s3 pointers via span processor#100

Open
Andrew Kent (realark) wants to merge 1 commit into
mainfrom
ark/auto-attachments
Open

convert inline base64 attachments to s3 pointers via span processor#100
Andrew Kent (realark) wants to merge 1 commit into
mainfrom
ark/auto-attachments

Conversation

@realark
Copy link
Copy Markdown
Collaborator

@realark Andrew Kent (realark) commented May 6, 2026

adds attachment process to standard braintrust tracing

  • scan for base64 images in braintrust.input_json and braintrust.output_json:
  • for each image:
  • extract the base64 string
  • upload to s3
  • replace base64 image with a pointer to the s3 image

@realark Andrew Kent (realark) force-pushed the ark/auto-attachments branch 8 times, most recently from b82bb65 to f782ca3 Compare May 13, 2026 01:34
@realark Andrew Kent (realark) changed the base branch from main to ark/vcr-improvements May 13, 2026 01:34
@realark Andrew Kent (realark) force-pushed the ark/auto-attachments branch 6 times, most recently from 9267811 to a781c63 Compare May 13, 2026 03:21
Base automatically changed from ark/vcr-improvements to main May 13, 2026 15:23
@realark Andrew Kent (realark) force-pushed the ark/auto-attachments branch 2 times, most recently from 5fe29f2 to 1733366 Compare May 13, 2026 16:38
@realark Andrew Kent (realark) marked this pull request as ready for review May 13, 2026 16:59
Comment thread braintrust-sdk/src/main/java/dev/braintrust/trace/AttachmentUploader.java Outdated
Comment thread braintrust-sdk/src/main/java/dev/braintrust/trace/AttachmentProcessor.java Outdated
Comment thread braintrust-sdk/src/main/java/dev/braintrust/trace/BraintrustShutdownHook.java Outdated
adds attachment process to standard braintrust tracing

- scan for base64 images in `braintrust.input_json` and `braintrust.output_json`:
- for each image:
-   extract the base64 string
-   upload to s3
-   replace base64 image with a pointer to the s3 image
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.

2 participants