berget 2.2.7 → 2.2.9

package/dist/tests/utils/opencode-validator.test.js

@@ -1,93 +1,92 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const node_fs_1 = require("node:fs");
 const vitest_1 = require("vitest");
 const opencode_validator_1 = require("../../src/utils/opencode-validator");
-const fs_1 = require("fs");
-(0, vitest_1.describe)("OpenCode Validator", () => {
-    (0, vitest_1.it)("should validate a correct OpenCode configuration", () => {
+(0, vitest_1.describe)('OpenCode Validator', () => {
+    (0, vitest_1.it)('should validate a correct OpenCode configuration', () => {
         const validConfig = {
-            $schema: "https://opencode.ai/config.json",
-            username: "test-user",
-            model: "gpt-4",
+            $schema: 'https://opencode.ai/config.json',
             agent: {
                 test: {
-                    model: "gpt-4",
-                    temperature: 0.7,
-                    prompt: "Test agent",
+                    model: 'gpt-4',
                     permission: {
-                        edit: "allow",
-                        bash: "allow",
-                        webfetch: "allow",
+                        bash: 'allow',
+                        edit: 'allow',
+                        webfetch: 'allow',
                     },
+                    prompt: 'Test agent',
+                    temperature: 0.7,
                 },
             },
+            model: 'gpt-4',
+            username: 'test-user',
         };
         const result = (0, opencode_validator_1.validateOpenCodeConfig)(validConfig);
         (0, vitest_1.expect)(result.valid).toBe(true);
         (0, vitest_1.expect)(result.errors).toBeUndefined();
     });
-    (0, vitest_1.it)("should reject invalid configuration", () => {
+    (0, vitest_1.it)('should reject invalid configuration', () => {
         const invalidConfig = {
-            username: 123, // Should be string
-            model: "gpt-4",
             agent: {
                 test: {
-                    model: "gpt-4",
-                    temperature: "high", // Should be number
-                    prompt: "Test agent",
+                    model: 'gpt-4',
                     permission: {
-                        edit: "invalid", // Should be enum value
-                        bash: "allow",
-                        webfetch: "allow",
+                        bash: 'allow',
+                        edit: 'invalid', // Should be enum value
+                        webfetch: 'allow',
                     },
+                    prompt: 'Test agent',
+                    temperature: 'high', // Should be number
                 },
             },
+            model: 'gpt-4',
+            username: 123, // Should be string
         };
         const result = (0, opencode_validator_1.validateOpenCodeConfig)(invalidConfig);
         (0, vitest_1.expect)(result.valid).toBe(false);
         (0, vitest_1.expect)(result.errors).toBeDefined();
         (0, vitest_1.expect)(result.errors.length).toBeGreaterThan(0);
     });
-    (0, vitest_1.it)("should fix common configuration issues", () => {
+    (0, vitest_1.it)('should fix common configuration issues', () => {
         const configWithIssues = {
-            username: "test-user",
-            model: "gpt-4",
-            tools: {
-                compact: { threshold: 80000 }, // Should be boolean
-            },
             maxTokens: 4000, // Invalid property
+            model: 'gpt-4',
             provider: {
                 berget: {
                     models: {
-                        "test-model": {
-                            name: "Test Model",
-                            maxTokens: 4000, // Should be moved to limit.context
+                        'test-model': {
                             contextWindow: 8000, // Should be moved to limit.context
+                            maxTokens: 4000, // Should be moved to limit.context
+                            name: 'Test Model',
                         },
                     },
                 },
             },
+            tools: {
+                compact: { threshold: 80000 }, // Should be boolean
+            },
+            username: 'test-user',
         };
         const fixed = (0, opencode_validator_1.fixOpenCodeConfig)(configWithIssues);
         // tools.compact should be boolean
-        (0, vitest_1.expect)(typeof fixed.tools.compact).toBe("boolean");
+        (0, vitest_1.expect)(typeof fixed.tools.compact).toBe('boolean');
         // maxTokens should be removed
         (0, vitest_1.expect)(fixed.maxTokens).toBeUndefined();
         // maxTokens and contextWindow should be moved to limit.context
-        (0, vitest_1.expect)(fixed.provider.berget.models["test-model"].limit).toBeDefined();
-        (0, vitest_1.expect)(fixed.provider.berget.models["test-model"].limit.context).toBe(8000);
-        (0, vitest_1.expect)(fixed.provider.berget.models["test-model"].maxTokens).toBeUndefined();
-        (0, vitest_1.expect)(fixed.provider.berget.models["test-model"].contextWindow).toBeUndefined();
+        (0, vitest_1.expect)(fixed.provider.berget.models['test-model'].limit).toBeDefined();
+        (0, vitest_1.expect)(fixed.provider.berget.models['test-model'].limit.context).toBe(8000);
+        (0, vitest_1.expect)(fixed.provider.berget.models['test-model'].maxTokens).toBeUndefined();
+        (0, vitest_1.expect)(fixed.provider.berget.models['test-model'].contextWindow).toBeUndefined();
     });
-    (0, vitest_1.it)("should validate the current opencode.json file", () => {
-        var _a;
+    (0, vitest_1.it)('should validate the current opencode.json file', () => {
         let currentConfig;
         try {
-            currentConfig = JSON.parse((0, fs_1.readFileSync)("opencode.json", "utf8"));
+            currentConfig = JSON.parse((0, node_fs_1.readFileSync)('opencode.json', 'utf8'));
         }
         catch (error) {
             // Skip when opencode.json is not present (e.g. in CI or clean checkouts)
-            console.log("Skipping: opencode.json not found:", error);
+            console.log('Skipping: opencode.json not found:', error);
             return;
         }
         // Apply fixes to handle common issues
@@ -97,8 +96,10 @@ const fs_1 = require("fs");
         // The fixed config should be valid according to the JSON Schema
         (0, vitest_1.expect)(result.valid).toBe(true);
         if (!result.valid) {
-            console.log("Fixed opencode.json validation errors:");
-            (_a = result.errors) === null || _a === void 0 ? void 0 : _a.forEach(err => console.log(`  - ${err}`));
+            console.log('Fixed opencode.json validation errors:');
+            if (result.errors)
+                for (const error of result.errors)
+                    console.log(`  - ${error}`);
         }
     });
 });

package/dist/vitest.config.js

@@ -3,7 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const config_1 = require("vitest/config");
 exports.default = (0, config_1.defineConfig)({
     test: {
+        environment: 'node',
         globals: true,
-        environment: "node",
     },
 });

package/eslint.config.mjs

@@ -1,47 +1,82 @@
-import typescriptEslint from "@typescript-eslint/eslint-plugin";
-import typescriptParser from "@typescript-eslint/parser";
-import prettier from "eslint-plugin-prettier";
+import vitest from '@vitest/eslint-plugin';
+import perfectionist from 'eslint-plugin-perfectionist';
+import prettier from 'eslint-plugin-prettier';
+import promise from 'eslint-plugin-promise';
+import sonarjs from 'eslint-plugin-sonarjs';
+import tseslint from 'typescript-eslint';
 
-export default [
+export default tseslint.config(
   {
     ignores: [
-      "node_modules/**",
-      "dist/**",
-      "build/**",
-      "*.lock",
-      ".husky/**",
-      "coverage/**",
-      ".nyc_output/**",
-      "test-results/**",
-      "playwright-report/**",
-      ".pi/**",
+      'node_modules/**',
+      'dist/**',
+      'build/**',
+      '*.lock',
+      '.husky/**',
+      'coverage/**',
+      '.nyc_output/**',
+      'test-results/**',
+      'playwright-report/**',
+      '.pi/**',
+      'src/types/api.d.ts',
     ],
   },
+  ...tseslint.configs.recommended,
+  ...tseslint.configs.strict,
+  sonarjs.configs.recommended,
+  promise.configs['flat/recommended'],
+  perfectionist.configs['recommended-natural'],
   {
-    files: ["**/*.ts", "**/*.tsx"],
+    files: ['**/*.test.ts', '**/*.spec.ts', '**/*.test.tsx', '**/*.spec.tsx'],
     languageOptions: {
-      parser: typescriptParser,
+      globals: {
+        ...vitest.environments.env.globals,
+      },
+    },
+    plugins: {
+      vitest,
+    },
+    rules: {
+      ...vitest.configs.recommended.rules,
+      '@typescript-eslint/no-explicit-any': 'off',
+      '@typescript-eslint/no-non-null-assertion': 'off',
+    },
+  },
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    languageOptions: {
+      ecmaVersion: 'latest',
       parserOptions: {
-        ecmaVersion: "latest",
-        sourceType: "commonjs",
-        project: "./tsconfig.json",
+        project: './tsconfig.json',
         tsconfigRootDir: process.cwd(),
       },
-      globals: {
-        node: "readonly",
-        es2022: "readonly",
-      },
+      sourceType: 'commonjs',
     },
     plugins: {
-      "@typescript-eslint": typescriptEslint,
-      prettier: prettier,
+      prettier,
     },
     rules: {
       ...prettier.configs.recommended.rules,
-      "@typescript-eslint/no-unused-vars": ["warn", { argsIgnorePattern: "^_" }],
-      "@typescript-eslint/no-explicit-any": "warn",
-      "@typescript-eslint/explicit-function-return-type": "off",
-      "@typescript-eslint/no-floating-promises": "off",
+      '@typescript-eslint/no-dynamic-delete': 'off',
+      '@typescript-eslint/no-explicit-any': 'off',
+      '@typescript-eslint/no-non-null-assertion': 'off',
+      '@typescript-eslint/no-unused-vars': [
+        'error',
+        { argsIgnorePattern: '^_', varsIgnorePattern: '^_' },
+      ],
+      'sonarjs/cognitive-complexity': 'off',
+      'sonarjs/different-types-comparison': 'off',
+      'sonarjs/function-return-type': 'off',
+      'sonarjs/no-duplicated-branches': 'off',
+      'sonarjs/no-nested-conditional': 'off',
+      'sonarjs/no-nested-functions': 'off',
+      'sonarjs/no-nested-template-literals': 'off',
+      'sonarjs/no-os-command-from-path': 'off',
+      'sonarjs/no-unused-vars': 'off',
+      'sonarjs/prefer-regexp-exec': 'off',
+      'sonarjs/publicly-writable-directories': 'off',
+      'sonarjs/redundant-type-aliases': 'off',
+      'sonarjs/slow-regex': 'off',
     },
   },
-];
+);

package/index.ts

@@ -1,16 +1,15 @@
-#!/usr/bin/env node
+import chalk from 'chalk';
+import { Option, program } from 'commander';
 
-import { program, Option } from "commander";
-import { registerCommands } from "./src/commands";
-import { checkBergetConfig } from "./src/utils/config-checker";
-import chalk from "chalk";
-import { version } from "./package.json";
-process.env.DOTENV_CONFIG_OVERRIDE = "true";
-import "dotenv/config";
+import { version } from './package.json';
+import { registerCommands } from './src/commands';
+import { checkBergetConfig } from './src/utils/config-checker';
+process.env.DOTENV_CONFIG_OVERRIDE = 'true';
+import 'dotenv/config';
 
 // Set version and description
 program
-  .name("berget")
+  .name('berget')
   .description(
     `______                     _      ___  _____ 
 | ___ \\                   | |    / _ \\|_   _|
@@ -20,12 +19,12 @@ program
 \\____/ \\___|_|  \\__, |\\___|\\_\\_ \\_| |_/\\___/ 
                  __/ |                      
                 |___/   AI on European terms
-Version: ${version}`
+Version: ${version}`,
   )
-  .version(version, "-v, --version")
-  .addOption(new Option("--local").default(false).hideHelp())
-  .addOption(new Option("--stage").default(false).hideHelp())
-  .option("--debug", "Enable debug output", false);
+  .version(version, '-v, --version')
+  .addOption(new Option('--local').default(false).hideHelp())
+  .addOption(new Option('--stage').default(false).hideHelp())
+  .option('--debug', 'Enable debug output', false);
 
 // Register all commands
 registerCommands(program);
@@ -41,42 +40,42 @@ if (process.argv.length <= 2) {
 
 // Add helpful suggestions for common command mistakes
 const commonMistakes: Record<string, string> = {
-  login: "auth login",
-  logout: "auth logout",
-  whoami: "auth whoami",
-  "list-models": "models list",
-  "list-keys": "api-keys list",
-  "create-key": "api-keys create",
-  "list-clusters": "clusters list",
-  usage: "billing usage",
-  init: "code init",
+  'create-key': 'api-keys create',
+  init: 'code init',
+  'list-clusters': 'clusters list',
+  'list-keys': 'api-keys list',
+  'list-models': 'models list',
+  login: 'auth login',
+  logout: 'auth logout',
+  usage: 'billing usage',
+  whoami: 'auth whoami',
 };
 
 // Add error handler for unknown commands
-program.on("command:*", operands => {
+program.on('command:*', (operands) => {
   const unknownCommand = operands[0] as string;
   console.error(chalk.red(`Error: unknown command '${unknownCommand}'`));
 
   // Check if this is a known mistake and suggest the correct command
   if (unknownCommand in commonMistakes) {
     console.log(
-      chalk.yellow(`Did you mean? ${chalk.bold(`berget ${commonMistakes[unknownCommand]}`)}`)
+      chalk.yellow(`Did you mean? ${chalk.bold(`berget ${commonMistakes[unknownCommand]}`)}`),
     );
   } else {
     // Try to find similar commands
-    const availableCommands = program.commands.map(cmd => cmd.name());
+    const availableCommands = program.commands.map((cmd) => cmd.name());
     const similarCommands = availableCommands.filter(
-      cmd => cmd.includes(unknownCommand) || unknownCommand.includes(cmd)
+      (cmd) => cmd.includes(unknownCommand) || unknownCommand.includes(cmd),
     );
 
     if (similarCommands.length > 0) {
-      console.log(chalk.yellow("Similar commands:"));
-      similarCommands.forEach(cmd => {
+      console.log(chalk.yellow('Similar commands:'));
+      for (const cmd of similarCommands) {
         console.log(chalk.yellow(`  ${chalk.bold(`berget ${cmd}`)}`));
-      });
+      }
     }
 
-    console.log(chalk.blue("\nRun `berget --help` for a list of available commands."));
+    console.log(chalk.blue('\nRun `berget --help` for a list of available commands.'));
   }
 
   process.exit(1);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "berget",
-  "version": "2.2.7",
+  "version": "2.2.9",
   "main": "dist/index.js",
   "bin": {
     "berget": "dist/index.js"
@@ -37,16 +37,19 @@
     "@types/marked": "^5.0.2",
     "@types/marked-terminal": "^6.1.1",
     "@types/node": "^20.11.20",
-    "@typescript-eslint/eslint-plugin": "^8.59.3",
-    "@typescript-eslint/parser": "^8.59.3",
+    "@vitest/eslint-plugin": "^1.6.17",
     "eslint": "^10.3.0",
     "eslint-config-prettier": "^10.1.8",
+    "eslint-plugin-perfectionist": "^5.9.0",
     "eslint-plugin-prettier": "^5.5.5",
+    "eslint-plugin-promise": "^7.3.0",
+    "eslint-plugin-sonarjs": "^3.0.2",
     "husky": "^9.1.7",
     "lint-staged": "^17.0.4",
     "prettier": "^3.8.3",
     "tsx": "^4.19.3",
     "typescript": "^5.3.3",
+    "typescript-eslint": "^8.59.3",
     "vitest": "^1.0.0"
   },
   "dependencies": {
@@ -57,6 +60,7 @@
     "commander": "^12.0.0",
     "dotenv": "^17.2.3",
     "fs-extra": "^11.3.0",
+    "globals": "^17.6.0",
     "jsonc-parser": "^3.3.1",
     "marked": "^9.1.6",
     "marked-terminal": "^6.2.0",

package/src/agents/app.ts

@@ -1,17 +1,17 @@
-import { Agent } from "./types.js";
+import { Agent } from './types.js';
 
 export const agent: Agent = {
   config: {
-    name: "app",
-    description: "Expo + React Native apps; props-first, offline-aware, shared tokens.",
-    mode: "primary",
-    temperature: 0.4,
-    top_p: 0.9,
+    description: 'Expo + React Native apps; props-first, offline-aware, shared tokens.',
+    mode: 'primary',
+    name: 'app',
     permission: {
-      edit: "allow",
-      bash: "deny",
-      webfetch: "allow",
+      bash: 'deny',
+      edit: 'allow',
+      webfetch: 'allow',
     },
+    temperature: 0.4,
+    top_p: 0.9,
   },
   systemPrompt: `You are Berget Code App agent. Voice: Scandinavian calm—precise, concise, confident. Expo + React Native + TypeScript. Structure by components/hooks/services/navigation. Components are pure; data via props; refactor shared logic into hooks/stores. Share tokens with frontend. Mock data in /data via typed hooks; later replace with live APIs. Offline via SQLite/MMKV; notifications via Expo. Request permissions only when needed. Subtle, meaningful motion; light/dark parity.
 

package/src/agents/backend.ts

@@ -1,10 +1,10 @@
-import { Agent } from "./types.js";
+import { Agent } from './types.js';
 
 export const agent: Agent = {
   config: {
-    name: "backend",
-    description: "Functional, modular Koa + TypeScript services",
-    mode: "primary",
+    description: 'Functional, modular Koa + TypeScript services',
+    mode: 'primary',
+    name: 'backend',
   },
   systemPrompt: `# Backend Agent
 

package/src/agents/devops.ts

@@ -1,17 +1,17 @@
-import { Agent } from "./types.js";
+import { Agent } from './types.js';
 
 export const agent: Agent = {
   config: {
-    name: "devops",
-    description: "Declarative GitOps infra with FluxCD, Kustomize, Helm, operators.",
-    mode: "primary",
-    temperature: 0.3,
-    top_p: 0.8,
+    description: 'Declarative GitOps infra with FluxCD, Kustomize, Helm, operators.',
+    mode: 'primary',
+    name: 'devops',
     permission: {
-      edit: "allow",
-      bash: "allow",
-      webfetch: "allow",
+      bash: 'allow',
+      edit: 'allow',
+      webfetch: 'allow',
     },
+    temperature: 0.3,
+    top_p: 0.8,
   },
   systemPrompt: `You are Berget Code DevOps agent. Voice: Scandinavian calm—precise, concise, confident. Start simple: k8s/{deployment,service,ingress}. Add FluxCD sync to repo and image automation. Use Kustomize bases/overlays (staging, production). Add dependencies via Helm from upstream sources; prefer native operators when available (CloudNativePG, cert-manager, external-dns). SemVer with -rc tags keeps CI environments current. Observability with Prometheus/Grafana. No manual kubectl in production—Git is the source of truth.
 

package/src/agents/frontend.ts

@@ -1,10 +1,10 @@
-import { Agent } from "./types.js";
+import { Agent } from './types.js';
 
 export const agent: Agent = {
   config: {
-    name: "frontend",
-    description: "Scandinavian, type-safe UIs with React, Tailwind, and Shadcn",
-    mode: "primary",
+    description: 'Scandinavian, type-safe UIs with React, Tailwind, and Shadcn',
+    mode: 'primary',
+    name: 'frontend',
   },
   systemPrompt: `# Frontend Agent
 

package/src/agents/fullstack.ts

@@ -1,10 +1,10 @@
-import { Agent } from "./types.js";
+import { Agent } from './types.js';
 
 export const agent: Agent = {
   config: {
-    name: "fullstack",
-    description: "Router/coordinator agent for full-stack development",
-    mode: "primary",
+    description: 'Router/coordinator agent for full-stack development',
+    mode: 'primary',
+    name: 'fullstack',
   },
   systemPrompt: `# Fullstack Agent
 

package/src/agents/index.ts

@@ -1,31 +1,43 @@
-import type { Agent, AgentConfig } from "./types.js";
-import { agent as fullstack } from "./fullstack.js";
-import { agent as frontend } from "./frontend.js";
-import { agent as backend } from "./backend.js";
-import { agent as devops } from "./devops.js";
-import { agent as app } from "./app.js";
-import { agent as quality } from "./quality.js";
-import { agent as security } from "./security.js";
+import type { Agent } from './types.js';
+
+import { agent as app } from './app.js';
+import { agent as backend } from './backend.js';
+import { agent as devops } from './devops.js';
+import { agent as frontend } from './frontend.js';
+import { agent as fullstack } from './fullstack.js';
+import { agent as quality } from './quality.js';
+import { agent as security } from './security.js';
 
 const agents: Record<string, Agent> = {
-  fullstack,
-  frontend,
+  app,
   backend,
   devops,
-  app,
+  frontend,
+  fullstack,
   quality,
   security,
 };
 
 export { agents };
-export type { Agent, AgentConfig };
+
+export function getAgent(name: string): Agent | undefined {
+  return agents[name];
+}
 
 export function getAllAgents(): Agent[] {
   return Object.values(agents);
 }
 
-export function getAgent(name: string): Agent | undefined {
-  return agents[name];
+export function toAgentTemplate(agent: Agent): {
+  content: string;
+  description: string;
+  name: string;
+} {
+  return {
+    content: agent.systemPrompt,
+    description: agent.config.description,
+    name: agent.config.name,
+  };
 }
 
 export function toMarkdown(agent: Agent): string {
@@ -58,14 +70,4 @@ export function toPiPrompt(agent: Agent): string {
   return agent.systemPrompt;
 }
 
-export function toAgentTemplate(agent: Agent): {
-  name: string;
-  content: string;
-  description: string;
-} {
-  return {
-    name: agent.config.name,
-    description: agent.config.description,
-    content: agent.systemPrompt,
-  };
-}
+export { type Agent, type AgentConfig } from './types.js';

package/src/agents/quality.ts

@@ -1,17 +1,17 @@
-import { Agent } from "./types.js";
+import { Agent } from './types.js';
 
 export const agent: Agent = {
   config: {
-    name: "quality",
-    description: "Quality assurance specialist for testing, building, and complete PR management.",
-    mode: "subagent",
-    temperature: 0.1,
-    top_p: 0.9,
+    description: 'Quality assurance specialist for testing, building, and complete PR management.',
+    mode: 'subagent',
+    name: 'quality',
     permission: {
-      edit: "allow",
-      bash: "allow",
-      webfetch: "allow",
+      bash: 'allow',
+      edit: 'allow',
+      webfetch: 'allow',
     },
+    temperature: 0.1,
+    top_p: 0.9,
   },
   systemPrompt: `Voice: Scandinavian calm—precise, concise, confident. You are Berget Code Quality agent. Specialist in code quality assurance, testing, building, and pull request lifecycle management.
 

package/src/agents/security.ts

@@ -1,18 +1,18 @@
-import { Agent } from "./types.js";
+import { Agent } from './types.js';
 
 export const agent: Agent = {
   config: {
-    name: "security",
     description:
-      "Security specialist for pentesting, OWASP compliance, and vulnerability assessments.",
-    mode: "subagent",
-    temperature: 0.2,
-    top_p: 0.8,
+      'Security specialist for pentesting, OWASP compliance, and vulnerability assessments.',
+    mode: 'subagent',
+    name: 'security',
     permission: {
-      edit: "deny",
-      bash: "allow",
-      webfetch: "allow",
+      bash: 'allow',
+      edit: 'deny',
+      webfetch: 'allow',
     },
+    temperature: 0.2,
+    top_p: 0.8,
   },
   systemPrompt: `Voice: Scandinavian calm—precise, concise, confident. You are Berget Code Security agent. Expert in application security, penetration testing, and OWASP standards. Core responsibilities: Conduct security assessments and penetration tests, Validate OWASP Top 10 compliance, Review code for security vulnerabilities, Implement security headers and Content Security Policy (CSP), Audit API security, Check for sensitive data exposure, Validate input sanitization and output encoding, Assess dependency security and supply chain risks. Tools and techniques: OWASP ZAP, Burp Suite, security linters, dependency scanners, manual code review. Always provide specific, actionable security recommendations with priority levels.