berget 2.2.5 → 2.2.7

package/src/utils/token-manager.ts

@@ -1,38 +1,57 @@
-import * as fs from 'fs'
-import * as path from 'path'
-import * as os from 'os'
-import chalk from 'chalk'
-import { logger } from './logger'
+import * as fs from "fs";
+import * as path from "path";
+import * as os from "os";
+import { logger } from "./logger";
 
 interface TokenData {
-  access_token: string
-  refresh_token: string
-  expires_at: number // timestamp in milliseconds
+  access_token: string;
+  refresh_token: string;
+  expires_at: number; // timestamp in milliseconds
+}
+
+/**
+ * Extract the expiration time from a JWT token
+ * @param accessToken The JWT access token
+ * @returns The expiration timestamp in milliseconds, or 0 if invalid
+ */
+function extractJwtExpiresAt(accessToken: string): number {
+  try {
+    const parts = accessToken.split(".");
+    if (parts.length !== 3) return 0;
+    const payload = Buffer.from(parts[1], "base64url").toString("utf8");
+    const decoded = JSON.parse(payload);
+    if (typeof decoded.exp === "number") {
+      return decoded.exp * 1000; // JWT exp is in seconds, convert to milliseconds
+    }
+  } catch {
+    // If decoding fails, return 0 (treated as expired)
+  }
+  return 0;
 }
 
 /**
  * Manages authentication tokens including refresh functionality
  */
 export class TokenManager {
-  private static instance: TokenManager
-  private tokenFilePath: string
-  private tokenData: TokenData | null = null
+  private static instance: TokenManager;
+  private tokenFilePath: string;
+  private tokenData: TokenData | null = null;
 
   private constructor() {
     // Set up token file path in user's home directory
-    const bergetDir = path.join(os.homedir(), '.berget')
+    const bergetDir = path.join(os.homedir(), ".berget");
     if (!fs.existsSync(bergetDir)) {
-      fs.mkdirSync(bergetDir, { recursive: true })
+      fs.mkdirSync(bergetDir, { recursive: true });
     }
-    this.tokenFilePath = path.join(bergetDir, 'auth.json')
-    this.loadToken()
+    this.tokenFilePath = path.join(bergetDir, "auth.json");
+    this.loadToken();
   }
 
   public static getInstance(): TokenManager {
     if (!TokenManager.instance) {
-      TokenManager.instance = new TokenManager()
+      TokenManager.instance = new TokenManager();
     }
-    return TokenManager.instance
+    return TokenManager.instance;
   }
 
   /**
@@ -41,12 +60,12 @@ export class TokenManager {
   private loadToken(): void {
     try {
       if (fs.existsSync(this.tokenFilePath)) {
-        const data = fs.readFileSync(this.tokenFilePath, 'utf8')
-        this.tokenData = JSON.parse(data)
+        const data = fs.readFileSync(this.tokenFilePath, "utf8");
+        this.tokenData = JSON.parse(data);
       }
-    } catch (error) {
-      logger.error('Failed to load authentication token')
-      this.tokenData = null
+    } catch {
+      logger.error("Failed to load authentication token");
+      this.tokenData = null;
     }
   }
 
@@ -56,20 +75,17 @@ export class TokenManager {
   private saveToken(): void {
     try {
       if (this.tokenData) {
-        fs.writeFileSync(
-          this.tokenFilePath,
-          JSON.stringify(this.tokenData, null, 2),
-        )
+        fs.writeFileSync(this.tokenFilePath, JSON.stringify(this.tokenData, null, 2));
         // Set file permissions to be readable only by the owner
-        fs.chmodSync(this.tokenFilePath, 0o600)
+        fs.chmodSync(this.tokenFilePath, 0o600);
       } else {
         // If token data is null, remove the file
         if (fs.existsSync(this.tokenFilePath)) {
-          fs.unlinkSync(this.tokenFilePath)
+          fs.unlinkSync(this.tokenFilePath);
         }
       }
-    } catch (error) {
-      logger.error('Failed to save authentication token')
+    } catch {
+      logger.error("Failed to save authentication token");
     }
   }
 
@@ -78,8 +94,8 @@ export class TokenManager {
    * @returns The access token or null if not available
    */
   public getAccessToken(): string | null {
-    if (!this.tokenData) return null
-    return this.tokenData.access_token
+    if (!this.tokenData) return null;
+    return this.tokenData.access_token;
   }
 
   /**
@@ -87,8 +103,8 @@ export class TokenManager {
    * @returns The refresh token or null if not available
    */
   public getRefreshToken(): string | null {
-    if (!this.tokenData) return null
-    return this.tokenData.refresh_token
+    if (!this.tokenData) return null;
+    return this.tokenData.refresh_token;
   }
 
   /**
@@ -96,34 +112,34 @@ export class TokenManager {
    * @returns true if expired or about to expire (within 10% of lifetime or 30 seconds), false otherwise
    */
   public isTokenExpired(): boolean {
-    if (!this.tokenData || !this.tokenData.expires_at) return true
+    if (!this.tokenData || !this.tokenData.expires_at) return true;
 
     try {
-      const now = Date.now()
-      const expiresAt = this.tokenData.expires_at
-      const timeUntilExpiry = expiresAt - now
+      const now = Date.now();
+      const expiresAt = this.tokenData.expires_at;
+      const timeUntilExpiry = expiresAt - now;
 
       // Use 10% of remaining lifetime or 30 seconds, whichever is smaller
       // This ensures we don't refresh tokens that were just issued
-      const minBuffer = 30 * 1000 // 30 seconds minimum
-      const percentBuffer = timeUntilExpiry * 0.1 // 10% of lifetime
-      const expirationBuffer = Math.min(minBuffer, percentBuffer)
+      const minBuffer = 30 * 1000; // 30 seconds minimum
+      const percentBuffer = timeUntilExpiry * 0.1; // 10% of lifetime
+      const expirationBuffer = Math.min(minBuffer, percentBuffer);
 
-      const isExpired = now + expirationBuffer >= expiresAt
+      const isExpired = now + expirationBuffer >= expiresAt;
 
       if (isExpired) {
         logger.debug(
-          `Token expired or expiring soon. Current time: ${new Date().toISOString()}, Expiry: ${new Date(expiresAt).toISOString()}`,
-        )
+          `Token expired or expiring soon. Current time: ${new Date().toISOString()}, Expiry: ${new Date(expiresAt).toISOString()}`
+        );
       }
 
-      return isExpired
+      return isExpired;
     } catch (error) {
       // If there's any error checking expiration, assume token is expired
       logger.error(
-        `Error checking token expiration: ${error instanceof Error ? error.message : String(error)}`,
-      )
-      return true
+        `Error checking token expiration: ${error instanceof Error ? error.message : String(error)}`
+      );
+      return true;
     }
   }
 
