acontext 0.0.17__tar.gz → 0.0.18__tar.gz

{acontext-0.0.17 → acontext-0.0.18}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: acontext
-Version: 0.0.17
+Version: 0.0.18
 Summary: Python SDK for the Acontext API
 Keywords: acontext,sdk,client,api
 Requires-Dist: httpx>=0.28.1

{acontext-0.0.17 → acontext-0.0.18}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "acontext"
-version = "0.0.17"
+version = "0.0.18"
 description = "Python SDK for the Acontext API"
 readme = "README.md"
 requires-python = ">=3.10"

acontext-0.0.18/src/acontext/agent/skill.py

@@ -0,0 +1,253 @@
+"""
+Skill tools for agent operations.
+"""
+
+from dataclasses import dataclass, field
+
+from .base import BaseContext, BaseTool, BaseToolPool
+from ..client import AcontextClient
+from ..types.skill import Skill
+
+
+@dataclass
+class SkillContext(BaseContext):
+    """Context for skill tools with preloaded skill name mapping."""
+
+    client: AcontextClient
+    skills: dict[str, Skill] = field(default_factory=dict)
+
+    @classmethod
+    def create(cls, client: AcontextClient, skill_ids: list[str]) -> "SkillContext":
+        """Create a SkillContext by preloading skills from a list of skill IDs.
+
+        Args:
+            client: The Acontext client instance.
+            skill_ids: List of skill UUIDs to preload.
+
+        Returns:
+            SkillContext with preloaded skills mapped by name.
+
+        Raises:
+            ValueError: If duplicate skill names are found.
+        """
+        skills: dict[str, Skill] = {}
+        for skill_id in skill_ids:
+            skill = client.skills.get(skill_id)
+            if skill.name in skills:
+                raise ValueError(
+                    f"Duplicate skill name '{skill.name}' found. "
+                    f"Existing ID: {skills[skill.name].id}, New ID: {skill.id}"
+                )
+            skills[skill.name] = skill
+        return cls(client=client, skills=skills)
+
+    def get_skill(self, skill_name: str) -> Skill:
+        """Get a skill by name from the preloaded skills.
+
+        Args:
+            skill_name: The name of the skill.
+
+        Returns:
+            The Skill object.
+
+        Raises:
+            ValueError: If the skill is not found in the context.
+        """
+        if skill_name not in self.skills:
+            available = ", ".join(self.skills.keys()) if self.skills else "[none]"
+            raise ValueError(
+                f"Skill '{skill_name}' not found in context. Available skills: {available}"
+            )
+        return self.skills[skill_name]
+
+    def list_skill_names(self) -> list[str]:
+        """Return list of available skill names in this context."""
+        return list(self.skills.keys())
+
+
+class GetSkillTool(BaseTool):
+    """Tool for getting a skill by name."""
+
+    @property
+    def name(self) -> str:
+        return "get_skill"
+
+    @property
+    def description(self) -> str:
+        return (
+            "Get a skill by its name. Returns the skill information including "
+            "the relative paths of the files and their mime type categories."
+        )
+
+    @property
+    def arguments(self) -> dict:
+        return {
+            "skill_name": {
+                "type": "string",
+                "description": "The name of the skill.",
+            },
+        }
+
+    @property
+    def required_arguments(self) -> list[str]:
+        return ["skill_name"]
+
+    def execute(self, ctx: SkillContext, llm_arguments: dict) -> str:
+        """Get a skill by name."""
+        skill_name = llm_arguments.get("skill_name")
+
+        if not skill_name:
+            raise ValueError("skill_name is required")
+
+        skill = ctx.get_skill(skill_name)
+
+        file_count = len(skill.file_index)
+
+        # Format all files with path and MIME type
+        if skill.file_index:
+            file_list = "\n".join(
+                [
+                    f"  - {file_info.path} ({file_info.mime})"
+                    for file_info in skill.file_index
+                ]
+            )
+        else:
+            file_list = "  [NO FILES]"
+
+        return (
+            f"Skill: {skill.name} (ID: {skill.id})\n"
+            f"Description: {skill.description}\n"
+            f"Files: {file_count} file(s)\n"
+            f"{file_list}"
+        )
+
+
+class GetSkillFileTool(BaseTool):
+    """Tool for getting a file from a skill."""
+
+    @property
+    def name(self) -> str:
+        return "get_skill_file"
+
+    @property
+    def description(self) -> str:
+        return (
+            "Get a file from a skill by name. The file_path should be a relative "
+            "path within the skill (e.g., 'scripts/extract_text.json')."
+            "Tips: SKILL.md is the first file you should read to understand the full picture of this skill's content."
+        )
+
+    @property
+    def arguments(self) -> dict:
+        return {
+            "skill_name": {
+                "type": "string",
+                "description": "The name of the skill.",
+            },
+            "file_path": {
+                "type": "string",
+                "description": "Relative path to the file within the skill (e.g., 'scripts/extract_text.json').",
+            },
+            "expire": {
+                "type": "integer",
+                "description": "URL expiration time in seconds (only used for non-parseable files). Defaults to 900 (15 minutes).",
+            },
+        }
+
+    @property
+    def required_arguments(self) -> list[str]:
+        return ["skill_name", "file_path"]
+
+    def execute(self, ctx: SkillContext, llm_arguments: dict) -> str:
+        """Get a skill file."""
+        skill_name = llm_arguments.get("skill_name")
+        file_path = llm_arguments.get("file_path")
+        expire = llm_arguments.get("expire")
+
+        if not skill_name:
+            raise ValueError("skill_name is required")
+        if not file_path:
+            raise ValueError("file_path is required")
+
+        skill = ctx.get_skill(skill_name)
+
+        result = ctx.client.skills.get_file(
+            skill_id=skill.id,
+            file_path=file_path,
+            expire=expire,
+        )
+
+        output_parts = [
+            f"File '{result.path}' (MIME: {result.mime}) from skill '{skill_name}':"
+        ]
+
+        if result.content:
+            output_parts.append(f"\nContent (type: {result.content.type}):")
+            output_parts.append(result.content.raw)
+
+        if result.url:
+            expire_seconds = expire if expire is not None else 900
+            output_parts.append(
+                f"\nDownload URL (expires in {expire_seconds} seconds):"
+            )
+            output_parts.append(result.url)
+
+        if not result.content and not result.url:
+            return f"File '{result.path}' retrieved but no content or URL returned."
+
+        return "\n".join(output_parts)
+
+
+class ListSkillsTool(BaseTool):
+    """Tool for listing available skills in the context."""
+
+    @property
+    def name(self) -> str:
+        return "list_skills"
+
+    @property
+    def description(self) -> str:
+        return "List all available skills in the current context with their names and descriptions."
+
+    @property
+    def arguments(self) -> dict:
+        return {}
+
+    @property
+    def required_arguments(self) -> list[str]:
+        return []
+
+    def execute(self, ctx: SkillContext, llm_arguments: dict) -> str:
+        """List all available skills."""
+        if not ctx.skills:
+            return "No skills available in the current context."
+
+        skill_list = []
+        for skill_name, skill in ctx.skills.items():
+            skill_list.append(f"- {skill_name}: {skill.description}")
+
+        return f"Available skills ({len(ctx.skills)}):\n" + "\n".join(skill_list)
+
+
+class SkillToolPool(BaseToolPool):
+    """Tool pool for skill operations on Acontext skills."""
+
+    def format_context(
+        self, client: AcontextClient, skill_ids: list[str]
+    ) -> SkillContext:
+        """Create a SkillContext by preloading skills from a list of skill IDs.
+
+        Args:
+            client: The Acontext client instance.
+            skill_ids: List of skill UUIDs to preload.
+
+        Returns:
+            SkillContext with preloaded skills mapped by name.
+        """
+        return SkillContext.create(client=client, skill_ids=skill_ids)
+
+
+SKILL_TOOLS = SkillToolPool()
+SKILL_TOOLS.add_tool(ListSkillsTool())
+SKILL_TOOLS.add_tool(GetSkillTool())
+SKILL_TOOLS.add_tool(GetSkillFileTool())

