diff --git a/metagpt/planner/test_action_planner.py b/metagpt/planner/test_action_planner.py index e5176d4d3..dcb023a0a 100644 --- a/metagpt/planner/test_action_planner.py +++ b/metagpt/planner/test_action_planner.py @@ -35,4 +35,4 @@ async def test_action_planner(): role.recv(Message(content=task, cause_by=BossRequirement)) await role._think() # it will choose mathskill.Add - assert "1100" == (await role._act()).content.result + assert "1100" == (await role._act()).content diff --git a/metagpt/planner/test_basic_planner.py b/metagpt/planner/test_basic_planner.py index b5cb62b6d..afe003725 100644 --- a/metagpt/planner/test_basic_planner.py +++ b/metagpt/planner/test_basic_planner.py @@ -39,4 +39,4 @@ async def test_basic_planner(): # assuming sk_agent will think he needs WriterSkill.Brainstorm and WriterSkill.Translate assert "WriterSkill.Brainstorm" in role.plan.generated_plan.result assert "WriterSkill.Translate" in role.plan.generated_plan.result - await role._act() + # assert "SALUT" in (await role._act()).content #content will be some French diff --git a/metagpt/roles/sk_agent.py b/metagpt/roles/sk_agent.py index e12144ca9..a6645b485 100644 --- a/metagpt/roles/sk_agent.py +++ b/metagpt/roles/sk_agent.py @@ -65,7 +65,7 @@ class SkAgent(Role): if isinstance(self.planner, BasicPlanner): result = await self.planner.execute_plan_async(self.plan, self.kernel) elif any(isinstance(self.planner, cls) for cls in [SequentialPlanner, ActionPlanner]): - result = await self.plan.invoke_async() + result = (await self.plan.invoke_async()).result logger.info(result) msg = Message(content=result, role=self.profile, cause_by=type(self._rc.todo))