[waleedlatif1]

Summary

Validated ServiceNow, Salesforce, and HubSpot against each vendor's docs. Endpoints, methods, params, and pagination were already correct; the fixes are in the request path and the agent tool-call path.

• ServiceNow: on agent tool calls, the block mapper overwrote the model's state, comments, closeCode, closeNotes, Search Knowledge query, List Attachments limit, displayValue, and field projection with blank dedicated subBlocks (Resolve Incident sent no close code/notes). Blank dedicated fields now fall back to the model's argument; a stored Create/Update JSON body is never sent as a projection. Canvas runs send exactly what they did before
• Salesforce: query_more resolved an absolute nextRecordsUrl into https://org/https://org/...; it now reduces a same-origin URL to its path and rejects other hosts or paths outside /services/data/. Record, report, and dashboard ids are trimmed and path-encoded (report/dashboard ids were checked trimmed but sent untrimmed)
• HubSpot: every path id goes through safeUrlPathSegment (emails used with idProperty=email containing #/? broke the URL). A get-operation properties value (comma-separated) no longer leaks into the search tools, which JSON.parse it and threw

Type of Change

• Bug fix

Testing

• New ServiceNow block tests for the tool-call path (each fails on the old mapper), Salesforce utils.test.ts cases (fail on old code)
• Affected integration suites (182 tests), blocks tests, check-block-registry, check:tool-metadata, check:docs, check:docs-manifest, biome
• Independent audit of every hunk against vendor docs and saved-workflow compatibility

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]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Nov 14, 2026 1:52am UTC

[cubic-dev-ai]

All reported issues were addressed across 60 files

<sub>Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic</sub>

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Summary

The PR fixes request paths and tool-call arguments across three integrations.

• ServiceNow tool calls keep model values when dedicated fields are blank.
• Salesforce tools keep pagination and resource IDs within the connected org's paths.
• HubSpot tools keep IDs inside one URL segment and pass search its own properties.

The latest changes add compatible TypeScript annotations. No new actionable issues or rule violations were found.

_{Reviews (2) · Last reviewed commit: "fix(servicenow): type the mapper context..." · Reviewed by Greptile}

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@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]

No issues found across 60 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}