@sylphx/flow 2.1.3 → 2.1.5

package/src/types/session.types.ts

@@ -49,7 +49,7 @@ export type MessagePart =
   | {
       type: 'error';
       error: string;
-      status: 'completed';  // Errors are immediately completed
+      status: 'completed'; // Errors are immediately completed
     };
 
 /**
@@ -94,8 +94,8 @@ export interface TokenUsage {
  * - content: Shown in UI AND sent to LLM
  */
 export interface MessageMetadata {
-  cpu?: string;         // CPU usage at creation time (e.g., "45.3% (8 cores)")
-  memory?: string;      // Memory usage at creation time (e.g., "4.2GB/16.0GB")
+  cpu?: string; // CPU usage at creation time (e.g., "45.3% (8 cores)")
+  memory?: string; // Memory usage at creation time (e.g., "4.2GB/16.0GB")
   // Future: add more fields as needed (sessionId, requestId, modelVersion, etc.)
 }
 
@@ -136,14 +136,14 @@ export interface MessageMetadata {
  */
 export interface SessionMessage {
   role: 'user' | 'assistant';
-  content: MessagePart[];  // UI display (without system status)
+  content: MessagePart[]; // UI display (without system status)
   timestamp: number;
-  status?: 'active' | 'completed' | 'error' | 'abort';  // Message lifecycle state (default: 'completed')
-  metadata?: MessageMetadata;  // System info for LLM context (not shown in UI)
-  todoSnapshot?: Todo[];   // Full todo state at message creation time (for rewind + LLM context)
+  status?: 'active' | 'completed' | 'error' | 'abort'; // Message lifecycle state (default: 'completed')
+  metadata?: MessageMetadata; // System info for LLM context (not shown in UI)
+  todoSnapshot?: Todo[]; // Full todo state at message creation time (for rewind + LLM context)
   attachments?: FileAttachment[];
-  usage?: TokenUsage;          // For UI/monitoring, not sent to LLM
-  finishReason?: string;       // For flow control (stop/tool-calls/length/error), not sent to LLM
+  usage?: TokenUsage; // For UI/monitoring, not sent to LLM
+  finishReason?: string; // For flow control (stop/tool-calls/length/error), not sent to LLM
 }
 
 /**
@@ -203,8 +203,8 @@ export interface Session {
   provider: ProviderId;
   model: string;
   messages: SessionMessage[];
-  todos: Todo[];         // Per-session todo list (not global!)
-  nextTodoId: number;    // Next todo ID for this session (starts at 1)
+  todos: Todo[]; // Per-session todo list (not global!)
+  nextTodoId: number; // Next todo ID for this session (starts at 1)
 
   // Note: Streaming state derived from message.status, not stored here
   // To check if streaming: messages.some(m => m.status === 'active')

package/src/types/target.types.ts

@@ -4,13 +4,15 @@
  */
 
 import type { CommonOptions, SetupResult } from './common.types.js';
-import type { MCPServerConfigFlags, MCPServerConfigUnion } from './mcp.types.js';
+import type { MCPServerConfigUnion } from './mcp.types.js';
 
 /**
  * Target-specific configuration
  * Defines how agents, configs, and other artifacts are structured for each target
  */
 export interface TargetConfig {
+  /** Base configuration directory (e.g., '.claude', '.opencode') */
+  configDir: string;
   /** Directory where agents are installed */
   agentDir: string;
   /** File extension for agent files */

package/src/types/todo.types.ts

@@ -10,7 +10,7 @@ export interface Todo {
   content: string;
   status: TodoStatus;
   activeForm: string; // Present continuous form (e.g., "Building feature X")
-  ordering: number;   // For custom ordering (higher = earlier in list)
+  ordering: number; // For custom ordering (higher = earlier in list)
 }
 
 export interface TodoUpdate {

package/src/types.ts

@@ -12,9 +12,9 @@
 
 // Re-export all types for backward compatibility
 export type {
+  CommandConfig,
   CommandHandler,
   CommandOptions,
-  CommandConfig,
   RunCommandOptions,
 } from './types/cli.types.js';
 export type {

package/src/utils/__tests__/package-manager-detector.test.ts

@@ -1,14 +1,14 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+import { afterEach, beforeEach, describe, expect, it } from 'vitest';
 import {
-  detectPackageManagerFromUserAgent,
-  detectPackageManagerFromLockFiles,
   detectPackageManager,
+  detectPackageManagerFromLockFiles,
+  detectPackageManagerFromUserAgent,
   getPackageManagerInfo,
   getUpgradeCommand,
 } from '../package-manager-detector';
-import fs from 'node:fs';
-import path from 'node:path';
-import os from 'node:os';
 
 describe('Package Manager Detection', () => {
   const originalEnv = process.env;

package/src/utils/agent-enhancer.ts

@@ -12,7 +12,7 @@
 
 import fs from 'node:fs/promises';
 import path from 'node:path';
-import { getOutputStylesDir, getRulesDir, getAgentsDir } from './config/paths.js';
+import { getAgentsDir, getOutputStylesDir, getRulesDir } from './config/paths.js';
 import { yamlUtils } from './config/target-utils.js';
 import { CLIError } from './error-handler.js';
 
@@ -58,7 +58,7 @@ async function loadRules(ruleNames?: string[]): Promise<string> {
         // Strip YAML front matter
         const stripped = await yamlUtils.stripFrontMatter(content);
         sections.push(stripped);
-      } catch (error) {
+      } catch (_error) {
         // Log warning if rule file not found, but continue with other rules
         console.warn(`Warning: Rule file not found: ${ruleName}.md`);
       }
@@ -88,7 +88,7 @@ async function loadOutputStyles(styleNames?: string[]): Promise<string> {
           const content = await fs.readFile(filePath, 'utf8');
           const stripped = await yamlUtils.stripFrontMatter(content);
           sections.push(stripped);
-        } catch (error) {
+        } catch (_error) {
           console.warn(`Warning: Output style file not found: ${styleName}.md`);
         }
       }
@@ -150,7 +150,7 @@ function filterRules(agentRules?: string[], enabledRules?: string[]): string[] |
   }
 
   // Return intersection: rules that are both in agent's list AND globally enabled
-  const filtered = agentRules.filter(rule => enabledRules.includes(rule));
+  const filtered = agentRules.filter((rule) => enabledRules.includes(rule));
   return filtered.length > 0 ? filtered : undefined;
 }
 

package/src/utils/config/paths.ts

@@ -39,7 +39,9 @@ export function findPackageRoot(context?: string): string {
     }
 
     const parentDir = path.dirname(currentDir);
-    if (parentDir === currentDir) break; // reached filesystem root
+    if (parentDir === currentDir) {
+      break; // reached filesystem root
+    }
     currentDir = parentDir;
   }
 

package/src/utils/config/target-utils.ts

@@ -288,7 +288,7 @@ export const pathUtils = {
     // Try to extract from content title
     const titleMatch = content.match(/^#\s+(.+?)(?:\s+Agent)?$/m);
     if (titleMatch) {
-      const title = titleMatch[1]!.trim().toLowerCase();
+      const title = titleMatch[1]?.trim().toLowerCase();
       const kebabTitle = title.replace(/\s+/g, '-').replace(/[^a-z0-9-]/g, '');
       return kebabTitle + (kebabTitle.includes('agent') ? '' : '-agent');
     }
@@ -319,7 +319,7 @@ export const pathUtils = {
   extractDescription(content: string): string {
     const firstParagraph = content.match(/^#\s+.+?\n\n(.+?)(?:\n\n|\n#|$)/s);
     if (firstParagraph) {
-      return firstParagraph[1]!.trim().replace(/\n+/g, ' ');
+      return firstParagraph[1]?.trim().replace(/\n+/g, ' ');
     }
     return 'Development agent for specialized tasks';
   },

package/src/utils/display/banner.ts

@@ -3,8 +3,8 @@
  * Welcome messages and branding
  */
 
-import chalk from 'chalk';
 import boxen from 'boxen';
+import chalk from 'chalk';
 
 /**
  * Display welcome banner
@@ -13,7 +13,7 @@ export function showWelcome(): void {
   console.log(
     boxen(
       `${chalk.cyan.bold('Sylphx Flow')} ${chalk.dim('- AI-Powered Development Framework')}\n` +
-      `${chalk.dim('Auto-initialization • Smart upgrades • One-click launch')}`,
+        `${chalk.dim('Auto-initialization • Smart upgrades • One-click launch')}`,
       {
         padding: 1,
         margin: { bottom: 1 },

package/src/utils/display/notifications.ts

@@ -6,10 +6,14 @@
 import { playNotificationSound } from './audio-player.js';
 
 // Terminal notification with sound
-export function sendTerminalNotification(title: string, message: string, options?: {
-  sound?: boolean;
-  duration?: number;
-}) {
+export function sendTerminalNotification(
+  _title: string,
+  _message: string,
+  options?: {
+    sound?: boolean;
+    duration?: number;
+  }
+) {
   const { sound = true, duration = 3000 } = options || {};
 
   // Play system sound using cross-platform audio player
@@ -30,40 +34,45 @@ export function sendTerminalNotification(title: string, message: string, options
 }
 
 // OS-level notification using system APIs
-export async function sendOSNotification(title: string, message: string, options?: {
-  icon?: string;
-  urgency?: 'low' | 'normal' | 'critical';
-  sound?: boolean;
-  timeout?: number;
-}) {
+export async function sendOSNotification(
+  title: string,
+  message: string,
+  options?: {
+    icon?: string;
+    urgency?: 'low' | 'normal' | 'critical';
+    sound?: boolean;
+    timeout?: number;
+  }
+) {
   const {
     icon = '🌀', // Flow-themed spiral emoji for Sylphx Flow notifications
     urgency = 'normal',
     sound = true,
-    timeout = 5000
+    timeout = 5000,
   } = options || {};
-  
+
   try {
     if (process.platform === 'darwin') {
       // macOS: use osascript with simplified approach
-      const { spawn } = require('child_process');
-      
+      const { spawn } = require('node:child_process');
+
       await new Promise<void>((resolve, reject) => {
         // Simple notification without complex escaping
         const args = [
-          '-e', `display notification "${message.replace(/"/g, '\\"')}" with title "${title.replace(/"/g, '\\"')}"`
+          '-e',
+          `display notification "${message.replace(/"/g, '\\"')}" with title "${title.replace(/"/g, '\\"')}"`,
         ];
-        
-        const proc = spawn('osascript', args, { 
+
+        const proc = spawn('osascript', args, {
           stdio: 'pipe',
-          env: { ...process.env, PATH: '/usr/bin:/bin:/usr/sbin:/sbin' }
+          env: { ...process.env, PATH: '/usr/bin:/bin:/usr/sbin:/sbin' },
         });
-        
+
         let stderr = '';
         proc.stderr?.on('data', (data) => {
           stderr += data.toString();
         });
-        
+
         proc.on('exit', (code) => {
           if (code === 0) {
             // Play sound separately if needed using cross-platform audio player
@@ -79,28 +88,32 @@ export async function sendOSNotification(title: string, message: string, options
         });
         proc.on('error', reject);
       });
-      
     } else if (process.platform === 'linux') {
       // Linux: use notify-send
-      const { spawn } = require('child_process');
+      const { spawn } = require('node:child_process');
       await new Promise<void>((resolve, reject) => {
         const proc = spawn('notify-send', [
-          '--urgency', urgency,
-          '--icon', icon,
-          '--expire-time', timeout.toString(),
+          '--urgency',
+          urgency,
+          '--icon',
+          icon,
+          '--expire-time',
+          timeout.toString(),
           title,
-          message
+          message,
         ]);
         proc.on('exit', (code) => {
-          if (code === 0) resolve();
-          else reject(new Error(`notify-send failed with code ${code}`));
+          if (code === 0) {
+            resolve();
+          } else {
+            reject(new Error(`notify-send failed with code ${code}`));
+          }
         });
         proc.on('error', reject);
       });
-
     } else if (process.platform === 'win32') {
       // Windows: use PowerShell toast notifications
-      const { spawn } = require('child_process');
+      const { spawn } = require('node:child_process');
       const powershellScript = `
         Add-Type -AssemblyName System.Windows.Forms
         $notification = New-Object System.Windows.Forms.NotifyIcon
@@ -113,12 +126,15 @@ export async function sendOSNotification(title: string, message: string, options
         Start-Sleep -Milliseconds ${timeout + 1000}
         $notification.Dispose()
       `;
-      
+
       await new Promise<void>((resolve, reject) => {
         const proc = spawn('powershell', ['-Command', powershellScript]);
         proc.on('exit', (code) => {
-          if (code === 0) resolve();
-          else reject(new Error(`PowerShell failed with code ${code}`));
+          if (code === 0) {
+            resolve();
+          } else {
+            reject(new Error(`PowerShell failed with code ${code}`));
+          }
         });
         proc.on('error', reject);
       });
@@ -132,24 +148,20 @@ export async function sendOSNotification(title: string, message: string, options
 
 // Combined notification - sends both terminal and OS notifications
 export function sendNotification(
-  title: string, 
-  message: string, 
+  title: string,
+  message: string,
   options?: {
     osNotification?: boolean;
     terminalNotification?: boolean;
     sound?: boolean;
   }
 ) {
-  const {
-    osNotification = true,
-    terminalNotification = true,
-    sound = true
-  } = options || {};
-  
+  const { osNotification = true, terminalNotification = true, sound = true } = options || {};
+
   if (terminalNotification) {
     sendTerminalNotification(title, message, { sound });
   }
-  
+
   if (osNotification) {
     sendOSNotification(title, message, { sound });
   }
@@ -163,7 +175,8 @@ export async function checkNotificationSupport(): Promise<{
 }> {
   return {
     terminalSupported: true, // Always supported
-    osSupported: process.platform === 'darwin' || process.platform === 'linux' || process.platform === 'win32',
-    platform: process.platform
+    osSupported:
+      process.platform === 'darwin' || process.platform === 'linux' || process.platform === 'win32',
+    platform: process.platform,
   };
-}
+}

package/src/utils/display/status.ts

@@ -13,9 +13,7 @@ import { isVersionOutdated } from '../version.js';
 export async function showStatus(state: ProjectState): Promise<void> {
   console.log(chalk.cyan.bold('📊 Project Status\n'));
 
-  if (!state.initialized) {
-    console.log('  ' + chalk.yellow('⚠  Not initialized'));
-  } else {
+  if (state.initialized) {
     console.log(`  ${chalk.green('✓')} Initialized (Flow v${state.version || 'unknown'})`);
 
     if (state.target) {
@@ -26,21 +24,38 @@ export async function showStatus(state: ProjectState): Promise<void> {
     // Component status
     const components = state.components;
     console.log(`\n  ${chalk.cyan('Components:')}`);
-    console.log(`    Agents: ${components.agents.installed ? chalk.green(`✓ ${components.agents.count}`) : chalk.red('✗')}`);
-    console.log(`    Rules: ${components.rules.installed ? chalk.green(`✓ ${components.rules.count}`) : chalk.red('✗')}`);
+    console.log(
+      `    Agents: ${components.agents.installed ? chalk.green(`✓ ${components.agents.count}`) : chalk.red('✗')}`
+    );
+    console.log(
+      `    Rules: ${components.rules.installed ? chalk.green(`✓ ${components.rules.count}`) : chalk.red('✗')}`
+    );
     console.log(`    Hooks: ${components.hooks.installed ? chalk.green('✓') : chalk.red('✗')}`);
-    console.log(`    MCP: ${components.mcp.installed ? chalk.green(`✓ ${components.mcp.serverCount} servers`) : chalk.red('✗')}`);
-    console.log(`    Output styles: ${components.outputStyles.installed ? chalk.green('✓') : chalk.red('✗')}`);
-    console.log(`    Slash commands: ${components.slashCommands.installed ? chalk.green(`✓ ${components.slashCommands.count}`) : chalk.red('✗')}`);
+    console.log(
+      `    MCP: ${components.mcp.installed ? chalk.green(`✓ ${components.mcp.serverCount} servers`) : chalk.red('✗')}`
+    );
+    console.log(
+      `    Output styles: ${components.outputStyles.installed ? chalk.green('✓') : chalk.red('✗')}`
+    );
+    console.log(
+      `    Slash commands: ${components.slashCommands.installed ? chalk.green(`✓ ${components.slashCommands.count}`) : chalk.red('✗')}`
+    );
 
     // Outdated warnings
     if (state.outdated) {
-      console.log(`\n  ${chalk.yellow('⚠')} Flow version outdated: ${state.version} → ${state.latestVersion}`);
+      console.log(
+        `\n  ${chalk.yellow('⚠')} Flow version outdated: ${state.version} → ${state.latestVersion}`
+      );
     }
 
-    if (state.targetVersion && state.targetLatestVersion &&
-        isVersionOutdated(state.targetVersion, state.targetLatestVersion)) {
-      console.log(`  ${chalk.yellow('⚠')} ${state.target} update available: v${state.targetVersion} → v${state.targetLatestVersion}`);
+    if (
+      state.targetVersion &&
+      state.targetLatestVersion &&
+      isVersionOutdated(state.targetVersion, state.targetLatestVersion)
+    ) {
+      console.log(
+        `  ${chalk.yellow('⚠')} ${state.target} update available: v${state.targetVersion} → v${state.targetLatestVersion}`
+      );
     }
 
     if (state.lastUpdated) {
@@ -49,6 +64,8 @@ export async function showStatus(state: ProjectState): Promise<void> {
         console.log(`\n  ${chalk.yellow('⚠')} Last updated: ${days} days ago`);
       }
     }
+  } else {
+    console.log(`  ${chalk.yellow('⚠  Not initialized')}`);
   }
 
   console.log('');

package/src/utils/files/file-operations.ts

@@ -5,8 +5,8 @@
 
 import fs from 'node:fs/promises';
 import path from 'node:path';
-import { pathSecurity } from '../security/security.js';
 import { formatFileSize as formatFileSizeCore } from '../../core/formatting/bytes.js';
+import { pathSecurity } from '../security/security.js';
 
 export interface FileReadOptions {
   encoding?: BufferEncoding;

package/src/utils/files/sync-utils.ts

@@ -1,18 +1,21 @@
 import fs from 'node:fs';
 import path from 'node:path';
 import chalk from 'chalk';
-import type { Target } from '../../types.js';
 import { MCP_SERVER_REGISTRY } from '../../config/servers.js';
-import { getAgentsDir, getSlashCommandsDir, getRulesDir } from '../config/paths.js';
+import type { Target } from '../../types.js';
+import { getAgentsDir, getRulesDir, getSlashCommandsDir } from '../config/paths.js';
 
 /**
  * Scan directory for .md files and return basenames
  */
 function scanTemplateDir(dir: string): string[] {
-  if (!fs.existsSync(dir)) return [];
+  if (!fs.existsSync(dir)) {
+    return [];
+  }
 
   try {
-    return fs.readdirSync(dir, { withFileTypes: true })
+    return fs
+      .readdirSync(dir, { withFileTypes: true })
       .filter((f) => f.isFile() && f.name.endsWith('.md'))
       .map((f) => f.name);
   } catch {
@@ -31,9 +34,9 @@ const FLOW_RULES = scanTemplateDir(getRulesDir());
  * Categorized files for sync
  */
 interface CategorizedFiles {
-  inFlow: string[];      // Files that exist locally and in Flow templates (will reinstall)
-  unknown: string[];     // Files not in Flow templates (custom or removed)
-  missing: string[];     // Flow templates that don't exist locally (will install)
+  inFlow: string[]; // Files that exist locally and in Flow templates (will reinstall)
+  unknown: string[]; // Files not in Flow templates (custom or removed)
+  missing: string[]; // Flow templates that don't exist locally (will install)
 }
 
 /**
@@ -48,8 +51,8 @@ interface SyncManifest {
     notInRegistry: string[];
   };
   hooks: {
-    inConfig: string[];      // Hooks from config (will be synced)
-    orphaned: string[];      // Hooks that exist locally but not in config (ask user)
+    inConfig: string[]; // Hooks from config (will be synced)
+    orphaned: string[]; // Hooks that exist locally but not in config (ask user)
   };
   preserve: string[];
 }
@@ -151,14 +154,14 @@ export async function buildSyncManifest(cwd: string, target: Target): Promise<Sy
         const installedServers = Object.keys(mcpConfig.mcpServers);
         const registryServers = Object.keys(MCP_SERVER_REGISTRY);
 
-        manifest.mcpServers.inRegistry = installedServers.filter(id =>
+        manifest.mcpServers.inRegistry = installedServers.filter((id) =>
           registryServers.includes(id)
         );
-        manifest.mcpServers.notInRegistry = installedServers.filter(id =>
-          !registryServers.includes(id)
+        manifest.mcpServers.notInRegistry = installedServers.filter(
+          (id) => !registryServers.includes(id)
         );
       }
-    } catch (error) {
+    } catch (_error) {
       console.warn(chalk.yellow('⚠ Failed to read .mcp.json'));
     }
   }
@@ -178,25 +181,21 @@ export async function buildSyncManifest(cwd: string, target: Target): Promise<Sy
           // In the future, this could be read from Flow config
           const expectedHookTypes = ['Notification'];
 
-          manifest.hooks.inConfig = existingHookTypes.filter(type =>
+          manifest.hooks.inConfig = existingHookTypes.filter((type) =>
             expectedHookTypes.includes(type)
           );
-          manifest.hooks.orphaned = existingHookTypes.filter(type =>
-            !expectedHookTypes.includes(type)
+          manifest.hooks.orphaned = existingHookTypes.filter(
+            (type) => !expectedHookTypes.includes(type)
           );
         }
-      } catch (error) {
+      } catch (_error) {
         console.warn(chalk.yellow('⚠ Failed to read settings.json'));
       }
     }
   }
 
   // Files to preserve
-  manifest.preserve = [
-    '.sylphx-flow/',
-    '.secrets/',
-    target.config.configFile || '',
-  ]
+  manifest.preserve = ['.sylphx-flow/', '.secrets/', target.config.configFile || '']
     .filter(Boolean)
     .map((p) => path.join(cwd, p));
 
@@ -379,7 +378,9 @@ export interface SelectedToRemove {
 /**
  * Select unknown files to remove
  */
-export async function selectUnknownFilesToRemove(manifest: SyncManifest): Promise<SelectedToRemove> {
+export async function selectUnknownFilesToRemove(
+  manifest: SyncManifest
+): Promise<SelectedToRemove> {
   const unknownFiles: Array<{ name: string; value: string; type: string }> = [];
 
   // Collect all unknown files
@@ -463,10 +464,7 @@ export async function selectUnknownFilesToRemove(manifest: SyncManifest): Promis
 /**
  * Show final summary before execution
  */
-export function showFinalSummary(
-  manifest: SyncManifest,
-  selectedUnknowns: SelectedToRemove
-): void {
+export function showFinalSummary(manifest: SyncManifest, selectedUnknowns: SelectedToRemove): void {
   console.log(chalk.cyan.bold('\n━━━ 📋 Final Summary\n'));
 
   // Will delete + reinstall
@@ -490,7 +488,10 @@ export function showFinalSummary(
   }
 
   // Will remove (selected unknowns)
-  const totalToRemove = selectedUnknowns.files.length + selectedUnknowns.mcpServers.length + selectedUnknowns.hooks.length;
+  const totalToRemove =
+    selectedUnknowns.files.length +
+    selectedUnknowns.mcpServers.length +
+    selectedUnknowns.hooks.length;
   if (totalToRemove > 0) {
     console.log(chalk.red('Remove (selected):\n'));
     selectedUnknowns.files.forEach((file) => {
@@ -506,7 +507,11 @@ export function showFinalSummary(
   }
 
   // Will preserve
-  const allSelected = [...selectedUnknowns.files, ...selectedUnknowns.mcpServers, ...selectedUnknowns.hooks];
+  const allSelected = [
+    ...selectedUnknowns.files,
+    ...selectedUnknowns.mcpServers,
+    ...selectedUnknowns.hooks,
+  ];
   const preservedUnknowns = [
     ...manifest.agents.unknown,
     ...manifest.slashCommands.unknown,
@@ -618,14 +623,10 @@ export async function removeMCPServers(cwd: string, serversToRemove: string[]):
     }
 
     // Write back
-    await fs.promises.writeFile(
-      mcpPath,
-      JSON.stringify(mcpConfig, null, 2) + '\n',
-      'utf-8'
-    );
+    await fs.promises.writeFile(mcpPath, `${JSON.stringify(mcpConfig, null, 2)}\n`, 'utf-8');
 
     return removedCount;
-  } catch (error) {
+  } catch (_error) {
     console.warn(chalk.yellow('⚠ Failed to update .mcp.json'));
     return 0;
   }
@@ -659,14 +660,10 @@ export async function removeHooks(cwd: string, hooksToRemove: string[]): Promise
     }
 
     // Write back
-    await fs.promises.writeFile(
-      settingsPath,
-      JSON.stringify(settings, null, 2) + '\n',
-      'utf-8'
-    );
+    await fs.promises.writeFile(settingsPath, `${JSON.stringify(settings, null, 2)}\n`, 'utf-8');
 
     return removedCount;
-  } catch (error) {
+  } catch (_error) {
     console.warn(chalk.yellow('⚠ Failed to update settings.json'));
     return 0;
   }