fix: add SKILL_SPACE_NAME env in execute_skills tool

Author: BhAem

veadk/tools/builtin_tools/execute_skills.py

@@ -71,19 +71,18 @@ def execute_skills(
     workflow_prompt: str,
     skills: Optional[List[str]] = None,
     tool_context: ToolContext = None,
-    timeout: int = 900,
 ) -> str:
     """execute skills in a code sandbox and return the output.
     For C++ code, don't execute it directly, compile and execute via Python; write sources and object files to /tmp.
 
     Args:
         workflow_prompt (str): instruction of workflow
         skills (Optional[List[str]]): The skills will be invoked
-        timeout (int, optional): The timeout in seconds for the code execution, less than or equal to 900. Defaults to 900.
 
     Returns:
         str: The output of the code execution.
     """
+    timeout = 900  # The timeout in seconds for the code execution, less than or equal to 900. Defaults to 900. Hard-coded to prevent the Agent from adjusting this parameter.
 
     tool_id = getenv("AGENTKIT_TOOL_ID")
 
@@ -131,7 +130,10 @@ def execute_skills(
     if skills:
         cmd.extend(["--skills"] + skills)
 
-    env_vars = {"TOOL_USER_SESSION_ID": tool_user_session_id}
+    env_vars = {
+        "SKILL_SPACE_NAME": os.getenv("SKILL_SPACE_NAME", ""),
+        "TOOL_USER_SESSION_ID": tool_user_session_id,
+    }
 
     code = f"""
 import subprocess

veadk/tools/skills_tools/skills_tool.py

@@ -38,8 +38,9 @@ class SkillsTool(BaseTool):
     def __init__(self, skills_directory: str | Path, skills_space_name: Optional[str]):
         self.skills_directory = Path(skills_directory).resolve()
         if not self.skills_directory.exists():
-            raise ValueError(
-                f"Skills directory does not exist: {self.skills_directory}"
+            self.skills_directory.mkdir(parents=True, exist_ok=True)
+            logger.info(
+                f"Skills directory does not exist: {self.skills_directory}, automatically created"
             )
 
         self.skills_space_name = skills_space_name
@@ -171,6 +172,7 @@ def _discover_skills(self) -> str:
                 logger.error(f"Failed to discover skill from skill space: {e}")
 
         if not skills_entries:
+            logger.warning("No skills found in local and skill space.")
             return "<available_skills>\n<!-- No skills found in skills directory and skill space -->\n</available_skills>\n"
 
         return (
@@ -210,6 +212,7 @@ def _invoke_skill(self, skill_name: str) -> str:
         """Load and return the full content of a skill."""
         # Check cache first
         if skill_name in self._skill_cache:
+            logger.info(f"Invoke skill '{skill_name}' from cache successfully.")
             return self._skill_cache[skill_name]
 
         # Find skill directory
@@ -325,12 +328,12 @@ def _invoke_skill(self, skill_name: str) -> str:
             # Cache the formatted content
             self._skill_cache[skill_name] = formatted_content
 
-            logger.info(f"Loaded skill '{skill_name}' successfully.")
+            logger.info(f"Invoke skill '{skill_name}' successfully.")
             return formatted_content
 
         except Exception as e:
-            logger.error(f"Failed to load skill {skill_name}: {e}")
-            return f"Error loading skill '{skill_name}': {e}"
+            logger.error(f"Failed to invoke skill {skill_name}: {e}")
+            return f"Error invoking skill '{skill_name}': {e}"
 
     def _parse_skill_metadata(self, skill_file: Path) -> Dict[str, str] | None:
         """Parse YAML frontmatter from a SKILL.md file."""