@gguf/coder 0.3.1 → 0.3.3

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

@@ -1,169 +0,0 @@
-import test from 'ava';
-import {APICallError} from 'ai';
-import {parseAPIError} from './error-parser.js';
-
-test('parseAPIError handles non-Error values', t => {
-	const result = parseAPIError('not an error');
-	t.is(result, 'An unknown error occurred while communicating with the model');
-});
-
-test('parseAPIError handles APICallError with 400 status', t => {
-	const error = new APICallError({
-		message: 'Invalid request',
-		url: 'https://api.example.com',
-		requestBodyValues: {},
-		statusCode: 400,
-		responseBody: JSON.stringify({error: {message: 'Bad input'}}),
-		responseHeaders: {},
-		isRetryable: false,
-	});
-
-	const result = parseAPIError(error);
-	t.is(result, 'Bad request: Bad input');
-});
-
-test('parseAPIError handles APICallError with 401 status', t => {
-	const error = new APICallError({
-		message: 'Unauthorized',
-		url: 'https://api.example.com',
-		requestBodyValues: {},
-		statusCode: 401,
-		responseBody: '{}',
-		responseHeaders: {},
-		isRetryable: false,
-	});
-
-	const result = parseAPIError(error);
-	t.is(result, 'Authentication failed: Invalid API key or credentials');
-});
-
-test('parseAPIError handles APICallError with 404 status', t => {
-	const error = new APICallError({
-		message: 'Not found',
-		url: 'https://api.example.com',
-		requestBodyValues: {},
-		statusCode: 404,
-		responseBody: '{}',
-		responseHeaders: {},
-		isRetryable: false,
-	});
-
-	const result = parseAPIError(error);
-	t.is(
-		result,
-		'Model not found: The requested model may not exist or is unavailable',
-	);
-});
-
-test('parseAPIError handles APICallError with 429 status and quota message', t => {
-	const error = new APICallError({
-		message: 'Rate limited',
-		url: 'https://api.example.com',
-		requestBodyValues: {},
-		statusCode: 429,
-		responseBody: JSON.stringify({
-			error: {message: 'You have exceeded your usage limit'},
-		}),
-		responseHeaders: {},
-		isRetryable: true,
-	});
-
-	const result = parseAPIError(error);
-	t.is(result, 'Rate limit: You have exceeded your usage limit');
-});
-
-test('parseAPIError handles APICallError with 429 status without quota message', t => {
-	const error = new APICallError({
-		message: 'Rate limited',
-		url: 'https://api.example.com',
-		requestBodyValues: {},
-		statusCode: 429,
-		responseBody: '{}',
-		responseHeaders: {},
-		isRetryable: true,
-	});
-
-	const result = parseAPIError(error);
-	t.is(
-		result,
-		'Rate limit exceeded: Too many requests. Please wait and try again',
-	);
-});
-
-test('parseAPIError handles APICallError with 500 status', t => {
-	const error = new APICallError({
-		message: 'Server error',
-		url: 'https://api.example.com',
-		requestBodyValues: {},
-		statusCode: 500,
-		responseBody: JSON.stringify({message: 'Internal server error'}),
-		responseHeaders: {},
-		isRetryable: true,
-	});
-
-	const result = parseAPIError(error);
-	t.is(result, 'Server error: Internal server error');
-});
-
-test('parseAPIError handles Error with status code in message', t => {
-	const error = new Error('400 Bad Request: Invalid parameters');
-	const result = parseAPIError(error);
-	t.is(result, 'Bad request: Invalid parameters');
-});
-
-test('parseAPIError handles Ollama unmarshal errors', t => {
-	const error = new Error(
-		'invalid character after top-level value: unmarshal error',
-	);
-	const result = parseAPIError(error);
-	t.true(result.includes('Ollama server error'));
-	t.true(result.includes('malformed JSON'));
-});
-
-test('parseAPIError handles timeout errors', 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 connection errors', 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 context length errors', t => {
-	const error = new Error('context length exceeded');
-	const result = parseAPIError(error);
-	t.is(
-		result,
-		'Context too large: Please reduce the conversation length or message size',
-	);
-});
-
-test('parseAPIError handles token limit errors', t => {
-	const error = new Error('Please reduce the number of tokens in your message');
-	const result = parseAPIError(error);
-	t.is(
-		result,
-		'Too many tokens: Please shorten your message or clear conversation history',
-	);
-});
-
-test('parseAPIError handles generic errors', t => {
-	const error = new Error('Something went wrong');
-	const result = parseAPIError(error);
-	t.is(result, 'Something went wrong');
-});
-
-test('parseAPIError removes "Error:" prefix from generic errors', t => {
-	const error = new Error('Error: Something went wrong');
-	const result = parseAPIError(error);
-	t.is(result, 'Something went wrong');
-});
-
-test('parseAPIError extracts first line from multiline errors', t => {
-	const error = new Error('First line\nSecond line\nThird line');
-	const result = parseAPIError(error);
-	t.is(result, 'First line');
-});

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

