@gguf/coder 0.3.1 → 0.3.3

package/source/ai-sdk-client-error-handling.spec.ts

@@ -1,186 +0,0 @@
-import test from 'ava';
-import {parseAPIError} from '@/ai-sdk-client';
-
-// Tests for parseAPIError function
-// Now using the actual exported function instead of a duplicated copy
-
-test('parseAPIError - handles Ollama unmarshal error from issue #87', t => {
-	const error = new Error(
-		"RetryError [AI_RetryError]: Failed after 3 attempts. Last error: unmarshal: invalid character '{' after top-level value",
-	);
-
-	const result = parseAPIError(error);
-
-	t.true(result.includes('Ollama server error'));
-	t.true(result.includes('malformed JSON'));
-	t.true(result.includes('Restart Ollama'));
-	t.true(result.includes('Re-pull the model'));
-	t.true(result.includes('Check Ollama logs'));
-	t.true(result.includes('Try a different model'));
-	t.true(result.includes('Original error:'));
-});
-
-test('parseAPIError - handles unmarshal error without retry wrapper', t => {
-	const error = new Error(
-		"unmarshal: invalid character '{' after top-level value",
-	);
-
-	const result = parseAPIError(error);
-
-	t.true(result.includes('Ollama server error'));
-	t.true(result.includes('malformed JSON'));
-});
-
-test('parseAPIError - handles 500 error with invalid character (status code takes precedence)', t => {
-	// This test verifies that HTTP status codes are parsed FIRST,
-	// so a 500 error with "invalid character" in the message is treated
-	// as a server error, not an Ollama-specific error
-	const error = new Error(
-		"500 Internal Server Error: invalid character 'x' after top-level value",
-	);
-
-	const result = parseAPIError(error);
-
-	// Status code parsing takes precedence over Ollama-specific pattern matching
-	t.is(result, "Server error: invalid character 'x' after top-level value");
-});
-
-test('parseAPIError - handles 500 error without JSON parsing issue', t => {
-	const error = new Error(
-		'500 Internal Server Error: database connection failed',
-	);
-
-	const result = parseAPIError(error);
-
-	t.is(result, 'Server error: database connection failed');
-});
-
-test('parseAPIError - handles 404 error', t => {
-	const error = new Error('404 Not Found: model not available');
-
-	const result = parseAPIError(error);
-
-	t.is(
-		result,
-		'Model not found: The requested model may not exist or is unavailable',
-	);
-});
-
-test('parseAPIError - handles connection refused', t => {
-	const error = new Error('ECONNREFUSED: Connection refused');
-
-	const result = parseAPIError(error);
-
-	t.is(result, 'Connection failed: Unable to reach the model server');
-});
-
-test('parseAPIError - handles timeout error', t => {
-	const error = new Error('Request timeout: ETIMEDOUT');
-
-	const result = parseAPIError(error);
-
-	t.is(result, 'Request timed out: The model took too long to respond');
-});
-
-test('parseAPIError - handles non-Error objects', t => {
-	const result = parseAPIError('string error');
-
-	t.is(result, 'An unknown error occurred while communicating with the model');
-});
-
-test('parseAPIError - handles context length errors', t => {
-	const error = new Error('context length exceeded');
-
-	const result = parseAPIError(error);
-
-	// Use exact assertion instead of OR condition
-	t.is(
-		result,
-		'Context too large: Please reduce the conversation length or message size',
-	);
-});
-
-test('parseAPIError - handles too many tokens errors', t => {
-	const error = new Error('too many tokens in the request');
-
-	const result = parseAPIError(error);
-
-	t.is(
-		result,
-		'Context too large: Please reduce the conversation length or message size',
-	);
-});
-
-test('parseAPIError - handles 400 with context length in message', t => {
-	const error = new Error('400 Bad Request: context length exceeded');
-
-	const result = parseAPIError(error);
-
-	// The 400 status code pattern matches first, so we get the full message
-	t.is(result, 'Bad request: context length exceeded');
-});
-
-test('parseAPIError - handles 401 authentication error', t => {
-	const error = new Error('401 Unauthorized: Invalid API key');
-
-	const result = parseAPIError(error);
-
-	t.is(result, 'Authentication failed: Invalid API key or credentials');
-});
-
-test('parseAPIError - handles 403 forbidden error', t => {
-	const error = new Error('403 Forbidden: Access denied');
-
-	const result = parseAPIError(error);
-
-	t.is(result, 'Access forbidden: Check your API permissions');
-});
-
-test('parseAPIError - handles 429 rate limit error', t => {
-	const error = new Error('429 Too Many Requests: Rate limit exceeded');
-
-	const result = parseAPIError(error);
-
-	t.is(
-		result,
-		'Rate limit exceeded: Too many requests. Please wait and try again',
-	);
-});
-
-test('parseAPIError - handles 502 bad gateway error', t => {
-	const error = new Error('502 Bad Gateway: upstream error');
-
-	const result = parseAPIError(error);
-
-	t.is(result, 'Server error: upstream error');
-});
-
-test('parseAPIError - handles 503 service unavailable error', t => {
-	const error = new Error('503 Service Unavailable: server overloaded');
-
-	const result = parseAPIError(error);
-
-	t.is(result, 'Server error: server overloaded');
-});
-
-test('parseAPIError - handles reduce tokens message', t => {
-	const error = new Error('Please reduce the number of tokens in your request');
-
-	const result = parseAPIError(error);
-
-	t.is(
-		result,
-		'Too many tokens: Please shorten your message or clear conversation history',
-	);
-});
-
-test('parseAPIError - cleans up unknown errors', t => {
-	const error = new Error(
-		'Error: Something unexpected happened\nWith more details',
-	);
-
-	const result = parseAPIError(error);
-
-	// Should strip "Error: " prefix and only return first line
-	t.is(result, 'Something unexpected happened');
-});

