letscode 0.5.0__py3-none-any.whl → 0.6.1__py3-none-any.whl

letscode/agent/loop.py

@@ -108,7 +108,9 @@ async def agent_loop(  # noqa: PLR0913, C901  # central turn state machine — b
     ``env`` defaults to a fresh :class:`~letscode.agent.execution_env.LocalEnv`
     so callers that don't care about the abstraction keep working.
     """
-    registry = _build_tool_registry(state.tools)
+    registry = ToolRegistry()
+    for tool_impl in state.tools:
+        registry.add(tool_impl)
     resolved_env: ExecutionEnv = env if env is not None else LocalEnv()
 
     state.messages.extend(inputs)
@@ -630,13 +632,6 @@ async def _close_aborted_run(  # noqa: RUF029  # async generator: consumed via `
     yield AgentEnd(messages=_snapshot(state.messages), error="aborted")
 
 
-def _build_tool_registry(tools: list[Tool]) -> ToolRegistry:
-    registry = ToolRegistry()
-    for tool_impl in tools:
-        registry.add(tool_impl)
-    return registry
-
-
 def _assemble_tool_calls(
     names: dict[str, str],
     args_strs: dict[str, str],

letscode/agent/tools.py

@@ -26,10 +26,11 @@ from pydantic import BaseModel, Field
 from letscode.llm.types import (
     ContentPart,  # noqa: TC001  # Pydantic resolves at runtime
 )
+from letscode.plugins.registries import Registry
 
 if TYPE_CHECKING:
     import asyncio
-    from collections.abc import Awaitable, Callable, Iterator
+    from collections.abc import Awaitable, Callable
     from pathlib import Path
 
     from letscode.agent.execution_env import ExecutionEnv
@@ -164,13 +165,6 @@ def tool(
 
 
 def _callable_name(fn: Callable[..., object]) -> str:
-    """Best-effort name for any callable.
-
-    ``Callable`` is broader than "function with ``__name__``" — a
-    ``functools.partial``, bound-method-on-an-instance, or callable
-    class instance may not have it. Falls back to ``repr`` so error
-    messages stay informative.
-    """
     return getattr(fn, "__name__", repr(fn))
 
 
@@ -210,36 +204,5 @@ def _extract_params_model(
 # ---------------------------------------------------------------------------
 
 
-class ToolRegistry:
+class ToolRegistry(Registry["Tool"]):
     """Mutable, name-keyed collection of registered tools."""
-
-    def __init__(self) -> None:
-        self._tools: dict[str, Tool] = {}
-
-    def add(self, tool: Tool, *, replace: bool = False) -> None:
-        """Register ``tool`` under its ``name``.
-
-        Raises ``ValueError`` on duplicate names unless ``replace=True``.
-        """
-        if tool.name in self._tools and not replace:
-            msg = (
-                f"Tool {tool.name!r} already registered. Pass replace=True to override."
-            )
-            raise ValueError(msg)
-        self._tools[tool.name] = tool
-
-    def get(self, name: str) -> Tool:
-        """Look up a tool by name. Raises ``KeyError`` if absent."""
-        if name not in self._tools:
-            msg = f"Tool {name!r} not registered"
-            raise KeyError(msg)
-        return self._tools[name]
-
-    def __contains__(self, name: object) -> bool:
-        return name in self._tools
-
-    def __iter__(self) -> Iterator[Tool]:
-        return iter(self._tools.values())
-
-    def __len__(self) -> int:
-        return len(self._tools)

letscode/cli/rpc.py

@@ -14,7 +14,7 @@ import contextlib
 import json
 import sys
 from importlib.metadata import PackageNotFoundError, version
-from typing import TYPE_CHECKING, Any, ClassVar
+from typing import TYPE_CHECKING, Any, ClassVar, Self
 
 from letscode.agent.agent import Agent
 from letscode.agent.events import MessageEnd
@@ -23,7 +23,7 @@ from letscode.llm.client import LLMClient
 from letscode.session.store import save_message
 
 if TYPE_CHECKING:
-    from collections.abc import Awaitable, Callable, Sequence
+    from collections.abc import Callable, Coroutine, Sequence
     from pathlib import Path
     from typing import TextIO
 
@@ -154,7 +154,7 @@ class _RpcSession:
         if handler is None:
             self._error(cid, f"unknown command: {name!r}")
             return False
-        return await handler(self, cmd, cid)
+        return await handler(self, cmd, cid)  # type: ignore[arg-type]  # mypy: Self vs _RpcSession in the table
 
     async def _h_hello(self, cmd: dict[str, Any], cid: str | None) -> bool:
         proto = cmd.get("protocol")
@@ -218,7 +218,9 @@ class _RpcSession:
         return True
 
     _HANDLERS: ClassVar[
-        dict[str, Callable[[_RpcSession, dict[str, Any], str | None], Awaitable[bool]]]
+        dict[
+            str, Callable[[Self, dict[str, Any], str | None], Coroutine[Any, Any, bool]]
+        ]
     ] = {
         "hello": _h_hello,
         "prompt": _h_prompt,

letscode/plugins/registries.py

@@ -14,7 +14,7 @@ from __future__ import annotations
 
 import logging
 from dataclasses import dataclass
-from typing import TYPE_CHECKING
+from typing import TYPE_CHECKING, Protocol
 
 from letscode.skills.discovery import iter_skill_files
 from letscode.skills.loader import Skill, parse_skill
@@ -30,6 +30,54 @@ if TYPE_CHECKING:
 
 _logger = logging.getLogger(__name__)
 
+# ---------------------------------------------------------------------------
+# Generic name-keyed registry
+# ---------------------------------------------------------------------------
+
+
+class _HasName(Protocol):
+    """Structural bound: any item with a ``name: str`` attribute."""
+
+    name: str
+
+
+class Registry[T: _HasName]:
+    """Name-keyed collection of registered items sharing a ``.name`` attribute.
+
+    Concrete registries (:class:`CommandRegistry`, :class:`ToolRegistry`,
+    :class:`FrontendRegistry`, :class:`SkillRegistry`) subclass this
+    to get their own type for ``isinstance`` checks and imports.
+    """
+
+    def __init__(self) -> None:
+        self._items: dict[str, T] = {}
+
+    def add(self, item: T, *, replace: bool = False) -> None:
+        """Register ``item`` under its ``name``.
+
+        Raises ``ValueError`` on duplicate names unless ``replace=True``.
+        """
+        if item.name in self._items and not replace:
+            msg = f"{item.name!r} already registered. Pass replace=True to override."
+            raise ValueError(msg)
+        self._items[item.name] = item
+
+    def get(self, name: str) -> T:
+        """Look up an item by name. Raises ``KeyError`` if absent."""
+        if name not in self._items:
+            msg = f"{name!r} not registered"
+            raise KeyError(msg)
+        return self._items[name]
+
+    def __contains__(self, name: object) -> bool:
+        return name in self._items
+
+    def __iter__(self) -> Iterator[T]:
+        return iter(self._items.values())
+
+    def __len__(self) -> int:
+        return len(self._items)
+
 
 # ---------------------------------------------------------------------------
 # Value types
@@ -77,6 +125,7 @@ __all__ = [
     "CommandRegistry",
     "FrontendFactory",
     "FrontendRegistry",
+    "Registry",
     "Skill",
     "SkillRegistry",
 ]
@@ -87,38 +136,11 @@ __all__ = [
 # ---------------------------------------------------------------------------
 
 
-class CommandRegistry:
+class CommandRegistry(Registry[Command]):
     """Name-keyed registry of slash commands."""
 
-    def __init__(self) -> None:
-        self._items: dict[str, Command] = {}
-
-    def add(self, command: Command, *, replace: bool = False) -> None:
-        if command.name in self._items and not replace:
-            msg = (
-                f"Command {command.name!r} already registered. "
-                "Pass replace=True to override."
-            )
-            raise ValueError(msg)
-        self._items[command.name] = command
 
-    def get(self, name: str) -> Command:
-        if name not in self._items:
-            msg = f"Command {name!r} not registered"
-            raise KeyError(msg)
-        return self._items[name]
-
-    def __contains__(self, name: object) -> bool:
-        return name in self._items
-
-    def __iter__(self) -> Iterator[Command]:
-        return iter(self._items.values())
-
-    def __len__(self) -> int:
-        return len(self._items)
-
-
-class SkillRegistry:
+class SkillRegistry(Registry[Skill]):
     """Name-keyed registry of skills plus a list of source directories.
 
     Plugins typically call :meth:`add_source` to register a directory
@@ -127,18 +149,9 @@ class SkillRegistry:
     """
 
     def __init__(self) -> None:
-        self._items: dict[str, Skill] = {}
+        super().__init__()
         self._sources: list[Path] = []
 
-    def add(self, skill: Skill, *, replace: bool = False) -> None:
-        if skill.name in self._items and not replace:
-            msg = (
-                f"Skill {skill.name!r} already registered. "
-                "Pass replace=True to override."
-            )
-            raise ValueError(msg)
-        self._items[skill.name] = skill
-
     def add_source(self, path: Path) -> None:
         """Register a directory the loader should scan for ``SKILL.md``."""
         if path not in self._sources:
@@ -192,51 +205,9 @@ class SkillRegistry:
         """Read-only view of registered source directories (insertion order)."""
         return tuple(self._sources)
 
-    def get(self, name: str) -> Skill:
-        if name not in self._items:
-            msg = f"Skill {name!r} not registered"
-            raise KeyError(msg)
-        return self._items[name]
-
-    def __contains__(self, name: object) -> bool:
-        return name in self._items
-
-    def __iter__(self) -> Iterator[Skill]:
-        return iter(self._items.values())
-
-    def __len__(self) -> int:
-        return len(self._items)
-
 
-class FrontendRegistry:
+class FrontendRegistry(Registry[FrontendFactory]):
     """Name-keyed registry of frontend factories.
 
     The CLI's ``--frontend NAME`` flag (added in T20) selects from here.
     """
-
-    def __init__(self) -> None:
-        self._items: dict[str, FrontendFactory] = {}
-
-    def add(self, frontend: FrontendFactory, *, replace: bool = False) -> None:
-        if frontend.name in self._items and not replace:
-            msg = (
-                f"Frontend {frontend.name!r} already registered. "
-                "Pass replace=True to override."
-            )
-            raise ValueError(msg)
-        self._items[frontend.name] = frontend
-
-    def get(self, name: str) -> FrontendFactory:
-        if name not in self._items:
-            msg = f"Frontend {name!r} not registered"
-            raise KeyError(msg)
-        return self._items[name]
-
-    def __contains__(self, name: object) -> bool:
-        return name in self._items
-
-    def __iter__(self) -> Iterator[FrontendFactory]:
-        return iter(self._items.values())
-
-    def __len__(self) -> int:
-        return len(self._items)

{letscode-0.5.0.dist-info → letscode-0.6.1.dist-info}/METADATA

@@ -1,9 +1,7 @@
 Metadata-Version: 2.4
 Name: letscode
-Version: 0.5.0
+Version: 0.6.1
 Summary: A minimal coding agent for the terminal.
-Project-URL: Homepage, https://git.sr.ht/~sfermigier/letscode
-Project-URL: Source Code, https://git.sr.ht/~sfermigier/letscode
 Author-email: Stefane Fermigier <sf@abilian.com>
 Requires-Python: >=3.12
 Requires-Dist: aiofiles>=24.0
@@ -13,6 +11,7 @@ Requires-Dist: openai>=1.0
 Requires-Dist: pluggy>=1.5
 Requires-Dist: prompt-toolkit>=3.0
 Requires-Dist: pydantic>=2.0
+Requires-Dist: pyyaml>=6.0
 Requires-Dist: rich>=13.0
 Description-Content-Type: text/markdown
 
@@ -20,10 +19,12 @@ Description-Content-Type: text/markdown
 
 A minimal, OpenAI-compatible coding agent for the terminal — written in Python. Point it at any OpenAI-API-compatible endpoint (Ollama, Fireworks, OpenRouter, vLLM, llama.cpp's `llama-server`, …) and get a streaming agent loop with the four tools that cover 95% of coding sessions — `read`, `write`, `edit`, `bash` — plus skills, slash commands, and a plugin system you can extend.
 
-**Status:** v0.5, alpha.
+**Status:** v0.6, alpha.
 
 The full docs live in [`docs/`](docs/) and build via [Zensical](https://zensical.org) — run `make docs-serve` for a live preview, or `make docs` for a static build.
 
+Or go to <https://letscode.hop3.abilian.com> for the online doc.
+
 ## What it does
 
 - **Streaming agent loop** with parallel tool execution.

{letscode-0.5.0.dist-info → letscode-0.6.1.dist-info}/RECORD

@@ -6,13 +6,13 @@ letscode/agent/events.py,sha256=FC0ZjvBF4NofxG57O-6Hc_XruqwurRLBF-sUzUd0P7g,3926
 letscode/agent/execution_env.py,sha256=QCFOeDi-mxRvgLKG_Zm--v7Ns1FbESi7birlLVaNWgI,11438
 letscode/agent/hooks.py,sha256=5XnFYZbM3rBHlnLBYjylM58j588_11Tsms1o4LPXMx8,1479
 letscode/agent/hookspecs.py,sha256=LwAjGdJXdbQmx5fEW-XLWt8BKry-e39ekZSvJmprmkY,7929
-letscode/agent/loop.py,sha256=X3Dkluqrq-58rIUj4YshGkvlkrJ5OE5f6ADTYO3syKY,24202
+letscode/agent/loop.py,sha256=2XyUADNDdfabilvqG4uH1DrMvOQ22Gcp7lKbyldiWNQ,24076
 letscode/agent/messages.py,sha256=ceBMb8k_uJDskqCCbZuKlRWmyL2Pwuc_LlYt7iaMn6c,3290
 letscode/agent/state.py,sha256=O4cUWs6nu51tGIZ93qCRNyIhpfWETT3IKAtqUV5MvEI,2052
-letscode/agent/tools.py,sha256=gTKLXfmIPqeN0ev6JiisJQSYEzqVe6H89uRndVEIS-8,8200
+letscode/agent/tools.py,sha256=B5Q3nhkv2-VRYWVSsXm7WOY1NDFT0RtWjGrfmiSExtQ,6953
 letscode/cli/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 letscode/cli/app.py,sha256=sfDaTO_6vNOHTvIYzR6U5MA_BQQbd-ELTpZZ1OkoCPY,21159
-letscode/cli/rpc.py,sha256=JYO2CZrlypY1Wbrc-8ODWYOD1dq1pqjqBlMlIncQEJ0,10877
+letscode/cli/rpc.py,sha256=N8h1vW8Ira-GkIORsxHZY4XWOr4iXD4MnT0ZZG5KfSw,10976
 letscode/commands/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 letscode/commands/builtin.py,sha256=2-Kqu0l-eW6b3-NYRjAqKrb5iz5d2dmS53qRAVH_XRU,12615
 letscode/commands/dispatch.py,sha256=sxr3z5NlErG4W5fwdTVIjTEK9gzID3NitwHRDHKHDcs,4080
@@ -34,7 +34,7 @@ letscode/llm/types.py,sha256=yjPyZzZ983vbRTOWTD5hexTuHCNeWMVJtwzvRpAvfAg,1377
 letscode/plugins/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 letscode/plugins/builtin.py,sha256=JRIlv2fD3i0rliKSgzTEXI37my_HcVffkmsZfXGdPDc,4552
 letscode/plugins/manager.py,sha256=u2hgbt5nIW8_MKeofy-XmPyDAN0bcrtrVcWDIC9jFO4,2760
-letscode/plugins/registries.py,sha256=pQ8tc0gmSjIGrmUeVdmvLxgfM6vEl604Ubd5Z0MsFvI,8367
+letscode/plugins/registries.py,sha256=S9igLcStCbHCGeg6d4IPN-j9z5J3K0ssNS90-dh8G-k,7496
 letscode/session/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 letscode/session/format.py,sha256=XIqQ7Z4yRzNghoXgI2V7J6uyKXL4oLIZmHg8rSM7CAM,2714
 letscode/session/store.py,sha256=m0llMEq6qhT4vMMRy2kjgpGD4YdXX_7hwalPf4OdnH4,6732
@@ -48,7 +48,7 @@ letscode/tools/builtin.py,sha256=bx-nPUwESAHTam4ts13BXgJYoZbx8KBC8estBe6YuqM,653
 letscode/tools/edit.py,sha256=QyCeouEco9NsO7aAZao3xzQcYTi4um8RCpHXawtuJHU,4319
 letscode/tools/read.py,sha256=TJ4GFBu-AahNu7fByAB1x9mFW3O-j_ikh3usF3znHQk,3129
 letscode/tools/write.py,sha256=JSBoKqwZ6OZ0Ut34__WiCqVzwREx3vxYOGsRNpClZm8,2445
-letscode-0.5.0.dist-info/METADATA,sha256=q-Ep5Lgm9-8eVrw7swZ7llCLPwgUxlrXOTJ-Sh-fz2E,11968
-letscode-0.5.0.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
-letscode-0.5.0.dist-info/entry_points.txt,sha256=7wTui6fYLJ-_V7rwBMFloXcRbc7rYTPkcvVwZ7SV9p8,51
-letscode-0.5.0.dist-info/RECORD,,
+letscode-0.6.1.dist-info/METADATA,sha256=Ui51BAgKqJy7M7pHqYTxomORl-UW3PRxnB4bHZxkYVQ,11934
+letscode-0.6.1.dist-info/WHEEL,sha256=mffPy8wBnZQn2VnJUU5jE99KsxaSfiyMHV9Yt0aLVxs,87
+letscode-0.6.1.dist-info/entry_points.txt,sha256=7wTui6fYLJ-_V7rwBMFloXcRbc7rYTPkcvVwZ7SV9p8,51
+letscode-0.6.1.dist-info/RECORD,,

{letscode-0.5.0.dist-info → letscode-0.6.1.dist-info}/WHEEL

@@ -1,4 +1,4 @@
 Wheel-Version: 1.0
-Generator: hatchling 1.29.0
+Generator: hatchling 1.30.1
 Root-Is-Purelib: true
 Tag: py3-none-any