deepflow-openagent-cli 0.1.1__tar.gz → 0.1.4__tar.gz

{deepflow_openagent_cli-0.1.1 → deepflow_openagent_cli-0.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepflow-openagent-cli
-Version: 0.1.1
+Version: 0.1.4
 Summary: Command-line client for OpenAgent OpenAPI (agents, conversations, SSE chat, tools, channels, knowledge bases).
 Author: DeepFlow AI
 Project-URL: Repository, https://github.com/DeepFlowAI/OpenAgent
@@ -67,6 +67,14 @@ deepflow_openagent_cli knowledge list --page 1 --per-page 10
 deepflow_openagent_cli knowledge documents 1
 deepflow_openagent_cli knowledge query-document 6670 --slice-id 512208 --line 1720
 deepflow_openagent_cli chat 1 --message "Hello"
+deepflow_openagent_cli steps 1 42
+deepflow_openagent_cli steps get 1 42 100
+deepflow_openagent_cli steps feedback 1 42 100 --rating like --comment "Helpful answer"
+
+# Python code tool (create → update config → debug execute)
+deepflow_openagent_cli tools create 1 --name order_query --tool-type python_code
+deepflow_openagent_cli tools update 1 2 --json '{"config":{"code":"import openagent\nopenagent.set_output(openagent.args)","parameters_json":"{\"type\":\"object\",\"properties\":{\"order_id\":{\"type\":\"string\"}},\"required\":[\"order_id\"]}"}}'
+deepflow_openagent_cli tools execute 1 2 --json '{"brief":"查订单","order_id":"A1"}'
 ```
 
 ## Development (build wheel/sdist locally)

{deepflow_openagent_cli-0.1.1 → deepflow_openagent_cli-0.1.4}/README.md

@@ -46,6 +46,14 @@ deepflow_openagent_cli knowledge list --page 1 --per-page 10
 deepflow_openagent_cli knowledge documents 1
 deepflow_openagent_cli knowledge query-document 6670 --slice-id 512208 --line 1720
 deepflow_openagent_cli chat 1 --message "Hello"
+deepflow_openagent_cli steps 1 42
+deepflow_openagent_cli steps get 1 42 100
+deepflow_openagent_cli steps feedback 1 42 100 --rating like --comment "Helpful answer"
+
+# Python code tool (create → update config → debug execute)
+deepflow_openagent_cli tools create 1 --name order_query --tool-type python_code
+deepflow_openagent_cli tools update 1 2 --json '{"config":{"code":"import openagent\nopenagent.set_output(openagent.args)","parameters_json":"{\"type\":\"object\",\"properties\":{\"order_id\":{\"type\":\"string\"}},\"required\":[\"order_id\"]}"}}'
+deepflow_openagent_cli tools execute 1 2 --json '{"brief":"查订单","order_id":"A1"}'
 ```
 
 ## Development (build wheel/sdist locally)

{deepflow_openagent_cli-0.1.1 → deepflow_openagent_cli-0.1.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "deepflow-openagent-cli"
-version = "0.1.1"
+version = "0.1.4"
 description = "Command-line client for OpenAgent OpenAPI (agents, conversations, SSE chat, tools, channels, knowledge bases)."
 readme = "README.md"
 requires-python = ">=3.11"

{deepflow_openagent_cli-0.1.1 → deepflow_openagent_cli-0.1.4}/src/deepflow_openagent_cli/__init__.py

@@ -1,3 +1,3 @@
 """DeepFlow OpenAgent CLI — published PyPI package."""
 
-__version__ = "0.1.1"
+__version__ = "0.1.4"

{deepflow_openagent_cli-0.1.1 → deepflow_openagent_cli-0.1.4}/src/deepflow_openagent_cli/cli.py

@@ -241,6 +241,40 @@ ENDPOINTS: tuple[EndpointDoc, ...] = (
         errors=["401 missing or invalid API key", "404 conversation not found"],
         example="deepflow_openagent_cli steps 1 42",
     ),
+    EndpointDoc(
+        key="steps.get",
+        method="GET",
+        path="/api/v1/agents/{agent_id}/conversations/{conversation_id}/steps/{step_id}",
+        scope="chat",
+        summary="Get full execution step detail, including LLM request/response payload fields.",
+        path_params=[
+            _param("agent_id", "integer", True, "Agent ID."),
+            _param("conversation_id", "integer", True, "Conversation ID."),
+            _param("step_id", "integer", True, "Step ID."),
+        ],
+        response="StepDetailResponse with request_messages, request_tools, request_params, response fields, and child tool calls.",
+        errors=["401 missing or invalid API key", "404 step not found"],
+        example="deepflow_openagent_cli steps get 1 42 100",
+    ),
+    EndpointDoc(
+        key="steps.feedback",
+        method="POST",
+        path="/api/v1/agents/{agent_id}/conversations/{conversation_id}/steps/{step_id}/feedback",
+        scope="chat",
+        summary="Submit or overwrite like/dislike feedback for one completed assistant_message step.",
+        path_params=[
+            _param("agent_id", "integer", True, "Agent ID."),
+            _param("conversation_id", "integer", True, "Conversation ID."),
+            _param("step_id", "integer", True, "Assistant message Step ID."),
+        ],
+        body_fields=[
+            _param("rating", "string", True, "Feedback rating: like or dislike."),
+            _param("comment", "string", False, "Optional feedback comment, max 500 characters."),
+        ],
+        response="StepFeedbackResponse with step_id, feedback_rating, feedback_comment, and feedback_updated_at.",
+        errors=["400 invalid step state", "401 missing or invalid API key", "403 scope missing", "404 step not found", "422 validation error"],
+        example="deepflow_openagent_cli steps feedback 1 42 100 --rating like --comment 'Helpful answer'",
+    ),
     EndpointDoc(
         key="agents.list",
         method="GET",
@@ -316,7 +350,7 @@ ENDPOINTS: tuple[EndpointDoc, ...] = (
         summary="Partially update system_prompt, model, selected_tool_ids, context, or pre_recall.",
         path_params=[_param("agent_id", "integer", True, "Agent ID.")],
         body_fields=[
-            _param("system_prompt", "string", False, "System prompt, max 10000 chars."),
+            _param("system_prompt", "string", False, "System prompt, max 30000 chars."),
             _param("model", "object", False, "Model config."),
             _param("selected_tool_ids", "array[integer]", False, "Enabled tool IDs."),
             _param("context", "object", False, "Context policy."),
@@ -345,14 +379,24 @@ ENDPOINTS: tuple[EndpointDoc, ...] = (
         summary="Create a custom tool for an agent.",
         path_params=[_param("agent_id", "integer", True, "Agent ID.")],
         body_fields=[
-            _param("name", "string", True, "Tool name, 1-128 chars."),
+            _param("name", "string", True, "Tool name, 1-128 chars, pattern ^[a-zA-Z0-9_-]+$."),
             _param("description", "string", False, "Tool description."),
             _param("tool_type", "string", True, "search, doc_query, doc_grep, notebook, tool_response_fetch, or python_code."),
-            _param("config", "object", False, "Tool config."),
+            _param(
+                "config",
+                "object",
+                False,
+                "Tool config. python_code uses code (Python source) and parameters_json (JSON Schema text string).",
+            ),
         ],
         response="AgentToolResponse.",
-        errors=["401 missing or invalid API key", "403 scope missing", "404 agent not found", "422 validation error"],
-        example="deepflow_openagent_cli tools create 1 --name Search --tool-type search --json '{\"config\":{\"kb_ids\":[1]}}'",
+        errors=["401 missing or invalid API key", "403 scope missing", "404 agent not found", "400/422 validation error"],
+        example=(
+            "deepflow_openagent_cli tools create 1 --name order_query --tool-type python_code "
+            "--json '{\"config\":{\"code\":\"import openagent\\nopenagent.set_output(openagent.args)\","
+            "\"parameters_json\":\"{\\\"type\\\":\\\"object\\\",\\\"properties\\\":{\\\"order_id\\\":{\\\"type\\\":\\\"string\\\"}},"
+            "\\\"required\\\":[\\\"order_id\\\"]}\"}}'"
+        ),
     ),
     EndpointDoc(
         key="tools.get",
@@ -379,13 +423,22 @@ ENDPOINTS: tuple[EndpointDoc, ...] = (
             _param("tool_id", "integer", True, "Tool ID."),
         ],
         body_fields=[
-            _param("name", "string", False, "Tool name."),
+            _param("name", "string", False, "Tool name, pattern ^[a-zA-Z0-9_-]+$."),
             _param("description", "string", False, "Tool description."),
-            _param("config", "object", False, "Tool config."),
+            _param(
+                "config",
+                "object",
+                False,
+                "Tool config. python_code: {code, parameters_json}; search/doc_*: {kb_ids}; human_handoff: {service_hours_id, route_fields}.",
+            ),
         ],
         response="AgentToolResponse.",
-        errors=["401 missing or invalid API key", "403 scope missing", "404 tool not found", "422 validation error"],
-        example="deepflow_openagent_cli tools update 1 2 --json '{\"config\":{\"kb_ids\":[1,2]}}'",
+        errors=["401 missing or invalid API key", "403 scope missing", "404 tool not found", "400/422 validation error"],
+        example=(
+            "deepflow_openagent_cli tools update 1 2 --json "
+            "'{\"config\":{\"code\":\"import openagent\\nopenagent.set_output(\\\"ok\\\")\","
+            "\"parameters_json\":\"{\\\"type\\\":\\\"object\\\",\\\"properties\\\":{}}\"}}'"
+        ),
     ),
     EndpointDoc(
         key="tools.toggle",
@@ -427,10 +480,12 @@ ENDPOINTS: tuple[EndpointDoc, ...] = (
             _param("tool_id", "integer", True, "Tool ID."),
         ],
         query_params=[_param("conversation_id", "integer", False, "Conversation ID for tool context.")],
-        body_fields=[_param("*", "object", False, "Tool-specific arguments.")],
-        response="ToolExecuteResponse.",
+        body_fields=[
+            _param("*", "object", False, "Tool-specific arguments. python_code example: brief + business params from parameters_json."),
+        ],
+        response="ToolExecuteResponse with tool_name, tool_type, arguments, result, duration_ms.",
         errors=["401 missing or invalid API key", "403 scope missing", "404 tool not found", "422 validation error"],
-        example="deepflow_openagent_cli tools execute 1 2 --json '{\"query\":\"pricing\"}'",
+        example="deepflow_openagent_cli tools execute 1 2 --json '{\"brief\":\"查订单\",\"order_id\":\"A1\"}'",
     ),
     EndpointDoc(
         key="channels.list",
@@ -785,10 +840,39 @@ def handle_chat(args: argparse.Namespace) -> int:
 
 
 def handle_steps(args: argparse.Namespace) -> int:
+    action, values = parse_steps_args(args.steps_args)
+    if action == "get":
+        agent_id, conversation_id, step_id = values
+        return run_request(
+            args,
+            "steps.get",
+            {
+                "agent_id": agent_id,
+                "conversation_id": conversation_id,
+                "step_id": step_id,
+            },
+        )
+    if action == "feedback":
+        agent_id, conversation_id, step_id = values
+        body = merge_payload(args, {"rating": args.rating, "comment": args.comment})
+        if "rating" not in body:
+            raise CliError("Missing feedback rating. Pass --rating like|dislike or include rating in --json.")
+        return run_request(
+            args,
+            "steps.feedback",
+            {
+                "agent_id": agent_id,
+                "conversation_id": conversation_id,
+                "step_id": step_id,
+            },
+            body=body,
+        )
+
+    agent_id, conversation_id = values
     return run_request(
         args,
         "steps.list",
-        {"agent_id": args.agent_id, "conversation_id": args.conversation_id},
+        {"agent_id": agent_id, "conversation_id": conversation_id},
     )
 
 
@@ -967,9 +1051,20 @@ def build_parser() -> argparse.ArgumentParser:
     add_json_args(chat)
     chat.set_defaults(handler=handle_chat)
 
-    steps = subparsers.add_parser("steps", help="List conversation execution steps.")
-    steps.add_argument("agent_id", type=int)
-    steps.add_argument("conversation_id", type=int)
+    steps = subparsers.add_parser(
+        "steps",
+        help="List, inspect, or submit feedback for conversation execution steps.",
+        description=(
+            "List conversation execution steps, get full step detail, or submit feedback. "
+            "Forms: steps <agent_id> <conversation_id>; "
+            "steps get <agent_id> <conversation_id> <step_id>; "
+            "steps feedback <agent_id> <conversation_id> <step_id> --rating like."
+        ),
+    )
+    steps.add_argument("steps_args", nargs="+", metavar="ARGS")
+    steps.add_argument("--rating", choices=["like", "dislike"])
+    steps.add_argument("--comment")
+    add_json_args(steps)
     steps.set_defaults(handler=handle_steps)
 
     agents = subparsers.add_parser("agents", help="Agent endpoints.")
@@ -1063,6 +1158,31 @@ def parse_bool(value: str) -> bool:
     raise argparse.ArgumentTypeError("Expected boolean: true/false")
 
 
+def parse_steps_args(values: list[str]) -> tuple[str, tuple[int, ...]]:
+    action = "list"
+    raw_values = values
+    if values[0] in {"list", "get", "detail", "feedback"}:
+        action = "get" if values[0] == "detail" else values[0]
+        raw_values = values[1:]
+    elif len(values) == 3:
+        action = "get"
+
+    expected = 3 if action in {"get", "feedback"} else 2
+    if len(raw_values) != expected:
+        raise CliError(
+            "Usage: steps <agent_id> <conversation_id> or "
+            "steps get <agent_id> <conversation_id> <step_id> or "
+            "steps feedback <agent_id> <conversation_id> <step_id> --rating like|dislike."
+        )
+
+    try:
+        parsed = tuple(int(value) for value in raw_values)
+    except ValueError as exc:
+        raise CliError("steps arguments must be integers.") from exc
+
+    return action, parsed
+
+
 def main(argv: list[str] | None = None) -> int:
     parser = build_parser()
     args = parser.parse_args(argv)

{deepflow_openagent_cli-0.1.1 → deepflow_openagent_cli-0.1.4}/src/deepflow_openagent_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepflow-openagent-cli
-Version: 0.1.1
+Version: 0.1.4
 Summary: Command-line client for OpenAgent OpenAPI (agents, conversations, SSE chat, tools, channels, knowledge bases).
 Author: DeepFlow AI
 Project-URL: Repository, https://github.com/DeepFlowAI/OpenAgent
@@ -67,6 +67,14 @@ deepflow_openagent_cli knowledge list --page 1 --per-page 10
 deepflow_openagent_cli knowledge documents 1
 deepflow_openagent_cli knowledge query-document 6670 --slice-id 512208 --line 1720
 deepflow_openagent_cli chat 1 --message "Hello"
+deepflow_openagent_cli steps 1 42
+deepflow_openagent_cli steps get 1 42 100
+deepflow_openagent_cli steps feedback 1 42 100 --rating like --comment "Helpful answer"
+
+# Python code tool (create → update config → debug execute)
+deepflow_openagent_cli tools create 1 --name order_query --tool-type python_code
+deepflow_openagent_cli tools update 1 2 --json '{"config":{"code":"import openagent\nopenagent.set_output(openagent.args)","parameters_json":"{\"type\":\"object\",\"properties\":{\"order_id\":{\"type\":\"string\"}},\"required\":[\"order_id\"]}"}}'
+deepflow_openagent_cli tools execute 1 2 --json '{"brief":"查订单","order_id":"A1"}'
 ```
 
 ## Development (build wheel/sdist locally)

deepflow_openagent_cli-0.1.4/tests/test_cli.py

@@ -0,0 +1,354 @@
+import json
+from pathlib import Path
+import tomllib
+
+import pytest
+
+from deepflow_openagent_cli import __version__
+from deepflow_openagent_cli.cli import (
+    CliError,
+    ENDPOINTS,
+    OpenApiClient,
+    build_parser,
+    client_from_args,
+    get_endpoint,
+    load_json_payload,
+    main,
+)
+
+
+def test_version_matches_release() -> None:
+    pyproject = Path(__file__).resolve().parents[1] / "pyproject.toml"
+    metadata = tomllib.loads(pyproject.read_text(encoding="utf-8"))
+
+    assert __version__ == metadata["project"]["version"]
+
+
+def test_catalog_has_stable_unique_keys() -> None:
+    keys = [endpoint.key for endpoint in ENDPOINTS]
+
+    assert len(keys) == len(set(keys))
+    assert "knowledge.list" in keys
+    assert "knowledge.documents" in keys
+    assert "documents.query" in keys
+    assert "chat.stream" in keys
+    assert "steps.get" in keys
+    assert "channels.secret_key" in keys
+
+
+def test_get_endpoint_unknown_key_raises_clear_error() -> None:
+    with pytest.raises(CliError, match="Unknown endpoint"):
+        get_endpoint("missing.endpoint")
+
+
+def test_load_json_payload_rejects_non_object() -> None:
+    with pytest.raises(CliError, match="must be an object"):
+        load_json_payload("[1, 2, 3]", None)
+
+
+def test_build_url_formats_path_and_query() -> None:
+    client = OpenApiClient("http://localhost:5001/", "sk-test")
+    endpoint = get_endpoint("knowledge.documents")
+
+    url = client._build_url(
+        endpoint,
+        {"kb_id": 1},
+        {"page": 2, "per_page": 20, "source": None},
+    )
+
+    assert url == "http://localhost:5001/api/v1/knowledge-bases/1/documents?page=2&per_page=20"
+
+
+def test_client_from_args_reads_openagent_env(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setenv("OPENAGENT_HOST", "https://api.example.com")
+    monkeypatch.setenv("OPENAGENT_API_KEY", "sk-openagent")
+    args = build_parser().parse_args(["docs", "list"])
+
+    client = client_from_args(args)
+
+    assert client.host == "https://api.example.com"
+    assert client.api_key == "sk-openagent"
+
+
+def test_client_from_args_reads_legacy_newagent_env(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.delenv("OPENAGENT_HOST", raising=False)
+    monkeypatch.delenv("OPENAGENT_API_KEY", raising=False)
+    monkeypatch.setenv("NEWAGENT_HOST", "https://legacy.example.com")
+    monkeypatch.setenv("NEWAGENT_API_KEY", "sk-legacy")
+    args = build_parser().parse_args(["docs", "list"])
+
+    client = client_from_args(args)
+
+    assert client.host == "https://legacy.example.com"
+    assert client.api_key == "sk-legacy"
+
+
+def test_docs_show_json_outputs_endpoint_detail(capsys: pytest.CaptureFixture[str]) -> None:
+    exit_code = main(["docs", "show", "documents.query", "--json"])
+
+    captured = capsys.readouterr()
+    assert exit_code == 0
+    payload = json.loads(captured.out)
+    assert payload["method"] == "POST"
+    assert payload["scope"] == "chat"
+
+
+def test_query_document_command_posts_body(
+    monkeypatch: pytest.MonkeyPatch,
+    capsys: pytest.CaptureFixture[str],
+) -> None:
+    calls = []
+
+    def fake_request(self, endpoint_key, path_params=None, query=None, body=None, *, stream=False):
+        calls.append((endpoint_key, path_params, query, body, stream))
+        return b'{"ok": true}'
+
+    monkeypatch.setattr(OpenApiClient, "request", fake_request)
+
+    exit_code = main(
+        [
+            "--host",
+            "http://localhost:5001",
+            "--api-key",
+            "sk-test",
+            "knowledge",
+            "query-document",
+            "6670",
+            "--slice-id",
+            "512208",
+            "--line",
+            "1720",
+        ]
+    )
+
+    captured = capsys.readouterr()
+    assert exit_code == 0
+    assert json.loads(captured.out) == {"ok": True}
+    assert calls == [
+        (
+            "documents.query",
+            None,
+            None,
+            {"doc_id": 6670, "slice_id": 512208, "line": 1720},
+            False,
+        )
+    ]
+
+
+def test_steps_list_keeps_legacy_short_form(
+    monkeypatch: pytest.MonkeyPatch,
+    capsys: pytest.CaptureFixture[str],
+) -> None:
+    calls = []
+
+    def fake_request(self, endpoint_key, path_params=None, query=None, body=None, *, stream=False):
+        calls.append((endpoint_key, path_params, query, body, stream))
+        return b'{"ok": true}'
+
+    monkeypatch.setattr(OpenApiClient, "request", fake_request)
+
+    exit_code = main(
+        [
+            "--host",
+            "http://localhost:5001",
+            "--api-key",
+            "sk-test",
+            "steps",
+            "1",
+            "42",
+        ]
+    )
+
+    captured = capsys.readouterr()
+    assert exit_code == 0
+    assert json.loads(captured.out) == {"ok": True}
+    assert calls == [
+        (
+            "steps.list",
+            {"agent_id": 1, "conversation_id": 42},
+            None,
+            None,
+            False,
+        )
+    ]
+
+
+def test_steps_get_fetches_full_step_detail(
+    monkeypatch: pytest.MonkeyPatch,
+    capsys: pytest.CaptureFixture[str],
+) -> None:
+    calls = []
+
+    def fake_request(self, endpoint_key, path_params=None, query=None, body=None, *, stream=False):
+        calls.append((endpoint_key, path_params, query, body, stream))
+        return b'{"ok": true}'
+
+    monkeypatch.setattr(OpenApiClient, "request", fake_request)
+
+    exit_code = main(
+        [
+            "--host",
+            "http://localhost:5001",
+            "--api-key",
+            "sk-test",
+            "steps",
+            "get",
+            "1",
+            "42",
+            "100",
+        ]
+    )
+
+    captured = capsys.readouterr()
+    assert exit_code == 0
+    assert json.loads(captured.out) == {"ok": True}
+    assert calls == [
+        (
+            "steps.get",
+            {"agent_id": 1, "conversation_id": 42, "step_id": 100},
+            None,
+            None,
+            False,
+        )
+    ]
+
+
+def test_tools_create_accepts_doc_grep(
+    monkeypatch: pytest.MonkeyPatch,
+    capsys: pytest.CaptureFixture[str],
+) -> None:
+    calls = []
+
+    def fake_request(self, endpoint_key, path_params=None, query=None, body=None, *, stream=False):
+        calls.append((endpoint_key, path_params, query, body, stream))
+        return b'{"ok": true}'
+
+    monkeypatch.setattr(OpenApiClient, "request", fake_request)
+
+    exit_code = main(
+        [
+            "--host",
+            "http://localhost:5001",
+            "--api-key",
+            "sk-test",
+            "tools",
+            "create",
+            "1",
+            "--name",
+            "Doc Grep",
+            "--tool-type",
+            "doc_grep",
+        ]
+    )
+
+    captured = capsys.readouterr()
+    assert exit_code == 0
+    assert json.loads(captured.out) == {"ok": True}
+    assert calls[0][0] == "tools.create"
+    assert calls[0][1] == {"agent_id": 1}
+    assert calls[0][3]["tool_type"] == "doc_grep"
+
+
+def test_tools_create_accepts_python_code(
+    monkeypatch: pytest.MonkeyPatch,
+    capsys: pytest.CaptureFixture[str],
+) -> None:
+    calls = []
+
+    def fake_request(self, endpoint_key, path_params=None, query=None, body=None, *, stream=False):
+        calls.append((endpoint_key, path_params, query, body, stream))
+        return b'{"ok": true}'
+
+    monkeypatch.setattr(OpenApiClient, "request", fake_request)
+
+    config = {
+        "code": "import openagent\nopenagent.set_output(openagent.args)",
+        "parameters_json": '{"type":"object","properties":{"order_id":{"type":"string"}},"required":["order_id"]}',
+    }
+    exit_code = main(
+        [
+            "--host",
+            "http://localhost:5001",
+            "--api-key",
+            "sk-test",
+            "tools",
+            "create",
+            "1",
+            "--name",
+            "order_query",
+            "--tool-type",
+            "python_code",
+            "--json",
+            json.dumps({"config": config}),
+        ]
+    )
+
+    captured = capsys.readouterr()
+    assert exit_code == 0
+    assert json.loads(captured.out) == {"ok": True}
+    assert calls[0][0] == "tools.create"
+    assert calls[0][1] == {"agent_id": 1}
+    assert calls[0][3]["tool_type"] == "python_code"
+    assert calls[0][3]["name"] == "order_query"
+    assert calls[0][3]["config"] == config
+
+
+def test_tools_update_and_execute_python_code(
+    monkeypatch: pytest.MonkeyPatch,
+    capsys: pytest.CaptureFixture[str],
+) -> None:
+    calls = []
+
+    def fake_request(self, endpoint_key, path_params=None, query=None, body=None, *, stream=False):
+        calls.append((endpoint_key, path_params, query, body, stream))
+        return b'{"ok": true}'
+
+    monkeypatch.setattr(OpenApiClient, "request", fake_request)
+
+    update_body = {
+        "config": {
+            "code": "import openagent\nopenagent.set_output('ok')",
+            "parameters_json": '{"type":"object","properties":{}}',
+        }
+    }
+    exit_code = main(
+        [
+            "--host",
+            "http://localhost:5001",
+            "--api-key",
+            "sk-test",
+            "tools",
+            "update",
+            "1",
+            "2",
+            "--json",
+            json.dumps(update_body),
+        ]
+    )
+    captured = capsys.readouterr()
+    assert exit_code == 0
+    assert json.loads(captured.out) == {"ok": True}
+    assert calls[0][0] == "tools.update"
+    assert calls[0][1] == {"agent_id": 1, "tool_id": 2}
+    assert calls[0][3] == update_body
+
+    exit_code = main(
+        [
+            "--host",
+            "http://localhost:5001",
+            "--api-key",
+            "sk-test",
+            "tools",
+            "execute",
+            "1",
+            "2",
+            "--json",
+            '{"brief":"查订单","order_id":"A1"}',
+        ]
+    )
+    captured = capsys.readouterr()
+    assert exit_code == 0
+    assert json.loads(captured.out) == {"ok": True}
+    assert calls[1][0] == "tools.execute"
+    assert calls[1][1] == {"agent_id": 1, "tool_id": 2}
+    assert calls[1][3] == {"brief": "查订单", "order_id": "A1"}

deepflow_openagent_cli-0.1.1/tests/test_cli.py

@@ -1,165 +0,0 @@
-import json
-
-import pytest
-
-from deepflow_openagent_cli import __version__
-from deepflow_openagent_cli.cli import (
-    CliError,
-    ENDPOINTS,
-    OpenApiClient,
-    build_parser,
-    client_from_args,
-    get_endpoint,
-    load_json_payload,
-    main,
-)
-
-
-def test_version_matches_release() -> None:
-    assert __version__ == "0.1.1"
-
-
-def test_catalog_has_stable_unique_keys() -> None:
-    keys = [endpoint.key for endpoint in ENDPOINTS]
-
-    assert len(keys) == len(set(keys))
-    assert "knowledge.list" in keys
-    assert "knowledge.documents" in keys
-    assert "documents.query" in keys
-    assert "chat.stream" in keys
-    assert "channels.secret_key" in keys
-
-
-def test_get_endpoint_unknown_key_raises_clear_error() -> None:
-    with pytest.raises(CliError, match="Unknown endpoint"):
-        get_endpoint("missing.endpoint")
-
-
-def test_load_json_payload_rejects_non_object() -> None:
-    with pytest.raises(CliError, match="must be an object"):
-        load_json_payload("[1, 2, 3]", None)
-
-
-def test_build_url_formats_path_and_query() -> None:
-    client = OpenApiClient("http://localhost:5001/", "sk-test")
-    endpoint = get_endpoint("knowledge.documents")
-
-    url = client._build_url(
-        endpoint,
-        {"kb_id": 1},
-        {"page": 2, "per_page": 20, "source": None},
-    )
-
-    assert url == "http://localhost:5001/api/v1/knowledge-bases/1/documents?page=2&per_page=20"
-
-
-def test_client_from_args_reads_openagent_env(monkeypatch: pytest.MonkeyPatch) -> None:
-    monkeypatch.setenv("OPENAGENT_HOST", "https://api.example.com")
-    monkeypatch.setenv("OPENAGENT_API_KEY", "sk-openagent")
-    args = build_parser().parse_args(["docs", "list"])
-
-    client = client_from_args(args)
-
-    assert client.host == "https://api.example.com"
-    assert client.api_key == "sk-openagent"
-
-
-def test_client_from_args_reads_legacy_newagent_env(monkeypatch: pytest.MonkeyPatch) -> None:
-    monkeypatch.delenv("OPENAGENT_HOST", raising=False)
-    monkeypatch.delenv("OPENAGENT_API_KEY", raising=False)
-    monkeypatch.setenv("NEWAGENT_HOST", "https://legacy.example.com")
-    monkeypatch.setenv("NEWAGENT_API_KEY", "sk-legacy")
-    args = build_parser().parse_args(["docs", "list"])
-
-    client = client_from_args(args)
-
-    assert client.host == "https://legacy.example.com"
-    assert client.api_key == "sk-legacy"
-
-
-def test_docs_show_json_outputs_endpoint_detail(capsys: pytest.CaptureFixture[str]) -> None:
-    exit_code = main(["docs", "show", "documents.query", "--json"])
-
-    captured = capsys.readouterr()
-    assert exit_code == 0
-    payload = json.loads(captured.out)
-    assert payload["method"] == "POST"
-    assert payload["scope"] == "chat"
-
-
-def test_query_document_command_posts_body(
-    monkeypatch: pytest.MonkeyPatch,
-    capsys: pytest.CaptureFixture[str],
-) -> None:
-    calls = []
-
-    def fake_request(self, endpoint_key, path_params=None, query=None, body=None, *, stream=False):
-        calls.append((endpoint_key, path_params, query, body, stream))
-        return b'{"ok": true}'
-
-    monkeypatch.setattr(OpenApiClient, "request", fake_request)
-
-    exit_code = main(
-        [
-            "--host",
-            "http://localhost:5001",
-            "--api-key",
-            "sk-test",
-            "knowledge",
-            "query-document",
-            "6670",
-            "--slice-id",
-            "512208",
-            "--line",
-            "1720",
-        ]
-    )
-
-    captured = capsys.readouterr()
-    assert exit_code == 0
-    assert json.loads(captured.out) == {"ok": True}
-    assert calls == [
-        (
-            "documents.query",
-            None,
-            None,
-            {"doc_id": 6670, "slice_id": 512208, "line": 1720},
-            False,
-        )
-    ]
-
-
-def test_tools_create_accepts_doc_grep(
-    monkeypatch: pytest.MonkeyPatch,
-    capsys: pytest.CaptureFixture[str],
-) -> None:
-    calls = []
-
-    def fake_request(self, endpoint_key, path_params=None, query=None, body=None, *, stream=False):
-        calls.append((endpoint_key, path_params, query, body, stream))
-        return b'{"ok": true}'
-
-    monkeypatch.setattr(OpenApiClient, "request", fake_request)
-
-    exit_code = main(
-        [
-            "--host",
-            "http://localhost:5001",
-            "--api-key",
-            "sk-test",
-            "tools",
-            "create",
-            "1",
-            "--name",
-            "Doc Grep",
-            "--tool-type",
-            "doc_grep",
-        ]
-    )
-
-    captured = capsys.readouterr()
-    assert exit_code == 0
-    assert json.loads(captured.out) == {"ok": True}
-    assert calls[0][0] == "tools.create"
-    assert calls[0][1] == {"agent_id": 1}
-    assert calls[0][3]["tool_type"] == "doc_grep"