newpr 1.0.8 → 1.0.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "newpr",
-  "version": "1.0.8",
+  "version": "1.0.11",
   "description": "AI-powered large PR review tool - understand PRs with 1000+ lines of changes",
   "module": "src/cli/index.ts",
   "type": "module",

package/src/web/client/components/Markdown.tsx

@@ -94,7 +94,7 @@ function MediaEmbed({ src }: { src: string }) {
 	);
 }
 
-const ANCHOR_RE = /\[\[(group|file):([^\]]+)\]\]/g;
+const ANCHOR_RE = /\[\[(group|file):(.*?)\]\]/g;
 
 const BOLD_CJK_RE = /\*\*(.+?)\*\*/g;
 
@@ -131,16 +131,14 @@ function replaceLineAnchors(text: string): string {
 		result += text.slice(i, start);
 
 		const idStart = start + OPEN.length;
-		const closeBracket = text.indexOf("]", idStart);
-		if (closeBracket === -1) {
+		const closeBrackets = text.indexOf("]]", idStart);
+		if (closeBrackets === -1) {
 			result += text.slice(start);
 			break;
 		}
 
-		const id = text.slice(idStart, closeBracket);
-		let afterClose = closeBracket + 1;
-
-		if (text[afterClose] === "]") afterClose++;
+		const id = text.slice(idStart, closeBrackets);
+		let afterClose = closeBrackets + 2;
 
 		let label: string | null = null;
 		if (text[afterClose] === "(") {
@@ -157,8 +155,6 @@ function replaceLineAnchors(text: string): string {
 			}
 		}
 
-		if (text[afterClose] === "]") afterClose++;
-
 		const encoded = encodeURIComponent(id);
 		if (label) {
 			result += `<span data-line-ref="${encoded}">${inlineMarkdownToHtml(label)}</span>`;

package/src/web/server/routes.ts

@@ -953,9 +953,22 @@ Before posting an inline comment, ALWAYS call \`get_file_diff\` first to find th
 			const encoder = new TextEncoder();
 			const stream = new ReadableStream({
 				async start(controller) {
+					let closed = false;
 					const send = (eventType: string, data: string) => {
+						if (closed) return;
 						controller.enqueue(encoder.encode(`event: ${eventType}\ndata: ${data}\n\n`));
 					};
+					const safeClose = () => {
+						if (closed) return;
+						closed = true;
+						clearInterval(heartbeat);
+						setTimeout(() => { try { controller.close(); } catch {} }, 50);
+					};
+					const heartbeat = setInterval(() => {
+						if (closed) return;
+						try { controller.enqueue(encoder.encode(":keepalive\n\n")); } catch { safeClose(); }
+					}, 15_000);
+
 					try {
 						if (config.openrouter_api_key) {
 							await chatWithTools(
@@ -999,7 +1012,7 @@ Before posting an inline comment, ALWAYS call \`get_file_diff\` first to find th
 					} catch (err) {
 						send("chat_error", JSON.stringify({ message: err instanceof Error ? err.message : String(err) }));
 					} finally {
-						controller.close();
+						safeClose();
 					}
 				},
 			});
@@ -1424,9 +1437,21 @@ Before posting an inline comment, ALWAYS call \`get_file_diff\` first to find th
 			const encoder = new TextEncoder();
 			const stream = new ReadableStream({
 				async start(controller) {
+					let closed = false;
 					const send = (eventType: string, data: string) => {
+						if (closed) return;
 						controller.enqueue(encoder.encode(`event: ${eventType}\ndata: ${data}\n\n`));
 					};
+					const safeClose = () => {
+						if (closed) return;
+						closed = true;
+						clearInterval(heartbeat);
+						setTimeout(() => { try { controller.close(); } catch {} }, 50);
+					};
+					const heartbeat = setInterval(() => {
+						if (closed) return;
+						try { controller.enqueue(encoder.encode(":keepalive\n\n")); } catch { safeClose(); }
+					}, 15_000);
 
 					let fullText = "";
 					const collectedToolCalls: ChatToolCall[] = [];
@@ -1526,7 +1551,7 @@ Before posting an inline comment, ALWAYS call \`get_file_diff\` first to find th
 					} catch (err) {
 						send("chat_error", JSON.stringify({ message: err instanceof Error ? err.message : String(err) }));
 					} finally {
-						controller.close();
+						safeClose();
 					}
 				},
 			});

package/src/web/server.ts

@@ -17,17 +17,30 @@ interface WebServerOptions {
 
 function getCssPaths() {
 	const webDir = import.meta.dir;
-	const packageRoot = join(webDir, "..", "..");
+
+	let bin: string;
+	try {
+		const resolved = import.meta.resolve("@tailwindcss/cli/package.json");
+		const cliDir = resolved.replace(/^file:\/\//, "").replace(/\/package\.json$/, "");
+		bin = join(cliDir, "dist", "index.mjs");
+	} catch {
+		const packageRoot = join(webDir, "..", "..");
+		bin = join(packageRoot, "node_modules", ".bin", "tailwindcss");
+	}
+
 	return {
 		input: join(webDir, "styles", "globals.css"),
 		output: join(webDir, "styles", "built.css"),
-		bin: join(packageRoot, "node_modules", ".bin", "tailwindcss"),
+		bin,
 	};
 }
 
 async function buildCss(bin: string, input: string, output: string): Promise<void> {
+	const args = bin.endsWith(".mjs") || bin.endsWith(".js")
+		? ["bun", bin, "-i", input, "-o", output, "--minify"]
+		: [bin, "-i", input, "-o", output, "--minify"];
 	const proc = Bun.spawn(
-		[bin, "-i", input, "-o", output, "--minify"],
+		args,
 		{ cwd: process.cwd(), stderr: "pipe", stdout: "pipe" },
 	);
 	const exitCode = await proc.exited;