Skip to content

fix: merge PR #1378 #1376 #1356#1556

Closed
xuwei-xy wants to merge 8 commits intosipeed:mainfrom
xuwei-xy:fix-pr-batch-16
Closed

fix: merge PR #1378 #1376 #1356#1556
xuwei-xy wants to merge 8 commits intosipeed:mainfrom
xuwei-xy:fix-pr-batch-16

Conversation

@xuwei-xy
Copy link
Copy Markdown

Summary

Merges fixes from open PRs #1378 #1376 #1356

treepudding and others added 8 commits March 11, 2026 17:45
Fixes sipeed#1322

Previously, when spawn tool was called with agent_id parameter, the
subagent would use the caller agent's model instead of the target
agent's configured model.

Changes:
- Add modelResolver field to SubagentManager
- Add SetModelResolver() method to inject model resolution function
- In runTask(), use target agent's model when agent_id is specified
- Add test case for model resolver functionality

Test Plan:
- All existing tests pass
- New TestSubagentManager_ModelResolver test passes
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ treepudding
✅ Alix-007
❌ OpenClaw-User


OpenClaw-User seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@wj-xiao wj-xiao closed this Mar 14, 2026
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.

5 participants