From 1899333b95d058761433da13a25a895e8fe7142c Mon Sep 17 00:00:00 2001
From: hetaoBackend <hetao7@pku.edu.cn>
Date: Sat, 31 Jan 2026 21:54:29 +0800
Subject: [PATCH] refactor: update skills XML format in prompt template

Change skills rendering from attribute-based format to nested element format
with <available_skills>, <skill>, <name>, <description>, and <location> tags
for better readability and structure.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 backend/src/agents/lead_agent/prompt.py | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/backend/src/agents/lead_agent/prompt.py b/backend/src/agents/lead_agent/prompt.py
index 2e64086..cb2ca5f 100644
--- a/backend/src/agents/lead_agent/prompt.py
+++ b/backend/src/agents/lead_agent/prompt.py
@@ -97,9 +97,7 @@ You have access to skills that provide optimized workflows for specific tasks. E
 
 **Skills are located at:** {skills_base_path}
 
-<all_available_skills>
 {skills_list}
-</all_available_skills>
 
 </skill_system>
 
@@ -181,10 +179,17 @@ def apply_prompt_template() -> str:
         container_base_path = "/mnt/skills"
 
     # Generate skills list XML with paths (path points to SKILL.md file)
-    skills_list = "\n".join(f'<skill name="{skill.name}" path="{skill.get_container_file_path(container_base_path)}">\n{skill.description}\n</skill>' for skill in skills)
-
-    # If no skills found, provide empty list
-    if not skills_list:
+    if skills:
+        skill_items = "\n".join(
+            f"    <skill>\n"
+            f"        <name>{skill.name}</name>\n"
+            f"        <description>{skill.description}</description>\n"
+            f"        <location>{skill.get_container_file_path(container_base_path)}</location>\n"
+            f"    </skill>"
+            for skill in skills
+        )
+        skills_list = f"<available_skills>\n{skill_items}\n</available_skills>"
+    else:
         skills_list = "<!-- No skills available -->"
 
     # Format the prompt with dynamic skills