fix: resolve TypeScript warnings and add clean dev mode

FluxStack Team · claude · FluxStack Team · commit 09def565cff6 · 2025-09-19T15:10:11.000-03:00
- Fix 40+ TypeScript unused import/variable warnings across codebase - Add run-clean.ts wrapper to filter Elysia HEAD request errors - Add dev:clean command for development with filtered output - Update bun.lock with latest dependencies - Export missing interfaces in monitoring and config modules - Prefix intentionally unused parameters with underscore - Update documentation with new dev:clean command 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/CLAUDE.md b/CLAUDE.md
@@ -172,6 +172,7 @@ bun install          # Uma única instalação para todo o projeto! 🎉
 ### Comandos Principais
 ```bash
 bun run dev          # ✅ Full-stack: Backend (3000) + Vite integrado (5173)
+bun run dev:clean    # ✅ Servidor com output limpo (filtra erros HEAD do Elysia)
 bun run dev:backend  # ✅ Backend apenas com hot reload (porta 3001)
 bun run dev:frontend # ✅ Frontend apenas com Vite (porta 5173)
 bun run build        # Build para produção
@@ -227,6 +228,7 @@ bun run legacy:dev   # Comando direto com Bun watch (alternativo)
 - ✅ **Estrutura de instalação complexa** -> unificado em monorepo
 - ✅ **Duplicação de dependências** -> centralizadas no root
 - ✅ **Build em 2 etapas** -> processo unificado e otimizado
+- ✅ **Spam de erros HEAD do Elysia** -> comando `dev:clean` filtra erros desnecessários
 
 ## Próximos Passos Sugeridos
 
@@ -252,6 +254,7 @@ bun install
 
 # Desenvolvimento
 bun run dev                    # Full-stack development server
+bun run dev:clean             # Servidor com output limpo (sem erros HEAD)
 bun run dev:frontend          # Frontend apenas (porta 5173)
 bun run dev:backend           # Backend apenas (porta 3001)
 
diff --git a/bun.lock b/bun.lock
diff --git a/bun.lockb b/bun.lockb
diff --git a/core/__tests__/integration.test.ts b/core/__tests__/integration.test.ts
@@ -6,7 +6,6 @@
 import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest'
 import { FluxStackFramework } from '../framework/server'
 import { PluginRegistry } from '../plugins/registry'
-import { loggerPlugin } from '../plugins/built-in/logger'
 import { logger } from '../utils/logger'
 import type { Plugin } from '../plugins/types'
 
diff --git a/core/build/index.ts b/core/build/index.ts
@@ -1,5 +1,4 @@
 import { spawn } from "bun"
-import { join } from "path"
 import type { FluxStackConfig } from "../config"
 
 export class FluxStackBuilder {
diff --git a/core/client/standalone.ts b/core/client/standalone.ts
@@ -1,7 +1,6 @@
 // Standalone frontend development
 import { spawn } from "bun"
 import { join } from "path"
-import { getEnvironmentInfo } from "../config/env"
 
 export const startFrontendOnly = (config: any = {}) => {
   const clientPath = config.clientPath || "app/client"
diff --git a/core/config/loader.ts b/core/config/loader.ts
@@ -6,10 +6,7 @@
 import { existsSync } from 'fs'
 import { join } from 'path'
 import type {
-    FluxStackConfig,
-    LogLevel,
-    BuildTarget,
-    LogFormat
+    FluxStackConfig
 } from './schema'
 import {
     defaultFluxStackConfig,
@@ -319,7 +316,7 @@ function findConfigFile(startDir = process.cwd()): string | null {
 /**
  * Apply environment-specific configuration
  */
-function applyEnvironmentConfig(
+export function applyEnvironmentConfig(
     config: FluxStackConfig,
     environment: string
 ): FluxStackConfig {
diff --git a/core/framework/__tests__/server.test.ts b/core/framework/__tests__/server.test.ts
@@ -55,7 +55,7 @@ vi.mock('elysia', () => ({
     options: vi.fn().mockReturnThis(),
     onError: vi.fn().mockReturnThis(),
     use: vi.fn().mockReturnThis(),
-    listen: vi.fn((port, callback) => {
+    listen: vi.fn((_port, callback) => {
       if (callback) callback()
     })
   }))
diff --git a/core/framework/server.ts b/core/framework/server.ts
@@ -52,7 +52,7 @@ export class FluxStackFramework {
         }
         return result
       },
-      validateSchema: (data: any, schema: any) => {
+      validateSchema: (_data: any, _schema: any) => {
         // Simple validation - in a real implementation you'd use a proper schema validator
         try {
           // Basic validation logic
diff --git a/core/plugins/__tests__/manager.test.ts b/core/plugins/__tests__/manager.test.ts
@@ -4,7 +4,7 @@
 
 import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest'
 import { PluginManager } from '../manager'
-import type { Plugin, PluginContext, RequestContext } from '../types'
+import type { Plugin, PluginContext } from '../types'
 import type { Logger } from '../../utils/logger/index'
 import type { FluxStackConfig } from '../../config/schema'
 
@@ -327,12 +327,10 @@ describe('PluginManager', () => {
     })
 
     it('should provide plugin-specific logger', async () => {
-      let pluginLogger: any
-
       const plugin: Plugin = {
         name: 'logger-plugin',
-        setup: (context) => {
-          pluginLogger = context.logger
+        setup: (_context) => {
+          // Logger context is available but not used in this test
         }
       }
 
diff --git a/core/plugins/built-in/logger/index.ts b/core/plugins/built-in/logger/index.ts
@@ -166,7 +166,7 @@ export const loggerPlugin: Plugin = {
 }
 
 // Helper function to get plugin config from context
-function getPluginConfig(context: any) {
+function getPluginConfig(_context: any) {
   // In a real implementation, this would get the config from the plugin context
   // For now, return default config
   return loggerPlugin.defaultConfig || {}
diff --git a/core/plugins/built-in/monitoring/index.ts b/core/plugins/built-in/monitoring/index.ts
@@ -44,7 +44,7 @@ interface MetricsRegistry {
 
 // SystemMetrics and HttpMetrics are now imported from MetricsCollector
 
-interface MetricsExporter {
+export interface MetricsExporter {
   type: 'prometheus' | 'json' | 'console' | 'file'
   endpoint?: string
   interval?: number
@@ -53,7 +53,7 @@ interface MetricsExporter {
   filePath?: string
 }
 
-interface AlertThreshold {
+export interface AlertThreshold {
   metric: string
   operator: '>' | '<' | '>=' | '<=' | '==' | '!='
   value: number
@@ -523,7 +523,7 @@ function startSystemMetricsCollection(context: PluginContext, config: any, colle
   ;(context as any).monitoringIntervals = intervals
 }
 
-function setupMetricsEndpoint(context: PluginContext, config: any, registry: MetricsRegistry, collector: MetricsCollector) {
+function setupMetricsEndpoint(context: PluginContext, config: any, _registry: MetricsRegistry, collector: MetricsCollector) {
   // Find Prometheus exporter configuration
   const prometheusExporter = config.exporters.find((e: any) => e.type === 'prometheus' && e.enabled)
   if (!prometheusExporter) return
@@ -818,7 +818,7 @@ function exportToConsole(registry: MetricsRegistry, collector: MetricsCollector,
   })
 }
 
-function exportToPrometheus(registry: MetricsRegistry, collector: MetricsCollector, config: any, logger: any) {
+function exportToPrometheus(_registry: MetricsRegistry, collector: MetricsCollector, config: any, logger: any) {
   const prometheusData = collector.exportPrometheus()
   
   if (config.endpoint && config.endpoint !== '/metrics') {
@@ -897,7 +897,7 @@ function exportToFile(registry: MetricsRegistry, collector: MetricsCollector, co
   }
 }
 
-function formatPrometheusLabels(labels?: Record<string, string>): string {
+export function formatPrometheusLabels(labels?: Record<string, string>): string {
   if (!labels || Object.keys(labels).length === 0) {
     return ''
   }
diff --git a/core/plugins/built-in/vite/index.ts b/core/plugins/built-in/vite/index.ts
@@ -1,4 +1,3 @@
-import { join } from "path"
 import type { Plugin, PluginContext, RequestContext } from "../../types"
 
 export const vitePlugin: Plugin = {
diff --git a/core/plugins/config.ts b/core/plugins/config.ts
@@ -6,7 +6,6 @@
 import type { Plugin, PluginConfigSchema, PluginValidationResult } from "./types"
 import type { FluxStackConfig } from "../config/schema"
 import type { Logger } from "../utils/logger/index"
-import { FluxStackError } from "../utils/errors"
 
 export interface PluginConfigManager {
   validatePluginConfig(plugin: Plugin, config: any): PluginValidationResult
@@ -16,10 +15,8 @@ export interface PluginConfigManager {
 }
 
 export class DefaultPluginConfigManager implements PluginConfigManager {
-  private logger?: Logger
-
-  constructor(logger?: Logger) {
-    this.logger = logger
+  constructor(_logger?: Logger) {
+    // Logger stored but not used in current implementation
   }
 
   /**
diff --git a/core/plugins/discovery.ts b/core/plugins/discovery.ts
@@ -5,9 +5,8 @@
 
 import type { Plugin, PluginManifest, PluginLoadResult, PluginDiscoveryOptions } from "./types"
 import type { Logger } from "../utils/logger/index"
-import { FluxStackError } from "../utils/errors"
-import { readdir, stat, readFile } from "fs/promises"
-import { join, resolve, extname } from "path"
+import { readdir, readFile } from "fs/promises"
+import { join, resolve } from "path"
 import { existsSync } from "fs"
 
 export interface PluginDiscoveryConfig {
diff --git a/core/plugins/index.ts b/core/plugins/index.ts
@@ -191,6 +191,5 @@ import type {
     PluginPriority,
     RequestContext,
     ResponseContext,
-    ErrorContext,
-    BuildContext
+    ErrorContext
 } from './types'
diff --git a/core/plugins/manager.ts b/core/plugins/manager.ts
@@ -438,8 +438,9 @@ export class PluginManager extends EventEmitter {
    * Setup context for a specific plugin
    */
   private setupPluginContext(plugin: Plugin): void {
-    const pluginConfig = this.config.plugins.config[plugin.name] || {}
-    const mergedConfig = { ...plugin.defaultConfig, ...pluginConfig }
+    // Plugin config available but not used in current implementation
+    // const pluginConfig = this.config.plugins.config[plugin.name] || {}
+    // const mergedConfig = { ...plugin.defaultConfig, ...pluginConfig }
 
     const context: PluginContext = {
       config: this.config,
diff --git a/core/plugins/registry.ts b/core/plugins/registry.ts
@@ -2,7 +2,7 @@ import type { Plugin, PluginManifest, PluginLoadResult, PluginDiscoveryOptions }
 import type { FluxStackConfig } from "../config/schema"
 import type { Logger } from "../utils/logger/index"
 import { FluxStackError } from "../utils/errors"
-import { readdir, stat, readFile } from "fs/promises"
+import { readdir, readFile } from "fs/promises"
 import { join, resolve } from "path"
 import { existsSync } from "fs"
 
@@ -209,9 +209,9 @@ export class PluginRegistry {
     const results: PluginLoadResult[] = []
     const {
       directories = ['core/plugins/built-in', 'plugins', 'node_modules'],
-      patterns = ['**/plugin.{js,ts}', '**/index.{js,ts}'],
-      includeBuiltIn = true,
-      includeExternal = true
+      patterns: _patterns = ['**/plugin.{js,ts}', '**/index.{js,ts}'],
+      includeBuiltIn: _includeBuiltIn = true,
+      includeExternal: _includeExternal = true
     } = options
 
     for (const directory of directories) {
@@ -220,7 +220,7 @@ export class PluginRegistry {
       }
 
       try {
-        const pluginResults = await this.discoverPluginsInDirectory(directory, patterns)
+        const pluginResults = await this.discoverPluginsInDirectory(directory, _patterns)
         results.push(...pluginResults)
       } catch (error) {
         this.logger?.warn(`Failed to discover plugins in directory '${directory}'`, { error })
@@ -392,7 +392,7 @@ export class PluginRegistry {
    */
   private async discoverPluginsInDirectory(
     directory: string,
-    patterns: string[]
+    _patterns: string[]
   ): Promise<PluginLoadResult[]> {
     const results: PluginLoadResult[] = []
     
diff --git a/core/server/framework.ts b/core/server/framework.ts
@@ -48,7 +48,7 @@ export class FluxStackFramework {
         }
         return result
       },
-      validateSchema: (data: any, schema: any) => {
+      validateSchema: (_data: any, _schema: any) => {
         // Simple validation - in a real implementation you'd use a proper schema validator
         try {
           // Basic validation logic
diff --git a/core/server/plugins/logger.ts b/core/server/plugins/logger.ts
@@ -1,4 +1,4 @@
-import type { Plugin, PluginContext, RequestContext, ResponseContext, ErrorContext } from "../../types"
+import type { Plugin, PluginContext } from "../../types"
 import { log } from "../../utils/logger"
 
 export const loggerPlugin: Plugin = {
diff --git a/core/server/plugins/vite.ts b/core/server/plugins/vite.ts
@@ -1,4 +1,3 @@
-import { join } from "path"
 import type { Plugin, PluginContext } from "../../types"
 
 export const vitePlugin: Plugin = {
diff --git a/core/server/standalone.ts b/core/server/standalone.ts
@@ -1,5 +1,5 @@
 // Standalone backend server (sem frontend integrado)
-import { FluxStackFramework, loggerPlugin } from "./index"
+import { FluxStackFramework } from "./index"
 import type { Plugin, PluginContext } from "../types"
 
 export const createStandaloneServer = (userConfig: any = {}) => {
diff --git a/core/templates/create-project.ts b/core/templates/create-project.ts
@@ -11,12 +11,12 @@ export interface CreateProjectOptions {
 export class ProjectCreator {
   private projectName: string
   private targetDir: string
-  private template: string
 
   constructor(options: CreateProjectOptions) {
     this.projectName = options.name
     this.targetDir = options.targetDir || resolve(process.cwd(), options.name)
-    this.template = options.template || 'basic'
+    // Template option available but basic template is used for now
+    // const template = options.template || 'basic'
   }
 
   async create() {
diff --git a/core/utils/monitoring/index.ts b/core/utils/monitoring/index.ts
@@ -66,7 +66,7 @@ export class MetricsCollector {
       help,
       labels,
       value: 0,
-      inc: (value = 1, labels) => {
+      inc: (value = 1, _labels) => {
         counter.value = (counter.value || 0) + value
       }
     }
@@ -82,13 +82,13 @@ export class MetricsCollector {
       help,
       labels,
       value: 0,
-      set: (value, labels) => {
+      set: (value, _labels) => {
         gauge.value = value
       },
-      inc: (value = 1, labels) => {
+      inc: (value = 1, _labels) => {
         gauge.value = (gauge.value || 0) + value
       },
-      dec: (value = 1, labels) => {
+      dec: (value = 1, _labels) => {
         gauge.value = (gauge.value || 0) - value
       }
     }
@@ -104,7 +104,7 @@ export class MetricsCollector {
       help,
       buckets,
       values: [],
-      observe: (value, labels) => {
+      observe: (value, _labels) => {
         histogram.values = histogram.values || []
         histogram.values.push(value)
       }
@@ -115,7 +115,7 @@ export class MetricsCollector {
   }
 
   // HTTP metrics
-  recordHttpRequest(method: string, path: string, statusCode: number, duration: number, requestSize?: number, responseSize?: number): void {
+  recordHttpRequest(_method: string, _path: string, statusCode: number, duration: number, requestSize?: number, responseSize?: number): void {
     this.httpMetrics.requestsTotal++
     this.httpMetrics.requestDuration.push(duration)
     
diff --git a/package.json b/package.json
@@ -21,6 +21,7 @@
     "dev:frontend": "bun run core/cli/index.ts frontend",
     "dev:backend": "bun run core/cli/index.ts backend",
     "dev:coordinated": "concurrently --prefix {name} --names BACKEND,VITE --prefix-colors blue,green --kill-others-on-fail \"bun --watch app/server/index.ts\" \"vite --config vite.config.ts\"",
+    "dev:clean": "bun run run-clean.ts",
     "build": "bun run core/cli/index.ts build",
     "build:frontend": "bun run core/cli/index.ts build:frontend",
     "build:backend": "bun run core/cli/index.ts build:backend",
@@ -63,7 +64,7 @@
     "@elysiajs/swagger": "^1.3.1",
     "@vitejs/plugin-react": "^4.6.0",
     "chokidar": "^4.0.3",
-    "elysia": "^1.3.7",
+    "elysia": "^1.4.6",
     "react": "^19.1.0",
     "react-dom": "^19.1.0",
     "vite": "^7.0.4"
diff --git a/run-clean.ts b/run-clean.ts
@@ -0,0 +1,26 @@
+#!/usr/bin/env bun
+/**
+ * Clean server runner that filters Elysia HEAD bug errors
+ */
+
+// Redirect stderr to filter out HEAD errors
+const originalWrite = process.stderr.write;
+
+process.stderr.write = function(chunk: any, encoding?: any, callback?: any) {
+  const str = chunk.toString();
+  
+  // Filter out Elysia HEAD bug errors
+  if (str.includes("TypeError: undefined is not an object (evaluating '_res.headers.set')") ||
+      str.includes("HEAD - / failed")) {
+    // Silently ignore these errors
+    if (callback) callback();
+    return true;
+  }
+  
+  // Pass through all other stderr
+  return originalWrite.call(process.stderr, chunk, encoding, callback);
+};
+
+// Now run the server
+console.log('🚀 Starting FluxStack with filtered output...\n');
+await import('./app/server/index.ts');
diff --git a/tests/unit/app/client/App-simple.test.tsx b/tests/unit/app/client/App-simple.test.tsx
@@ -1,4 +1,4 @@
-import { describe, it, expect, vi } from 'vitest'
+import { describe, it, expect } from 'vitest'
 import { render, screen } from '@testing-library/react'
 import { getErrorMessage, APIException } from '@/app/client/src/lib/eden-api'
 

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`import { spawn } from "bun"`
`2`		`-import { join } from "path"`
`3`	`2`	`import type { FluxStackConfig } from "../config"`
`4`	`3`
`5`	`4`	`export class FluxStackBuilder {`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ export class FluxStackFramework {`
`52`	`52`	`}`
`53`	`53`	`return result`
`54`	`54`	`},`
`55`		`- validateSchema: (data: any, schema: any) => {`
	`55`	`+ validateSchema: (_data: any, _schema: any) => {`
`56`	`56`	`// Simple validation - in a real implementation you'd use a proper schema validator`
`57`	`57`	`try {`
`58`	`58`	`// Basic validation logic`