create-fluxstack 1.0.22 → 1.4.0

package/core/config/runtime-config.ts

@@ -1,18 +1,14 @@
 /**
  * Runtime Configuration System for FluxStack
- * Uses dynamic environment loading to solve Bun build issues
- * Drop-in replacement for process.env based configuration
+ * Uses declarative configuration system
  */
 
-import { env, createEnvNamespace, envValidation } from '../utils/env-runtime'
-import { 
-  dynamicEnvironmentProcessor, 
-  createDynamicConfig, 
-  validateProductionEnv,
-  getDynamicEnvironmentInfo 
-} from './env-dynamic'
+import { env, createNamespace } from '../utils/env'
 import type { FluxStackConfig } from './schema'
 import { defaultFluxStackConfig } from './schema'
+import { loggerConfig } from '../../config/logger.config'
+import { buildConfig } from '../../config/build.config'
+import { appConfig } from '../../config/app.config'
 
 /**
  * Runtime Configuration Builder
@@ -35,11 +31,21 @@ export class RuntimeConfigBuilder {
   }
 
   /**
-   * Load from dynamic environment variables
+   * Load from environment variables
    */
   private loadFromDynamicEnv(): this {
-    const envConfig = createDynamicConfig()
-    this.config = this.deepMerge(this.config, envConfig)
+    // Environment vars are loaded automatically by env loader
+    // Just merge common overrides here
+    const envOverrides: Partial<FluxStackConfig> = {}
+
+    if (env.has('PORT')) {
+      envOverrides.server = { ...this.config.server, port: env.PORT }
+    }
+    if (env.has('LOG_LEVEL')) {
+      envOverrides.logging = { ...this.config.logging, level: env.LOG_LEVEL }
+    }
+
+    this.config = this.deepMerge(this.config, envOverrides)
     return this
   }
 
