Merge branch 'data_analyst_ldy' into 'mgx_ops'

refine prompt

See merge request pub/MetaGPT!242

metagpt/prompts/di/data_analyst.py

@@ -1,10 +1,11 @@
 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 = """
+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.
+7. When you are making plan. It is highly recommend to plan all the coding plan and reviews plan in first response.
 """
 
 TASK_TYPE_DESC = "\n".join([f"- **{tt.type_name}**: {tt.value.desc}" for tt in TaskType])

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"]