package/source/ai-sdk-client-maxretries.spec.ts

@@ -1,114 +0,0 @@
-import test from 'ava';
-import {AISDKClient} from '@/ai-sdk-client';
-import type {AIProviderConfig} from '@/types/config';
-
-// Tests for maxRetries configuration
-// Now tests actual AISDKClient instantiation and behavior
-
-test('AISDKClient - maxRetries defaults to 2 when not specified', t => {
-	const config: AIProviderConfig = {
-		name: 'TestProvider',
-		type: 'openai-compatible',
-		models: ['test-model'],
-		config: {
-			baseURL: 'http://localhost:11434/v1',
-			apiKey: 'test-key',
-		},
-	};
-
-	const client = new AISDKClient(config);
-
-	// Verify the client's internal maxRetries is set to default of 2
-	t.is(client.getMaxRetries(), 2);
-});
-
-test('AISDKClient - maxRetries respects custom value', t => {
-	const config: AIProviderConfig = {
-		name: 'TestProvider',
-		type: 'openai-compatible',
-		models: ['test-model'],
-		maxRetries: 5,
-		config: {
-			baseURL: 'http://localhost:11434/v1',
-			apiKey: 'test-key',
-		},
-	};
-
-	const client = new AISDKClient(config);
-
-	// Verify the client uses the custom maxRetries value
-	t.is(client.getMaxRetries(), 5);
-});
-
-test('AISDKClient - maxRetries can be set to 0 to disable retries', t => {
-	// Important: This test verifies that 0 is treated as a valid value,
-	// not as falsy (which would incorrectly default to 2)
-	const config: AIProviderConfig = {
-		name: 'TestProvider',
-		type: 'openai-compatible',
-		models: ['test-model'],
-		maxRetries: 0,
-		config: {
-			baseURL: 'http://localhost:11434/v1',
-			apiKey: 'test-key',
-		},
-	};
-
-	const client = new AISDKClient(config);
-
-	// Verify that 0 is respected (nullish coalescing handles this correctly)
-	t.is(client.getMaxRetries(), 0);
-});
-
-test('AISDKClient - maxRetries handles value of 1', t => {
-	const config: AIProviderConfig = {
-		name: 'TestProvider',
-		type: 'openai-compatible',
-		models: ['test-model'],
-		maxRetries: 1,
-		config: {
-			baseURL: 'http://localhost:11434/v1',
-			apiKey: 'test-key',
-		},
-	};
-
-	const client = new AISDKClient(config);
-
-	t.is(client.getMaxRetries(), 1);
-});
-
-test('AIProviderConfig type - includes maxRetries in interface', t => {
-	// Compile-time test that maxRetries is part of the interface
-	const config: AIProviderConfig = {
-		name: 'TestProvider',
-		type: 'openai-compatible',
-		models: ['test-model'],
-		maxRetries: 3,
-		config: {
-			baseURL: 'http://localhost:11434/v1',
-		},
-	};
-
-	// TypeScript should not complain about maxRetries property
-	t.is(typeof config.maxRetries, 'number');
-	t.true('maxRetries' in config);
-});
-
-test('AISDKClient - undefined maxRetries uses default', t => {
-	// Explicitly set to undefined to test fallback behavior
-	const config: AIProviderConfig = {
-		name: 'TestProvider',
-		type: 'openai-compatible',
-		models: ['test-model'],
-		maxRetries: undefined,
-		config: {
-			baseURL: 'http://localhost:11434/v1',
-			apiKey: 'test-key',
-		},
-	};
-
-	const client = new AISDKClient(config);
-
-	// Verify undefined falls back to default of 2
-	t.is(client.getMaxRetries(), 2);
-});

