Disable eca_plan_edit_file in plan behavior until better idea on what plan behavior should do.

Author: ericdallo

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## Unreleased
 
+- Disable `eca_plan_edit_file` in plan behavior until better idea on what plan behavior should do.
+
 ## 0.32.3
 
 - Consider `AGENTS.md` instead of `AGENT.md`, following the https://agents.md standard.

resources/prompts/plan_behavior.txt

@@ -1 +1 @@
-Plan mode is active. The user indicated that they do not want you to execute yet -- you MUST NOT make any edits, run any non-readonly tools (including changing configs or making commits), or otherwise make any changes to the system. This supersedes any other instructions you have received (for example, to make edits). The only allowed tool related to code changes is the `eca_plan_edit_file` since system will take care to present to user the changes, only run if user ask to apply changes.
+Plan mode is active. The user indicated that they do not want you to execute yet -- you MUST NOT make any edits, run any non-readonly tools (including changing configs or making commits), or otherwise make any changes to the system. This supersedes any other instructions you have received (for example, to make edits).

src/eca/features/tools/filesystem.clj

@@ -294,7 +294,8 @@
                                                   :description "Whether to replace all occurences of the file or just the first one (default)"}}
                   :required ["path" "original_content" "new_content"]}
     :handler #'plan-edit-file
-    :enabled-fn (fn [{:keys [behavior]}] (= "plan" behavior))
+    ;; TODO improve plan behavior providing better tool for exit plan and present to user.
+    :enabled-fn (constantly false) #_(fn [{:keys [behavior]}] (= "plan" behavior))
     :summary-fn (constantly "Planning edit")}
    "eca_move_file"
    {:description (str "Move or rename files and directories. Can move files between directories "

test/eca/features/tools_test.clj

@@ -33,7 +33,7 @@
     (is (match?
          (m/embeds [(m/mismatch {:name "eca_directory_tree"})])
          (f.tools/all-tools "agent" {} {:nativeTools {:filesystem {:enabled false}}}))))
-  (testing "Do not include edit tool if plan behavior"
+  #_(testing "Do not include edit tool if plan behavior"
     (is (match?
          (m/embeds [{:name "eca_plan_edit_file"}
                     (m/mismatch {:name "eca_edit_file"})])