{acontext-0.0.17 → acontext-0.0.18}/src/acontext/resources/async_skills.py

@@ -12,8 +12,6 @@ from ..types.skill import (
     GetSkillFileResp,
     ListSkillsOutput,
     Skill,
-    SkillCatalogItem,
-    _ListSkillsResponse,
 )
 from ..uploads import FileUpload, normalize_file_upload
 
@@ -84,31 +82,19 @@ class AsyncSkillsAPI:
         data = await self._requester.request(
             "GET", "/agent_skills", params=params or None
         )
-        api_response = _ListSkillsResponse.model_validate(data)
-
-        # Convert to catalog format (name and description only)
-        return ListSkillsOutput(
-            items=[
-                SkillCatalogItem(name=skill.name, description=skill.description)
-                for skill in api_response.items
-            ],
-            next_cursor=api_response.next_cursor,
-            has_more=api_response.has_more,
-        )
+        # Pydantic ignores extra fields, so ListSkillsOutput directly extracts name/description
+        return ListSkillsOutput.model_validate(data)
 
-    async def get_by_name(self, name: str) -> Skill:
-        """Get a skill by its name.
+    async def get(self, skill_id: str) -> Skill:
+        """Get a skill by its ID.
 
         Args:
-            name: The name of the skill (unique within project).
+            skill_id: The UUID of the skill.
 
         Returns:
