improvement(gong): validate integration against API docs, fix pagination/wandConfig consistency#5361
improvement(gong): validate integration against API docs, fix pagination/wandConfig consistency#5361waleedlatif1 wants to merge 110 commits into
Conversation
…ership workflow edits via sockets, ui improvements
…ng improvements, posthog, secrets mutations
…ration, signup method feature flags, SSO improvements
…nts, secrets performance, polling refactors, drag resources in mothership
…y invalidation, HITL docs
…endar triggers, docs updates, integrations/models pages improvements
…ions, jira forms endpoints
…mat, logs performance improvements fix(csp): add missing analytics domains, remove unsafe-eval, fix workspace CSP gap (#4179) fix(landing): return 404 for invalid dynamic route slugs (#4182) improvement(seo): optimize sitemaps, robots.txt, and core web vitals across sim and docs (#4170) fix(gemini): support structured output with tools on Gemini 3 models (#4184) feat(brightdata): add Bright Data integration with 8 tools (#4183) fix(mothership): fix superagent credentials (#4185) fix(logs): close sidebar when selected log disappears from filtered list; cleanup (#4186)
v0.6.46: mothership streaming fixes, brightdata integration
…m integration, atlassian triggers
…dgebase connector, SSO provider ID allowlist, singleton memory leak fix
…rs, clickhouse integration
…ration, smooth streaming, security hardening, db fixes
…x, db migrations from ci, docs updates, read replicas v0.7.3: jira oauth scope fix, read-replica client, table wire data fix, db migrations from ci, docs updates, read replicas
…uting, trigger.dev, temporal, latex, quartr, brex, convex integrations
…rovements, styling consolidation
…, db perf improvements
…nges, code hygiene
…lassian assets and google calendar tools
…d, security fixes
…richment providers, deepseek models, db performance
…nce, file sharing, scheduled tasks granularity
…t harness, sakana fugu provider v0.7.13: pii redaction, react query frontend refactor, pi coding agent harness, sakana fugu provider
…ix, settings overhaul, thrive learning integration
…extension, workspace forking, slack trigger extension, new README
…to ID list fields - mark cursor output optional:true on aggregate_activity, interaction_stats, list_flows for consistency with list_calls - add wandConfig to comma-separated ID fields (callIds, primaryUserIds, userIds, scorecardIds, reviewedUserIds) matching repo convention for CSV inputs
PR SummaryLow Risk Overview The Gong block now enables wand (AI-assisted fill) on comma-separated ID inputs— Three paginated tools—aggregate activity, interaction stats, and list flows—mark the Reviewed by Cursor Bugbot for commit 77b3a2b. Configure here. |
Greptile SummaryThis PR makes two targeted consistency improvements to the Gong integration: it marks the
Confidence Score: 5/5Safe to merge — all changes are additive consistency fixes with no behavioral risk. The three tool files only add No files require special attention. Important Files Changed
Sequence Diagram%%{init: {'theme': 'neutral'}}%%
sequenceDiagram
participant User
participant Block as Gong Block (gong.ts)
participant Tool as Gong Tool (e.g. aggregate_activity)
participant API as Gong API
User->>Block: Provide ID field input (wandConfig AI-assist)
Block->>Tool: Execute with params (callIds / userIds / etc.)
Tool->>API: POST /v2/... with comma-separated IDs
API-->>Tool: Response with records + optional cursor
Note over Tool: cursor: optional: true (absent on last page)
alt More pages exist
Tool-->>Block: records + cursor
Block-->>User: Results + cursor for next page
else Last page
Tool-->>Block: records (no cursor)
Block-->>User: Results only
end
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
sequenceDiagram
participant User
participant Block as Gong Block (gong.ts)
participant Tool as Gong Tool (e.g. aggregate_activity)
participant API as Gong API
User->>Block: Provide ID field input (wandConfig AI-assist)
Block->>Tool: Execute with params (callIds / userIds / etc.)
Tool->>API: POST /v2/... with comma-separated IDs
API-->>Tool: Response with records + optional cursor
Note over Tool: cursor: optional: true (absent on last page)
alt More pages exist
Tool-->>Block: records + cursor
Block-->>User: Results + cursor for next page
else Last page
Tool-->>Block: records (no cursor)
Block-->>User: Results only
end
Reviews (1): Last reviewed commit: "improvement(gong): tighten pagination op..." | Re-trigger Greptile |
contentSelector.context/contextTiming were never set on the /v2/calls/extensive request, so the documented context (CRM/external-system links) output was silently always empty even though the field is declared in the tool's outputs.
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Summary
cursoroutputoptional: trueon aggregate_activity/interaction_stats/list_flows (matching list_calls convention), addedwandConfigto comma-separated ID fields (callIds, primaryUserIds, userIds, scorecardIds, reviewedUserIds)Type of Change
Testing
Tested manually —
bun run lintandtsc --noEmitboth cleanChecklist