Skip to content

fix(cli): clarify deployment replica target output - #616

Open
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/cli-sdk-updates-84e22019
Open

cursor[bot] wants to merge 1 commit into
mainfrom
cursor/cli-sdk-updates-84e22019

Conversation

@cursor

@cursor cursor Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Rename endpoint list replica output to show ready replicas versus target replicas
  • Show deployment desiredReplicas as the autoscaler target in endpoint retrieve output
  • Add focused CLI coverage for list/retrieve replica wording

Tests

  • python3 -m pytest tests/cli/test_beta_endpoints.py::TestFormatReplicas tests/cli/test_beta_endpoints.py::TestBetaEndpointsList tests/cli/test_beta_endpoints_retrieve.py::TestBetaEndpointsRetrieve
  • python3 -m ruff check src/together/lib/cli/api/beta/endpoints/list.py src/together/lib/cli/api/beta/endpoints/retrieve.py tests/cli/test_beta_endpoints.py tests/cli/test_beta_endpoints_retrieve.py
Open in Web View Automation 

Note

Low Risk
Display-only CLI label and formatting changes with no API or scaling behavior changes.

Overview
Clarifies beta endpoints CLI copy so ready replica counts are distinguished from autoscaler target (desiredReplicas), not a generic “replicas” count.

For tg beta endpoints ls, the table column is renamed to Ready/Target and cell values are compact ready/desired (still green when fully ready), dropping the trailing “ready” text.

For retrieve, endpoint deployment rows label the pair Ready / autoscaler target, and single-deployment detail prints a separate Autoscaler target field when desiredReplicas is set; the min/max line is labeled Autoscaling instead of Replicas. Tests cover _format_replicas, list header/output, and deployment retrieve wording.

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

Co-authored-by: Blaine Kasten <blainekasten@gmail.com>
@broly-code-security-scanner

Copy link
Copy Markdown

Broly Security Scan

Note

✅ Clean scan
No vulnerabilities detected in this PR.

Note

Re-scan this PR anytime with /broly scan — useful after /broly undismiss, or to refresh findings without a new push.

Broly — SAST (GLM-5.3-Flash) · Secrets · SCA · IaC · GH Actions · Base Images · Supply Chain Threats · Attack Hypotheses · Adversarial Verification

We're continuously improving Broly's accuracy and finding quality — your feedback is valuable. False positives, missed findings, bugs, and feature requests all welcome.

Ask in #security-engineering   Powered by Together AI

blainekasten
blainekasten previously approved these changes Oct 1, 2026
@blainekasten

Copy link
Copy Markdown
Collaborator

@cursoragent fix lint

@blainekasten
blainekasten marked this pull request as ready for review October 1, 2026 15:33
@blainekasten
blainekasten requested a review from zainhas October 1, 2026 15:33
@cursor

cursor Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

Taking a look!

Open in Web Open in Cursor 

@blainekasten
blainekasten dismissed their stale review October 9, 2026 20:25

removing my review

@blainekasten
blainekasten self-requested a review October 9, 2026 20:38

This branch has not been deployed

No deployments
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