feat: Wokforce support configurable fail handling #3488
Conversation
This reverts commit 516cfce.
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Wendong-Fan
requested review from
LuoPengcheng12138,
fengju0213 and
waleedalzarooni
hesamsheikh
left a comment
hesamsheikh
left a comment
There was a problem hiding this comment.
everything looks good to me. thanks @Wendong-Fan.
| }, | ||
| ] | ||
|
|
||
| # Generate available strategies text based on config |
There was a problem hiding this comment.
Calling _get_available_strategies_text() without checking if strategies are empty.
There was a problem hiding this comment.
_analyze_task would only be used for case with multi-strategy, so i think it should be fine
fengju0213
added
Waiting for Update
Wendong-Fan
removed
the
Waiting for Update
Description
The reassignment mechanism in Workforce was originally designed for scenarios where the initially assigned agent wasn't suitable for the task. However, as LLM capabilities have improved, the initial assignment is now almost always optimal. Consequently, if a task fails, reassigning it to a different agent rarely leads to success. Therefore, it is necessary to make this default reassignment setting configurable.
Checklist
Go over all the following points, and put an
xin all the boxes that apply.Fixes #issue-numberin the PR description (required)pyproject.tomlanduv lockIf you are unsure about any of these, don't hesitate to ask. We are here to help!