npm - flowent - Versions diffs - 0.0.6 → 0.0.7 - Mend

flowent 0.0.6 → 0.0.7

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (265) hide show

package/backend/src/flowent/static/assets/useAppRoute-FgSHBKhV.js ADDED Viewed

@@ -0,0 +1 @@

+ import{a as e}from"./rolldown-runtime-BYbx6iT9.js";import{d as t}from"./graph-vendor-DRq_-6fV.js";import{D as n,b as r}from"./index-C9HuekJm.js";var i=e(t(),1);function a(){return window.location.pathname}function o(e){return window.addEventListener(`popstate`,e),window.addEventListener(r,e),()=>{window.removeEventListener(`popstate`,e),window.removeEventListener(r,e)}}function s(){return(0,i.useSyncExternalStore)(o,a,a),n(window.location)}export{s as t};

package/backend/src/flowent/static/index.html CHANGED Viewed

@@ -21,13 +21,13 @@
         }
       })();
     </script>
-    <script type="module" crossorigin src="/assets/index-Biio-CoI.js"></script>
+    <script type="module" crossorigin src="/assets/index-C9HuekJm.js"></script>
     <link rel="modulepreload" crossorigin href="/assets/rolldown-runtime-BYbx6iT9.js">
     <link rel="modulepreload" crossorigin href="/assets/graph-vendor-DRq_-6fV.js">
     <link rel="modulepreload" crossorigin href="/assets/react-vendor-mEs_JJxa.js">
-    <link rel="modulepreload" crossorigin href="/assets/ui-vendor-UazN8rcv.js">
+    <link rel="modulepreload" crossorigin href="/assets/ui-vendor-C5pJa8N7.js">
     <link rel="stylesheet" crossorigin href="/assets/graph-vendor-CHpVij2M.css">
-    <link rel="stylesheet" crossorigin href="/assets/index-CmQvO7sl.css">
+    <link rel="stylesheet" crossorigin href="/assets/index--o_0fv0N.css">
   </head>
   <body class="min-h-dvh bg-background text-foreground">
     <div id="root"></div>

package/backend/src/flowent/tools/__init__.py CHANGED Viewed

@@ -16,14 +16,6 @@ MINIMUM_TOOLS = (
     "send",
 )
