@donotdev/functions 0.0.11 → 0.0.13

package/src/shared/billing/__tests__/createCheckout.test.ts

@@ -61,7 +61,9 @@ const BASE_REQUEST: CreateCheckoutSessionRequest = {
 // Mock factories
 // ---------------------------------------------------------------------------
 
-function makeStripeProvider(overrides?: Partial<StripeCheckoutProvider>): StripeCheckoutProvider {
+function makeStripeProvider(
+  overrides?: Partial<StripeCheckoutProvider>
+): StripeCheckoutProvider {
   return {
     createCheckoutSession: vi.fn().mockResolvedValue({
       id: 'cs_test_session123',
@@ -71,7 +73,9 @@ function makeStripeProvider(overrides?: Partial<StripeCheckoutProvider>): Stripe
   };
 }
 
-function makeAuthProvider(overrides?: Partial<AuthCheckoutProvider>): AuthCheckoutProvider {
+function makeAuthProvider(
+  overrides?: Partial<AuthCheckoutProvider>
+): AuthCheckoutProvider {
   return {
     getUser: vi.fn().mockResolvedValue({ customClaims: {} }),
     ...overrides,
@@ -112,14 +116,24 @@ describe('createCheckoutAlgorithm', () => {
     });
 
     it('verifies the user before creating the session', async () => {
-      await createCheckoutAlgorithm(BASE_REQUEST, stripeProvider, authProvider, BILLING_CONFIG);
+      await createCheckoutAlgorithm(
+        BASE_REQUEST,
+        stripeProvider,
+        authProvider,
+        BILLING_CONFIG
+      );
 
       expect(authProvider.getUser).toHaveBeenCalledOnce();
       expect(authProvider.getUser).toHaveBeenCalledWith('user_001');
     });
 
     it('forwards priceId, mode, successUrl, cancelUrl to Stripe', async () => {
-      await createCheckoutAlgorithm(BASE_REQUEST, stripeProvider, authProvider, BILLING_CONFIG);
+      await createCheckoutAlgorithm(
+        BASE_REQUEST,
+        stripeProvider,
+        authProvider,
+        BILLING_CONFIG
+      );
 
       expect(stripeProvider.createCheckoutSession).toHaveBeenCalledWith(
         expect.objectContaining({
@@ -132,7 +146,12 @@ describe('createCheckoutAlgorithm', () => {
     });
 
     it('ensures userId and billingConfigKey are set in session metadata', async () => {
-      await createCheckoutAlgorithm(BASE_REQUEST, stripeProvider, authProvider, BILLING_CONFIG);
+      await createCheckoutAlgorithm(
+        BASE_REQUEST,
+        stripeProvider,
+        authProvider,
+        BILLING_CONFIG
+      );
 
       expect(stripeProvider.createCheckoutSession).toHaveBeenCalledWith(
         expect.objectContaining({
@@ -150,7 +169,12 @@ describe('createCheckoutAlgorithm', () => {
         customerEmail: 'user@example.com',
       };
 
-      await createCheckoutAlgorithm(request, stripeProvider, authProvider, BILLING_CONFIG);
+      await createCheckoutAlgorithm(
+        request,
+        stripeProvider,
+        authProvider,
+        BILLING_CONFIG
+      );
 
       expect(stripeProvider.createCheckoutSession).toHaveBeenCalledWith(
         expect.objectContaining({ customerEmail: 'user@example.com' })
@@ -163,7 +187,12 @@ describe('createCheckoutAlgorithm', () => {
         allowPromotionCodes: undefined,
       };
 
-      await createCheckoutAlgorithm(request, stripeProvider, authProvider, BILLING_CONFIG);
+      await createCheckoutAlgorithm(
+        request,
+        stripeProvider,
+        authProvider,
+        BILLING_CONFIG
+      );
 
       expect(stripeProvider.createCheckoutSession).toHaveBeenCalledWith(
         expect.objectContaining({ allowPromotionCodes: true })
@@ -176,7 +205,12 @@ describe('createCheckoutAlgorithm', () => {
         mode: undefined,
       };
 
-      await createCheckoutAlgorithm(request, stripeProvider, authProvider, BILLING_CONFIG);
+      await createCheckoutAlgorithm(
+        request,
+        stripeProvider,
+        authProvider,
+        BILLING_CONFIG
+      );
 
       expect(stripeProvider.createCheckoutSession).toHaveBeenCalledWith(
         expect.objectContaining({ mode: 'payment' })
@@ -233,7 +267,12 @@ describe('createCheckoutAlgorithm', () => {
       };
 
       await expect(
-        createCheckoutAlgorithm(request, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          request,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow('Missing billingConfigKey in metadata');
     });
 
@@ -244,7 +283,12 @@ describe('createCheckoutAlgorithm', () => {
       };
 
       await expect(
-        createCheckoutAlgorithm(request, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          request,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow('Missing billingConfigKey in metadata');
     });
 
@@ -255,7 +299,12 @@ describe('createCheckoutAlgorithm', () => {
       };
 
       await expect(
-        createCheckoutAlgorithm(request, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          request,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow();
 
       expect(stripeProvider.createCheckoutSession).not.toHaveBeenCalled();
@@ -274,7 +323,12 @@ describe('createCheckoutAlgorithm', () => {
       };
 
       await expect(
-        createCheckoutAlgorithm(request, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          request,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow('Invalid billing config key: nonexistent_plan');
     });
 
@@ -286,7 +340,12 @@ describe('createCheckoutAlgorithm', () => {
       };
 
       await expect(
-        createCheckoutAlgorithm(request, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          request,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow('Price ID mismatch with configuration');
     });
 
@@ -298,7 +357,12 @@ describe('createCheckoutAlgorithm', () => {
       };
 
       await expect(
-        createCheckoutAlgorithm(request, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          request,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow();
 
       expect(stripeProvider.createCheckoutSession).not.toHaveBeenCalled();
@@ -311,7 +375,12 @@ describe('createCheckoutAlgorithm', () => {
 
   describe('auth validation', () => {
     it('calls getUser with the request userId', async () => {
-      await createCheckoutAlgorithm(BASE_REQUEST, stripeProvider, authProvider, BILLING_CONFIG);
+      await createCheckoutAlgorithm(
+        BASE_REQUEST,
+        stripeProvider,
+        authProvider,
+        BILLING_CONFIG
+      );
 
       expect(authProvider.getUser).toHaveBeenCalledWith('user_001');
     });
@@ -322,7 +391,12 @@ describe('createCheckoutAlgorithm', () => {
       });
 
       await expect(
-        createCheckoutAlgorithm(BASE_REQUEST, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          BASE_REQUEST,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow('User not found');
     });
 
@@ -332,7 +406,12 @@ describe('createCheckoutAlgorithm', () => {
       });
 
       await expect(
-        createCheckoutAlgorithm(BASE_REQUEST, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          BASE_REQUEST,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow();
 
       expect(stripeProvider.createCheckoutSession).not.toHaveBeenCalled();
@@ -346,11 +425,18 @@ describe('createCheckoutAlgorithm', () => {
   describe('Stripe error handling', () => {
     it('propagates Stripe API errors', async () => {
       stripeProvider = makeStripeProvider({
-        createCheckoutSession: vi.fn().mockRejectedValue(new Error('Stripe network error')),
+        createCheckoutSession: vi
+          .fn()
+          .mockRejectedValue(new Error('Stripe network error')),
       });
 
       await expect(
-        createCheckoutAlgorithm(BASE_REQUEST, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          BASE_REQUEST,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow('Stripe network error');
     });
 
@@ -362,7 +448,12 @@ describe('createCheckoutAlgorithm', () => {
       });
 
       await expect(
-        createCheckoutAlgorithm(BASE_REQUEST, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          BASE_REQUEST,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow('Your card was declined');
     });
 
@@ -374,7 +465,12 @@ describe('createCheckoutAlgorithm', () => {
       });
 
       await expect(
-        createCheckoutAlgorithm(BASE_REQUEST, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          BASE_REQUEST,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toThrow('Too many requests');
     });
 
@@ -386,7 +482,12 @@ describe('createCheckoutAlgorithm', () => {
       });
 
       await expect(
-        createCheckoutAlgorithm(BASE_REQUEST, stripeProvider, authProvider, BILLING_CONFIG)
+        createCheckoutAlgorithm(
+          BASE_REQUEST,
+          stripeProvider,
+          authProvider,
+          BILLING_CONFIG
+        )
       ).rejects.toBe(unexpected);
     });
   });

package/src/shared/billing/__tests__/webhookHandler.test.ts

@@ -80,7 +80,10 @@ vi.mock('../../logger.js', () => ({
 // Reset idempotency store between tests so events are never pre-marked
 import { resetIdempotencyStore } from '../idempotency.js';
 import { processWebhook } from '../webhookHandler.js';
-import type { UpdateSubscriptionFn, WebhookAuthProvider } from '../webhookHandler.js';
+import type {
+  UpdateSubscriptionFn,
+  WebhookAuthProvider,
+} from '../webhookHandler.js';
 
 // ---------------------------------------------------------------------------
 // Shared test fixtures
@@ -217,15 +220,15 @@ describe('processWebhook — signature verification', () => {
     const stripe = {
       webhooks: {
         constructEvent: vi.fn(() => {
-          throw new Error('No signatures found matching the expected signature');
+          throw new Error(
+            'No signatures found matching the expected signature'
+          );
         }),
       },
       subscriptions: { retrieve: vi.fn() },
     };
 
-    await expect(
-      callProcessWebhook({ stripe })
-    ).rejects.toThrow();
+    await expect(callProcessWebhook({ stripe })).rejects.toThrow();
   });
 });
 
@@ -260,7 +263,10 @@ describe('processWebhook — idempotency', () => {
       updateSubscription: updateSubscription2,
     });
 
-    expect(result).toEqual({ success: true, message: 'Event already processed' });
+    expect(result).toEqual({
+      success: true,
+      message: 'Event already processed',
+    });
     expect(updateSubscription2).not.toHaveBeenCalled();
   });
 
@@ -279,8 +285,16 @@ describe('processWebhook — idempotency', () => {
     const updateSubscription1 = makeUpdateSubscription();
     const updateSubscription2 = makeUpdateSubscription();
 
-    await callProcessWebhook({ stripe: makeStripe(event1), config, updateSubscription: updateSubscription1 });
-    await callProcessWebhook({ stripe: makeStripe(event2), config, updateSubscription: updateSubscription2 });
+    await callProcessWebhook({
+      stripe: makeStripe(event1),
+      config,
+      updateSubscription: updateSubscription1,
+    });
+    await callProcessWebhook({
+      stripe: makeStripe(event2),
+      config,
+      updateSubscription: updateSubscription2,
+    });
 
     expect(updateSubscription1).toHaveBeenCalledTimes(1);
     expect(updateSubscription2).toHaveBeenCalledTimes(1);
@@ -349,7 +363,11 @@ describe('event routing — checkout.session.completed (StripeSubscription)', ()
     const config = makeConfig();
     const updateSubscription = makeUpdateSubscription();
 
-    await callProcessWebhook({ stripe: makeStripe(event), config, updateSubscription });
+    await callProcessWebhook({
+      stripe: makeStripe(event),
+      config,
+      updateSubscription,
+    });
 
     expect(updateSubscription).toHaveBeenCalledOnce();
     const [userId, data] = (updateSubscription as any).mock.calls[0];
@@ -370,9 +388,16 @@ describe('event routing — checkout.session.completed (StripeSubscription)', ()
       customer: 'cus_hook',
     });
 
-    await callProcessWebhook({ stripe: makeStripe(event), config, updateSubscription: makeUpdateSubscription() });
+    await callProcessWebhook({
+      stripe: makeStripe(event),
+      config,
+      updateSubscription: makeUpdateSubscription(),
+    });
 
-    expect(onSubscriptionCreated).toHaveBeenCalledWith('u_hook', expect.any(Object));
+    expect(onSubscriptionCreated).toHaveBeenCalledWith(
+      'u_hook',
+      expect.any(Object)
+    );
   });
 
   it('does NOT throw when onSubscriptionCreated hook throws (non-critical)', async () => {
@@ -387,7 +412,11 @@ describe('event routing — checkout.session.completed (StripeSubscription)', ()
     });
 
     await expect(
-      callProcessWebhook({ stripe: makeStripe(event), config, updateSubscription: makeUpdateSubscription() })
+      callProcessWebhook({
+        stripe: makeStripe(event),
+        config,
+        updateSubscription: makeUpdateSubscription(),
+      })
     ).resolves.toEqual({ success: true, message: 'Webhook processed' });
   });
 
@@ -459,7 +488,11 @@ describe('event routing — checkout.session.completed (StripePayment)', () => {
     const config = makeConfig();
     const updateSubscription = makeUpdateSubscription();
 
-    await callProcessWebhook({ stripe: makeStripe(event), config, updateSubscription });
+    await callProcessWebhook({
+      stripe: makeStripe(event),
+      config,
+      updateSubscription,
+    });
 
     expect(updateSubscription).toHaveBeenCalledOnce();
     const [userId, data] = (updateSubscription as any).mock.calls[0];
@@ -486,9 +519,16 @@ describe('event routing — checkout.session.completed (StripePayment)', () => {
       customer: 'cus_pay_hook',
     });
 
-    await callProcessWebhook({ stripe: makeStripe(event), config, updateSubscription: makeUpdateSubscription() });
+    await callProcessWebhook({
+      stripe: makeStripe(event),
+      config,
+      updateSubscription: makeUpdateSubscription(),
+    });
 
-    expect(onPurchaseSuccess).toHaveBeenCalledWith('u_pay_hook', expect.any(Object));
+    expect(onPurchaseSuccess).toHaveBeenCalledWith(
+      'u_pay_hook',
+      expect.any(Object)
+    );
   });
 
   it('does NOT throw when onPurchaseSuccess hook throws (non-critical)', async () => {
@@ -509,7 +549,11 @@ describe('event routing — checkout.session.completed (StripePayment)', () => {
     });
 
     await expect(
-      callProcessWebhook({ stripe: makeStripe(event), config, updateSubscription: makeUpdateSubscription() })
+      callProcessWebhook({
+        stripe: makeStripe(event),
+        config,
+        updateSubscription: makeUpdateSubscription(),
+      })
     ).resolves.toEqual({ success: true, message: 'Webhook processed' });
   });
 });
@@ -568,9 +612,16 @@ describe('event routing — invoice.payment_succeeded', () => {
       metadata: { userId: 'u_hook_renewal', billingConfigKey: 'pro_monthly' },
     }));
 
-    await callProcessWebhook({ stripe, config, updateSubscription: makeUpdateSubscription() });
+    await callProcessWebhook({
+      stripe,
+      config,
+      updateSubscription: makeUpdateSubscription(),
+    });
 
-    expect(onSubscriptionRenewed).toHaveBeenCalledWith('u_hook_renewal', expect.any(Object));
+    expect(onSubscriptionRenewed).toHaveBeenCalledWith(
+      'u_hook_renewal',
+      expect.any(Object)
+    );
   });
 
   it('does NOT throw when onSubscriptionRenewed hook throws (non-critical)', async () => {
@@ -593,7 +644,11 @@ describe('event routing — invoice.payment_succeeded', () => {
     }));
 
     await expect(
-      callProcessWebhook({ stripe, config, updateSubscription: makeUpdateSubscription() })
+      callProcessWebhook({
+        stripe,
+        config,
+        updateSubscription: makeUpdateSubscription(),
+      })
     ).resolves.toEqual({ success: true, message: 'Webhook processed' });
   });
 
@@ -647,7 +702,10 @@ describe('event routing — invoice.payment_succeeded', () => {
     const stripe = makeStripe(event);
     stripe.subscriptions.retrieve = vi.fn(async () => ({
       id: 'sub_payment_type',
-      metadata: { userId: 'u_payment_type', billingConfigKey: 'lifetime_purchase' },
+      metadata: {
+        userId: 'u_payment_type',
+        billingConfigKey: 'lifetime_purchase',
+      },
     }));
     const updateSubscription = makeUpdateSubscription();
 
@@ -723,9 +781,16 @@ describe('event routing — customer.subscription.deleted', () => {
     };
     const event = makeEvent('customer.subscription.deleted', subscription);
 
-    await callProcessWebhook({ stripe: makeStripe(event), config, updateSubscription: makeUpdateSubscription() });
+    await callProcessWebhook({
+      stripe: makeStripe(event),
+      config,
+      updateSubscription: makeUpdateSubscription(),
+    });
 
-    expect(onSubscriptionCancelled).toHaveBeenCalledWith('u_cancel_hook', expect.any(Object));
+    expect(onSubscriptionCancelled).toHaveBeenCalledWith(
+      'u_cancel_hook',
+      expect.any(Object)
+    );
   });
 
   it('does NOT throw when onSubscriptionCancelled hook throws (non-critical)', async () => {
@@ -744,7 +809,11 @@ describe('event routing — customer.subscription.deleted', () => {
     const event = makeEvent('customer.subscription.deleted', subscription);
 
     await expect(
-      callProcessWebhook({ stripe: makeStripe(event), config, updateSubscription: makeUpdateSubscription() })
+      callProcessWebhook({
+        stripe: makeStripe(event),
+        config,
+        updateSubscription: makeUpdateSubscription(),
+      })
     ).resolves.toEqual({ success: true, message: 'Webhook processed' });
   });
 
@@ -776,7 +845,11 @@ describe('event routing — customer.subscription.deleted', () => {
     const event = makeEvent('customer.subscription.deleted', subscription);
     const updateSubscription = makeUpdateSubscription();
 
-    await callProcessWebhook({ stripe: makeStripe(event), config, updateSubscription });
+    await callProcessWebhook({
+      stripe: makeStripe(event),
+      config,
+      updateSubscription,
+    });
 
     expect(updateSubscription).not.toHaveBeenCalled();
   });
@@ -824,7 +897,10 @@ describe('event routing — invoice.payment_failed', () => {
 
     await callProcessWebhook({ stripe: makeStripe(event), config });
 
-    expect(onPurchaseFailure).toHaveBeenCalledWith('u_fail_pay', expect.any(Object));
+    expect(onPurchaseFailure).toHaveBeenCalledWith(
+      'u_fail_pay',
+      expect.any(Object)
+    );
   });
 
   it('calls onPaymentFailed for StripeSubscription billing items', async () => {
@@ -839,7 +915,10 @@ describe('event routing — invoice.payment_failed', () => {
 
     await callProcessWebhook({ stripe: makeStripe(event), config });
 
-    expect(onPaymentFailed).toHaveBeenCalledWith('u_fail_sub', expect.any(Object));
+    expect(onPaymentFailed).toHaveBeenCalledWith(
+      'u_fail_sub',
+      expect.any(Object)
+    );
   });
 
   it('does NOT throw when onPurchaseFailure hook throws (non-critical)', async () => {
@@ -922,7 +1001,11 @@ describe('event routing — invoice.payment_failed', () => {
     const event = makeEvent('invoice.payment_failed', invoice);
     const updateSubscription = makeUpdateSubscription();
 
-    await callProcessWebhook({ stripe: makeStripe(event), config, updateSubscription });
+    await callProcessWebhook({
+      stripe: makeStripe(event),
+      config,
+      updateSubscription,
+    });
 
     expect(updateSubscription).not.toHaveBeenCalled();
   });
@@ -952,7 +1035,9 @@ describe('event routing — unknown event types', () => {
   });
 
   it('handles payment_intent.succeeded gracefully (not in router)', async () => {
-    const event = makeEvent('payment_intent.succeeded', { id: 'pi_not_routed' });
+    const event = makeEvent('payment_intent.succeeded', {
+      id: 'pi_not_routed',
+    });
 
     const result = await callProcessWebhook({ stripe: makeStripe(event) });
 
@@ -997,7 +1082,10 @@ describe('state machine transitions', () => {
 
   it('checkout.session.completed → status=active (StripePayment)', async () => {
     const event = makeEvent('checkout.session.completed', {
-      metadata: { userId: 'u_sm_pay_active', billingConfigKey: 'lifetime_purchase' },
+      metadata: {
+        userId: 'u_sm_pay_active',
+        billingConfigKey: 'lifetime_purchase',
+      },
       subscription: null,
       customer: 'cus_sm_pay_active',
     });
@@ -1086,6 +1174,9 @@ describe('processWebhook — return value', () => {
     await callProcessWebhook({ stripe: makeStripe(event) });
     const result = await callProcessWebhook({ stripe: makeStripe(event) });
 
-    expect(result).toEqual({ success: true, message: 'Event already processed' });
+    expect(result).toEqual({
+      success: true,
+      message: 'Event already processed',
+    });
   });
 });

package/src/shared/errorHandling.ts

@@ -9,7 +9,6 @@
  * @author AMBROISE PARK Consulting
  */
 
-import { HttpsError } from 'firebase-functions/v2/https';
 import * as v from 'valibot';
 
 import { DoNotDevError, EntityHookError } from '@donotdev/core/server';
@@ -26,7 +25,7 @@ export { DoNotDevError };
  * @since 0.0.1
  * @author AMBROISE PARK Consulting
  */
-export function handleError(error: unknown): never {
+export async function handleError(error: unknown): Promise<never> {
   // W11: Log only in development to avoid double-logging with platform loggers.
   // Known DoNotDevError, ValiError, and EntityHookError are handled by callers.
   if (
@@ -97,7 +96,7 @@ export function handleError(error: unknown): never {
 
   // Platform-specific error throwing
   if (isFirebaseEnvironment()) {
-    throwFirebaseError(code, message, details);
+    await throwFirebaseError(code, message, details);
   } else if (isVercelEnvironment()) {
     throwVercelError(code, message, details);
   } else {
@@ -143,12 +142,13 @@ function isVercelEnvironment(): boolean {
  * @since 0.0.1
  * @author AMBROISE PARK Consulting
  */
-function throwFirebaseError(
+async function throwFirebaseError(
   code: string,
   message: string,
   details?: any
-): never {
-  // Static import - Firebase is always available in Firebase functions
+): Promise<never> {
+  // Dynamic import — firebase-functions is only loaded in Firebase environments
+  const { HttpsError } = await import('firebase-functions/v2/https');
   throw new HttpsError(code as any, message, details);
 }
 

package/src/shared/firebase.ts

@@ -212,4 +212,4 @@ export function prepareForFirestore<T = any>(
 
   // Return primitive values unchanged
   return data;
-}
+}

package/src/shared/index.ts

@@ -19,8 +19,9 @@ export * from './oauth/index.js';
 export {
   createTimestamp,
   toTimestamp,
-  toISOString,
   isTimestamp,
   transformFirestoreData,
   prepareForFirestore,
 } from './firebase.js';
+// Note: toISOString is exported from ./utils/external/date.js (handles DateValue).
+// The Firebase-specific toISOString (FirestoreTimestamp only) is available via direct import from ./firebase.js.

package/src/shared/logger.ts

@@ -28,10 +28,8 @@ if (typeof globalThis !== 'undefined' && 'window' in globalThis) {
   throw new Error('Server logger cannot be imported on client side');
 }
 
-// ServerShim: Ensure we're in a Node.js environment
-if (typeof process === 'undefined' || !process.versions?.node) {
-  throw new Error('Server logger requires Node.js environment');
-}
+// ServerShim: Warn if not in Node.js (e.g., Deno) but don't throw — allow fallback logging
+const _isNodeEnv = typeof process !== 'undefined' && !!process.versions?.node;
 
 let sentryEnabled = false;
 let sentryClient: any = null;
@@ -247,9 +245,13 @@ export const logger = {
       context,
       error: context?.error,
       metadata: {
-        pid: process.pid,
-        nodeVersion: process.version,
-        platform: process.platform,
+        ...(_isNodeEnv
+          ? {
+              pid: process.pid,
+              nodeVersion: process.version,
+              platform: process.platform,
+            }
+          : {}),
         ...context?.metadata,
       },
     };