test: adopt @doist/cli-core/testing shared helpers (#6)

Replace per-file test scaffolding with the published cli-core helpers
(cli-core#49): createTestProgram(register) collapses the 19 identical
local createProgram() definitions to a one-line wrapper, and
captureConsole(method?) replaces ~200 inline vi.spyOn(console, ...)
spies plus their manual mockRestore() calls (it auto-restores via
onTestFinished).

Test-only change across 21 files; no runtime behaviour affected.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: scottlovegrove

src/commands/account/account.test.ts

@@ -1,4 +1,4 @@
-import { Command } from 'commander'
+import { captureConsole, createTestProgram } from '@doist/cli-core/testing'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 
 const storeMocks = vi.hoisted(() => ({
@@ -28,12 +28,7 @@ import { TOKEN_ENV_VAR } from '../../lib/auth.js'
 import { CliError } from '../../lib/errors.js'
 import { registerAccountCommand } from './index.js'
 
-function createProgram() {
-    const program = new Command()
-    program.exitOverride()
-    registerAccountCommand(program)
-    return program
-}
+const createProgram = () => createTestProgram(registerAccountCommand)
 
 /**
  * Seed an in-memory store that mirrors the real keyring store closely enough
@@ -68,13 +63,11 @@ describe('account command', () => {
 
     beforeEach(() => {
         vi.clearAllMocks()
-        consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
-        errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
+        consoleSpy = captureConsole('log')
+        errorSpy = captureConsole('error')
     })
 
     afterEach(() => {
-        consoleSpy.mockRestore()
-        errorSpy.mockRestore()
         vi.unstubAllEnvs()
     })
 

src/commands/auth/auth.test.ts

@@ -1,3 +1,4 @@
+import { captureConsole, createTestProgram } from '@doist/cli-core/testing'
 import { Command } from 'commander'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 
@@ -89,12 +90,7 @@ const mockGetAuthMetadata = vi.mocked(getAuthMetadata)
 const mockCreateWrappedCommsClient = vi.mocked(createWrappedCommsClient)
 const mockAttachLoginCommand = vi.mocked(attachLoginCommand)
 
-function createProgram() {
-    const program = new Command()
-    program.exitOverride()
-    registerAuthCommand(program)
-    return program
-}
+const createProgram = () => createTestProgram(registerAuthCommand)
 
 const TEST_USER: User = {
     id: 1,
@@ -114,13 +110,8 @@ describe('auth command', () => {
         vi.clearAllMocks()
 
         // Mock console.log to capture output
-        consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
-        errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
-    })
-
-    afterEach(() => {
-        consoleSpy.mockRestore()
-        errorSpy.mockRestore()
+        consoleSpy = captureConsole('log')
+        errorSpy = captureConsole('error')
     })
 
     const STORED_ACCOUNT: CommsAccount = {

src/commands/changelog.test.ts

@@ -1,4 +1,4 @@
-import { Command } from 'commander'
+import { captureConsole, createTestProgram } from '@doist/cli-core/testing'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 
 vi.mock('node:fs/promises')
@@ -32,18 +32,13 @@ const SAMPLE_CHANGELOG = `# Changelog
 * prior release with a level-2 heading
 `
 
-function createProgram() {
-    const program = new Command()
-    program.exitOverride()
-    registerChangelogCommand(program)
-    return program
-}
+const createProgram = () => createTestProgram(registerChangelogCommand)
 
 describe('changelog wrapper', () => {
     let logSpy: ReturnType<typeof vi.spyOn>
 
     beforeEach(() => {
-        logSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
+        logSpy = captureConsole('log')
     })
 
     afterEach(() => {