style(backend): apply ESLint fixes and remove unused code

- Fix build script to ensure dist/database directory exists
- Apply consistent code formatting (quotes, line breaks)
- Remove unused variables and parameters
- Remove unused imports
- Improve code readability and maintainability

Author: alvagante

backend/package.json

@@ -5,7 +5,7 @@
   "main": "dist/server.js",
   "scripts": {
     "dev": "tsx watch src/server.ts",
-    "build": "tsc && cp src/database/schema.sql dist/database/ && cp src/database/migrations.sql dist/database/",
+    "build": "tsc && mkdir -p dist/database && cp src/database/schema.sql dist/database/ && cp src/database/migrations.sql dist/database/",
     "start": "node dist/server.js",
     "test": "vitest --run",
     "test:watch": "vitest",

backend/src/config/ConfigService.ts

@@ -1,6 +1,10 @@
-import { config as loadDotenv } from 'dotenv';
-import { AppConfigSchema, type AppConfig, type WhitelistConfig, type StreamingConfig } from './schema';
-import { z } from 'zod';
+import { config as loadDotenv } from "dotenv";
+import {
+  AppConfigSchema,
+  type AppConfig,
+  type WhitelistConfig,
+} from "./schema";
+import { z } from "zod";
 
 /**
  * Configuration service to load and validate application settings
@@ -25,19 +29,26 @@ export class ConfigService {
       // Parse command whitelist from JSON string
       let commandWhitelist: WhitelistConfig;
       try {
-        const whitelistJson = process.env.COMMAND_WHITELIST ?? '[]';
+        const whitelistJson = process.env.COMMAND_WHITELIST ?? "[]";
         const parsedWhitelist: unknown = JSON.parse(whitelistJson);
         const whitelistArray: string[] = Array.isArray(parsedWhitelist)
-          ? parsedWhitelist.filter((item): item is string => typeof item === 'string')
+          ? parsedWhitelist.filter(
+              (item): item is string => typeof item === "string",
+            )
           : [];
         const matchMode = process.env.COMMAND_WHITELIST_MATCH_MODE;
         commandWhitelist = {
-          allowAll: process.env.COMMAND_WHITELIST_ALLOW_ALL === 'true',
+          allowAll: process.env.COMMAND_WHITELIST_ALLOW_ALL === "true",
           whitelist: whitelistArray,
-          matchMode: (matchMode === 'exact' || matchMode === 'prefix') ? matchMode : 'exact',
+          matchMode:
+            matchMode === "exact" || matchMode === "prefix"
+              ? matchMode
+              : "exact",
         };
       } catch (error) {
-        throw new Error(`Failed to parse COMMAND_WHITELIST: ${error instanceof Error ? error.message : 'Unknown error'}`);
+        throw new Error(
+          `Failed to parse COMMAND_WHITELIST: ${error instanceof Error ? error.message : "Unknown error"}`,
+        );
       }
 
       // Parse package tasks from JSON string if provided
@@ -46,27 +57,43 @@ export class ConfigService {
         try {
           packageTasks = JSON.parse(process.env.PACKAGE_TASKS);
         } catch (error) {
-          throw new Error(`Failed to parse PACKAGE_TASKS: ${error instanceof Error ? error.message : 'Unknown error'}`);
+          throw new Error(
+            `Failed to parse PACKAGE_TASKS: ${error instanceof Error ? error.message : "Unknown error"}`,
+          );
         }
       }
 
       // Parse streaming configuration
       const streaming = {
-        bufferMs: process.env.STREAMING_BUFFER_MS ? parseInt(process.env.STREAMING_BUFFER_MS, 10) : undefined,
-        maxOutputSize: process.env.STREAMING_MAX_OUTPUT_SIZE ? parseInt(process.env.STREAMING_MAX_OUTPUT_SIZE, 10) : undefined,
-        maxLineLength: process.env.STREAMING_MAX_LINE_LENGTH ? parseInt(process.env.STREAMING_MAX_LINE_LENGTH, 10) : undefined,
+        bufferMs: process.env.STREAMING_BUFFER_MS
+          ? parseInt(process.env.STREAMING_BUFFER_MS, 10)
+          : undefined,
+        maxOutputSize: process.env.STREAMING_MAX_OUTPUT_SIZE
+          ? parseInt(process.env.STREAMING_MAX_OUTPUT_SIZE, 10)
+          : undefined,
+        maxLineLength: process.env.STREAMING_MAX_LINE_LENGTH
+          ? parseInt(process.env.STREAMING_MAX_LINE_LENGTH, 10)
+          : undefined,
       };
 
       // Parse cache configuration
       const cache = {
-        inventoryTtl: process.env.CACHE_INVENTORY_TTL ? parseInt(process.env.CACHE_INVENTORY_TTL, 10) : undefined,
-        factsTtl: process.env.CACHE_FACTS_TTL ? parseInt(process.env.CACHE_FACTS_TTL, 10) : undefined,
+        inventoryTtl: process.env.CACHE_INVENTORY_TTL
+          ? parseInt(process.env.CACHE_INVENTORY_TTL, 10)
+          : undefined,
+        factsTtl: process.env.CACHE_FACTS_TTL
+          ? parseInt(process.env.CACHE_FACTS_TTL, 10)
+          : undefined,
       };
 
       // Parse execution queue configuration
       const executionQueue = {
-        concurrentLimit: process.env.CONCURRENT_EXECUTION_LIMIT ? parseInt(process.env.CONCURRENT_EXECUTION_LIMIT, 10) : undefined,
-        maxQueueSize: process.env.MAX_QUEUE_SIZE ? parseInt(process.env.MAX_QUEUE_SIZE, 10) : undefined,
+        concurrentLimit: process.env.CONCURRENT_EXECUTION_LIMIT
+          ? parseInt(process.env.CONCURRENT_EXECUTION_LIMIT, 10)
+          : undefined,
+        maxQueueSize: process.env.MAX_QUEUE_SIZE
+          ? parseInt(process.env.MAX_QUEUE_SIZE, 10)
+          : undefined,
       };
 
       // Build configuration object
@@ -75,7 +102,9 @@ export class ConfigService {
         host: process.env.HOST,
         boltProjectPath: process.env.BOLT_PROJECT_PATH,
         commandWhitelist,
-        executionTimeout: process.env.EXECUTION_TIMEOUT ? parseInt(process.env.EXECUTION_TIMEOUT, 10) : undefined,
+        executionTimeout: process.env.EXECUTION_TIMEOUT
+          ? parseInt(process.env.EXECUTION_TIMEOUT, 10)
+          : undefined,
         logLevel: process.env.LOG_LEVEL,
         databasePath: process.env.DATABASE_PATH,
         packageTasks,
@@ -88,7 +117,9 @@ export class ConfigService {
       return AppConfigSchema.parse(rawConfig);
     } catch (error) {
       if (error instanceof z.ZodError) {
-        const issues = error.issues.map(issue => `${issue.path.join('.')}: ${issue.message}`).join(', ');
+        const issues = error.issues
+          .map((issue) => `${issue.path.join(".")}: ${issue.message}`)
+          .join(", ");
         throw new Error(`Configuration validation failed: ${issues}`);
       }
       throw error;

backend/src/routes/executions.ts

@@ -141,7 +141,7 @@ export function createExecutionsRouter(
    * GET /api/executions/queue/status
    * Return current execution queue status
    */
-  router.get('/queue/status', asyncHandler(async (req: Request, res: Response): Promise<void> => {
+  router.get('/queue/status', asyncHandler(async (_req: Request, res: Response): Promise<void> => {
     if (!executionQueue) {
       res.status(503).json({
         error: {

backend/src/services/ExecutionQueue.ts

@@ -119,7 +119,7 @@ export class ExecutionQueue {
     // Sort by enqueued time (oldest first)
     entries.sort((a, b) => a[1].enqueuedAt.getTime() - b[1].enqueuedAt.getTime());
 
-    const [executionId, execution] = entries[0];
+    const [executionId] = entries[0];
 
     // Move from queue to running
     this.queuedExecutions.delete(executionId);
@@ -140,8 +140,7 @@ export class ExecutionQueue {
    * @returns true if execution was cancelled, false if not found or already running
    */
   public cancel(executionId: string): boolean {
-    const execution = this.queuedExecutions.get(executionId);
-    if (!execution) {
+    if (!this.queuedExecutions.has(executionId)) {
       return false;
     }
 
@@ -216,7 +215,7 @@ export class ExecutionQueue {
    */
   public clearQueue(): void {
     // Reject all waiting promises
-    for (const [executionId, promise] of this.waitingPromises.entries()) {
+    for (const promise of this.waitingPromises.values()) {
       promise.reject(new Error('Queue cleared'));
     }
 

backend/test/database/ExecutionRepository.test.ts

@@ -2,7 +2,6 @@ import { describe, it, expect, beforeEach, afterEach } from "vitest";
 import sqlite3 from "sqlite3";
 import {
   ExecutionRepository,
-  ExecutionRecord,
   ExecutionType,
   ExecutionStatus,
 } from "../../src/database/ExecutionRepository";