test: expand cli coverage

- add e2e coverage for install/update + --cli-version
- add unit coverage for sync optional changelog
- add handler tests for delete/undelete

Author: steipete

convex/httpApi.handlers.test.ts

@@ -271,4 +271,42 @@ describe('httpApi handlers', () => {
     })
     expect(await response.json()).toEqual({ ok: true })
   })
+
+  it('cliSkillDeleteHandler supports undelete', async () => {
+    vi.mocked(requireApiTokenUser).mockResolvedValueOnce({ userId: 'user1' } as never)
+    const runMutation = vi.fn().mockResolvedValue({ ok: true })
+    const request = new Request('https://x/api/cli/skill/undelete', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ slug: 'demo' }),
+    })
+    const response = await __handlers.cliSkillDeleteHandler(
+      { runMutation } as never,
+      request,
+      false,
+    )
+    expect(response.status).toBe(200)
+    expect(runMutation).toHaveBeenCalledWith(expect.anything(), {
+      userId: 'user1',
+      slug: 'demo',
+      deleted: false,
+    })
+  })
+
+  it('cliSkillDeleteHandler returns 400 on invalid json', async () => {
+    const request = new Request('https://x/api/cli/skill/delete', { method: 'POST', body: '{' })
+    const response = await __handlers.cliSkillDeleteHandler({} as never, request, true)
+    expect(response.status).toBe(400)
+  })
+
+  it('cliSkillDeleteHandler returns 400 on invalid payload', async () => {
+    vi.mocked(requireApiTokenUser).mockResolvedValueOnce({ userId: 'user1' } as never)
+    const request = new Request('https://x/api/cli/skill/delete', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({}),
+    })
+    const response = await __handlers.cliSkillDeleteHandler({} as never, request, true)
+    expect(response.status).toBe(400)
+  })
 })

e2e/clawdhub.e2e.test.ts

@@ -10,6 +10,7 @@ import {
   ApiSearchResponseSchema,
   parseArk,
 } from 'clawdhub-schema'
+import { unzipSync } from 'fflate'
 import { describe, expect, it } from 'vitest'
 import { readGlobalConfig } from '../packages/clawdhub/src/config'
 
