Skip to content

fix(swarm): stop rejecting lane models against stale inherited providers - #143

Merged
rn7s2 merged 1 commit into
mainfrom
fix/swarm-inherited-model-provider
Oct 13, 2026
Merged

rn7s2 merged 1 commit into
mainfrom
fix/swarm-inherited-model-provider

Conversation

@rn7s2

@rn7s2 rn7s2 commented Oct 13, 2026

Copy link
Copy Markdown
Owner

Why

Selecting gpt-6-astra@openai-oauth for the coordinator caused lanes configured with DeepSeek-V4-Pro to fail validation against the inherited openai-oauth provider, even though runtime resolution selected traex correctly.

Change

Use the existing runtime provider-resolution rule in lane validation. Add regressions for inherited-provider overrides, bare lane flags, and serialized lane setup forms.

Verification

  • Reproduced the original failure with the installed evo-swarm check command.
  • The source CLI check now accepts the same local configuration with no problems.
  • Focused regressions fail before the fix and pass afterward.
  • Swarm unit suite passes.

This PR contains only the original targeted fix; mandatory qualified model identity will be handled separately.

Co-authored-by: EvoAgent evo@ruiqilei.com

Use runtime provider resolution when validating lane defaults, so an in-lanes model override is not rejected against the coordinator's unrelated provider. Cover the check and serialized lane setup paths.

Co-authored-by: EvoAgent <evo@ruiqilei.com>
Copilot AI balanced review requested due to automatic review settings October 13, 2026 12:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@rn7s2
rn7s2 merged commit 6d0c2a3 into main Oct 13, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants