WIP: feat(curry): add versioned ledger tools and direct commands #10

Draft
Athena wants to merge 5 commits from Athena/NexusOS:codex/curry-tools-pr into main
14 changed files with 2621 additions and 13 deletions
+1
View File
@@ -13,6 +13,7 @@ synapse/memory/memory.db
synapse/memory/memory.db-wal
synapse/memory/memory.db-shm
assets/gitnexus-logo.svg
/data/curry.db
*.db-wal
*.db-shm
.DS_Store
+6
View File
@@ -126,6 +126,12 @@ React 19 + Vite. No routing library — `App.jsx` manages page state in a single
### Persistent Storage
Most data lands in `synapse/memory/memory.db` (SQLite, WAL mode). Tables: memory facts, conversations, messages, app settings. `synapse/memory/store.py` (`PersistentMemoryStore`) owns the schema and all queries. Playbooks are the exception — they live as YAML files in `data/playbooks/` (see Playbook System). `nexus_config.py` defines all paths; it also ensures all required directories exist on import.
### Curry (`synapse/curry_core.py` + `synapse/curry_store.py`)
`curry_core.py` is vendored from [Athena-Pro/Curry](https://github.com/Athena-Pro/Curry), with two deliberate deviations from upstream documented in the file's own docstring (a sandbox-escape fix and a `check_same_thread=False` connection fix) — an immutable, versioned fact store (constants, functions, model registrations, inference provenance) backed by its own SQLite file (`CURRY_DB` in `nexus_config.py`, separate from `memory.db`). `curry_store.py` opens it into a module-level singleton (`curry_db`) at import time — the same pattern as `memory.store.store` / `playbooks.store.playbook_store` — so it's preloaded and callable from anywhere in the backend without extra setup. It ships inside the wheel (`bin/check.sh`'s packaging gate asserts this) and has no external dependencies of its own. Ten `curry_*` tools in `tools.py` expose it to chat (`curry_declare_constant`, `curry_call_function`, etc.); the five that write or execute are ACTION tools in `ALWAYS_ASK_ACTION_TOOLS`, same approval floor as `edit_source`. Re-sync `curry_core.py` from upstream by hand, not by script.
### Direct tool invocation (`synapse/slash_commands.py`)
A chat message that's nothing but `/tool_name(arg=val, ...)` (Python-call-shaped, arguments parsed via `ast.literal_eval` only — no names, no calls, no attribute access) dispatches straight through `tools.dispatch()`, skipping model selection, context assembly, and the ask-policy approval round-trip. A human typing it is the approval. Wired into `chat_stream_endpoint` as an early short-circuit; the TUI's `_handle_slash` falls through to the backend for anything shaped like a tool call that isn't one of its own local meta-commands (`/help`, `/model`, `/new`).
### Logs & Runtime State
- `runtime/backend.log`, `runtime/frontend.log`, `runtime/memory.log` — service stdout
- `runtime/logs/ollama.log`, `runtime/logs/chat.log`
+4
View File
@@ -63,6 +63,10 @@ if not any(n.startswith("synapse/_resources/web/") for n in names):
sys.exit("wheel is missing the compiled web UI (cd interface/web && npm run build)")
if not any(n.startswith("synapse/_resources/playbooks/") for n in names):
sys.exit("wheel is missing the seed playbooks")
if "synapse/curry_core.py" not in names or "synapse/curry_store.py" not in names:
sys.exit("wheel is missing vendored Curry (synapse/curry_core.py / curry_store.py)")
if "synapse/slash_commands.py" not in names:
sys.exit("wheel is missing synapse/slash_commands.py")
print(f"wheel OK: {len(names)} files")
PY
else
+14
View File
@@ -15,6 +15,7 @@ from typing import Any
import httpx
from synapse.nexus_config import settings
from synapse.slash_commands import parse_slash_command
from .monitor import collect_snapshot
@@ -344,6 +345,19 @@ class NexusTUI:
log.write(
f"[dim]model:[/] {_escape(self._model or '(auto)')}"
)
elif parse_slash_command(text) is not None:
# Shaped like /tool_name(arg=val, ...) rather than one of
# the local meta-commands above — not handled here, sent
# to the backend as-is. chat_stream_endpoint recognizes
# and dispatches it directly (see synapse/slash_commands.py);
# a malformed one still goes through so the user sees the
# backend's own error, with full context, in one place.
if self._busy:
log.write(
"[yellow]Still streaming — wait or Ctrl+C to interrupt[/]"
)
else:
self._start_chat(text)
else:
log.write(
f"[red]unknown command[/] /{_escape(cmd)} — try /help"
+8 -2
View File
@@ -167,10 +167,16 @@ async def _run_tool_loop(manager, messages, model, tool_schemas, temperature, nu
break
messages.append(msg)
# If any action tool needs per-call approval, pause and wait for the user.
# Curry write/execute tools always require approval when model-issued,
# even if the global policy allows lower-risk actions. A human-typed
# /tool(...) command is dispatched separately by main.py.
decisions = None
action_calls = [c for c in calls if _tools.is_action(c.get("function", {}).get("name", ""))]
if policy == "ask" and action_calls:
needs_approval = policy == "ask" or any(
c.get("function", {}).get("name", "") in _tools.ALWAYS_ASK_ACTION_TOOLS
for c in action_calls
)
if needs_approval and action_calls:
event = asyncio.Event()
# Single-use capability token, delivered only to the client that owns
# this stream. /chat/approve requires it, so knowing the (guessable,
File diff suppressed because it is too large Load Diff
+22
View File
@@ -0,0 +1,22 @@
"""NexusOS's own Curry instance: preloaded at import time, ready to be called.
Curry (curry_core.py, vendored alongside this file) is an immutable, versioned
fact store - constants, functions, model registrations, and inference
provenance, backed by SQLite. Nothing in NexusOS wires chat/model-authored
content into it yet; this module only makes it available - `from
synapse.curry_store import curry_db` and call `declare_constant`,
`get_constant_latest`, `declare_function`, `call_function`, etc. directly, the
same way `synapse.memory.store.store` and `synapse.playbooks.store.playbook_store`
are used elsewhere in this codebase.
Kept as a separate database file (CURRY_DB) from the memory/conversation store
on purpose: Curry's schema and lifecycle are independent of the memory store's.
"""
from __future__ import annotations
from .curry_core import Curry
from .nexus_config import CURRY_DB
curry_db = Curry(str(CURRY_DB))
__all__ = ["curry_db"]
+61 -4
View File
@@ -182,7 +182,9 @@ async def _generate_conversation_title(first_message: str, model: str) -> Option
from .memory.store import store, MemoryItem
from .playbooks.store import playbook_store, PlaybookItem
from .curry_store import curry_db # noqa: F401 - import triggers Curry's own preload at startup
from .search import needs_web_search, web_search
from . import slash_commands as _slash_commands
MEMORY_SERVICE = settings.memory_url
@@ -460,6 +462,38 @@ async def _resume_dropped_extractions() -> None:
# -------------------------
# Chat (streaming)
# -------------------------
async def _slash_command_stream(
slash: "_slash_commands.SlashCommand | _slash_commands.SlashCommandError",
conversation_id: str,
) -> AsyncGenerator[str, None]:
"""Dispatch an explicit slash-command without a model or approval round-trip."""
if isinstance(slash, _slash_commands.SlashCommandError):
yield f"event: error\ndata: {_json.dumps({'detail': slash.text})}\n\n"
return
if slash.tool not in _tools.REGISTRY:
detail = f"unknown tool: {slash.tool}"
yield f"event: error\ndata: {_json.dumps({'detail': detail})}\n\n"
return
yield f"event: status\ndata: {_json.dumps({'tool': slash.tool})}\n\n"
raw_result = await _tools.dispatch(slash.tool, slash.args)
content = raw_result
try:
parsed = _json.loads(raw_result)
if isinstance(parsed, dict) and isinstance(parsed.get("fence"), str):
content = parsed["fence"]
else:
content = _json.dumps(parsed, indent=2, ensure_ascii=False)
except (TypeError, ValueError):
pass
store.add_message(conversation_id, "assistant", content)
yield f"data: {_json.dumps(content)}\n\n"
yield "event: done\ndata: {}\n\n"
@app.post("/chat/stream")
async def chat_stream_endpoint(payload: Dict[str, Any]):
# Bound concurrent chats so a flood can't fan out unlimited model inference.
@@ -469,13 +503,39 @@ async def chat_stream_endpoint(payload: Dict[str, Any]):
_chat_slot_held = True
try:
message = payload.get("message", "")
conversation_id = payload.get("conversation_id") or str(_uuid.uuid4())
if not message:
raise HTTPException(status_code=400, detail="Missing 'message'")
# A whole-message /tool_name(arg=val, ...) command is an explicit human
# action. It skips model selection and approval but not the tool's own
# validation; slash_commands.py accepts literal keyword values only.
slash = _slash_commands.parse_slash_command(message)
if slash is not None:
project_id = store.conversation_project(conversation_id)
if project_id is None:
project_id = store.get_settings().get("active_project", "")
store.create_conversation(conversation_id, project_id or "")
store.add_message(conversation_id, "user", message)
slash_stream = _slash_command_stream(slash, conversation_id)
async def _slash_guarded() -> AsyncGenerator[str, None]:
try:
async for chunk in slash_stream:
yield chunk
finally:
_CHAT_INFLIGHT.release()
_chat_slot_held = False
return StreamingResponse(_slash_guarded(), media_type="text/event-stream")
app_settings = store.get_settings()
# Model precedence: explicit request > active playbook's pinned model > auto-select.
_active_pb = playbook_manager.get_main_playbook()
_pb_model = _active_pb.model if (_active_pb and _active_pb.model) else ""
model = payload.get("model") or _pb_model or await _auto_select_model(message)
context = payload.get("context", {})
conversation_id = payload.get("conversation_id") or str(_uuid.uuid4())
history = payload.get("history", [])
temperature = payload.get("temperature", app_settings.get("temperature"))
num_ctx = payload.get("num_ctx", app_settings.get("num_ctx", 0))
@@ -483,9 +543,6 @@ async def chat_stream_endpoint(payload: Dict[str, Any]):
gpu_offload = payload.get("gpu_offload", app_settings.get("gpu_offload", -1))
num_gpu = await get_ollama_manager().resolve_num_gpu(gpu_offload, model)
if not message:
raise HTTPException(status_code=400, detail="Missing 'message'")
# Resolve the project scope: an existing conversation keeps its bound project;
# a brand-new one inherits the current workspace (active_project setting).
# Everything project-scoped below (instructions, memory facts, RAG) uses it.
+7 -1
View File
@@ -160,6 +160,11 @@ SEED_PLAYBOOK_DIR = (
# --- DATABASE / STORAGE FILES (match your repo) ---
MEMORY_DB = _configured_path("memory_db", "NEXUS_MEMORY_DB", MEMORY_DIR / "memory.db")
# Vendored Curry (synapse/curry_core.py) database: immutable versioned
# constants/functions/models + inference provenance. Separate file from
# MEMORY_DB on purpose - Curry's schema and lifecycle are independent of the
# memory/conversation store.
CURRY_DB = _configured_path("curry_db", "NEXUS_CURRY_DB", DATA_DIR / "curry.db")
# --- LOG FILES ---
BACKEND_LOG = RUNTIME_DIR / "backend.log"
@@ -180,6 +185,7 @@ _REQUIRED_DIRS = (
UPLOADS_DIR,
EXPORTS_DIR,
MEMORY_DB.parent,
CURRY_DB.parent,
)
@@ -424,7 +430,7 @@ __all__ = ["Settings", "settings", "path", "VERSION",
"read_user_config", "write_user_config", "init_state", "INITIALIZED_FILES",
"DATA_DIR", "MODELS_DIR", "RUNTIME_DIR",
"MEMORY_DIR", "LOGS_DIR", "PLAYBOOK_DIR", "UPLOADS_DIR",
"EXPORTS_DIR", "MEMORY_DB", "WEB_DIST_DIR", "FRONTEND_SOURCE_DIR",
"EXPORTS_DIR", "MEMORY_DB", "CURRY_DB", "WEB_DIST_DIR", "FRONTEND_SOURCE_DIR",
"ASSETS_DIR", "SEED_PLAYBOOK_DIR",
"BACKEND_LOG", "OLLAMA_LOG", "CHAT_LOG",
"ALLOWED_HOSTS", "ALLOWED_ORIGINS",
+94
View File
@@ -0,0 +1,94 @@
"""Direct tool invocation from chat input: `/tool_name(arg=val, arg=val)`.
A human typing this IS the approval there's no one else to ask — so a
recognized slash-command skips the ask-policy round-trip entirely and
dispatches straight through `tools.dispatch()`, the same entry point a
model-issued tool call already goes through. It does not bypass anything a
tool validates internally (path boundaries, size caps, Curry's own sandbox
checks, etc.) only the human-approval step, which this message already is.
Argument values are parsed with `ast.literal_eval`, not `eval()`: strings,
numbers, booleans, None, and literal lists/dicts/tuples only. There is no way
to reference a name, call a function, or access an attribute in this syntax
a malformed or hostile-looking argument fails to parse rather than executing
anything, which is the "lint, not run" property that makes this different
from just typing Python.
The whole message must be nothing but the command this is a deliberate
command line, not a directive embedded in prose. Anything else (including a
message that merely starts with `/` but isn't shaped like this) falls through
to the normal chat/model path unchanged.
"""
from __future__ import annotations
import ast
import re
from dataclasses import dataclass
from typing import Any, Optional
# name(args) where name is a plain identifier — the same shape as a Python
# function call, so it reads the way the tool's own schema already documents
# it. re.DOTALL: argument values (e.g. a multi-line body= string) may
# legitimately contain newlines.
_COMMAND_RE = re.compile(r"^/([A-Za-z_][A-Za-z0-9_]*)\((.*)\)\s*$", re.DOTALL)
@dataclass
class SlashCommand:
tool: str
args: dict[str, Any]
@dataclass
class SlashCommandError:
text: str
def parse_slash_command(message: str) -> Optional[SlashCommand | SlashCommandError]:
"""Parse `/tool_name(arg=val, ...)`.
Returns None when `message` isn't shaped like a slash-command at all (the
caller should treat it as an ordinary chat message). Returns
SlashCommandError when it looks like one but is malformed that's worth
telling the user about rather than silently sending "/curry_call_fnction(...)"
to the model as if it were prose.
"""
stripped = (message or "").strip()
match = _COMMAND_RE.match(stripped)
if not match:
return None
tool_name, raw_args = match.group(1), match.group(2).strip()
if not raw_args:
return SlashCommand(tool=tool_name, args={})
# Parse "k1=v1, k2=v2" as keyword arguments to a call with no positional
# arguments and no function to actually call — ast.parse(mode='eval') on a
# synthetic call expression reuses Python's own keyword-argument grammar
# (quoting, nesting, trailing commas) instead of hand-rolling a parser for
# it, while call() as a bare name is never resolved or invoked.
try:
tree = ast.parse(f"call({raw_args})", mode="eval")
except SyntaxError as e:
return SlashCommandError(f"could not parse arguments for /{tool_name}(...): {e}")
call_node = tree.body
if not isinstance(call_node, ast.Call) or call_node.args:
return SlashCommandError(
f"/{tool_name}(...) arguments must be keyword form: arg=value, arg=value"
)
args: dict[str, Any] = {}
for kw in call_node.keywords:
if kw.arg is None: # **mapping unpacking — no source for that here
return SlashCommandError(f"/{tool_name}(...) does not support **-unpacking")
try:
args[kw.arg] = ast.literal_eval(kw.value)
except (ValueError, SyntaxError):
return SlashCommandError(
f"/{tool_name}(...): argument '{kw.arg}' must be a literal "
"(string, number, bool, None, list, dict, or tuple) — not an "
"expression, name, or call"
)
return SlashCommand(tool=tool_name, args=args)
+344 -6
View File
@@ -3,16 +3,20 @@
Ollama drives the calling: `/api/chat` with a `tools` param returns
`message.tool_calls`, and this module is just the registry + dispatch.
Most tools READ local state (memory, history, documents, models). A few act:
`web_search`/`fetch_url` make outbound HTTP requests, and `remember` WRITES a
memory fact. The per-playbook allowlist (`PlaybookItem.tools`) is the security
boundary an action tool only fires when a playbook explicitly lists it.
Most tools READ local state (memory, history, documents, models). Some act:
`web_search`/`fetch_url` make outbound HTTP requests, `remember` writes a
memory fact, and `curry_*` reads or writes NexusOS's vendored Curry ledger.
The per-playbook allowlist (`PlaybookItem.tools`) is the first gate. Curry
write/execute tools additionally require per-call approval when model-issued.
A message consisting only of `/tool_name(arg=val, ...)` dispatches directly;
see `synapse/slash_commands.py` for that explicit-human-command boundary.
"""
from __future__ import annotations
import json
from typing import Awaitable, Callable
from .curry_store import curry_db
from .memory.store import store, MemoryItem
from .ollama_manager import get_ollama_manager
@@ -215,6 +219,124 @@ async def _list_files(pattern: str = "", **_) -> str:
return json.dumps(sorted(hits))
# Curry (synapse/curry_core.py, vendored) — immutable, versioned constants and
# functions. Expected caller errors keep the same structured JSON shape as the
# other tools instead of falling through dispatch()'s generic error envelope.
_CURRY_FENCE_LANG = "nexus-curry"
def _curry_fence(payload: dict) -> str:
body = json.dumps(payload, ensure_ascii=False, default=str).replace("`", "\\u0060")
return f"```{_CURRY_FENCE_LANG}\n{body}\n```"
async def _curry_call(fn, *args, **kwargs) -> dict:
# Curry holds one SQLite connection. Calls stay on the event-loop thread,
# where these local database operations are short and naturally serialized.
try:
result = fn(*args, **kwargs)
return {"ok": True, "result": result}
except (KeyError, ValueError, TypeError, RuntimeError) as exc:
return {"ok": False, "error": str(exc)}
async def _curry_declare_constant(
id: str = "", version: int = 0, value=None, type_signature: str = "",
description: str = "", **_,
) -> str:
"""ACTION tool: declare a new, immutable version of a named constant."""
out = await _curry_call(
curry_db.declare_constant, id, version, value, type_signature, description or None
)
if out["ok"]:
out = {"ok": True, "id": id, "version": version}
out["fence"] = _curry_fence({"kind": "declare_constant", **out})
return json.dumps(out)
async def _curry_get_constant(id: str = "", version: int = 0, **_) -> str:
return json.dumps(await _curry_call(curry_db.get_constant, id, version))
async def _curry_get_constant_latest(id: str = "", **_) -> str:
return json.dumps(await _curry_call(curry_db.get_constant_latest, id))
async def _curry_list_constants(active_only: bool = True, **_) -> str:
return json.dumps(await _curry_call(curry_db.list_constants, active_only))
async def _curry_retire_constant(
id: str = "", version: int = 0, reason: str = "", **_,
) -> str:
out = await _curry_call(
curry_db.retire_constant_with_reason,
id,
version,
reason or "retired via tool call",
)
return json.dumps(out)
async def _curry_declare_function(
name: str = "", version: int = 0, body: str = "",
constant_bindings: dict | None = None, function_bindings: dict | None = None,
is_pure: bool = False, expected_args: list | None = None,
description: str = "", arg_descriptions: dict | None = None, **_,
) -> str:
"""ACTION tool: declare one statically validated expression."""
out = await _curry_call(
curry_db.declare_function,
name,
version,
body,
constant_bindings or {},
function_bindings or {},
is_pure,
expected_args,
description or None,
arg_descriptions,
)
if out["ok"]:
out = {"ok": True, "name": name, "version": version}
out["fence"] = _curry_fence({"kind": "declare_function", **out})
return json.dumps(out)
async def _curry_get_function(name: str = "", version: int = 0, **_) -> str:
return json.dumps(await _curry_call(curry_db.get_function, name, version))
async def _curry_list_functions(active_only: bool = True, **_) -> str:
return json.dumps(await _curry_call(curry_db.list_functions, active_only))
async def _curry_call_function(
name: str = "", version: int = 0, args: dict | None = None, **_,
) -> str:
out = await _curry_call(curry_db.call_function, name, version, args or {})
if out["ok"]:
out["fence"] = _curry_fence({
"kind": "call_function",
"name": name,
"version": version,
**out,
})
return json.dumps(out)
async def _curry_retire_function(
name: str = "", version: int = 0, reason: str = "", **_,
) -> str:
out = await _curry_call(
curry_db.retire_function_with_reason,
name,
version,
reason or "retired via tool call",
)
return json.dumps(out)
# name -> (schema, callable). Schema is the OpenAI/Ollama function-tool format.
REGISTRY: dict[str, tuple[dict, Callable[..., Awaitable[str]]]] = {
"search_memory": (
@@ -360,13 +482,229 @@ REGISTRY: dict[str, tuple[dict, Callable[..., Awaitable[str]]]] = {
},
_remember,
),
"curry_declare_constant": (
{
"type": "function",
"function": {
"name": "curry_declare_constant",
"description": (
"Declare a new immutable version of a Curry constant. "
"Requires per-call human approval when model-issued."
),
"parameters": {
"type": "object",
"properties": {
"id": {"type": "string", "description": "Constant identifier."},
"version": {"type": "integer", "description": "A new, higher version."},
"value": {"description": "Value matching type_signature."},
"type_signature": {
"type": "string",
"description": (
"Float64 | Int32 | String | Blob | Json | Tokens | "
"Currency | Bool"
),
},
"description": {"type": "string"},
},
"required": ["id", "version", "value", "type_signature"],
},
},
},
_curry_declare_constant,
),
"curry_get_constant": (
{
"type": "function",
"function": {
"name": "curry_get_constant",
"description": "Retrieve a Curry constant by exact id and version.",
"parameters": {
"type": "object",
"properties": {
"id": {"type": "string"},
"version": {"type": "integer"},
},
"required": ["id", "version"],
},
},
},
_curry_get_constant,
),
"curry_get_constant_latest": (
{
"type": "function",
"function": {
"name": "curry_get_constant_latest",
"description": "Retrieve the latest active version of a Curry constant.",
"parameters": {
"type": "object",
"properties": {"id": {"type": "string"}},
"required": ["id"],
},
},
},
_curry_get_constant_latest,
),
"curry_list_constants": (
{
"type": "function",
"function": {
"name": "curry_list_constants",
"description": "List Curry constants.",
"parameters": {
"type": "object",
"properties": {"active_only": {"type": "boolean"}},
},
},
},
_curry_list_constants,
),
"curry_retire_constant": (
{
"type": "function",
"function": {
"name": "curry_retire_constant",
"description": (
"Retire, but do not delete, a Curry constant version. "
"Requires per-call human approval when model-issued."
),
"parameters": {
"type": "object",
"properties": {
"id": {"type": "string"},
"version": {"type": "integer"},
"reason": {"type": "string"},
},
"required": ["id", "version"],
},
},
},
_curry_retire_constant,
),
"curry_declare_function": (
{
"type": "function",
"function": {
"name": "curry_declare_function",
"description": (
"Declare a new immutable Curry function version. The body is one "
"statically validated Python expression. Requires per-call human "
"approval when model-issued."
),
"parameters": {
"type": "object",
"properties": {
"name": {"type": "string"},
"version": {"type": "integer"},
"body": {"type": "string"},
"constant_bindings": {"type": "object"},
"function_bindings": {"type": "object"},
"is_pure": {"type": "boolean"},
"expected_args": {
"type": "array",
"items": {"type": "string"},
},
"description": {"type": "string"},
"arg_descriptions": {"type": "object"},
},
"required": ["name", "version", "body"],
},
},
},
_curry_declare_function,
),
"curry_get_function": (
{
"type": "function",
"function": {
"name": "curry_get_function",
"description": "Retrieve a Curry function by exact name and version.",
"parameters": {
"type": "object",
"properties": {
"name": {"type": "string"},
"version": {"type": "integer"},
},
"required": ["name", "version"],
},
},
},
_curry_get_function,
),
"curry_list_functions": (
{
"type": "function",
"function": {
"name": "curry_list_functions",
"description": "List Curry functions and their expected arguments.",
"parameters": {
"type": "object",
"properties": {"active_only": {"type": "boolean"}},
},
},
},
_curry_list_functions,
),
"curry_call_function": (
{
"type": "function",
"function": {
"name": "curry_call_function",
"description": (
"Execute an exact Curry function version with runtime arguments. "
"Requires per-call human approval when model-issued."
),
"parameters": {
"type": "object",
"properties": {
"name": {"type": "string"},
"version": {"type": "integer"},
"args": {"type": "object"},
},
"required": ["name", "version"],
},
},
},
_curry_call_function,
),
"curry_retire_function": (
{
"type": "function",
"function": {
"name": "curry_retire_function",
"description": (
"Retire, but do not delete, a Curry function version. "
"Requires per-call human approval when model-issued."
),
"parameters": {
"type": "object",
"properties": {
"name": {"type": "string"},
"version": {"type": "integer"},
"reason": {"type": "string"},
},
"required": ["name", "version"],
},
},
},
_curry_retire_function,
),
}
# Tools that act (write local state or reach the network). These require an
# explicit consent gate (settings.allow_action_tools) on top of the per-playbook
# allowlist — a playbook granting one isn't enough on its own.
ACTION_TOOLS = frozenset({"web_search", "fetch_url", "remember"})
# allowlist — a playbook granting one isn't enough on its own. Curry writes and
# execution additionally require per-call approval for model-issued calls.
CURRY_ALWAYS_ASK_TOOLS = frozenset({
"curry_declare_constant",
"curry_retire_constant",
"curry_declare_function",
"curry_retire_function",
"curry_call_function",
})
ACTION_TOOLS = frozenset({"web_search", "fetch_url", "remember"}) | CURRY_ALWAYS_ASK_TOOLS
ALWAYS_ASK_ACTION_TOOLS = CURRY_ALWAYS_ASK_TOOLS
def is_action(name: str) -> bool:
+53
View File
@@ -0,0 +1,53 @@
"""synapse/curry_core.py (vendored) + synapse/curry_store.py (NexusOS's preload).
Two concerns: the vendor sync didn't silently drop the sandbox fix from
https://github.com/Athena-Pro/Curry/pull/4, and curry_store gives NexusOS a
live instance for the registered chat tools.
"""
import pytest
from synapse.curry_core import Curry, TypeSignature
from synapse import curry_store
def test_curry_store_is_preloaded_and_open():
# curry_store.curry_db is a module-level singleton constructed at import
# time (mirrors synapse.memory.store.store / synapse.playbooks.store.playbook_store)
# - by the time this test runs, it has already opened its database file.
assert isinstance(curry_store.curry_db, Curry)
assert curry_store.curry_db.conn.execute("SELECT 1").fetchone()[0] == 1
def test_curry_db_path_matches_nexus_config(tmp_path, monkeypatch):
from synapse import nexus_config
assert str(curry_store.curry_db.db_path) == str(nexus_config.CURRY_DB)
def test_vendored_sandbox_fix_rejects_format_dunder_escape(tmp_path):
# Regression test for the vendored fix: a body that hides dunder-attribute
# traversal inside a str.format() field spec must still be rejected at
# declare time, not just the literal '.__class__' form. If a future
# re-vendor from upstream drops the fix, this is what catches it.
db = Curry(str(tmp_path / "sandbox_check.db"))
db.declare_function("helper", 1, "1")
exploit = "'{0.__globals__}'.format(helper)"
with pytest.raises(ValueError, match="format"):
db.declare_function("evil", 1, exploit, function_bindings={"helper": 1})
# the original, always-caught dunder-attribute form stays blocked too
with pytest.raises(ValueError):
db.declare_function("evil2", 1, "x.__class__", expected_args=["x"])
db.close()
def test_vendored_curry_basic_versioning_roundtrip(tmp_path):
db = Curry(str(tmp_path / "roundtrip.db"))
db.declare_constant("rate", 1, 0.1, TypeSignature.FLOAT64.value)
db.declare_function(
"apply_rate", 1, "amount * (1 + rate)",
constant_bindings={"rate": 1}, expected_args=["amount"],
)
assert db.call_function("apply_rate", 1, {"amount": 100}) == 110.00000000000001
db.close()
+204
View File
@@ -0,0 +1,204 @@
"""synapse/slash_commands.py (the /tool_name(arg=val) parser) and its wiring
into chat_stream_endpoint (direct dispatch, no model call, no approval
round-trip) plus the ten curry_* tools it can now reach.
"""
import json
import pytest
from fastapi.testclient import TestClient
from synapse.slash_commands import SlashCommand, SlashCommandError, parse_slash_command
from synapse.main import app
from synapse import tools
# ---------------------------------------------------------------------------
# Parser
# ---------------------------------------------------------------------------
def test_parses_keyword_arguments_as_python_literals():
result = parse_slash_command('/curry_call_function(name="x", version=1, args={"a": 1})')
assert result == SlashCommand(
tool="curry_call_function",
args={"name": "x", "version": 1, "args": {"a": 1}},
)
def test_parses_no_arguments():
assert parse_slash_command("/curry_list_functions()") == SlashCommand(tool="curry_list_functions", args={})
def test_non_slash_message_returns_none():
assert parse_slash_command("just chatting, not a command") is None
def test_slash_without_parens_returns_none():
# The TUI's own local commands (/model foo, /new) use this shape — must
# never be mistaken for a tool call.
assert parse_slash_command("/model gpt") is None
def test_slash_embedded_in_prose_returns_none():
assert parse_slash_command('hey /curry_call_function(name="x", version=1) run this') is None
def test_name_or_call_as_argument_value_is_rejected():
# ast.literal_eval only accepts literals — a bare name or a call is a
# parse failure, not a value, so nothing here is ever evaluated.
result = parse_slash_command("/curry_call_function(x=some_name)")
assert isinstance(result, SlashCommandError)
result2 = parse_slash_command('/curry_call_function(x=__import__("os"))')
assert isinstance(result2, SlashCommandError)
def test_positional_arguments_are_rejected():
result = parse_slash_command("/curry_call_function(1, 2)")
assert isinstance(result, SlashCommandError)
def test_double_star_unpacking_is_rejected():
result = parse_slash_command('/curry_call_function(**{"a": 1})')
assert isinstance(result, SlashCommandError)
def test_malformed_syntax_is_rejected():
result = parse_slash_command("/curry_call_function(name=)")
assert isinstance(result, SlashCommandError)
# ---------------------------------------------------------------------------
# Curry tool registration
# ---------------------------------------------------------------------------
_CURRY_ACTION_TOOLS = {
"curry_declare_constant", "curry_retire_constant",
"curry_declare_function", "curry_retire_function", "curry_call_function",
}
_CURRY_READ_TOOLS = {
"curry_get_constant", "curry_get_constant_latest", "curry_list_constants",
"curry_get_function", "curry_list_functions",
}
def test_all_curry_tools_registered():
for name in _CURRY_ACTION_TOOLS | _CURRY_READ_TOOLS:
assert name in tools.REGISTRY
def test_curry_write_and_execute_tools_are_gated_actions():
for name in _CURRY_ACTION_TOOLS:
assert tools.is_action(name), name
assert name in tools.ALWAYS_ASK_ACTION_TOOLS, name
def test_curry_read_tools_are_not_actions():
for name in _CURRY_READ_TOOLS:
assert not tools.is_action(name), name
# ---------------------------------------------------------------------------
# End-to-end HTTP: direct dispatch, no model call, no approval round-trip
# ---------------------------------------------------------------------------
@pytest.fixture
def client():
return TestClient(app)
def _sse_events(body: str) -> list[tuple[str, str]]:
events = []
event_type = "message"
for block in body.split("\n\n"):
for line in block.splitlines():
if line.startswith("event: "):
event_type = line[len("event: "):].strip()
elif line.startswith("data: "):
events.append((event_type, line[len("data: "):]))
event_type = "message"
return events
def test_slash_command_dispatches_without_model_call(client, monkeypatch):
from synapse import chat as chatmod
async def _boom(*a, **k):
raise AssertionError("the model must not be called for a slash-command")
monkeypatch.setattr(chatmod, "stream_chat_response", _boom)
resp = client.post("/chat/stream", json={
"message": '/curry_list_functions()',
"conversation_id": "test-slash-http-1",
})
events = _sse_events(resp.text)
assert ("status", json.dumps({"tool": "curry_list_functions"})) in events
assert any(t == "done" for t, _ in events)
def test_slash_command_skips_approval_round_trip(client, monkeypatch):
async def _fake_dispatch(name, args):
return json.dumps({"ok": True, "result": "did it"})
monkeypatch.setattr(tools, "dispatch", _fake_dispatch)
resp = client.post("/chat/stream", json={
"message": '/curry_call_function(name="x", version=1, args={})',
"conversation_id": "test-slash-http-2",
})
events = _sse_events(resp.text)
assert not any(t == "tool_request" for t, _ in events)
assert any(t == "done" for t, _ in events)
def test_slash_command_uses_fence_from_result_when_present(client, monkeypatch):
async def _fake_dispatch(name, args):
return json.dumps({"ok": True, "fence": "```nexus-curry\n{\"kind\": \"x\"}\n```"})
monkeypatch.setattr(tools, "dispatch", _fake_dispatch)
resp = client.post("/chat/stream", json={
"message": '/curry_call_function(name="x", version=1, args={})',
"conversation_id": "test-slash-http-3",
})
events = _sse_events(resp.text)
content = [d for t, d in events if t == "message"]
assert content and "nexus-curry" in content[0]
def test_slash_command_unknown_tool_yields_error_not_a_chat_reply(client):
resp = client.post("/chat/stream", json={
"message": "/not_a_real_tool(a=1)",
"conversation_id": "test-slash-http-4",
})
events = _sse_events(resp.text)
assert any(t == "error" for t, _ in events)
assert not any(t == "status" for t, _ in events)
def test_slash_command_malformed_yields_error(client):
resp = client.post("/chat/stream", json={
"message": "/curry_call_function(x=some_name)",
"conversation_id": "test-slash-http-5",
})
events = _sse_events(resp.text)
assert any(t == "error" for t, _ in events)
def test_message_with_leading_slash_but_not_command_shaped_goes_to_chat(client, monkeypatch):
# e.g. "/model gpt" or plain prose starting with "/" - must still reach
# the normal model path, not be swallowed as a broken slash-command.
called = {}
async def _fake_stream(*a, **k):
called["hit"] = True
return
yield # pragma: no cover - make this an async generator
# main.py did `from .chat import stream_chat_response`, a separate name
# binding from chat.stream_chat_response - patch the one main.py actually
# calls.
from synapse import main as mainmod
monkeypatch.setattr(mainmod, "stream_chat_response", _fake_stream)
client.post("/chat/stream", json={
"message": "/model gpt",
"conversation_id": "test-slash-http-6",
})
assert called.get("hit") is True
+83
View File
@@ -301,3 +301,86 @@ def test_interrupt_cancels_silent_stream_and_accepts_next_message(monkeypatch):
def test_escape_round_trip_helper():
assert "[" in _escape("x[y]") or "\\[" in _escape("x[y]")
def test_slash_tool_call_shape_forwards_to_start_chat(monkeypatch):
"""/tool_name(arg=val) isn't a local meta-command — it must reach the
backend (synapse/slash_commands.py + chat_stream_endpoint dispatch it),
not fall into the generic 'unknown command' branch."""
pytest.importorskip("textual")
from nexusos_cli.tui_app import NexusTUI
app = NexusTUI.build_app(api_url="http://127.0.0.1:9")
calls: list[str] = []
monkeypatch.setattr(app, "_start_chat", lambda text: calls.append(text))
async def _run():
async with app.run_test():
text = '/curry_call_function(name="double", version=1, args={"x": 21})'
app._handle_slash(text)
assert calls == [text]
log = app.query_one("#log")
assert not any("unknown command" in line.text for line in log.lines)
asyncio.run(_run())
def test_slash_malformed_tool_call_still_forwards_for_the_backend_error(monkeypatch):
"""Even a malformed /tool(...) is forwarded rather than swallowed locally
the backend's parser gives a clearer, more specific error than the
TUI's generic 'unknown command' would."""
pytest.importorskip("textual")
from nexusos_cli.tui_app import NexusTUI
app = NexusTUI.build_app(api_url="http://127.0.0.1:9")
calls: list[str] = []
monkeypatch.setattr(app, "_start_chat", lambda text: calls.append(text))
async def _run():
async with app.run_test():
text = "/curry_call_function(x=__import__('os'))"
app._handle_slash(text)
assert calls == [text]
asyncio.run(_run())
def test_slash_local_meta_commands_still_handled_locally(monkeypatch):
"""A known local command must still be handled in-TUI, never forwarded —
the new tool-call passthrough is strictly the fallback branch."""
pytest.importorskip("textual")
from nexusos_cli.tui_app import NexusTUI
app = NexusTUI.build_app(api_url="http://127.0.0.1:9")
calls: list[str] = []
monkeypatch.setattr(app, "_start_chat", lambda text: calls.append(text))
async def _run():
async with app.run_test():
app._handle_slash("/help")
assert calls == []
log = app.query_one("#log")
assert any("this list" in line.text for line in log.lines)
asyncio.run(_run())
def test_slash_unknown_bare_command_still_rejected(monkeypatch):
"""A genuinely unknown command (no parens, not a local command) keeps the
existing 'unknown command' behavior rather than silently forwarding
anything that starts with /."""
pytest.importorskip("textual")
from nexusos_cli.tui_app import NexusTUI
app = NexusTUI.build_app(api_url="http://127.0.0.1:9")
calls: list[str] = []
monkeypatch.setattr(app, "_start_chat", lambda text: calls.append(text))
async def _run():
async with app.run_test():
app._handle_slash("/frobnicate")
assert calls == []
log = app.query_one("#log")
assert any("unknown command" in line.text for line in log.lines)
asyncio.run(_run())