Fix tests

Author: ericdallo

src/eca/features/tools.clj

@@ -113,13 +113,11 @@
                                                                         (update :tools #(mapv with-tool-status %)))))})))
 
 (defn manual-approval? [name config]
-  (let [manual-approval? (get-in config [:toolCall :manualApproval] nil)]
-    (logger/info manual-approval?
-                 (coll? manual-approval?)
-                 name)
-    (if (coll? manual-approval?)
-      (some #(= name (str %)) manual-approval?)
-      manual-approval?)))
+  (boolean
+    (let [manual-approval? (get-in config [:toolCall :manualApproval] nil)]
+      (if (coll? manual-approval?)
+        (some #(= name (str %)) manual-approval?)
+        manual-approval?))))
 
 (defn tool-call-summary [all-tools name args]
   (when-let [summary-fn (:summary-fn (first (filter #(= name (:name %))

test/eca/features/chat_test.clj

@@ -198,8 +198,7 @@
              (h/messages)))))))
 
 (deftest basic-tool-calling-prompt-test
-  (testing "Asking to list directories, LLM will check
-for allowed directories and then list files"
+  (testing "Asking to list directories, LLM will check for allowed directories and then list files"
     (h/reset-components!)
     (let [{:keys [chat-id]}
           (complete!