Improve rules tests

Author: zikajk

docs/config/rules.md

@@ -8,6 +8,11 @@ Rules let you influence how the model behaves without changing an agent prompt.
 
 A rule is a Markdown file. By default, rule content is rendered into the system prompt for every chat where the rule matches. Add YAML frontmatter when a rule should apply only to specific agents, models, or paths.
 
+## Rules vs Skills
+
+- Use rules to make ECA automatically follow project conventions, safety constraints, or guidance for matching paths.
+- Use [skills](./skills.md) to teach ECA reusable workflows, tools, or domain knowledge it can load on demand for specific tasks.
+
 ## Rule locations
 
 ECA loads rules from 3 sources:

integration-test/integration/chat/commands_test.clj

@@ -35,6 +35,7 @@
                        {:name "config" :arguments []}
                        {:name "doctor" :arguments []}
                        {:name "repo-map-show" :arguments []}
+                       {:name "rules" :arguments []}
                        {:name "prompt-show" :arguments [{:name "optional-prompt"}]}
                        {:name "subagents" :arguments []}
                        {:name "plugins" :arguments []}

resources/META-INF/native-image/eca/eca/native-image.properties

@@ -39,6 +39,7 @@ Args=-J-Dborkdude.dynaload.aot=true \
      -H:IncludeResources=prompts/tools/skill.md \
      -H:IncludeResources=prompts/tools/spawn_agent.md \
      -H:IncludeResources=prompts/tools/write_file.md \
+     -H:IncludeResources=prompts/tools/fetch_rule.md \
      -H:IncludeResources=webpages/oauth.html \
      -H:IncludeResources=logo.svg \
      -H:IncludeResources=tls/local-eca-dev-fullchain.pem \

test/eca/features/rules_test.clj

@@ -430,12 +430,12 @@
 
 (deftest matching-path-scoped-rules-test
   (testing "returns every matching path-scoped rule with match details"
-    (with-redefs [f.rules/path-scoped-rules (constantly [{:id "/workspace-a/.eca/rules/format.md"
+    (with-redefs [f.rules/path-scoped-rules (constantly [{:id (h/file-path "/workspace-a/.eca/rules/format.md")
                                                           :name "format.md"
                                                           :scope :project
                                                           :workspace-root (h/file-path "/workspace-a")
                                                           :paths ["src/**.clj"]}
-                                                         {:id "/workspace-a/.eca/rules/notes.md"
+                                                         {:id (h/file-path "/workspace-a/.eca/rules/notes.md")
                                                           :name "notes.md"
                                                           :scope :project
                                                           :workspace-root (h/file-path "/workspace-a")
@@ -563,26 +563,27 @@
         (fs/create-dirs (fs/parent target-file))
         (spit rule-file "---\nagent: code\nmodel: \"openai/.*\"\npaths: \"src/**/*.clj\"\nenforce:\n  - read\n  - modify\n---\n\nUse project Clojure style.")
         (spit target-file "(ns foo)")
-        (let [workspace-root (shared/normalize-path tmp-dir)
-              roots [{:uri (shared/filename->uri (str tmp-dir))}]
-              rules (f.rules/path-scoped-rules {} roots "code" "openai/gpt-5.2")
-              rule (first rules)]
-          (is (= 1 (count rules)))
-          (is (match? {:name "clojure.md"
-                       :scope :project
-                       :workspace-root workspace-root
-                       :content "Use project Clojure style."
-                       :agents ["code"]
-                       :models ["openai/.*"]
-                       :paths ["src/**/*.clj"]
-                       :enforce ["read" "modify"]}
-                      rule))
-          (is (match? {:match? true
-                       :reason nil
-                       :workspace-root workspace-root
-                       :relative-path (str (fs/path "src" "nested" "foo.clj"))
-                       :matched-pattern "src/**/*.clj"}
-                      (f.rules/match-path-scoped-rule rule roots (str target-file)))))
+        (with-redefs [config/get-env (constantly (str (fs/path tmp-dir "no-global-config")))]
+          (let [workspace-root (shared/normalize-path tmp-dir)
+                roots [{:uri (shared/filename->uri (str tmp-dir))}]
+                rules (f.rules/path-scoped-rules {} roots "code" "openai/gpt-5.2")
+                rule (first rules)]
+            (is (= 1 (count rules)))
+            (is (match? {:name "clojure.md"
+                         :scope :project
+                         :workspace-root workspace-root
+                         :content "Use project Clojure style."
+                         :agents ["code"]
+                         :models ["openai/.*"]
+                         :paths ["src/**/*.clj"]
+                         :enforce ["read" "modify"]}
+                        rule))
+            (is (match? {:match? true
+                         :reason nil
+                         :workspace-root workspace-root
+                         :relative-path (str (fs/path "src" "nested" "foo.clj"))
+                         :matched-pattern "src/**/*.clj"}
+                        (f.rules/match-path-scoped-rule rule roots (str target-file))))))
         (finally
           (fs/delete-tree tmp-dir))))))