@alumbwe/anvil 1.0.0 → 1.0.2

package/src/commands/__tests__/diagnostics-command.test.ts

@@ -1,35 +0,0 @@
-import { describe, expect, mock, test } from 'bun:test'
-
-import { findCommand } from '../command-registry'
-
-import type { RouterParams } from '../command-registry'
-import type { ChatMessage } from '../../types/chat'
-
-describe('/diagnostics command', () => {
-  test('adds runtime process diagnostics to the chat', () => {
-    const command = findCommand('diagnostics')
-    expect(command).toBeDefined()
-
-    let messages: ChatMessage[] = []
-    const saveToHistory = mock(() => {})
-    const setInputValue = mock(() => {})
-    command!.handler(
-      {
-        inputValue: '/diagnostics',
-        setMessages: (update: Parameters<RouterParams['setMessages']>[0]) => {
-          messages = typeof update === 'function' ? update(messages) : update
-        },
-        saveToHistory,
-        setInputValue,
-      } as unknown as RouterParams,
-      '',
-    )
-
-    const output = messages.at(-1)?.content ?? ''
-    expect(output).toContain('process diagnostics')
-    expect(output).toContain(`Platform: ${process.platform} ${process.arch}`)
-    expect(output).toContain(`CLI binary: PID ${process.pid}`)
-    expect(saveToHistory).toHaveBeenCalledWith('/diagnostics')
-    expect(setInputValue).toHaveBeenCalled()
-  })
-})

package/src/commands/__tests__/freebuff-command-aliases.test.ts

@@ -1,56 +0,0 @@
-import { describe, expect, test } from 'bun:test'
-
-describe('freebuff command aliases', () => {
-  test('/model aliases /end-session in freebuff', () => {
-    const slashCommandsUrl = new URL(
-      '../../data/slash-commands.ts',
-      import.meta.url,
-    ).href
-    const commandRegistryUrl = new URL(
-      '../command-registry.ts',
-      import.meta.url,
-    ).href
-
-    const result = Bun.spawnSync({
-      cmd: [
-        'bun',
-        '--eval',
-        `
-          import { SLASH_COMMANDS } from ${JSON.stringify(slashCommandsUrl)}
-          import { findCommand } from ${JSON.stringify(commandRegistryUrl)}
-
-          const endSession = SLASH_COMMANDS.find((cmd) => cmd.id === 'end-session')
-          if (!endSession) throw new Error('end-session slash command missing')
-          if (!endSession.aliases?.includes('model')) {
-            throw new Error('end-session slash command is missing model alias')
-          }
-
-          const modelCommand = findCommand('model')
-          if (!modelCommand) throw new Error('model command alias missing')
-          if (modelCommand.name !== 'end-session') {
-            throw new Error('model alias did not resolve to end-session')
-          }
-        `,
-      ],
-      cwd: process.cwd(),
-      env: {
-        ...process.env,
-        FREEBUFF_MODE: 'true',
-        NODE_ENV: 'test',
-        NEXT_PUBLIC_CB_ENVIRONMENT: 'test',
-        NEXT_PUBLIC_ANVIL_APP_URL: 'https://app.codebuff.test',
-        NEXT_PUBLIC_SUPPORT_EMAIL: 'support@codebuff.test',
-        NEXT_PUBLIC_POSTHOG_API_KEY: 'phc_test_key',
-        NEXT_PUBLIC_POSTHOG_HOST_URL: 'https://posthog.codebuff.test',
-        NEXT_PUBLIC_STRIPE_PUBLISHABLE_KEY: 'pk_test_123',
-        NEXT_PUBLIC_STRIPE_CUSTOMER_PORTAL: 'https://stripe.codebuff.test',
-        NEXT_PUBLIC_WEB_PORT: '3000',
-      },
-      stderr: 'pipe',
-      stdout: 'pipe',
-    })
-
-    const stderr = new TextDecoder().decode(result.stderr)
-    expect(result.exitCode, stderr).toBe(0)
-  }, 15_000)
-})

package/src/commands/__tests__/image.test.ts

