feat: improved screenshot accuracy

app/biome.json

@@ -1,48 +1,48 @@
 {
-  "$schema": "https://biomejs.dev/schemas/2.4.6/schema.json",
-  "vcs": {
-    "enabled": true,
-    "clientKind": "git",
-    "useIgnoreFile": false
-  },
-  "formatter": {
-    "enabled": true,
-    "indentStyle": "tab",
-    "indentWidth": 2,
-    "lineWidth": 100
-  },
-  "linter": {
-    "enabled": true,
-    "rules": {
-      "recommended": true,
-      "correctness": {
-        "useExhaustiveDependencies": "warn",
-        "noUnusedImports": "warn",
-        "noUnusedVariables": "warn"
-      },
-      "style": {
-        "noNonNullAssertion": "off"
-      },
-      "suspicious": {
-        "noExplicitAny": "warn",
-        "noArrayIndexKey": "warn"
-      },
-      "a11y": {
-        "useButtonType": "warn"
-      },
-      "performance": {
-        "noImgElement": "warn"
-      },
-      "security": {
-        "noDangerouslySetInnerHtml": "warn"
-      }
-    }
-  },
-  "javascript": {
-    "formatter": {
-      "quoteStyle": "double",
-      "semicolons": "always",
-      "trailingCommas": "es5"
-    }
-  }
+	"$schema": "https://biomejs.dev/schemas/2.4.6/schema.json",
+	"vcs": {
+		"enabled": true,
+		"clientKind": "git",
+		"useIgnoreFile": false
+	},
+	"formatter": {
+		"enabled": true,
+		"indentStyle": "tab",
+		"indentWidth": 2,
+		"lineWidth": 100
+	},
+	"linter": {
+		"enabled": true,
+		"rules": {
+			"recommended": true,
+			"correctness": {
+				"useExhaustiveDependencies": "warn",
+				"noUnusedImports": "warn",
+				"noUnusedVariables": "warn"
+			},
+			"style": {
+				"noNonNullAssertion": "off"
+			},
+			"suspicious": {
+				"noExplicitAny": "warn",
+				"noArrayIndexKey": "warn"
+			},
+			"a11y": {
+				"useButtonType": "warn"
+			},
+			"performance": {
+				"noImgElement": "warn"
+			},
+			"security": {
+				"noDangerouslySetInnerHtml": "warn"
+			}
+		}
+	},
+	"javascript": {
+		"formatter": {
+			"quoteStyle": "double",
+			"semicolons": "always",
+			"trailingCommas": "es5"
+		}
+	}
 }

app/next.config.ts

@@ -1,18 +1,19 @@
 import type { NextConfig } from "next";
 
 const nextConfig: NextConfig = {
-  reactCompiler: true,
-  // Proxy tier2 streaming tool requests directly to the Python runner,
-  // bypassing the Next.js API route runtime which kills long-lived streams.
-  async rewrites() {
-    const runnerUrl = process.env.TOOL_RUNNER_URL || "http://localhost:9080";
-    return [
-      {
-        source: "/api/tools-stream/:toolId",
-        destination: `${runnerUrl}/api/tools/:toolId`,
-      },
-    ];
-  },
+
+	reactCompiler: true,
+	// Proxy tier2 streaming tool requests directly to the Python runner,
+	// bypassing the Next.js API route runtime which kills long-lived streams.
+	async rewrites() {
+		const runnerUrl = process.env.TOOL_RUNNER_URL || "http://localhost:9080";
+		return [
+			{
+				source: "/api/tools-stream/:toolId",
+				destination: `${runnerUrl}/api/tools/:toolId`,
+			},
+		];
+	},
 };
 
 export default nextConfig;

app/package.json

