feat: sync PROVIDER_NAMES with config schema and update README docs

Extract MODEL_PROVIDERS const from config schema as single source of truth
for provider names. PROVIDER_NAMES in TUI commands now imports from schema
instead of maintaining a hardcoded list. Adds tests verifying sync.

Updates README TUI Commands section with /model hot-swap documentation,
supported providers, and runtime model switching examples.
This commit is contained in:
William Valentin
2026-02-10 21:26:18 -08:00
parent 27ee3b2c10
commit 25482b8516
5 changed files with 74 additions and 9 deletions
+1 -1
View File
@@ -1,2 +1,2 @@
export { loadConfig, deepMerge } from './loader.js';
export { configSchema, type Config, type TelegramConfig, type ModelConfig, type CronJobConfig, type AgentsConfig, type CompactionConfig, type ToolProfile, type ToolOverrideConfig, type ToolsConfig, type SandboxConfig, type AgentConfigEntry, type RoutingConfig, type ServerConfig } from './schema.js';
export { configSchema, MODEL_PROVIDERS, type ModelProvider, type Config, type TelegramConfig, type ModelConfig, type CronJobConfig, type AgentsConfig, type CompactionConfig, type ToolProfile, type ToolOverrideConfig, type ToolsConfig, type SandboxConfig, type AgentConfigEntry, type RoutingConfig, type ServerConfig } from './schema.js';
+6 -1
View File
@@ -38,8 +38,13 @@ const serverSchema = z.object({
lock: z.boolean().default(false),
});
/** All supported model provider identifiers. Used by the config schema and TUI autocompletion. */
export const MODEL_PROVIDERS = ['anthropic', 'openai', 'gemini', 'ollama', 'llamacpp', 'openrouter', 'bedrock', 'github', 'zhipuai', 'xai'] as const;
export type ModelProvider = (typeof MODEL_PROVIDERS)[number];
const modelConfigBaseSchema = z.object({
provider: z.enum(['anthropic', 'openai', 'gemini', 'ollama', 'llamacpp', 'openrouter', 'bedrock', 'github', 'zhipuai', 'xai']),
provider: z.enum(MODEL_PROVIDERS),
model: z.string(),
endpoint: z.string().optional(),
api_key: z.string().optional(),
+32 -1
View File
@@ -1,5 +1,6 @@
import { describe, it, expect } from 'vitest';
import { parseCommand, getHelpText } from './commands.js';
import { parseCommand, getHelpText, getCommandCompletions, PROVIDER_NAMES } from './commands.js';
import { MODEL_PROVIDERS } from '../../config/index.js';
describe('parseCommand', () => {
it('parses /quit command', () => {
@@ -130,3 +131,33 @@ describe('/pair command', () => {
expect(parseCommand('/pair revoke telegram 12345')).toEqual({ type: 'pair', action: 'revoke', args: 'telegram 12345' });
});
});
describe('PROVIDER_NAMES', () => {
it('matches all providers from the config schema', () => {
expect(PROVIDER_NAMES).toEqual(MODEL_PROVIDERS);
});
it('includes zhipuai, openrouter, and xai', () => {
expect(PROVIDER_NAMES).toContain('zhipuai');
expect(PROVIDER_NAMES).toContain('openrouter');
expect(PROVIDER_NAMES).toContain('xai');
});
});
describe('getCommandCompletions', () => {
it('suggests all providers when typing /model default with partial provider', () => {
// Typing any character after the tier triggers provider suggestions
const completions = getCommandCompletions('/model default a');
expect(completions).toContain('/model default anthropic');
});
it('filters provider suggestions by partial input', () => {
const completions = getCommandCompletions('/model default zh');
expect(completions).toEqual(['/model default zhipuai']);
});
it('completes xai provider', () => {
const completions = getCommandCompletions('/model fast x');
expect(completions).toEqual(['/model fast xai']);
});
});
+4 -2
View File
@@ -139,6 +139,8 @@ Commands:
`.trim();
}
import { MODEL_PROVIDERS } from '../../config/index.js';
export type ModelAlias = 'local' | 'default' | 'fast' | 'complex' | 'opus' | 'sonnet' | 'haiku' | 'ollama';
// List of all slash commands for autocompletion
@@ -186,8 +188,8 @@ export const COMMAND_TOOLTIPS: Record<string, string> = {
// Model aliases for /model command autocompletion
export const MODEL_ALIASES = ['local', 'default', 'fast', 'complex', 'opus', 'sonnet', 'haiku', 'ollama'];
// Provider names for /model <tier> <provider/model> syntax
export const PROVIDER_NAMES = ['anthropic', 'openai', 'github-copilot', 'gemini', 'bedrock', 'ollama', 'llamacpp'];
// Provider names for /model <tier> <provider/model> syntax — derived from config schema
export const PROVIDER_NAMES: readonly string[] = MODEL_PROVIDERS;
// Model alias descriptions
export const MODEL_TOOLTIPS: Record<string, string> = {