@skilltap/core 0.4.4 → 0.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skilltap/core",
-  "version": "0.4.4",
+  "version": "0.5.0",
   "description": "Core library for skilltap — agent skill management",
   "type": "module",
   "license": "MIT",

package/src/self-update.ts

@@ -60,7 +60,7 @@ async function writeCache(configDir: string, latest: string): Promise<void> {
   }
 }
 
-async function fetchLatestVersion(): Promise<string | null> {
+export async function fetchLatestVersion(): Promise<string | null> {
   try {
     const response = await fetch(
       `https://api.github.com/repos/${GITHUB_OWNER}/${GITHUB_REPO}/releases/latest`,

package/src/taps.test.ts

@@ -9,7 +9,14 @@ import {
 import { loadConfig } from "./config";
 import { installSkill } from "./install";
 import type { TapEntry } from "./taps";
-import { addTap, loadTaps, removeTap, searchTaps, updateTap } from "./taps";
+import {
+  addTap,
+  loadTaps,
+  parseGitHubTapShorthand,
+  removeTap,
+  searchTaps,
+  updateTap,
+} from "./taps";
 
 type Env = {
   SKILLTAP_HOME?: string;
@@ -74,6 +81,61 @@ async function createLocalSkillRepo(
   return { path: repoDir, cleanup: () => removeTmpDir(repoDir) };
 }
 
+// ─── Unit tests: parseGitHubTapShorthand ───────────────────────────────────
+
+describe("parseGitHubTapShorthand", () => {
+  test("parses owner/repo", () => {
+    expect(parseGitHubTapShorthand("user/my-tap")).toEqual({
+      name: "my-tap",
+      url: "https://github.com/user/my-tap.git",
+    });
+  });
+
+  test("parses github:owner/repo", () => {
+    expect(parseGitHubTapShorthand("github:acme/skills")).toEqual({
+      name: "skills",
+      url: "https://github.com/acme/skills.git",
+    });
+  });
+
+  test("strips @ref suffix", () => {
+    expect(parseGitHubTapShorthand("user/tap@main")).toEqual({
+      name: "tap",
+      url: "https://github.com/user/tap.git",
+    });
+  });
+
+  test("returns null for bare names", () => {
+    expect(parseGitHubTapShorthand("my-tap")).toBeNull();
+  });
+
+  test("returns null for full URLs", () => {
+    expect(
+      parseGitHubTapShorthand("https://github.com/user/repo.git"),
+    ).toBeNull();
+  });
+
+  test("returns null for npm: prefix", () => {
+    expect(parseGitHubTapShorthand("npm:my-package")).toBeNull();
+  });
+
+  test("returns null for local paths", () => {
+    expect(parseGitHubTapShorthand("./local")).toBeNull();
+    expect(parseGitHubTapShorthand("/abs/path")).toBeNull();
+    expect(parseGitHubTapShorthand("~/home")).toBeNull();
+  });
+
+  test("returns null for three-part paths", () => {
+    expect(parseGitHubTapShorthand("a/b/c")).toBeNull();
+  });
+
+  test("returns null for git@ URLs", () => {
+    expect(
+      parseGitHubTapShorthand("git@github.com:user/repo.git"),
+    ).toBeNull();
+  });
+});
+
 // ─── Unit tests: searchTaps ────────────────────────────────────────────────
 
 describe("searchTaps", () => {

package/src/taps.ts

@@ -63,6 +63,36 @@ function registrySourceToRepo(source: RegistrySource): string {
   }
 }
 
+export type GitHubTapShorthand = { name: string; url: string };
+
+const GH_LOCAL_PREFIXES = ["./", "/", "~/"];
+const GH_URL_PROTOCOLS = ["https://", "http://", "git@", "ssh://", "npm:"];
+
+/** Parse GitHub shorthand (owner/repo) into a tap name + clone URL. Returns null if not shorthand. */
+export function parseGitHubTapShorthand(
+  source: string,
+): GitHubTapShorthand | null {
+  let s = source;
+  if (s.startsWith("github:")) s = s.slice("github:".length);
+  else if (!s.includes("/")) return null;
+
+  if (GH_URL_PROTOCOLS.some((p) => s.startsWith(p))) return null;
+  if (GH_LOCAL_PREFIXES.some((p) => s.startsWith(p))) return null;
+
+  // Strip @ref suffix (taps always clone HEAD)
+  const atIdx = s.lastIndexOf("@");
+  if (atIdx !== -1) s = s.slice(0, atIdx);
+
+  const parts = s.split("/").filter(Boolean);
+  if (parts.length !== 2) return null;
+
+  const [owner, repo] = parts;
+  return {
+    name: repo!,
+    url: `https://github.com/${owner}/${repo}.git`,
+  };
+}
+
 export async function addTap(
   name: string,
   url: string,