-MCP_BROWSING_TOOLS = (
-    "list_mcp_resources",
-    "list_mcp_resource_templates",
-    "read_mcp_resource",
-    "list_mcp_prompts",
-    "get_mcp_prompt",
-)
 ASSISTANT_ONLY_TOOLS = frozenset(
     {
         "create_workflow",
@@ -36,49 +28,11 @@ ASSISTANT_ONLY_TOOLS = frozenset(
     }
 )
-ASSISTANT_ONLY_MCP_TOOL_NAMES = frozenset(
-    {
-        "create_workflow",
-        "create_workflows",
-        "delete_workflow",
-        "delete_workflows",
-        "duplicate_workflow",
-        "duplicate_workflows",
-        "copy_workflow",
-        "copy_workflows",
-        "list_workflow",
-        "list_workflows",
-        "select_workflow",
-        "select_workflows",
-        "switch_workflow",
-        "switch_workflows",
-        "manage_provider",
-        "manage_providers",
-        "manage_role",
-        "manage_roles",
-        "manage_setting",
-        "manage_settings",
-        "manage_prompt",
-        "manage_prompts",
-    }
-)
 def is_assistant_only_tool_name(tool_name: str) -> bool:
     return tool_name in ASSISTANT_ONLY_TOOLS
-def is_assistant_only_mcp_tool_name(tool_name: str) -> bool:
-    normalized = tool_name.strip().lower().replace("-", "_")
-    candidate = (
-        normalized.rsplit("__", 1)[-1] if normalized.startswith("mcp__") else normalized
-    )
-    return (
-        normalized in ASSISTANT_ONLY_MCP_TOOL_NAMES
-        or candidate in ASSISTANT_ONLY_MCP_TOOL_NAMES
-    )
 class Tool(ABC):
     name: str
     description: str
@@ -117,36 +71,17 @@ class ToolRegistry:
         self._tools[tool.name] = tool
     def get(self, name: str) -> Tool | None:
-        tool = self._tools.get(name)
-        if tool is not None:
-            return tool
-        from flowent.mcp_service import mcp_service
-        from flowent.tools.mcp import DynamicMCPTool
-        descriptor = mcp_service.get_dynamic_tool_descriptor(name)
-        if descriptor is None:
-            return None
-        return DynamicMCPTool(descriptor=descriptor)
+        return self._tools.get(name)
     def list_tools(self, *, agent_visible_only: bool = False) -> list[Tool]:
         tools = list(self._tools.values())
-        from flowent.mcp_service import mcp_service
-        from flowent.tools.mcp import DynamicMCPTool
-        dynamic_tools = [
-            DynamicMCPTool(descriptor=descriptor)
-            for descriptor in mcp_service.list_discovered_tools()
-        ]
-        tools.extend(dynamic_tools)
         if not agent_visible_only:
             return tools
         return [tool for tool in tools if tool.agent_visible]
     def get_tools_for_agent(self, agent: Agent) -> list[Tool]:
-        from flowent.mcp_service import mcp_service
         from flowent.models import NodeType
         from flowent.settings import find_role, get_settings
-        from flowent.tools.mcp import DynamicMCPTool
         allowed = set(agent.config.tools) | set(MINIMUM_TOOLS)
         if agent.node_type != NodeType.ASSISTANT:
@@ -156,17 +91,7 @@ class ToolRegistry:
         ]
         role = find_role(get_settings(), agent.config.role_name or "")
         excluded = set(role.excluded_tools) if role is not None else set()
-        dynamic_tools = [
-            DynamicMCPTool(descriptor=descriptor)
-            for descriptor in mcp_service.list_agent_dynamic_tools(agent)
-            if descriptor.fully_qualified_id in allowed
-            and descriptor.fully_qualified_id not in excluded
-            and (
-                agent.node_type == NodeType.ASSISTANT
-                or not is_assistant_only_mcp_tool_name(descriptor.tool_name)
-            )
-        ]
-        return [*visible_tools, *dynamic_tools]
+        return [tool for tool in visible_tools if tool.name not in excluded]
     def get_tools_schema(self, agent: Agent) -> list[dict[str, Any]]:
         return [t.to_schema() for t in self.get_tools_for_agent(agent)]
@@ -189,13 +114,6 @@ def build_tool_registry() -> ToolRegistry:
     from flowent.tools.manage_providers import ManageProvidersTool
     from flowent.tools.manage_roles import ManageRolesTool
     from flowent.tools.manage_settings import ManageSettingsTool
-    from flowent.tools.mcp import (
-        GetMCPPromptTool,
-        ListMCPPromptsTool,
-        ListMCPResourcesTool,
-        ListMCPResourceTemplatesTool,
-        ReadMCPResourceTool,
-    )
     from flowent.tools.read import ReadTool
     from flowent.tools.send import SendTool
     from flowent.tools.set_permissions import SetPermissionsTool
@@ -225,11 +143,6 @@ def build_tool_registry() -> ToolRegistry:
         ListRolesTool,
         ListTabsTool,
         ListToolsTool,
-        ListMCPResourcesTool,
-        ListMCPResourceTemplatesTool,
-        ReadMCPResourceTool,
-        ListMCPPromptsTool,
-        GetMCPPromptTool,
     ]:
         reg.register(tool_cls())  # type: ignore[abstract]
     return reg
@@ -240,12 +153,9 @@ def list_agent_visible_tool_descriptors(
     include_assistant_only: bool = True,
 ) -> list[dict[str, Any]]:
     registry = build_tool_registry()
-    from flowent.mcp_service import mcp_service
     descriptors: list[dict[str, Any]] = []
     for tool in registry.list_tools(agent_visible_only=True):
