fix(actor): resolve provider from explicit field in v3 YAML before inferring from model #10930
Merged
hurui200320
merged 1 commits from 2026-04-30 02:28:35 +00:00
bugfix/m3-v3-actor-provider-resolution into master
1 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
 hurui200320
|
caf146e132 |
fix(actor): resolve provider from explicit field in v3 YAML before inferring from model
CI / benchmark-publish (push) Failing after 39s
CI / helm (push) Successful in 38s
CI / build (push) Successful in 54s
CI / lint (push) Successful in 1m2s
CI / push-validation (push) Successful in 21s
CI / quality (push) Successful in 1m26s
CI / typecheck (push) Successful in 1m46s
CI / security (push) Successful in 1m51s
CI / integration_tests (push) Successful in 4m16s
CI / unit_tests (push) Successful in 4m56s
CI / e2e_tests (push) Successful in 4m56s
CI / docker (push) Successful in 1m49s
CI / coverage (push) Successful in 10m51s
CI / status-check (push) Successful in 3s
CI / benchmark-publish (pull_request) Has been skipped
CI / coverage (pull_request) Successful in 14m50s
CI / docker (pull_request) Successful in 1m58s
CI / quality (pull_request) Successful in 1m18s
CI / e2e_tests (pull_request) Successful in 3m53s
CI / integration_tests (pull_request) Successful in 6m34s
CI / unit_tests (pull_request) Successful in 8m47s
CI / push-validation (pull_request) Successful in 20s
CI / lint (pull_request) Successful in 1m19s
CI / typecheck (pull_request) Successful in 1m26s
CI / security (pull_request) Successful in 1m31s
CI / helm (pull_request) Successful in 36s
CI / build (pull_request) Successful in 43s
CI / status-check (pull_request) Successful in 5s
Two bugs prevented built-in LLM actors from working with models that lack a '/'
separator (e.g. claude-sonnet-4-20250514):
1. _generate_builtin_actor_yaml (actor/registry.py) was constructing the model
field as '{provider}/{model}' (e.g. 'anthropic/claude-sonnet-4-20250514').
The Anthropic API expects just the bare model ID, causing 404 errors. Fixed
by using the bare model identifier and lowercasing the provider field to
match ProviderType enum values.
2. _build_from_v3 (reactive/config_parser.py) always inferred the provider from
the model string via infer_provider_from_model(). For models without a '/'
separator this returned 'custom', an invalid ProviderType, causing
'Unknown provider type: custom' at runtime. Fixed by checking for an
explicit top-level 'provider' field in the v3 YAML data (per spec resolution
order step 2) before falling back to inference.
As a side effect, these fixes also resolve issue #10861: agents actor run now
successfully invokes the LLM when using built-in actors. The @tdd_expected_fail
tag has been removed from features/tdd_actor_run_response.feature accordingly.
New BDD scenarios added to features/builtin_actor_v3_yaml.feature and
features/actor_v3_route_synthesis.feature validate both fixes and the fallback
inference behaviour for models with '/' separators.
All quality gates pass: lint, typecheck, 671 feature files (15673 scenarios),
1997 Robot integration tests, coverage 97.12%.
ISSUES CLOSED: #10926
|