@@ -131,39 +147,43 @@ export class TokenManager {
    * Set new token data
    * @param accessToken The new access token
    * @param refreshToken The new refresh token
-   * @param expiresIn Expiration time in seconds
+   * @param expiresIn Expiration time in seconds (fallback if JWT parsing fails)
    */
-  public setTokens(
-    accessToken: string,
-    refreshToken: string,
-    expiresIn: number,
-  ): void {
+  public setTokens(accessToken: string, refreshToken: string, expiresIn: number): void {
+    // Extract the actual expiry time from the JWT token
+    const jwtExpiresAt = extractJwtExpiresAt(accessToken);
+    const expiresAt = jwtExpiresAt > 0 ? jwtExpiresAt : Date.now() + expiresIn * 1000;
+
     this.tokenData = {
       access_token: accessToken,
       refresh_token: refreshToken,
-      expires_at: Date.now() + expiresIn * 1000,
-    }
-    this.saveToken()
+      expires_at: expiresAt,
+    };
+    this.saveToken();
   }
 
   /**
    * Update just the access token and its expiration
    * @param accessToken The new access token
-   * @param expiresIn Expiration time in seconds
+   * @param expiresIn Expiration time in seconds (fallback if JWT parsing fails)
    */
   public updateAccessToken(accessToken: string, expiresIn: number): void {
-    if (!this.tokenData) return
+    if (!this.tokenData) return;
+
+    // Extract the actual expiry time from the JWT token
+    const jwtExpiresAt = extractJwtExpiresAt(accessToken);
+    const expiresAt = jwtExpiresAt > 0 ? jwtExpiresAt : Date.now() + expiresIn * 1000;
 
-    this.tokenData.access_token = accessToken
-    this.tokenData.expires_at = Date.now() + expiresIn * 1000
-    this.saveToken()
+    this.tokenData.access_token = accessToken;
+    this.tokenData.expires_at = expiresAt;
+    this.saveToken();
   }
 
   /**
    * Clear all token data
    */
   public clearTokens(): void {
-    this.tokenData = null
-    this.saveToken()
+    this.tokenData = null;
+    this.saveToken();
   }
 }

