berget 2.2.8 → 2.2.9

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "berget",
-    "version": "2.2.8",
+    "version": "2.2.9",
     "main": "dist/index.js",
     "bin": {
         "berget": "dist/index.js"
@@ -43,6 +43,7 @@
         "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",
@@ -59,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/dist/src/commands/api-keys.js

@@ -341,19 +341,5 @@ function formatDate(dateString) {
     return chalk_1.default.gray(`${Math.floor(diffDays / 365)} years ago`);
 }
 function formatLastUsed(dateString) {
-    const date = new Date(dateString);
-    const now = new Date();
-    const diffTime = Math.abs(now.getTime() - date.getTime());
-    const diffDays = Math.ceil(diffTime / (1000 * 60 * 60 * 24));
-    if (diffDays === 0)
-        return chalk_1.default.green('Today');
-    if (diffDays === 1)
-        return chalk_1.default.yellow('Yesterday');
-    if (diffDays < 7)
-        return chalk_1.default.yellow(`${diffDays} days ago`);
-    if (diffDays < 30)
-        return chalk_1.default.blue(`${Math.floor(diffDays / 7)} weeks ago`);
-    if (diffDays < 365)
-        return chalk_1.default.magenta(`${Math.floor(diffDays / 30)} months ago`);
-    return chalk_1.default.gray(`${Math.floor(diffDays / 365)} years ago`);
+    return formatDate(dateString);
 }

package/dist/src/services/chat-service.js

@@ -87,7 +87,7 @@ class ChatService {
             if (hasValidAuth && !hasExplicitApiKey) {
                 logger_1.logger.debug('Using authenticated client with refresh token support');
                 // Create a copy without apiKey to let the authenticated client handle auth automatically
-                const { apiKey: _apiKey, ...optionsWithoutKey } = optionsCopy;
+                const { apiKey: _, ...optionsWithoutKey } = optionsCopy;
                 return this.executeCompletion(optionsWithoutKey, {});
             }
             // Check for environment variables first - prioritize this over everything else
@@ -244,7 +244,7 @@ class ChatService {
                 headers['Authorization'] = options.apiKey;
             }
             // Remove apiKey and onChunk from options before sending to API
-            const { apiKey: _apiKey, onChunk, ...requestOptions } = options;
+            const { apiKey: _, onChunk, ...requestOptions } = options;
             logger_1.logger.debug('Request options:');
             logger_1.logger.debug(JSON.stringify({
                 ...requestOptions,

package/dist/src/utils/config-checker.js

@@ -34,7 +34,7 @@ function checkBergetConfig() {
     if (fs.existsSync(configPath)) {
         try {
             const config = fs.readFileSync(configPath, 'utf8');
-            const match = config.match(/cluster:\s*(.+)/);
+            const match = /cluster:\s*(.+)/.exec(config);
             if (match && match[1]) {
                 const clusterName = match[1].trim();
                 console.log(`🔄 Berget: Switched to cluster "${clusterName}"`);

package/eslint.config.mjs

@@ -2,6 +2,7 @@ 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 tseslint.config(
@@ -22,6 +23,7 @@ export default tseslint.config(
   },
   ...tseslint.configs.recommended,
   ...tseslint.configs.strict,
+  sonarjs.configs.recommended,
   promise.configs['flat/recommended'],
   perfectionist.configs['recommended-natural'],
   {
@@ -62,6 +64,19 @@ export default tseslint.config(
         '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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "berget",
-  "version": "2.2.8",
+  "version": "2.2.9",
   "main": "dist/index.js",
   "bin": {
     "berget": "dist/index.js"
@@ -43,6 +43,7 @@
     "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",
@@ -59,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/commands/api-keys.ts

@@ -429,15 +429,5 @@ function formatDate(dateString: string): string {
 }
 
 function formatLastUsed(dateString: string): string {
-  const date = new Date(dateString);
-  const now = new Date();
-  const diffTime = Math.abs(now.getTime() - date.getTime());
-  const diffDays = Math.ceil(diffTime / (1000 * 60 * 60 * 24));
-
-  if (diffDays === 0) return chalk.green('Today');
-  if (diffDays === 1) return chalk.yellow('Yesterday');
-  if (diffDays < 7) return chalk.yellow(`${diffDays} days ago`);
-  if (diffDays < 30) return chalk.blue(`${Math.floor(diffDays / 7)} weeks ago`);
-  if (diffDays < 365) return chalk.magenta(`${Math.floor(diffDays / 30)} months ago`);
-  return chalk.gray(`${Math.floor(diffDays / 365)} years ago`);
+  return formatDate(dateString);
 }

package/src/services/chat-service.ts

@@ -96,7 +96,7 @@ export class ChatService {
       if (hasValidAuth && !hasExplicitApiKey) {
         logger.debug('Using authenticated client with refresh token support');
         // Create a copy without apiKey to let the authenticated client handle auth automatically
-        const { apiKey: _apiKey, ...optionsWithoutKey } = optionsCopy;
+        const { apiKey: _, ...optionsWithoutKey } = optionsCopy;
         return this.executeCompletion(optionsWithoutKey, {});
       }
 
@@ -278,7 +278,7 @@ export class ChatService {
       }
 
       // Remove apiKey and onChunk from options before sending to API
-      const { apiKey: _apiKey, onChunk, ...requestOptions } = options;
+      const { apiKey: _, onChunk, ...requestOptions } = options;
 
       logger.debug('Request options:');
       logger.debug(

package/src/utils/config-checker.ts

@@ -9,7 +9,7 @@ export function checkBergetConfig(): void {
   if (fs.existsSync(configPath)) {
     try {
       const config = fs.readFileSync(configPath, 'utf8');
-      const match = config.match(/cluster:\s*(.+)/);
+      const match = /cluster:\s*(.+)/.exec(config);
       if (match && match[1]) {
         const clusterName = match[1].trim();
         console.log(`🔄 Berget: Switched to cluster "${clusterName}"`);