fix: dont prevent execution when capturing async function

Author: skoshx

e2e/capture.spec.ts

@@ -61,6 +61,7 @@ test('should capture front-end bug in nextjs example', async ({ page }) => {
   // Start from the index page (the baseURL is set via the webServer in the playwright.config.ts)
   await page.goto(getBaseUrl(NEXTJS_PORT));
   await page.reload()
+  await waitMs(3_000)
 
   // Find the input
   await page.locator('input').fill('wrong')
@@ -73,7 +74,7 @@ test('should capture front-end bug in nextjs example', async ({ page }) => {
   const consoleLogsArgs = await consolePromise.args()
   // expect(await consoleLogsArgs[0]?.jsonValue()).toContain('[🐛 post] https://www.useflytrap.com/api/v1/captures')
 
-  await waitMs(10_000);
+  await waitMs(3_000);
 
   // Check the API for the capture
   const capturesAfter = await fetchCaptures('sk_Y8dYLe5VoNJfDI_CPEqF8AqMTEwWAvTwBi7Ml-pN9bzWsgsP');
@@ -89,7 +90,7 @@ test('should capture front-end bug in sveltekit example', async ({ page }) => {
   // Start from the index page (the baseURL is set via the webServer in the playwright.config.ts)
   await page.goto(getBaseUrl(SVELTE_PORT));
   await page.reload()
-  await waitMs(1000)
+  await waitMs(3_000)
 
   // Find the input
   await page.locator('input').fill('wrong')
@@ -116,7 +117,7 @@ test('should capture front-end bug in nuxt example', async ({ page }) => {
   // Start from the index page (the baseURL is set via the webServer in the playwright.config.ts)
   await page.goto(getBaseUrl(NUXT_PORT));
   await page.reload()
-  await waitMs(1000)
+  await waitMs(3_000)
 
   // Find the input
   await page.locator('input').fill('wrong')

examples/with-nuxt/package.json

@@ -12,7 +12,6 @@
 		"useflytrap": "workspace:*"
 	},
 	"devDependencies": {
-		"@types/node": "^18",
 		"nuxt": "^3.5.1"
 	}
 }

pnpm-lock.yaml

@@ -65,19 +65,18 @@ export function useFlytrapFunction<
 				 */
 				saveErrorForFunction(opts.id, error)
 				log.info('capture', `Captured error in async function with ID "${opts.id}".`, { error })
-				const { error: saveError } = await utilTryCatch(
-					getFlytrapStorage().saveCapture(_executingFunctions, _functionCalls, error as Error)
-				)
-				if (saveError) {
-					console.error(
-						createHumanLog({
-							event: 'capture_failed',
-							explanation: 'api_capture_error_response',
-							solution: 'try_again_contact_us'
-						}).toString()
-					)
-					console.error(saveError)
-				}
+				getFlytrapStorage()
+					.saveCapture(_executingFunctions, _functionCalls, error as Error)
+					.catch((saveError) => {
+						console.error(
+							createHumanLog({
+								event: 'capture_failed',
+								explanation: 'api_capture_error_response',
+								solution: 'try_again_contact_us'
+							}).toString()
+						)
+						console.error(saveError)
+					})
 
 				throw error
 			}
@@ -125,18 +124,17 @@ export function useFlytrapFunction<
 			 */
 			saveErrorForFunction(opts.id, error)
 			log.info('capture', `Captured error in function with ID "${opts.id}".`, { error })
-			const { error: saveError } = utilTryCatchSync(() =>
-				getFlytrapStorage().saveCapture(_executingFunctions, _functionCalls, error as Error)
-			)
-			if (saveError) {
-				const errorLog = createHumanLog({
-					event: 'capture_failed',
-					explanation: 'generic_unexpected_error',
-					solution: 'try_again_contact_us'
+			getFlytrapStorage()
+				.saveCapture(_executingFunctions, _functionCalls, error as Error)
+				.catch((saveError) => {
+					const errorLog = createHumanLog({
+						event: 'capture_failed',
+						explanation: 'generic_unexpected_error',
+						solution: 'try_again_contact_us'
+					})
+					console.error(errorLog.toString())
+					console.error(saveError)
 				})
-				console.error(errorLog.toString())
-				console.error(saveError)
-			}
 
 			throw error
 		}

src/index.ts

@@ -10,6 +10,7 @@ export default defineConfig({
 			useflytrap: fileURLToPath(new URL('./src/index.ts', import.meta.url).href)
 		}
 	},
+	// @ts-ignore
 	test: {
 		coverage: {
 			'100': true,