@@ -31,6 +32,15 @@ async function makeTempConfig(registry: string, token: string | null) {
 }
 
 describe('clawdhub e2e', () => {
+  it('prints CLI version via --cli-version', async () => {
+    const result = spawnSync('bun', ['clawdhub', '--cli-version'], {
+      cwd: process.cwd(),
+      encoding: 'utf8',
+    })
+    expect(result.status).toBe(0)
+    expect(result.stdout.trim()).toMatch(/^\d+\.\d+\.\d+/)
+  })
+
   it('search endpoint returns a results array (schema parse)', async () => {
     const registry = process.env.CLAWDHUB_REGISTRY?.trim() || 'https://clawdhub.com'
     const url = new URL(ApiRoutes.search, registry)
@@ -174,6 +184,7 @@ describe('clawdhub e2e', () => {
 
     const cfg = await makeTempConfig(registry, token)
     const workdir = await mkdtemp(join(tmpdir(), 'clawdhub-e2e-publish-'))
+    const installWorkdir = await mkdtemp(join(tmpdir(), 'clawdhub-e2e-install-'))
     const slug = `e2e-${Date.now()}`
     const skillDir = join(workdir, slug)
 
@@ -241,6 +252,73 @@ describe('clawdhub e2e', () => {
       expect(publish2.status).toBe(0)
       expect(publish2.stderr).not.toMatch(/changelog required/i)
 
+      const downloadUrl = new URL(ApiRoutes.download, registry)
+      downloadUrl.searchParams.set('slug', slug)
+      downloadUrl.searchParams.set('version', '1.0.1')
+      const zipRes = await fetch(downloadUrl.toString())
+      expect(zipRes.ok).toBe(true)
+      const zipBytes = new Uint8Array(await zipRes.arrayBuffer())
+      const unzipped = unzipSync(zipBytes)
+      expect(Object.keys(unzipped)).toContain('SKILL.md')
+
+      const install = spawnSync(
+        'bun',
+        [
+          'clawdhub',
+          'install',
+          slug,
+          '--version',
+          '1.0.0',
+          '--force',
+          '--site',
+          site,
+          '--registry',
+          registry,
+          '--workdir',
+          installWorkdir,
+        ],
+        {
+          cwd: process.cwd(),
+          env: { ...process.env, CLAWDHUB_CONFIG_PATH: cfg.path },
+          encoding: 'utf8',
+        },
+      )
+      expect(install.status).toBe(0)
+
+      const list = spawnSync(
+        'bun',
+        ['clawdhub', 'list', '--site', site, '--registry', registry, '--workdir', installWorkdir],
+        {
+          cwd: process.cwd(),
+          env: { ...process.env, CLAWDHUB_CONFIG_PATH: cfg.path },
+          encoding: 'utf8',
+        },
+      )
+      expect(list.status).toBe(0)
+      expect(list.stdout).toMatch(new RegExp(`${slug}\\s+1\\.0\\.0`))
+
+      const update = spawnSync(
+        'bun',
+        [
+          'clawdhub',
+          'update',
+          slug,
+          '--force',
+          '--site',
+          site,
+          '--registry',
+          registry,
+          '--workdir',
+          installWorkdir,
+        ],
+        {
+          cwd: process.cwd(),
+          env: { ...process.env, CLAWDHUB_CONFIG_PATH: cfg.path },
+          encoding: 'utf8',
+        },
+      )
+      expect(update.status).toBe(0)
+
       const metaUrl = new URL(ApiRoutes.skill, registry)
       metaUrl.searchParams.set('slug', slug)
       const metaRes = await fetch(metaUrl.toString(), { headers: { Accept: 'application/json' } })
@@ -273,9 +351,6 @@ describe('clawdhub e2e', () => {
       })
       expect(metaAfterDelete.status).toBe(404)
 
-      const downloadUrl = new URL(ApiRoutes.download, registry)
-      downloadUrl.searchParams.set('slug', slug)
-      downloadUrl.searchParams.set('version', '1.0.1')
       const downloadAfterDelete = await fetch(downloadUrl.toString())
       expect(downloadAfterDelete.status).toBe(404)
 
@@ -330,6 +405,7 @@ describe('clawdhub e2e', () => {
         // best-effort cleanup
       }
       await rm(workdir, { recursive: true, force: true })
+      await rm(installWorkdir, { recursive: true, force: true })
       await rm(cfg.dir, { recursive: true, force: true })
     }
   }, 180_000)

packages/clawdhub/src/cli/commands/sync.test.ts

@@ -6,6 +6,7 @@ import type { GlobalOpts } from '../types'
 const mockIntro = vi.fn()
 const mockOutro = vi.fn()
 const mockNote = vi.fn()
+let interactive = false
 
 vi.mock('@clack/prompts', () => ({
   intro: (value: string) => mockIntro(value),
@@ -39,7 +40,7 @@ vi.mock('../ui.js', () => ({
   createSpinner: vi.fn(() => mockSpinner),
   fail: (message: string) => mockFail(message),
   formatError: (error: unknown) => (error instanceof Error ? error.message : String(error)),
-  isInteractive: () => false,
+  isInteractive: () => interactive,
 }))
 
 vi.mock('../scanSkills.js', () => ({
@@ -86,6 +87,7 @@ afterEach(() => {
 
 describe('cmdSync', () => {
   it('classifies skills as new/update/synced (dry-run, mocked HTTP)', async () => {
+    interactive = false
     mockApiRequest.mockImplementation(async (_registry: string, args: { path: string }) => {
       if (args.path === '/api/cli/whoami') return { user: { handle: 'steipete' } }
       if (args.path.startsWith('/api/skill?slug=')) {
@@ -117,4 +119,37 @@ describe('cmdSync', () => {
     const dryRunOutro = mockOutro.mock.calls.at(-1)?.[0]
     expect(String(dryRunOutro)).toMatch(/Dry run: would upload 2 skill/)
   })
+
+  it('allows empty changelog for updates (interactive)', async () => {
+    interactive = true
+    mockApiRequest.mockImplementation(async (_registry: string, args: { path: string }) => {
+      if (args.path === '/api/cli/whoami') return { user: { handle: 'steipete' } }
+      if (args.path.startsWith('/api/skill?slug=')) {
+        const slug = new URL(`https://x.test${args.path}`).searchParams.get('slug')
+        if (slug === 'new-skill') return { latestVersion: undefined, skill: null }
+        if (slug === 'synced-skill') return { latestVersion: { version: '1.2.3' }, skill: {} }
+        if (slug === 'update-skill') return { latestVersion: { version: '1.0.0' }, skill: {} }
+      }
+      if (args.path.startsWith('/api/skill/resolve?')) {
+        const u = new URL(`https://x.test${args.path}`)
+        const slug = u.searchParams.get('slug')
+        if (slug === 'synced-skill') {
+          return { match: { version: '1.2.3' }, latestVersion: { version: '1.2.3' } }
+        }
+        if (slug === 'update-skill') {
+          return { match: null, latestVersion: { version: '1.0.0' } }
+        }
+      }
+      throw new Error(`Unexpected apiRequest: ${args.path}`)
+    })
+
+    await cmdSync(makeOpts(), { root: ['/scan'], all: true, dryRun: false, bump: 'patch' }, true)
+
+    const calls = mockCmdPublish.mock.calls.map(
+      (call) => call[2] as { slug: string; changelog: string },
+    )
+    const update = calls.find((c) => c.slug === 'update-skill')
+    if (!update) throw new Error('Missing update-skill publish')
+    expect(update.changelog).toBe('')
+  })
 })

packages/schema/src/schemas.test.ts

@@ -5,6 +5,7 @@ import { parseArk } from './ark'
 import {
   ApiSearchResponseSchema,
   CliPublishRequestSchema,
+  CliSkillDeleteRequestSchema,
   LockfileSchema,
   WellKnownConfigSchema,
 } from './schemas'
@@ -89,4 +90,10 @@ describe('clawdhub-schema', () => {
     expect(parsed.results).toHaveLength(2)
     expect(parsed.results[0]?.slug).toBe('a')
   })
+
+  it('parses delete request payload', () => {
+    expect(parseArk(CliSkillDeleteRequestSchema, { slug: 'demo' }, 'Delete')).toEqual({
+      slug: 'demo',
+    })
+  })
 })