-        if tool.name.startswith("mcp__"):
-            continue
         if not include_assistant_only and is_assistant_only_tool_name(tool.name):
             continue
         schema = tool.to_schema()
@@ -263,13 +173,4 @@ def list_agent_visible_tool_descriptors(
                 "source": "builtin",
             }
         )
-    for descriptor in mcp_service.list_discovered_tool_descriptors():
-        tool_name = descriptor.get("tool_name")
-        if (
-            not include_assistant_only
-            and isinstance(tool_name, str)
-            and is_assistant_only_mcp_tool_name(tool_name)
-        ):
-            continue
-        descriptors.append(descriptor)
     return descriptors

package/backend/src/flowent/tools/__pycache__/__init__.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/connect.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/contacts.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/create_agent.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/create_tab.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/delete_tab.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/edit.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/exec.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/fetch.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/idle.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/list_roles.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/list_tabs.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/list_tools.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/manage_prompts.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/manage_providers.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/manage_roles.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/manage_settings.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/read.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/send.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/set_permissions.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/sleep.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/__pycache__/todo.cpython-313.pyc CHANGED Viewed

Binary file

package/backend/src/flowent/tools/list_roles.py CHANGED Viewed

@@ -24,7 +24,6 @@ class ListRolesTool(Tool):
         from flowent.tools import (
             MINIMUM_TOOLS,
             build_tool_registry,
-            is_assistant_only_mcp_tool_name,
             is_assistant_only_tool_name,
         )
@@ -32,12 +31,6 @@ class ListRolesTool(Tool):
         tool_registry = build_tool_registry()
         all_tool_names: list[str] = []
         for tool in tool_registry.list_tools(agent_visible_only=True):