package/templates/agents/app.md

@@ -0,0 +1,23 @@
+---
+description: Expo + React Native apps; props-first, offline-aware, shared tokens.
+mode: primary
+temperature: 0.4
+top_p: 0.9
+permission:
+  edit: allow
+  bash: deny
+  webfetch: allow
+---
+
+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.
+
+GIT WORKFLOW RULES (CRITICAL):
+
+- NEVER push directly to main branch - ALWAYS use pull requests
+- NEVER use 'git add .' - ALWAYS add specific files with 'git add path/to/file'
+- ALWAYS clean up test files, documentation files, and temporary artifacts before committing
+- ALWAYS ensure git history maintains production quality - no test commits, no debugging code
+- ALWAYS create descriptive commit messages following project conventions
+- ALWAYS run tests and build before creating PR
+
+CRITICAL: When all app implementation tasks are complete and ready for merge, ALWAYS invoke @quality subagent to handle testing, building, and complete PR management including URL provision.

package/templates/agents/backend.md

@@ -0,0 +1,23 @@
+---
+description: Functional, modular Koa + TypeScript services; schema-first with code quality focus.
+mode: primary
+temperature: 0.3
+top_p: 0.9
+permission:
+  edit: allow
+  bash: allow
+  webfetch: allow
+---
+
+You are Berget Code Backend agent. Voice: Scandinavian calm—precise, concise, confident. TypeScript + Koa. Prefer many small pure functions; avoid big try/catch blocks. Routes thin; logic in services/adapters/domain. Validate with Zod; auto-generate OpenAPI. Adapters isolate external systems; domain never depends on framework. Test with supertest; idempotent and stateless by default. Each microservice emits an OpenAPI contract; changes propagate upward to types. Code Quality & Refactoring Principles: Apply Single Responsibility Principle, fail fast with explicit errors, eliminate code duplication, remove nested complexity, use descriptive error codes, keep functions under 30 lines. Always leave code cleaner and more readable than you found it.
+
+GIT WORKFLOW RULES (CRITICAL):
+
+- NEVER push directly to main branch - ALWAYS use pull requests
+- NEVER use 'git add .' - ALWAYS add specific files with 'git add path/to/file'
+- ALWAYS clean up test files, documentation files, and temporary artifacts before committing
+- ALWAYS ensure git history maintains production quality - no test commits, no debugging code
+- ALWAYS create descriptive commit messages following project conventions
+- ALWAYS run tests and build before creating PR
+
+CRITICAL: When all backend implementation tasks are complete and ready for merge, ALWAYS invoke @quality subagent to handle testing, building, and complete PR management including URL provision.

package/templates/agents/devops.md

@@ -0,0 +1,30 @@
+---
+description: Declarative GitOps infra with FluxCD, Kustomize, Helm, operators.
+mode: primary
+temperature: 0.3
+top_p: 0.8
+permission:
+  edit: allow
+  bash: allow
+  webfetch: allow
+---
+
+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.
+
+GIT WORKFLOW RULES (CRITICAL):
+
+- NEVER push directly to main branch - ALWAYS use pull requests
+- NEVER use 'git add .' - ALWAYS add specific files with 'git add path/to/file'
+- ALWAYS clean up test files, documentation files, and temporary artifacts before committing
+- ALWAYS ensure git history maintains production quality - no test commits, no debugging code
+- ALWAYS create descriptive commit messages following project conventions
+- ALWAYS run tests and build before creating PR
+
+Helm Values Configuration Process:
+
+1. Documentation First Approach: Always fetch official documentation from Artifact Hub/GitHub for the specific chart version before writing values. Search Artifact Hub for exact chart version documentation, check the chart's GitHub repository for official docs and examples, verify the exact version being used in the deployment.
+2. Validation Requirements: Check for available validation schemas before committing YAML files. Use Helm's built-in validation tools (helm lint, helm template). Validate against JSON schema if available for the chart. Ensure YAML syntax correctness with linters.
+3. Standard Workflow: Identify chart name and exact version. Fetch official documentation from Artifact Hub/GitHub. Check for available schemas and validation tools. Write values according to official documentation. Validate against schema (if available). Test with helm template or helm lint. Commit validated YAML files.
+4. Quality Assurance: Never commit unvalidated Helm values. Use helm dependency update when adding new charts. Test rendering with helm template --dry-run before deployment. Document any custom values with comments referencing official docs.
+
+CRITICAL: When all devops implementation tasks are complete and ready for merge, ALWAYS invoke @quality subagent to handle testing, building, and complete PR management including URL provision.

