Fix CI failures on master

Two fixes:
- read-chat missing-path test asserted a unix-style path in the error
  message, failing on Windows where io/file renders backslashes. Build
  the expected string with io/file too.
- Marking MCP servers running right after tools listing introduced an
  unconditional second serverUpdated notification after prompts and
  resources listing, even when the server has none. That extra
  notification broke the mcp-remote list_changed integration test on
  all platforms. Only notify again when prompts or resources arrived.

Author: ericdallo

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## Unreleased
 
+- Skip redundant MCP `tool/serverUpdated` notification after prompts/resources listing when the server has none.
+
 ## 0.154.0
 
 - Improve `/login` UX: markdown links, provider/method selection via `chat/askQuestion` when supported by client, GitHub Copilot login completes automatically, secrets not echoed.

src/eca/features/tools/mcp.clj

@@ -509,9 +509,14 @@
                                      (logger/info logger-tag (format "Started MCP server %s" name))
                                      ;; Fetch after flipping to running: servers may take long or never
                                      ;; answer these list requests, and they should not gate the status.
-                                     (swap! db* assoc-in [:mcp-clients name :prompts] (list-server-prompts client list-timeout-ms))
-                                     (swap! db* assoc-in [:mcp-clients name :resources] (list-server-resources client list-timeout-ms))
-                                     (on-server-updated (->server name server-config :running @db*))
+                                     (let [prompts (list-server-prompts client list-timeout-ms)
+                                           resources (list-server-resources client list-timeout-ms)]
+                                       (swap! db* assoc-in [:mcp-clients name :prompts] prompts)
+                                       (swap! db* assoc-in [:mcp-clients name :resources] resources)
+                                       ;; Notify again only when something actually arrived, avoiding
+                                       ;; a redundant duplicate notification for servers without them.
+                                       (when (or (seq prompts) (seq resources))
+                                         (on-server-updated (->server name server-config :running @db*))))
                                      :ok)))
                              (catch Exception e
                                (try (pp/stop-client-transport! transport false) (catch Exception _))

test/eca/read_chat_test.clj

@@ -421,7 +421,7 @@
                                           (catch clojure.lang.ExceptionInfo e
                                             e))]
       (is e)
-      (is (string/includes? (.getMessage e) "DB cache file not found: /nonexistent/path"))
+      (is (string/includes? (.getMessage e) (str "DB cache file not found: " (io/file "/nonexistent/path"))))
       (is (string/includes? (.getMessage e) "Check --db-cache-path")))))
 
 (deftest run-missing-workspace-derived-path-test