acontext 0.0.17__py3-none-any.whl → 0.0.18__py3-none-any.whl

acontext/agent/skill.py

@@ -2,15 +2,67 @@
 Skill tools for agent operations.
 """
 
-from dataclasses import dataclass
+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):
@@ -23,37 +75,41 @@ class GetSkillTool(BaseTool):
     @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" 
+            "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 {
-            "name": {
+            "skill_name": {
                 "type": "string",
-                "description": "The name of the skill (unique within project).",
+                "description": "The name of the skill.",
             },
         }
 
     @property
     def required_arguments(self) -> list[str]:
-        return ["name"]
+        return ["skill_name"]
 
     def execute(self, ctx: SkillContext, llm_arguments: dict) -> str:
         """Get a skill by name."""
-        name = llm_arguments.get("name")
+        skill_name = llm_arguments.get("skill_name")
 
-        if not name:
-            raise ValueError("name is required")
+        if not skill_name:
+            raise ValueError("skill_name is required")
 
-        skill = ctx.client.skills.get_by_name(name)
+        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]
+                [
+                    f"  - {file_info.path} ({file_info.mime})"
+                    for file_info in skill.file_index
+                ]
             )
         else:
             file_list = "  [NO FILES]"
@@ -62,9 +118,7 @@ class GetSkillTool(BaseTool):
             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}"
+            f"{file_list}"
         )
 
 
@@ -78,7 +132,9 @@ class GetSkillFileTool(BaseTool):
     @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'). "
+            "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
@@ -108,18 +164,22 @@ class GetSkillFileTool(BaseTool):
         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")
+        if not file_path:
+            raise ValueError("file_path is required")
+
+        skill = ctx.get_skill(skill_name)
 
-        result = ctx.client.skills.get_file_by_name(
-            skill_name=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}':"]
+        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}):")
@@ -127,7 +187,9 @@ class GetSkillFileTool(BaseTool):
 
         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(
+                f"\nDownload URL (expires in {expire_seconds} seconds):"
+            )
             output_parts.append(result.url)
 
         if not result.content and not result.url:
@@ -136,13 +198,56 @@ class GetSkillFileTool(BaseTool):
         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) -> SkillContext:
-        return SkillContext(client=client)
+    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/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/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/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.dist-info → acontext-0.0.18.dist-info}/METADATA

@@ -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.dist-info → acontext-0.0.18.dist-info}/RECORD

@@ -4,7 +4,7 @@ acontext/_utils.py,sha256=GKQH45arKh0sDu64u-5jwrII_ctnU_oChYlgR5lRkfE,1250
 acontext/agent/__init__.py,sha256=PAUhm1rBYOdKw3lCD-H9d72emkGL4KoxEvVq4GeS9ig,158
 acontext/agent/base.py,sha256=kN2KzV6fOlvmMSi_0yu2QqwVLtHh09TznAy_SDtC8iM,2998
 acontext/agent/disk.py,sha256=dSDjCejP0o4Z4nHJgfKfIXLFZaCA9AFV852Yo9E9GHk,16131
-acontext/agent/skill.py,sha256=opsOTQyuedqQMlwDXWMVwCwg0u9cyehfyELDEbat3bE,4451
+acontext/agent/skill.py,sha256=iRdIqBw8-t5A2iiOuzq4cgj4SDjYQ_xFJA3VbRhrhoA,7792
 acontext/async_client.py,sha256=sqdSNjl3tx8ijoRNYoEwh1Lxdlhfk0zyd-h-xffIJJE,8372
 acontext/client.py,sha256=3xha-zbHYLwI1njlhmoB_BYK5JIJoXFyrCpvIi63b9o,8097
 acontext/client_types.py,sha256=uVBWzLbZyXrqkljG49ojdQL_xX6N3n_HGt4Bs_TEE48,987
@@ -15,14 +15,14 @@ acontext/resources/__init__.py,sha256=Oww11t9hmwdNFK2nC8YW9mf4BBSz5YLPAXAWLlZuNT
 acontext/resources/async_blocks.py,sha256=e_iJpgcAdwS2tXQ16MMCnySlddp5JV3BadN_EyqZSF4,5555
 acontext/resources/async_disks.py,sha256=rwcRwQ79ZYbMMWAj5YDDpKQmfAIOl9M3CTYxOZTy_UU,6830
 acontext/resources/async_sessions.py,sha256=Ld9T8l884r3UuUprRz7MwafSI8BaPfLA0pg0dPfNM1A,14363
-acontext/resources/async_skills.py,sha256=mbEzyUH7S2_jkZ5tfl0FzpHJ7Io3x_biw5FirhokzAQ,5062
+acontext/resources/async_skills.py,sha256=VoeqO9Z6E7qPCE2tMJaO7coONaCKCoLJRANQ5yWVbiE,4666
 acontext/resources/async_spaces.py,sha256=b8DNG2L43B6iSgfmH8M-0cxAp681IjSCjw-4alfjV2A,6712
 acontext/resources/async_tools.py,sha256=RbGaF2kX65Mun-q-Fp5H1J8waWTLIdCOfbdY19jpn1o,1091
 acontext/resources/async_users.py,sha256=m0nK_Luo6Uuvw2azGx8Wg2cXnipnTnyheolr79fX16Y,2020
 acontext/resources/blocks.py,sha256=HJdAy5HdyTcHCYCPmqNdvApYKZ6aWs-ORIi_wQt3TUM,5447
 acontext/resources/disks.py,sha256=pvxs9gdVBothuy0LSkPmuS061nNOgJX3iffypqLHW1Y,9032
 acontext/resources/sessions.py,sha256=qUoLs01tzmYChHy3kozzGQ3ZeAz_JyWGtCIsVL20BO4,14491
-acontext/resources/skills.py,sha256=FPp1YWhH85I9BU0fJJC9FbePIbFhyX6j2FYxjPuBEAc,4935
+acontext/resources/skills.py,sha256=ZECLUfy4QWHnzk9nnq1VvEeFTx-u7NvEhECfbq5Reiw,4615
 acontext/resources/spaces.py,sha256=Ktvkkn3Jj7CZoKMbn6fYuo0zqImoCWj2EnMsnL2q-0A,6571
 acontext/resources/tools.py,sha256=II_185B0HYKSP43hizE6C1zs7kjkkPLKihuEG8s-DRY,1046
 acontext/resources/users.py,sha256=6xwU7UeMWRIGn6n7LKgsptV9pzYFNWCbJftieptVg6k,1961
@@ -31,11 +31,11 @@ acontext/types/block.py,sha256=CzKByunk642rWXNNnh8cx67UzKLKDAxODmC_whwjP90,1078
 acontext/types/common.py,sha256=5kLwzszlIofz8qZ9-Wj_zcBBiF22mAWgH9uWPkcgWCE,327
 acontext/types/disk.py,sha256=nMjyXLUre4Xuu_YfBbzzdRQDRxB8qX1Zx9LsNqGCi9E,2223
 acontext/types/session.py,sha256=WIaTiiAw8OujeSnk18Vy_Fn_AffW60pfs-unlPXYuT8,11076
-acontext/types/skill.py,sha256=qBDNysW9ptKX6RcMiZd4PfwWwvcYCzj_dC-6RbXqyx4,2531
+acontext/types/skill.py,sha256=VlLp5NpsJSpyoQTlTuV6jJ4G6ArHr1SDpcqEmFkHyeY,2364
 acontext/types/space.py,sha256=9BkGBYGeQDVwYTmPLoIjMY-IUdQzjrt8I7CXca2_5Vc,2600
 acontext/types/tool.py,sha256=-mVn-vgk2SENK0Ubt-ZgWFZxKa-ddABqcAgXQ69YY-E,805
 acontext/types/user.py,sha256=JjE0xZDwUAs2BL741U7NINyyPAso64A1YmgMBPqSuvo,1331
 acontext/uploads.py,sha256=6twnqQOY_eerNuEjeSKsE_3S0IfJUiczXtAy4aXqDl8,1379
-acontext-0.0.17.dist-info/WHEEL,sha256=eycQt0QpYmJMLKpE3X9iDk8R04v2ZF0x82ogq-zP6bQ,79
-acontext-0.0.17.dist-info/METADATA,sha256=Z6rVfzFv3K3o_hSyymhrvdQU2ZGZ20GEwQPjPkNR7Rw,859
-acontext-0.0.17.dist-info/RECORD,,
+acontext-0.0.18.dist-info/WHEEL,sha256=eycQt0QpYmJMLKpE3X9iDk8R04v2ZF0x82ogq-zP6bQ,79
+acontext-0.0.18.dist-info/METADATA,sha256=Cx7aF0w2eFg2ovJ4FyaW-NGg7NW-dnS5iPPRPJ7lZh8,859
+acontext-0.0.18.dist-info/RECORD,,