fix(logs, workflows): snapshot fetch optionality, restore consistent draft read snapshots - #6594

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Aug 12, 2026 4:21am

PR Summary

Medium Risk
Touches workflow draft-read and log-detail query paths, including stronger workspace scoping on snapshot loads. Incorrect filtering or join changes could affect read correctness or access boundaries.

Overview
Avoids loading unused execution snapshots in log detail, and restores consistent, workspace-scoped draft workflow reads.

fetchLogDetail no longer goes through getPublicWorkflowLog (which joined workflowExecutionSnapshots / user). It queries the detail fields it needs directly, so log detail no longer materializes heavy workflowState snapshot data.

Draft readWorkflowDefinition now uses the canonical loadWorkflowReadSnapshot path (same consistent transaction as other reads) instead of loading normalized tables alone. loadWorkflowReadSnapshot also requires workspaceId and filters the workflow row by workspace + non-archived at query time; callers like readWorkflow are updated accordingly.

^{Reviewed by Cursor Bugbot for commit 3125237. Configure here.}

Greptile Summary

This PR avoids fetching unused execution snapshots for internal log details and restores consistent workflow-row and draft-state reads within one repeatable-read transaction.

• Replaces the broad public-log query with a workspace-scoped detail query that omits unused snapshot and owner joins.
• Requires workflow read snapshots to be scoped to the canonical workspace and active workflow.
• Uses the canonical snapshot for draft workflow-definition responses while preserving immutable deployment-state reads.
• Adds focused tests for log-query optionality and draft snapshot consistency.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete blocking or non-blocking defects identified.

The replacement log query preserves all fields used by log-detail responses, and every workflow snapshot caller supplies the canonical workspace while retaining authorization and active-workflow checks.

sequenceDiagram
  participant Caller
  participant UseCase as Authorized workflow use case
  participant Context as Context resolver
  participant DB as PostgreSQL
  Caller->>UseCase: Read draft workflow
  UseCase->>Context: Resolve active workflow and workspace
  Context->>DB: Load canonical active workflow
  DB-->>Context: Workflow context
  UseCase->>DB: Begin REPEATABLE READ snapshot
  par Consistent draft state
    DB->>DB: Load normalized workflow tables
  and Consistent workflow metadata
    DB->>DB: Load active workflow scoped by workspace
  end
  DB-->>UseCase: Workflow record and normalized state
  UseCase-->>Caller: Consistent draft definition

_{Reviews (1): Last reviewed commit: "fix(workflows): restore consistent draft..." | Re-trigger Greptile}