Gong - Add cursor to allow pagination #18359

jocarino · 2025-09-12T16:37:20Z

WHY

Summary by CodeRabbit

New Features
- Added an optional Cursor input to the Gong “List calls” action, letting you resume listing from where a previous step ended. Enables smoother pagination and multi-step workflows; no change if no cursor is provided.
Chores
- Bumped the action version to 0.0.4 and package metadata version to 0.3.1.

Co-authored-by: Jorge Cortes <[email protected]>

…dream

vercel · 2025-09-12T16:37:25Z

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

1 Skipped Deployment

Project	Deployment	Preview	Comments	Updated (UTC)
pipedream-docs-redirect-do-not-edit	Ignored			Sep 12, 2025 7:20pm

coderabbitai · 2025-09-12T16:37:29Z

Walkthrough

Adds an optional cursor prop to the Gong “List calls” action, forwards it to app.listCalls via params, and increments the action version to 0.0.4; also bumps the Gong package version to 0.3.1.

Changes

Cohort / File(s)	Summary
Gong list-calls action `components/gong/actions/list-calls/list-calls.mjs`	Version bumped 0.0.3 → 0.0.4. Added optional `cursor` prop (string; label "Cursor") and updated `run` to destructure `cursor` and pass it in `params` to `app.listCalls`.
Gong package metadata `components/gong/package.json`	Package `version` bumped 0.3.0 → 0.3.1. No behavioral changes.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor U as User
  participant A as Action: list-calls
  participant G as Gong API

  U->>A: Invoke with params (optional cursor)
  A->>G: listCalls(params + cursor)
  G-->>A: Returns calls list + next cursor
  A-->>U: Return calls and pagination data

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I twitch my ears at cursors’ call,
Hop-hop through pages, one and all.
A gentle nudge—0.0.4—
Now I fetch a little more.
Trail of crumbs, I won’t defer—
Follow the cursor, whirr-whirr-whirr!

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 307ae04 and 6677182.

📒 Files selected for processing (1)

components/gong/package.json (1 hunks)

✅ Files skipped from review due to trivial changes (1)

components/gong/package.json

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)

GitHub Check: Lint Code Base
GitHub Check: pnpm publish
GitHub Check: Publish TypeScript components
GitHub Check: Verify TypeScript components

Pre-merge checks (2 passed, 1 warning)

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description Check	⚠️ Warning	The PR description contains only the template header "## WHY" with the placeholder "" and provides no rationale, implementation details, or testing instructions despite the raw_summary showing concrete changes (added cursor prop to list-calls and version bumps). Because the repository's template requires an explicit WHY explaining the reason and impact of the change, the current description is incomplete and does not give reviewers necessary context. Therefore this check fails.	Please replace the placeholder with a concise WHY that explains why pagination via a cursor is needed, summarizes the key changes (added optional cursor prop in components/gong/actions/list-calls/list-calls.mjs and bumped action/package versions), and lists verification steps (how to test the cursor is passed and pagination behaves as expected). Also add any backward-compatibility or migration notes and a link to an issue or design doc if one exists to aid reviewers. Updating the description with this information will satisfy the template and speed up review.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title "Gong - Add cursor to allow pagination" succinctly captures the primary change in the changeset (adding a cursor prop to support pagination in the Gong list-calls action) and is specific, concise, and free of extraneous details, so a teammate scanning history will understand the main change. It matches the modifications described in the raw_summary (added cursor prop and version bumps) and avoids file lists or vague phrasing. Therefore it aligns with the repository's title guidance.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests

Create PR with unit tests
Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

pipedream-component-development · 2025-09-12T16:37:39Z

Thank you so much for submitting this! We've added it to our backlog to review, and our team has been notified.

pipedream-component-development · 2025-09-12T16:37:39Z

coderabbitai

Actionable comments posted: 1

🧹 Nitpick comments (2)

components/gong/actions/list-calls/list-calls.mjs (2)
35-35: Nit: disambiguate local variable from imported app

Minor readability: avoid shadowing the imported app by renaming the destructured instance.
-      app,
+      app: gong,
       cursor,
       ...params
