fix(observation): warn without error on observations without sessions

src/logic/capture/captureObservationEvent.ts

@@ -21,7 +21,27 @@ export const captureObservationEvent = async ({
   details: AppUsageEvent['details'];
 }) => {
   if (typeof window === 'undefined') return; // do nothing on serverside, since no installation-uuid available
-  await waitForApplicationOfSession();
+
+  // wait for the app session to be loaded
+  try {
+    await waitForApplicationOfSession();
+  } catch (error) {
+    if (!(error instanceof Error)) throw error;
+
+    // if the error was due to not having the app session, then just warn and exit. dont pass up the error
+    if (error.message.includes('application wait attempts exceeded')) {
+      console.warn(
+        'could not capture observation event, app.session was not instantiated. was the AppUsageEventProvider loaded?',
+        { type, details },
+      );
+      return;
+    }
+
+    // otherwise, pass up the error since we dont know what to do with it
+    throw error;
+  }
+
+  // capture the event
   await captureAppUsageEvent({
     source: AppUsageEventSource.OBSERVATION,
     type,

src/logic/session/getApplicationOfSession.ts

@@ -1,3 +1,6 @@
+import { createCache } from 'simple-in-memory-cache';
+import { withSimpleCaching } from 'with-simple-caching';
+
 import { AppUsageApplication } from '../../domain/AppUsageApplication';
 
 // track the application in memory
@@ -13,10 +16,13 @@ export const getApplicationOfSession = (): AppUsageApplication => {
 
 export const sleep = (ms: number) =>
   new Promise((resolve) => setTimeout(resolve, ms));
-export const waitForApplicationOfSession = async () => {
-  for (let attempt = 0; attempt < 5; attempt++) {
-    if (application) return application;
-    await sleep(300);
-  }
-  throw new Error('application wait attempts exceeded');
-};
+export const waitForApplicationOfSession = withSimpleCaching(
+  async () => {
+    for (let attempt = 0; attempt < 5; attempt++) {
+      if (application) return application;
+      await sleep(300);
+    }
+    throw new Error('application wait attempts exceeded');
+  },
+  { cache: createCache() }, // cache the promise in memory, so we only ever create one promise for this no matter how many duplicate requests
+);