Rename unify.pm.issues to unify.ticketing.tickets#1
Conversation
Mirror the Unify API migration from the /pm/issues endpoint to /ticketing/tickets: - Rename src/unify/pm.ts -> ticketing.ts (PM class -> Ticketing, namespace "pm" -> "ticketing", issues() -> tickets()) - Update the unify.ticketing accessor in src/unify.ts - Update tests, README, and examples Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014zVgT6VdGxc5a96oMxJU87
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4e2b4c2146
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| async issues({ limit = 100, after, include_raw = false }: Params = {}) { | ||
| const url = this.buildUrl('issues', { limit, after, include_raw }); | ||
| async tickets({ limit = 100, after, include_raw = false }: Params = {}) { | ||
| const url = this.buildUrl('tickets', { limit, after, include_raw }); |
There was a problem hiding this comment.
The published Unify Swagger page (https://unify.bundleup.io/) still advertises GET /v1/pm/issues and does not expose /v1/ticketing/tickets; with this change tickets() now builds /v1/ticketing/tickets, so ticket listing calls from the SDK will hit an unavailable route for current deployments. Please retain the live pm/issues route or add a compatibility fallback until the server migration is deployed.
Useful? React with 👍 / 👎.
2 participants
Mirrors the Unify API migration from the
/pm/issuesendpoint to/ticketing/tickets.Changes
src/unify/pm.ts→src/unify/ticketing.ts:PMclass →Ticketing,namespace'pm'→'ticketing',issues()→tickets()(now requestingticketing/tickets)src/unify.ts:get pm()→get ticketing()src/__tests__/unify.test.ts), README, and examplesVerification
npm run buildsucceedsnpm test— all 58 tests passscripts/check-sdk-parity.jsin the API repo reports no missingticketing.ticketsmethod for this SDKNote
This SDK was already missing the
crmanddrivenamespaces and thechat.users/chat.message/git.branchesmethods relative to the canonical API surface. That drift is pre-existing and out of scope for this change.🤖 Generated with Claude Code
https://claude.ai/code/session_014zVgT6VdGxc5a96oMxJU87
Generated by Claude Code