Minor docs

Author: ericdallo

docs/configuration.md

@@ -560,11 +560,11 @@ To configure, add your OTLP collector config via `:otlp` map following [otlp aut
       "toolCall": {
         "approval": {
           "byDefault": "ask",
-          "allow": {"directory_tree": {},
-                    "read_file": {},
-                    "grep": {},
-                    "preview_file_change": {},
-                    "editor_diagnostics": {}},
+          "allow": {"eca__directory_tree": {},
+                    "eca__read_file": {},
+                    "eca__grep": {},
+                    "eca__preview_file_change": {},
+                    "eca__editor_diagnostics": {}},
           "ask": {},
           "deny": {}
         },
@@ -583,7 +583,7 @@ To configure, add your OTLP collector config via `:otlp` map following [otlp aut
                   "disabledTools": ["preview_file_change"]},
         "plan": {"systemPromptFile": "prompts/plan_behavior.md",
                   "disabledTools": ["edit_file", "write_file", "move_file"],
-                  "toolCall": {"approval": {"deny": {"shell_command":
+                  "toolCall": {"approval": {"deny": {"eca__shell_command":
                                                      {"argsMatchers": {"command" [".*>.*",
                                                                                   ".*\\|\\s*(tee|dd|xargs).*",
                                                                                   ".*\\b(sed|awk|perl)\\s+.*-i.*",

docs/features.md

@@ -28,6 +28,8 @@ Behavior affect the prompt passed to LLM and the tools to include, ECA allow to
 ECA leverage tools to give more power to the LLM, this is the best way to make LLMs have more context about your codebase and behave like an agent.
 It supports both MCP server tools + ECA native tools.
 
+When passing tools to LLM, ECA will always prefix the tool with the server name, like `eca__directory_tree` or `my-mcp__foo_bar`.
+
 !!! info "Approval / permissions"
 
     By default, ECA ask to approve all non-read only tools or mcp tools, you can easily configure that, check `toolCall approval` [config](./configuration.md) or try the `plan` behavior.

src/eca/config.clj

@@ -84,13 +84,13 @@
                        :disabledTools ["preview_file_change"]}
               "plan" {:systemPromptFile "prompts/plan_behavior.md"
                       :disabledTools ["edit_file" "write_file" "move_file"]
-                      :toolCall {:approval {:allow {"shell_command"
+                      :toolCall {:approval {:allow {"eca__shell_command"
                                                     {:argsMatchers {"command" ["pwd"]}}
-                                                    "preview_file_change" {}
-                                                    "grep" {}
-                                                    "read_file" {}
-                                                    "directory_tree" {}}
-                                            :deny {"shell_command"
+                                                    "eca__preview_file_change" {}
+                                                    "eca__grep" {}
+                                                    "eca__read_file" {}
+                                                    "eca__directory_tree" {}}
+                                            :deny {"eca__shell_command"
                                                    {:argsMatchers {"command" ["[12&]?>>?\\s*(?!/dev/null($|\\s))\\S+"
                                                                               ".*>.*",
                                                                               ".*\\|\\s*(tee|dd|xargs).*",
@@ -101,7 +101,7 @@
                                                                               ".*-c\\s+[\"'].*open.*[\"']w[\"'].*",
                                                                               ".*bash.*-c.*>.*"]}}}}}}}
    :defaultModel nil
-   :hooks {} 
+   :hooks {}
    :rules []
    :commands []
    :disabledTools []
@@ -139,7 +139,6 @@
    :rewrite {:systemPromptFile "prompts/rewrite.md"}
    :netrcFile nil
    :env "prod"})
-   
 
 (def ^:private fallback-behavior "agent")