Skip to content

fix: follow pagination in HubSpot and DynamoDB list ops#517

Merged
cdayAI merged 1 commit into
mainfrom
claude/codex-prs-stale-branches-oIxF6
Jun 1, 2026
Merged

fix: follow pagination in HubSpot and DynamoDB list ops#517
cdayAI merged 1 commit into
mainfrom
claude/codex-prs-stale-branches-oIxF6

Conversation

@cdayAI
Copy link
Copy Markdown
Owner

@cdayAI cdayAI commented Jun 1, 2026

Completes #455 — the last two connectors with first-page-only truncation, finishing the connector-pagination sweep.

Connector Op Mechanism
HubSpot contacts, companies, deals (GET list path) paging.next.after offset token
DynamoDB query, scan LastEvaluatedKeyExclusiveStartKey

DynamoDB's Limit is a per-page scanned cap, not a total — a filtered page can return fewer (or zero) items while more match, so a single call truncated. Both loops collect up to the existing per-op limit and are bounded by a hard page cap.

#455 is now complete

All eight connectors paginate: Stripe, Slack (#496); Linear, Google Drive, Asana, Shopify (#516); HubSpot, DynamoDB (this PR).

Verification

Multi-page regression tests for both (HubSpot follows after; DynamoDB follows ExclusiveStartKey). test_q3_2026_batch11.py + test_q3_2026_batch12.py → 87 passed; ruff check and the shell=True lint guard clean.

🤖 Generated with Claude Code


Generated by Claude Code

Completes #455: the last two connectors that fetched only the first page.
- HubSpot contacts/companies/deals (GET list path): follow paging.next.after.
- DynamoDB query/scan: follow LastEvaluatedKey via ExclusiveStartKey. Limit
  is a per-page scanned cap (a filtered page can return fewer/zero while more
  match), so a single call truncated.

Both collect up to the existing per-op limit and are bounded by a hard page
cap. Adds multi-page regression tests for each. Closes #455.
@cdayAI cdayAI merged commit 104310e into main Jun 1, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants