address comments.

Author: ItzNotABug

package.json

@@ -19,7 +19,7 @@
 	"keywords": [],
 	"author": "",
 	"license": "ISC",
-	"packageManager": "pnpm@10.20.0",
+	"packageManager": "bun@1.3.2",
 	"dependencies": {
 		"lighthouse": "^12.2.1",
 		"playwright-core": "^1.52.0",

src/config/index.ts

@@ -1,4 +1,4 @@
 export * from "./browser.js";
 export * from "./lighthouse.js";
 
-export const port = Number(process.env.PORT) || 3000;
+export const port = process.env.PORT ? Number(process.env.PORT) : 3000;

src/routes/reports.ts

@@ -1,9 +1,11 @@
-import type { BrowserContextOptions } from "playwright-core";
+import type { BrowserContext, BrowserContextOptions } from "playwright-core";
 import { playAudit } from "playwright-lighthouse";
 import { browser, defaultContext, lighthouseConfigs } from "../config";
 import { lighthouseSchema } from "../schemas";
 
 export async function handleReportsRequest(req: Request): Promise<Response> {
+	let context: BrowserContext | undefined;
+
 	try {
 		const json = await req.json();
 		const body = lighthouseSchema.parse(json);
@@ -19,7 +21,7 @@ export async function handleReportsRequest(req: Request): Promise<Response> {
 		if (body.locale) contextOptions.locale = body.locale;
 		if (body.timezoneId) contextOptions.timezoneId = body.timezoneId;
 
-		const context = await browser.newContext(contextOptions);
+		context = await browser.newContext(contextOptions);
 
 		// Grant permissions if specified
 		if (body.permissions && body.permissions.length > 0) {
@@ -72,7 +74,6 @@ export async function handleReportsRequest(req: Request): Promise<Response> {
 			thresholds,
 		});
 
-		await context.close();
 		const report = Array.isArray(results.report)
 			? results.report.join("")
 			: results.report;
@@ -85,5 +86,7 @@ export async function handleReportsRequest(req: Request): Promise<Response> {
 			status: 400,
 			headers: { "Content-Type": "application/json" },
 		});
+	} finally {
+		await context?.close();
 	}
 }

src/routes/screenshots.ts

@@ -1,4 +1,5 @@
 import type {
+	BrowserContext,
 	BrowserContextOptions,
 	PageScreenshotOptions,
 } from "playwright-core";
@@ -8,6 +9,8 @@ import { screenshotSchema } from "../schemas";
 export async function handleScreenshotsRequest(
 	req: Request,
 ): Promise<Response> {
+	let context: BrowserContext | undefined;
+
 	try {
 		const json = await req.json();
 		const body = screenshotSchema.parse(json);
@@ -28,7 +31,7 @@ export async function handleScreenshotsRequest(
 		if (body.timezoneId) contextOptions.timezoneId = body.timezoneId;
 		if (body.geolocation) contextOptions.geolocation = body.geolocation;
 
-		const context = await browser.newContext(contextOptions);
+		context = await browser.newContext(contextOptions);
 
 		// Grant permissions if specified
 		if (body.permissions && body.permissions.length > 0) {
@@ -78,8 +81,6 @@ export async function handleScreenshotsRequest(
 
 		const screen = await page.screenshot(screenshotOptions);
 
-		await context.close();
-
 		return new Response(Buffer.from(screen), {
 			headers: {
 				"Content-Type": `image/${body.format}`,
@@ -91,5 +92,7 @@ export async function handleScreenshotsRequest(
 			status: 400,
 			headers: { "Content-Type": "application/json" },
 		});
+	} finally {
+		await context?.close();
 	}
 }

src/routes/test.ts

@@ -3,17 +3,21 @@ import { generateTestHTML } from "../utils/test-page.js";
 
 export async function handleTestRequest(_req: Request): Promise<Response> {
 	const context = await browser.newContext(defaultContext);
-	const page = await context.newPage();
+	try {
+		const page = await context.newPage();
 
-	await page.goto("about:blank");
+		await page.goto("about:blank");
 
-	const html = await page.evaluate(() => {
-		return new Date().toISOString();
-	});
+		const html = await page.evaluate(() => {
+			return new Date().toISOString();
+		});
 
-	await context.close();
+		await context.close();
 
-	return new Response(generateTestHTML(html), {
-		headers: { "Content-Type": "text/html" },
-	});
+		return new Response(generateTestHTML(html), {
+			headers: { "Content-Type": "text/html" },
+		});
+	} finally {
+		await context.close();
+	}
 }