fix: resolve Biome lint errors for CI

- Sort import statements (organizeImports)
- Fix noUselessConstructor in MockModeManager
- Fix noNonNullAssertion by adding explicit null check
- Simplify return statements in initializer.ts and loader.ts
- Add comprehensive JSDoc documentation

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: j4rviscmd

src/config/command-installer.ts

@@ -9,7 +9,7 @@
  * @module config/command-installer
  */
 
-import { copyFileSync, mkdirSync, readdirSync, existsSync } from 'node:fs'
+import { copyFileSync, existsSync, mkdirSync, readdirSync } from 'node:fs'
 import { homedir } from 'node:os'
 import { dirname, join } from 'node:path'
 import { fileURLToPath } from 'node:url'

src/config/initializer.test.ts

@@ -1,8 +1,8 @@
-import { describe, test, expect, beforeEach, afterEach } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, test } from 'bun:test'
+import { mkdirSync, rmSync } from 'node:fs'
 import { join } from 'node:path'
-import { rmSync, mkdirSync } from 'node:fs'
 import { validateConfig } from './initializer.ts'
-import type { ModeSwitcherConfig, ModePreset } from './types.ts'
+import type { ModePreset, ModeSwitcherConfig } from './types.ts'
 
 describe('initializer', () => {
   describe('validateConfig', () => {

src/config/initializer.ts

@@ -1,15 +1,19 @@
-import type { ModeSwitcherConfig, ModePreset, AgentPreset } from './types.ts'
-import { DEFAULT_ECONOMY_MODEL } from './types.ts'
 import {
-  loadOpencodeConfig,
   loadOhMyOpencodeConfig,
+  loadOpencodeConfig,
   loadPluginConfig,
-  savePluginConfig,
   pluginConfigExists,
+  savePluginConfig,
 } from './loader.ts'
+import type { AgentPreset, ModePreset, ModeSwitcherConfig } from './types.ts'
+import { DEFAULT_ECONOMY_MODEL } from './types.ts'
 
 /**
- * Default opencode agent names
+ * Default opencode agent names.
+ *
+ * These are the standard agents used by OpenCode for various tasks.
+ *
+ * @constant
  */
 const OPENCODE_AGENTS = [
   'build',
@@ -22,7 +26,19 @@ const OPENCODE_AGENTS = [
 ] as const
 
 /**
- * Build a preset from existing configurations
+ * Builds a performance preset from existing OpenCode configurations.
+ *
+ * This function scans the current `opencode.json` and `oh-my-opencode.json`
+ * files to extract the currently configured models for each agent. These
+ * models are saved as the "performance" preset, preserving the user's
+ * high-performance model choices.
+ *
+ * @returns Promise resolving to a ModePreset with performance-oriented models
+ * @example
+ * ```typescript
+ * const preset = await buildPerformancePreset();
+ * console.log(preset.opencode.build.model); // "anthropic/claude-sonnet-4"
+ * ```
  */
 async function buildPerformancePreset(): Promise<ModePreset> {
   const opencodeConfig = await loadOpencodeConfig()
@@ -64,7 +80,19 @@ async function buildPerformancePreset(): Promise<ModePreset> {
 }
 
 /**
- * Build economy preset with free model
+ * Builds an economy preset using the default free model.
+ *
+ * This function creates a cost-efficient preset where all agents
+ * (both OpenCode and oh-my-opencode) are configured to use the
+ * `opencode/glm-4.7-free` model. This provides a budget-friendly
+ * alternative to performance mode for routine tasks.
+ *
+ * @returns Promise resolving to a ModePreset with economy-oriented models
+ * @example
+ * ```typescript
+ * const preset = await buildEconomyPreset();
+ * console.log(preset.model); // "opencode/glm-4.7-free"
+ * ```
  */
 async function buildEconomyPreset(): Promise<ModePreset> {
   const opencodeConfig = await loadOpencodeConfig()
@@ -101,8 +129,27 @@ async function buildEconomyPreset(): Promise<ModePreset> {
 }
 
 /**
- * Initialize the plugin configuration if it doesn't exist
- * @returns The configuration (existing or newly created)
+ * Initializes the plugin configuration if it doesn't exist.
+ *
+ * This function performs the following steps:
+ * 1. Checks if a configuration file already exists
+ * 2. If exists, loads and returns it
+ * 3. If not, creates a new configuration by:
+ *    - Building a performance preset from current settings
+ *    - Building an economy preset with free models
+ *    - Setting default mode to "performance"
+ *    - Saving the configuration to disk
+ *
+ * This is called on plugin startup to ensure a valid configuration
+ * is always available.
+ *
+ * @returns Promise resolving to the configuration (existing or newly created)
+ * @throws {Error} If configuration creation or file I/O fails
+ * @example
+ * ```typescript
+ * const config = await initializeConfig();
+ * console.log(config.currentMode); // "performance"
+ * ```
  */
 export async function initializeConfig(): Promise<ModeSwitcherConfig> {
   const exists = await pluginConfigExists()
@@ -134,7 +181,24 @@ export async function initializeConfig(): Promise<ModeSwitcherConfig> {
 }
 
 /**
- * Ensure configuration is valid and has required presets
+ * Validates that a configuration object is well-formed and has required presets.
+ *
+ * This function performs the following checks:
+ * - `currentMode` field is present and non-empty
+ * - `presets` object exists and contains at least one preset
+ * - A preset exists for the current mode
+ *
+ * @param config - The configuration object to validate
+ * @returns True if configuration is valid, false otherwise
+ * @example
+ * ```typescript
+ * const config = await loadPluginConfig();
+ * if (validateConfig(config)) {
+ *   console.log('Configuration is valid');
+ * } else {
+ *   console.error('Invalid configuration detected');
+ * }
+ * ```
  */
 export function validateConfig(config: ModeSwitcherConfig): boolean {
   if (!config.currentMode) {
@@ -143,8 +207,5 @@ export function validateConfig(config: ModeSwitcherConfig): boolean {
   if (!config.presets || Object.keys(config.presets).length === 0) {
     return false
   }
-  if (!config.presets[config.currentMode]) {
-    return false
-  }
-  return true
+  return Boolean(config.presets[config.currentMode])
 }

src/config/loader.test.ts

@@ -1,21 +1,21 @@
-import { describe, test, expect, beforeEach, afterEach } from 'bun:test'
-import { join } from 'node:path'
+import { afterEach, beforeEach, describe, expect, test } from 'bun:test'
+import { mkdirSync, rmSync } from 'node:fs'
 import { homedir } from 'node:os'
-import { rmSync, mkdirSync } from 'node:fs'
+import { join } from 'node:path'
 import { parse as parseJsonc } from 'jsonc-parser'
+import { type ModificationOptions, applyEdits, modify } from 'jsonc-parser'
 import {
+  clearContentCache,
   expandPath,
-  getPluginConfigPath,
-  getOpencodeConfigPath,
   getOhMyOpencodeConfigPath,
-  clearContentCache,
+  getOpencodeConfigPath,
+  getPluginConfigPath,
   setContentCache,
 } from './loader.ts'
-import { modify, applyEdits, type ModificationOptions } from 'jsonc-parser'
 import type {
   ModeSwitcherConfig,
-  OpencodeConfig,
   OhMyOpencodeConfig,
+  OpencodeConfig,
 } from './types.ts'
 
 describe('loader', () => {

src/config/loader.ts

@@ -1,15 +1,15 @@
 import { homedir } from 'node:os'
 import { join } from 'node:path'
 import {
-  parse as parseJsonc,
-  modify,
-  applyEdits,
   type ModificationOptions,
+  applyEdits,
+  modify,
+  parse as parseJsonc,
 } from 'jsonc-parser'
 import type {
   ModeSwitcherConfig,
-  OpencodeConfig,
   OhMyOpencodeConfig,
+  OpencodeConfig,
 } from './types.ts'
 
 /**
@@ -344,8 +344,7 @@ export async function saveOhMyOpencodeConfig(
  * @returns True if agent-mode-switcher.json exists, false otherwise
  */
 export async function pluginConfigExists(): Promise<boolean> {
-  const file = Bun.file(getPluginConfigPath())
-  return await file.exists()
+  return Bun.file(getPluginConfigPath()).exists()
 }
 
 /**

src/index.ts

@@ -10,8 +10,8 @@
 
 import { tool } from '@opencode-ai/plugin'
 import type { Plugin } from '@opencode-ai/plugin'
-import { ModeManager } from './modes/index.ts'
 import { copyCommandFiles } from './config/index.ts'
+import { ModeManager } from './modes/index.ts'
 
 /**
  * OpenCode Agent Mode Switcher Plugin.

src/modes/manager.test.ts

@@ -8,15 +8,24 @@ import type {
 import { createMockOpencodeClient, sampleConfigs } from '../test-utils/mocks.ts'
 
 /**
- * Create a mock ModeManager for testing
- * This avoids the complexity of mocking file system operations
+ * Mock implementation of ModeManager for testing purposes.
+ *
+ * This test double avoids the complexity of mocking file system operations
+ * by storing configuration in memory. It implements the same public API
+ * as the real ModeManager but uses in-memory state instead of reading/writing
+ * JSON files.
+ *
+ * @internal
  */
 class MockModeManager {
   private config: ModeSwitcherConfig | null = null
   private opencodeConfig: OpencodeConfig | null = null
   private ohMyConfig: OhMyOpencodeConfig | null = null
+  private client: OpencodeClient
 
-  constructor(_client: OpencodeClient) {}
+  constructor(client: OpencodeClient) {
+    this.client = client
+  }
 
   setConfig(config: ModeSwitcherConfig): void {
     this.config = config
@@ -43,7 +52,10 @@ class MockModeManager {
     if (!this.config) {
       await this.initialize()
     }
-    return this.config!
+    if (!this.config) {
+      throw new Error('Config not initialized')
+    }
+    return this.config
   }
 
   async getCurrentMode(): Promise<string> {