berget 2.0.6 → 2.1.1

package/dist/tests/utils/config-loader.test.js

@@ -1,182 +1,267 @@
 "use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 const vitest_1 = require("vitest");
 const config_loader_1 = require("../../src/utils/config-loader");
-const path = __importStar(require("path"));
-const fs_1 = require("fs");
-(0, vitest_1.describe)('ConfigLoader Singleton Pattern', () => {
-    const testConfigPath = path.join(process.cwd(), 'test-config.json');
-    const testConfigPath2 = path.join(process.cwd(), 'test-config2.json');
-    const testConfig = {
-        $schema: 'https://opencode.ai/config.json',
-        username: 'test-user',
-        model: 'test-model',
-        agent: {
-            test: {
-                model: 'test-model',
-                temperature: 0.5,
-                mode: 'primary',
-                permission: { edit: 'allow', bash: 'allow', webfetch: 'allow' },
-                description: 'Test agent',
-                prompt: 'Test prompt'
-            }
-        }
-    };
+// Mock fs module
+const mockFs = vitest_1.vi.hoisted(() => ({
+    existsSync: vitest_1.vi.fn(),
+    readFileSync: vitest_1.vi.fn(),
+    writeFileSync: vitest_1.vi.fn(),
+    mkdirSync: vitest_1.vi.fn(),
+}));
+vitest_1.vi.mock('fs', () => mockFs);
+(0, vitest_1.describe)('ConfigLoader', () => {
+    const testConfigPath = '/tmp/test-opencode.json';
+    let configLoader;
     (0, vitest_1.beforeEach)(() => {
-        // Clean up any existing test files
-        if ((0, fs_1.existsSync)(testConfigPath))
-            (0, fs_1.unlinkSync)(testConfigPath);
-        if ((0, fs_1.existsSync)(testConfigPath2))
-            (0, fs_1.unlinkSync)(testConfigPath2);
-        config_loader_1.ConfigLoader.instance = null;
+        // Reset mocks and clear singleton
+        vitest_1.vi.clearAllMocks();
+        config_loader_1.ConfigLoader.clearInstance();
+        // Create new instance for each test using getInstance
+        configLoader = config_loader_1.ConfigLoader.getInstance(testConfigPath);
     });
     (0, vitest_1.afterEach)(() => {
-        // Clean up test files
-        if ((0, fs_1.existsSync)(testConfigPath))
-            (0, fs_1.unlinkSync)(testConfigPath);
-        if ((0, fs_1.existsSync)(testConfigPath2))
-            (0, fs_1.unlinkSync)(testConfigPath2);
-        config_loader_1.ConfigLoader.instance = null;
-    });
-    (0, vitest_1.it)('should create singleton instance on first call', () => {
-        const instance1 = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        const instance2 = config_loader_1.ConfigLoader.getInstance();
-        (0, vitest_1.expect)(instance1).toBe(instance2);
-        (0, vitest_1.expect)(instance1.getConfigPath()).toBe(testConfigPath);
-    });
-    (0, vitest_1.it)('should ignore configPath parameter after first instantiation', () => {
-        const instance1 = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        const instance2 = config_loader_1.ConfigLoader.getInstance(testConfigPath2); // Should be ignored
-        (0, vitest_1.expect)(instance1).toBe(instance2);
-        (0, vitest_1.expect)(instance1.getConfigPath()).toBe(testConfigPath); // Still uses first path
-    });
-    (0, vitest_1.it)('should load and cache configuration', () => {
-        (0, fs_1.writeFileSync)(testConfigPath, JSON.stringify(testConfig));
-        const loader = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        const config1 = loader.loadConfig();
-        const config2 = loader.loadConfig();
-        (0, vitest_1.expect)(config1).toBe(config2); // Same object reference (cached)
-        (0, vitest_1.expect)(config1.username).toBe('test-user');
+        vitest_1.vi.clearAllMocks();
+        config_loader_1.ConfigLoader.clearInstance();
     });
-    (0, vitest_1.it)('should reload configuration when reloadConfig is called', () => {
-        (0, fs_1.writeFileSync)(testConfigPath, JSON.stringify(testConfig));
-        const loader = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        const config1 = loader.loadConfig();
-        // Modify file
-        const modifiedConfig = Object.assign(Object.assign({}, testConfig), { username: 'modified-user' });
-        (0, fs_1.writeFileSync)(testConfigPath, JSON.stringify(modifiedConfig));
-        const config2 = loader.loadConfig(); // Should still be cached
-        (0, vitest_1.expect)(config2).toBe(config1);
-        (0, vitest_1.expect)(config2.username).toBe('test-user');
-        const config3 = loader.reloadConfig(); // Should reload
-        (0, vitest_1.expect)(config3).not.toBe(config1);
-        (0, vitest_1.expect)(config3.username).toBe('modified-user');
-    });
-    (0, vitest_1.it)('should change config path and clear cache with setConfigPath', () => {
-        (0, fs_1.writeFileSync)(testConfigPath, JSON.stringify(testConfig));
-        (0, fs_1.writeFileSync)(testConfigPath2, JSON.stringify(Object.assign(Object.assign({}, testConfig), { username: 'user2' })));
-        const loader = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        const config1 = loader.loadConfig();
-        (0, vitest_1.expect)(config1.username).toBe('test-user');
-        loader.setConfigPath(testConfigPath2);
-        const config2 = loader.loadConfig();
-        (0, vitest_1.expect)(config2.username).toBe('user2');
-        (0, vitest_1.expect)(loader.getConfigPath()).toBe(testConfigPath2);
-    });
-    (0, vitest_1.it)('should throw error when config file does not exist', () => {
-        const loader = config_loader_1.ConfigLoader.getInstance('/nonexistent/config.json');
-        (0, vitest_1.expect)(() => loader.loadConfig()).toThrow('Configuration file not found');
-    });
-    (0, vitest_1.it)('should throw error when config file has invalid JSON', () => {
-        (0, fs_1.writeFileSync)(testConfigPath, 'invalid json content');
-        const loader = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        (0, vitest_1.expect)(() => loader.loadConfig()).toThrow('Invalid JSON in configuration file');
-    });
-    (0, vitest_1.it)('should provide fallback model configuration with warning', () => {
-        const minimalConfig = { username: 'test' }; // No model settings
-        (0, fs_1.writeFileSync)(testConfigPath, JSON.stringify(minimalConfig));
-        const loader = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        const modelConfig = loader.getModelConfig();
-        (0, vitest_1.expect)(modelConfig.primary).toBe('berget/deepseek-r1');
-        (0, vitest_1.expect)(modelConfig.small).toBe('berget/gpt-oss');
-    });
-    (0, vitest_1.it)('should provide fallback provider models with warning', () => {
-        const minimalConfig = { username: 'test' }; // No provider settings
-        (0, fs_1.writeFileSync)(testConfigPath, JSON.stringify(minimalConfig));
-        const loader = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        const providerModels = loader.getProviderModels();
-        (0, vitest_1.expect)(providerModels['deepseek-r1']).toBeDefined();
-        (0, vitest_1.expect)(providerModels['gpt-oss']).toBeDefined();
-        (0, vitest_1.expect)(providerModels['llama-8b']).toBeDefined();
-    });
-    (0, vitest_1.it)('should provide fallback watcher configuration with warning', () => {
-        const minimalConfig = { username: 'test' }; // No watcher settings
-        (0, fs_1.writeFileSync)(testConfigPath, JSON.stringify(minimalConfig));
-        const loader = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        const watcherConfig = loader.getWatcherConfig();
-        (0, vitest_1.expect)(watcherConfig.ignore).toContain('node_modules');
-        (0, vitest_1.expect)(watcherConfig.ignore).toContain('dist');
-        (0, vitest_1.expect)(watcherConfig.ignore).toContain('.git');
-        (0, vitest_1.expect)(watcherConfig.ignore).toContain('coverage');
+    (0, vitest_1.describe)('when config file does not exist', () => {
+        (0, vitest_1.beforeEach)(() => {
+            mockFs.existsSync.mockReturnValue(false);
+        });
+        (0, vitest_1.describe)('getModelConfig', () => {
+            (0, vitest_1.it)('should return default values when config file does not exist', () => {
+                const modelConfig = configLoader.getModelConfig();
+                (0, vitest_1.expect)(modelConfig).toEqual({
+                    primary: 'berget/glm-4.7',
+                    small: 'berget/gpt-oss'
+                });
+            });
+            (0, vitest_1.it)('should return default values when using convenience function', () => {
+                const modelConfig = (0, config_loader_1.getModelConfig)(testConfigPath);
+                (0, vitest_1.expect)(modelConfig).toEqual({
+                    primary: 'berget/glm-4.7',
+                    small: 'berget/gpt-oss'
+                });
+            });
+        });
+        (0, vitest_1.describe)('getProviderModels', () => {
+            (0, vitest_1.it)('should return default provider models when config file does not exist', () => {
+                const models = configLoader.getProviderModels();
+                (0, vitest_1.expect)(models).toEqual({
+                    'glm-4.7': {
+                        name: 'GLM-4.7',
+                        limit: { output: 4000, context: 90000 }
+                    },
+                    'gpt-oss': {
+                        name: 'GPT-OSS',
+                        limit: { output: 4000, context: 128000 },
+                        modalities: {
+                            input: ['text', 'image'],
+                            output: ['text']
+                        }
+                    },
+                    'llama-8b': {
+                        name: 'llama-3.1-8b',
+                        limit: { output: 4000, context: 128000 }
+                    }
+                });
+            });
+            (0, vitest_1.it)('should return default provider models when using convenience function', () => {
+                const models = (0, config_loader_1.getProviderModels)(testConfigPath);
+                (0, vitest_1.expect)(models).toEqual({
+                    'glm-4.7': {
+                        name: 'GLM-4.7',
+                        limit: { output: 4000, context: 90000 }
+                    },
+                    'gpt-oss': {
+                        name: 'GPT-OSS',
+                        limit: { output: 4000, context: 128000 },
+                        modalities: {
+                            input: ['text', 'image'],
+                            output: ['text']
+                        }
+                    },
+                    'llama-8b': {
+                        name: 'llama-3.1-8b',
+                        limit: { output: 4000, context: 128000 }
+                    }
+                });
+            });
+        });
+        (0, vitest_1.describe)('getAllAgentConfigs', () => {
+            (0, vitest_1.it)('should return empty object when config file does not exist', () => {
+                const agents = configLoader.getAllAgentConfigs();
+                (0, vitest_1.expect)(agents).toEqual({});
+            });
+            (0, vitest_1.it)('should return empty object when using convenience function', () => {
+                const agents = (0, config_loader_1.getAllAgentConfigs)(testConfigPath);
+                (0, vitest_1.expect)(agents).toEqual({});
+            });
+        });
+        (0, vitest_1.describe)('getAgentConfig', () => {
+            (0, vitest_1.it)('should return null when config file does not exist', () => {
+                const agent = configLoader.getAgentConfig('fullstack');
+                (0, vitest_1.expect)(agent).toBeNull();
+            });
+        });
     });
-    (0, vitest_1.it)('should use actual configuration when provided', () => {
-        const fullConfig = Object.assign(Object.assign({}, testConfig), { model: 'custom-primary-model', small_model: 'custom-small-model', provider: {
+    (0, vitest_1.describe)('when config file exists', () => {
+        const mockConfig = {
+            model: 'custom-model',
+            small_model: 'custom-small-model',
+            agent: {
+                fullstack: {
+                    model: 'custom-agent-model',
+                    temperature: 0.5,
+                    mode: 'primary',
+                    permission: {
+                        edit: 'allow',
+                        bash: 'allow',
+                        webfetch: 'allow'
+                    }
+                }
+            },
+            command: {
+                test: {
+                    description: 'Test command'
+                }
+            },
+            watcher: {
+                ignore: ['custom-ignore']
+            },
+            provider: {
                 berget: {
                     models: {
                         'custom-model': {
                             name: 'Custom Model',
-                            limit: { output: 2000, context: 4000 }
+                            limit: { output: 8000, context: 160000 }
                         }
                     }
                 }
-            }, watcher: {
-                ignore: ['custom-ignore']
-            } });
-        (0, fs_1.writeFileSync)(testConfigPath, JSON.stringify(fullConfig));
-        const loader = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        const modelConfig = loader.getModelConfig();
-        (0, vitest_1.expect)(modelConfig.primary).toBe('custom-primary-model');
-        (0, vitest_1.expect)(modelConfig.small).toBe('custom-small-model');
-        const providerModels = loader.getProviderModels();
-        (0, vitest_1.expect)(providerModels['custom-model']).toBeDefined();
-        (0, vitest_1.expect)(providerModels['custom-model'].name).toBe('Custom Model');
-        const watcherConfig = loader.getWatcherConfig();
-        (0, vitest_1.expect)(watcherConfig.ignore).toEqual(['custom-ignore']);
+            }
+        };
+        (0, vitest_1.beforeEach)(() => {
+            mockFs.existsSync.mockReturnValue(true);
+            mockFs.readFileSync.mockReturnValue(JSON.stringify(mockConfig));
+        });
+        (0, vitest_1.describe)('getModelConfig', () => {
+            (0, vitest_1.it)('should return values from config file', () => {
+                const modelConfig = configLoader.getModelConfig();
+                (0, vitest_1.expect)(modelConfig).toEqual({
+                    primary: 'custom-model',
+                    small: 'custom-small-model'
+                });
+            });
+        });
+        (0, vitest_1.describe)('getProviderModels', () => {
+            (0, vitest_1.it)('should return models from config file', () => {
+                const models = configLoader.getProviderModels();
+                (0, vitest_1.expect)(models).toEqual({
+                    'custom-model': {
+                        name: 'Custom Model',
+                        limit: { output: 8000, context: 160000 }
+                    }
+                });
+            });
+        });
+        (0, vitest_1.describe)('getAllAgentConfigs', () => {
+            (0, vitest_1.it)('should return agents from config file', () => {
+                const agents = configLoader.getAllAgentConfigs();
+                (0, vitest_1.expect)(agents).toEqual(mockConfig.agent);
+            });
+        });
+        (0, vitest_1.describe)('getAgentConfig', () => {
+            (0, vitest_1.it)('should return specific agent from config file', () => {
+                const agent = configLoader.getAgentConfig('fullstack');
+                (0, vitest_1.expect)(agent).toEqual(mockConfig.agent.fullstack);
+            });
+            (0, vitest_1.it)('should return null for non-existent agent', () => {
+                const agent = configLoader.getAgentConfig('nonexistent');
+                (0, vitest_1.expect)(agent).toBeNull();
+            });
+        });
+    });
+    (0, vitest_1.describe)('when config file is invalid JSON', () => {
+        (0, vitest_1.beforeEach)(() => {
+            mockFs.existsSync.mockReturnValue(true);
+            mockFs.readFileSync.mockReturnValue('invalid json {');
+        });
+        (0, vitest_1.it)('should fall back to defaults for getModelConfig', () => {
+            const modelConfig = configLoader.getModelConfig();
+            (0, vitest_1.expect)(modelConfig).toEqual({
+                primary: 'berget/glm-4.7',
+                small: 'berget/gpt-oss'
+            });
+        });
+        (0, vitest_1.it)('should fall back to defaults for getProviderModels', () => {
+            const models = configLoader.getProviderModels();
+            (0, vitest_1.expect)(models).toEqual({
+                'glm-4.7': {
+                    name: 'GLM-4.7',
+                    limit: { output: 4000, context: 90000 }
+                },
+                'gpt-oss': {
+                    name: 'GPT-OSS',
+                    limit: { output: 4000, context: 128000 },
+                    modalities: {
+                        input: ['text', 'image'],
+                        output: ['text']
+                    }
+                },
+                'llama-8b': {
+                    name: 'llama-3.1-8b',
+                    limit: { output: 4000, context: 128000 }
+                }
+            });
+        });
+        (0, vitest_1.it)('should fall back to defaults for getAllAgentConfigs', () => {
+            const agents = configLoader.getAllAgentConfigs();
+            (0, vitest_1.expect)(agents).toEqual({});
+        });
+    });
+    (0, vitest_1.describe)('singleton pattern', () => {
+        (0, vitest_1.it)('should return the same instance for same path', () => {
+            config_loader_1.ConfigLoader.clearInstance();
+            const loader1 = config_loader_1.ConfigLoader.getInstance(testConfigPath);
+            const loader2 = config_loader_1.ConfigLoader.getInstance(testConfigPath);
+            (0, vitest_1.expect)(loader1).toBe(loader2);
+        });
+        (0, vitest_1.it)('should return the same instance even for different paths (true singleton)', () => {
+            config_loader_1.ConfigLoader.clearInstance();
+            const loader1 = config_loader_1.ConfigLoader.getInstance('/path1/config.json');
+            const loader2 = config_loader_1.ConfigLoader.getInstance('/path2/config.json');
+            // ConfigLoader is a true singleton - it returns the same instance regardless of path
+            (0, vitest_1.expect)(loader1).toBe(loader2);
+        });
     });
-    (0, vitest_1.it)('should maintain consistent getter method behavior', () => {
-        (0, fs_1.writeFileSync)(testConfigPath, JSON.stringify(testConfig));
-        const loader = config_loader_1.ConfigLoader.getInstance(testConfigPath);
-        // All getter methods should use the same cached config
-        const agentConfig = loader.getAgentConfig('test');
-        const allAgents = loader.getAllAgentConfigs();
-        const commandConfigs = loader.getCommandConfigs();
-        const providerConfig = loader.getProviderConfig();
-        (0, vitest_1.expect)(agentConfig).toBeDefined();
-        (0, vitest_1.expect)(allAgents.test).toBeDefined();
-        (0, vitest_1.expect)(commandConfigs).toEqual({});
-        (0, vitest_1.expect)(providerConfig).toEqual({});
+    (0, vitest_1.describe)('init scenario regression tests', () => {
+        (0, vitest_1.it)('should handle missing config file during init scenario', () => {
+            // This test specifically verifies the fix for the init issue
+            mockFs.existsSync.mockReturnValue(false);
+            // All these methods should work without throwing errors
+            (0, vitest_1.expect)(() => configLoader.getModelConfig()).not.toThrow();
+            (0, vitest_1.expect)(() => configLoader.getProviderModels()).not.toThrow();
+            (0, vitest_1.expect)(() => configLoader.getAllAgentConfigs()).not.toThrow();
+            (0, vitest_1.expect)(() => configLoader.getAgentConfig('fullstack')).not.toThrow();
+            // And return sensible defaults
+            (0, vitest_1.expect)(configLoader.getModelConfig()).toEqual({
+                primary: 'berget/glm-4.7',
+                small: 'berget/gpt-oss'
+            });
+            (0, vitest_1.expect)(configLoader.getAllAgentConfigs()).toEqual({});
+            (0, vitest_1.expect)(configLoader.getAgentConfig('fullstack')).toBeNull();
+        });
+        (0, vitest_1.it)('should work with convenience functions during init scenario', () => {
+            // This test verifies that convenience functions also work during init
+            mockFs.existsSync.mockReturnValue(false);
+            (0, vitest_1.expect)(() => (0, config_loader_1.getModelConfig)(testConfigPath)).not.toThrow();
+            (0, vitest_1.expect)(() => (0, config_loader_1.getProviderModels)(testConfigPath)).not.toThrow();
+            (0, vitest_1.expect)(() => (0, config_loader_1.getAllAgentConfigs)(testConfigPath)).not.toThrow();
+            (0, vitest_1.expect)((0, config_loader_1.getModelConfig)(testConfigPath)).toEqual({
+                primary: 'berget/glm-4.7',
+                small: 'berget/gpt-oss'
+            });
+            (0, vitest_1.expect)((0, config_loader_1.getAllAgentConfigs)(testConfigPath)).toEqual({});
+        });
     });
 });