berget 1.3.1 → 2.0.0

package/tests/utils/opencode-validator.test.ts

@@ -0,0 +1,118 @@
+import { describe, it, expect } from 'vitest'
+import {
+  validateOpenCodeConfig,
+  fixOpenCodeConfig,
+} from '../../src/utils/opencode-validator'
+import { readFileSync } from 'fs'
+
+describe('OpenCode Validator', () => {
+  it('should validate a correct OpenCode configuration', () => {
+    const validConfig = {
+      $schema: 'https://opencode.ai/config.json',
+      username: 'test-user',
+      model: 'gpt-4',
+      agent: {
+        test: {
+          model: 'gpt-4',
+          temperature: 0.7,
+          prompt: 'Test agent',
+          permission: {
+            edit: 'allow',
+            bash: 'allow',
+            webfetch: 'allow',
+          },
+        },
+      },
+    }
+
+    const result = validateOpenCodeConfig(validConfig)
+    expect(result.valid).toBe(true)
+    expect(result.errors).toBeUndefined()
+  })
+
+  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',
+          permission: {
+            edit: 'invalid', // Should be enum value
+            bash: 'allow',
+            webfetch: 'allow',
+          },
+        },
+      },
+    }
+
+    const result = validateOpenCodeConfig(invalidConfig)
+    expect(result.valid).toBe(false)
+    expect(result.errors).toBeDefined()
+    expect(result.errors!.length).toBeGreaterThan(0)
+  })
+
+  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
+      provider: {
+        berget: {
+          models: {
+            'test-model': {
+              name: 'Test Model',
+              maxTokens: 4000, // Should be moved to limit.context
+              contextWindow: 8000, // Should be moved to limit.context
+            },
+          },
+        },
+      },
+    }
+
+    const fixed = fixOpenCodeConfig(configWithIssues)
+
+    // tools.compact should be boolean
+    expect(typeof fixed.tools.compact).toBe('boolean')
+
+    // maxTokens should be removed
+    expect(fixed.maxTokens).toBeUndefined()
+
+    // maxTokens and contextWindow should be moved to limit.context
+    expect(fixed.provider.berget.models['test-model'].limit).toBeDefined()
+    expect(fixed.provider.berget.models['test-model'].limit.context).toBe(8000)
+    expect(fixed.provider.berget.models['test-model'].maxTokens).toBeUndefined()
+    expect(
+      fixed.provider.berget.models['test-model'].contextWindow,
+    ).toBeUndefined()
+  })
+
+  it('should validate the current opencode.json file', () => {
+    try {
+      const currentConfig = JSON.parse(readFileSync('opencode.json', 'utf8'))
+
+      // Apply fixes to handle common issues
+      const fixedConfig = fixOpenCodeConfig(currentConfig)
+
+      // Validate the fixed config
+      const result = validateOpenCodeConfig(fixedConfig)
+
+      // The fixed config should be valid according to the JSON Schema
+      expect(result.valid).toBe(true)
+
+      if (!result.valid) {
+        console.log('Fixed opencode.json validation errors:')
+        result.errors?.forEach((err) => console.log(`  - ${err}`))
+      }
+    } catch (error) {
+      // If we can't read the file, that's ok for this test
+      console.log('Could not read opencode.json for testing:', error)
+      expect.fail('Should be able to read opencode.json')
+    }
+  })
+})

package/tsconfig.json

@@ -11,7 +11,7 @@
     // "disableReferencedProjectLoad": true,             /* Reduce the number of projects loaded automatically by TypeScript. */
 
     /* Language and Environment */
-    "target": "es2016",                                  /* Set the JavaScript language version for emitted JavaScript and include compatible library declarations. */
+    "target": "es2016" /* Set the JavaScript language version for emitted JavaScript and include compatible library declarations. */,
     // "lib": [],                                        /* Specify a set of bundled library declaration files that describe the target runtime environment. */
     // "jsx": "preserve",                                /* Specify what JSX code is generated. */
     // "experimentalDecorators": true,                   /* Enable experimental support for legacy experimental decorators. */
