refactor(clawhub): remove unused skill list endpoint

vincentkoc · vincentkoc · commit 3c1b346115b5 · 2026-06-17T23:53:47.000+08:00
diff --git a/src/infra/clawhub.ts b/src/infra/clawhub.ts
@@ -386,27 +386,6 @@ export type ClawHubSkillSecurityVerdictsResponse = {
   items: ClawHubSkillSecurityVerdictItem[];
 };
 
-export type ClawHubSkillListResponse = {
-  items: Array<{
-    slug: string;
-    displayName: string;
-    summary?: string;
-    tags?: Record<string, string>;
-    latestVersion?: {
-      version: string;
-      createdAt: number;
-      changelog?: string;
-    } | null;
-    metadata?: {
-      os?: string[] | null;
-      systems?: string[] | null;
-    } | null;
-    createdAt: number;
-    updatedAt: number;
-  }>;
-  nextCursor?: string | null;
-};
-
 export type ClawHubDownloadResult = {
   archivePath: string;
   integrity: string;
@@ -1171,25 +1150,6 @@ export async function fetchClawHubSkillCard(params: {
   return new TextDecoder().decode(bytes);
 }
 
-export async function listClawHubSkills(params: {
-  baseUrl?: string;
-  token?: string;
-  timeoutMs?: number;
-  fetchImpl?: FetchLike;
-  limit?: number;
-}): Promise<ClawHubSkillListResponse> {
-  return await fetchJson<ClawHubSkillListResponse>({
-    baseUrl: params.baseUrl,
-    path: "/api/v1/skills",
-    token: params.token,
-    timeoutMs: params.timeoutMs,
-    fetchImpl: params.fetchImpl,
-    search: {
-      limit: params.limit ? String(params.limit) : undefined,
-    },
-  });
-}
-
 export async function downloadClawHubPackageArchive(params: {
   name: string;
   version?: string;
diff --git a/src/skills/lifecycle/clawhub.test.ts b/src/skills/lifecycle/clawhub.test.ts
@@ -12,7 +12,6 @@ const fetchClawHubSkillVerificationMock = vi.fn();
 const downloadClawHubSkillArchiveMock = vi.fn();
 const downloadClawHubSkillArchiveUrlMock = vi.fn();
 const downloadClawHubGitHubSkillArchiveMock = vi.fn();
-const listClawHubSkillsMock = vi.fn();
 const reportClawHubSkillInstallTelemetryMock = vi.fn();
 const resolveClawHubBaseUrlMock = vi.fn(() => "https://clawhub.ai");
 const isDefaultClawHubBaseUrlMock = vi.fn((baseUrl?: string) => !baseUrl);
@@ -31,7 +30,6 @@ vi.mock("../../infra/clawhub.js", () => ({
   downloadClawHubSkillArchive: downloadClawHubSkillArchiveMock,
   downloadClawHubSkillArchiveUrl: downloadClawHubSkillArchiveUrlMock,
   downloadClawHubGitHubSkillArchive: downloadClawHubGitHubSkillArchiveMock,
-  listClawHubSkills: listClawHubSkillsMock,
   reportClawHubSkillInstallTelemetry: reportClawHubSkillInstallTelemetryMock,
   isDefaultClawHubBaseUrl: isDefaultClawHubBaseUrlMock,
   resolveClawHubBaseUrl: resolveClawHubBaseUrlMock,
@@ -180,7 +178,6 @@ describe("skills-clawhub", () => {
     downloadClawHubSkillArchiveMock.mockReset();
     downloadClawHubSkillArchiveUrlMock.mockReset();
     downloadClawHubGitHubSkillArchiveMock.mockReset();
-    listClawHubSkillsMock.mockReset();
     reportClawHubSkillInstallTelemetryMock.mockReset();
     resolveClawHubBaseUrlMock.mockReset();
     isDefaultClawHubBaseUrlMock.mockReset();
@@ -1392,7 +1389,6 @@ describe("skills-clawhub", () => {
       limit: 20,
       baseUrl: undefined,
     });
-    expect(listClawHubSkillsMock).not.toHaveBeenCalled();
   });
 });
 
