From 73f9b4a961cf2f4837eca063fce93f4acf6d907e Mon Sep 17 00:00:00 2001
From: Kaushal_26 <kaushalsolanki260102@gmail.com>
Date: Mon, 3 Jun 2024 18:20:17 +0530
Subject: [PATCH] refactor: Update in the last commit of werewolf game

- Reviewed changes: https://github.com/geekan/MetaGPT/pull/1316#discussion_r1624248864
---
 metagpt/ext/werewolf/roles/base_player.py | 2 +-
 metagpt/ext/werewolf/roles/moderator.py   | 2 +-
 metagpt/ext/werewolf/roles/werewolf.py    | 2 +-
 metagpt/ext/werewolf/roles/witch.py       | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/metagpt/ext/werewolf/roles/base_player.py b/metagpt/ext/werewolf/roles/base_player.py
index 936e7181d..a108718e0 100644
--- a/metagpt/ext/werewolf/roles/base_player.py
+++ b/metagpt/ext/werewolf/roles/base_player.py
@@ -86,7 +86,7 @@ class BasePlayer(Role):
             # FIXME: hard code to split, restricted为"Moderator"或"Moderator, 角色profile"
             # Moderator加密发给自己的，意味着要执行角色的特殊动作
             self.rc.todo = self.special_actions[0]()
-        return self.rc.todo is not None
+        return True
 
     async def _act(self):
         # todo为_think时确定的，有两种情况，Speak或Protect
diff --git a/metagpt/ext/werewolf/roles/moderator.py b/metagpt/ext/werewolf/roles/moderator.py
index c959729b7..9621d9acb 100644
--- a/metagpt/ext/werewolf/roles/moderator.py
+++ b/metagpt/ext/werewolf/roles/moderator.py
@@ -171,7 +171,7 @@ class Moderator(BasePlayer):
         else:
             # 上一轮消息是游戏角色的发言，解析角色的发言
             self.rc.todo = ParseSpeak()
-        return self.rc.todo is not None
+        return True
 
     def _init_fields_from_obj(self, obs: dict[str, Union[int, str, list[str]]]):
         self.game_setup = obs.get("game_setup", "")
diff --git a/metagpt/ext/werewolf/roles/werewolf.py b/metagpt/ext/werewolf/roles/werewolf.py
index f28a6f655..8c2ef283a 100644
--- a/metagpt/ext/werewolf/roles/werewolf.py
+++ b/metagpt/ext/werewolf/roles/werewolf.py
@@ -13,4 +13,4 @@ class Werewolf(BasePlayer):
         await super()._think()
         if isinstance(self.rc.todo, Speak):
             self.rc.todo = Impersonate()
-        return self.rc.todo is not None
+        return True
diff --git a/metagpt/ext/werewolf/roles/witch.py b/metagpt/ext/werewolf/roles/witch.py
index e5191b48b..34f9c77d2 100644
--- a/metagpt/ext/werewolf/roles/witch.py
+++ b/metagpt/ext/werewolf/roles/witch.py
@@ -26,4 +26,4 @@ class Witch(BasePlayer):
                 self.rc.todo = Poison()
             else:
                 raise ValueError("Moderator's instructions must include save or poison keyword")
-        return self.rc.todo is not None
+        return True