@getvision/server 0.2.1-d0f3a53-develop → 0.2.1

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # @getvision/server
 
+## 0.2.1
+
+### Patch Changes
+
+- ec9cf8b: Redis password is now passed correctly and REDIS_URL is honored.
+- d0f3a53: Add event handler context
+
+  Event handlers receive a Hono-like `Context` as the second argument. You can run service-level middleware to inject resources using `c.set(...)` and then access them in the handler via `c.get(...)`.
+
+- 648a711: If Redis is configured use production mode in PubSub
+
 ## 0.2.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@getvision/server",
-  "version": "0.2.1-d0f3a53-develop",
+  "version": "0.2.1",
   "type": "module",
   "description": "Vision Server - Meta-framework with built-in observability, pub/sub, and type-safe APIs",
   "exports": {

package/src/event-bus.ts

@@ -26,12 +26,52 @@ export class EventBus {
   private devModeHandlers = new Map<string, Array<(data: any) => Promise<void>>>()
 
   constructor(config: EventBusConfig = {}) {
-    this.config = {
-      devMode: config.devMode ?? process.env.NODE_ENV === 'development',
-      redis: config.redis ?? {
+    // Build Redis config from environment variables
+    const envUrl = process.env.REDIS_URL
+    let envRedis: { host?: string; port?: number; password?: string } | undefined
+    if (envUrl) {
+      try {
+        const u = new URL(envUrl)
+        envRedis = {
+          host: u.hostname || undefined,
+          port: u.port ? parseInt(u.port) : 6379,
+          // URL password takes precedence over REDIS_PASSWORD
+          password: u.password || process.env.REDIS_PASSWORD || undefined,
+        }
+      } catch {
+        // Fallback to individual env vars if URL is invalid
+        envRedis = undefined
+      }
+    }
+
+    if (!envRedis) {
+      envRedis = {
         host: process.env.REDIS_HOST || 'localhost',
         port: parseInt(process.env.REDIS_PORT || '6379'),
-      },
+        password: process.env.REDIS_PASSWORD || undefined,
+      }
+    }
+
+    // Merge: explicit config.redis overrides env-derived values
+    const mergedRedis = { ...(envRedis || {}), ...(config.redis || {}) }
+
+    // Determine if Redis is configured by env or explicit config
+    const hasRedisFromEnv = Boolean(envUrl)
+    const hasRedisFromConfig = Boolean(
+      config.redis && (config.redis.host || config.redis.port || config.redis.password)
+    )
+    const hasRedis = hasRedisFromEnv || hasRedisFromConfig
+
+    // devMode precedence:
+    // 1) Respect explicit config.devMode when provided (true/false)
+    // 2) Otherwise, if Redis is configured (env or config), use production mode (devMode=false)
+    // 3) Otherwise, default to devMode=true (in-memory)
+    const resolvedDevMode =
+      typeof config.devMode === 'boolean' ? config.devMode : !hasRedis
+
+    this.config = {
+      devMode: resolvedDevMode,
+      redis: mergedRedis,
     }
   }
 

package/src/vision-app.ts

@@ -17,6 +17,30 @@ export interface VisionALSContext {
 
 const visionContext = new AsyncLocalStorage<VisionALSContext>()
 
+// Simple deep merge utility (objects only, arrays are overwritten by source)
+function deepMerge<T extends Record<string, any>>(target: T, source: Partial<T>): T {
+  const output: any = { ...target }
+  if (source && typeof source === 'object') {
+    for (const key of Object.keys(source)) {
+      const srcVal = source[key]
+      const tgtVal = output[key]
+      if (
+        srcVal &&
+        typeof srcVal === 'object' &&
+        !Array.isArray(srcVal) &&
+        tgtVal &&
+        typeof tgtVal === 'object' &&
+        !Array.isArray(tgtVal)
+      ) {
+        output[key] = deepMerge(tgtVal, srcVal)
+      } else {
+        output[key] = srcVal
+      }
+    }
+  }
+  return output as T
+}
+
 /**
  * Vision Server configuration
  */
@@ -61,20 +85,13 @@ export interface VisionConfig {
  *   service: {
  *     name: 'My API',
  *     version: '1.0.0'
- *   },
- *   pubsub: {
- *     schemas: {
- *       'user/created': {
- *         data: z.object({ userId: z.string() })
- *       }
- *     }
  *   }
  * })
  * 
  * const userService = app.service('users')
- *   .endpoint('GET', '/users/:id', schema, handler)
  *   .on('user/created', handler)
- * 
+ *   .endpoint('GET', '/users/:id', schema, handler)
+ *
  * app.start(3000)
  * ```
  */
@@ -100,24 +117,16 @@ export class Vision<
         enabled: false,
         port: 9500,
       },
-      pubsub: {
-        devMode: true,
-      },
+      // Do not set a default devMode here; let EventBus derive from Redis presence
+      pubsub: {},
       routes: {
         autodiscover: true,
         dirs: ['app/routes'],
       },
     }
 
-    // Merge shallowly (good enough for our config structure)
-    this.config = {
-      ...defaultConfig,
-      ...(config || {}),
-      service: { ...defaultConfig.service, ...(config?.service || {}) },
-      vision: { ...defaultConfig.vision, ...(config?.vision || {}) },
-      pubsub: { ...defaultConfig.pubsub, ...(config?.pubsub || {}) },
-      routes: { ...defaultConfig.routes, ...(config?.routes || {}) },
-    }
+    // Deep merge to respect nested overrides
+    this.config = deepMerge(defaultConfig, config || {})
     
     // Initialize Vision Core
     const visionEnabled = this.config.vision?.enabled !== false