Fix org-contacts UUID format and cross-platform skill improvements#3
Closed
mycollablab wants to merge 3 commits into
Closed
Fix org-contacts UUID format and cross-platform skill improvements#3mycollablab wants to merge 3 commits into
mycollablab wants to merge 3 commits into
Conversation
- Replace hardcoded /tmp/ with ${TMPDIR:-/tmp}/ (respects macOS per-user
temp dir; falls back to /tmp on Linux; adds Windows PowerShell comments)
- Expand Claude Desktop config path from macOS-only ~/Library/Application
Support/Claude/ to a per-platform table (macOS / Linux / Windows)
- Split sub-skills table into base-path-per-platform + relative sub-path
so Windows users use %USERPROFILE%\.claude\skills\webex-cli\ correctly
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The /contacts/organizations/{orgId}/* API requires UUID format, not the
base64 format used by most other Admin endpoints. The codegen was sending
base64 because the blanket --org-id → base64 rule in root.go applied to
all non-CC commands indiscriminately.
Fix: extend the orgId normalization condition in generate_cli.py to also
apply UUID handling (--orgid flag, {orgid} path) for any endpoint whose
path contains /contacts/organizations/, matching the explicit Postman note
"orgId used in path are the org UUIDs." All 7 org-contacts commands now
use --orgid and auto-receive a decoded UUID from root.go.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Documents the CC=UUID vs admin/calling=base64 rule and tells the model to try the other format when an API returns a 400/404 org-related error. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
Author
|
Superseded by a cleaner branch — see new PR |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
webex admin org-contactswas sending base64 org IDs to the/contacts/organizations/{orgId}/*API, which explicitly requires UUID format. All 7 org-contacts commands now correctly send UUID.generate_cli.pyorgId normalization rule to also apply UUID handling for any endpoint whose path contains/contacts/organizations/, so future Postman collection updates pick this up automatically./tmp/paths and~/Library/Application Support/Claude/config path with per-platform alternatives covering macOS, Linux, and Windows.Changes
codegen/generate_cli.py/contacts/organizations/pathscmd/admin/org_contacts.go--orgid(UUID)skill/SKILL.mdTest plan
go build ./...passeswebex admin org-contacts list --debug— URL contains UUID, not base64webex cc site list --debug— URL contains UUIDwebex admin people list --debug—?orgId=query param contains base64webex calling locations list --debug—?orgId=query param contains base64webex admin licenses list --debug—?orgId=query param contains base64🤖 Generated with Claude Code