@@ -1,36 +1,36 @@
 {
-  "name": "oxtools",
-  "version": "1.0.0",
-  "private": true,
-  "scripts": {
-    "dev": "next dev",
-    "build": "next build",
-    "start": "next start",
-    "lint": "biome check",
-    "format": "biome format --write"
-  },
-  "dependencies": {
-    "@ansospace/ui": "^0.0.3",
-    "@icons-pack/react-simple-icons": "^13.13.0",
-    "idb-keyval": "^6.2.2",
-    "lucide-react": "^1.7.0",
-    "mermaid": "^11.14.0",
-    "next": "16.2.2",
-    "openai": "^6.33.0",
-    "react": "19.2.4",
-    "react-dom": "19.2.4",
-    "react-markdown": "^10.1.0",
-    "rehype-highlight": "^7.0.2",
-    "remark-gfm": "^4.0.1"
-  },
-  "devDependencies": {
-    "@biomejs/biome": "2.4.6",
-    "@tailwindcss/postcss": "^4",
-    "@types/node": "^20",
-    "@types/react": "^19",
-    "@types/react-dom": "^19",
-    "babel-plugin-react-compiler": "1.0.0",
-    "tailwindcss": "^4",
-    "typescript": "^5"
-  }
+	"name": "oxtools",
+	"version": "1.0.0",
+	"private": true,
+	"scripts": {
+		"dev": "next dev",
+		"build": "next build",
+		"start": "next start",
+		"lint": "biome check",
+		"format": "biome format --write"
+	},
+	"dependencies": {
+		"@ansospace/ui": "^0.0.3",
+		"@icons-pack/react-simple-icons": "^13.13.0",
+		"idb-keyval": "^6.2.2",
+		"lucide-react": "^1.7.0",
+		"mermaid": "^11.14.0",
+		"next": "16.2.2",
+		"openai": "^6.33.0",
+		"react": "19.2.4",
+		"react-dom": "19.2.4",
+		"react-markdown": "^10.1.0",
+		"rehype-highlight": "^7.0.2",
+		"remark-gfm": "^4.0.1"
+	},
+	"devDependencies": {
+		"@biomejs/biome": "2.4.6",
+		"@tailwindcss/postcss": "^4",
+		"@types/node": "^20",
+		"@types/react": "^19",
+		"@types/react-dom": "^19",
+		"babel-plugin-react-compiler": "1.0.0",
+		"tailwindcss": "^4",
+		"typescript": "^5"
+	}
 }

app/postcss.config.mjs

@@ -1,8 +1,8 @@
 /** @type {import('postcss-load-config').Config} */
 const config = {
-  plugins: {
-    "@tailwindcss/postcss": {},
-  },
+	plugins: {
+		"@tailwindcss/postcss": {},
+	},
 };
 
 export default config;

app/src/app/api/tools/[toolId]/route.ts

@@ -1,24 +1,12 @@
-import type { NextRequest } from "next/server";
-import { NextResponse } from "next/server";
-
+import { type NextRequest, NextResponse } from "next/server";
 import { createToolRoute } from "@/lib/create-tool-route";
 import { getToolById } from "@/lib/tools/registry";
-
-// Allow long-running tool executions (up to 5 min locally, 300s on Vercel)
+// Issue 5: maxDuration is set to 300s (5 minutes).
+// We align the AbortController timeout to match this limit exactly,
+// so that requests gracefully abort rather than hanging when Vercel kills the function.
 export const maxDuration = 300;
-// Force dynamic rendering — prevents Next.js from evaluating this route
-// at build time (which would fail without OXLO_API_KEY in CI)
 export const dynamic = "force-dynamic";
 
-/**
- * Dynamic API route for ALL tools.
- *
- * TIER 1: Dispatches to createToolRoute (calls Oxlo LLM API directly).
- * TIER 2: Proxies to the unified Python tool runner on port 9080.
- *
- * The unified runner handles ALL Python tools on ONE port.
- * Route: POST http://localhost:9080/api/tools/{toolId}
- */
 export async function POST(
 	request: NextRequest,
 	{ params }: { params: Promise<{ toolId: string }> }
@@ -33,31 +21,20 @@ export async function POST(
 		);
 	}
 
-	// --- Tier 2: Proxy to unified Python tool runner ---
 	if (tool.tier === "tier2") {
 		return proxyToToolRunner(request, toolId);
 	}
 
-	// --- Tier 1: Handle via LLM prompt (default) ---
 	const handler = createToolRoute({
 		requiredFields: tool.requiredFields,
 		buildSystemPrompt: tool.buildSystemPrompt,
 		buildUserPrompt: tool.buildUserPrompt,
 		defaultModel: tool.defaultModel,
 		errorMessage: `Failed to execute ${tool.name}`,
 	});
-
 	return handler(request);
 }
 
-/**
- * Proxy a request to the unified Python tool runner.
- *
- * ALL Tier 2 tools run on ONE service (port 9080) via:
- *   POST http://runner:9080/api/tools/{toolId}
- *
- * No more port-per-tool. One port, one container, unlimited tools.
- */
 async function proxyToToolRunner(request: NextRequest, toolId: string) {
 	const runnerUrl = process.env.TOOL_RUNNER_URL || "http://localhost:9080";
 	const targetUrl = `${runnerUrl}/api/tools/${toolId}`;
@@ -66,12 +43,12 @@ async function proxyToToolRunner(request: NextRequest, toolId: string) {
 		const body = await request.text();
 		const contentType = request.headers.get("content-type") || "application/json";
 
-		// 10 minute timeout — security scans with LLM retries can take 5-10 min
+		// 5 minute timeout — security scans with LLM retries can take 5 min
 		const response = await fetch(targetUrl, {
 			method: "POST",
 			headers: { "Content-Type": contentType },
 			body,
-			signal: AbortSignal.timeout(600_000),
+			signal: AbortSignal.timeout(300_000),
 		});
 
 		if (!response.ok) {