package/templates/agents/frontend.md

@@ -0,0 +1,25 @@
+---
+description: Builds Scandinavian, type-safe UIs with React, Tailwind, Shadcn.
+mode: primary
+temperature: 0.4
+top_p: 0.9
+permission:
+  edit: allow
+  bash: deny
+  webfetch: allow
+---
+
+You are Berget Code Frontend agent. Voice: Scandinavian calm—precise, concise, confident. React 18 + TypeScript. Tailwind + Shadcn UI only via the design system (index.css, tailwind.config.ts). Use semantic tokens for color/spacing/typography/motion; never ad-hoc classes or inline colors. Components are pure and responsive; props-first data; minimal global state (Zustand/Jotai). Accessibility and keyboard navigation mandatory. Mock data only at init under /data via typed hooks (e.g., useProducts() reading /data/products.json). Design: minimal, balanced, quiet motion.
+
+IMPORTANT: You have NO bash access and cannot run git commands. When your frontend implementation tasks are complete, inform the user that changes are ready and suggest using /pr command to create a pull request with proper testing and quality checks.
+
+CODE QUALITY RULES:
+
+- Write clean, production-ready code
+- Follow React and TypeScript best practices
+- Ensure accessibility and responsive design
+- Use semantic tokens from design system
+- Test your components manually when possible
+- Document any complex logic with comments
+
+CRITICAL: When frontend implementation is complete, ALWAYS inform the user to use "/pr" command to handle testing, building, and pull request creation.

package/templates/agents/fullstack.md

@@ -0,0 +1,23 @@
+---
+description: Router/coordinator agent for full-stack development with schema-driven architecture
+mode: primary
+temperature: 0.3
+top_p: 0.9
+permission:
+  edit: allow
+  bash: allow
+  webfetch: allow
+---
+
+Voice: Scandinavian calm—precise, concise, confident; no fluff. You are Berget Code Fullstack agent. Act as a router and coordinator in a monorepo. Bottom-up schema: database → OpenAPI → generated types. Top-down types: API → UI → components. Use openapi-fetch and Zod at every boundary; compile-time errors are desired when contracts change. Routing rules: if task/paths match /apps/frontend or React (.tsx) → use frontend; if /apps/app or Expo/React Native → app; if /infra, /k8s, flux-system, kustomization.yaml, Helm values → devops; if /services, Koa routers, services/adapters/domain → backend. If ambiguous, remain fullstack and outline the end-to-end plan, then delegate subtasks to the right persona. Security: validate inputs; secrets via FluxCD SOPS/Sealed Secrets. Documentation is generated from code—never duplicated.
+
+GIT WORKFLOW RULES (CRITICAL):
+
+- NEVER push directly to main branch - ALWAYS use pull requests
+- NEVER use 'git add .' - ALWAYS add specific files with 'git add path/to/file'
+- ALWAYS clean up test files, documentation files, and temporary artifacts before committing
+- ALWAYS ensure git history maintains production quality - no test commits, no debugging code
+- ALWAYS create descriptive commit messages following project conventions
+- ALWAYS run tests and build before creating PR
+
+CRITICAL: When all implementation tasks are complete and ready for merge, ALWAYS invoke @quality subagent to handle testing, building, and complete PR management including URL provision.

package/templates/agents/quality.md

