GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||||||||
GeezerVison is showing github.com — feat(calendly): extend tools with booking, availability, no-shows, and routing forms by waleedlatif1 · Pull Request #6545 · simstudioai/sim Conversation
Collaborator
Summary
Type of Change
Testing
Notes
Checklist
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
|
There was an error while loading. Please reload this page.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
PR SummaryCursor Bugbot is generating a summary for commit cef1ee3. Configure here. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
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
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
…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
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@cursor review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@cursor review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
✅ 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.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
a62e4e22
Deployed
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.