From 8a1237460eb1afd77be3d8db6d61adbcdcf271a2 Mon Sep 17 00:00:00 2001
From: geekan <geekan@foxmail.com>
Date: Tue, 19 Dec 2023 19:27:11 +0800
Subject: [PATCH] remove useless fields

---
 metagpt/actions/action.py               | 12 +-----------
 metagpt/actions/search_and_summarize.py |  3 +--
 metagpt/roles/role.py                   |  2 +-
 3 files changed, 3 insertions(+), 14 deletions(-)

diff --git a/metagpt/actions/action.py b/metagpt/actions/action.py
index 7bb26ea91..1292b6684 100644
--- a/metagpt/actions/action.py
+++ b/metagpt/actions/action.py
@@ -26,22 +26,12 @@ class Action(ABC):
         self.llm = llm
         self.context = context
         self.prefix = ""  # aask*时会加上prefix，作为system_message
-        self.profile = ""  # FIXME: USELESS
         self.desc = ""  # for skill manager
         self.nodes = ...
 
-        # Output, useless
-        # self.content = ""
-        # self.instruct_content = None
-        # self.env = None
-
-    # def set_env(self, env):
-    #     self.env = env
-
-    def set_prefix(self, prefix, profile):
+    def set_prefix(self, prefix):
         """Set prefix for later usage"""
         self.prefix = prefix
-        self.profile = profile
         return self
 
     def __str__(self):
diff --git a/metagpt/actions/search_and_summarize.py b/metagpt/actions/search_and_summarize.py
index 5e4cdaea0..a1d81bc65 100644
--- a/metagpt/actions/search_and_summarize.py
+++ b/metagpt/actions/search_and_summarize.py
@@ -130,8 +130,7 @@ class SearchAndSummarize(Action):
         system_prompt = [system_text]
 
         prompt = SEARCH_AND_SUMMARIZE_PROMPT.format(
-            # PREFIX = self.prefix,
-            ROLE=self.profile,
+            ROLE=self.prefix,
             CONTEXT=rsp,
             QUERY_HISTORY="\n".join([str(i) for i in context[:-1]]),
             QUERY=str(context[-1]),
diff --git a/metagpt/roles/role.py b/metagpt/roles/role.py
index e13bf454b..bf37a6637 100644
--- a/metagpt/roles/role.py
+++ b/metagpt/roles/role.py
@@ -146,7 +146,7 @@ class Role:
         self._actions = []
 
     def _init_action_system_message(self, action: Action):
-        action.set_prefix(self._get_prefix(), self.profile)
+        action.set_prefix(self._get_prefix())
 
     def _init_actions(self, actions):
         self._reset()