fix: delete skill name and description when registering skill

Author: BhAem

veadk/skills/utils.py

@@ -89,11 +89,14 @@ def load_skills_from_cloud(skill_space_ids: str) -> list[Skill]:
                 secret_key = cred.secret_access_key
                 session_token = cred.session_token
 
+            request_body = {
+                "SkillSpaceId": skill_space_id,
+                "InnerTags": {"source": "sandbox"},
+            }
+            logger.debug(f"ListSkillsBySpaceId request body: {request_body}")
+
             response = ve_request(
-                request_body={
-                    "SkillSpaceId": skill_space_id,
-                    "InnerTags": {"source": "sandbox"},
-                },
+                request_body=request_body,
                 action="ListSkillsBySpaceId",
                 ak=access_key,
                 sk=secret_key,

veadk/tools/skills_tools/register_skills_tool.py

@@ -17,6 +17,7 @@
 import json
 import os
 import zipfile
+import frontmatter
 from pathlib import Path
 from datetime import datetime
 
@@ -31,16 +32,12 @@
 
 
 def register_skills_tool(
-    skill_name: str,
-    skill_description: str,
     skill_local_path: str,
     tool_context: ToolContext,
 ) -> str:
     """Register a skill to the remote skill space by uploading its zip package to TOS and calling the CreateSkill API.
 
     Args:
-        skill_name (str): The name of the skill.
-        skill_description (str): The description of the skill.
         skill_local_path (str): The local path of the skill directory.
             - The format of the skill directory is as follows:
                 skill_local_path/
@@ -68,6 +65,16 @@ def register_skills_tool(
         logger.error(f"Skill path '{skill_path}' has no SKILL.md file.")
         return f"Skill path '{skill_path}' has no SKILL.md file."
 
+    try:
+        skill = frontmatter.load(str(skill_readme))
+        skill_name = skill.get("name", "")
+        # skill_description = skill.get("description", "")
+    except Exception as e:
+        logger.error(
+            f"Failed to get skill name and description from {skill_readme}: {e}"
+        )
+        return f"Failed to get skill name and description from {skill_readme}: {e}"
+
     zip_file_path = working_dir / "outputs" / f"{skill_name}.zip"
 
     with zipfile.ZipFile(zip_file_path, "w", zipfile.ZIP_DEFLATED) as zipf:
@@ -138,13 +145,14 @@ def register_skills_tool(
             x.strip() for x in skill_space_ids.split(",") if x.strip()
         ]
 
+        request_body = {
+            "TosUrl": tos_url,
+            "SkillSpaces": skill_space_ids_list,
+        }
+        logger.debug(f"CreateSkill request body: {request_body}")
+
         response = ve_request(
-            request_body={
-                "Name": skill_name,
-                "Description": skill_description,
-                "TosUrl": tos_url,
-                "SkillSpaces": skill_space_ids_list,
-            },
+            request_body=request_body,
             action="CreateSkill",
             ak=access_key,
             sk=secret_key,