vstorm-co / subagents-pydantic-ai / 31023255231
100%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 16
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 100.0%. Remained the same
31023255231

push

github

web-flow 
fix(toolset): build the general-purpose delegate through the host factory (#78)

## What this fixes

`default_model` defaulted to `"openai:gpt-4.1"`, so a consumer that
never set it
got every fallback subagent — and always the general-purpose one,
compiled at
construction — running on that model. That string resolves whatever
provider
credential the process environment happens to hold: on a deployment with
no such
key the build raised; on one with `OPENAI_API_KEY` set it ran a caller's
work on
a credential that was not theirs, off any per-caller model, budget or
vault the
host meant to enforce. A host that resolves a model per tenant had no
single
`default_model` to give the general-purpose delegate at all.

Fixes it the way the tracking issue's first option describes: **there is
no
implicit default any more.** `default_model` defaults to `None`, and the
general-purpose delegate is built the way every other subagent is — from
`default_model` or from a `default_agent_factory`.

Refs vstorm-co/agenticos#174.

## What changed

- `src/subagents_pydantic_ai/toolset.py`
  - `default_model` defaults to `None` on `create_subagent_toolset`,
    `SubAgentToolset` (and `SubAgentCapability`).
  - `_create_general_purpose_config` carries `default_model` /
`default_agent_factory` onto the config, so the delegate is compiled
through
    `_compile_subagent` — from the host's own factory when there is one.
- `include_general_purpose=True` with neither raises at construction,
naming
    the three ways out.
- `create_agent`, `delegate` and `task` refuse a call that names no
model
(tool result) when there is no default; `_compile_subagent` raises for a
    configured subagent that names no model, agent or factory.
  - Split the general-purpose / reachability guards into
`_reject_unreachable_configuration` to keep `_validate` under the
complexity
    ceiling — no behaviour change in the move.
- `src/subagents_pydantic_ai/factory.py` — `default_model` defaults to
`None`;
`... (continued)

1595 of 1595 relevant lines covered (100.0%)

1.0 hits per line

Jobs
ID Job ID Ran Files Coverage
1 31023255231.1 16
100.0
 GitHub Action Run
Source Files on build 31023255231