Flatten the ponytail skill description for discovery

Author: TheGreatAxios

plugins/corbits-skills/skills/ponytail/SKILL.md

@@ -1,19 +1,7 @@
 ---
 name: ponytail
 user-invocable: false
-description: >
-  Forces the laziest solution that actually works, simplest, shortest, most
-  minimal. Channels a senior dev who has seen everything: question whether the
-  task needs to exist at all (YAGNI), reach for the standard library before
-  custom code, native platform features before dependencies, one line before
-  fifty. Supports intensity levels: lite, full (default), ultra. Use on ANY
-  coding task: writing, adding, refactoring, fixing, reviewing, or designing
-  code, and choosing libraries or dependencies. Also use whenever the user
-  says "ponytail", "be lazy", "lazy mode", "simplest solution", "minimal
-  solution", "yagni", "do less", or "shortest path", or complains about
-  over-engineering, bloat, boilerplate, or unnecessary dependencies. Do NOT
-  use for non-coding requests (general knowledge, prose, translation,
-  summaries, recipes).
+description: Less-is-more quality bar: YAGNI, reuse, shortest working diff. Baked into implement and critic; load when writing or reviewing product code.
 argument-hint: "[lite|full|ultra]"
 license: MIT
 ---

src/agent/directors/identity.test.ts

@@ -63,6 +63,18 @@ describe("formatDirectorSystemPrompt", () => {
     expect(text).toContain(ponytail);
   });
 
+  test("bakes real ponytail body for critic including Corbits host", () => {
+    const text = formatDirectorSystemPrompt(DIRECTOR_REGISTRY.critic);
+    const ponytail = stripFrontmatter(
+      readFileSync(
+        join(import.meta.dirname, "../../../plugins/corbits-skills/skills/ponytail/SKILL.md"),
+        "utf8",
+      ),
+    );
+    expect(text).toContain(ponytail);
+    expect(text).toContain("## Corbits host");
+  });
+
   test("does not bake skill bodies when optionalSkills is empty", () => {
     const text = formatDirectorSystemPrompt(DIRECTOR_REGISTRY.intern);
     expect(text).not.toContain("# Baked skill guidance");

src/agent/directors/skywalker/package.test.ts

@@ -207,4 +207,10 @@ describe("skywalkerPackage", () => {
     expect(p).toContain("ship → verify → fix → re-verify");
     expect(p).toContain("Cap re-fix rounds");
   });
+
+  test("systemPrompt treats implement read-first as expected progress", () => {
+    expect(skywalkerPackage.systemPrompt).toContain(
+      "Implement's bounded read-first phase is expected progress",
+    );
+  });
 });

tests/unit/corbits-skills-catalog.test.ts

@@ -2,6 +2,7 @@ import { existsSync } from "node:fs";
 import { readdir } from "node:fs/promises";
 import { join } from "node:path";
 import { expect, test } from "bun:test";
+import { discoverSkills } from "../../src/extensions/skills.js";
 import { loadSkillCommands } from "../../src/plugins/skill-commands.js";
 
 const pluginRoot = join(import.meta.dirname, "../../plugins/corbits-skills");
@@ -305,4 +306,29 @@ test("ponytail skill is use-skill-only with a Corbits host addendum", async () =
   const skill = await Bun.file(join(pluginRoot, "skills/ponytail/SKILL.md")).text();
   expect(skill).toContain(USER_INVOCABLE_FALSE);
   expect(skill).toContain("## Corbits host");
+  const descLine = skill.split("\n").find((line) => line.startsWith("description:"));
+  expect(descLine).toBeDefined();
+  if (descLine === undefined) return;
+  const value = descLine.slice("description:".length).trim();
+  expect(value).not.toBe(">");
+  expect(value).not.toBe("|");
+  expect(value).toBe(
+    "Less-is-more quality bar: YAGNI, reuse, shortest working diff. Baked into implement and critic; load when writing or reviewing product code.",
+  );
+});
+
+test("discoverSkills lists one-line skill descriptions, not YAML fold markers", async () => {
+  const skills = await discoverSkills(".", [pluginRoot]);
+  const ponytail = skills.find((s) => s.name === "ponytail");
+  expect(ponytail).toBeDefined();
+  if (ponytail === undefined) return;
+  expect(ponytail.description).toBe(
+    "Less-is-more quality bar: YAGNI, reuse, shortest working diff. Baked into implement and critic; load when writing or reviewing product code.",
+  );
+  expect(ponytail.description.length).toBeGreaterThan(8);
+  expect(ponytail.description).not.toContain("\n");
+  for (const skill of skills) {
+    expect(skill.description).not.toBe(">");
+    expect(skill.description).not.toBe("|");
+  }
 });