@@ -64,8 +70,8 @@ export class RuntimeConfigBuilder {
    */
   build(): FluxStackConfig {
     // Validate production environment if needed
-    if (env.get('NODE_ENV') === 'production') {
-      validateProductionEnv()
+    if (env.NODE_ENV === 'production') {
+      env.require(['NODE_ENV'])
     }
 
     return this.config as FluxStackConfig
@@ -140,8 +146,8 @@ export const runtimeConfig = {
    */
   development(): FluxStackConfig {
     return new RuntimeConfigBuilder()
-      .override('logging.level', env.get('LOG_LEVEL', 'debug'))
-      .override('logging.format', env.get('LOG_FORMAT', 'pretty'))
+      .override('logging.level', loggerConfig.level)
+      .override('logging.format', 'pretty')
       .override('build.optimization.minify', false)
       .override('build.sourceMaps', true)
       .override('monitoring.enabled', false)
@@ -153,11 +159,11 @@ export const runtimeConfig = {
    */
   production(): FluxStackConfig {
     return new RuntimeConfigBuilder()
-      .override('logging.level', env.get('LOG_LEVEL', 'warn'))
-      .override('logging.format', env.get('LOG_FORMAT', 'json'))
+      .override('logging.level', loggerConfig.level)
+      .override('logging.format', 'json')
       .override('build.optimization.minify', true)
       .override('build.sourceMaps', false)
-      .override('monitoring.enabled', env.bool('MONITORING_ENABLED', true))
+      .override('monitoring.enabled', buildConfig.monitoringEnabled)
       .build()
   },
 
@@ -166,7 +172,7 @@ export const runtimeConfig = {
    */
   test(): FluxStackConfig {
     return new RuntimeConfigBuilder()
-      .override('logging.level', env.get('LOG_LEVEL', 'error'))
+      .override('logging.level', loggerConfig.level)
       .override('server.port', 0) // Random port for tests
       .override('client.port', 0)
       .override('monitoring.enabled', false)
@@ -177,7 +183,7 @@ export const runtimeConfig = {
    * Auto-detect environment and create appropriate config
    */
   auto(overrides?: Partial<FluxStackConfig>): FluxStackConfig {
-    const environment = env.get('NODE_ENV', 'development') as 'development' | 'production' | 'test'
+    const environment = appConfig.env
     
     let config: FluxStackConfig
     
@@ -214,27 +220,27 @@ export const envLoaders = {
   /**
    * Database environment loader
    */
-  database: createEnvNamespace('DATABASE_'),
-  
+  database: createNamespace('DATABASE_'),
+
   /**
    * JWT environment loader
    */
-  jwt: createEnvNamespace('JWT_'),
-  
+  jwt: createNamespace('JWT_'),
+
   /**
    * SMTP environment loader
    */
-  smtp: createEnvNamespace('SMTP_'),
-  
+  smtp: createNamespace('SMTP_'),
+
   /**
    * CORS environment loader
    */
-  cors: createEnvNamespace('CORS_'),
-  
+  cors: createNamespace('CORS_'),
+
   /**
    * FluxStack specific environment loader
    */
-  fluxstack: createEnvNamespace('FLUXSTACK_')
+  fluxstack: createNamespace('FLUXSTACK_')
 }
 
 /**
@@ -245,15 +251,12 @@ export const configHelpers = {
    * Get database URL with validation
    */
   getDatabaseUrl(): string | null {
-    const url = env.get('DATABASE_URL') as string | undefined
-    
-    if (url) {
-      envValidation.validate('DATABASE_URL', 
-        (value) => value.includes('://'),
-        'Must be a valid URL'
-      )
+    const url = env.DATABASE_URL
+
+    if (url && !url.includes('://')) {
+      throw new Error('DATABASE_URL must be a valid URL')
     }
-    
+
     return url || null
   },
 
@@ -261,18 +264,18 @@ export const configHelpers = {
    * Get CORS origins with proper defaults
    */
   getCorsOrigins(): string[] {
-    const origins = env.array('CORS_ORIGINS')
-    
-    if (origins.length === 0) {
-      const environment = env.get('NODE_ENV', 'development') as 'development' | 'production' | 'test'
-      
+    const origins = env.CORS_ORIGINS
+
+    if (origins.length === 0 || (origins.length === 1 && origins[0] === '*')) {
+      const environment = env.NODE_ENV
+
       if (environment === 'development') {
         return ['http://localhost:3000', 'http://localhost:5173']
       } else if (environment === 'production') {
         return [] // Must be explicitly configured in production
       }
     }
-    
+
     return origins
   },
 
@@ -281,15 +284,15 @@ export const configHelpers = {
    */
   getServerConfig() {
     return {
-      port: env.num('PORT', 3000),
-      host: env.get('HOST', 'localhost'),
-      apiPrefix: env.get('API_PREFIX', '/api'),
+      port: env.PORT,
+      host: env.HOST,
+      apiPrefix: env.API_PREFIX,
       cors: {
         origins: this.getCorsOrigins(),
-        methods: env.array('CORS_METHODS', ['GET', 'POST', 'PUT', 'DELETE', 'OPTIONS']),
-        headers: env.array('CORS_HEADERS', ['Content-Type', 'Authorization']),
-        credentials: env.bool('CORS_CREDENTIALS', false),
-        maxAge: env.num('CORS_MAX_AGE', 86400)
+        methods: env.CORS_METHODS,
+        headers: env.CORS_HEADERS,
+        credentials: env.CORS_CREDENTIALS,
+        maxAge: env.CORS_MAX_AGE
       }
     }
   },
@@ -299,16 +302,16 @@ export const configHelpers = {
    */
   getClientConfig() {
     return {
-      port: env.num('VITE_PORT', 5173),
+      port: env.VITE_PORT,
       proxy: {
-        target: env.get('API_URL', 'http://localhost:3000'),
-        changeOrigin: env.bool('PROXY_CHANGE_ORIGIN', true)
+        target: buildConfig.apiUrl,
+        changeOrigin: buildConfig.proxyChangeOrigin
       },
       build: {
-        outDir: env.get('CLIENT_BUILD_DIR', 'dist/client'),
-        sourceMaps: env.bool('CLIENT_SOURCEMAPS', env.get('NODE_ENV') === 'development'),
-        minify: env.bool('CLIENT_MINIFY', env.get('NODE_ENV') === 'production'),
-        target: env.get('CLIENT_TARGET', 'es2020')
+        outDir: buildConfig.clientBuildDir,
+        sourceMaps: buildConfig.clientSourceMaps,
+        minify: buildConfig.clientMinify,
+        target: buildConfig.clientTarget
       }
     }
   }

package/core/config/schema.ts

@@ -35,6 +35,7 @@ export interface ServerConfig {
   apiPrefix: string
   cors: CorsConfig
   middleware: MiddlewareConfig[]
+  showBanner?: boolean // Show startup banner (default: true)
 }
 
 export interface ProxyConfig {

package/core/framework/server.ts

@@ -5,6 +5,7 @@ import { PluginRegistry } from "../plugins/registry"
 import { PluginManager } from "../plugins/manager"
 import { getConfigSync, getEnvironmentInfo } from "../config"
 import { logger } from "../utils/logger"
+import { displayStartupBanner, type StartupInfo } from "../utils/logger/startup-banner"
 import { createErrorHandler } from "../utils/errors/handlers"
 import { createTimer, formatBytes, isProduction, isDevelopment } from "../utils/helpers"
 
@@ -73,7 +74,7 @@ export class FluxStackFramework {
       info: (message: string, meta?: any) => logger.info(message, meta),
       warn: (message: string, meta?: any) => logger.warn(message, meta),
       error: (message: string, meta?: any) => logger.error(message, meta),
-      child: (context: any) => (logger as any).child(context),
+      child: (context: any) => (logger as any).child ? (logger as any).child(context) : logger,
       time: (label: string) => (logger as any).time(label),
       timeEnd: (label: string) => (logger as any).timeEnd(label),
       request: (method: string, path: string, status?: number, duration?: number) =>
@@ -95,10 +96,11 @@ export class FluxStackFramework {
     })
 
     this.setupCors()
+    this.setupHeadHandler()
     this.setupHooks()
     this.setupErrorHandling()
 
-    logger.framework('FluxStack framework initialized', {
+    logger.debug('FluxStack framework initialized', {
       environment: envInfo.name,
       port: fullConfig.server.port
     })
@@ -113,7 +115,7 @@ export class FluxStackFramework {
     try {
       await this.pluginManager.initialize()
       const stats = this.pluginManager.getRegistry().getStats()
-      logger.framework('Automatic plugins loaded successfully', {
+      logger.debug('Automatic plugins loaded successfully', {
         pluginCount: stats.totalPlugins,
         enabledPlugins: stats.enabledPlugins,
         disabledPlugins: stats.disabledPlugins
@@ -141,6 +143,37 @@ export class FluxStackFramework {
       })
   }
 
+  private setupHeadHandler() {
+    // Global HEAD handler to prevent Elysia's automatic HEAD conversion bug
+    this.app.head("*", ({ request, set }) => {
+      const url = new URL(request.url)
+      
+      // Handle API routes
+      if (url.pathname.startsWith(this.context.config.server.apiPrefix)) {
+        set.status = 200
+        set.headers['Content-Type'] = 'application/json'
+        set.headers['Content-Length'] = '0'
+        return ""
+      }
+      
+      // Handle static files (assume they're HTML if no extension)
+      const isStatic = url.pathname === '/' || !url.pathname.includes('.')
+      if (isStatic) {
+        set.status = 200
+        set.headers['Content-Type'] = 'text/html'
+        set.headers['Content-Length'] = '478' // approximate size of index.html
+        set.headers['Cache-Control'] = 'no-cache'
+        return ""
+      }
+      
+      // Handle other file types
+      set.status = 200
+      set.headers['Content-Type'] = 'application/octet-stream'
+      set.headers['Content-Length'] = '0'
+      return ""
+    })
+  }
+
   private setupHooks() {
     // Setup onRequest hook and onBeforeRoute hook
     this.app.onRequest(async ({ request, set }) => {
@@ -388,7 +421,7 @@ export class FluxStackFramework {
         ;(this.pluginRegistry as any).loadOrder = loadOrder
       }
       
-      logger.framework(`Plugin '${plugin.name}' registered`, {
+      logger.debug(`Plugin '${plugin.name}' registered`, {
         version: plugin.version,
         dependencies: plugin.dependencies
       })
@@ -446,7 +479,7 @@ export class FluxStackFramework {
       }
 
       this.isStarted = true
-      logger.framework('All plugins loaded successfully', {
+      logger.debug('All plugins loaded successfully', {
         pluginCount: loadOrder.length
       })
 
@@ -503,15 +536,26 @@ export class FluxStackFramework {
     const apiPrefix = this.context.config.server.apiPrefix
 
     this.app.listen(port, () => {
-      logger.framework(`Server started on port ${port}`, {
+      const showBanner = this.context.config.server.showBanner !== false // default: true
+      const vitePluginActive = this.pluginRegistry.has('vite')
+
+      // Prepare startup info for banner or callback
+      const startupInfo: StartupInfo = {
+        port,
         apiPrefix,
         environment: this.context.environment,
-        pluginCount: this.pluginRegistry.getAll().length
-      })
+        pluginCount: this.pluginRegistry.getAll().length,
+        vitePort: this.context.config.client?.port,
+        viteEmbedded: vitePluginActive, // Vite is embedded when plugin is active
+        swaggerPath: '/swagger' // TODO: Get from swagger plugin config
+      }
+
+      // Display banner if enabled
+      if (showBanner) {
+        displayStartupBanner(startupInfo)
+      }
 
-      console.log(`🚀 API ready at http://localhost:${port}${apiPrefix}`)
-      console.log(`📋 Health check: http://localhost:${port}${apiPrefix}/health`)
-      console.log()
+      // Call user callback with startup info
       callback?.()
     })
 

package/core/plugins/built-in/index.ts

@@ -1,17 +1,17 @@
 /**
  * Built-in Plugins for FluxStack
  * Core plugins that provide essential functionality
+ *
+ * Note: Logger is NOT a plugin - it's core infrastructure used by plugins
  */
 
 // Import all built-in plugins
-import { loggerPlugin } from './logger'
 import { swaggerPlugin } from './swagger'
 import { vitePlugin } from './vite'
 import { staticPlugin } from './static'
 import { monitoringPlugin } from './monitoring'
 
 // Export individual plugins
-export { loggerPlugin } from './logger'
 export { swaggerPlugin } from './swagger'
 export { vitePlugin } from './vite'
 export { staticPlugin } from './static'
@@ -19,7 +19,6 @@ export { monitoringPlugin } from './monitoring'
 
 // Export as a collection
 export const builtInPlugins = {
-  logger: loggerPlugin,
   swagger: swaggerPlugin,
   vite: vitePlugin,
   static: staticPlugin,
@@ -28,7 +27,6 @@ export const builtInPlugins = {
 
 // Export as an array for easy registration
 export const builtInPluginsList = [
-  loggerPlugin,
   swaggerPlugin,
   vitePlugin,
   staticPlugin,
@@ -37,22 +35,14 @@ export const builtInPluginsList = [
 
 // Plugin categories
 export const pluginCategories = {
-  core: [loggerPlugin, staticPlugin],
+  core: [staticPlugin],
   development: [vitePlugin],
   documentation: [swaggerPlugin],
-  monitoring: [loggerPlugin, monitoringPlugin]
+  monitoring: [monitoringPlugin]
 } as const
 
 // Default plugin configuration
 export const defaultPluginConfig = {
-  logger: {
-    logRequests: true,
-    logResponses: true,
-    logErrors: true,
-    includeHeaders: false,
-    includeBody: false,
-    slowRequestThreshold: 1000
-  },
   swagger: {
     enabled: true,
     path: '/swagger',
@@ -104,15 +94,15 @@ export const defaultPluginConfig = {
  * Get default plugins for a specific environment
  */
 export function getDefaultPlugins(environment: 'development' | 'production' | 'test' = 'development') {
-  const basePlugins = [loggerPlugin, staticPlugin]
-  
+  const basePlugins = [staticPlugin]
+
   switch (environment) {
     case 'development':
       return [...basePlugins, vitePlugin, swaggerPlugin, monitoringPlugin]
     case 'production':
       return [...basePlugins, monitoringPlugin]
     case 'test':
-      return [loggerPlugin] // Minimal plugins for testing
+      return [] // Minimal plugins for testing
     default:
       return basePlugins
   }

package/core/plugins/built-in/static/index.ts

@@ -104,8 +104,8 @@ export const staticPlugin: Plugin = {
       compression: config.compression.enabled
     })
     
-    // Setup static file handling in Elysia
-    context.app.get("/*", async ({ request, set }: { request: Request, set: any }) => {
+    // Helper function for handling both GET and HEAD requests
+    const handleStaticRequest = async ({ request, set }: { request: Request, set: any }) => {
       const url = new URL(request.url)
       
       // Skip API routes
@@ -123,7 +123,7 @@ export const staticPlugin: Plugin = {
         // This plugin only handles static files serving in production or fallback
         
         // Serve static files
-        return await serveStaticFile(url.pathname, config, context, set)
+        return await serveStaticFile(url.pathname, config, context, set, request.method === 'HEAD')
         
       } catch (error) {
         context.logger.error("Error serving static file", { 
@@ -134,7 +134,11 @@ export const staticPlugin: Plugin = {
         set.status = 500
         return "Internal Server Error"
       }
-    })
+    }
+
+    // Setup static file handling in Elysia - handle both GET and HEAD
+    context.app.get("/*", handleStaticRequest)
+    context.app.head("/*", handleStaticRequest)
   },
 
   onServerStart: async (context: PluginContext) => {
@@ -162,7 +166,8 @@ async function serveStaticFile(
   pathname: string, 
   config: any, 
   context: PluginContext,
-  set: any
+  set: any,
+  isHead: boolean = false
 ): Promise<any> {
   const isDev = context.utils.isDevelopment()
   
@@ -209,7 +214,7 @@ async function serveStaticFile(
     if (config.spa.enabled && !pathname.includes('.')) {
       const indexPath = join(process.cwd(), baseDir, config.spa.fallback)
       if (existsSync(indexPath)) {
-        return serveFile(indexPath, config, set, context)
+        return serveFile(indexPath, config, set, context, isHead)
       }
     }
     
@@ -222,18 +227,18 @@ async function serveStaticFile(
   if (stats.isDirectory()) {
     const indexPath = join(filePath, config.indexFile)
     if (existsSync(indexPath)) {
-      return serveFile(indexPath, config, set, context)
+      return serveFile(indexPath, config, set, context, isHead)
     }
     
     set.status = 404
     return "Not Found"
   }
   
-  return serveFile(filePath, config, set, context)
+  return serveFile(filePath, config, set, context, isHead)
 }
 
 // Serve individual file
-function serveFile(filePath: string, config: any, set: any, context: PluginContext) {
+function serveFile(filePath: string, config: any, set: any, context: PluginContext, isHead: boolean = false) {
   const ext = extname(filePath)
   const file = Bun.file(filePath)
   
@@ -258,6 +263,9 @@ function serveFile(filePath: string, config: any, set: any, context: PluginConte
   const contentType = mimeTypes[ext] || 'application/octet-stream'
   set.headers['Content-Type'] = contentType
   
+  // Set content-length for both GET and HEAD requests
+  set.headers['Content-Length'] = file.size.toString()
+  
   // Set cache headers
   if (config.cacheControl.enabled) {
     if (ext === '.html') {
@@ -280,9 +288,15 @@ function serveFile(filePath: string, config: any, set: any, context: PluginConte
   
   context.logger.debug(`Serving static file: ${filePath}`, {
     contentType,
-    size: file.size
+    size: file.size,
+    method: isHead ? 'HEAD' : 'GET'
   })
   
+  // For HEAD requests, return empty body but keep all headers
+  if (isHead) {
+    return ""
+  }
+  
   return file
 }