@@ -25,7 +25,7 @@
     // "moduleDetection": "auto",                        /* Control what method is used to detect module-format JS files. */
 
     /* Modules */
-    "module": "commonjs",                                /* Specify what module code is generated. */
+    "module": "commonjs" /* Specify what module code is generated. */,
     // "rootDir": "./",                                  /* Specify the root folder within your source files. */
     // "moduleResolution": "node10",                     /* Specify how TypeScript looks up a file from a given module specifier. */
     // "baseUrl": "./",                                  /* Specify the base directory to resolve non-relative module names. */
@@ -39,7 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
-    "resolveJsonModule": true,                        /* Enable importing .json files. */
+    "resolveJsonModule": true /* Enable importing .json files. */,
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */
 
@@ -55,7 +55,7 @@
     // "sourceMap": true,                                /* Create source map files for emitted JavaScript files. */
     // "inlineSourceMap": true,                          /* Include sourcemap files inside the emitted JavaScript. */
     // "outFile": "./",                                  /* Specify a file that bundles all outputs into one JavaScript file. If 'declaration' is true, also designates a file that bundles all .d.ts output. */
-     "outDir": "./dist/",                                   /* Specify an output folder for all emitted files. */
+    "outDir": "./dist/" /* Specify an output folder for all emitted files. */,
     // "removeComments": true,                           /* Disable emitting comments. */
     // "noEmit": true,                                   /* Disable emitting files from a compilation. */
     // "importHelpers": true,                            /* Allow importing helper functions from tslib once per project, instead of including them per-file. */
@@ -77,12 +77,12 @@
     // "isolatedModules": true,                          /* Ensure that each file can be safely transpiled without relying on other imports. */
     // "verbatimModuleSyntax": true,                     /* Do not transform or elide any imports or exports not marked as type-only, ensuring they are written in the output file's format based on the 'module' setting. */
     // "allowSyntheticDefaultImports": true,             /* Allow 'import x from y' when a module doesn't have a default export. */
-    "esModuleInterop": true,                             /* Emit additional JavaScript to ease support for importing CommonJS modules. This enables 'allowSyntheticDefaultImports' for type compatibility. */
+    "esModuleInterop": true /* Emit additional JavaScript to ease support for importing CommonJS modules. This enables 'allowSyntheticDefaultImports' for type compatibility. */,
     // "preserveSymlinks": true,                         /* Disable resolving symlinks to their realpath. This correlates to the same flag in node. */
-    "forceConsistentCasingInFileNames": true,            /* Ensure that casing is correct in imports. */
+    "forceConsistentCasingInFileNames": true /* Ensure that casing is correct in imports. */,
 
     /* Type Checking */
-    "strict": true,                                      /* Enable all strict type-checking options. */
+    "strict": true /* Enable all strict type-checking options. */,
     // "noImplicitAny": true,                            /* Enable error reporting for expressions and declarations with an implied 'any' type. */
     // "strictNullChecks": true,                         /* When type checking, take into account 'null' and 'undefined'. */
     // "strictFunctionTypes": true,                      /* When assigning functions, check to ensure parameters and the return values are subtype-compatible. */
@@ -104,6 +104,6 @@
 
     /* Completeness */
     // "skipDefaultLibCheck": true,                      /* Skip type checking .d.ts files that are included with TypeScript. */
-    "skipLibCheck": true                                 /* Skip type checking all .d.ts files. */
+    "skipLibCheck": true /* Skip type checking all .d.ts files. */
   }
 }

package/vitest.config.ts

@@ -0,0 +1,8 @@
+import { defineConfig } from 'vitest/config'
+
+export default defineConfig({
+  test: {
+    globals: true,
+    environment: 'node',
+  },
+})