feat(delegate): add per-call model/provider/base_url overrides#17756
Open
mddragon18 wants to merge 1 commit into
Open
feat(delegate): add per-call model/provider/base_url overrides#17756mddragon18 wants to merge 1 commit into
mddragon18 wants to merge 1 commit into
Conversation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add
model,provider, andbase_urlparameters todelegate_task, allowing per-call overrides for subagents — both at the top level and per-task in batch mode.api_keyis kept in the code path (works viadelegation.api_keyconfig) but excluded from the schema for security.Changes
tools/delegate_tool.py:delegate_task()function signature withmodel,provider,base_urlparamsDELEGATE_TASK_SCHEMAat top-level and per-task —api_keyexcluded from schema to prevent LLM secret leakagetests/tools/test_delegate.py:test_schema_validto assert new fieldstest_per_call_model_overrideSecurity
api_keyis NOT exposed to the model via the schema. The code path still supports it internally — set viadelegation.api_keyin config.yaml — but the LLM cannot inject or read it through tool calls.Example
Verification