@@ -0,0 +1,69 @@
+---
+description: Quality assurance specialist for testing, building, and complete PR management.
+mode: subagent
+temperature: 0.1
+top_p: 0.9
+permission:
+  edit: allow
+  bash: allow
+  webfetch: allow
+---
+
+Voice: Scandinavian calm—precise, concise, confident. You are Berget Code Quality agent. Specialist in code quality assurance, testing, building, and pull request lifecycle management.
+
+Core responsibilities:
+
+- Run comprehensive test suites (npm test, npm run test, jest, vitest)
+- Execute build processes (npm run build, webpack, vite, tsc)
+- Create and manage pull requests with proper descriptions
+- Handle merge conflicts and keep main updated
+- Monitor GitHub for reviewer comments and address them
+- Ensure code quality standards are met
+- Validate linting and formatting (npm run lint, prettier)
+- Check test coverage and performance benchmarks
+- Handle CI/CD pipeline validation
+
+Complete PR Workflow:
+
+1. Ensure all tests pass: npm test
+2. Build successfully: npm run build
+3. Commit all changes with proper message
+4. Push to feature branch
+5. Update main branch and handle merge conflicts
+6. Create or update PR with comprehensive description
+7. Monitor for reviewer comments
+8. Address feedback and push updates
+9. Always provide PR URL for user review
+
+Essential CLI commands:
+
+- npm test or npm run test (run test suite)
+- npm run build (build project)
+- npm run lint (run linting)
+- npm run format (format code)
+- npm run test:coverage (check coverage)
+- git add <specific-files> && git commit -m "message" && git push (commit and push)
+- git checkout main && git pull origin main (update main)
+- git checkout feature-branch && git merge main (handle conflicts)
+- gh pr create --title "title" --body "body" (create PR)
+- gh pr view --comments (check PR comments)
+- gh pr edit --title "title" --body "body" (update PR)
+
+PR Creation Process:
+
+- Always include clear summary of changes
+- List technical details and improvements
+- Include testing and validation results
+- Add any breaking changes or migration notes
+- Provide PR URL immediately after creation
+
+GIT WORKFLOW RULES (CRITICAL - ENFORCE STRICTLY):
+
+- NEVER push directly to main branch - ALWAYS use pull requests
+- NEVER use 'git add .' - ALWAYS add specific files with 'git add path/to/file'
+- ALWAYS clean up test files, documentation files, and temporary artifacts before committing
+- ALWAYS ensure git history maintains production quality - no test commits, no debugging code
+- ALWAYS create descriptive commit messages following project conventions
+- ALWAYS run tests and build before creating PR
+
+Always provide specific command examples and wait for processes to complete before proceeding.

package/templates/agents/security.md

@@ -0,0 +1,21 @@
+---
+description: Security specialist for pentesting, OWASP compliance, and vulnerability assessments.
+mode: subagent
+temperature: 0.2
+top_p: 0.8
+permission:
+  edit: deny
+  bash: allow
+  webfetch: allow
+---
+
+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.
+
+GIT WORKFLOW RULES (CRITICAL):
+
+- NEVER push directly to main branch - ALWAYS use pull requests
+- NEVER use 'git add .' - ALWAYS add specific files with 'git add path/to/file'
+- ALWAYS clean up test files, documentation files, and temporary artifacts before committing
+- ALWAYS ensure git history maintains production quality - no test commits, no debugging code
+- ALWAYS create descriptive commit messages following project conventions
+- ALWAYS run tests and build before creating PR

package/tests/commands/chat.test.ts

@@ -1,110 +1,102 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
-import { Command } from 'commander'
-import { registerChatCommands } from '../../src/commands/chat'
-import { ChatService } from '../../src/services/chat-service'
-import { DefaultApiKeyManager } from '../../src/utils/default-api-key'
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+import { Command } from "commander";
+import { registerChatCommands } from "../../src/commands/chat";
+import { ChatService } from "../../src/services/chat-service";
+import { DefaultApiKeyManager } from "../../src/utils/default-api-key";
 
 // Mock dependencies
-vi.mock('../../src/services/chat-service')
-vi.mock('../../src/utils/default-api-key')
-vi.mock('readline', () => ({
+vi.mock("../../src/services/chat-service");
+vi.mock("../../src/utils/default-api-key");
+vi.mock("readline", () => ({
   createInterface: vi.fn(() => ({
     question: vi.fn(),
     close: vi.fn(),
   })),
-}))
+}));
 
-describe('Chat Commands', () => {
-  let program: Command
-  let mockChatService: any
-  let mockDefaultApiKeyManager: any
+describe("Chat Commands", () => {
+  let program: Command;
+  let mockChatService: any;
+  let mockDefaultApiKeyManager: any;
 
   beforeEach(() => {
-    program = new Command()
+    program = new Command();
 
     // Mock ChatService
     mockChatService = {
       createCompletion: vi.fn(),
       listModels: vi.fn(),
-    }
-    vi.mocked(ChatService.getInstance).mockReturnValue(mockChatService)
+    };
+    vi.mocked(ChatService.getInstance).mockReturnValue(mockChatService);
 
     // Mock DefaultApiKeyManager
     mockDefaultApiKeyManager = {
       getDefaultApiKeyData: vi.fn(),
       promptForDefaultApiKey: vi.fn(),
-    }
-    vi.mocked(DefaultApiKeyManager.getInstance).mockReturnValue(
-      mockDefaultApiKeyManager,
-    )
+    };
+    vi.mocked(DefaultApiKeyManager.getInstance).mockReturnValue(mockDefaultApiKeyManager);
 
-    registerChatCommands(program)
-  })
+    registerChatCommands(program);
+  });
 
   afterEach(() => {
-    vi.clearAllMocks()
-  })
+    vi.clearAllMocks();
+  });
 