@@ -1,161 +0,0 @@
-import {APICallError} from 'ai';
-import {extractRootError} from './error-extractor.js';
-
-/**
- * Parses API errors into user-friendly messages.
- * Exported for testing purposes.
- */
-export function parseAPIError(error: unknown): string {
-	// First extract the root error from any wrappers
-	const rootError = extractRootError(error);
-
-	if (!(rootError instanceof Error)) {
-		return 'An unknown error occurred while communicating with the model';
-	}
-
-	// Handle AI SDK APICallError - it has statusCode and responseBody
-	if (APICallError.isInstance(rootError)) {
-		const statusCode = rootError.statusCode;
-		// Try to extract a clean message from responseBody or use the error message
-		let cleanMessage = rootError.message;
-
-		// Parse the response body if available for more details
-		if (rootError.responseBody) {
-			try {
-				const body = JSON.parse(rootError.responseBody) as {
-					error?: {message?: string};
-					message?: string;
-				};
-				if (body.error?.message) {
-					cleanMessage = body.error.message;
-				} else if (body.message) {
-					cleanMessage = body.message;
-				}
-			} catch {
-				// If not JSON, try to extract message from the raw response
-				const msgMatch = rootError.responseBody.match(
-					/["']?message["']?\s*[:=]\s*["']([^"']+)["']/i,
-				);
-				if (msgMatch) {
-					cleanMessage = msgMatch[1];
-				}
-			}
-		}
-
-		// Format based on status code
-		if (statusCode) {
-			switch (statusCode) {
-				case 400:
-					return `Bad request: ${cleanMessage}`;
-				case 401:
-					return 'Authentication failed: Invalid API key or credentials';
-				case 403:
-					return 'Access forbidden: Check your API permissions';
-				case 404:
-					return 'Model not found: The requested model may not exist or is unavailable';
-				case 429:
-					if (
-						cleanMessage.includes('usage limit') ||
-						cleanMessage.includes('quota')
-					) {
-						return `Rate limit: ${cleanMessage}`;
-					}
-					return 'Rate limit exceeded: Too many requests. Please wait and try again';
-				case 500:
-				case 502:
-				case 503:
-					return `Server error: ${cleanMessage}`;
-				default:
-					return `Request failed (${statusCode}): ${cleanMessage}`;
-			}
-		}
-	}
-
-	const errorMessage = rootError.message;
-
-	// Extract status code and clean message from common error patterns FIRST
-	// This ensures HTTP status codes are properly parsed before falling through
-	// to more generic pattern matching (like Ollama-specific errors)
-	const statusMatch = errorMessage.match(
-		/(?:Error: )?(\d{3})\s+(?:\d{3}\s+)?(?:Bad Request|[^:]+):\s*(.+)/i,
-	);
-	if (statusMatch) {
-		const [, statusCode, message] = statusMatch;
-		const cleanMessage = message.trim();
-
-		switch (statusCode) {
-			case '400':
-				return `Bad request: ${cleanMessage}`;
-			case '401':
-				return 'Authentication failed: Invalid API key or credentials';
-			case '403':
-				return 'Access forbidden: Check your API permissions';
-			case '404':
-				return 'Model not found: The requested model may not exist or is unavailable';
-			case '429':
-				// Include the original message if it has useful details
-				if (
-					cleanMessage.includes('usage limit') ||
-					cleanMessage.includes('quota')
-				) {
-					return `Rate limit: ${cleanMessage}`;
-				}
-				return 'Rate limit exceeded: Too many requests. Please wait and try again';
-			case '500':
-			case '502':
-			case '503':
-				return `Server error: ${cleanMessage}`;
-			default:
-				return `Request failed (${statusCode}): ${cleanMessage}`;
-		}
-	}
-
-	// Handle Ollama-specific unmarshal/JSON parsing errors
-	// This runs AFTER status code parsing to avoid misclassifying HTTP errors
-	// that happen to contain JSON parsing error text in their message
-	if (
-		errorMessage.includes('unmarshal') ||
-		(errorMessage.includes('invalid character') &&
-			errorMessage.includes('after top-level value'))
-	) {
-		return (
-			'Ollama server error: The model returned malformed JSON. ' +
-			'This usually indicates an issue with the Ollama server or model. ' +
-			'Try:\n' +
-			'  1. Restart Ollama: systemctl restart ollama (Linux) or restart the Ollama app\n' +
-			'  2. Re-pull the model: ollama pull <model-name>\n' +
-			'  3. Check Ollama logs for more details\n' +
-			'  4. Try a different model to see if the issue is model-specific\n' +
-			`Original error: ${errorMessage}`
-		);
-	}
-
-	// Handle timeout errors
-	if (errorMessage.includes('timeout') || errorMessage.includes('ETIMEDOUT')) {
-		return 'Request timed out: The model took too long to respond';
-	}
-
-	// Handle network errors
-	if (
-		errorMessage.includes('ECONNREFUSED') ||
-		errorMessage.includes('connect')
-	) {
-		return 'Connection failed: Unable to reach the model server';
-	}
-
-	// Handle context length errors
-	if (
-		errorMessage.includes('context length') ||
-		errorMessage.includes('too many tokens')
-	) {
-		return 'Context too large: Please reduce the conversation length or message size';
-	}
-
-	// Handle token limit errors
-	if (errorMessage.includes('reduce the number of tokens')) {
-		return 'Too many tokens: Please shorten your message or clear conversation history';
-	}
-
-	// If we can't parse it, return a cleaned up version
-	return errorMessage.replace(/^Error:\s*/i, '').split('\n')[0];
-}

package/source/ai-sdk-client/index.ts

@@ -1,7 +0,0 @@
-// Main client export
-export {AISDKClient} from './AISDKClient.js';
-
-// Exported for testing purposes
-export {parseAPIError} from './error-handling/error-parser.js';
-export {isEmptyAssistantMessage} from './converters/message-converter.js';
-export type {TestableMessage} from './types.js';

package/source/ai-sdk-client/providers/provider-factory.spec.ts

@@ -1,71 +0,0 @@
-import test from 'ava';
-import type {AIProviderConfig} from '@/types/index';
-import {Agent} from 'undici';
-import {createProvider} from './provider-factory.js';
-
-test('createProvider creates provider with basic config', t => {
-	const config: AIProviderConfig = {
-		name: 'TestProvider',
-		type: 'openai',
-		models: ['test-model'],
-		config: {
-			baseURL: 'https://api.test.com',
-			apiKey: 'test-key',
-		},
-	};
-
-	const agent = new Agent();
-	const provider = createProvider(config, agent);
-
-	t.truthy(provider);
-	t.is(typeof provider, 'function');
-});
-
-test('createProvider adds OpenRouter headers for openrouter provider', t => {
-	const config: AIProviderConfig = {
-		name: 'OpenRouter',
-		type: 'openai',
-		models: ['test-model'],
-		config: {
-			baseURL: 'https://openrouter.ai/api/v1',
-			apiKey: 'test-key',
-		},
-	};
-
-	const agent = new Agent();
-	const provider = createProvider(config, agent);
-
-	t.truthy(provider);
-});
-
-test('createProvider handles provider with no API key', t => {
-	const config: AIProviderConfig = {
-		name: 'TestProvider',
-		type: 'openai',
-		models: ['test-model'],
-		config: {
-			baseURL: 'https://api.test.com',
-		},
-	};
-
-	const agent = new Agent();
-	const provider = createProvider(config, agent);
-
-	t.truthy(provider);
-});
-
-test('createProvider handles provider with no baseURL', t => {
-	const config: AIProviderConfig = {
-		name: 'TestProvider',
-		type: 'openai',
-		models: ['test-model'],
-		config: {
-			apiKey: 'test-key',
-		},
-	};
-
-	const agent = new Agent();
-	const provider = createProvider(config, agent);
-
-	t.truthy(provider);
-});

package/source/ai-sdk-client/providers/provider-factory.ts

@@ -1,41 +0,0 @@
-import type {AIProviderConfig} from '@/types/index';
-import {createOpenAICompatible} from '@ai-sdk/openai-compatible';
-import {type Agent, fetch as undiciFetch} from 'undici';
-
-/**
- * Creates an OpenAI-compatible provider with custom fetch using undici
- */
-export function createProvider(
-	providerConfig: AIProviderConfig,
-	undiciAgent: Agent,
-): ReturnType<typeof createOpenAICompatible> {
-	const {config} = providerConfig;
-
-	// Custom fetch using undici
-	const customFetch = (
-		url: string | URL | Request,
-		options?: RequestInit,
-	): Promise<Response> => {
-		// Type cast to string | URL since undici's fetch accepts these types
-		// Request objects are converted to URL internally by the fetch spec
-		return undiciFetch(url as string | URL, {
-			...options,
-			dispatcher: undiciAgent,
-		}) as Promise<Response>;
-	};
-
-	// Add OpenRouter-specific headers for app attribution
-	const headers: Record<string, string> = {};
-	if (providerConfig.name.toLowerCase() === 'openrouter') {
-		headers['HTTP-Referer'] = 'https://github.com/Nano-Collective/coder';
-		headers['X-Title'] = 'Coder';
-	}
-
-	return createOpenAICompatible({
-		name: providerConfig.name,
-		baseURL: config.baseURL ?? '',
-		apiKey: config.apiKey ?? 'dummy-key',
-		fetch: customFetch,
-		headers,
-	});
-}

package/source/ai-sdk-client/types.ts

@@ -1,9 +0,0 @@
-/**
- * Message type used for testing the empty assistant message filter.
- * This is a simplified version of the AI SDK's internal message format.
- */
-export interface TestableMessage {
-	role: 'system' | 'user' | 'assistant' | 'tool';
-	content: string | unknown[];
-	toolCalls?: unknown[];
-}

package/source/ai-sdk-client-empty-message.spec.ts

@@ -1,141 +0,0 @@
-import test from 'ava';
-import {isEmptyAssistantMessage, type TestableMessage} from '@/ai-sdk-client';
-
-// Tests for isEmptyAssistantMessage function
-// This function is used to filter out empty assistant messages that would cause API errors:
-// "400 Bad Request: Assistant message must have either content or tool_calls, but not none."
-
-// ============================================================================
-// Non-assistant messages should never be considered empty
-// ============================================================================
-
-test('isEmptyAssistantMessage - user message is not empty', t => {
-	const message: TestableMessage = {
-		role: 'user',
-		content: '',
-	};
-	t.false(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - system message is not empty', t => {
-	const message: TestableMessage = {
-		role: 'system',
-		content: '',
-	};
-	t.false(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - tool message is not empty', t => {
-	const message: TestableMessage = {
-		role: 'tool',
-		content: '',
-	};
-	t.false(isEmptyAssistantMessage(message));
-});
-
-// ============================================================================
-// Empty assistant messages (should be filtered out)
-// ============================================================================
-
-test('isEmptyAssistantMessage - assistant with empty string content is empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: '',
-	};
-	t.true(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - assistant with whitespace-only content is empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: '   \n\t  ',
-	};
-	t.true(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - assistant with empty array content is empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: [],
-	};
-	t.true(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - assistant with empty content and empty toolCalls is empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: '',
-		toolCalls: [],
-	};
-	t.true(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - assistant with undefined toolCalls is empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: '',
-		toolCalls: undefined,
-	};
-	t.true(isEmptyAssistantMessage(message));
-});
-
-// ============================================================================
-// Valid assistant messages (should NOT be filtered out)
-// ============================================================================
-
-test('isEmptyAssistantMessage - assistant with text content is not empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: 'Hello, how can I help you?',
-	};
-	t.false(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - assistant with single character content is not empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: 'a',
-	};
-	t.false(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - assistant with array content is not empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: [{type: 'text', text: 'Hello'}],
-	};
-	t.false(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - assistant with toolCalls is not empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: '',
-		toolCalls: [
-			{toolName: 'read_file', toolCallId: '123', input: {path: 'test.txt'}},
-		],
-	};
-	t.false(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - assistant with content and toolCalls is not empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: 'Let me read that file for you.',
-		toolCalls: [
-			{toolName: 'read_file', toolCallId: '123', input: {path: 'test.txt'}},
-		],
-	};
-	t.false(isEmptyAssistantMessage(message));
-});
-
-test('isEmptyAssistantMessage - assistant with whitespace content but toolCalls is not empty', t => {
-	const message: TestableMessage = {
-		role: 'assistant',
-		content: '   ',
-		toolCalls: [
-			{toolName: 'read_file', toolCallId: '123', input: {path: 'test.txt'}},
-		],
-	};
-	t.false(isEmptyAssistantMessage(message));
-});