@dealdeploy/skl 1.3.0 → 1.4.0

package/lib.test.ts

@@ -16,6 +16,7 @@ import {
   findSkillEntries,
   parseRepoArg,
   buildAddArgs,
+  readSkillFrontmatterName,
   detectProjectAgents,
   parseSkillsListOutput,
   planUpdates,
@@ -326,6 +327,67 @@ describe("buildAddArgs", () => {
     const args = buildAddArgs("/my/catalog", "custom-skill", true);
     expect(args).toEqual(["add", "/my/catalog/custom-skill", "-y", "-g"]);
   });
+
+  test("uses SKILL.md frontmatter name for --skill flag when it differs from dir name", () => {
+    const catalog = catalogDir();
+    mkdirSync(join(catalog, "code-review"), { recursive: true });
+    writeFileSync(
+      join(catalog, "code-review", "SKILL.md"),
+      "---\nname: code-review:code-review\ndescription: Review PRs\n---\n# Code Review"
+    );
+
+    const entry: LockEntry = {
+      source: "owner/repo",
+      sourceUrl: "https://github.com/owner/repo",
+      skillPath: "skills/code-review",
+      treeSHA: "abc",
+      addedAt: "2026-01-01",
+    };
+    addToLock("code-review", entry);
+
+    const args = buildAddArgs(catalog, "code-review", true);
+    expect(args).toContain("--skill");
+    expect(args).toContain("code-review:code-review");
+    expect(args).not.toContain("--agent");
+    expect(args).toContain("-g");
+  });
+});
+
+// ── readSkillFrontmatterName ────────────────────────────────────────
+
+describe("readSkillFrontmatterName", () => {
+  test("reads name from valid frontmatter", () => {
+    const catalog = catalogDir();
+    mkdirSync(join(catalog, "my-skill"), { recursive: true });
+    writeFileSync(
+      join(catalog, "my-skill", "SKILL.md"),
+      "---\nname: my-skill:variant\ndescription: test\n---\n# Content"
+    );
+    expect(readSkillFrontmatterName(catalog, "my-skill")).toBe("my-skill:variant");
+  });
+
+  test("returns null when no frontmatter", () => {
+    const catalog = catalogDir();
+    mkdirSync(join(catalog, "no-fm"), { recursive: true });
+    writeFileSync(join(catalog, "no-fm", "SKILL.md"), "# Just markdown");
+    expect(readSkillFrontmatterName(catalog, "no-fm")).toBeNull();
+  });
+
+  test("returns null when SKILL.md missing", () => {
+    const catalog = catalogDir();
+    mkdirSync(join(catalog, "empty"), { recursive: true });
+    expect(readSkillFrontmatterName(catalog, "empty")).toBeNull();
+  });
+
+  test("returns null when name field missing from frontmatter", () => {
+    const catalog = catalogDir();
+    mkdirSync(join(catalog, "no-name"), { recursive: true });
+    writeFileSync(
+      join(catalog, "no-name", "SKILL.md"),
+      "---\ndescription: has no name\n---\n# Content"
+    );
+    expect(readSkillFrontmatterName(catalog, "no-name")).toBeNull();
+  });
 });
 
 // ── detectProjectAgents ─────────────────────────────────────────────

package/lib.ts

@@ -160,6 +160,19 @@ export function detectProjectAgents(cwd: string): string[] {
   return agents;
 }
 
+/** Read the frontmatter `name` from a skill's SKILL.md in the catalog. */
+export function readSkillFrontmatterName(catalogPath: string, dirName: string): string | null {
+  try {
+    const content = readFileSync(join(catalogPath, dirName, "SKILL.md"), "utf-8");
+    const match = content.match(/^---\s*\n([\s\S]*?)\n---/);
+    if (!match) return null;
+    const nameMatch = match[1]!.match(/^name:\s*(.+)$/m);
+    return nameMatch ? nameMatch[1]!.trim() : null;
+  } catch {
+    return null;
+  }
+}
+
 /** Build npx skills add args for a skill. */
 export function buildAddArgs(
   catalogPath: string,
@@ -172,7 +185,8 @@ export function buildAddArgs(
     ? ["--agent", ...projectAgents]
     : [];
   if (lockEntry) {
-    const args = ["add", lockEntry.sourceUrl, "--skill", name, "-y", ...agentArgs];
+    const skillName = readSkillFrontmatterName(catalogPath, name) ?? name;
+    const args = ["add", lockEntry.sourceUrl, "--skill", skillName, "-y", ...agentArgs];
     if (isGlobal) args.push("-g");
     return args;
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dealdeploy/skl",
-  "version": "1.3.0",
+  "version": "1.4.0",
   "description": "TUI skill manager for Claude Code agents",
   "module": "index.ts",
   "bin": {

package/tui.ts

@@ -239,6 +239,14 @@ export function createTui(renderer: CliRenderer, deps: TuiDeps) {
   const rows: RowRefs[] = new Array(allSkills.length);
 
   scrollBox.add(searchRow);
+  scrollBox.add(
+    new BoxRenderable(renderer, {
+      id: "spacer-search",
+      height: 1,
+      width: "100%",
+      backgroundColor: C.rowBg,
+    })
+  );
 
   for (let di = 0; di < displayItems.length; di++) {
     const item = displayItems[di]!;
@@ -490,7 +498,7 @@ export function createTui(renderer: CliRenderer, deps: TuiDeps) {
           if (rows[item.skillIndex]!.row.visible) visibleBefore++;
         }
       }
-      scrollBox.scrollTo(Math.max(0, visibleBefore + 1 - 2));
+      scrollBox.scrollTo(Math.max(0, visibleBefore + 2 - 2));
     }
   }