@dealdeploy/skl 1.3.0 → 1.5.0

package/add-tui.ts

@@ -202,7 +202,9 @@ export function createAddTui(renderer: CliRenderer, deps: AddTuiDeps) {
   renderer.keyInput.on("keypress", (key: KeyEvent) => {
     const prevCursor = cursor;
 
-    switch (key.name) {
+    const navKey = (key.ctrl && { h: "left", j: "down", k: "up", l: "right" }[key.name]) || key.name;
+
+    switch (navKey) {
       case "j":
       case "down":
         if (cursor < skills.length - 1) cursor++;

package/lib.test.ts

@@ -288,7 +288,7 @@ describe("parseRepoArg", () => {
 // ── buildAddArgs ────────────────────────────────────────────────────
 
 describe("buildAddArgs", () => {
-  test("uses lock entry sourceUrl for remote skill with project agents", () => {
+  test("uses owner/repo/skillPath shorthand for remote skill with project agents", () => {
     const entry: LockEntry = {
       source: "owner/repo",
       sourceUrl: "https://github.com/owner/repo",
@@ -299,7 +299,7 @@ describe("buildAddArgs", () => {
     addToLock("myskill", entry);
 
     const args = buildAddArgs("/catalog", "myskill", false, ["claude-code", "roo"]);
-    expect(args).toEqual(["add", "https://github.com/owner/repo", "--skill", "myskill", "-y", "--agent", "claude-code", "roo"]);
+    expect(args).toEqual(["add", "owner/repo/skills/myskill", "-y", "--agent", "claude-code", "roo"]);
   });
 
   test("global install does not pass --agent", () => {
@@ -315,6 +315,7 @@ describe("buildAddArgs", () => {
     const args = buildAddArgs("/catalog", "myskill", true, ["claude-code"]);
     expect(args).toContain("-g");
     expect(args).not.toContain("--agent");
+    expect(args).toContain("owner/repo/skills/myskill");
   });
 
   test("uses catalog path for hand-authored skill with project agents", () => {
@@ -326,6 +327,21 @@ describe("buildAddArgs", () => {
     const args = buildAddArgs("/my/catalog", "custom-skill", true);
     expect(args).toEqual(["add", "/my/catalog/custom-skill", "-y", "-g"]);
   });
+
+  test("uses subpath for nested skill paths", () => {
+    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).toEqual(["add", "owner/repo/skills/code-review", "-y", "-g"]);
+    expect(args).not.toContain("--skill");
+  });
 });
 
 // ── detectProjectAgents ─────────────────────────────────────────────

package/lib.ts

@@ -172,7 +172,9 @@ export function buildAddArgs(
     ? ["--agent", ...projectAgents]
     : [];
   if (lockEntry) {
-    const args = ["add", lockEntry.sourceUrl, "--skill", name, "-y", ...agentArgs];
+    // Use owner/repo/skillPath shorthand so the CLI narrows via subpath
+    const source = `${lockEntry.source}/${lockEntry.skillPath}`;
+    const args = ["add", source, "-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.5.0",
   "description": "TUI skill manager for Claude Code agents",
   "module": "index.ts",
   "bin": {

package/tui.ts

@@ -227,7 +227,7 @@ export function createTui(renderer: CliRenderer, deps: TuiDeps) {
     width: "100%",
   });
 
-  type HeaderRef = { spacer: BoxRenderable | null; row: BoxRenderable; text: TextRenderable };
+  type HeaderRef = { spacer: BoxRenderable; row: BoxRenderable; text: TextRenderable };
   const headerRefs = new Map<number, HeaderRef>();
 
   type RowRefs = {
@@ -243,16 +243,13 @@ export function createTui(renderer: CliRenderer, deps: TuiDeps) {
   for (let di = 0; di < displayItems.length; di++) {
     const item = displayItems[di]!;
     if (item.type === "header") {
-      let spacer: BoxRenderable | null = null;
-      if (di > 0) {
-        spacer = new BoxRenderable(renderer, {
-          id: `spacer-${di}`,
-          height: 1,
-          width: "100%",
-          backgroundColor: C.rowBg,
-        });
-        scrollBox.add(spacer);
-      }
+      const spacer = new BoxRenderable(renderer, {
+        id: `spacer-${di}`,
+        height: 1,
+        width: "100%",
+        backgroundColor: C.rowBg,
+      });
+      scrollBox.add(spacer);
       const hRow = new BoxRenderable(renderer, {
         id: `header-${di}`,
         flexDirection: "row",
@@ -368,7 +365,7 @@ export function createTui(renderer: CliRenderer, deps: TuiDeps) {
         const visible = visibleRepos.has(item.repo);
         const ref = headerRefs.get(di)!;
         ref.row.visible = visible;
-        if (ref.spacer) ref.spacer.visible = visible;
+        ref.spacer.visible = visible;
       } else {
         const visible = matchingSkills.has(item.skillIndex);
         rows[item.skillIndex]!.row.visible = visible;
@@ -484,13 +481,13 @@ export function createTui(renderer: CliRenderer, deps: TuiDeps) {
           const ref = headerRefs.get(di)!;
           if (ref.row.visible) {
             visibleBefore++;
-            if (ref.spacer?.visible) visibleBefore++;
+            if (ref.spacer.visible) visibleBefore++;
           }
         } else {
           if (rows[item.skillIndex]!.row.visible) visibleBefore++;
         }
       }
-      scrollBox.scrollTo(Math.max(0, visibleBefore + 1 - 2));
+      scrollBox.scrollTo(Math.max(0, visibleBefore + 2 - 2));
     }
   }
 
@@ -614,7 +611,7 @@ export function createTui(renderer: CliRenderer, deps: TuiDeps) {
         }
         return;
       }
-      if (key.name === "down" || key.name === "return") {
+      if (key.name === "down" || key.name === "return" || (key.ctrl && key.name === "j")) {
         focusArea = "grid";
         refreshAll();
         ensureVisible();
@@ -634,7 +631,15 @@ export function createTui(renderer: CliRenderer, deps: TuiDeps) {
     // ── Grid-focused input ──
 
     if (key.name === "backspace") return;
-    if (key.ctrl) return;
+    const ctrlNav: Record<string, string> = { h: "left", j: "down", k: "up", l: "right" };
+    let navKey = key.name;
+    if (key.ctrl) {
+      if (ctrlNav[key.name]) {
+        navKey = ctrlNav[key.name]!;
+      } else {
+        return;
+      }
+    }
 
     if (key.sequence === "q") {
       deps.onQuit();
@@ -664,7 +669,7 @@ export function createTui(renderer: CliRenderer, deps: TuiDeps) {
     }
 
     // ── Navigation & actions ──
-    switch (key.name) {
+    switch (navKey) {
       case "down":
         if (cursor < filteredDisplayIndices.length - 1) cursor++;
         break;