mirror of
https://github.com/NCBM/plyngent.git
synced 2026-11-07 15:37:55 +08:00
core/tools: add READ_ONLY tool attribute
New ToolTag.READ_ONLY marks tools that never mutate host state, plus ToolRegistry.clone(read_only_only=True) to select just those (for safe side turns / planning). Builtin read-only set: read_file, get_truncated, glob_paths, grep_files, listdir, tree, todo_list, wait, ask_user_*, vcs_* and fetch. fetch is tagged READ_ONLY but can mutate via method; a new read_only_context ContextVar makes it accept GET only while set.
This commit is contained in:
@@ -42,6 +42,7 @@ class ToolTag(Flag):
|
||||
YOLO = auto() # soft-confirm: eligible for YOLO auto-approve
|
||||
INSTANCE_STATE = auto() # needs instance-scoped state
|
||||
SESSION_STATE = auto() # needs session-scoped state
|
||||
READ_ONLY = auto() # never mutates host state; safe for read-only contexts
|
||||
|
||||
|
||||
class ToolDefinition:
|
||||
@@ -266,14 +267,22 @@ class ToolRegistry:
|
||||
session_state: object | None = None,
|
||||
yolo: bool | None = None,
|
||||
auto_bind_state: bool | None = None,
|
||||
read_only_only: bool = False,
|
||||
) -> ToolRegistry:
|
||||
"""New registry with the same tools/hooks and rebound host state.
|
||||
|
||||
*instance_state* / *session_state* are applied as given (use the parent
|
||||
registry's handles only when the caller passes them through).
|
||||
|
||||
With *read_only_only*, only :attr:`ToolTag.READ_ONLY` tools are kept
|
||||
(e.g. for safe side turns / planning); hooks and policy flags are
|
||||
preserved.
|
||||
"""
|
||||
definitions = self.definitions()
|
||||
if read_only_only:
|
||||
definitions = [d for d in definitions if d.tags & ToolTag.READ_ONLY]
|
||||
return ToolRegistry(
|
||||
self.definitions(),
|
||||
definitions,
|
||||
danger=self._danger,
|
||||
on_confirm=self._on_confirm,
|
||||
yolo=self._yolo if yolo is None else yolo,
|
||||
|
||||
@@ -4,7 +4,7 @@ from plyngent.agent import ToolTag, tool
|
||||
from plyngent.prompting import NonInteractiveError, ask_async
|
||||
|
||||
|
||||
@tool(name="ask_user_line", tags=ToolTag.LOCAL)
|
||||
@tool(name="ask_user_line", tags=ToolTag.LOCAL | ToolTag.READ_ONLY)
|
||||
async def ask_user(question: str, default: str = "") -> str:
|
||||
"""Ask the human a free-form one-line question and return their answer.
|
||||
|
||||
|
||||
@@ -36,7 +36,7 @@ def parse_options(options: list[object]) -> list[ChoiceOption]:
|
||||
return out
|
||||
|
||||
|
||||
@tool(name="ask_user_choice", tags=ToolTag.LOCAL)
|
||||
@tool(name="ask_user_choice", tags=ToolTag.LOCAL | ToolTag.READ_ONLY)
|
||||
async def choose_user(
|
||||
question: str,
|
||||
options: list[str] | list[dict[str, object]],
|
||||
|
||||
@@ -46,7 +46,7 @@ def parse_fields(fields: list[dict[str, object]]) -> list[FormField]:
|
||||
return out
|
||||
|
||||
|
||||
@tool(name="ask_user_form", tags=ToolTag.LOCAL)
|
||||
@tool(name="ask_user_form", tags=ToolTag.LOCAL | ToolTag.READ_ONLY)
|
||||
async def form_user(
|
||||
title: str,
|
||||
fields: list[dict[str, object]],
|
||||
|
||||
@@ -8,7 +8,7 @@ from plyngent.prompting import get_prompt_backend, read_line_with_timeout, run_p
|
||||
WAIT_PROMPT = "Waiting for {duration}s. Press Enter to disturb. Feel free to ship any reason before pressing Enter. "
|
||||
|
||||
|
||||
@tool(name="wait", tags=ToolTag.LOCAL)
|
||||
@tool(name="wait", tags=ToolTag.LOCAL | ToolTag.READ_ONLY)
|
||||
async def wait(duration: int, *, reason: str | None = None) -> str:
|
||||
"""Wait ``duration`` seconds before continuing.
|
||||
|
||||
|
||||
@@ -137,6 +137,29 @@ def bind_session(state: SessionState | None) -> Generator[SessionState | None]:
|
||||
_session.reset(token)
|
||||
|
||||
|
||||
_read_only: ContextVar[bool] = ContextVar("plyngent_read_only_context", default=False)
|
||||
|
||||
|
||||
@contextmanager
|
||||
def read_only_context() -> Generator[None]:
|
||||
"""Mark the current context as read-only for tool execution.
|
||||
|
||||
Tools that may mutate depending on arguments (e.g. ``fetch`` with a
|
||||
non-GET method) check :func:`is_read_only_context` and refuse to mutate
|
||||
while it is set.
|
||||
"""
|
||||
token = _read_only.set(True)
|
||||
try:
|
||||
yield
|
||||
finally:
|
||||
_read_only.reset(token)
|
||||
|
||||
|
||||
def is_read_only_context() -> bool:
|
||||
"""Whether tools currently run inside a read-only context."""
|
||||
return _read_only.get()
|
||||
|
||||
|
||||
@contextmanager
|
||||
def bind_tool_context(
|
||||
*,
|
||||
|
||||
@@ -15,7 +15,7 @@ from plyngent.tools.truncate_token import (
|
||||
)
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def get_truncated(token: str, *, max_chars: int = DEFAULT_TOOL_RESULT_MAX_CHARS) -> str:
|
||||
"""Fetch the next chunk of a truncated result using its ``truncate_token``.
|
||||
|
||||
|
||||
@@ -65,7 +65,7 @@ def _resolve_glob_base(path: str) -> tuple[Path, Path] | str:
|
||||
return root, base
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def glob_paths(
|
||||
pattern: str,
|
||||
path: str = ".",
|
||||
|
||||
@@ -127,7 +127,7 @@ def _compile_pattern(pattern: str, *, case_insensitive: bool) -> re.Pattern[str]
|
||||
return f"error: invalid regex: {exc}"
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def grep_files(
|
||||
pattern: str,
|
||||
path: str = ".",
|
||||
|
||||
@@ -4,7 +4,7 @@ from plyngent.agent import ToolTag, tool
|
||||
from plyngent.tools.workspace import resolve_path
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def listdir(path: str = ".") -> str:
|
||||
"""List entries in a directory under the workspace (name and type)."""
|
||||
target = resolve_path(path)
|
||||
|
||||
@@ -49,7 +49,7 @@ def read_raw_text(path: str) -> tuple[str | None, str]:
|
||||
return target.read_text(encoding="utf-8", errors="replace"), ""
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def read_file(
|
||||
path: str,
|
||||
*,
|
||||
|
||||
@@ -151,7 +151,7 @@ def _resolve_skip_basenames(skip_dirs: Sequence[str] | None) -> frozenset[str]:
|
||||
return frozenset(name for name in skip_dirs if name)
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def tree(
|
||||
path: str = ".",
|
||||
*,
|
||||
|
||||
@@ -3,6 +3,7 @@
|
||||
from __future__ import annotations
|
||||
|
||||
from plyngent.agent import ToolTag, tool
|
||||
from plyngent.tools.context import is_read_only_context
|
||||
from plyngent.tools.net.client import http_fetch
|
||||
from plyngent.tools.net.policy import (
|
||||
DEFAULT_MAX_BODY_CHARS_IN,
|
||||
@@ -66,7 +67,7 @@ def format_fetch_result(
|
||||
return "\n".join(parts)
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.YOLO | ToolTag.TRUSTABLE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.YOLO | ToolTag.TRUSTABLE | ToolTag.READ_ONLY)
|
||||
async def fetch(
|
||||
url: str,
|
||||
*,
|
||||
@@ -92,9 +93,13 @@ async def fetch(
|
||||
Private/loopback/link-local hosts require an explicit human policy grant
|
||||
(not skipped by YOLO). HTTPS→HTTP redirects are blocked unless
|
||||
``allow_http_downgrade`` is true.
|
||||
|
||||
Read-only contexts (e.g. ``/btw --tools=read``) accept only ``GET``.
|
||||
"""
|
||||
try:
|
||||
verb = normalize_method(method)
|
||||
if is_read_only_context() and verb != "GET":
|
||||
return "error: fetch method must be GET in a read-only context (POST/PUT/DELETE mutate)"
|
||||
parsed = parse_fetch_url(url)
|
||||
hdrs = normalize_request_headers(headers, user_agent=user_agent)
|
||||
body_bytes: bytes | None = None
|
||||
|
||||
@@ -48,7 +48,7 @@ async def _with_todo_stack(mutator: Callable[[TodoStack], str]) -> str:
|
||||
return result
|
||||
|
||||
|
||||
@tool(name="todo_list", tags=ToolTag.LOCAL | ToolTag.PUBLIC | ToolTag.SESSION_STATE)
|
||||
@tool(name="todo_list", tags=ToolTag.LOCAL | ToolTag.PUBLIC | ToolTag.SESSION_STATE | ToolTag.READ_ONLY)
|
||||
async def todo_list() -> str:
|
||||
"""Show the LIFO stack of **task groups** (TOP group = current breakdown level).
|
||||
|
||||
|
||||
@@ -22,7 +22,7 @@ def _backend_or_error() -> VcsBackend | str:
|
||||
return backend
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def vcs_kind() -> str:
|
||||
"""Return the detected VCS kind under the workspace (e.g. ``git``), or an error."""
|
||||
backend = _backend_or_error()
|
||||
@@ -31,7 +31,7 @@ async def vcs_kind() -> str:
|
||||
return backend.kind
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def vcs_status() -> str:
|
||||
"""Show working-tree status for the detected VCS (read-only)."""
|
||||
backend = _backend_or_error()
|
||||
@@ -40,7 +40,7 @@ async def vcs_status() -> str:
|
||||
return backend.status()
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def vcs_diff(path: str = "", *, staged: bool = False) -> str:
|
||||
"""Show a unified diff for the detected VCS (read-only).
|
||||
|
||||
@@ -54,7 +54,7 @@ async def vcs_diff(path: str = "", *, staged: bool = False) -> str:
|
||||
return backend.diff(staged=staged, path=rel)
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def vcs_log(limit: int = 10) -> str:
|
||||
"""Show recent commits for the detected VCS (read-only)."""
|
||||
if limit < 1:
|
||||
@@ -65,7 +65,7 @@ async def vcs_log(limit: int = 10) -> str:
|
||||
return backend.log(limit=limit)
|
||||
|
||||
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE)
|
||||
@tool(tags=ToolTag.LOCAL | ToolTag.INSTANCE_STATE | ToolTag.READ_ONLY)
|
||||
async def vcs_branch() -> str:
|
||||
"""Show the current branch / named head for the detected VCS (read-only)."""
|
||||
backend = _backend_or_error()
|
||||
|
||||
Reference in New Issue
Block a user