fix: request util not awaiting json parsing, import replay package while replaying

Author: skoshx

e2e/replay.spec.ts

@@ -1,7 +1,7 @@
 import { expect, test } from "@playwright/test";
 import { REPLAY_NEXTJS_PORT, getBaseUrl, waitMs } from "./util";
 
-test('should replay front-end bug in nextjs example', async ({ page }) => {
+test.only('should replay front-end bug in nextjs example', async ({ page }) => {
 	// const capturesBefore = await fetchCaptures('sk_Y8dYLe5VoNJfDI_CPEqF8AqMTEwWAvTwBi7Ml-pN9bzWsgsP');
 	// Start from the index page (the baseURL is set via the webServer in the playwright.config.ts)
   await page.goto(getBaseUrl(REPLAY_NEXTJS_PORT));

src/core/requestUtils.ts

@@ -25,8 +25,10 @@ export async function request<DataType = unknown>(
 			...(options.headers && { headers: options.headers }),
 			body
 		})
-		if (data.status >= 400 && data.status < 600) throw await data.text()
-		return Ok(data.json() as DataType)
+		if (data.status >= 400 && data.status < 600) {
+			throw await data.text()
+		}
+		return Ok((await data.json()) as DataType)
 	} catch (error) {
 		return Err(
 			createHumanLog({

src/core/types.ts

@@ -135,6 +135,8 @@ export type FlytrapConfig = {
 	/**
 	 * Use Flytrap in environments with access to only browser APIs, eg. Cloudflare
 	 * Workers & Pages and Deno Deploy.
+	 *
+	 * @deprecated Starting from v0.8 the Flytrap SDK only uses browser & edge-friendly APIs, meaning that the `browser` option is no longer needed.
 	 */
 	browser?: true
 	transformOptions?: {

src/transform/imports.ts

@@ -67,14 +67,14 @@ export function addMissingFlytrapImports(
 			s.appendLeft(
 				startingIndex,
 				`\n\nconst { ${importsToBeAdded.join(', ')} } = require('${FLYTRAP_PACKAGE_NAME}${
-					config?.browser === true ? '/browser' : ''
+					config?.mode === 'replay' ? '/replay' : ''
 				}');\n\n`
 			)
 		} else {
 			s.appendLeft(
 				startingIndex,
 				`\n\nimport { ${importsToBeAdded.join(', ')} } from '${FLYTRAP_PACKAGE_NAME}${
-					config?.browser === true ? '/browser' : ''
+					config?.mode === 'replay' ? '/replay' : ''
 				}';\n\n`
 			)
 		}