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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Mar 29, 2026 0:37am

@greptile review

@BugBot review

PR Summary

Medium Risk
Broad logging changes across workflow execution and copilot/orchestrator paths could increase log volume and affect alerting/performance, but do not change core execution behavior.

Overview
Improves observability by introducing Logger.withMetadata() in @sim/logger and updating log formatting to automatically append key/value metadata to every line.

Refactors workflow execution (/api/workflows/[id]/execute and executor engine/block execution) to use request/execution-scoped loggers with consistent metadata (requestId, workflowId, workspaceId, userId, executionId) and cleans up many log messages.

Downgrades many copilot/chat/orchestrator lifecycle logs from error to info (including stream resume and tool execution) so successful flows are visible without being flagged as errors.

^{Written by Cursor Bugbot for commit 584ac10. This will update automatically on new commits. Configure here.}

Greptile Summary

This PR delivers two main changes under a single branch: (1) structured logging improvements for the /execute endpoint and copilot routes, and (2) a set of UI enhancements to the sidebar, markdown file viewer, and plus-menu dropdown.

Logging changes:

• Adds a withMetadata() method to the core Logger class, enabling child loggers that automatically prepend key-value context (workflowId, workspaceId, userId, executionId) to every log line without repeating it in every call site.
• Migrates the /execute route from ad-hoc [requestId] string prefixes to a structured reqLogger / asyncLogger built via withMetadata(), matching the new format shown in the PR description.
• Reverts a previous mistake where informational copilot/mothership chat events were logged at ERROR level — all corrected back to INFO.
• Good test coverage added for withMetadata (inheritance, merge, override, undefined filtering).

UI changes:

• Interactive checkbox toggling in the Markdown file preview, using a render-phase counter ref to track checkbox indices across ReactMarkdown renders.
• PlusMenuDropdown refactored to open at an arbitrary viewport anchor (e.g. the @ caret position in the textarea), using a virtual zero-size fixed trigger element.
• Sidebar improvements: keyboard shortcuts for new workflow/task (Mod+Shift+P / Mod+Shift+K), international keyboard layout fix via codeToKey(), sidebar expand via keyboard, and a search-modal centering fix that accounts for sidebar width.

Notable findings:

• The production log level default remains ERROR; the new INFO-level execute route logs require LOG_LEVEL=INFO to be set in the environment to take effect.
• reqLogger is updated twice with userId in handleExecutePost — once from auth.userId and once from actorUserId — which can produce inconsistent context in logs for workspace API key requests.

Confidence Score: 5/5

Safe to merge; all remaining findings are P2 style/informational suggestions with no runtime impact

The core logging feature is well-implemented and thoroughly tested. Copilot route fixes are straightforward level downgrades. UI changes are self-contained. No P0/P1 issues found — both flagged items are P2 observations that don't block correct operation.

No files require special attention; the two P2 notes in packages/logger/src/index.ts and apps/sim/app/api/workflows/[id]/execute/route.ts are informational only

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Client
    participant ExecuteRoute as /execute route
    participant Logger as Logger (withMetadata)
    participant DB

    Client->>ExecuteRoute: POST /api/workflows/{id}/execute
    Note over ExecuteRoute: reqLogger = logger.withMetadata({requestId, workflowId})
    ExecuteRoute->>ExecuteRoute: checkHybridAuth()
    Note over ExecuteRoute: executionId = uuidv4()<br/>reqLogger = reqLogger.withMetadata({userId, executionId})
    ExecuteRoute->>Logger: reqLogger.info("Starting server-side execution")
    ExecuteRoute->>DB: authorizeWorkflow + preprocessExecution
    DB-->>ExecuteRoute: workflow record (incl. workspaceId, actorUserId)
    Note over ExecuteRoute: reqLogger = reqLogger.withMetadata({workspaceId, userId: actorUserId})
    ExecuteRoute->>Logger: reqLogger.info("Preprocessing passed")
    ExecuteRoute->>ExecuteRoute: run workflow (SSE or non-SSE)
    ExecuteRoute->>Logger: reqLogger.info/error (block events, errors)
    ExecuteRoute-->>Client: response

_{Reviews (2): Last reviewed commit: "improvement(ui): sidebar (#3832)" | Re-trigger Greptile}

@greptile review