Skip to content

fix(integrations): return the Dataverse page size and correct Contacts search outputs - #9965

Merged
waleedlatif1 merged 2 commits into
stagingfrom
fix/release-9963-google-microsoft
Nov 13, 2026
Merged

waleedlatif1 merged 2 commits into
stagingfrom
fix/release-9963-google-microsoft

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

These address the two release PR #9963 review threads on the Google/Microsoft integration sweep.

  • Dataverse List Records now returns pageSize: the page size it actually sent as odata.maxpagesize, or null when Max Results applied instead.
    • Dataverse requires the same page size on every nextLink page, and the value can't be recovered from the link.
    • A loop can now pass the output straight back in.
    • The request header and the output share one resolvePageSize helper, so they can't drift.
  • Dataverse Page Size field: the placeholder and input description now say the default of 100 applies only when Max Results is empty, and that the value must stay the same across Next Page Link pages.
  • Google Contacts: the metadata description no longer claims search returns nextPageToken/totalItems. People API searchContacts returns only results.
  • No request changes. The new output is additive and the other edits are copy, so nothing changes for existing workflows.

Type of Change

  • Bug fix

Testing

  • vitest on tools/microsoft_dataverse and blocks/blocks.test.ts: passing.
  • These checks pass:
    • check-block-registry.ts origin/staging
    • check:tool-metadata, check:docs, check:integration-catalog
    • check:utils, check:test-patterns
    • explicit-any
    • biome

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Ready Ready Preview Nov 13, 2026 12:47am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no new actionable issues were found.

Summary

Adds Dataverse’s sent pageSize to the returned output and corrects the Google Contacts search description.

  • Dataverse list results include the page size used.
  • Google Contacts metadata distinguishes list and search results.

Reviews (2) · Last reviewed commit: "fix(integrations): document the null Dat..." · Reviewed by Greptile

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 6 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

…s search outputs

- Dataverse List Records returns the page size it sent (odata.maxpagesize) so a loop can pass it
  back with nextLink, since Dataverse requires the same value on every page; the request header
  and the output share one resolver
- Page Size placeholder and input description now say the default applies only without Max
  Results and that the value must stay the same across Next Page Link pages
- Google Contacts metadata description no longer claims search returns nextPageToken/totalItems
@waleedlatif1
waleedlatif1 force-pushed the fix/release-9963-google-microsoft branch from 00bee48 to 9fa9c69 Compare November 13, 2026 00:41
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 6 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit 34b6101 into staging Nov 13, 2026
83 checks passed
@waleedlatif1
waleedlatif1 deleted the fix/release-9963-google-microsoft branch November 13, 2026 01:30

This branch was successfully deployed

1 active deployment
Preview — 9fa9c69f Deployed Nov 13, 2026 by vercel[bot]
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.

1 participant