Skip to content

fix(artifacts): delimit sandbox metadata - #173

Merged
iamjr15 merged 1 commit into
mainfrom
fix/artifact-stdout-protocol
Aug 5, 2026
Merged

fix(artifacts): delimit sandbox metadata#173
iamjr15 merged 1 commit into
mainfrom
fix/artifact-stdout-protocol

Conversation

@iamjr15

@iamjr15 iamjr15 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Why

After the large run-code transport fix, production PDF rendering executed successfully but artifact parsing still failed when sandbox or renderer diagnostics shared stdout with the final JSON metadata.

What changed

  • emit a reserved final-artifact marker immediately before metadata JSON
  • parse only the payload after the last marker
  • retain compatibility with a clean JSON-only response
  • document the sandbox artifact stdout protocol

Verification

  • pnpm lint
  • pnpm typecheck
  • pnpm turbo build --force
  • pnpm deadcode
  • pnpm architecture:check
  • pnpm turbo skills:build
  • live Daytona diagnostic with an injected renderer log followed by marked PDF metadata
  • confirmed the marker remains the final output boundary and the rendered PDF payload is valid

Production deep-research and PDF acceptance will be rerun after deployment.

@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
cheatcode Skipped Skipped Aug 5, 2026 10:17am

@iamjr15
iamjr15 merged commit 7e6fa96 into main Aug 5, 2026
6 checks passed
@iamjr15
iamjr15 deleted the fix/artifact-stdout-protocol branch August 5, 2026 10:19
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