feat: forward orchestrators list through reserve_session#37
Conversation
Let reserve_session() select across a list (or comma-delimited string) of orchestrator URLs directly, without standing up a remote signer aggregator. runner_selector/discover_orchestrator_runners already accept orchestrators; this just forwards the param through the convenience wrapper. Closes #36 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Looks like claude didn't follow my instructions correctly. |
Forwards the existing
orchestratorsparam (already supported byrunner_selector/discover_orchestrator_runners/orchestrator_discovery_urls) through thereserve_session()convenience wrapper, so callers can select across a list or comma-delimited string of orchestrator URLs directly:Useful for testing/selecting across a chosen set of orchs without standing up a remote signer +
/discover-orchestratorsaggregator. Direct per-orch discovery, so it works with any app id. Two-line passthrough, no behavior change when the param is omitted.Closes #36