-            descriptor = getattr(tool, "_descriptor", None)
-            descriptor_tool_name = getattr(descriptor, "tool_name", None)
-            if isinstance(
-                descriptor_tool_name, str
-            ) and is_assistant_only_mcp_tool_name(descriptor_tool_name):
-                continue
             all_tool_names.append(tool.name)
         if agent.node_type != NodeType.ASSISTANT:
             all_tool_names = [
@@ -53,8 +46,8 @@ class ListRolesTool(Tool):
                 builtin_tools = [
                     tool_name
                     for tool_name in builtin_tools
-                    if not is_assistant_only_tool_name(tool_name)
-                    and not is_assistant_only_mcp_tool_name(tool_name)
+                    if not tool_name.startswith("mcp__")
+                    and not is_assistant_only_tool_name(tool_name)
                 ]
             optional_tools = [
                 tool_name

package/backend/src/flowent/tools/manage_settings.py CHANGED Viewed

@@ -1,6 +1,7 @@
 from __future__ import annotations
 import json
+from collections.abc import Callable
 from typing import TYPE_CHECKING, Any, ClassVar
 if TYPE_CHECKING:
@@ -15,6 +16,99 @@ from flowent.settings_management import (
 from flowent.tools import Tool
+def _error(message: str) -> str:
+    return json.dumps({"error": message})
+def _validate_optional_string(value: object, field_name: str) -> str | None:
+    if value is not None and not isinstance(value, str):
+        return f"{field_name} must be a string"
+    return None
+def _validate_manage_settings_args(args: dict[str, Any]) -> str | None:
+    from flowent.settings import (
+        build_model_auto_compact_token_limit,
+        build_model_context_window_tokens,
+        build_model_input_image,
+        build_model_max_retries,
+        build_model_output_image,
+        build_model_retry_backoff_cap_retries,
+        build_model_retry_initial_delay_seconds,
+        build_model_retry_max_delay_seconds,
+        build_model_retry_policy,
+        build_model_structured_output,
+        build_model_timeout_ms,
+    )
+    action = args.get("action")
+    if not isinstance(action, str):
+        return "action must be a string"
+    for field_name in (
+        "assistant_role_name",
+        "working_dir",
+        "leader_role_name",
+        "active_provider_id",
+        "active_model",
+        "timestamp_format",
+    ):
+        error = _validate_optional_string(args.get(field_name), field_name)
+        if error is not None:
+            return error
+    assistant_allow_network = args.get("assistant_allow_network")
+    if assistant_allow_network is not None and not isinstance(
+        assistant_allow_network, bool
+    ):
+        return "assistant_allow_network must be a boolean"
+    assistant_write_dirs = args.get("assistant_write_dirs")
+    if assistant_write_dirs is not None and not isinstance(assistant_write_dirs, list):
+        return "assistant_write_dirs must be an array of strings"
+    builders: tuple[tuple[str, Callable[..., object]], ...] = (
+        ("retry_policy", build_model_retry_policy),
+        ("timeout_ms", build_model_timeout_ms),
+        ("max_retries", build_model_max_retries),
+        ("retry_initial_delay_seconds", build_model_retry_initial_delay_seconds),
+        ("retry_max_delay_seconds", build_model_retry_max_delay_seconds),
+        ("retry_backoff_cap_retries", build_model_retry_backoff_cap_retries),
+    )
+    for field_name, builder in builders:
+        if args.get(field_name) is None:
+            continue
+        try:
+            builder(args.get(field_name), field_name=field_name)
+        except ValueError as exc:
+            return str(exc)
+    optional_builders: tuple[tuple[str, Callable[..., object]], ...] = (
+        ("input_image", build_model_input_image),
+        ("output_image", build_model_output_image),
+        ("structured_output", build_model_structured_output),
+        ("context_window_tokens", build_model_context_window_tokens),
+        ("auto_compact_token_limit", build_model_auto_compact_token_limit),
+    )
+    for field_name, builder in optional_builders:
+        if field_name not in args:
+            continue
+        try:
+            builder(args.get(field_name), field_name=field_name)
+        except ValueError as exc:
+            return str(exc)
+    model_params = args.get("model_params")
+    if model_params is not None and not isinstance(model_params, (dict, type(None))):
+        return "model_params must be an object or null"
+    return None
+def _provided(args: dict[str, Any], field_name: str) -> object:
+    return args.get(field_name, MISSING)
 class ManageSettingsTool(Tool):
     name = "manage_settings"
     description = (
@@ -133,146 +227,12 @@ class ManageSettingsTool(Tool):
     def execute(self, agent: Agent, args: dict[str, Any], **_kwargs: Any) -> str:
         from flowent.graph_service import sync_assistant_role, sync_tab_leaders
         from flowent.providers.gateway import gateway
-        from flowent.settings import (
-            build_model_auto_compact_token_limit,
-            build_model_context_window_tokens,
-            build_model_input_image,
-            build_model_max_retries,
-            build_model_output_image,
-            build_model_retry_backoff_cap_retries,
-            build_model_retry_initial_delay_seconds,
-            build_model_retry_max_delay_seconds,
-            build_model_retry_policy,
-            build_model_structured_output,
-            build_model_timeout_ms,
-            get_settings,
-            save_settings,
-        )
+        from flowent.settings import get_settings, save_settings
         action = args.get("action")
-        assistant_role_name = args.get("assistant_role_name")
-        assistant_allow_network = args.get("assistant_allow_network")
-        assistant_write_dirs = args.get("assistant_write_dirs")
-        working_dir = args.get("working_dir")
-        leader_role_name = args.get("leader_role_name")
-        active_provider_id = args.get("active_provider_id")
-        active_model = args.get("active_model")
-        timeout_ms = args.get("timeout_ms")
-        retry_policy = args.get("retry_policy")
-        max_retries = args.get("max_retries")
-        retry_initial_delay_seconds = args.get("retry_initial_delay_seconds")
-        retry_max_delay_seconds = args.get("retry_max_delay_seconds")
-        retry_backoff_cap_retries = args.get("retry_backoff_cap_retries")
-        context_window_tokens = args.get("context_window_tokens")
-        input_image = args.get("input_image")
-        output_image = args.get("output_image")
-        structured_output = args.get("structured_output")
-        auto_compact_token_limit = args.get("auto_compact_token_limit")
-        model_params = args.get("model_params")
-        timestamp_format = args.get("timestamp_format")
-        if not isinstance(action, str):
-            return json.dumps({"error": "action must be a string"})
-        if assistant_role_name is not None and not isinstance(assistant_role_name, str):
-            return json.dumps({"error": "assistant_role_name must be a string"})
-        if assistant_allow_network is not None and not isinstance(
-            assistant_allow_network, bool
-        ):
-            return json.dumps({"error": "assistant_allow_network must be a boolean"})
-        if assistant_write_dirs is not None and not isinstance(
-            assistant_write_dirs, list
-        ):
-            return json.dumps(
-                {"error": "assistant_write_dirs must be an array of strings"}
-            )
-        if working_dir is not None and not isinstance(working_dir, str):
-            return json.dumps({"error": "working_dir must be a string"})
-        if leader_role_name is not None and not isinstance(leader_role_name, str):
-            return json.dumps({"error": "leader_role_name must be a string"})
-        if active_provider_id is not None and not isinstance(active_provider_id, str):
-            return json.dumps({"error": "active_provider_id must be a string"})
-        if active_model is not None and not isinstance(active_model, str):
-            return json.dumps({"error": "active_model must be a string"})
-        if retry_policy is not None:
-            try:
-                build_model_retry_policy(retry_policy, field_name="retry_policy")
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if timeout_ms is not None:
-            try:
-                build_model_timeout_ms(timeout_ms, field_name="timeout_ms")
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if max_retries is not None:
-            try:
-                build_model_max_retries(max_retries, field_name="max_retries")
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if retry_initial_delay_seconds is not None:
-            try:
-                build_model_retry_initial_delay_seconds(
-                    retry_initial_delay_seconds,
-                    field_name="retry_initial_delay_seconds",
-                )
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if retry_max_delay_seconds is not None:
-            try:
-                build_model_retry_max_delay_seconds(
-                    retry_max_delay_seconds,
-                    field_name="retry_max_delay_seconds",
-                )
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if retry_backoff_cap_retries is not None:
-            try:
-                build_model_retry_backoff_cap_retries(
-                    retry_backoff_cap_retries,
-                    field_name="retry_backoff_cap_retries",
-                )
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if "input_image" in args:
-            try:
-                build_model_input_image(input_image, field_name="input_image")
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if "output_image" in args:
-            try:
-                build_model_output_image(output_image, field_name="output_image")
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if "structured_output" in args:
-            try:
-                build_model_structured_output(
-                    structured_output,
-                    field_name="structured_output",
-                )
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if "context_window_tokens" in args:
-            try:
-                build_model_context_window_tokens(
-                    context_window_tokens,
-                    field_name="context_window_tokens",
-                )
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if "auto_compact_token_limit" in args:
-            try:
-                build_model_auto_compact_token_limit(
-                    auto_compact_token_limit,
-                    field_name="auto_compact_token_limit",
-                )
-            except ValueError as exc:
-                return json.dumps({"error": str(exc)})
-        if model_params is not None and not isinstance(
-            model_params, (dict, type(None))
-        ):
-            return json.dumps({"error": "model_params must be an object or null"})
-        if timestamp_format is not None and not isinstance(timestamp_format, str):
-            return json.dumps({"error": "timestamp_format must be a string"})
+        validation_error = _validate_manage_settings_args(args)
+        if validation_error is not None:
+            return _error(validation_error)
         settings = get_settings()
@@ -280,61 +240,63 @@ class ManageSettingsTool(Tool):
             return json.dumps(serialize_manage_settings(settings))
         if action != "update":
-            return json.dumps({"error": f"Unsupported action: {action}"})
+            return _error(f"Unsupported action: {action}")
         try:
             resolved = resolve_settings_update(
                 settings,
-                working_dir=working_dir,
-                assistant_role_name=assistant_role_name,
+                working_dir=args.get("working_dir"),
+                assistant_role_name=args.get("assistant_role_name"),
                 assistant_allow_network=(
-                    assistant_allow_network
-                    if assistant_allow_network is not None
+                    args.get("assistant_allow_network")
+                    if args.get("assistant_allow_network") is not None
                     else MISSING
                 ),
                 assistant_write_dirs=(
-                    assistant_write_dirs
-                    if assistant_write_dirs is not None
+                    args.get("assistant_write_dirs")
+                    if args.get("assistant_write_dirs") is not None
                     else MISSING
                 ),
-                leader_role_name=leader_role_name,
-                active_provider_id=active_provider_id,
-                active_model=active_model,
-                context_window_tokens=(
-                    context_window_tokens
-                    if "context_window_tokens" in args
+                leader_role_name=args.get("leader_role_name"),
+                active_provider_id=args.get("active_provider_id"),
+                active_model=args.get("active_model"),
+                context_window_tokens=_provided(args, "context_window_tokens"),
+                input_image=_provided(args, "input_image"),
+                output_image=_provided(args, "output_image"),
+                structured_output=_provided(args, "structured_output"),
+                max_retries=(
+                    args.get("max_retries")
+                    if args.get("max_retries") is not None
                     else MISSING
                 ),
-                input_image=input_image if "input_image" in args else MISSING,
-                output_image=output_image if "output_image" in args else MISSING,
-                structured_output=(
-                    structured_output if "structured_output" in args else MISSING
+                retry_policy=(
+                    args.get("retry_policy")
+                    if args.get("retry_policy") is not None
+                    else MISSING
+                ),
+                timeout_ms=(
+                    args.get("timeout_ms")
+                    if args.get("timeout_ms") is not None
+                    else MISSING
                 ),
-                max_retries=max_retries if max_retries is not None else MISSING,
-                retry_policy=retry_policy if retry_policy is not None else MISSING,
-                timeout_ms=timeout_ms if timeout_ms is not None else MISSING,
                 retry_initial_delay_seconds=(
-                    retry_initial_delay_seconds
-                    if retry_initial_delay_seconds is not None
+                    args.get("retry_initial_delay_seconds")
+                    if args.get("retry_initial_delay_seconds") is not None
                     else MISSING
                 ),
                 retry_max_delay_seconds=(
-                    retry_max_delay_seconds
-                    if retry_max_delay_seconds is not None
+                    args.get("retry_max_delay_seconds")
+                    if args.get("retry_max_delay_seconds") is not None
                     else MISSING
                 ),
                 retry_backoff_cap_retries=(
-                    retry_backoff_cap_retries
-                    if retry_backoff_cap_retries is not None
-                    else MISSING
-                ),
-                auto_compact_token_limit=(
-                    auto_compact_token_limit
-                    if "auto_compact_token_limit" in args
+                    args.get("retry_backoff_cap_retries")
+                    if args.get("retry_backoff_cap_retries") is not None
                     else MISSING
                 ),
-                model_params=model_params if "model_params" in args else MISSING,
-                timestamp_format=timestamp_format,
+                auto_compact_token_limit=_provided(args, "auto_compact_token_limit"),
+                model_params=_provided(args, "model_params"),
+                timestamp_format=args.get("timestamp_format"),
                 assistant_role_field_name="assistant_role_name",
                 assistant_allow_network_field_name="assistant_allow_network",
                 assistant_write_dirs_field_name="assistant_write_dirs",
@@ -353,7 +315,7 @@ class ManageSettingsTool(Tool):
                 auto_compact_token_limit_field_name="auto_compact_token_limit",
             )
         except ValueError as exc:
-            return json.dumps({"error": str(exc)})
+            return _error(str(exc))
         apply_resolved_settings_update(settings, resolved)