Mount IT contacts on their own ItContacts service - #110
Conversation
Default resolution produced organizationsItContacts.listOrganizationItContacts and createItContactInvite. Mount the OrganizationsItContacts tag on ItContacts and name the operations after the resource/action, matching Groups. No SDK has generated IT Contacts yet, so no compat baseline is affected. Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Original prompt from jonatas
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Greptile SummaryThe PR gives organization-scoped IT Contacts a dedicated generated SDK service and concise resource-oriented operation names.
Confidence Score: 5/5The PR appears safe to merge, with the mount and operation hints consistently matching the current IT Contacts API surface. The exact service remount and all five operation-hint identities align with the current spec, and no conflicting mounts, overrides, or method names were identified. Important Files Changed
Reviews (1): Last reviewed commit: "Mount IT contacts on their own ItContact..." | Re-trigger Greptile |
Summary
The IT Contacts endpoints landed in the spec (#109) under the
organizations.it-contactstag, and default resolution produces a service and method names that leak the tag and the path prefix:This mounts the tag on
ItContactsand names the operations after the resource/action, so it reads likeGroups— an org-scoped resource with its own service, org passed as an option:Node then generates
workos.itContacts.listItContacts({ organizationId })etc.No SDK has generated IT Contacts yet, so there is no compat baseline to break — this is the last cheap point to fix the names, and the hints are language-agnostic so every SDK gets them.
Verified with
npm run sdk:resolve:Link to Devin session: https://app.devin.ai/sessions/2633e183d1b146d6a18a87e0e1b9c42b
Requested by: @jonatascastro12