Skip to content

fix(hubspot): remove unused scopes#4846

Merged
icecrasher321 merged 1 commit into
stagingfrom
chore/remove-unused-hspot-scopes
Jun 2, 2026
Merged

fix(hubspot): remove unused scopes#4846
icecrasher321 merged 1 commit into
stagingfrom
chore/remove-unused-hspot-scopes

Conversation

@icecrasher321
Copy link
Copy Markdown
Collaborator

Summary

Remove unused scopes from Hubspot block

Type of Change

  • Other: Cleanup

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 2, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jun 2, 2026 4:30am

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented Jun 2, 2026

PR Summary

Low Risk
Config-only scope reduction for new authorizations; no runtime logic changes, with minor risk only if a workflow still depended on a removed write/import scope without re-auth.

Overview
Narrows the HubSpot OAuth consent request in oauth.ts by dropping six scopes that are not needed for current HubSpot block behavior: write access for users, marketing events, quotes, and carts, plus crm.import.

New HubSpot connections will be asked for fewer permissions; existing tokens are unchanged until users reconnect. Read scopes for quotes, marketing events, carts, and users remain where still listed.

Reviewed by Cursor Bugbot for commit 9635d99. Configure here.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jun 2, 2026

Greptile Summary

This PR removes five unused OAuth scopes from the HubSpot provider configuration in apps/sim/lib/oauth/oauth.ts. Cross-referencing the apps/sim/tools/hubspot/ directory confirms that no write tools exist for users, marketing events, quotes, or carts, and no import tooling is present — making these scopes genuinely dead weight.

  • Removes crm.objects.users.write, crm.objects.marketing_events.write, crm.objects.quotes.write, crm.objects.carts.write, and crm.import from the HubSpot OAuth scope list.
  • All five removed scopes are confirmed unused: the tools directory contains only read-side implementations (get_users.ts, get_marketing_event.ts, get_quote.ts, get_cart.ts) with no corresponding write or import tools.

Confidence Score: 5/5

Safe to merge — removes write scopes that have no corresponding tool implementations in the codebase.

All five removed scopes have no matching write or import tool files under apps/sim/tools/hubspot/, confirming they are genuinely unused. The remaining scope list is consistent with the read/write tools that do exist.

No files require special attention.

Important Files Changed

Filename Overview
apps/sim/lib/oauth/oauth.ts Removes 5 unused write/import scopes from the HubSpot OAuth provider — verified safe by inspecting the HubSpot tools directory, which contains no write tools for users, marketing events, quotes, carts, or import.

Reviews (1): Last reviewed commit: "fix(hubspot): remove unused scopes" | Re-trigger Greptile

@icecrasher321 icecrasher321 merged commit cf24779 into staging Jun 2, 2026
14 checks passed
@waleedlatif1 waleedlatif1 deleted the chore/remove-unused-hspot-scopes branch June 2, 2026 16:54
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