feat(logrocket): add LogRocket integration - #6678
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Tool wiring maps each operation to Also swaps the deployments block icon from Reviewed by Cursor Bugbot for commit 3a758aa. Configure here. |
Greptile SummaryThe PR adds a registered LogRocket block with six REST operations, documentation, generated metadata, and model-input projection for Galileo questions.
Confidence Score: 5/5The PR appears safe to merge because no blocking failure remains from the previous review threads. The previously reported identity validation, whitespace handling, cursor typing, and configuration-documentation issues are fixed, while the response-ID and terminal-status reports were invalidated by the documented API behavior.
|
| Filename | Overview |
|---|---|
| apps/sim/blocks/blocks/logrocket.ts | Defines the LogRocket block UI, operation routing, parameter aliases, outputs, templates, and skills; the reviewed prior mapping and comment concerns do not remain blocking. |
| apps/sim/tools/logrocket/request_highlights.ts | Implements Highlights requests with model-input projection, trimmed identity handling, time-range validation, and the documented POST id response. |
| apps/sim/tools/logrocket/get_highlights.ts | Implements Highlights polling and exposes the documented PENDING, READY, and FAILED status flow. |
| apps/sim/tools/logrocket/list_exported_sessions.ts | Implements Data Export pagination while preserving opaque cursors as strings. |
| apps/sim/tools/logrocket/get_audit_logs.ts | Implements paginated audit-log retrieval with normalized entries and string cursors. |
| apps/sim/tools/logrocket/identify_user.ts | Implements profile updates with trimmed optional fields, timestamp validation, and JSON-object trait parsing. |
| apps/sim/tools/logrocket/utils.ts | Centralizes scoped URL construction, API-key authorization headers, and response error parsing. |
| apps/sim/tools/model-input-selectors.test.ts | Verifies that only the Galileo question is classified as model input. |
Sequence Diagram
sequenceDiagram
participant Workflow as Sim Workflow
participant Tool as LogRocket Tool
participant API as LogRocket API
Workflow->>Tool: Resolved block inputs
Tool->>Tool: Validate and transform parameters
Tool->>API: Authenticated REST request
API-->>Tool: Operation response
Tool-->>Workflow: Normalized block outputs
Reviews (4): Last reviewed commit: "fix(logrocket): declare the release vers..." | Re-trigger Greptile
…mp and cover releases in the catalog copy
…rsors as opaque strings
|
@cursor review |
|
@cursor review |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 3a758aa. Configure here.
Summary
Authorization: token), scoped by org + project ID, with an advanced API host override for Private Cloudrequest.modelInputon the highlightsquestionfield so resolved secrets are projected to{{NAME}}placeholders before reaching Galileo's LLMType of Change
Testing
Tested manually. Added a model-input projection test and verified it fails without the selector.
Checklist