improvement(tools): validate integrations, add Gong activity tools, regenerate docs#4937
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Gong adds Day-by-Day Activity and Aggregate by Period tools; Get Extensive Calls now documents AI spotlight fields (brief, outline, key points, call outcome). The Gong block outputs are expanded from a single generic Databricks gains four operations—Get Statement, List Warehouses, Get Job, Get Cluster—with new tools, block wiring, and Dub adds bulk create/update/delete, Count Links, List Events, and Get QR Code, plus block UI for cloaking/archived and the new filters; list-links sort params are dropped from docs. ServiceNow adds aggregate, list/download/upload attachments, a upload-attachment API route (auth, file access checks), contracts export, and Enrich adds Search Jobs and LinkedIn post reactions/comments by URL. Fireflies adds list skip, bite media type, and delete returns the removed transcript. DuckDuckGo exposes definition and bang redirect outputs. Google PageSpeed splits the monolithic Workday fixes Change Job SOAP (supervisory org, proposed position, nested job details) and Update Worker payload key. Okta omits blank params on partial updates so empty strings do not clear fields. Branding/docs: several integrations switch block/card Reviewed by Cursor Bugbot for commit 4615046. Configure here. |
Read Records and List Attachments shared the subBlock id 'limit', so the single-value-per-id store could bleed the value across operations. Rename the new list-attachments field to attachmentLimit and map it back to the tool's limit param.
…pecheck secureFetchWithValidation's response.json() resolves to unknown under the build's stricter typecheck; cast the parsed body so data.result is accessible.
…convention Match the SharePoint/OneDrive upload pattern: import the named ServiceNowAttachment type from the tool's types, cast response.json() to it, and extract specific fields with ?? null instead of passing data.result through as an opaque unknown blob.
|
@greptile |
|
@cursor review |
…wandConfig 'custom' is not a member of the GenerationType union, breaking the build typecheck. No valid type fits a comma-separated field list, so drop the wandConfig (consistent with the block's other field-list inputs).
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 4615046. Configure here.
Greptile SummaryThis PR audits and validates ~15 tool integrations against their respective APIs, adds two new Gong activity tools (day-by-day and aggregate-by-period), enriches the
Confidence Score: 4/5The changes are primarily additive API fixes and new tool additions; existing functionality is not broken, and the new routes follow established auth and validation patterns. The bulk of the PR is API-correctness hardening and well-contained new tools. One new parameter (fileContent) uses the wrong visibility level, which is a style inconsistency with the project's visibility conventions rather than a runtime defect. All auth, routing, and data-flow logic looks sound. apps/sim/tools/servicenow/upload_attachment.ts — the fileContent visibility level warrants a quick look. Important Files Changed
Reviews (1): Last reviewed commit: "fix(servicenow): remove invalid 'custom'..." | Re-trigger Greptile |
…attachment Per project rule, visibility:'hidden' is reserved for framework-injected tokens, not user-supplied data. fileContent was a copied-over legacy fallback with no caller (the block uploads via the canonical file/UserFile path), so remove it from the tool, types, contract, and route.
Bulk Update Links could run with only Update Data and no Link IDs or External IDs, sending Dub a request with no target links and producing a confusing API error. Guard the tool to throw a clear validation error when neither selector is provided, and clarify the block placeholder.

Summary
Type of Change
Testing
Tested manually — lint, typecheck, and api-validation pass; block registry + Gong webhook tests green
Checklist