Fix crash on invalid YAML frontmatter in plugin skill files

Author: ericdallo

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - Clarify Task tool prompt for task ordering, task granularity, concurrent starts, and clearing finished task lists.
 - Replace MCP Java SDK with plumcp for MCP client communication. SSE transport is no longer supported (deprecated in MCP spec 2025-03-26).
+- Fix crash on invalid YAML frontmatter in plugin skill files.
 
 ## 0.112.1
 

src/eca/features/skills.clj

@@ -4,18 +4,25 @@
    [clojure.java.io :as io]
    [eca.config :as config]
    [eca.interpolation :as interpolation]
+   [eca.logger :as logger]
    [eca.shared :as shared]))
 
 (set! *warn-on-reflection* true)
 
+(def ^:private logger-tag "[skills]")
+
 (defn ^:private skill-file->skill [skill-file]
-  (let [content (interpolation/replace-dynamic-strings (slurp (str skill-file)) (fs/parent skill-file) nil)
-        {:keys [name description body]} (shared/parse-md (or content ""))]
-    (when (and name description)
-      {:name name
-       :description description
-       :body body
-       :dir (str (fs/canonicalize (fs/parent skill-file)))})))
+  (try
+    (let [content (interpolation/replace-dynamic-strings (slurp (str skill-file)) (fs/parent skill-file) nil)
+          {:keys [name description body]} (shared/parse-md (or content ""))]
+      (when (and name description)
+        {:name name
+         :description description
+         :body body
+         :dir (str (fs/canonicalize (fs/parent skill-file)))}))
+    (catch Exception e
+      (logger/warn logger-tag (format "Error parsing skill file '%s': %s" (str skill-file) (.getMessage e)))
+      nil)))
 
 (defn global-skills-dir []
   (let [xdg-config-home (or (config/get-env "XDG_CONFIG_HOME")