docs(changeset): chore: added better-auth on error handling for production (console.error(error, AuthContext)

chore: added upstash vs valkey getter handling (upstash does not resturn
string)
chore: added wrangler dev, bun dev, and production runtime handling for server
chore: context must be re-created (not a singleton) for edge runtime)

Author: bauchdj

.changeset/fancy-moments-slide.md

@@ -0,0 +1,8 @@
+---
+"@nimbus/auth": patch
+"@nimbus/server": patch
+---
+
+chore: added better-auth on error handling for production (console.error(error, AuthContext) chore: added upstash vs
+valkey getter handling (upstash does not resturn string) chore: added wrangler dev, bun dev, and production runtime
+handling for server chore: context must be re-created (not a singleton) for edge runtime)

apps/server/package.json

@@ -12,7 +12,7 @@
 		"check": "tsc --noEmit",
 		"build": "bun run check && bun build src/index.ts --target bun --minify --outdir dist",
 		"start": "bun dist/index.js",
-		"cf:dev": "wrangler dev",
+		"cf:dev": "sudo wrangler dev",
 		"cf:deploy:preview": "wrangler deploy --env preview",
 		"cf:deploy:staging": "wrangler deploy --env staging",
 		"cf:deploy:production": "wrangler deploy --env production",

apps/server/src/context.ts

@@ -23,7 +23,7 @@ export class ContextManager {
 	}
 
 	public static getInstance(): ContextManager {
-		if (!ContextManager.instance) {
+		if (!ContextManager.instance || ContextManager.instance.env.IS_EDGE_RUNTIME) {
 			ContextManager.instance = new ContextManager();
 		}
 		return ContextManager.instance;

apps/server/src/index.ts

@@ -29,7 +29,9 @@ const app = createPublicRouter()
 		try {
 			await next();
 		} finally {
-			if (env.IS_EDGE_RUNTIME) {
+			// WARNING: make sure to add WRANGLER_DEV to .dev.vars for wrangler dev
+			// for local dev, always keep context open UNLESS wrangler dev, close context
+			if (env.IS_EDGE_RUNTIME && (env.NODE_ENV === "production" || process.env.WRANGLER_DEV === "true")) {
 				await ContextManager.getInstance().close();
 			}
 		}
@@ -41,17 +43,10 @@ export type AppType = typeof app;
 
 // Create a wrapper that handles the environment initialization
 const handler = {
+	port: process.env.SERVER_PORT,
 	async fetch(request: Request, env: any, ctx: any) {
 		return app.fetch(request, env, ctx);
 	},
 };
 
-// For non-edge environments, we'll use the original app with the port
-if (process.env.IS_EDGE_RUNTIME !== "true") {
-	Object.assign(handler, {
-		port: process.env.SERVER_PORT,
-		...app,
-	});
-}
-
 export default handler;

apps/server/wrangler.toml

@@ -5,7 +5,9 @@ compatibility_flags = ["nodejs_compat"]
 observability = { enabled = true }
 
 [dev]
-port = 1284
+# wrangler dev must be run on port 80 and with sudo because wrangler sets
+# Access-Control-Allow-Origin to http://localhost:1284:1284:3000
+port = 80
 host = "localhost"
 local_protocol = "http"
 

packages/auth/src/auth.ts

@@ -1,7 +1,7 @@
+import { type Account, type AuthContext, betterAuth } from "better-auth";
 import type { Redis as UpstashRedis } from "@upstash/redis/cloudflare";
 import { drizzleAdapter } from "better-auth/adapters/drizzle";
 import schema, { user as userTable } from "@nimbus/db/schema";
-import { type Account, betterAuth } from "better-auth";
 import type { Redis as ValkeyRedis } from "iovalkey";
 import type { RedisClient } from "@nimbus/cache";
 import { providerSchema } from "@nimbus/shared";
@@ -24,7 +24,13 @@ export const createAuth = (env: Env, db: DB, redisClient: RedisClient, resend: R
 		appName: "Nimbus",
 		baseURL: env.BACKEND_URL,
 		trustedOrigins: [...env.TRUSTED_ORIGINS, env.BACKEND_URL],
-		// onApiError: { errorUrl: env.FRONTEND_URL },
+		onApiError: {
+			// errorUrl: env.FRONTEND_URL,
+			// throwError: true,
+			onError: (error: unknown, ctx: AuthContext) => {
+				console.error("API error", error, ctx);
+			},
+		},
 
 		// Ensure state is properly handled
 		state: {
@@ -100,7 +106,14 @@ export const createAuth = (env: Env, db: DB, redisClient: RedisClient, resend: R
 
 		secondaryStorage: {
 			get: async (key: string) => {
-				return await redisClient.get(key);
+				if (env.IS_EDGE_RUNTIME) {
+					const value = await (redisClient as UpstashRedis).get(key);
+					const string = JSON.stringify(value);
+					return string as string | null;
+				} else {
+					const value = await (redisClient as ValkeyRedis).get(key);
+					return value as string | null;
+				}
 			},
 			set: async (key: string, value: string, ttl?: number) => {
 				if (ttl) {