Also update the call site:
-    return app.listCalls({
+    return gong.listCalls({
41-44: Only include cursor when defined; add guard against mixing with dates; consider auto-pagination option

Prevent sending cursor=undefined to the API.

Guard against cursor + date filters together.

Optional: expose a paginate boolean (and limit/maxPages) to fetch multiple pages automatically per Pipedream guidelines.

Minimal change to avoid undefined:
-      params: {
-        ...params,
-        cursor,
-      },
+      params: {
+        ...params,
+        ...(cursor != null ? { cursor } : {}),
+      },
Add this guard near destructuring:
if (cursor && (params.fromDateTime || params.toDateTime)) {
  throw new Error("Provide either a cursor or date range, not both.");
}
If you’d like, I can draft a follow-up that implements paginate, accumulates results until no next-cursor, and surfaces nextCursor in the step’s output/summary.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 194d1ef and 307ae04.

📒 Files selected for processing (1)

components/gong/actions/list-calls/list-calls.mjs (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

GitHub Check: Publish TypeScript components
GitHub Check: Verify TypeScript components
GitHub Check: Lint Code Base

🔇 Additional comments (1)

components/gong/actions/list-calls/list-calls.mjs (1)

8-8: Version bump looks good

Aligns with the new public prop addition.

components/gong/actions/list-calls/list-calls.mjs

michelle0927

LGTM! Ready for QA!

vunguyenhung · 2025-09-14T14:01:57Z

Hi everyone, all test cases are passed! Ready for release!

Test report
https://vunguyenhung.notion.site/Gong-Add-cursor-to-allow-pagination-26cbf548bb5e8135b553cab5356f3050

* fix: pagination prop and params struct * fix: no need for paginate here * chore: update version * chore: cleanup * chore: update package * feat: allow raw response * chore: bump package * fix: buffer response instead * Update components/google_drive/actions/download-file/download-file.mjs Co-authored-by: Jorge Cortes <[email protected]> * versions * pnpm-lock.yaml * pnpm-lock.yaml * pnpm-lock.yaml * feat: add content selector * chore: bump package * fix: comments * chore: bump versions * chore: fix versions * fixes: QA fixes * feat: add cursor to req * package.json --------- Co-authored-by: joao <[email protected]> Co-authored-by: joaocoform <[email protected]> Co-authored-by: Jorge Cortes <[email protected]> Co-authored-by: Michelle Bergeron <[email protected]> Co-authored-by: Luan Cazarine <[email protected]>

* Leonardo AI components * added unzoom image action * fixing link errors * more lint fixes * Merging pull request #18359 * fix: pagination prop and params struct * fix: no need for paginate here * chore: update version * chore: cleanup * chore: update package * feat: allow raw response * chore: bump package * fix: buffer response instead * Update components/google_drive/actions/download-file/download-file.mjs Co-authored-by: Jorge Cortes <[email protected]> * versions * pnpm-lock.yaml * pnpm-lock.yaml * pnpm-lock.yaml * feat: add content selector * chore: bump package * fix: comments * chore: bump versions * chore: fix versions * fixes: QA fixes * feat: add cursor to req * package.json --------- Co-authored-by: joao <[email protected]> Co-authored-by: joaocoform <[email protected]> Co-authored-by: Jorge Cortes <[email protected]> Co-authored-by: Michelle Bergeron <[email protected]> Co-authored-by: Luan Cazarine <[email protected]> * Merging pull request #18361 * update siteId prop * pnpm-lock.yaml * package.json version * Google Sheets - update row refresh fields (#18369) * change prop order and refresh fields * bump package.json * Pipedrive - fix app name (#18370) * use pipedriveApp instead of app * bump package.json * Pipedrive - pipelineId integer (#18372) * pipelineId - integer * bump versions * Adding app scaffolding for lightspeed_ecom_c_series * Adding app scaffolding for financial_data * Adding app scaffolding for microsoft_authenticator * Merging pull request #18345 * updates * versions * versions * Merging pull request #18368 * updates * remove console.log * versions * Coinbase Developer Platform - New Wallet Event (#18342) * new component * pnpm-lock.yaml * updates * updates * Hubspot - update search-crm (#18360) * update search-crm * limit results to one page * update version * package.json version * Merging pull request #18347 * widget props * fix version * Adding app scaffolding for rundeck * Merging pull request #18378 * Update Taiga component with new actions and sources - Bump version to 0.1.0 in package.json and add dependency on @pipedream/platform. - Introduce new actions for creating, updating, and deleting issues, tasks, and user stories. - Add sources for tracking changes and deletions of issues and tasks. - Implement utility functions for parsing and cleaning objects in common/utils.mjs. - Enhance prop definitions for better integration with Taiga API. * pnpm update * Refactor Taiga actions to utilize parseObject utility - Added parseObject utility for tags, watchers, and points in update-issue, update-task, and update-userstory actions. - Removed the update-project action as it is no longer needed. - Enhanced base source to include secret key validation for webhook security. * Merging pull request #18382 * add testSources prop * pnpm-lock.yaml * fix * Merging pull request #18323 * Added actions * Added actions * Added actions * Merging pull request #18377 * Added actions * Update components/weaviate/actions/create-class/create-class.mjs Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> --------- Co-authored-by: Luan Cazarine <[email protected]> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Merging pull request #18376 * Adding app scaffolding for etrusted * Adding app scaffolding for intelliflo_office * Adding app scaffolding for thoughtspot * Adding app scaffolding for kordiam * Adding app scaffolding for ticketsauce * trustpilot fixes (#18152) * trustpilot fixes * more fixes * update versions * more version updates * fixes * Bump all Trustpilot actions to version 0.1.0 Major improvements and API updates across all actions: - Enhanced private API support with proper authentication - Improved parameter handling and validation - Better error handling and response structures - Added new conversation flow for product reviews - Fixed endpoint URLs to match latest API documentation - Streamlined request/response processing 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]> * up version and clean up sources * merge * fix business ID * delete temp action * Update components/trustpilot/sources/new-product-reviews/new-product-reviews.mjs Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Update components/trustpilot/sources/new-product-reviews/new-product-reviews.mjs Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Update components/trustpilot/sources/new-product-reviews/new-product-reviews.mjs Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Update components/trustpilot/sources/new-service-reviews/new-service-reviews.mjs Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * comments * Pagination * fixes * comments * missed some `$`'s * unduplicated * more fixes * final comments * more comments * . --------- Co-authored-by: Claude <[email protected]> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Adding app scaffolding for peekalink * 18314 twilio (#18350) * Update Twilio component versions and dependencies - Update Twilio Send Message action adding detailed description for 'from' prop and refactoring phone number validation logic. - Incremented action versions for several Twilio actions. * pnpm update * Updating LinkedIn API version (#18399) * Merging pull request #18394 * Databricks API - Jobs action components (#18371) * Notion property building improvements (#18381) * validate property types * versions * Google Business - add debug log (#18407) * add debug log * bump versions * Notion API Key - update @pipedream/notion version (#18409) * update @pipedream/notion dependency version * pnpm-lock.yaml * Adding app scaffolding for reduct_video * Adding app scaffolding for shopware * Adding app scaffolding for instamojo * Hubspot - bug fix to sources w/ property changes (#18379) * updates * versions * Google sheets type fix (#18411) * Fixing worksheetId prop type from string to integer * Version bumps --------- Co-authored-by: Leo Vu <[email protected]> * Merging pull request #18393 * new components * remove console.log * versions * update * Merging pull request #18408 * 403 error message * versions * update * Merging pull request #18419 * Changes per PR Review * Removes leonardo_ai_actions.mdc not indented for merging * synced lockfile after install * fully lock form-data for leonardo_ai * conflict solving * lint fixes * Chipped down Readme, implemented async options in gen motion --------- Co-authored-by: jocarino <[email protected]> Co-authored-by: joao <[email protected]> Co-authored-by: joaocoform <[email protected]> Co-authored-by: Jorge Cortes <[email protected]> Co-authored-by: Michelle Bergeron <[email protected]> Co-authored-by: Luan Cazarine <[email protected]> Co-authored-by: michelle0927 <[email protected]> Co-authored-by: Andrew Chuang <[email protected]> Co-authored-by: danhsiung <[email protected]> Co-authored-by: Lucas Caresia <[email protected]> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Job <[email protected]> Co-authored-by: Claude <[email protected]> Co-authored-by: Guilherme Falcão <[email protected]> Co-authored-by: Leo Vu <[email protected]>

joao and others added 30 commits June 3, 2025 10:08

fix: pagination prop and params struct

7971dad

fix: no need for paginate here

0eaba8e

chore: update version

330430e

Merge branch 'master' into master

3d6802e

chore: cleanup

379e23d

chore: update package

6f320c1

Merge remote-tracking branch 'upstream/master'

39e9dfe

feat: allow raw response

a051253

chore: bump package

77b2e13

fix: buffer response instead

05d29c5

Update components/google_drive/actions/download-file/download-file.mjs

0303c22

Co-authored-by: Jorge Cortes <[email protected]>

Merge remote-tracking branch 'upstream/master'

a9a743d

Merge branches 'master' and 'master' of github-personal:jocarino/pipe…

7a7cb53

…dream

versions

8bedd60

pnpm-lock.yaml

5e2888c

pnpm-lock.yaml

033fb6d

Merge remote-tracking branch 'upstream/master'

8412b56

pnpm-lock.yaml

3fa1060

Merge remote-tracking branch 'upstream/master'

e08c027

feat: add content selector

63c13d8

Merge branch 'master' of github-personal:jocarino/pipedream

5cd0755

Merge remote-tracking branch 'upstream/master'

be9cca9

chore: bump package

066dae1

Merge branch 'master' into master

700c9eb

fix: comments

ef77de5

Merge branch 'master' of github-personal:jocarino/pipedream

8581a8f

Merge remote-tracking branch 'upstream/master'

78f78e2

chore: bump versions

ec8d317

chore: fix versions

aa2aa24

fixes: QA fixes

5de5e9e

jocarino added 3 commits August 21, 2025 11:03

Merge remote-tracking branch 'upstream/master'

d673426

Merge remote-tracking branch 'upstream/master'

d1e6e3b

feat: add cursor to req

7f32907

adolfo-pd added the User submitted Submitted by a user label Sep 12, 2025

Merge branch 'master' into master

307ae04

coderabbitai bot reviewed Sep 12, 2025

View reviewed changes

components/gong/actions/list-calls/list-calls.mjs Show resolved Hide resolved

andrewjschuang added this to Component (Source and Action) Backlog Sep 12, 2025

andrewjschuang moved this to Ready for PR Review in Component (Source and Action) Backlog Sep 12, 2025

pipedream-component-development requested a review from michelle0927 September 12, 2025 18:18

michelle0927 moved this from Ready for PR Review to In Review in Component (Source and Action) Backlog Sep 12, 2025

package.json

6677182

michelle0927 approved these changes Sep 12, 2025

View reviewed changes

michelle0927 moved this from In Review to Ready for QA in Component (Source and Action) Backlog Sep 12, 2025

vunguyenhung moved this from Ready for QA to Ready for Release in Component (Source and Action) Backlog Sep 14, 2025

vunguyenhung merged commit 28a8f4a into PipedreamHQ:master Sep 14, 2025
9 checks passed

github-project-automation bot moved this from Ready for Release to Done in Component (Source and Action) Backlog Sep 14, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Gong - Add cursor to allow pagination #18359

Gong - Add cursor to allow pagination #18359

Uh oh!

jocarino commented Sep 12, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

vercel bot commented Sep 12, 2025 •

edited

Loading

Uh oh!

coderabbitai bot commented Sep 12, 2025 •

edited

Loading

Uh oh!

pipedream-component-development commented Sep 12, 2025

Uh oh!

pipedream-component-development commented Sep 12, 2025

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

michelle0927 left a comment

Uh oh!

vunguyenhung commented Sep 14, 2025

Uh oh!

Uh oh!

Uh oh!

Gong - Add cursor to allow pagination #18359

Gong - Add cursor to allow pagination #18359

Uh oh!

Conversation

jocarino commented Sep 12, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

WHY

Summary by CodeRabbit

Uh oh!

vercel bot commented Sep 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai bot commented Sep 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Poem

Pre-merge checks (2 passed, 1 warning)

Uh oh!

pipedream-component-development commented Sep 12, 2025

Uh oh!

pipedream-component-development commented Sep 12, 2025

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

michelle0927 left a comment

Choose a reason for hiding this comment

Uh oh!

vunguyenhung commented Sep 14, 2025

Uh oh!

Uh oh!

Uh oh!

jocarino commented Sep 12, 2025 •

edited by coderabbitai bot

Loading

vercel bot commented Sep 12, 2025 •

edited

Loading

coderabbitai bot commented Sep 12, 2025 •

edited

Loading