papilot 0.8.1 → 0.8.3

package/{vendor/latex-review → engine}/src/web/review.tsx

@@ -1,5 +1,7 @@
 import {
   Fragment,
+  lazy,
+  Suspense,
   useEffect,
   useMemo,
   useRef,
@@ -18,7 +20,11 @@ import type {
 } from "../types.ts";
 import { PdfStrip } from "./PdfStrip.tsx";
 import { ChangeMap } from "./ChangeMap.tsx";
+import { placeable, type PlacedChange } from "./paper.ts";
+// pdf.js loads only when the full paper is shown.
+const FullPaper = lazy(() => import("./FullPaper.tsx").then((module) => ({ default: module.FullPaper })));
 import { EFFECTS_EVENT, EFFECTS_STORAGE, readExpandedEffects } from "./effect-preferences.ts";
+import { readView, VIEW_EVENT, VIEW_STORAGE, type ReviewView } from "./view-preferences.ts";
 import { STATUS } from "./overview.tsx";
 import { language, locale, t, tn, withLanguage } from "./i18n.ts";
 import {
@@ -167,6 +173,22 @@ export function ReviewApp({
     scrollUntil = useRef(0),
     snapshot = useRef({ state, detail, busy, composer, current });
   const [shortcuts, setShortcuts] = useState(readShortcuts);
+  // The whole paper with the changes in place, or the changed sections only.
+  // A host may fix the view in the address (Papilot's manuscript chooses its
+  // PDF with its own switch); otherwise it follows the stored choice.
+  const [pinnedView] = useState<ReviewView | null>(() => {
+    const value = new URLSearchParams(location.search).get("view");
+    return value === "paper" || value === "changes" ? value : null;
+  });
+  const [view, setView] = useState<ReviewView>(() => pinnedView ?? readView());
+  useEffect(() => {
+    if (pinnedView) return;
+    const chosen = (event: Event) => setView((event as CustomEvent).detail === "changes" ? "changes" : "paper");
+    const stored = (event: StorageEvent) => { if (event.key === VIEW_STORAGE || event.key === null) setView(readView()); };
+    window.addEventListener(VIEW_EVENT, chosen);
+    window.addEventListener("storage", stored);
+    return () => { window.removeEventListener(VIEW_EVENT, chosen); window.removeEventListener("storage", stored); };
+  }, [pinnedView]);
   const [expandedEffects, setExpandedEffects] = useState(readExpandedEffects);
   // Per-edit disclosure is temporary and scoped to its round. Changing the
   // global preference resets those overrides without reloading the review.
@@ -195,6 +217,16 @@ export function ReviewApp({
       [detail],
     ),
     stops = useMemo(() => (detail ? buildStops(detail) : []), [detail]),
+    // The change blocks in the order and numbering of the changes view.
+    placed = useMemo(() => {
+      const list: PlacedChange[] = [];
+      if (!detail) return list;
+      for (const section of detail.sections)
+        for (const block of section.blocks)
+          if (block.kind === "change") list.push({ block, index: list.length });
+      for (const block of detail.loose) list.push({ block, index: list.length });
+      return list;
+    }, [detail]),
     // An edit's places show under the first change that includes it.
     firstStop = useMemo(() => {
       const found = new Map<string, number>();
@@ -1002,7 +1034,7 @@ export function ReviewApp({
           <p className="reference-problem reference-uncited">
             {uncited
               .map((m) => t("Cites {key}, which no bibliography entry defines.", { key: m.key }))
-              .join(language() === "zh-CN" ? "" : " ")}
+              .join(language().startsWith("zh") ? "" : " ")}
           </p>
         )}
         {bib.length > 0 && (
@@ -1094,6 +1126,32 @@ export function ReviewApp({
       </article>
     );
   }
+  const fullPaper = !pinnedView && Boolean(detail?.pdf) && placeable(placed),
+    paperView = fullPaper && view === "paper";
+  // A host shows its own switch while the whole paper can be shown.
+  useEffect(() => {
+    const root = document.documentElement;
+    if (fullPaper) root.dataset.fullPaper = "available";
+    else delete root.dataset.fullPaper;
+  }, [fullPaper]);
+  // Keep the selected change in view when the view changes, from here or the host.
+  const shownView = useRef(paperView);
+  useEffect(() => {
+    if (shownView.current === paperView) return;
+    shownView.current = paperView;
+    requestAnimationFrame(() => {
+      const article = paper.current?.querySelector<HTMLElement>(`[data-stop="${snapshot.current.current}"]`);
+      if (article) reveal(article);
+    });
+  }, [paperView]);
+  function chooseView(next: ReviewView) {
+    setView(next);
+    try {
+      localStorage.setItem(VIEW_STORAGE, next);
+    } catch {
+      /* The choice lasts for this page. */
+    }
+  }
   let index = 0;
   return (
     <>
@@ -1233,6 +1291,24 @@ export function ReviewApp({
               {t("Undo revert")}
             </button>
           )}
+          {fullPaper && (
+            <div className="view-switch" role="group" aria-label={t("Review view")}>
+              <button
+                className={`ghost${view === "paper" ? " active" : ""}`}
+                aria-pressed={view === "paper"}
+                onClick={() => chooseView("paper")}
+              >
+                PDF
+              </button>
+              <button
+                className={`ghost${view === "changes" ? " active" : ""}`}
+                aria-pressed={view === "changes"}
+                onClick={() => chooseView("changes")}
+              >
+                {t("Changes")}
+              </button>
+            </div>
+          )}
           {/* History is a toolbar action; its own row appears only while
               browsing earlier decisions. */}
           {detail && !history && !changes && (
@@ -1462,7 +1538,28 @@ export function ReviewApp({
               />
             )}
           <div className="document">
-            {detail ? (
+            {detail && paperView ? (
+              <>
+                <Suspense fallback={<div className="paper-status" role="status"><div className="pulse" /></div>}>
+                <FullPaper
+                  url={detail.pdf!}
+                  changes={placed}
+                  render={(block, at) => change(block, at, true)}
+                  scrollRoot={paper}
+                  onReady={() => {
+                    const article = paper.current?.querySelector<HTMLElement>(
+                      `[data-stop="${snapshot.current.current}"]`,
+                    );
+                    if (article) reveal(article);
+                  }}
+                  onUnavailable={() => chooseView("changes")}
+                />
+                </Suspense>
+                {!stops.length && !changes && (
+                  <p className="empty">{t("No changes to review.")}</p>
+                )}
+              </>
+            ) : detail ? (
               <>
                 {detail.sections.map((section) => (
                   <section className="part" key={section.node}>

package/{vendor/latex-review → engine}/src/web/style.css

@@ -247,6 +247,33 @@ kbd {
   border-color: var(--line);
 }
 
+/* Full paper or changes only, as one segmented control (a host such as
+   Papilot shows its own instead). */
+.view-switch {
+  display: inline-grid;
+  grid-template-columns: repeat(2, 1fr);
+  flex: none;
+  gap: 2px;
+  padding: 2px;
+  border: 1px solid var(--line);
+  border-radius: 9px;
+  background: var(--head);
+}
+.view-switch .ghost {
+  padding: 3px 10px;
+  border-radius: 6px;
+  font-weight: 500;
+}
+.view-switch .ghost:hover {
+  text-decoration: none;
+  border-color: transparent;
+}
+.view-switch .ghost.active {
+  color: var(--ink);
+  background: var(--paper);
+  box-shadow: 0 1px 2px rgb(0 0 0 / 0.06), 0 0 0 1px var(--line);
+}
+
 /* Document */
 #paper {
   overflow: auto;
@@ -399,6 +426,7 @@ kbd {
   mix-blend-mode: multiply;
 }
 .side::after {
+  content: attr(aria-label);
   position: absolute;
   top: 4px;
   right: 8px;
@@ -409,20 +437,9 @@ kbd {
   opacity: 0.9;
   pointer-events: none;
 }
-.side.before::after {
-  content: "before";
-}
 .side.after::after {
-  content: "after";
   color: var(--after-edge);
 }
-/* :where keeps the original weight, so later rules still hide the labels. */
-:where(:lang(zh-CN)) .side.before::after {
-  content: "修改前";
-}
-:where(:lang(zh-CN)) .side.after::after {
-  content: "修改后";
-}
 .side.after.identical::after,
 .change[data-resolved] .side::after {
   content: none;
@@ -908,6 +925,49 @@ kbd {
   text-align: center;
   padding: 90px 16px;
 }
+/* The whole paper: the new PDF's pages, opened where each change is shown
+   as in the changes view. A host's narrow-screen crop of the strips
+   (--pdf-crop-scale, --pdf-crop-shift) applies to the pages too. */
+.paper-page {
+  background: var(--pdf-page, var(--paper));
+  box-shadow: var(--shadow);
+  border-radius: 3px;
+  margin-bottom: 14px;
+}
+.paper-context {
+  line-height: 0;
+}
+.paper-slice {
+  position: relative;
+  width: 100%;
+}
+.paper-slice canvas {
+  position: absolute;
+  inset: 0;
+  width: 100%;
+  height: 100%;
+  filter: var(--pdf-filter, none);
+}
+@media (max-width: 680px) {
+  .paper-slice {
+    width: calc(100% * var(--pdf-crop-scale, 1));
+    margin-left: calc(-100% * var(--pdf-crop-scale, 1) * var(--pdf-crop-shift, 0));
+  }
+  .paper-context {
+    overflow-x: clip;
+  }
+}
+.paper-status {
+  text-align: center;
+  padding: 90px 16px;
+  color: var(--muted);
+}
+.paper-status p {
+  margin: 6px 0 14px;
+}
+.paper-clock {
+  font-variant-numeric: tabular-nums;
+}
 .waiting p {
   margin: 6px 0;
 }

package/engine/src/web/view-preferences.ts

@@ -0,0 +1,13 @@
+// Whole paper or changed sections only. A host such as Papilot shows its own
+// switch: it stores the choice and tells this frame; storage events also
+// update other tabs.
+export const VIEW_STORAGE = "latex-review:view";
+export const VIEW_EVENT = "latex-review:view-changed";
+export type ReviewView = "paper" | "changes";
+export function readView(): ReviewView {
+  try {
+    return localStorage.getItem(VIEW_STORAGE) === "changes" ? "changes" : "paper";
+  } catch {
+    return "paper";
+  }
+}

package/{vendor/latex-review → engine}/src/web/zh-CN.ts

@@ -66,6 +66,9 @@ export const zhCN: Record<string, string> = {
   "Review history": "审阅历史",
   "Keyboard shortcuts": "键盘快捷键",
   "Review view": "审阅视图",
+  "Changes": "改动",
+  "Preparing the full PDF…": "正在准备完整 PDF…",
+  "The full PDF could not be loaded.": "无法加载完整 PDF。",
   "Undo {action} · {changes}": "撤销{action} · {changes}",
   "Change {list}": "修改 {list}",
   "Changes {list}": "修改 {list}",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "papilot",
-  "version": "0.8.1",
+  "version": "0.8.3",
   "description": "A Cursor-like workspace for your papers: annotate and review PDFs directly without ever touching LaTeX.",
   "keywords": [
     "latex",
@@ -48,34 +48,40 @@
     "scripts/demo.ts",
     "scripts/service.ts",
     "examples/papilot-paper",
-    "vendor/latex-review/src",
-    "vendor/latex-review/dist",
+    "engine/src",
+    "engine/dist",
+    "engine/package.json",
+    "engine/package-lock.json",
+    "engine/LICENSE",
     "vendor/latex-review/package.json",
-    "vendor/latex-review/package-lock.json",
-    "vendor/latex-review/LICENSE"
+    "vendor/latex-review/package-lock.json"
   ],
   "engines": {
     "node": ">=22.22.0"
   },
+  "desktopShell": 2,
   "scripts": {
     "dev": "tsx watch src/server/main.ts",
     "start": "tsx src/server/main.ts",
     "build": "tsc --noEmit && vite build",
     "typecheck": "tsc --noEmit && tsc -p desktop/tsconfig.json --noEmit",
     "test": "tsx --test tests/*.test.ts",
-    "setup:review": "npm --prefix vendor/latex-review ci && npm --prefix vendor/latex-review run build",
+    "setup:review": "npm --prefix engine ci && npm --prefix engine run build",
     "demo": "tsx scripts/demo.ts",
     "doctor": "node bin/papilot.mjs doctor",
     "service:install": "tsx scripts/service.ts install",
     "service:stop": "tsx scripts/service.ts stop",
-    "test:ui": "npm run build && node scripts/check-ui.mjs && node scripts/check-appearance.mjs && node scripts/check-updates.mjs && node scripts/check-papers.mjs && node scripts/check-imports.mjs && node scripts/check-compilation.mjs && node scripts/check-build-problems.mjs && node scripts/check-review-notes.mjs && node --import tsx scripts/check-note-groups.mjs && node scripts/check-note-selection.mjs && node scripts/check-ai-review.mjs && node --import tsx scripts/check-note-results.mjs && node --import tsx scripts/check-review-flow.mjs && node scripts/check-skills.mjs && node --import tsx scripts/check-skill-library.mjs && node scripts/check-attachments.mjs && node --import tsx scripts/check-accounts.mjs",
+    "test:ui": "npm run build && node scripts/check-ui.mjs && node scripts/check-task-model.mjs && node scripts/check-appearance.mjs && node scripts/check-updates.mjs && node scripts/check-papers.mjs && node scripts/check-imports.mjs && node scripts/check-compilation.mjs && node scripts/check-build-problems.mjs && node scripts/check-review-notes.mjs && node --import tsx scripts/check-note-groups.mjs && node scripts/check-note-selection.mjs && node scripts/check-ai-review.mjs && node --import tsx scripts/check-note-results.mjs && node --import tsx scripts/check-review-flow.mjs && node scripts/check-skills.mjs && node --import tsx scripts/check-skill-library.mjs && node scripts/check-attachments.mjs && node --import tsx scripts/check-accounts.mjs",
     "site": "node scripts/build-site.mjs",
     "test:site": "npm run site && node scripts/check-site.mjs",
     "prepack": "npm run build && npm run setup:review",
     "desktop:compile": "tsc -p desktop/tsconfig.json",
     "desktop": "npm run setup:review && npm run build && npm run desktop:compile && node scripts/desktop.mjs dev",
     "desktop:build": "npm run setup:review && npm run build && npm run desktop:compile && node scripts/desktop.mjs package",
-    "test:desktop": "node scripts/check-desktop.mjs"
+    "test:desktop": "node scripts/check-desktop.mjs",
+    "i18n": "tsx scripts/i18n.ts",
+    "i18n:check-app": "tsx scripts/i18n/app-locales.ts",
+    "test:locales": "npm run build && npm --prefix engine run build && node --import tsx scripts/check-locales.mjs"
   },
   "dependencies": {
     "@fastify/compress": "^9.2.2",
@@ -109,6 +115,9 @@
     "remark-gfm": "^4.0.1",
     "remark-math": "^6.0.0",
     "typescript": "^7.0.2",
-    "vite": "^8.3.5"
+    "vite": "^8.3.5",
+    "parse5": "^8.0.1",
+    "@babel/parser": "^8.0.10",
+    "@babel/types": "^8.0.10"
   }
 }

package/src/conversation.ts

@@ -33,7 +33,7 @@ export function skillPrompt(skill: { name: string; body: string; folder?: string
 }
 
 interface FeedbackItem { kind?: string; round?: number | null; text?: string; changes?: (string | { file?: string; lines?: [number, number] })[] }
-// The review engine's feedback (see vendor/latex-review FeedbackItem) as the chat shows it.
+// The review engine's feedback (see engine FeedbackItem) as the chat shows it.
 export function summarizeFeedback(feedback: { feedback?: FeedbackItem[]; build_error?: unknown }): FeedbackSummary | undefined {
   const items = feedback.feedback || [];
   if (!items.length && !feedback.build_error) return undefined;

package/src/server/acp.ts

@@ -218,7 +218,8 @@ export class AcpAgent extends Adapter {
       const previous = this.catalog.find(entry => entry.model === option.value);
       const entry = { model: option.value, displayName: option.name, isDefault: option.value === this.defaultModel,
         supportedReasoningEfforts: option.value === current ? levels.map(choice => ({ reasoningEffort: choice.value })) : previous?.supportedReasoningEfforts || [],
-        defaultReasoningEffort: option.value === current ? level : previous?.defaultReasoningEffort || null };
+        // A turn's explicit effort must not become the catalog's default.
+        defaultReasoningEffort: previous?.supportedReasoningEfforts.length ? previous.defaultReasoningEffort : option.value === current ? level : null };
       if (previous) Object.assign(previous, entry); else this.catalog.push(entry);
     }
   }
@@ -239,10 +240,13 @@ export class AcpAgent extends Adapter {
     };
     try {
       const session = await this.session(turn.threadId, thread.cwd, thread.model);
-      if (effort) {
-        const option = session.configOptions?.find(option => option.category === 'thought_level' || option.id === 'thinking');
-        if (!option || !choices(option).some(choice => choice.value === effort)) throw new Error(`${this.name} does not support reasoning effort ${effort} for this model.`);
-        await this.rpc('session/set_config_option', { sessionId: turn.threadId, configId: option.id, value: effort });
+      const option = session.configOptions?.find(option => option.category === 'thought_level' || option.id === 'thinking');
+      const defaultEffort = choices(option).some(choice => choice.value === 'default') ? 'default' : this.catalog.find(model => model.model === thread.model)?.defaultReasoningEffort;
+      const chosenEffort = effort || defaultEffort;
+      if (chosenEffort) {
+        if (!option || !choices(option).some(choice => choice.value === chosenEffort)) throw new Error(`${this.name} does not support reasoning effort ${chosenEffort} for this model.`);
+        const result = await this.rpc('session/set_config_option', { sessionId: turn.threadId, configId: option.id, value: chosenEffort });
+        if (result?.configOptions) session.configOptions = result.configOptions;
       }
       const prompt: any[] = [{ type: 'text', text: thread.started ? text : `${thread.instructions}\n\n---\n\n${text}` }];
       if (images.length && !this.capabilities.promptCapabilities?.image) throw new Error(`This ${this.name} version does not support image input. Update it, or send the images to another agent.`);

package/src/server/agents.ts

@@ -8,7 +8,7 @@ import { createRequire } from 'node:module';
 import os from 'node:os';
 import path from 'node:path';
 import { pathToFileURL } from 'node:url';
-import type { AccountInfo, ModelInfo, Options, PermissionResult, Query, SDKMessage, SDKRateLimitInfo, SDKUserMessage } from '@anthropic-ai/claude-agent-sdk';
+import type { AccountInfo, ModelInfo, Options, PermissionResult, SDKMessage, SDKRateLimitInfo, SDKUserMessage } from '@anthropic-ai/claude-agent-sdk';
 import { AGENT_NAMES } from '../shared.ts';
 import { DATA, ROOT } from './paths.ts';
 import { imageType } from './attachments.ts';
@@ -79,13 +79,15 @@ export abstract class Adapter extends EventEmitter implements AgentRuntime {
     if (method === 'turn/start') {
       const thread = this.threads.get(String(params.threadId));
       if (!thread) throw new Error('Start or resume the thread before starting a turn.');
+      if (params.model !== undefined) thread.model = params.model || null;
       const text = (params.input || []).filter((part: { text?: unknown }) => typeof part.text === 'string').map((part: { text: string }) => part.text).join('\n');
       const images = (params.input || []).filter((part: { type?: string; path?: string }) => part.type === 'localImage' && typeof part.path === 'string').map((part: { path: string }) => part.path);
       const turn: Turn = { id: randomUUID(), threadId: String(params.threadId), clientId: params.clientUserMessageId, status: 'inProgress', stop: () => {} };
       turn.stop = () => this.finish(turn, 'interrupted');
       this.turns.set(turn.id, turn);
       // Papilot records the turn ID before the first event arrives.
-      setImmediate(() => { if (turn.status === 'inProgress') this.run(turn, { ...thread, cwd: params.cwd || thread.cwd }, text, params.effort || null, images); });
+      const settings = { ...thread, cwd: params.cwd || thread.cwd };
+      setImmediate(() => { if (turn.status === 'inProgress') this.run(turn, settings, text, params.effort || null, images); });
       return { turn: { id: turn.id } };
     }
     if (method === 'turn/interrupt') { this.turns.get(String(params.turnId))?.stop(); return {}; }

package/src/server/app.ts

@@ -263,6 +263,13 @@ export async function buildApp(options: AppOptions) {
     const task = store.task(request.params.id);
     return workspace.commitMessage(task.projectId, task.review?.files || [], { taskId: task.id, agent: task.agent, fresh: (request.body as any)?.fresh === true });
   });
+  app.post<{ Params: { id: string } }>('/api/tasks/:id/model', task, request => {
+    const b = (request.body || {}) as Record<string, unknown>;
+    if (Object.keys(b).some(key => !['model', 'effort', 'serviceTier'].includes(key))) throw new Error('Only the model and its settings can be changed.');
+    if (b.model !== undefined && (typeof b.model !== 'string' || b.model.length > 200)) throw new Error('Invalid model.');
+    if (b.effort !== undefined && b.effort !== '' && !effort(b.effort)) throw new Error('Invalid reasoning effort.');
+    return workspace.updateTaskModel(request.params.id, { ...(b.model !== undefined ? { model: b.model as string } : {}), ...(b.effort !== undefined ? { effort: b.effort as string } : {}), ...(b.serviceTier !== undefined ? { serviceTier: string(b.serviceTier, 'Service tier', 64) } : {}) });
+  });
   app.post<{ Params: { id: string } }>('/api/tasks/:id/rename', task, request => workspace.renameTask(request.params.id, string((request.body as any).title, 'Task title', 120)));
   app.post<{ Params: { id: string } }>('/api/tasks/:id/title', task, request => workspace.requestTaskTitle(request.params.id));
   app.post<{ Params: { id: string } }>('/api/tasks/:id/archive', task, request => workspace.archiveTask(request.params.id, (request.body as any)?.archived, (request.body as any)?.finished === true));
@@ -466,6 +473,10 @@ export async function buildApp(options: AppOptions) {
     if (typeof b?.id !== 'string' || typeof b?.confirmed !== 'boolean') throw new Error('Choose an entry to confirm.');
     return changes.confirmReference(project.id, project.path, Number(request.params.number), b.id, b.confirmed);
   });
+  app.get<{ Params: { projectId: string; number: string } }>('/review/:projectId/changes/pdf/:number', changesAccess, (request, reply) => {
+    const pdf = changes.pdf(store.project(request.params.projectId).id, Number(request.params.number));
+    return reply.type('application/pdf').sendFile(path.basename(pdf), path.dirname(pdf), { cacheControl: false });
+  });
   app.get<{ Params: { projectId: string; number: string; file: string } }>('/review/:projectId/changes/files/:number/:file', changesAccess, (request, reply) => {
     const file = changes.file(store.project(request.params.projectId).id, Number(request.params.number), request.params.file);
     return reply.header('Cache-Control', 'private, max-age=31536000, immutable').type(file.endsWith('.svg') ? 'image/svg+xml' : 'image/png').send(fs.createReadStream(file));
@@ -484,6 +495,14 @@ export async function buildApp(options: AppOptions) {
     return serialGit(project.id, () => workspace.exclusiveCommit(project.id, () => changes.undo(project.id, project.path, csrfFor(request.viewer!), id)));
   });
 
+  // The complete PDF of a round's new version, for the review's full paper.
+  // A round may compile only the changed chapters; this compiles them all once.
+  app.get<{ Params: { projectId: string; name: string; round: string } }>('/review/:projectId/pdf/:name/:round', { config: { access: { project: 'projectId' } } }, async (request, reply) => {
+    const project = store.project(request.params.projectId), name = request.params.name, round = Number(request.params.round);
+    if (!/^[a-z0-9-]+$/.test(name) || !Number.isSafeInteger(round) || round < 1) return reply.code(404).send({ error: 'Review resource not found.' });
+    const pdf = await fullPaperPdf(project.path, name, round, 'after');
+    return reply.type('application/pdf').sendFile(path.basename(pdf), path.dirname(pdf), { cacheControl: false });
+  });
   app.route<{ Params: { projectId: string; '*': string } }>({
     // The review page posts with the review engine's token rather than Papilot's.
     method: ['GET', 'POST'], url: '/review/:projectId/*', config: { access: { project: 'projectId' }, csrf: false }, handler: async (request, reply) => {
@@ -521,6 +540,10 @@ export async function buildApp(options: AppOptions) {
               await workspace.refresh(task.id);
             }
           }
+          // A round's new version as one complete PDF, which Papilot compiles,
+          // lets the review show the changes in the whole paper.
+          const round = request.method === 'GET' && response.ok ? /^\/api\/tasks\/([a-z0-9-]+)\/rounds\/(\d+)$/.exec(route) : null;
+          if (round) (value as { pdf?: string }).pdf = `${prefix}/pdf/${round[1]}/${round[2]}`;
           return rewriteReviewJson(value, prefix);
         }
         if (reviewTask && writesFiles) throw new Error('The review engine returned an unexpected response. Select Check status before continuing.');

package/src/server/changes.ts

@@ -17,7 +17,7 @@ type Op = ['equal' | 'replace' | 'insert' | 'delete', number, number, number, nu
 export interface Unit { id: string; file: string; status: 'added' | 'deleted' | 'modified'; kind: string; whole_file: boolean; ops: Op[]; old_start: number; old_end: number; new_start: number; new_end: number; old_text: string; new_text: string; truncated: boolean }
 type Decision = 'pending' | 'accepted' | 'rejected' | 'commented';
 export type ChangeAction = 'stage' | 'unstage' | 'revert';
-interface Round { number: number; before: string; after: string; units: Unit[]; folder: string; warnings: string[]; decisions: Record<string, Decision> }
+interface Round { number: number; before: string; after: string; units: Unit[]; folder: string; warnings: string[]; decisions: Record<string, Decision>; pdf?: string }
 interface Job { key: string; phase: 'build' | 'render'; status: 'running' | 'failed'; created: number; error?: string; details?: string[]; log?: string }
 interface Snapshot { before: string | null; after: string }
 interface Entry { mode: string; sha: string }
@@ -139,7 +139,7 @@ export class Changes {
       const { Builder, BuildError, Repo, loadProject, withUnits, render } = await engine();
       const project = loadProject(root), repo = new Repo(root), number = comparison.next++;
       const units = await this.units(projectId, root, snapshot), folder = path.join(this.dir, projectId, String(number));
-      let warnings: string[] = [];
+      let warnings: string[] = [], pdf: string | undefined;
       fs.rmSync(folder, { recursive: true, force: true }); fs.mkdirSync(folder, { recursive: true });
       if (units.length) {
         const builder = new Builder(project, repo);
@@ -154,8 +154,9 @@ export class Changes {
         await render(folder, { main: project.main, source_dir: project.source_dir, figures: project.figures, old, new: next, units },
           (subset: Unit[]) => builder.build(withUnits(repo, snapshot.before!, snapshot.after, subset), ['full']));
         warnings = next.warnings.filter((warning: string) => !old.warnings.includes(warning));
+        pdf = next.pdf;
       } else fs.writeFileSync(path.join(folder, 'round.json'), JSON.stringify({ units: {}, sections: [], loose: [] }));
-      comparison.rounds.push({ number, before: snapshot.before!, after: snapshot.after, units, folder, warnings, decisions: {} });
+      comparison.rounds.push({ number, before: snapshot.before!, after: snapshot.after, units, folder, warnings, decisions: {}, pdf });
       // The viewer may still show the round before; older ones go.
       for (const old of comparison.rounds.splice(0, Math.max(0, comparison.rounds.length - 3))) fs.rmSync(old.folder, { recursive: true, force: true });
       if (comparison.job === job) comparison.job = null;
@@ -203,6 +204,8 @@ export class Changes {
       task: 'changes', round: round.number, before: round.before, after: round.after,
       units: round.units.map(unit => ({ ...unit, ...render.units[unit.id] })), sections: render.sections, loose: render.loose,
       warnings: round.warnings, files: `/review/${projectId}/changes/files/${round.number}/`,
+      // The working tree compiles in full, so its PDF shows the whole paper.
+      ...(round.pdf ? { pdf: `/review/${projectId}/changes/pdf/${round.number}` } : {}),
       ...(render.references ? { references: render.references } : {}),
     };
   }
@@ -238,6 +241,12 @@ export class Changes {
     return { places: await more(round.folder, unit) };
   }
 
+  pdf(projectId: string, number: number) {
+    const file = this.comparison(projectId).rounds.find(item => item.number === number)?.pdf;
+    if (!file || !fs.existsSync(file)) throw Object.assign(new Error('This comparison is out of date. Refresh the page.'), { statusCode: 404 });
+    return file;
+  }
+
   file(projectId: string, number: number, name: string) {
     const round = this.comparison(projectId).rounds.find(item => item.number === number);
     const file = round && /^[\w.-]+\.(svg|png)$/.test(name) ? path.join(round.folder, name) : '';