berget 2.2.6 → 2.2.8

package/dist/tests/commands/code.test.js

@@ -22,22 +22,13 @@ var __importStar = (this && this.__importStar) || function (mod) {
     __setModuleDefault(result, mod);
     return result;
 };
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
-        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
-        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
 Object.defineProperty(exports, "__esModule", { value: true });
-const vitest_1 = require("vitest");
 const commander_1 = require("commander");
+const fs = __importStar(require("node:fs"));
+const promises_1 = require("node:fs/promises");
+const vitest_1 = require("vitest");
 const code_1 = require("../../src/commands/code");
 const api_key_service_1 = require("../../src/services/api-key-service");
-const fs = __importStar(require("fs"));
-const promises_1 = require("fs/promises");
 const env_manager_1 = require("../../src/utils/env-manager");
 // Mock dependencies
 vitest_1.vi.mock('../../src/services/api-key-service');
@@ -57,8 +48,8 @@ vitest_1.vi.mock('child_process', () => ({
 }));
 vitest_1.vi.mock('readline', () => ({
     createInterface: vitest_1.vi.fn(() => ({
-        question: vitest_1.vi.fn(),
         close: vitest_1.vi.fn(),
+        question: vitest_1.vi.fn(),
     })),
 }));
 (0, vitest_1.describe)('Code Commands', () => {
@@ -95,35 +86,35 @@ vitest_1.vi.mock('readline', () => ({
     (0, vitest_1.describe)('code init command', () => {
         (0, vitest_1.it)('should register init command with correct description', () => {
             const codeCommand = program.commands.find((cmd) => cmd.name() === 'code');
-            const initCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'init');
+            const initCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'init');
             (0, vitest_1.expect)(initCommand).toBeDefined();
-            (0, vitest_1.expect)(initCommand === null || initCommand === void 0 ? void 0 : initCommand.description()).toBe('Initialize project for AI coding assistant');
+            (0, vitest_1.expect)(initCommand?.description()).toBe('Initialize project for AI coding assistant');
         });
         (0, vitest_1.it)('should have name, force, and yes options', () => {
             const codeCommand = program.commands.find((cmd) => cmd.name() === 'code');
-            const initCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'init');
+            const initCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'init');
             (0, vitest_1.expect)(initCommand).toBeDefined();
-            const nameOption = initCommand === null || initCommand === void 0 ? void 0 : initCommand.options.find((opt) => opt.long === '--name');
-            const forceOption = initCommand === null || initCommand === void 0 ? void 0 : initCommand.options.find((opt) => opt.long === '--force');
-            const yesOption = initCommand === null || initCommand === void 0 ? void 0 : initCommand.options.find((opt) => opt.long === '--yes');
+            const nameOption = initCommand?.options.find((opt) => opt.long === '--name');
+            const forceOption = initCommand?.options.find((opt) => opt.long === '--force');
+            const yesOption = initCommand?.options.find((opt) => opt.long === '--yes');
             (0, vitest_1.expect)(nameOption).toBeDefined();
-            (0, vitest_1.expect)(nameOption === null || nameOption === void 0 ? void 0 : nameOption.description).toContain('Project name');
+            (0, vitest_1.expect)(nameOption?.description).toContain('Project name');
             (0, vitest_1.expect)(forceOption).toBeDefined();
-            (0, vitest_1.expect)(forceOption === null || forceOption === void 0 ? void 0 : forceOption.description).toContain('Overwrite existing configuration');
+            (0, vitest_1.expect)(forceOption?.description).toContain('Overwrite existing configuration');
             (0, vitest_1.expect)(yesOption).toBeDefined();
-            (0, vitest_1.expect)(yesOption === null || yesOption === void 0 ? void 0 : yesOption.description).toContain('Automatically answer yes');
+            (0, vitest_1.expect)(yesOption?.description).toContain('Automatically answer yes');
         });
         (0, vitest_1.it)('should check if opencode is installed', () => {
             const codeCommand = program.commands.find((cmd) => cmd.name() === 'code');
-            const initCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'init');
+            const initCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'init');
             (0, vitest_1.expect)(initCommand).toBeDefined();
             // The command should attempt to spawn opencode --version
             // This is tested implicitly through the spawn mock
         });
-        (0, vitest_1.it)('should list existing API keys and allow selection', () => __awaiter(void 0, void 0, void 0, function* () {
+        (0, vitest_1.it)('should list existing API keys and allow selection', async () => {
             // Mock successful opencode installation check
-            mockSpawn.mockImplementation((command, args) => {
-                if (command === 'opencode' && args[0] === '--version') {
+            mockSpawn.mockImplementation((command, arguments_) => {
+                if (command === 'opencode' && arguments_[0] === '--version') {
                     return {
                         on: vitest_1.vi.fn().mockImplementation((event, callback) => {
                             if (event === 'close')
@@ -136,31 +127,31 @@ vitest_1.vi.mock('readline', () => ({
             // Mock existing API keys
             const mockExistingKeys = [
                 {
+                    created: '2023-01-01T00:00:00.000Z',
                     id: 1,
+                    lastUsed: null,
                     name: 'existing-key-1',
                     prefix: 'sk_ber',
-                    created: '2023-01-01T00:00:00.000Z',
-                    lastUsed: null,
                 },
                 {
+                    created: '2023-01-02T00:00:00.000Z',
                     id: 2,
+                    lastUsed: '2023-01-03T00:00:00.000Z',
                     name: 'existing-key-2',
                     prefix: 'sk_ber',
-                    created: '2023-01-02T00:00:00.000Z',
-                    lastUsed: '2023-01-03T00:00:00.000Z',
                 },
             ];
             mockApiKeyService.list.mockResolvedValue(mockExistingKeys);
             // Mock file operations
             mockFs.existsSync.mockReturnValue(false);
-            mockFsPromises.writeFile.mockResolvedValue(undefined);
+            mockFsPromises.writeFile.mockResolvedValue();
             // Verify that the list method is called
             (0, vitest_1.expect)(mockApiKeyService.list).toBeDefined();
-        }));
-        (0, vitest_1.it)('should create new API key with project-based naming', () => __awaiter(void 0, void 0, void 0, function* () {
+        });
+        (0, vitest_1.it)('should create new API key with project-based naming', async () => {
             // Mock successful opencode installation check
-            mockSpawn.mockImplementation((command, args) => {
-                if (command === 'opencode' && args[0] === '--version') {
+            mockSpawn.mockImplementation((command, arguments_) => {
+                if (command === 'opencode' && arguments_[0] === '--version') {
                     return {
                         on: vitest_1.vi.fn().mockImplementation((event, callback) => {
                             if (event === 'close')
@@ -175,33 +166,33 @@ vitest_1.vi.mock('readline', () => ({
             // Mock successful API key creation
             const mockApiKeyData = {
                 id: 123,
-                name: 'opencode-testproject-1234567890',
                 key: 'test-api-key-12345',
+                name: 'opencode-testproject-1234567890',
             };
             mockApiKeyService.create.mockResolvedValue(mockApiKeyData);
             // Mock file operations
             mockFs.existsSync.mockReturnValue(false);
-            mockFsPromises.writeFile.mockResolvedValue(undefined);
+            mockFsPromises.writeFile.mockResolvedValue();
             // Verify that the create method is available
             (0, vitest_1.expect)(mockApiKeyService.create).toBeDefined();
-        }));
-        (0, vitest_1.it)('should create opencode.json with correct structure', () => __awaiter(void 0, void 0, void 0, function* () {
+        });
+        (0, vitest_1.it)('should create opencode.json with correct structure', async () => {
             // This tests the expected config structure
             const expectedConfig = {
-                model: 'berget/glm-4-6',
                 apiKey: 'test-api-key',
+                created: vitest_1.expect.any(String),
+                model: 'berget/glm-4-6',
                 projectName: 'testproject',
                 provider: 'berget',
-                created: vitest_1.expect.any(String),
                 version: '1.0.0',
             };
             (0, vitest_1.expect)(expectedConfig.model).toBe('berget/glm-4-6');
             (0, vitest_1.expect)(expectedConfig.provider).toBe('berget');
             (0, vitest_1.expect)(expectedConfig.version).toBe('1.0.0');
-        }));
+        });
         (0, vitest_1.it)('should handle existing config file', () => {
             const codeCommand = program.commands.find((cmd) => cmd.name() === 'code');
-            const initCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'init');
+            const initCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'init');
             (0, vitest_1.expect)(initCommand).toBeDefined();
             // Should check if opencode.json exists before proceeding
             (0, vitest_1.expect)(mockFs.existsSync).toBeDefined();
@@ -210,39 +201,39 @@ vitest_1.vi.mock('readline', () => ({
     (0, vitest_1.describe)('code run command', () => {
         (0, vitest_1.it)('should register run command with correct description', () => {
             const codeCommand = program.commands.find((cmd) => cmd.name() === 'code');
-            const runCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'run');
+            const runCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'run');
             (0, vitest_1.expect)(runCommand).toBeDefined();
-            (0, vitest_1.expect)(runCommand === null || runCommand === void 0 ? void 0 : runCommand.description()).toBe('Run AI coding assistant');
+            (0, vitest_1.expect)(runCommand?.description()).toBe('Run AI coding assistant');
         });
         (0, vitest_1.it)('should accept prompt argument and model, no-config, and yes options', () => {
             const codeCommand = program.commands.find((cmd) => cmd.name() === 'code');
-            const runCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'run');
+            const runCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'run');
             (0, vitest_1.expect)(runCommand).toBeDefined();
-            const modelOption = runCommand === null || runCommand === void 0 ? void 0 : runCommand.options.find((opt) => opt.long === '--model');
-            const noConfigOption = runCommand === null || runCommand === void 0 ? void 0 : runCommand.options.find((opt) => opt.long === '--no-config');
-            const yesOption = runCommand === null || runCommand === void 0 ? void 0 : runCommand.options.find((opt) => opt.long === '--yes');
+            const modelOption = runCommand?.options.find((opt) => opt.long === '--model');
+            const noConfigOption = runCommand?.options.find((opt) => opt.long === '--no-config');
+            const yesOption = runCommand?.options.find((opt) => opt.long === '--yes');
             (0, vitest_1.expect)(modelOption).toBeDefined();
-            (0, vitest_1.expect)(modelOption === null || modelOption === void 0 ? void 0 : modelOption.description).toContain('Model to use');
+            (0, vitest_1.expect)(modelOption?.description).toContain('Model to use');
             (0, vitest_1.expect)(noConfigOption).toBeDefined();
-            (0, vitest_1.expect)(noConfigOption === null || noConfigOption === void 0 ? void 0 : noConfigOption.description).toContain('Run without loading project config');
+            (0, vitest_1.expect)(noConfigOption?.description).toContain('Run without loading project config');
             (0, vitest_1.expect)(yesOption).toBeDefined();
-            (0, vitest_1.expect)(yesOption === null || yesOption === void 0 ? void 0 : yesOption.description).toContain('Automatically answer yes');
+            (0, vitest_1.expect)(yesOption?.description).toContain('Automatically answer yes');
         });
-        (0, vitest_1.it)('should load configuration from opencode.json', () => __awaiter(void 0, void 0, void 0, function* () {
+        (0, vitest_1.it)('should load configuration from opencode.json', async () => {
             const mockConfig = {
-                model: 'berget/glm-4-6',
                 apiKey: 'test-api-key',
+                created: '2023-01-01T00:00:00.000Z',
+                model: 'berget/glm-4-6',
                 projectName: 'testproject',
                 provider: 'berget',
-                created: '2023-01-01T00:00:00.000Z',
                 version: '1.0.0',
             };
             // Mock file exists and contains config
             mockFs.existsSync.mockReturnValue(true);
             mockFsPromises.readFile.mockResolvedValue(JSON.stringify(mockConfig));
             // Mock successful opencode check
-            mockSpawn.mockImplementation((command, args) => {
-                if (command === 'opencode' && args[0] === '--version') {
+            mockSpawn.mockImplementation((command, arguments_) => {
+                if (command === 'opencode' && arguments_[0] === '--version') {
                     return {
                         on: vitest_1.vi.fn().mockImplementation((event, callback) => {
                             if (event === 'close')
@@ -256,17 +247,17 @@ vitest_1.vi.mock('readline', () => ({
             (0, vitest_1.expect)(mockConfig.model).toBe('berget/glm-4-6');
             (0, vitest_1.expect)(mockConfig.apiKey).toBe('test-api-key');
             (0, vitest_1.expect)(mockConfig.projectName).toBe('testproject');
-        }));
+        });
         (0, vitest_1.it)('should spawn opencode with correct arguments', () => {
             const codeCommand = program.commands.find((cmd) => cmd.name() === 'code');
-            const runCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'run');
+            const runCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'run');
             (0, vitest_1.expect)(runCommand).toBeDefined();
             // Should spawn opencode with appropriate arguments
             (0, vitest_1.expect)(mockSpawn).toBeDefined();
         });
         (0, vitest_1.it)('should handle missing configuration file', () => {
             const codeCommand = program.commands.find((cmd) => cmd.name() === 'code');
-            const runCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'run');
+            const runCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'run');
             (0, vitest_1.expect)(runCommand).toBeDefined();
             // Should check if opencode.json exists
             (0, vitest_1.expect)(mockFs.existsSync).toBeDefined();
@@ -279,8 +270,8 @@ vitest_1.vi.mock('readline', () => ({
         });
         (0, vitest_1.it)('should offer to install opencode if not found', () => {
             // Mock opencode not installed
-            mockSpawn.mockImplementation((command, args) => {
-                if (command === 'opencode' && args[0] === '--version') {
+            mockSpawn.mockImplementation((command, arguments_) => {
+                if (command === 'opencode' && arguments_[0] === '--version') {
                     return {
                         on: vitest_1.vi.fn().mockImplementation((event, callback) => {
                             if (event === 'close')
@@ -301,19 +292,19 @@ vitest_1.vi.mock('readline', () => ({
     (0, vitest_1.describe)('automation support', () => {
         (0, vitest_1.it)('should support -y flag for automated initialization', () => {
             const codeCommand = program.commands.find((cmd) => cmd.name() === 'code');
-            const initCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'init');
+            const initCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'init');
             (0, vitest_1.expect)(initCommand).toBeDefined();
-            const yesOption = initCommand === null || initCommand === void 0 ? void 0 : initCommand.options.find((opt) => opt.long === '--yes');
+            const yesOption = initCommand?.options.find((opt) => opt.long === '--yes');
             (0, vitest_1.expect)(yesOption).toBeDefined();
-            (0, vitest_1.expect)(yesOption === null || yesOption === void 0 ? void 0 : yesOption.description).toContain('automation');
+            (0, vitest_1.expect)(yesOption?.description).toContain('automation');
         });
         (0, vitest_1.it)('should support -y flag for automated run', () => {
             const codeCommand = program.commands.find((cmd) => cmd.name() === 'code');
-            const runCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'run');
+            const runCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'run');
             (0, vitest_1.expect)(runCommand).toBeDefined();
-            const yesOption = runCommand === null || runCommand === void 0 ? void 0 : runCommand.options.find((opt) => opt.long === '--yes');
+            const yesOption = runCommand?.options.find((opt) => opt.long === '--yes');
             (0, vitest_1.expect)(yesOption).toBeDefined();
-            (0, vitest_1.expect)(yesOption === null || yesOption === void 0 ? void 0 : yesOption.description).toContain('automation');
+            (0, vitest_1.expect)(yesOption?.description).toContain('automation');
         });
         (0, vitest_1.it)('should use BERGET_API_KEY environment variable in automation mode', () => {
             // Test that environment variable is used when -y flag is set
@@ -324,65 +315,65 @@ vitest_1.vi.mock('readline', () => ({
         });
     });
     (0, vitest_1.describe)('.env file handling', () => {
-        let mockUpdateEnvFile;
+        let mockUpdateEnvironmentFile;
         (0, vitest_1.beforeEach)(() => {
-            mockUpdateEnvFile = vitest_1.vi.mocked(env_manager_1.updateEnvFile);
+            mockUpdateEnvironmentFile = vitest_1.vi.mocked(env_manager_1.updateEnvFile);
         });
-        (0, vitest_1.it)('should call updateEnvFile when creating new project', () => __awaiter(void 0, void 0, void 0, function* () {
-            mockUpdateEnvFile.mockResolvedValue(true);
+        (0, vitest_1.it)('should call updateEnvFile when creating new project', async () => {
+            mockUpdateEnvironmentFile.mockResolvedValue(true);
             mockFs.existsSync.mockReturnValue(false); // .env doesn't exist
-            mockFsPromises.writeFile.mockResolvedValue(undefined);
+            mockFsPromises.writeFile.mockResolvedValue();
             // This would be tested by actually calling the init command
             // For now we verify the mock is properly set up
-            (0, vitest_1.expect)(mockUpdateEnvFile).toBeDefined();
-        }));
-        (0, vitest_1.it)('should not overwrite existing BERGET_API_KEY in .env', () => __awaiter(void 0, void 0, void 0, function* () {
+            (0, vitest_1.expect)(mockUpdateEnvironmentFile).toBeDefined();
+        });
+        (0, vitest_1.it)('should not overwrite existing BERGET_API_KEY in .env', async () => {
             const consoleSpy = vitest_1.vi.spyOn(console, 'log').mockImplementation(() => { });
             // Mock existing .env with BERGET_API_KEY
             mockFs.existsSync.mockReturnValue(true);
             mockFs.readFileSync.mockReturnValue('BERGET_API_KEY=existing_key\nOTHER_KEY=value\n');
             // Mock updateEnvFile to simulate the check
-            mockUpdateEnvFile.mockImplementation((options) => __awaiter(void 0, void 0, void 0, function* () {
+            mockUpdateEnvironmentFile.mockImplementation(async (options) => {
                 if (options.key === 'BERGET_API_KEY' && !options.force) {
                     console.log(`⚠ ${options.key} already exists in .env - leaving unchanged`);
                     return false;
                 }
                 return true;
-            }));
-            yield (0, env_manager_1.updateEnvFile)({
+            });
+            await (0, env_manager_1.updateEnvFile)({
                 key: 'BERGET_API_KEY',
                 value: 'new_key',
             });
             (0, vitest_1.expect)(consoleSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('BERGET_API_KEY already exists in .env - leaving unchanged'));
             consoleSpy.mockRestore();
-        }));
-        (0, vitest_1.it)('should add new key to existing .env file', () => __awaiter(void 0, void 0, void 0, function* () {
+        });
+        (0, vitest_1.it)('should add new key to existing .env file', async () => {
             mockFs.existsSync.mockReturnValue(true);
             mockFs.readFileSync.mockReturnValue('EXISTING_KEY=value\n');
-            mockUpdateEnvFile.mockResolvedValue(true);
-            yield (0, env_manager_1.updateEnvFile)({
+            mockUpdateEnvironmentFile.mockResolvedValue(true);
+            await (0, env_manager_1.updateEnvFile)({
+                comment: 'Berget AI Configuration',
                 key: 'BERGET_API_KEY',
                 value: 'new_api_key',
-                comment: 'Berget AI Configuration',
             });
-            (0, vitest_1.expect)(mockUpdateEnvFile).toHaveBeenCalledWith({
+            (0, vitest_1.expect)(mockUpdateEnvironmentFile).toHaveBeenCalledWith({
+                comment: 'Berget AI Configuration',
                 key: 'BERGET_API_KEY',
                 value: 'new_api_key',
-                comment: 'Berget AI Configuration',
             });
-        }));
-        (0, vitest_1.it)('should create new .env file when none exists', () => __awaiter(void 0, void 0, void 0, function* () {
+        });
+        (0, vitest_1.it)('should create new .env file when none exists', async () => {
             mockFs.existsSync.mockReturnValue(false);
-            mockUpdateEnvFile.mockResolvedValue(true);
-            yield (0, env_manager_1.updateEnvFile)({
+            mockUpdateEnvironmentFile.mockResolvedValue(true);
+            await (0, env_manager_1.updateEnvFile)({
                 key: 'BERGET_API_KEY',
                 value: 'new_api_key',
             });
-            (0, vitest_1.expect)(mockUpdateEnvFile).toHaveBeenCalledWith({
+            (0, vitest_1.expect)(mockUpdateEnvironmentFile).toHaveBeenCalledWith({
                 key: 'BERGET_API_KEY',
                 value: 'new_api_key',
             });
-        }));
+        });
     });
     (0, vitest_1.describe)('error handling', () => {
         (0, vitest_1.it)('should handle API key creation failures', () => {
@@ -402,26 +393,26 @@ vitest_1.vi.mock('readline', () => ({
             });
             (0, vitest_1.expect)(mockSpawn).toBeDefined();
         });
-        (0, vitest_1.it)('should handle .env update failures', () => __awaiter(void 0, void 0, void 0, function* () {
-            const mockUpdateEnvFile = vitest_1.vi.mocked(env_manager_1.updateEnvFile);
-            mockUpdateEnvFile.mockRejectedValue(new Error('Env update failed'));
-            yield (0, vitest_1.expect)((0, env_manager_1.updateEnvFile)({
+        (0, vitest_1.it)('should handle .env update failures', async () => {
+            const mockUpdateEnvironmentFile = vitest_1.vi.mocked(env_manager_1.updateEnvFile);
+            mockUpdateEnvironmentFile.mockRejectedValue(new Error('Env update failed'));
+            await (0, vitest_1.expect)((0, env_manager_1.updateEnvFile)({
                 key: 'TEST_KEY',
                 value: 'test_value',
             })).rejects.toThrow('Env update failed');
-        }));
+        });
     });
     (0, vitest_1.describe)('experimental features', () => {
-        let originalEnv;
+        let originalEnvironment;
         (0, vitest_1.beforeEach)(() => {
-            originalEnv = process.env.BERGET_EXPERIMENTAL;
+            originalEnvironment = process.env.BERGET_EXPERIMENTAL;
         });
         (0, vitest_1.afterEach)(() => {
-            if (originalEnv === undefined) {
+            if (originalEnvironment === undefined) {
                 delete process.env.BERGET_EXPERIMENTAL;
             }
             else {
-                process.env.BERGET_EXPERIMENTAL = originalEnv;
+                process.env.BERGET_EXPERIMENTAL = originalEnvironment;
             }
         });
         (0, vitest_1.it)('should NOT show setup command when BERGET_EXPERIMENTAL is not set', () => {
@@ -429,7 +420,7 @@ vitest_1.vi.mock('readline', () => ({
             const freshProgram = new commander_1.Command();
             (0, code_1.registerCodeCommands)(freshProgram);
             const codeCommand = freshProgram.commands.find((cmd) => cmd.name() === 'code');
-            const setupCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'setup');
+            const setupCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'setup');
             (0, vitest_1.expect)(setupCommand).toBeUndefined();
         });
         (0, vitest_1.it)('should show setup command when BERGET_EXPERIMENTAL is set', () => {
@@ -437,9 +428,9 @@ vitest_1.vi.mock('readline', () => ({
             const freshProgram = new commander_1.Command();
             (0, code_1.registerCodeCommands)(freshProgram);
             const codeCommand = freshProgram.commands.find((cmd) => cmd.name() === 'code');
-            const setupCommand = codeCommand === null || codeCommand === void 0 ? void 0 : codeCommand.commands.find((cmd) => cmd.name() === 'setup');
+            const setupCommand = codeCommand?.commands.find((cmd) => cmd.name() === 'setup');
             (0, vitest_1.expect)(setupCommand).toBeDefined();
-            (0, vitest_1.expect)(setupCommand === null || setupCommand === void 0 ? void 0 : setupCommand.description()).toBe('Interactive setup for Berget AI coding tools');
+            (0, vitest_1.expect)(setupCommand?.description()).toBe('Interactive setup for Berget AI coding tools');
         });
     });
 });

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

@@ -5,9 +5,9 @@ const config_loader_1 = require("../../src/utils/config-loader");
 // Mock fs module
 const mockFs = vitest_1.vi.hoisted(() => ({
     existsSync: vitest_1.vi.fn(),
+    mkdirSync: 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', () => {
@@ -33,14 +33,14 @@ vitest_1.vi.mock('fs', () => mockFs);
                 const modelConfig = configLoader.getModelConfig();
                 (0, vitest_1.expect)(modelConfig).toEqual({
                     primary: 'berget/glm-4.7',
-                    small: 'berget/gpt-oss'
+                    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'
+                    small: 'berget/gpt-oss',
                 });
             });
         });
@@ -49,42 +49,42 @@ vitest_1.vi.mock('fs', () => mockFs);
                 const models = configLoader.getProviderModels();
                 (0, vitest_1.expect)(models).toEqual({
                     'glm-4.7': {
+                        limit: { context: 90000, output: 4000 },
                         name: 'GLM-4.7',
-                        limit: { output: 4000, context: 90000 }
                     },
                     'gpt-oss': {
-                        name: 'GPT-OSS',
-                        limit: { output: 4000, context: 128000 },
+                        limit: { context: 128000, output: 4000 },
                         modalities: {
                             input: ['text', 'image'],
-                            output: ['text']
-                        }
+                            output: ['text'],
+                        },
+                        name: 'GPT-OSS',
                     },
                     'llama-8b': {
+                        limit: { context: 128000, output: 4000 },
                         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': {
+                        limit: { context: 90000, output: 4000 },
                         name: 'GLM-4.7',
-                        limit: { output: 4000, context: 90000 }
                     },
                     'gpt-oss': {
-                        name: 'GPT-OSS',
-                        limit: { output: 4000, context: 128000 },
+                        limit: { context: 128000, output: 4000 },
                         modalities: {
                             input: ['text', 'image'],
-                            output: ['text']
-                        }
+                            output: ['text'],
+                        },
+                        name: 'GPT-OSS',
                     },
                     'llama-8b': {
+                        limit: { context: 128000, output: 4000 },
                         name: 'llama-3.1-8b',
-                        limit: { output: 4000, context: 128000 }
-                    }
+                    },
                 });
             });
         });
@@ -107,38 +107,38 @@ vitest_1.vi.mock('fs', () => mockFs);
     });
     (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',
+                    model: 'custom-agent-model',
                     permission: {
-                        edit: 'allow',
                         bash: 'allow',
-                        webfetch: 'allow'
-                    }
-                }
+                        edit: 'allow',
+                        webfetch: 'allow',
+                    },
+                    temperature: 0.5,
+                },
             },
             command: {
                 test: {
-                    description: 'Test command'
-                }
-            },
-            watcher: {
-                ignore: ['custom-ignore']
+                    description: 'Test command',
+                },
             },
+            model: 'custom-model',
             provider: {
                 berget: {
                     models: {
                         'custom-model': {
+                            limit: { context: 160000, output: 8000 },
                             name: 'Custom Model',
-                            limit: { output: 8000, context: 160000 }
-                        }
-                    }
-                }
-            }
+                        },
+                    },
+                },
+            },
+            small_model: 'custom-small-model',
+            watcher: {
+                ignore: ['custom-ignore'],
+            },
         };
         (0, vitest_1.beforeEach)(() => {
             mockFs.existsSync.mockReturnValue(true);
@@ -149,7 +149,7 @@ vitest_1.vi.mock('fs', () => mockFs);
                 const modelConfig = configLoader.getModelConfig();
                 (0, vitest_1.expect)(modelConfig).toEqual({
                     primary: 'custom-model',
-                    small: 'custom-small-model'
+                    small: 'custom-small-model',
                 });
             });
         });
@@ -158,9 +158,9 @@ vitest_1.vi.mock('fs', () => mockFs);
                 const models = configLoader.getProviderModels();
                 (0, vitest_1.expect)(models).toEqual({
                     'custom-model': {
+                        limit: { context: 160000, output: 8000 },
                         name: 'Custom Model',
-                        limit: { output: 8000, context: 160000 }
-                    }
+                    },
                 });
             });
         });
@@ -190,28 +190,28 @@ vitest_1.vi.mock('fs', () => mockFs);
             const modelConfig = configLoader.getModelConfig();
             (0, vitest_1.expect)(modelConfig).toEqual({
                 primary: 'berget/glm-4.7',
-                small: 'berget/gpt-oss'
+                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': {
+                    limit: { context: 90000, output: 4000 },
                     name: 'GLM-4.7',
-                    limit: { output: 4000, context: 90000 }
                 },
                 'gpt-oss': {
-                    name: 'GPT-OSS',
-                    limit: { output: 4000, context: 128000 },
+                    limit: { context: 128000, output: 4000 },
                     modalities: {
                         input: ['text', 'image'],
-                        output: ['text']
-                    }
+                        output: ['text'],
+                    },
+                    name: 'GPT-OSS',
                 },
                 'llama-8b': {
+                    limit: { context: 128000, output: 4000 },
                     name: 'llama-3.1-8b',
-                    limit: { output: 4000, context: 128000 }
-                }
+                },
             });
         });
         (0, vitest_1.it)('should fall back to defaults for getAllAgentConfigs', () => {
@@ -246,7 +246,7 @@ vitest_1.vi.mock('fs', () => mockFs);
             // And return sensible defaults
             (0, vitest_1.expect)(configLoader.getModelConfig()).toEqual({
                 primary: 'berget/glm-4.7',
-                small: 'berget/gpt-oss'
+                small: 'berget/gpt-oss',
             });
             (0, vitest_1.expect)(configLoader.getAllAgentConfigs()).toEqual({});
             (0, vitest_1.expect)(configLoader.getAgentConfig('fullstack')).toBeNull();
@@ -259,7 +259,7 @@ vitest_1.vi.mock('fs', () => mockFs);
             (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'
+                small: 'berget/gpt-oss',
             });
             (0, vitest_1.expect)((0, config_loader_1.getAllAgentConfigs)(testConfigPath)).toEqual({});
         });