feat: make server commands respect workspace context#256
Conversation
ListServers now accepts ownerID and passes it to the servers(ownerID:) GraphQL query, matching the existing pattern used by ListProjects. All server subcommands (list, get, reboot, ssh, sshinfo, rename) now pass f.CurrentOwnerID() so they return the correct workspace's servers when --workspace is set or workspace switch has been used. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (8)
📝 WalkthroughWalkthroughThe 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
BruceDu521
left a comment
There was a problem hiding this comment.
LGTM. The server listing path now follows the same owner-scoped pattern as projects, and the affected interactive server commands pass the effective workspace owner through correctly.
Summary
ListServersnow acceptsownerIDand passes it toservers(ownerID:)GraphQL query, matching the pattern used byListProjectslist,get,reboot,ssh,sshinfo,rename) now passf.CurrentOwnerID()so they return the correct workspace's serversContext
When using
--workspaceorworkspace switch, server commands still returned personal servers becauseListServersdidn't pass the owner context to the backend. This is a sub-task of DES-715 (team-level agent quota).Closes DES-718
Test plan
zeabur server listreturns personal servers (no workspace set)zeabur server list --workspace <teamId>returns team serverszeabur workspace switch <team>thenzeabur server listreturns team servers🤖 Generated with Claude Code
Need help on this PR? Tag
/codesmithwith what you need. Autofix is disabled.Summary by CodeRabbit
Release Notes