fix(cli): unbox sync output

Author: steipete

CHANGELOG.md

@@ -8,6 +8,7 @@
 
 ### Changed
 - CLI sync: default `--concurrency` is now 4 (was 8).
+- CLI sync: replace boxed notes with plain output for long lists.
 
 ### Fixed
 - CLI sync: wrap note output to avoid terminal overflow; cap list lengths.

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

@@ -5,7 +5,7 @@ import type { GlobalOpts } from '../types'
 
 const mockIntro = vi.fn()
 const mockOutro = vi.fn()
-const mockNote = vi.fn()
+const mockLog = vi.fn()
 const mockMultiselect = vi.fn(async (_args?: unknown) => [] as string[])
 let interactive = false
 
@@ -21,7 +21,6 @@ const defaultFindSkillFolders = async (root: string) => {
 vi.mock('@clack/prompts', () => ({
   intro: (value: string) => mockIntro(value),
   outro: (value: string) => mockOutro(value),
-  note: (message: string, body?: string) => mockNote(message, body),
   multiselect: (args: unknown) => mockMultiselect(args),
   text: vi.fn(async () => ''),
   isCancel: () => false,
@@ -90,6 +89,10 @@ afterEach(async () => {
   vi.mocked(findSkillFolders).mockImplementation(defaultFindSkillFolders)
 })
 
+vi.spyOn(console, 'log').mockImplementation((...args) => {
+  mockLog(args.map(String).join(' '))
+})
+
 describe('cmdSync', () => {
   it('classifies skills as new/update/synced (dry-run, mocked HTTP)', async () => {
     interactive = false
@@ -115,8 +118,9 @@ describe('cmdSync', () => {
 
     expect(mockCmdPublish).not.toHaveBeenCalled()
 
-    const alreadySyncedNote = mockNote.mock.calls.find((call) => call[0] === 'Already synced')
-    expect(alreadySyncedNote?.[1]).toMatch(/synced-skill/)
+    const output = mockLog.mock.calls.map((call) => String(call[0])).join('\n')
+    expect(output).toMatch(/Already synced/)
+    expect(output).toMatch(/synced-skill/)
 
     const dryRunOutro = mockOutro.mock.calls.at(-1)?.[0]
     expect(String(dryRunOutro)).toMatch(/Dry run: would upload 2 skill/)
@@ -148,12 +152,12 @@ describe('cmdSync', () => {
 
     await cmdSync(makeOpts(), { root: ['/scan'], all: false, dryRun: false, bump: 'patch' }, true)
 
-    const toSyncNote = mockNote.mock.calls.find((call) => call[0] === 'To sync')
-    expect(toSyncNote?.[1]).toMatch(/- new-skill/)
-    expect(toSyncNote?.[1]).toMatch(/- update-skill/)
-
-    const syncedNote = mockNote.mock.calls.find((call) => call[0] === 'Already synced')
-    expect(syncedNote?.[1]).toMatch(/- synced-skill/)
+    const output = mockLog.mock.calls.map((call) => String(call[0])).join('\n')
+    expect(output).toMatch(/To sync/)
+    expect(output).toMatch(/- new-skill/)
+    expect(output).toMatch(/- update-skill/)
+    expect(output).toMatch(/Already synced/)
+    expect(output).toMatch(/- synced-skill/)
 
     const lastCall = mockMultiselect.mock.calls.at(-1)
     const promptArgs = lastCall ? (lastCall[0] as { initialValues: string[] }) : undefined
@@ -173,10 +177,11 @@ describe('cmdSync', () => {
 
     await cmdSync(makeOpts(), { root: ['/scan'], all: true, dryRun: false }, true)
 
-    const syncedNote = mockNote.mock.calls.find((call) => call[0] === 'Already synced')
-    expect(syncedNote?.[1]).toMatch(/new-skill@1.0.0/)
-    expect(syncedNote?.[1]).toMatch(/synced-skill@1.0.0/)
-    expect(String(syncedNote?.[1])).not.toMatch(/\n-/)
+    const output = mockLog.mock.calls.map((call) => String(call[0])).join('\n')
+    expect(output).toMatch(/Already synced/)
+    expect(output).toMatch(/new-skill@1.0.0/)
+    expect(output).toMatch(/synced-skill@1.0.0/)
+    expect(output).not.toMatch(/\n-/)
 
     const outro = mockOutro.mock.calls.at(-1)?.[0]
     expect(String(outro)).toMatch(/Nothing to sync/)
@@ -204,8 +209,9 @@ describe('cmdSync', () => {
     await cmdSync(makeOpts(), { root: ['/scan'], all: true, dryRun: false }, true)
 
     expect(mockCmdPublish).toHaveBeenCalledTimes(1)
-    const duplicateNote = mockNote.mock.calls.find((call) => call[0] === 'Skipped duplicate slugs')
-    expect(duplicateNote?.[1]).toMatch(/dup-skill/)
+    const output = mockLog.mock.calls.map((call) => String(call[0])).join('\n')
+    expect(output).toMatch(/Skipped duplicate slugs/)
+    expect(output).toMatch(/dup-skill/)
   })
 
   it('allows empty changelog for updates (interactive)', async () => {

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

@@ -1,7 +1,7 @@
 import { realpath } from 'node:fs/promises'
 import { homedir } from 'node:os'
 import { resolve } from 'node:path'
-import { intro, isCancel, multiselect, note, outro, text } from '@clack/prompts'
+import { intro, isCancel, multiselect, outro, text } from '@clack/prompts'
 import semver from 'semver'
 import { readGlobalConfig } from '../../config.js'
 import { apiRequest, downloadZip } from '../../http.js'
@@ -61,17 +61,17 @@ export async function cmdSync(opts: GlobalOpts, options: SyncOptions, inputAllow
     spinner.stop()
     if (scan.skills.length === 0)
       fail('No skills found (checked workdir and known Clawdis/Clawd locations)')
-    note(
+    printSection(
       `No skills in workdir. Found ${scan.skills.length} in fallback locations.`,
-      wrapNoteBody(formatList(scan.rootsWithSkills, 10)),
+      formatList(scan.rootsWithSkills, 10),
     )
   } else {
     spinner.stop()
   }
   const deduped = dedupeSkillsBySlug(scan.skills)
   const skills = deduped.skills
   if (deduped.duplicates.length > 0) {
-    note('Skipped duplicate slugs', wrapNoteBody(formatCommaList(deduped.duplicates, 16)))
+    printSection('Skipped duplicate slugs', formatCommaList(deduped.duplicates, 16))
   }
   const parsingSpinner = createSpinner('Parsing local skills')
   const locals: LocalSkill[] = []
@@ -123,23 +123,21 @@ export async function cmdSync(opts: GlobalOpts, options: SyncOptions, inputAllow
 
   if (actionable.length === 0) {
     if (synced.length > 0) {
-      note('Already synced', wrapNoteBody(formatCommaList(synced.map(formatSyncedSummary), 16)))
+      printSection('Already synced', formatCommaList(synced.map(formatSyncedSummary), 16))
     }
     outro('Nothing to sync.')
     return
   }
 
-  note(
+  printSection(
     'To sync',
-    wrapNoteBody(
-      formatBulletList(
-        actionable.map((candidate) => formatActionableLine(candidate, bump)),
-        20,
-      ),
+    formatBulletList(
+      actionable.map((candidate) => formatActionableLine(candidate, bump)),
+      20,
     ),
   )
   if (synced.length > 0) {
-    note('Already synced', wrapNoteBody(formatSyncedDisplay(synced)))
+    printSection('Already synced', formatSyncedDisplay(synced))
   }
 
   const selected = await selectToUpload(actionable, {
@@ -392,6 +390,19 @@ function formatList(values: string[], max: number) {
   return [...head, `… +${rest} more`].join('\n')
 }
 
+function printSection(title: string, body?: string) {
+  const trimmed = body?.trim()
+  if (!trimmed) {
+    console.log(title)
+    return
+  }
+  if (trimmed.includes('\n')) {
+    console.log(`\n${title}\n${trimmed}`)
+    return
+  }
+  console.log(`${title}: ${trimmed}`)
+}
+
 function abbreviatePath(value: string) {
   const home = homedir()
   if (value.startsWith(home)) return `~${value.slice(home.length)}`
@@ -449,58 +460,6 @@ function formatSyncedDisplay(synced: Candidate[]) {
   return formatCommaList(synced.map(formatSyncedSummary), 24)
 }
 
-function wrapNoteBody(text: string) {
-  const width = noteWrapWidth()
-  return text
-    .split('\n')
-    .map((line) => wrapLine(line, width))
-    .join('\n')
-}
-
-function noteWrapWidth() {
-  const columns = process.stdout.columns ?? 80
-  return Math.min(80, Math.max(20, columns - 4))
-}
-
-function wrapLine(line: string, width: number) {
-  if (line.length <= width) return line
-  if (line.startsWith('- ')) {
-    return wrapWords(line.slice(2), width - 2, '- ', '  ')
-  }
-  return wrapWords(line, width, '', '')
-}
-
-function wrapWords(text: string, width: number, firstPrefix: string, nextPrefix: string) {
-  const words = text.trim().split(/\s+/).filter(Boolean)
-  if (words.length === 0) return firstPrefix.trimEnd()
-  const lines: string[] = []
-  let current = ''
-  for (const word of words) {
-    if (word.length > width) {
-      if (current) {
-        lines.push(current)
-        current = ''
-      }
-      for (let i = 0; i < word.length; i += width) {
-        lines.push(word.slice(i, i + width))
-      }
-      continue
-    }
-    if (!current) {
-      current = word
-      continue
-    }
-    if (current.length + 1 + word.length <= width) {
-      current = `${current} ${word}`
-      continue
-    }
-    lines.push(current)
-    current = word
-  }
-  if (current) lines.push(current)
-  return lines.map((line, index) => `${index === 0 ? firstPrefix : nextPrefix}${line}`).join('\n')
-}
-
 function formatCommaList(values: string[], max: number) {
   if (values.length === 0) return ''
   if (values.length <= max) return values.join(', ')