@gguf/coder 0.3.1 → 0.3.2

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));
-});

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);
-});