Skip to content

Walk every sandbox page in list commands - #367

Draft
davixcky wants to merge 1 commit into
eng-1161-jobs-paginationfrom
eng-1161-sandboxes-pagination
Draft

davixcky wants to merge 1 commit into
eng-1161-jobs-paginationfrom
eng-1161-sandboxes-pagination

Conversation

@davixcky

@davixcky davixcky commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • walk every sandbox cursor page for sandbox list
  • reuse the same complete sandbox set for local override list
  • keep table, JSON, and YAML output as the historical plain array
  • preserve an empty list as [] rather than null
  • reject an invalid hasMore response that omits nextCursor

Stack and dependencies

This PR intentionally does not update go.mod yet. It remains a draft until the SDK stack is merged and released, at which point the dependency can be bumped to the released version.

Test plan

  • go test ./internal/sdkclient ./internal/command/jobs ./internal/command/sandbox ./internal/command/local/override
  • tested against the exact clean go-sdk#93 commit via a temporary local replace that is not committed
  • gofmt and git diff --check

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.

1 participant