move extra instruction to DA

metagpt/prompts/di/data_analyst.py

@@ -1,7 +1,9 @@
 from metagpt.strategy.task_type import TaskType
 
-BROWSER_INSTRUCTION = """
-4. Carefully choose to use or not use the browser tool to assist you in web tasks. 
+EXTRA_INSTRUCTION = """
+4. Don't forget to append task first when all existing tasks are finished and new tasks are required.
+5. Avoid repeating tasks you have already completed. And end loop when all requirements are met.
+6. Carefully choose to use or not use the browser tool to assist you in web tasks. 
     - When no click action is required, no need to use the browser tool to navigate to the webpage before scraping.
     - If you need detail HTML content, write code to get it but not to use the browser tool.
     - Make sure the command_name are certainly in Available Commands when you use the browser tool.

metagpt/prompts/di/role_zero.py

@@ -7,8 +7,6 @@ Note:
 1. If you keeping encountering errors, unexpected situation, or you are not sure of proceeding, use RoleZero.ask_human to ask for help.
 2. Carefully review your progress at the current task, if your actions so far has not fulfilled the task instruction, you should continue with current task. Otherwise, finish current task by Plan.finish_current_task explicitly.
 3. Each time you finish a task, use RoleZero.reply_to_human to report your progress.
-4. Don't forget to append task first when all existing tasks are finished and new tasks are required.
-5. Avoid repeating tasks you have already completed. And end loop when all requirements are met.
 """
 # To ensure compatibility with hard-coded experience, do not add any other content between "# Example" and "# Available Commands".
 CMD_PROMPT = """

metagpt/roles/di/data_analyst.py

@@ -8,7 +8,7 @@ from pydantic import Field, model_validator
 from metagpt.actions.di.execute_nb_code import ExecuteNbCode
 from metagpt.actions.di.write_analysis_code import WriteAnalysisCode
 from metagpt.logs import logger
-from metagpt.prompts.di.data_analyst import BROWSER_INSTRUCTION, TASK_TYPE_DESC, CODE_STATUS, BROWSER_INFO
+from metagpt.prompts.di.data_analyst import EXTRA_INSTRUCTION, TASK_TYPE_DESC, CODE_STATUS, BROWSER_INFO
 from metagpt.prompts.di.role_zero import ROLE_INSTRUCTION
 from metagpt.roles.di.role_zero import RoleZero
 from metagpt.schema import TaskResult, Message
@@ -22,7 +22,7 @@ class DataAnalyst(RoleZero):
     name: str = "David"
     profile: str = "DataAnalyst"
     goal: str = "Take on any data-related tasks, such as data analysis, machine learning, deep learning, web browsing, web scraping, web searching, web deployment, terminal operation, git and github operation, etc."
-    instruction: str = ROLE_INSTRUCTION + BROWSER_INSTRUCTION
+    instruction: str = ROLE_INSTRUCTION + EXTRA_INSTRUCTION
     task_type_desc: str = TASK_TYPE_DESC
 
     tools: list[str] = ["Plan", "DataAnalyst", "RoleZero", "Browser"]