Skip to content

Feat/sandbox logs data field#301

Draft
matthewlouisbrockman wants to merge 3 commits intomainfrom
feat/sandbox-logs-data-field
Draft

Feat/sandbox logs data field#301
matthewlouisbrockman wants to merge 3 commits intomainfrom
feat/sandbox-logs-data-field

Conversation

@matthewlouisbrockman
Copy link
Copy Markdown

for preview

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 23, 2026

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

Project Deployment Actions Updated (UTC)
web Ready Ready Preview, Comment Apr 23, 2026 1:30am
web-juliett Ready Ready Preview, Comment Apr 23, 2026 1:30am

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 23, 2026

PR Summary

Medium Risk
Moderate risk: changes log-mapping semantics (splitting a single infra entry into multiple UI log rows) and adds new virtualized expandable-row UI, which could affect pagination/cursor behavior and log rendering performance.

Overview
Adds structured sandbox log support end-to-end: the backend now parses log.fields.data (including JSON and JSON-lines), normalizes level/logger/message, and may emit multiple SandboxLogModel rows per infra log entry with optional fields, origin, and capturedBy metadata.

Updates sandbox log APIs to use mapInfraSandboxLogToModels and adjusts cursor/hasMore logic, and enhances the dashboard log viewer with a new Logger column plus expandable per-row details showing parsed fields/structured entries, with actions to copy a deep link (?log=...) or the log JSON.

Reviewed by Cursor Bugbot for commit 92e789b. Bugbot is set up for automated code reviews on this repo. Configure here.

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