package/source/ai-sdk-client-preparestep.spec.ts

@@ -1,279 +0,0 @@
-import test from 'ava';
-import type {ModelMessage} from 'ai';
-import {isEmptyAssistantMessage, type TestableMessage} from '@/ai-sdk-client';
-
-// Tests for prepareStep message filtering logic
-// This simulates the filtering logic used in AISDKClient.chat() to prevent:
-// 1. "400 Bad Request: Assistant message must have either content or tool_calls"
-// 2. "400 Bad Request: Unexpected role 'tool' after role 'user'"
-
-/**
- * Simulates the prepareStep filtering logic from ai-sdk-client.ts
- * This is extracted for testing purposes
- */
-function simulatePrepareStepFiltering(
-	messages: ModelMessage[],
-): ModelMessage[] {
-	const filteredMessages: ModelMessage[] = [];
-	const indicesToSkip = new Set<number>();
-
-	// First pass: identify empty assistant messages and their orphaned tool results
-	for (let i = 0; i < messages.length; i++) {
-		if (isEmptyAssistantMessage(messages[i] as unknown as TestableMessage)) {
-			indicesToSkip.add(i);
-
-			// Mark any immediately following tool messages as orphaned
-			let j = i + 1;
-			while (j < messages.length && messages[j].role === 'tool') {
-				indicesToSkip.add(j);
-				j++;
-			}
-		}
-	}
-
-	// Second pass: build filtered array
-	for (let i = 0; i < messages.length; i++) {
-		if (!indicesToSkip.has(i)) {
-			filteredMessages.push(messages[i]);
-		}
-	}
-
-	return filteredMessages;
-}
-
-// ============================================================================
-// Basic filtering tests
-// ============================================================================
-
-test('prepareStep - keeps valid messages unchanged', t => {
-	const messages: ModelMessage[] = [
-		{role: 'user', content: 'Hello'},
-		{role: 'assistant', content: 'Hi there!'},
-	];
-
-	const filtered = simulatePrepareStepFiltering(messages);
-	t.deepEqual(filtered, messages);
-});
-
-test('prepareStep - filters out empty assistant message', t => {
-	const messages: ModelMessage[] = [
-		{role: 'user', content: 'Hello'},
-		{role: 'assistant', content: ''},
-		{role: 'user', content: 'Are you there?'},
-	];
-
-	const filtered = simulatePrepareStepFiltering(messages);
-	t.is(filtered.length, 2);
-	t.is(filtered[0].role, 'user');
-	t.is(filtered[1].role, 'user');
-});
-
-// ============================================================================
-// Orphaned tool message tests (critical for fixing the bug)
-// ============================================================================
-
-test('prepareStep - filters out orphaned tool messages after empty assistant', t => {
-	// This is the scenario that caused the bug:
-	// user -> empty assistant -> tool -> tool
-	// After filtering empty assistant, we'd have: user -> tool -> tool (INVALID!)
-	const messages: ModelMessage[] = [
-		{role: 'user', content: 'Read a file'},
-		{role: 'assistant', content: ''}, // Empty assistant
-		{
-			role: 'tool',
-			content: [
-				{
-					type: 'tool-result',
-					toolCallId: 'tool_1',
-					toolName: 'read_file',
-					output: {type: 'text', value: 'file contents'},
-				},
-			],
-		},
-		{
-			role: 'tool',
-			content: [
-				{
-					type: 'tool-result',
-					toolCallId: 'tool_2',
-					toolName: 'read_file',
-					output: {type: 'text', value: 'other file contents'},
-				},
-			],
-		},
-	];
-
-	const filtered = simulatePrepareStepFiltering(messages);
-
-	// Should only have the user message - empty assistant and orphaned tools removed
-	t.is(filtered.length, 1);
-	t.is(filtered[0].role, 'user');
-});
-
-test('prepareStep - keeps tool messages after valid assistant', t => {
-	// Valid sequence: user -> assistant with content -> tool results
-	// Note: tool_calls are part of the model response, not input messages
-	const messages: ModelMessage[] = [
-		{role: 'user', content: 'Read a file'},
-		{
-			role: 'assistant',
-			content: 'Let me read that file for you.',
-		},
-		{
-			role: 'tool',
-			content: [
-				{
-					type: 'tool-result',
-					toolCallId: 'tool_1',
-					toolName: 'read_file',
-					output: {type: 'text', value: 'file contents'},
-				},
-			],
-		},
-	];
-
-	const filtered = simulatePrepareStepFiltering(messages);
-
-	// All messages should be kept
-	t.is(filtered.length, 3);
-	t.is(filtered[0].role, 'user');
-	t.is(filtered[1].role, 'assistant');
-	t.is(filtered[2].role, 'tool');
-});
-
-test('prepareStep - handles multiple empty assistants with tool messages', t => {
-	const messages: ModelMessage[] = [
-		{role: 'user', content: 'First request'},
-		{role: 'assistant', content: ''}, // Empty
-		{
-			role: 'tool',
-			content: [
-				{
-					type: 'tool-result',
-					toolCallId: 'tool_1',
-					toolName: 'read_file',
-					output: {type: 'text', value: 'file1'},
-				},
-			],
-		},
-		{role: 'user', content: 'Second request'},
-		{role: 'assistant', content: ''}, // Empty
-		{
-			role: 'tool',
-			content: [
-				{
-					type: 'tool-result',
-					toolCallId: 'tool_2',
-					toolName: 'read_file',
-					output: {type: 'text', value: 'file2'},
-				},
-			],
-		},
-	];
-
-	const filtered = simulatePrepareStepFiltering(messages);
-
-	// Should only have the two user messages
-	t.is(filtered.length, 2);
-	t.is(filtered[0].role, 'user');
-	t.is(filtered[1].role, 'user');
-});
-
-test('prepareStep - stops filtering tool messages at next non-tool message', t => {
-	const messages: ModelMessage[] = [
-		{role: 'user', content: 'Request'},
-		{role: 'assistant', content: ''}, // Empty
-		{
-			role: 'tool',
-			content: [
-				{
-					type: 'tool-result',
-					toolCallId: 'tool_1',
-					toolName: 'read_file',
-					output: {type: 'text', value: 'file1'},
-				},
-			],
-		},
-		{role: 'assistant', content: 'Here is the result'}, // Valid assistant
-		{
-			role: 'tool',
-			content: [
-				{
-					type: 'tool-result',
-					toolCallId: 'tool_2',
-					toolName: 'read_file',
-					output: {type: 'text', value: 'file2'},
-				},
-			],
-		},
-	];
-
-	const filtered = simulatePrepareStepFiltering(messages);
-
-	// Empty assistant and its tool are removed, but valid assistant and its tool remain
-	// However, the last tool message has no parent assistant with tool_calls, so it's orphaned
-	// Actually, looking at the logic more carefully: we only filter tool messages that
-	// IMMEDIATELY follow an empty assistant. So the last tool message would remain.
-	t.is(filtered.length, 3);
-	t.is(filtered[0].role, 'user');
-	t.is(filtered[1].role, 'assistant');
-	t.is(filtered[2].role, 'tool');
-});
-
-// ============================================================================
-// Complex scenarios
-// ============================================================================
-
-test('prepareStep - handles empty assistant between valid messages', t => {
-	const messages: ModelMessage[] = [
-		{role: 'user', content: 'First request'},
-		{role: 'assistant', content: 'First response'},
-		{role: 'user', content: 'Second request'},
-		{role: 'assistant', content: ''}, // Empty
-		{
-			role: 'tool',
-			content: [
-				{
-					type: 'tool-result',
-					toolCallId: 'tool_1',
-					toolName: 'read_file',
-					output: {type: 'text', value: 'file1'},
-				},
-			],
-		},
-		{role: 'assistant', content: 'Final response'},
-	];
-
-	const filtered = simulatePrepareStepFiltering(messages);
-
-	// Empty assistant and orphaned tool removed, valid messages remain
-	t.is(filtered.length, 4);
-	t.is(filtered[0].role, 'user');
-	t.is(filtered[1].role, 'assistant');
-	t.is(filtered[2].role, 'user');
-	t.is(filtered[3].role, 'assistant');
-});
-
-test('prepareStep - whitespace-only assistant content is considered empty', t => {
-	const messages: ModelMessage[] = [
-		{role: 'user', content: 'Request'},
-		{role: 'assistant', content: '   \n\t  '}, // Whitespace only
-		{
-			role: 'tool',
-			content: [
-				{
-					type: 'tool-result',
-					toolCallId: 'tool_1',
-					toolName: 'read_file',
-					output: {type: 'text', value: 'file1'},
-				},
-			],
-		},
-	];
-
-	const filtered = simulatePrepareStepFiltering(messages);
-
-	// Whitespace-only assistant and orphaned tool are filtered
-	t.is(filtered.length, 1);
-	t.is(filtered[0].role, 'user');
-});

package/source/app/App.spec.tsx

@@ -1,32 +0,0 @@
-import test from 'ava';
-import type {AppProps} from './types';
-
-test('App types are properly exported', t => {
-	const props: AppProps = {
-		vscodeMode: false,
-		nonInteractiveMode: false,
-		loggingConfig: {},
-	};
-
-	t.truthy(props);
-	t.is(props.vscodeMode, false);
-	t.is(props.nonInteractiveMode, false);
-});
-
-test('AppProps accepts all optional parameters', t => {
-	const minimalProps: AppProps = {};
-	const fullProps: AppProps = {
-		vscodeMode: true,
-		vscodePort: 3000,
-		nonInteractivePrompt: 'test prompt',
-		nonInteractiveMode: true,
-		loggingConfig: {level: 'info'},
-	};
-
-	t.truthy(minimalProps);
-	t.truthy(fullProps);
-	t.is(fullProps.vscodeMode, true);
-	t.is(fullProps.vscodePort, 3000);
-	t.is(fullProps.nonInteractivePrompt, 'test prompt');
-	t.is(fullProps.nonInteractiveMode, true);
-});