Skip to content

Show remediation instance names in CLI#373

Merged
blainekasten merged 3 commits into
nextfrom
cursor/sdk-cli-parameter-changes-4e43
May 20, 2026
Merged

Show remediation instance names in CLI#373
blainekasten merged 3 commits into
nextfrom
cursor/sdk-cli-parameter-changes-4e43

Conversation

@cursor
Copy link
Copy Markdown
Contributor

@cursor cursor Bot commented May 20, 2026

Summary

  • Update tg beta clusters remediations list to display the new SDK instance_name field when present while preserving the instance ID.
  • Keep falling back to instance_id for responses that do not include instance_name.
  • Add CLI tests for the new display behavior and fallback.

Tests

  • .venv/bin/python -m pytest tests/cli/test_beta_clusters.py -q
  • .venv/bin/python -m ruff check src/together/lib/cli/api/beta/clusters/remediations/list.py tests/cli/test_beta_clusters.py

PR routing

  • Target branch: next
  • Requested reviewers: @blainekasten and @stainless-app
Open in Web View Automation 

Co-authored-by: Blaine Kasten <blainekasten@gmail.com>
@blainekasten blainekasten changed the base branch from main to next May 20, 2026 16:48
@blainekasten blainekasten self-requested a review May 20, 2026 16:48
@blainekasten blainekasten marked this pull request as ready for review May 20, 2026 16:48
@blainekasten
Copy link
Copy Markdown
Contributor

@cursoragent fix merge conflicts

@cursor
Copy link
Copy Markdown
Contributor Author

cursor Bot commented May 20, 2026

Done.

  • Merged origin/next into cursor/sdk-cli-parameter-changes-4e43
  • Resolved the conflict in tests/cli/test_beta_clusters.py
    • Kept the PR’s instance-name/fallback tests
    • Kept next’s remediation filter test
  • Committed and pushed:
    • a31f6f10 Merge next and resolve remediation test conflicts
  • Git status is clean.

Checks passed:

.venv/bin/python -m pytest tests/cli/test_beta_clusters.py -q
# 23 passed

.venv/bin/python -m ruff check src/together/lib/cli/api/beta/clusters/remediations/list.py tests/cli/test_beta_clusters.py
# All checks passed

Note: this cloud image was missing python3.12-venv, pytest, and ruff, so I installed python3.12-venv and created a local ignored .venv. For future agents, consider running an env setup agent at https://cursor.com/onboard with:

Set up together-py cloud agents with Python 3.12 venv support and install the repo dev/test tooling so .venv/bin/python -m pytest and .venv/bin/python -m ruff work out of the box.
Open in Web Open in Cursor 

Co-authored-by: Blaine Kasten <blainekasten@gmail.com>
@blainekasten
Copy link
Copy Markdown
Contributor

@cursoragent fix lint by running the formatter

@cursor
Copy link
Copy Markdown
Contributor Author

cursor Bot commented May 20, 2026

Done.

  • CI lint failure was uv run ruff format --check
  • Ran formatter on the 2 reported files:
    • src/together/lib/cli/api/beta/clusters/remediations/list.py
    • tests/cli/test_beta_clusters.py
  • Committed/pushed:
    • f0cf1587 Run formatter on remediation CLI changes
  • PR updated.
  • Git status clean.

Checks passed:

.venv/bin/python -m ruff format --check
# 386 files already formatted

.venv/bin/python -m ruff check src/together/lib/cli/api/beta/clusters/remediations/list.py tests/cli/test_beta_clusters.py
# All checks passed

.venv/bin/python -m pytest tests/cli/test_beta_clusters.py -q
# 23 passed
Open in Web Open in Cursor 

Co-authored-by: Blaine Kasten <blainekasten@gmail.com>
@blainekasten blainekasten merged commit 1edd32e into next May 20, 2026
9 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.

2 participants