codeer-cli 0.1.12__tar.gz → 0.1.13__tar.gz

{codeer_cli-0.1.12 → codeer_cli-0.1.13}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.4
+Metadata-Version: 2.5
 Name: codeer-cli
-Version: 0.1.12
+Version: 0.1.13
 Summary: Command line tools for managing Codeer agents over the Codeer API.
 Project-URL: Homepage, https://www.codeer.ai
 Author: Codeer.AI

{codeer_cli-0.1.12 → codeer_cli-0.1.13}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "codeer-cli"
-version = "0.1.12"
+version = "0.1.13"
 description = "Command line tools for managing Codeer agents over the Codeer API."
 readme = "README.md"
 requires-python = ">=3.11"

{codeer_cli-0.1.12 → codeer_cli-0.1.13}/src/codeer_cli/agents.py

@@ -20,6 +20,7 @@ def create(
     unified_tools: Optional[List[dict]] = None,
     use_search: bool = False,
     llm_model: Optional[str] = None,
+    llm_model_settings: Optional[dict[str, dict[str, Any]]] = None,
     description: Optional[str] = None,
     suggested_questions: Optional[List[str]] = None,
     primary_object_ids: Optional[List[int]] = None,
@@ -41,6 +42,8 @@ def create(
         body["description"] = description
     if llm_model is not None:
         body["llm_model"] = llm_model
+    if llm_model_settings is not None:
+        body["llm_model_settings"] = llm_model_settings
     if validated_handoff is not None:
         body["human_handoff"] = validated_handoff
     return client.post("/external/agents", json=body)
@@ -57,6 +60,7 @@ def update(
     version_note: str = "",
     description: Optional[str] = None,
     llm_model: Optional[str] = None,
+    llm_model_settings: Optional[dict[str, dict[str, Any]]] = None,
     suggested_questions: Optional[List[str]] = None,
     primary_object_ids: Optional[List[int]] = None,
     attachment_ids: Optional[List[str]] = None,
@@ -79,6 +83,8 @@ def update(
         body["description"] = description
     if llm_model is not None:
         body["llm_model"] = llm_model
+    if llm_model_settings is not None:
+        body["llm_model_settings"] = llm_model_settings
     if validated_handoff is not None:
         body["human_handoff"] = validated_handoff
     return client.patch(f"/external/agents/{agent_id}", json=body)

{codeer_cli-0.1.12 → codeer_cli-0.1.13}/src/codeer_cli/commands/agent.py

@@ -89,7 +89,9 @@ def register(subparsers):
 def _tool_summary(tools: list[dict] | None) -> list[dict]:
     out = []
     for t in tools or []:
-        form = t.get("custom_form_schema") if isinstance(t.get("custom_form_schema"), dict) else {}
+        form = t.get("custom_form_schema")
+        if not isinstance(form, dict):
+            form = {}
         out.append({
             "id": t.get("id"),
             "type": t.get("type"),
@@ -169,6 +171,8 @@ def run_apply(args, client) -> int:
         log(f"error: invalid agent payload: {exc}")
         return 2
 
+    llm_model_settings_provided = "llm_model_settings" in body
+
     if args.dry_run:
         operation = "update" if args.agent_id else "create"
         result = {
@@ -181,6 +185,8 @@ def run_apply(args, client) -> int:
             "tool_count": len(validated_tools),
             "use_search": body.get("use_search", False),
             "llm_model": body.get("llm_model"),
+            "llm_model_settings_provided": llm_model_settings_provided,
+            "llm_model_settings": body.get("llm_model_settings"),
             "human_handoff": {
                 "enabled": bool((validated_handoff or {}).get("enabled")),
                 "idle_timeout_minutes": (validated_handoff or {}).get("idle_timeout_minutes"),
@@ -193,6 +199,12 @@ def run_apply(args, client) -> int:
         print_json(result)
         return 0
 
+    model_settings_kwargs = (
+        {"llm_model_settings": body["llm_model_settings"]}
+        if llm_model_settings_provided
+        else {}
+    )
+
     if args.agent_id:
         body.pop("workspace_id", None)
         agents_mod.update(
@@ -208,6 +220,7 @@ def run_apply(args, client) -> int:
             primary_object_ids=body.get("primary_object_ids") or [],
             attachment_ids=body.get("attachment_ids") or [],
             human_handoff=validated_handoff,
+            **model_settings_kwargs,
         )
         agent_id = args.agent_id
         log(f"PUT /agents/{agent_id} ok")
@@ -227,6 +240,7 @@ def run_apply(args, client) -> int:
             primary_object_ids=body.get("primary_object_ids") or [],
             attachment_ids=body.get("attachment_ids") or [],
             human_handoff=validated_handoff,
+            **model_settings_kwargs,
         )
         agent_id = agent["id"]
         log(f"POST /agents ok, id={agent_id}")

codeer_cli-0.1.13/tests/test_agent_model_settings.py

@@ -0,0 +1,230 @@
+from __future__ import annotations
+
+import contextlib
+import io
+import json
+import tempfile
+import unittest
+from pathlib import Path
+from types import SimpleNamespace
+from unittest.mock import patch
+
+from codeer_cli import agents
+from codeer_cli.commands import agent as agent_cmd
+
+
+MODEL_ID = "openai/gpt-5.6-luna"
+MODEL_SETTINGS = {MODEL_ID: {"thinking": "xhigh"}}
+
+
+class FakeClient:
+    def __init__(self) -> None:
+        self.calls: list[tuple[str, str, dict]] = []
+
+    def post(self, path: str, **kwargs):
+        self.calls.append(("POST", path, kwargs))
+        return {"id": "agent-1"}
+
+    def patch(self, path: str, **kwargs):
+        self.calls.append(("PATCH", path, kwargs))
+        return {"id": "agent-1"}
+
+
+class AgentModelSettingsPayloadTests(unittest.TestCase):
+    def test_create_forwards_model_settings(self) -> None:
+        client = FakeClient()
+
+        agents.create(
+            client,  # type: ignore[arg-type]
+            workspace_id="workspace-1",
+            name="Agent",
+            system_prompt="Help safely.",
+            llm_model=MODEL_ID,
+            llm_model_settings=MODEL_SETTINGS,
+        )
+
+        self.assertEqual(client.calls[0][0:2], ("POST", "/external/agents"))
+        self.assertEqual(
+            client.calls[0][2]["json"]["llm_model_settings"],
+            MODEL_SETTINGS,
+        )
+
+    def test_update_forwards_model_settings(self) -> None:
+        client = FakeClient()
+
+        agents.update(
+            client,  # type: ignore[arg-type]
+            "agent-1",
+            name="Agent",
+            system_prompt="Help safely.",
+            unified_tools=[],
+            use_search=False,
+            llm_model=MODEL_ID,
+            llm_model_settings=MODEL_SETTINGS,
+        )
+
+        self.assertEqual(
+            client.calls[0][0:2],
+            ("PATCH", "/external/agents/agent-1"),
+        )
+        self.assertEqual(
+            client.calls[0][2]["json"]["llm_model_settings"],
+            MODEL_SETTINGS,
+        )
+
+    def test_update_forwards_empty_map_to_clear_settings(self) -> None:
+        client = FakeClient()
+
+        agents.update(
+            client,  # type: ignore[arg-type]
+            "agent-1",
+            name="Agent",
+            system_prompt="Help safely.",
+            unified_tools=[],
+            use_search=False,
+            llm_model_settings={},
+        )
+
+        self.assertEqual(client.calls[0][2]["json"]["llm_model_settings"], {})
+
+    def test_update_omits_model_settings_when_unspecified(self) -> None:
+        client = FakeClient()
+
+        agents.update(
+            client,  # type: ignore[arg-type]
+            "agent-1",
+            name="Agent",
+            system_prompt="Help safely.",
+            unified_tools=[],
+            use_search=False,
+        )
+
+        self.assertNotIn("llm_model_settings", client.calls[0][2]["json"])
+
+
+class AgentModelSettingsCommandTests(unittest.TestCase):
+    def _run_apply(
+        self,
+        payload: dict,
+        *,
+        agent_id: str | None = None,
+        dry_run: bool = False,
+    ) -> tuple[int, dict]:
+        with tempfile.TemporaryDirectory() as tmpdir:
+            payload_path = Path(tmpdir) / "agent.json"
+            payload_path.write_text(json.dumps(payload))
+            args = SimpleNamespace(
+                payload=str(payload_path),
+                agent_id=agent_id,
+                dry_run=dry_run,
+                note="settings regression",
+                out=None,
+            )
+            stdout = io.StringIO()
+            with contextlib.redirect_stdout(stdout):
+                status = agent_cmd.run_apply(args, object())
+        return status, json.loads(stdout.getvalue()) if stdout.getvalue() else {}
+
+    def test_apply_create_forwards_model_settings(self) -> None:
+        payload = {
+            "workspace_id": "workspace-1",
+            "name": "Agent",
+            "system_prompt": "Help safely.",
+            "llm_model": MODEL_ID,
+            "llm_model_settings": MODEL_SETTINGS,
+        }
+        history = [{"id": "history-1", "version_number": 1, "status": "draft"}]
+
+        with (
+            patch.object(
+                agent_cmd.agents_mod,
+                "create",
+                return_value={"id": "agent-1"},
+            ) as create,
+            patch.object(agent_cmd.agents_mod, "list_versions", return_value=history),
+        ):
+            status, _ = self._run_apply(payload)
+
+        self.assertEqual(status, 0)
+        self.assertEqual(
+            create.call_args.kwargs["llm_model_settings"],
+            MODEL_SETTINGS,
+        )
+
+    def test_apply_update_preserves_present_and_empty_settings(self) -> None:
+        history = [{"id": "history-2", "version_number": 2, "status": "draft"}]
+
+        for model_settings in (MODEL_SETTINGS, {}):
+            with self.subTest(model_settings=model_settings):
+                payload = {
+                    "name": "Agent",
+                    "system_prompt": "Help safely.",
+                    "llm_model_settings": model_settings,
+                }
+                with (
+                    patch.object(agent_cmd.agents_mod, "update") as update,
+                    patch.object(
+                        agent_cmd.agents_mod,
+                        "list_versions",
+                        return_value=history,
+                    ),
+                ):
+                    status, _ = self._run_apply(payload, agent_id="agent-1")
+
+                self.assertEqual(status, 0)
+                self.assertEqual(
+                    update.call_args.kwargs["llm_model_settings"],
+                    model_settings,
+                )
+
+    def test_apply_update_does_not_supply_omitted_settings(self) -> None:
+        payload = {
+            "name": "Agent",
+            "system_prompt": "Help safely.",
+        }
+        history = [{"id": "history-2", "version_number": 2, "status": "draft"}]
+
+        with (
+            patch.object(agent_cmd.agents_mod, "update") as update,
+            patch.object(agent_cmd.agents_mod, "list_versions", return_value=history),
+        ):
+            status, _ = self._run_apply(payload, agent_id="agent-1")
+
+        self.assertEqual(status, 0)
+        self.assertNotIn("llm_model_settings", update.call_args.kwargs)
+
+    def test_apply_dry_run_distinguishes_settings_states(self) -> None:
+        cases = (
+            (MODEL_SETTINGS, True, MODEL_SETTINGS),
+            ({}, True, {}),
+            (None, False, None),
+        )
+
+        for model_settings, expected_provided, expected_settings in cases:
+            with self.subTest(model_settings=model_settings):
+                payload: dict[str, object] = {
+                    "name": "Agent",
+                    "system_prompt": "Help safely.",
+                }
+                if expected_provided:
+                    payload["llm_model_settings"] = model_settings
+
+                status, result = self._run_apply(
+                    payload,
+                    agent_id="agent-1",
+                    dry_run=True,
+                )
+
+                self.assertEqual(status, 0)
+                self.assertEqual(
+                    result["llm_model_settings_provided"],
+                    expected_provided,
+                )
+                self.assertEqual(
+                    result["llm_model_settings"],
+                    expected_settings,
+                )
+
+
+if __name__ == "__main__":
+    unittest.main()

{codeer_cli-0.1.12 → codeer_cli-0.1.13}/uv.lock

@@ -26,7 +26,7 @@ wheels = [
 
 [[package]]
 name = "codeer-cli"
-version = "0.1.12"
+version = "0.1.13"
 source = { editable = "." }
 dependencies = [
     { name = "httpx" },