@@ -1,99 +0,0 @@
-import { describe, test, expect } from 'bun:test'
-
-/**
- * Tests for the handleImageCommand argument parsing behavior.
- *
- * These tests verify the parsing logic independently of the actual
- * validateAndAddImage implementation by testing the parsing function directly.
- */
-
-// Extract the parsing logic that handleImageCommand uses
-// New simplified implementation: split on whitespace
-function parseImageCommandArgs(args: string): {
-  imagePath: string | null
-  message: string
-} {
-  const [imagePath, ...rest] = args.trim().split(/\s+/)
-
-  if (!imagePath) {
-    return { imagePath: null, message: '' }
-  }
-
-  return { imagePath, message: rest.join(' ') }
-}
-
-describe('handleImageCommand parsing', () => {
-  describe('argument parsing', () => {
-    test('parses image path only', () => {
-      const result = parseImageCommandArgs('./screenshot.png')
-      expect(result.imagePath).toBe('./screenshot.png')
-      expect(result.message).toBe('')
-    })
-
-    test('parses image path with message', () => {
-      const result = parseImageCommandArgs(
-        './screenshot.png please analyze this',
-      )
-      expect(result.imagePath).toBe('./screenshot.png')
-      expect(result.message).toBe('please analyze this')
-    })
-
-    test('parses image path with multi-word message', () => {
-      const result = parseImageCommandArgs(
-        './image.jpg what is in this picture?',
-      )
-      expect(result.imagePath).toBe('./image.jpg')
-      expect(result.message).toBe('what is in this picture?')
-    })
-
-    test('handles absolute paths with message', () => {
-      const result = parseImageCommandArgs('/path/to/file.png describe the UI')
-      expect(result.imagePath).toBe('/path/to/file.png')
-      expect(result.message).toBe('describe the UI')
-    })
-
-    test('trims whitespace from input', () => {
-      const result = parseImageCommandArgs('  ./image.png  ')
-      expect(result.imagePath).toBe('./image.png')
-      expect(result.message).toBe('')
-    })
-
-    test('handles multiple spaces between path and message', () => {
-      const result = parseImageCommandArgs('./image.png    hello world')
-      expect(result.imagePath).toBe('./image.png')
-      // The regex only captures content after the first whitespace group
-      expect(result.message).toBe('hello world')
-    })
-  })
-
-  describe('invalid input handling', () => {
-    test('returns null imagePath for empty input', () => {
-      const result = parseImageCommandArgs('')
-      expect(result.imagePath).toBeNull()
-      expect(result.message).toBe('')
-    })
-
-    test('returns null imagePath for whitespace-only input', () => {
-      const result = parseImageCommandArgs('   ')
-      expect(result.imagePath).toBeNull()
-      expect(result.message).toBe('')
-    })
-  })
-
-  describe('edge cases', () => {
-    test('handles filenames with extensions', () => {
-      const result = parseImageCommandArgs('image.jpeg')
-      expect(result.imagePath).toBe('image.jpeg')
-    })
-
-    test('handles relative paths', () => {
-      const result = parseImageCommandArgs('../screenshots/test.png')
-      expect(result.imagePath).toBe('../screenshots/test.png')
-    })
-
-    test('handles tilde paths', () => {
-      const result = parseImageCommandArgs('~/Downloads/image.png')
-      expect(result.imagePath).toBe('~/Downloads/image.png')
-    })
-  })
-})

package/src/commands/__tests__/init.test.ts

