@getvision/server 0.2.1-ec9cf8b-develop → 0.2.2-a88ad97-develop

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-ec9cf8b-develop",
+  "version": "0.2.2-a88ad97-develop",
   "type": "module",
   "description": "Vision Server - Meta-framework with built-in observability, pub/sub, and type-safe APIs",
   "exports": {
@@ -14,7 +14,7 @@
   "license": "MIT",
   "dependencies": {
     "@getvision/core": "0.0.3",
-    "@hono/node-server": "^1.19.5",
+    "@hono/node-server": "^1.19.6",
     "bullmq": "^5.62.0",
     "hono-rate-limiter": "^0.4.2",
     "hono": "^4.10.4",

package/src/event-bus.ts

@@ -55,8 +55,22 @@ export class EventBus {
     // 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: config.devMode ?? process.env.NODE_ENV === 'development',
+      devMode: resolvedDevMode,
       redis: mergedRedis,
     }
   }

package/src/index.ts

@@ -51,6 +51,10 @@ export type { VisionConfig } from './vision-app'
 // Service builder (usually accessed via app.service())
 export { ServiceBuilder } from './service'
 
+// Event bus
+export { EventBus } from './event-bus'
+export type { EventBusConfig } from './event-bus'
+
 // Types
 export type { 
   EndpointConfig, 

package/src/router.ts

@@ -2,6 +2,7 @@ import type { Hono } from 'hono'
 import { readdirSync, statSync } from 'fs'
 import { join, resolve, relative, sep } from 'path'
 import { pathToFileURL } from 'url'
+import type { EventBus } from './event-bus'
 
 /**
  * Autoload Vision/Hono sub-apps from a directory structure like app/routes/.../index.ts
@@ -12,7 +13,7 @@ import { pathToFileURL } from 'url'
  * - app/routes/users/[id]/index.ts   -> /users/:id
  * - app/routes/index.ts              -> /
  */
-export async function loadSubApps(app: Hono, routesDir: string = './app/routes'): Promise<Array<{ name: string; routes: any[] }>> {
+export async function loadSubApps(app: Hono, routesDir: string = './app/routes', eventBus?: EventBus): Promise<Array<{ name: string; routes: any[] }>> {
   const mounted: Array<{ base: string }> = []
   const allSubAppSummaries: Array<{ name: string; routes: any[] }> = []
 
@@ -37,6 +38,10 @@ export async function loadSubApps(app: Hono, routesDir: string = './app/routes')
       const mod: any = await import(modUrl)
       const subApp = mod?.default
       if (subApp) {
+        // Inject EventBus into sub-app if it's a Vision instance
+        if (eventBus && typeof subApp?.setEventBus === 'function') {
+          subApp.setEventBus(eventBus)
+        }
         const base = toBasePath(dir)
         // If it's a Vision sub-app, build its services before mounting
         try {

package/src/service.ts

@@ -459,8 +459,10 @@ export class ServiceBuilder<
                 throw error
               }
             }
-            
-            // Add emit() method to context with type-safe event validation
+          }
+
+          // Always provide emit() so events work in sub-apps without local VisionCore
+          if (!(c as any).emit) {
             (c as any).emit = async <K extends keyof TEvents>(
               eventName: K,
               data: TEvents[K]

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
  */
@@ -49,6 +73,7 @@ export interface VisionConfig {
       password?: string
     }
     devMode?: boolean  // Use in-memory event bus (no Redis required)
+    eventBus?: EventBus  // Share EventBus instance across apps (for sub-apps)
   }
 }
 
@@ -61,20 +86,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 +118,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
@@ -184,8 +194,9 @@ export class Vision<
       this.visionCore = null as any
     }
     
-    // Initialize EventBus
-    this.eventBus = new EventBus({
+    // Use provided EventBus or create a new one
+    // Root app creates EventBus, sub-apps can share it via config.pubsub.eventBus
+    this.eventBus = this.config.pubsub?.eventBus || new EventBus({
       redis: this.config.pubsub?.redis,
       devMode: this.config.pubsub?.devMode,
     })
@@ -536,7 +547,8 @@ export class Vision<
     let allSubAppSummaries: Array<{ name: string; routes: any[] }> = []
     for (const d of existing) {
       try {
-        const summaries = await loadSubApps(this as any, d)
+        // Pass EventBus to sub-apps so they share the same instance
+        const summaries = await loadSubApps(this as any, d, this.eventBus)
         allSubAppSummaries = allSubAppSummaries.concat(summaries)
       } catch (e) {
         console.error(`❌ Failed to load sub-apps from ${d}:`, (e as any)?.message || e)
@@ -610,6 +622,13 @@ export class Vision<
       return this
     }
   }
+
+  /**
+   * Set the EventBus instance (used internally by router to inject shared EventBus)
+   */
+  setEventBus(eventBus: EventBus): void {
+    this.eventBus = eventBus
+  }
 }
 
 /**