fix(skill): normalize multiline frontmatter descriptions

Author: ulises-jeremias

packages/opencode/src/skill/index.ts

@@ -58,6 +58,12 @@ function isSkillFrontmatter(data: unknown): data is { name: string; description?
   )
 }
 
+function normalizeDescription(description: string | undefined) {
+  if (description === undefined) return undefined
+  const normalized = description.replace(/\s+/g, " ").trim()
+  return normalized.length > 0 ? normalized : undefined
+}
+
 export class InvalidError extends Schema.TaggedErrorClass<InvalidError>()("SkillInvalidError", {
   path: Schema.String,
   message: Schema.optional(Schema.String),
@@ -133,7 +139,7 @@ const add = Effect.fnUntraced(function* (state: State, match: string, events: Ev
   state.dirs.add(path.dirname(match))
   state.skills[md.data.name] = {
     name: md.data.name,
-    description: md.data.description,
+    description: normalizeDescription(md.data.description),
     location: match,
     content: md.content,
   }

packages/opencode/test/skill/skill.test.ts

@@ -108,6 +108,36 @@ Instructions here.
     ),
   )
 
+  it.live("normalizes multiline YAML descriptions in SKILL.md frontmatter", () =>
+    provideTmpdirInstance(
+      (dir) =>
+        Effect.gen(function* () {
+          yield* Effect.promise(() =>
+            Bun.write(
+              path.join(dir, ".opencode", "skill", "multiline-desc", "SKILL.md"),
+              `---
+name: multiline-desc
+description: |
+  This is a multiline description
+  that spans multiple lines
+  using YAML pipe syntax
+---
+
+# Multiline Skill
+`,
+            ),
+          )
+
+          const skill = yield* Skill.Service
+          const list = (yield* skill.all()).filter((s) => s.location !== "<built-in>")
+          const item = list.find((x) => x.name === "multiline-desc")
+          expect(item).toBeDefined()
+          expect(item!.description).toBe("This is a multiline description that spans multiple lines using YAML pipe syntax")
+        }),
+      { git: true },
+    ),
+  )
+
   it.live("returns skill directories from Skill.dirs", () =>
     provideTmpdirInstance(
       (dir) =>