Skip to content

feat: allow updating and deleting nexus endpoint via id#87

Merged
chene40 merged 1 commit into
mainfrom
eric/nexus-id-update-delete-cmd
Jun 25, 2026
Merged

feat: allow updating and deleting nexus endpoint via id#87
chene40 merged 1 commit into
mainfrom
eric/nexus-id-update-delete-cmd

Conversation

@chene40

@chene40 chene40 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

What was changed

  • add support for cloud nexus endpoint updates and delete via in (in addition to endpoint name)

Why?

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

Note

Low Risk
CLI-only change reusing existing Cloud API lookups; no new server behavior beyond optional ID-based resolution.

Overview
temporal cloud nexus endpoint delete and update can target an endpoint with --id as well as --name. Exactly one of those flags is required; using both or neither returns a validation error. Help text and commands.yml were updated to match.

Lookup for get, delete, and update now goes through shared resolveNexusEndpoint, which calls GetNexusEndpoint when an ID is set and the existing name-based list filter otherwise.

Tests cover delete/update by ID and the name/id validation cases.

Reviewed by Cursor Bugbot for commit 0bcb4de. Bugbot is set up for automated code reviews on this repo. Configure here.

@chene40 chene40 requested a review from a team as a code owner June 25, 2026 17:24
@CLAassistant

CLAassistant commented Jun 25, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit 0bcb4de. Configure here.

Comment thread temporalcloudcli/commands.nexus.go
@chene40 chene40 merged commit e94ec6f into main Jun 25, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants