newpr 1.0.3 → 1.0.5

package/package.json

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

package/src/stack/publish.ts

@@ -46,9 +46,9 @@ export async function publishStack(input: PublishInput): Promise<StackPublishRes
 			? `[${order}/${total}] ${gc.pr_title}`
 			: `[Stack ${order}/${total}] ${gc.group_id}`;
 
-		const body = buildPrBody(gc.group_id, order, total, exec_result, pr_meta);
+		const placeholder = buildPlaceholderBody(gc.group_id, order, total, pr_meta);
 
-		const prResult = await Bun.$`gh pr create --repo ${ghRepo} --base ${prBase} --head ${gc.branch_name} --title ${title} --body ${body} --draft`.quiet().nothrow();
+		const prResult = await Bun.$`gh pr create --repo ${ghRepo} --base ${prBase} --head ${gc.branch_name} --title ${title} --body ${placeholder} --draft`.quiet().nothrow();
 
 		if (prResult.exitCode === 0) {
 			const prUrl = prResult.stdout.toString().trim();
@@ -68,29 +68,84 @@ export async function publishStack(input: PublishInput): Promise<StackPublishRes
 		}
 	}
 
+	await updatePrBodies(ghRepo, prs, pr_meta);
+
 	return { branches, prs };
 }
 
-function buildPrBody(
+async function updatePrBodies(ghRepo: string, prs: PrInfo[], prMeta: PrMeta): Promise<void> {
+	if (prs.length === 0) return;
+
+	for (let i = 0; i < prs.length; i++) {
+		const pr = prs[i]!;
+		const body = buildFullBody(pr, i, prs, prMeta);
+
+		await Bun.$`gh pr edit ${pr.number} --repo ${ghRepo} --body ${body}`.quiet().nothrow();
+	}
+}
+
+function buildPlaceholderBody(
 	groupId: string,
 	order: number,
 	total: number,
-	_execResult: StackExecResult,
 	prMeta: PrMeta,
 ): string {
-	const prevPr = order > 1 ? `Previous: Stack ${order - 1}/${total}` : "Previous: (base branch)";
-	const nextPr = order < total ? `Next: Stack ${order + 1}/${total}` : "Next: (top of stack)";
-
-	const lines = [
-		`> This is part of a stacked PR chain created by [newpr](${prMeta.pr_url})`,
-		`>`,
-		`> **Stack order**: ${order}/${total}`,
-		`> **${prevPr}** | **${nextPr}**`,
+	return [
+		`> This is part of a stacked PR chain created by [newpr](https://github.com/jiwonMe/newpr).`,
+		`> Stack order: ${order}/${total} — body will be updated with links shortly.`,
 		``,
 		`## ${groupId}`,
 		``,
 		`*From PR #${prMeta.pr_number}: ${prMeta.pr_title}*`,
-	];
+	].join("\n");
+}
+
+function buildFullBody(
+	current: PrInfo,
+	index: number,
+	allPrs: PrInfo[],
+	prMeta: PrMeta,
+): string {
+	const total = allPrs.length;
+	const order = index + 1;
+
+	const stackTable = allPrs.map((pr, i) => {
+		const num = i + 1;
+		const isCurrent = i === index;
+		const marker = isCurrent ? "👉" : statusEmoji(i, index);
+		const link = `[#${pr.number}](${pr.url})`;
+		const titleText = pr.title.replace(/^\[\d+\/\d+\]\s*/, "");
+		return `| ${marker} | ${num}/${total} | ${link} | ${titleText} |`;
+	}).join("\n");
+
+	const prev = index > 0
+		? `⬅️ Previous: [#${allPrs[index - 1]!.number}](${allPrs[index - 1]!.url})`
+		: "⬅️ Previous: base branch";
+	const next = index < total - 1
+		? `➡️ Next: [#${allPrs[index + 1]!.number}](${allPrs[index + 1]!.url})`
+		: "➡️ Next: top of stack";
+
+	return [
+		`> **Stack ${order}/${total}** — This PR is part of a stacked PR chain created by [newpr](https://github.com/jiwonMe/newpr).`,
+		`> Source: #${prMeta.pr_number} ${prMeta.pr_title}`,
+		``,
+		`### 📚 Stack Navigation`,
+		``,
+		`| | Order | PR | Title |`,
+		`|---|---|---|---|`,
+		stackTable,
+		``,
+		`${prev} | ${next}`,
+		``,
+		`---`,
+		``,
+		`## ${current.group_id}`,
+		``,
+		`*From PR [#${prMeta.pr_number}](${prMeta.pr_url}): ${prMeta.pr_title}*`,
+	].join("\n");
+}
 
-	return lines.join("\n");
+function statusEmoji(prIndex: number, currentIndex: number): string {
+	if (prIndex < currentIndex) return "✅";
+	return "⬜";
 }

package/src/web/client/App.tsx

@@ -202,6 +202,7 @@ export function App() {
 					onTabChange={handleTabChange}
 					onReanalyze={(prUrl: string) => { analysis.start(prUrl); }}
 					enabledPlugins={features.enabledPlugins}
+					onTrackAnalysis={bgAnalyses.track}
 				/>
 			)}
 			{analysis.phase === "error" && (

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

@@ -51,6 +51,7 @@ export function ResultsScreen({
 	onTabChange,
 	onReanalyze,
 	enabledPlugins,
+	onTrackAnalysis,
 }: {
 	data: NewprOutput;
 	onBack: () => void;
@@ -61,6 +62,7 @@ export function ResultsScreen({
 	onTabChange?: (tab: string) => void;
 	onReanalyze?: (prUrl: string) => void;
 	enabledPlugins?: string[];
+	onTrackAnalysis?: (analysisSessionId: string, prUrl: string) => void;
 }) {
 	const { meta, summary } = data;
 	const [tab, setTab] = useState<TabValue>(getInitialTab);
@@ -274,7 +276,7 @@ export function ResultsScreen({
 				/>
 			</TabsContent>
 			<TabsContent value="stack">
-				<StackPanel sessionId={sessionId} />
+				<StackPanel sessionId={sessionId} onTrackAnalysis={onTrackAnalysis} />
 			</TabsContent>
 			<TabsContent value="slides">
 				<SlidesPanel data={data} sessionId={sessionId} />

package/src/web/client/hooks/useStack.ts

@@ -133,7 +133,11 @@ function applyServerState(server: ServerStackState): Partial<StackState> {
 	};
 }
 
-export function useStack(sessionId: string | null | undefined) {
+interface UseStackOptions {
+	onTrackAnalysis?: (analysisSessionId: string, prUrl: string) => void;
+}
+
+export function useStack(sessionId: string | null | undefined, options?: UseStackOptions) {
 	const [state, setState] = useState<StackState>({
 		phase: "idle",
 		error: null,
@@ -212,8 +216,9 @@ export function useStack(sessionId: string | null | undefined) {
 		});
 
 		es.onerror = () => {
-			es.close();
-			eventSourceRef.current = null;
+			if (es.readyState === EventSource.CLOSED) {
+				eventSourceRef.current = null;
+			}
 		};
 	}, []);
 
@@ -253,11 +258,29 @@ export function useStack(sessionId: string | null | undefined) {
 			const data = await res.json();
 			if (!res.ok) throw new Error(data.error ?? "Publishing failed");
 
+			const publishResult = data.publish_result as PublishResultData;
+
 			setState((s) => ({
 				...s,
 				phase: "done",
-				publishResult: data.publish_result,
+				publishResult,
 			}));
+
+			if (options?.onTrackAnalysis && publishResult?.prs?.length > 0) {
+				for (const pr of publishResult.prs) {
+					try {
+						const analysisRes = await fetch("/api/analysis", {
+							method: "POST",
+							headers: { "Content-Type": "application/json" },
+							body: JSON.stringify({ pr: pr.url }),
+						});
+						const analysisData = await analysisRes.json() as { sessionId?: string };
+						if (analysisData.sessionId) {
+							options.onTrackAnalysis(analysisData.sessionId, pr.url);
+						}
+					} catch {}
+				}
+			}
 		} catch (err) {
 			setState((s) => ({
 				...s,
@@ -265,7 +288,7 @@ export function useStack(sessionId: string | null | undefined) {
 				error: err instanceof Error ? err.message : String(err),
 			}));
 		}
-	}, [sessionId]);
+	}, [sessionId, options]);
 
 	const reset = useCallback(() => {
 		eventSourceRef.current?.close();

package/src/web/client/panels/StackPanel.tsx

@@ -68,8 +68,13 @@ function PipelineTimeline({ phase }: { phase: StackPhase }) {
 	);
 }
 
-export function StackPanel({ sessionId }: { sessionId?: string | null }) {
-	const stack = useStack(sessionId);
+interface StackPanelProps {
+	sessionId?: string | null;
+	onTrackAnalysis?: (analysisSessionId: string, prUrl: string) => void;
+}
+
+export function StackPanel({ sessionId, onTrackAnalysis }: StackPanelProps) {
+	const stack = useStack(sessionId, { onTrackAnalysis });
 
 	if (stack.phase === "idle") {
 		return (