fix(logs): restore narrow v1 detail query - #6588
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview The inline select joins only Reviewed by Cursor Bugbot for commit 3ca8818. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThis PR restores a route-specific query for v1 log details while preserving the endpoint’s existing authorization and response behavior.
Confidence Score: 5/5The PR appears safe to merge with no actionable correctness or security regressions identified. The narrowed query preserves every field used by the response, handles deleted workflows through the existing nullable join behavior, and retains the authentication and workspace authorization gates.
|
| Filename | Overview |
|---|---|
| apps/sim/app/api/v1/logs/[id]/route.ts | Replaces the broad shared log query with a narrow projection while retaining all fields consumed by the route and the existing workspace-access check. |
Reviews (1): Last reviewed commit: "fix(logs): restore narrow v1 detail quer..." | Re-trigger Greptile
Summary
Type of Change
Testing
bun run lintbun run apps/sim/scripts/check-block-registry.ts origin/stagingbun run check:auditsChecklist