-            Skill containing the skill information.
+            Skill containing the full skill information including file_index.
         """
-        params = {"name": name}
-        data = await self._requester.request(
-            "GET", "/agent_skills/by_name", params=params
-        )
+        data = await self._requester.request("GET", f"/agent_skills/{skill_id}")
         return Skill.model_validate(data)
 
     async def delete(self, skill_id: str) -> None:
@@ -119,32 +105,31 @@ class AsyncSkillsAPI:
         """
         await self._requester.request("DELETE", f"/agent_skills/{skill_id}")
 
-    async def get_file_by_name(
+    async def get_file(
         self,
         *,
-        skill_name: str,
+        skill_id: str,
         file_path: str,
         expire: int | None = None,
     ) -> GetSkillFileResp:
-        """Get a file from a skill by name.
+        """Get a file from a skill by skill ID.
 
         The backend automatically returns content for parseable text files, or a presigned URL
         for non-parseable files (binary, images, etc.).
 
         Args:
-            skill_name: The name of the skill.
+            skill_id: The UUID of the skill.
             file_path: Relative path to the file within the skill (e.g., 'scripts/extract_text.json').
             expire: URL expiration time in seconds. Defaults to 900 (15 minutes).
 
         Returns:
             GetSkillFileResp containing the file path, MIME type, and either content or URL.
         """
-        endpoint = f"/agent_skills/by_name/{skill_name}/file"
+        endpoint = f"/agent_skills/{skill_id}/file"
 
-        params = {"file_path": file_path}
+        params: dict[str, Any] = {"file_path": file_path}
         if expire is not None:
             params["expire"] = expire
 
         data = await self._requester.request("GET", endpoint, params=params)
         return GetSkillFileResp.model_validate(data)
-

{acontext-0.0.17 → acontext-0.0.18}/src/acontext/resources/skills.py

@@ -12,8 +12,6 @@ from ..types.skill import (
     GetSkillFileResp,
     ListSkillsOutput,
     Skill,
-    SkillCatalogItem,
-    _ListSkillsResponse,
 )
 from ..uploads import FileUpload, normalize_file_upload
 
@@ -25,9 +23,11 @@ class SkillsAPI:
     def create(
         self,
         *,
-        file: FileUpload
-        | tuple[str, BinaryIO | bytes]
-        | tuple[str, BinaryIO | bytes, str],
+        file: (
+            FileUpload
+            | tuple[str, BinaryIO | bytes]
+            | tuple[str, BinaryIO | bytes, str]
+        ),
         user: str | None = None,
         meta: Mapping[str, Any] | None = None,
     ) -> Skill:
@@ -80,31 +80,23 @@ class SkillsAPI:
             along with pagination information (next_cursor and has_more).
         """
         effective_limit = limit if limit is not None else 100
-        params = build_params(user=user, limit=effective_limit, cursor=cursor, time_desc=time_desc)
-        data = self._requester.request("GET", "/agent_skills", params=params or None)
-        api_response = _ListSkillsResponse.model_validate(data)
-
-        # Convert to catalog format (name and description only)
-        return ListSkillsOutput(
-            items=[
-                SkillCatalogItem(name=skill.name, description=skill.description)
-                for skill in api_response.items
-            ],
-            next_cursor=api_response.next_cursor,
-            has_more=api_response.has_more,
+        params = build_params(
+            user=user, limit=effective_limit, cursor=cursor, time_desc=time_desc
         )
+        data = self._requester.request("GET", "/agent_skills", params=params or None)
+        # Pydantic ignores extra fields, so ListSkillsOutput directly extracts name/description
+        return ListSkillsOutput.model_validate(data)
 
-    def get_by_name(self, name: str) -> Skill:
-        """Get a skill by its name.
+    def get(self, skill_id: str) -> Skill:
+        """Get a skill by its ID.
 
         Args:
-            name: The name of the skill (unique within project).
+            skill_id: The UUID of the skill.
 
         Returns:
-            Skill containing the skill information.
+            Skill containing the full skill information including file_index.
         """
-        params = {"name": name}
-        data = self._requester.request("GET", "/agent_skills/by_name", params=params)
+        data = self._requester.request("GET", f"/agent_skills/{skill_id}")
         return Skill.model_validate(data)
 
     def delete(self, skill_id: str) -> None:
@@ -115,32 +107,31 @@ class SkillsAPI:
         """
         self._requester.request("DELETE", f"/agent_skills/{skill_id}")
 
-    def get_file_by_name(
+    def get_file(
         self,
         *,
-        skill_name: str,
+        skill_id: str,
         file_path: str,
         expire: int | None = None,
     ) -> GetSkillFileResp:
-        """Get a file from a skill by name.
+        """Get a file from a skill by skill ID.
 
         The backend automatically returns content for parseable text files, or a presigned URL
         for non-parseable files (binary, images, etc.).
 
         Args:
-            skill_name: The name of the skill.
+            skill_id: The UUID of the skill.
             file_path: Relative path to the file within the skill (e.g., 'scripts/extract_text.json').
             expire: URL expiration time in seconds. Defaults to 900 (15 minutes).
 
         Returns:
             GetSkillFileResp containing the file path, MIME type, and either content or URL.
         """
-        endpoint = f"/agent_skills/by_name/{skill_name}/file"
+        endpoint = f"/agent_skills/{skill_id}/file"
 
-        params = {"file_path": file_path}
+        params: dict[str, Any] = {"file_path": file_path}
         if expire is not None:
             params["expire"] = expire
 
         data = self._requester.request("GET", endpoint, params=params)
         return GetSkillFileResp.model_validate(data)
-

{acontext-0.0.17 → acontext-0.0.18}/src/acontext/types/skill.py

@@ -24,9 +24,7 @@ class Skill(BaseModel):
     file_index: list[FileInfo] = Field(
         ..., description="List of file information (path and MIME type) in the skill"
     )
-    meta: dict[str, Any] | None = Field(
-        None, description="Custom metadata dictionary"
-    )
+    meta: dict[str, Any] | None = Field(None, description="Custom metadata dictionary")
     created_at: str = Field(..., description="ISO 8601 formatted creation timestamp")
     updated_at: str = Field(..., description="ISO 8601 formatted update timestamp")
 
@@ -39,7 +37,11 @@ class SkillCatalogItem(BaseModel):
 
 
 class ListSkillsOutput(BaseModel):
-    """Response model for listing skills (catalog format with name and description only)."""
+    """Response model for listing skills (catalog format with name and description only).
+
+    Pydantic ignores extra fields, so this directly parses API responses
+    and extracts only name/description from each item.
+    """
 
     items: list[SkillCatalogItem] = Field(
         ..., description="List of skills with name and description"
@@ -48,22 +50,16 @@ class ListSkillsOutput(BaseModel):
     has_more: bool = Field(..., description="Whether there are more items")
 
 
-class _ListSkillsResponse(BaseModel):
-    """Internal response model for API pagination (full Skill objects).
-    
-    This is used internally to parse the raw API response before converting
-    to the catalog format (ListSkillsOutput).
-    """
-    items: list[Skill]
-    next_cursor: str | None = None
-    has_more: bool = False
-
-
 class GetSkillFileResp(BaseModel):
     """Response model for getting a skill file."""
 
     path: str = Field(..., description="File path")
     mime: str = Field(..., description="MIME type of the file")
-    url: str | None = Field(None, description="Presigned URL for downloading the file (present if file is not parseable)")
-    content: FileContent | None = Field(None, description="Parsed file content if available (present if file is parseable)")
-
+    url: str | None = Field(
+        None,
+        description="Presigned URL for downloading the file (present if file is not parseable)",
+    )
+    content: FileContent | None = Field(
+        None,
+        description="Parsed file content if available (present if file is parseable)",
+    )

acontext-0.0.17/src/acontext/agent/skill.py

@@ -1,148 +0,0 @@
-"""
-Skill tools for agent operations.
-"""
-
-from dataclasses import dataclass
-
-from .base import BaseContext, BaseTool, BaseToolPool
-from ..client import AcontextClient
-
-
-@dataclass
-class SkillContext(BaseContext):
-    client: AcontextClient
-
-
-class GetSkillTool(BaseTool):
-    """Tool for getting a skill by name."""
-
-    @property
-    def name(self) -> str:
-        return "get_skill"
-
-    @property
-    def description(self) -> str:
-        return (
-            "Get a skill by its name. Return the skill information including the relative paths of the files and their mime type categories" 
-        )
-
-    @property
-    def arguments(self) -> dict:
-        return {
-            "name": {
-                "type": "string",
-                "description": "The name of the skill (unique within project).",
-            },
-        }
-
-    @property
-    def required_arguments(self) -> list[str]:
-        return ["name"]
-
-    def execute(self, ctx: SkillContext, llm_arguments: dict) -> str:
-        """Get a skill by name."""
-        name = llm_arguments.get("name")
-
-        if not name:
-            raise ValueError("name is required")
-
-        skill = ctx.client.skills.get_by_name(name)
-
-        file_count = len(skill.file_index)
-        
-        # Format all files with path and MIME type
-        if skill.file_index:
-            file_list = "\n".join(
-                [f"  - {file_info.path} ({file_info.mime})" for file_info in skill.file_index]
-            )
-        else:
-            file_list = "  [NO FILES]"
-
-        return (
-            f"Skill: {skill.name} (ID: {skill.id})\n"
-            f"Description: {skill.description}\n"
-            f"Files: {file_count} file(s)\n"
-            f"{file_list}\n"
-            f"Created: {skill.created_at}\n"
-            f"Updated: {skill.updated_at}"
-        )
-
-
-class GetSkillFileTool(BaseTool):
-    """Tool for getting a file from a skill."""
-
-    @property
-    def name(self) -> str:
-        return "get_skill_file"
-
-    @property
-    def description(self) -> str:
-        return (
-            "Get a file from a skill by name. The file_path should be a relative path within the skill (e.g., 'scripts/extract_text.json'). "
-        )
-
-    @property
-    def arguments(self) -> dict:
-        return {
-            "skill_name": {
-                "type": "string",
-                "description": "The name of the skill.",
-            },
-            "file_path": {
-                "type": "string",
-                "description": "Relative path to the file within the skill (e.g., 'scripts/extract_text.json').",
-            },
-            "expire": {
-                "type": "integer",
-                "description": "URL expiration time in seconds (only used for non-parseable files). Defaults to 900 (15 minutes).",
-            },
-        }
-
-    @property
-    def required_arguments(self) -> list[str]:
-        return ["skill_name", "file_path"]
-
-    def execute(self, ctx: SkillContext, llm_arguments: dict) -> str:
-        """Get a skill file."""
-        skill_name = llm_arguments.get("skill_name")
-        file_path = llm_arguments.get("file_path")
-        expire = llm_arguments.get("expire")
-
-        if not file_path:
-            raise ValueError("file_path is required")
-        if not skill_name:
-            raise ValueError("skill_name is required")
-
-        result = ctx.client.skills.get_file_by_name(
-            skill_name=skill_name,
-            file_path=file_path,
-            expire=expire,
-        )
-
-        output_parts = [f"File '{result.path}' (MIME: {result.mime}) from skill '{skill_name}':"]
-
-        if result.content:
-            output_parts.append(f"\nContent (type: {result.content.type}):")
-            output_parts.append(result.content.raw)
-
-        if result.url:
-            expire_seconds = expire if expire is not None else 900
-            output_parts.append(f"\nDownload URL (expires in {expire_seconds} seconds):")
-            output_parts.append(result.url)
-
-        if not result.content and not result.url:
-            return f"File '{result.path}' retrieved but no content or URL returned."
-
-        return "\n".join(output_parts)
-
-
-class SkillToolPool(BaseToolPool):
-    """Tool pool for skill operations on Acontext skills."""
-
-    def format_context(self, client: AcontextClient) -> SkillContext:
-        return SkillContext(client=client)
-
-
-SKILL_TOOLS = SkillToolPool()
-SKILL_TOOLS.add_tool(GetSkillTool())
-SKILL_TOOLS.add_tool(GetSkillFileTool())