Add workspace scoping to all existing RPCs #33

Activity

1. rhuss

   added

   proto-gapProto API surface not yet covered by SDKProto API surface not yet covered by SDK

   drop-cTarget: Drop C (Provider + Workspace)Target: Drop C (Provider + Workspace)

   on Jul 29, 2026
2. rhuss commented on Jul 29, 2026

   

   rhuss

   on Jul 29, 2026

   OwnerAuthor

   More actions

   Implementation Hints

   This issue deserves a full spec (cross-cutting change touching every client, converter, fake, and the public API surface).

   Setup spex (if not already initialized)

   specify --init      # Initialize spex in the repo
   specify --refresh   # Update templates

   Or: /spex:init in Claude Code.

   Recommended workflow

   1. Specify: /spex:spec describing workspace scoping across all RPCs.
   2. Plan: /spex:plan to design the approach.
   3. Tasks: /spex:tasks to break down the work.
   4. Implement: /spex:implement to execute.
   5. Verify: /spex:verify for final checks.

   Key design decisions the spec should address

   • Where does workspace live in the client API? Options:
     • Parameter on each method (e.g., Create(ctx, name, spec, labels, workspace)) -- breaking change
     • CreateOptions struct passed variadically -- non-breaking, extensible
     • Context-based (workspace in context) -- implicit, harder to discover
   • How does all_workspaces interact with ListOptions?
   • Should Get/Delete accept workspace as a parameter or option?
   • Retroactive scoping: update converters for sandbox (Drop A) and exec/file/health (Drop B) RPCs

   Depends on

   • Add Workspace CRUD client (7 RPCs) #32 (Workspace CRUD) must land first for the Workspace type definitions

   Affected files (every sub-client)

   • sandbox_client.go, sandbox.go (SandboxInterface)
   • types/options.go (ListOptions)
   • All future client files (exec, file, health, provider, policy, service, tcp, ssh)
   • All fake implementations
   • All converter functions that build request messages
3. rhuss

   added 3 commits that reference this issue on Jul 31, 2026

   Proto Sync from Upstream PR #2445 (#40)

   ...

   d510716

   feat(sdk): add workspace scoping to all RPCs

   ...

   71b9312

   Add Workspace Scoping to All RPCs [Spec + Impl] (#41)

   ...

   69ce066
4. rhuss

   closed this as completedin #41on Jul 31, 2026