refactor(cli): remove unused terminal-link dep and dead exports

- Remove terminal-link from dependencies (never imported)
- Remove unused exports: printSuccess, printWarning, printInfo (human.ts),
  outputTableList (table.ts)
- Remove corresponding test for printSuccess

Author: jerome-benoit

pnpm-lock.yaml

@@ -33,7 +33,6 @@
     "cli-table3": "^0.6.5",
     "commander": "^14.0.0",
     "ora": "^8.2.0",
-    "terminal-link": "^3.0.0",
     "ui-common": "workspace:*",
     "ws": "^8.20.0"
   },

ui/cli/package.json

@@ -1,18 +1,6 @@
 import chalk from 'chalk'
 import process from 'node:process'
 
-export const printSuccess = (message: string): void => {
-  process.stdout.write(chalk.green(`✓ ${message}\n`))
-}
-
 export const printError = (message: string): void => {
   process.stderr.write(chalk.red(`✗ ${message}\n`))
 }
-
-export const printWarning = (message: string): void => {
-  process.stderr.write(chalk.yellow(`⚠ ${message}\n`))
-}
-
-export const printInfo = (message: string): void => {
-  process.stderr.write(chalk.dim(`  ${message}\n`))
-}

ui/cli/src/output/human.ts

@@ -41,18 +41,3 @@ const displayGenericPayload = (payload: ResponsePayload): void => {
     process.stdout.write(chalk.green('✓ Success\n'))
   }
 }
-
-export const outputTableList = <T extends Record<string, unknown>>(
-  items: T[],
-  columns: { header: string; key: keyof T }[]
-): void => {
-  if (items.length === 0) {
-    process.stdout.write(chalk.dim('(no items)\n'))
-    return
-  }
-  const table = new Table({ head: columns.map(c => chalk.white(c.header)) })
-  for (const item of items) {
-    table.push(columns.map(c => String(item[c.key] ?? '')))
-  }
-  process.stdout.write(table.toString() + '\n')
-}

ui/cli/src/output/table.ts

@@ -3,7 +3,7 @@ import { describe, it } from 'node:test'
 import { ResponseStatus } from 'ui-common'
 
 import { createFormatter } from '../src/output/formatter.js'
-import { printError, printSuccess } from '../src/output/human.js'
+import { printError } from '../src/output/human.js'
 import { outputJson, outputJsonError } from '../src/output/json.js'
 import { outputTable } from '../src/output/table.js'
 
@@ -117,13 +117,6 @@ await describe('output formatters', async () => {
     assert.ok(output.includes('Success'))
   })
 
-  await it('should write success message via printSuccess', () => {
-    const output = captureStdout(() => {
-      printSuccess('done')
-    })
-    assert.ok(output.includes('done'))
-  })
-
   await it('should write error message via printError', () => {
     const output = captureStderr(() => {
       printError('oops')