-  describe('chat run command', () => {
-    it('should use berget/glm-4.7 as default model', () => {
-      const chatCommand = program.commands.find((cmd) => cmd.name() === 'chat')
-      const runCommand = chatCommand?.commands.find(
-        (cmd) => cmd.name() === 'run',
-      )
+  describe("chat run command", () => {
+    it("should use berget/glm-4.7 as default model", () => {
+      const chatCommand = program.commands.find(cmd => cmd.name() === "chat");
+      const runCommand = chatCommand?.commands.find(cmd => cmd.name() === "run");
 
-      expect(runCommand).toBeDefined()
+      expect(runCommand).toBeDefined();
 
       // Check the help text which contains the default model
-      const helpText = runCommand?.helpInformation()
-      expect(helpText).toContain('glm-4.7')
-    })
+      const helpText = runCommand?.helpInformation();
+      expect(helpText).toContain("glm-4.7");
+    });
 
-    it('should have streaming enabled by default', () => {
-      const chatCommand = program.commands.find((cmd) => cmd.name() === 'chat')
-      const runCommand = chatCommand?.commands.find(
-        (cmd) => cmd.name() === 'run',
-      )
+    it("should have streaming enabled by default", () => {
+      const chatCommand = program.commands.find(cmd => cmd.name() === "chat");
+      const runCommand = chatCommand?.commands.find(cmd => cmd.name() === "run");
 
-      expect(runCommand).toBeDefined()
+      expect(runCommand).toBeDefined();
 
       // Check that the option is --no-stream (meaning streaming is default)
-      const streamOption = runCommand?.options.find(
-        (opt) => opt.long === '--no-stream',
-      )
-      expect(streamOption).toBeDefined()
-      expect(streamOption?.description).toContain('Disable streaming')
-    })
-
-    it('should create completion with correct default options', async () => {
+      const streamOption = runCommand?.options.find(opt => opt.long === "--no-stream");
+      expect(streamOption).toBeDefined();
+      expect(streamOption?.description).toContain("Disable streaming");
+    });
+
+    it("should create completion with correct default options", async () => {
       // Mock API key
-      process.env.BERGET_API_KEY = 'test-key'
+      process.env.BERGET_API_KEY = "test-key";
 
       // Mock successful completion
       mockChatService.createCompletion.mockResolvedValue({
         choices: [
           {
-            message: { content: 'Test response' },
+            message: { content: "Test response" },
           },
         ],
-      })
+      });
 
       // This would normally test the actual command execution
       // but since it involves readline interaction, we just verify
       // that the service would be called with correct defaults
-      expect(mockChatService.createCompletion).not.toHaveBeenCalled()
+      expect(mockChatService.createCompletion).not.toHaveBeenCalled();
 
       // Clean up
-      delete process.env.BERGET_API_KEY
-    })
-  })
+      delete process.env.BERGET_API_KEY;
+    });
+  });
 
-  describe('chat list command', () => {
-    it('should list available models', async () => {
+  describe("chat list command", () => {
+    it("should list available models", async () => {
       const mockModels = {
         data: [
           {
-            id: 'gpt-oss',
-            owned_by: 'openai',
+            id: "gpt-oss",
+            owned_by: "openai",
             active: true,
             capabilities: {
               vision: false,
@@ -113,17 +105,15 @@ describe('Chat Commands', () => {
             },
           },
         ],
-      }
+      };
 
-      mockChatService.listModels.mockResolvedValue(mockModels)
+      mockChatService.listModels.mockResolvedValue(mockModels);
 
-      const chatCommand = program.commands.find((cmd) => cmd.name() === 'chat')
-      const listCommand = chatCommand?.commands.find(
-        (cmd) => cmd.name() === 'list',
-      )
+      const chatCommand = program.commands.find(cmd => cmd.name() === "chat");
+      const listCommand = chatCommand?.commands.find(cmd => cmd.name() === "list");
 
-      expect(listCommand).toBeDefined()
-      expect(listCommand?.description()).toBe('List available chat models')
-    })
-  })
-})
+      expect(listCommand).toBeDefined();
+      expect(listCommand?.description()).toBe("List available chat models");
+    });
+  });
+});