feat(calendly): extend tools with booking, availability, no-shows, and routing forms - #6545
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryCursor Bugbot is generating a summary for commit cef1ee3. Configure here. |
Greptile SummaryThe PR substantially expands the Calendly integration and corrects existing request-building behavior. The follow-up specifically replaces unsafe typing in the block/tool alignment tests with real repository types and explicit union narrowing.
Confidence Score: 5/5The PR appears safe to merge because the previously reported alignment-test type-safety issue has been fully addressed and no blocking failure remains. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/blocks/blocks/calendly.test.ts | The previous type-safety issue is resolved using concrete block/tool types and safe narrowing for function-valued options and conditions. |
| apps/sim/blocks/blocks/calendly.ts | Expands the Calendly block’s operations, fields, parameter normalization, and tool access configuration without leaving a follow-up-eligible defect. |
| apps/sim/tools/calendly/calendly.test.ts | Adds request-builder coverage for the expanded Calendly operation set and corrected query behavior. |
| apps/sim/tools/calendly/index.ts | Exports the newly added Calendly tools through the typed integration barrel. |
Reviews (3): Last reviewed commit: "fix(calendly): normalize webhook organiz..." | Re-trigger Greptile
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
…d routing forms Adds 12 tools verified against the Calendly OpenAPI spec: get_user, get_event_invitee, create_event_invitee, list_event_type_available_times, list_user_busy_times, list_user_availability_schedules, create_scheduling_link, create/delete_invitee_no_show, list_organization_memberships, list_routing_forms, and list_routing_form_submissions. Also fixes issues found while validating the existing tools: - list_webhooks dropped the scope query param the API requires, so every call with scope unset returned 400 - list_event_types could only send active=true, making inactive event types unlistable - user and organization filters now accept a bare UUID or a full URI consistently across every operation - json array params (eventGuests, events) are normalized whether they arrive as an array or a JSON string
cef1ee3 to
9ebed91
Compare
|
@cursor review |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
1 issue from previous review remains unresolved.
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit a62e4e2. Configure here.
1 participant
Summary
list_webhooksdropping thescopequery param the API requires — every call with scope unset was returning 400list_event_typesonly ever sendingactive=true, which made inactive event types unlistableuser/organizationfilters now accept a bare UUID or a full URI consistently across every operationjsonarray params (eventGuests,events) are normalized whether they arrive as an array or a JSON stringType of Change
Testing
type-check,lint,tool-metadata:check, andintegration-catalog:checkall passNotes
POST /inviteesrequires a paid Calendly plan; free plans get a 403questions_and_answers,location, andtracking— their schemas resolve to an empty__bundled__map in the spec, so I left them out rather than guessadmin_managedanduser_availability_scheduleon event types,groupon scheduled events and webhooksChecklist