@@ -1,456 +0,0 @@
-import * as fs from 'fs'
-import path from 'path'
-
-import {
-  describe,
-  test,
-  expect,
-  beforeEach,
-  afterEach,
-  mock,
-  spyOn,
-} from 'bun:test'
-
-import * as projectFiles from '../../project-files'
-import { handleInitializationFlowLocally } from '../init'
-
-import type { ChatMessage } from '../../types/chat'
-
-/** Helper to extract text content from ChatMessages returned by getSystemMessage */
-const getMessageText = (messages: ChatMessage[]): string => {
-  return messages
-    .map((m) => {
-      // ChatMessage has content as a string, not an array
-      if (typeof m.content === 'string') {
-        return m.content
-      }
-      return ''
-    })
-    .join('')
-}
-
-describe('handleInitializationFlowLocally', () => {
-  const TEST_PROJECT_ROOT = '/test/project'
-  const KNOWLEDGE_FILE_NAME = 'AGENTS.md'
-
-  let existsSyncSpy: ReturnType<typeof spyOn>
-  let writeFileSyncSpy: ReturnType<typeof spyOn>
-  let mkdirSyncSpy: ReturnType<typeof spyOn>
-  let getProjectRootSpy: ReturnType<typeof spyOn>
-
-  beforeEach(() => {
-    // Mock getProjectRoot
-    getProjectRootSpy = spyOn(projectFiles, 'getProjectRoot').mockReturnValue(
-      TEST_PROJECT_ROOT,
-    )
-
-    // Mock fs functions
-    existsSyncSpy = spyOn(fs, 'existsSync').mockReturnValue(false)
-    writeFileSyncSpy = spyOn(fs, 'writeFileSync').mockImplementation(() => {})
-    mkdirSyncSpy = spyOn(fs, 'mkdirSync').mockImplementation(() => undefined)
-  })
-
-  afterEach(() => {
-    mock.restore()
-  })
-
-  describe('knowledge file creation', () => {
-    test('creates AGENTS.md when it does not exist', () => {
-      existsSyncSpy.mockImplementation((_p: string) => false)
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      expect(writeFileSyncSpy).toHaveBeenCalledWith(
-        path.join(TEST_PROJECT_ROOT, KNOWLEDGE_FILE_NAME),
-        expect.stringContaining('# Project knowledge'),
-      )
-
-      // Check message indicates creation
-      const messages = postUserMessage([])
-      expect(messages.length).toBeGreaterThan(0)
-      expect(getMessageText(messages)).toContain('✅ Created `AGENTS.md`')
-    })
-
-    test('skips AGENTS.md creation when it already exists', () => {
-      existsSyncSpy.mockImplementation((p: unknown) =>
-        p === path.join(TEST_PROJECT_ROOT, KNOWLEDGE_FILE_NAME),
-      )
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      // writeFileSync should not be called for knowledge.md
-      const knowledgeWriteCalls = writeFileSyncSpy.mock.calls.filter(
-        (call: unknown[]) => call[0] === path.join(TEST_PROJECT_ROOT, KNOWLEDGE_FILE_NAME),
-      )
-      expect(knowledgeWriteCalls.length).toBe(0)
-
-      // Check message indicates file already exists
-      const messages = postUserMessage([])
-      expect(getMessageText(messages)).toContain('📋 `AGENTS.md` already exists')
-    })
-  })
-
-  describe('.agents directory creation', () => {
-    test('creates .agents directory when it does not exist', () => {
-      existsSyncSpy.mockReturnValue(false)
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      expect(mkdirSyncSpy).toHaveBeenCalledWith(
-        path.join(TEST_PROJECT_ROOT, '.agents'),
-        { recursive: true },
-      )
-
-      const messages = postUserMessage([])
-      expect(getMessageText(messages)).toContain('✅ Created `.agents/`')
-    })
-
-    test('skips .agents directory creation when it already exists', () => {
-      existsSyncSpy.mockImplementation((p: unknown) =>
-        p === path.join(TEST_PROJECT_ROOT, '.agents'),
-      )
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      // mkdirSync should not be called for .agents directory
-      const agentsDirCalls = mkdirSyncSpy.mock.calls.filter(
-        (call: unknown[]) => call[0] === path.join(TEST_PROJECT_ROOT, '.agents'),
-      )
-      expect(agentsDirCalls.length).toBe(0)
-
-      const messages = postUserMessage([])
-      expect(getMessageText(messages)).toContain('📋 `.agents/` already exists')
-    })
-  })
-
-  describe('.agents/types directory creation', () => {
-    test('creates .agents/types directory when it does not exist', () => {
-      existsSyncSpy.mockReturnValue(false)
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      expect(mkdirSyncSpy).toHaveBeenCalledWith(
-        path.join(TEST_PROJECT_ROOT, '.agents', 'types'),
-        { recursive: true },
-      )
-
-      const messages = postUserMessage([])
-      expect(getMessageText(messages)).toContain('✅ Created `.agents/types/`')
-    })
-
-    test('skips .agents/types directory creation when it already exists', () => {
-      existsSyncSpy.mockImplementation((p: unknown) => {
-        // .agents exists, .agents/types exists
-        return (
-          p === path.join(TEST_PROJECT_ROOT, '.agents') ||
-          p === path.join(TEST_PROJECT_ROOT, '.agents', 'types')
-        )
-      })
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      // mkdirSync should not be called for .agents/types directory
-      const typesDirCalls = mkdirSyncSpy.mock.calls.filter(
-        (call: unknown[]) => call[0] === path.join(TEST_PROJECT_ROOT, '.agents', 'types'),
-      )
-      expect(typesDirCalls.length).toBe(0)
-
-      const messages = postUserMessage([])
-      expect(getMessageText(messages)).toContain('📋 `.agents/types/` already exists')
-    })
-  })
-
-  describe('type file copying', () => {
-    test('copies type files when they do not exist', () => {
-      existsSyncSpy.mockReturnValue(false)
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      // Check that writeFileSync was called for type files
-      const typeFiles = ['agent-definition.ts', 'tools.ts', 'util-types.ts']
-      for (const fileName of typeFiles) {
-        const fileCalls = writeFileSyncSpy.mock.calls.filter((call: unknown[]) =>
-          (call[0] as string).endsWith(fileName),
-        )
-        expect(fileCalls.length).toBe(1)
-      }
-
-      const messages = postUserMessage([])
-      const messageContent = getMessageText(messages)
-
-      // Should have success messages for copied files
-      expect(messageContent).toContain('`.agents/types/agent-definition.ts`')
-      expect(messageContent).toContain('`.agents/types/tools.ts`')
-      expect(messageContent).toContain('`.agents/types/util-types.ts`')
-    })
-
-    test('skips type files that already exist', () => {
-      const typesDir = path.join(TEST_PROJECT_ROOT, '.agents', 'types')
-      existsSyncSpy.mockImplementation((p: unknown) => {
-        // Only agent-definition.ts exists
-        return p === path.join(typesDir, 'agent-definition.ts')
-      })
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      // agent-definition.ts should NOT be written
-      const agentDefCalls = writeFileSyncSpy.mock.calls.filter(
-        (call: unknown[]) => (call[0] as string).endsWith('agent-definition.ts'),
-      )
-      expect(agentDefCalls.length).toBe(0)
-
-      // tools.ts and util-types.ts should be written
-      const toolsCalls = writeFileSyncSpy.mock.calls.filter((call: unknown[]) =>
-        (call[0] as string).endsWith('tools.ts'),
-      )
-      expect(toolsCalls.length).toBe(1)
-
-      const utilTypesCalls = writeFileSyncSpy.mock.calls.filter((call: unknown[]) =>
-        (call[0] as string).endsWith('util-types.ts'),
-      )
-      expect(utilTypesCalls.length).toBe(1)
-
-      const messages = postUserMessage([])
-      expect(getMessageText(messages)).toContain(
-        '📋 `.agents/types/agent-definition.ts` already exists',
-      )
-    })
-  })
-
-  describe('message accumulation', () => {
-    test('returns multiple messages for all operations', () => {
-      existsSyncSpy.mockReturnValue(false)
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      const messages = postUserMessage([])
-
-      // Should have messages for:
-      // 1. AGENTS.md creation
-      // 2. .agents/ creation
-      // 3. .agents/types/ creation
-      // 4-6. Three type file copies
-      expect(messages.length).toBeGreaterThanOrEqual(6)
-    })
-
-    test('preserves previous messages in postUserMessage', () => {
-      existsSyncSpy.mockReturnValue(false)
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      // ChatMessage has content as a string, not an array
-      const previousMessages: ChatMessage[] = [
-        {
-          id: 'user-123',
-          variant: 'user',
-          content: 'Previous message',
-          timestamp: '2024-01-01T00:00:00Z',
-        },
-      ]
-
-      const messages = postUserMessage(previousMessages)
-
-      // First message should be the previous one
-      expect(messages[0]).toEqual(previousMessages[0])
-      // Should have additional messages
-      expect(messages.length).toBeGreaterThan(1)
-    })
-  })
-
-  describe('error handling', () => {
-    test('handles writeFileSync errors for type files gracefully', () => {
-      existsSyncSpy.mockReturnValue(false)
-      writeFileSyncSpy.mockImplementation((p: unknown) => {
-        if ((p as string).endsWith('tools.ts')) {
-          throw new Error('Permission denied')
-        }
-      })
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      const messages = postUserMessage([])
-      const messageContent = getMessageText(messages)
-
-      // Should have error message for tools.ts
-      expect(messageContent).toContain('⚠️ Failed to copy `.agents/types/tools.ts`')
-      expect(messageContent).toContain('Permission denied')
-    })
-
-    test('handles writeFileSync errors for AGENTS.md gracefully', () => {
-      existsSyncSpy.mockReturnValue(false)
-      writeFileSyncSpy.mockImplementation((p: unknown) => {
-        if ((p as string).endsWith(KNOWLEDGE_FILE_NAME)) {
-          throw new Error('Disk full')
-        }
-      })
-
-      // The function should throw when the AGENTS.md write fails
-      // since it is not wrapped in try-catch
-      expect(() => handleInitializationFlowLocally()).toThrow('Disk full')
-    })
-
-    test('handles mkdirSync errors for .agents directory gracefully', () => {
-      existsSyncSpy.mockReturnValue(false)
-      mkdirSyncSpy.mockImplementation((p: unknown) => {
-        if ((p as string).endsWith('.agents')) {
-          throw new Error('Cannot create directory')
-        }
-        return undefined
-      })
-
-      // The function should throw when .agents directory creation fails
-      // since mkdirSync is not wrapped in try-catch
-      expect(() => handleInitializationFlowLocally()).toThrow('Cannot create directory')
-    })
-
-    test('handles mkdirSync errors for .agents/types directory gracefully', () => {
-      existsSyncSpy.mockImplementation((p: unknown) => {
-        // .agents exists but .agents/types doesn't
-        return p === path.join(TEST_PROJECT_ROOT, '.agents')
-      })
-      mkdirSyncSpy.mockImplementation((p: unknown) => {
-        if ((p as string).endsWith('types')) {
-          throw new Error('Permission denied for types dir')
-        }
-        return undefined
-      })
-
-      // The function should throw when .agents/types directory creation fails
-      expect(() => handleInitializationFlowLocally()).toThrow('Permission denied for types dir')
-    })
-
-    test('continues copying other files when one type file fails', () => {
-      existsSyncSpy.mockReturnValue(false)
-      writeFileSyncSpy.mockImplementation((p: unknown) => {
-        // Only fail for agent-definition.ts
-        if ((p as string).endsWith('agent-definition.ts')) {
-          throw new Error('File locked')
-        }
-      })
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-      const messages = postUserMessage([])
-      const messageContent = getMessageText(messages)
-
-      // Should have error for agent-definition.ts
-      expect(messageContent).toContain('⚠️ Failed to copy `.agents/types/agent-definition.ts`')
-      expect(messageContent).toContain('File locked')
-
-      // But should still succeed for tools.ts and util-types.ts
-      expect(messageContent).toContain('✅ Copied `.agents/types/tools.ts`')
-      expect(messageContent).toContain('✅ Copied `.agents/types/util-types.ts`')
-    })
-
-    test('handles non-Error exceptions in type file copying', () => {
-      existsSyncSpy.mockReturnValue(false)
-      writeFileSyncSpy.mockImplementation((p: unknown) => {
-        if ((p as string).endsWith('util-types.ts')) {
-          // Throw a non-Error value
-          throw 'string error'
-        }
-      })
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-      const messages = postUserMessage([])
-      const messageContent = getMessageText(messages)
-
-      // Should handle non-Error exceptions gracefully
-      expect(messageContent).toContain('⚠️ Failed to copy `.agents/types/util-types.ts`')
-      expect(messageContent).toContain('string error')
-    })
-
-    test('handles null/undefined exceptions in type file copying', () => {
-      existsSyncSpy.mockReturnValue(false)
-      writeFileSyncSpy.mockImplementation((p: unknown) => {
-        if ((p as string).endsWith('tools.ts')) {
-          // Throw null
-          throw null
-        }
-      })
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-      const messages = postUserMessage([])
-      const messageContent = getMessageText(messages)
-
-      // Should handle null exceptions with 'Unknown' fallback
-      expect(messageContent).toContain('⚠️ Failed to copy `.agents/types/tools.ts`')
-      expect(messageContent).toContain('Unknown')
-    })
-  })
-
-  describe('integration scenarios', () => {
-    test('handles partial initialization state correctly', () => {
-      const agentsDir = path.join(TEST_PROJECT_ROOT, '.agents')
-      const typesDir = path.join(agentsDir, 'types')
-
-      // Scenario: AGENTS.md exists, .agents exists, but .agents/types and type files don't exist
-      existsSyncSpy.mockImplementation((p: unknown) => {
-        return (
-          p === path.join(TEST_PROJECT_ROOT, KNOWLEDGE_FILE_NAME) ||
-          p === agentsDir
-        )
-      })
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      // Should NOT create AGENTS.md
-      const knowledgeWriteCalls = writeFileSyncSpy.mock.calls.filter(
-        (call: unknown[]) => call[0] === path.join(TEST_PROJECT_ROOT, KNOWLEDGE_FILE_NAME),
-      )
-      expect(knowledgeWriteCalls.length).toBe(0)
-
-      // Should NOT create .agents directory
-      const agentsDirCalls = mkdirSyncSpy.mock.calls.filter(
-        (call: unknown[]) => call[0] === agentsDir,
-      )
-      expect(agentsDirCalls.length).toBe(0)
-
-      // Should create .agents/types directory
-      const typesDirCalls = mkdirSyncSpy.mock.calls.filter(
-        (call: unknown[]) => call[0] === typesDir,
-      )
-      expect(typesDirCalls.length).toBe(1)
-
-      // Should copy type files
-      const typeFileCalls = writeFileSyncSpy.mock.calls.filter((call: unknown[]) =>
-        (call[0] as string).startsWith(typesDir),
-      )
-      expect(typeFileCalls.length).toBe(3)
-
-      const messages = postUserMessage([])
-      const messageContent = getMessageText(messages)
-
-      expect(messageContent).toContain('📋 `AGENTS.md` already exists')
-      expect(messageContent).toContain('📋 `.agents/` already exists')
-      expect(messageContent).toContain('✅ Created `.agents/types/`')
-    })
-
-    test('handles fully initialized project correctly', () => {
-      // Everything exists
-      existsSyncSpy.mockReturnValue(true)
-
-      const { postUserMessage } = handleInitializationFlowLocally()
-
-      // Nothing should be created
-      expect(writeFileSyncSpy).not.toHaveBeenCalled()
-      expect(mkdirSyncSpy).not.toHaveBeenCalled()
-
-      const messages = postUserMessage([])
-      const messageContent = getMessageText(messages)
-
-      // All messages should indicate existing files
-      expect(messageContent).toContain('📋 `AGENTS.md` already exists')
-      expect(messageContent).toContain('📋 `.agents/` already exists')
-      expect(messageContent).toContain('📋 `.agents/types/` already exists')
-      expect(messageContent).toContain(
-        '📋 `.agents/types/agent-definition.ts` already exists',
-      )
-      expect(messageContent).toContain(
-        '📋 `.agents/types/tools.ts` already exists',
-      )
-      expect(messageContent).toContain(
-        '📋 `.agents/types/util-types.ts` already exists',
-      )
-    })
-  })
-})