dsh-diff-approval 0.29.0 → 0.30.1

package/lib/types/client/discussion.d.ts

@@ -7,6 +7,7 @@
  *
  * @module dsh-diff-approval/client/discussion
  */
+import type { CommentQuoteLine } from '../types.ts';
 /** One row range in the current diff model (inclusive, 0-based row indices). */
 export interface DiscussionAnchor {
     /** First row of the range, in the model this block is currently drawn against. */
@@ -97,23 +98,8 @@ export interface Discussion {
     lost?: boolean;
     /** The answer being streamed for the question in flight. */
     reply?: string;
-    /**
-     * Where the session's transcript stood when the last question was sent — its node count.
-     *
-     * Kept with the block rather than only in the panel's refs, so a question still waiting
-     * for its answer can be picked up again after the panel is closed and reopened: the search
-     * for our own prompt starts here, which is what keeps an older, identical-looking prompt
-     * from being mistaken for it.
-     */
-    baseline?: number;
     /** The turn for the last question is running. */
     asking?: boolean;
-    /**
-     * The question is sent but still behind another turn of the session, so nothing
-     * streamed yet belongs to this discussion. The block says it is waiting instead
-     * of showing the other turn's answer.
-     */
-    queued?: boolean;
     /** The answer failed (the session recorded a prompt error). */
     failed?: boolean;
     /**
@@ -234,6 +220,25 @@ export declare function discussionRows(discussion: Discussion): number;
  * @returns a map from row index to the rows added below it.
  */
 export declare function discussionRowExtras(discussions: readonly Discussion[], rowCount: number): Map<number, number>;
+/**
+ * Where each block's own box starts, in rows, measured from the BASE of the stack it hangs in.
+ *
+ * Several blocks can end on one row, and then they are drawn one below the other under it, in the
+ * order they are held: `discussionRowExtras` charges the row the sum of them all, and the row stream
+ * (and the split view's card layer) draws one reserving box per block, in that same order. So the
+ * first block of a stack starts at the base, the second one box down, and so on — and a jump that
+ * names a block has to add that many boxes to the base, or it lands the top of the stack whatever
+ * block the reader asked for.
+ *
+ * Nothing is reordered here and no height changes: this is a reading of the drawing order the views
+ * already use, keyed by block id so a jump can ask for the box it was sent for.
+ *
+ * @param discussions - the blocks currently attached to this file, in the order they are drawn.
+ * @param anchorOf - the row the block hangs under, clamped exactly as the caller's row stream clamps
+ *   it (its own height table and its own idea of the last row are what the base below is measured in).
+ * @returns a map from block id to the block's own start, in whole rows below its stack's base.
+ */
+export declare function discussionStackOffsets(discussions: readonly Discussion[], anchorOf: (discussion: Discussion) => number): Map<string, number>;
 /**
  * The discussion attached to exactly this range, if there is one.
  *
@@ -271,7 +276,12 @@ export declare function discussionOverlapping(discussions: readonly Discussion[]
  * is what says whether they still hold the code the comment was written about. When they do
  * not — the lines were edited under it, or the hunk moved — the quote is looked for elsewhere
  * in the model and the thread follows it, nearest occurrence first when the same code shows up
- * more than once. When nothing matches, the block keeps the rows it last matched and is marked
+ * more than once. A match farther from the stored rows than `REANCHOR_MAX_LINES` is not that code
+ * having moved but another place in the file that happens to read the same — a look-alike
+ * declaration, which the recorded ±1 context cannot tell apart — so it is refused and the block is
+ * marked outdated instead. That is the one bound the host half applies in lines as well, so the
+ * figure the list ships and the place this half draws the card cannot come from two rules. When
+ * nothing matches, the block keeps the rows it last matched and is marked
  * outdated: it says so and keeps its quote for the reader, rather than being silently moved
  * onto whatever took those lines. Only when the whole range is gone from the model does the
  * block move, and then by line number, back to where the range used to be (see `gone`). The mark
@@ -301,6 +311,49 @@ export declare function remapDiscussion(discussion: Discussion, lineOf: (row: nu
  * @returns the re-anchored blocks, in the order they came in.
  */
 export declare function remapDiscussions(discussions: readonly Discussion[], lineOf: (row: number) => number | undefined, textOf: (row: number) => string, rowCount: number, hasNewLine?: (row: number) => boolean): readonly Discussion[];
+/**
+ * The frame the reader drew, as a block of quoted lines for the question to carry.
+ *
+ * A reference is one number, and one number cannot say what the comment is about: a reader annotating
+ * a DELETED line produced `a.txt:2`, and the agent read line 2 of the file — which is another line
+ * entirely, because a removed line has no number in the file as it reads now (its `2` is the old
+ * file's). This block says what the number cannot: every row of the frame, in order, with the side it
+ * was on (`-` removed, `+` added, a space for context) and the number THAT side's file gave it.
+ *
+ * `quote` and `quoteLines` are what the record already holds — one entry per frame row, in row order —
+ * so nothing new is recorded for this: the question is built from the same snapshot the re-anchor uses.
+ * A record from before `quoteLines` was kept (or one the agent annotated, which names current lines
+ * only) has the text alone, and the block is that text with no numbers rather than a made-up side.
+ *
+ * @param quote - the anchored lines as they read when the comment was written.
+ * @param quoteLines - their gutter numbers and sides, in the same order.
+ * @returns the block, or `''` when there is nothing to show.
+ */
+export declare function quotedFrame(quote: string | undefined, quoteLines: readonly DiscussionQuoteLine[] | undefined): string;
+/**
+ * Whether a frame's numbers all come from the OLD file.
+ *
+ * A frame of removed rows alone names no line the file still has, so the numbers its record carries
+ * (`anchor`, and the line a jump is handed) are the old file's. Everything that would otherwise
+ * present them as current lines asks this first: the label a card and a list item wear, and the
+ * landing a jump to that comment makes. A record that kept no sides answers false — what it holds is
+ * read exactly as it always was.
+ *
+ * @param quoteLines - the frame's gutter numbers and sides, when the record kept them.
+ * @returns true when the frame holds at least one row and none of them has a new-file line.
+ */
+export declare function frameNamesNoCurrentLine(quoteLines: readonly DiscussionQuoteLine[] | readonly CommentQuoteLine[] | undefined): boolean;
+/**
+ * Whether a frame covers a line the file no longer has.
+ *
+ * The one thing a frame can hide behind its numbers: `anchor` is documented as new-file lines, but a
+ * frame of removed rows alone has none of those, so the numbers it carries are the old file's. A
+ * caller that must not present them as current lines asks this.
+ *
+ * @param quoteLines - the frame's gutter numbers and sides, when the record kept them.
+ * @returns true when at least one row of the frame is a removed line.
+ */
+export declare function frameCoversRemovedRow(quoteLines: readonly DiscussionQuoteLine[] | undefined): boolean;
 /** What the selection frame shows for the current selection. */
 export interface SelectionFrameActions {
     /** Whether keep/revert apply (the selection fully covers change blocks). */

package/lib/types/client/dock.d.ts

@@ -133,11 +133,6 @@ export interface DiffDockBodyProps extends PropsLocale<'diff-approval'> {
     readonly onDockClose?: ((close: (() => void) | undefined) => void) | undefined;
     [key: string]: unknown;
 }
-/** One live dock adapter: the face, and the teardown that unsubscribes. */
-export interface DiffDock {
-    readonly face: DockFace;
-    dispose(): void;
-}
 /** The cordis pieces this adapter uses, structurally: optional service lookup and
  *  a scoped effect. Both are what the plugin sandbox always forwards — its
  *  whitelist covers `effect`/`on`/timers but NOT cordis's dynamic `inject`, and it

package/lib/types/client/dsh-icons.d.ts

@@ -0,0 +1,60 @@
+/**
+ * The shell's own icons and controls, fetched by every name the shells have given them.
+ *
+ * 0.1.7 renamed the whole product icon set: `IconCloseOutline16` and `IconChevronDownOutline14` became
+ * `IconCloseOutlineMedium` and `IconChevronDownOutlineMedium` (the `Medium`/`Regular` stroke pair,
+ * where the numbers used to say the size). A static import of the old name survives compilation and
+ * then hands React `undefined` to render, which is a crash of the whole mount — React #130, measured
+ * here on 0.1.7-rc.2: `DiffApprovalHeaderEntry` took the panel down with it, and nothing in the plugin
+ * worked until the shell was changed again.
+ *
+ * So each glyph is resolved in two steps, and neither step can throw:
+ *
+ * 1. the NEW name, looked up on the module namespace at run time — present on 0.1.7 and later, and
+ *    absent (harmlessly) anywhere else;
+ * 2. the legacy name, imported statically, which is what every shell before 0.1.7 exports.
+ *
+ * A glyph no shell offers renders nothing rather than failing. That is the point of this module: a
+ * rename in the shell becomes a shell that draws fewer icons, not a plugin that dies. The plugin's
+ * peer floor stays `0.1.0-rc.5`.
+ *
+ * The exports carry the names this plugin's code already uses (`IconCloseOutline16` and the rest), so
+ * a caller changes only where it imports from.
+ *
+ * @module dsh-diff-approval/client/dsh-icons
+ */
+import * as primitives from '@deepseek-ai/dsh-client-ui-primitives';
+/** The pending-changes glyph: the sidebar's badge and the header entry carry it. */
+export declare const IconListPenOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+/** The list's fold chevrons. */
+export declare const IconChevronDownOutline14: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+export declare const IconChevronUpOutline14: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+/** The path picker's tree disclosure. */
+export declare const IconChevronRightOutline14: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+/** Closing a card, a dialog or a search bar. */
+export declare const IconCloseOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+/** The thread's own overflow menu. */
+export declare const IconEllipsisOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+/** The add-path dialog's browse button. */
+export declare const IconBrowseOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+/** A folder in the picker's tree, open and closed. */
+export declare const IconFolderOpenOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+export declare const IconFolderOpen16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+export declare const IconFolderClose16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+/** The presentation switch: dock the panel in the sidebar. */
+export declare const IconPanelLeftOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+/** The settings tab's refresh, which the older set spelled with the small size. */
+export declare const IconRefreshOutline14: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+/** Adding a path, the diff refresh, the search box, and Settings. */
+export declare const IconPlusOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+export declare const IconRefreshOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+export declare const IconSearchOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+export declare const IconSettingsOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;
+/**
+ * The package's own menus, toasts and clipboard helper.
+ *
+ * Re-exported so one module names everything this plugin takes from the shell's primitives: these
+ * names have survived both releases, and a caller has no reason to reach past this file.
+ */
+export { Menu, Toast, Tooltip, writeClipboard } from '@deepseek-ai/dsh-client-ui-primitives';
+export type { MenuEntry } from '@deepseek-ai/dsh-client-ui-primitives';

package/lib/types/client/header-entry.d.ts

@@ -38,15 +38,30 @@ export interface HeaderEntryFace {
 export interface HeaderEntrySeat {
     /** The selected session and its rows; the entry is inert without one. */
     useSessions?: ((select: (state: HeaderEntrySessions) => boolean) => boolean) | undefined;
+    /**
+     * The session this seat is about, composed by the shell.
+     *
+     * This is where 0.1.7 hands it over: the session-list store it gives `useSessions` carries
+     * `{ ids, byId, phase, projectionsBySession }` and no selected session at all, while the shell's
+     * own session-scoped seats take the id from their props and ask the store about that row
+     * (`useSessions(s => s.byId[sessionId]?.blank)`). Older shells put `current` in the store, which is
+     * why both are read (see `session-seat.ts`).
+     */
+    sessionId?: SessionId | undefined;
 }
-/** The slice of the session list this entry reads. */
+/** The slice of the session list this entry reads: whichever of these a shell carries. */
 export interface HeaderEntrySessions {
-    /** The session the app is showing. */
+    /** The session the app is showing, on the shells that keep it here (0.1.5 and earlier). */
     current?: SessionId | undefined;
     /** Known sessions by id, for the blank-session check. */
-    byId: Record<string, {
+    byId?: Record<string, {
         blank?: boolean;
-    } | undefined>;
+    } | undefined> | undefined;
+    /** A blank flag carried by the STATE itself on the shells that place it there. */
+    blank?: boolean | undefined;
+    /** Every other field a shell may carry: this plugin reads none of them, and a shape it does not
+     *  know must not disable the entry (see `session-seat.ts`). */
+    [field: string]: unknown;
 }
 /** Full props of the header entry: the injected face's hooks bound as
  *  `usePending`/`useDock`, the framework seats, and the locale `t`. */
@@ -56,4 +71,4 @@ export type DiffApprovalHeaderEntryProps = InjectFace<HeaderEntryFace> & HeaderE
  * @param props - the bound hooks, the session seat, and the translator.
  * @returns the icon button, with the pending count while there is one.
  */
-export declare function DiffApprovalHeaderEntry({ usePending, useSessions, useDock, t }: DiffApprovalHeaderEntryProps): ReactNode;
+export declare function DiffApprovalHeaderEntry({ usePending, useSessions, useDock, sessionId, t }: DiffApprovalHeaderEntryProps): ReactNode;

package/lib/types/client/ids.d.ts

@@ -0,0 +1,18 @@
+/**
+ * A fresh identifier for a record the browser half mints.
+ *
+ * `crypto.randomUUID` is a SECURE CONTEXT API. Reaching this host from a phone or a tablet over the
+ * LAN means plain HTTP, where `crypto.randomUUID` is undefined and calling it throws a TypeError —
+ * synchronously, before the code around it does anything. That is how writing a comment died on a
+ * tablet: the composer stayed open with the reader's words in it, nothing was sent, and no failure was
+ * shown, because the throw happened before the request and outside the `try` that would have said so.
+ * The desktop was never affected: `localhost` IS a secure context.
+ *
+ * `crypto.getRandomValues` carries no such restriction, so the fallback builds the id from it — a
+ * version-4 UUID, the shape the host and the client both treat as opaque but which reads as one
+ * everywhere it is printed. `randomUUID` is still preferred when it is there: it is the same thing
+ * done by the platform.
+ *
+ * @returns an identifier, unique for practical purposes.
+ */
+export declare function newId(): string;

package/lib/types/client/locales.d.ts

@@ -9,7 +9,6 @@ export declare const zh: {
     'panel.dockFailed': string;
     'panel.loading': string;
     'panel.readFailed': string;
-    'panel.group.current': string;
     'panel.group.others': string;
     'panel.tabs': string;
     'panel.tab.pending': string;
@@ -22,7 +21,9 @@ export declare const zh: {
     'panel.blockAtEnd': string;
     'panel.blockAtStart': string;
     'panel.blockSingle': string;
-    'panel.viewDiff': string;
+    'chip.openDefault': string;
+    'chip.reviewInPanel': string;
+    'chip.gone': string;
     'panel.fileNotPending': string;
     'panel.selectHint': string;
     'panel.searchPlaceholder': string;
@@ -67,6 +68,8 @@ export declare const zh: {
     'action.gotoCancel': string;
     'panel.gotoMissing': string;
     'panel.persistFailed': string;
+    'panel.commentPersistFailed': string;
+    'panel.undoFailed': string;
     'settings.keybindings': string;
     'settings.keybindingsDesc': string;
     'settings.comment': string;
@@ -101,6 +104,20 @@ export declare const zh: {
     'row.keepListed': string;
     'row.keepRemove': string;
     'row.revertRemove': string;
+    'panel.batchKeepAsk': string;
+    'panel.batchKeepRemoveAsk': string;
+    'panel.batchRevertAsk': string;
+    'panel.batchRevertRemoveAsk': string;
+    'panel.batchKeepAllAsk': string;
+    'panel.batchRevertAllAsk': string;
+    'panel.batchCloseAsk': string;
+    'panel.batchDeletes': string;
+    'panel.batchAndMore': string;
+    'panel.batchGo': string;
+    'panel.removeOneAsk': string;
+    'panel.removeCommentsOne': string;
+    'panel.removeCommentsMany': string;
+    'panel.removeCommentsMore': string;
     'row.added': string;
     'row.removed': string;
     'action.keep': string;
@@ -110,15 +127,17 @@ export declare const zh: {
     'action.discussionCollapse': string;
     'discussion.placeholder': string;
     'discussion.marker': string;
+    'discussion.frameLegend': string;
+    'discussion.deletedLabel': string;
     'discussion.promptRule': string;
     'discussion.promptRuleSkill': string;
     'discussion.thinking': string;
-    'discussion.queued': string;
-    'discussion.answering': string;
     'discussion.outdatedQuote': string;
     'discussion.hidden': string;
     'discussion.failed': string;
     'discussion.stopped': string;
+    'discussion.addMissing': string;
+    'discussion.askMissing': string;
     'action.revert': string;
     'action.delete': string;
     'action.discussionEnd': string;
@@ -138,10 +157,6 @@ export declare const zh: {
     'action.viewUnified': string;
     'action.viewPreview': string;
     'action.viewSource': string;
-    'action.viewBefore': string;
-    'action.viewAfter': string;
-    'action.viewSingle': string;
-    'action.viewDouble': string;
     'panel.mdPreviewFailed': string;
     'action.copyHint': string;
     'action.copied': string;
@@ -161,7 +176,6 @@ export declare const zh: {
     'panel.importFailed': string;
     'action.search': string;
     'action.settings': string;
-    'action.presentation': string;
     'action.presentationCurrent': string;
     'action.cover': string;
     'cover.left': string;
@@ -186,13 +200,10 @@ export declare const zh: {
     'action.summonHint': string;
     'action.retry': string;
     'panel.crashed': string;
-    'status.kept': string;
-    'status.reverted': string;
-    'status.missing': string;
     'panel.resolvedAsk': string;
     'panel.keepInList': string;
-    'panel.removalQuiet': string;
-    'panel.removalQuietHint': string;
+    'panel.keepInListQuiet': string;
+    'panel.keepInListQuietHint': string;
     'panel.fileKeptAsk': string;
     'panel.fileRevertedAsk': string;
     'panel.confirmFileRemove': string;
@@ -247,7 +258,6 @@ export declare const en: {
     'panel.dockFailed': string;
     'panel.loading': string;
     'panel.readFailed': string;
-    'panel.group.current': string;
     'panel.group.others': string;
     'panel.tabs': string;
     'panel.tab.pending': string;
@@ -260,7 +270,9 @@ export declare const en: {
     'panel.blockAtEnd': string;
     'panel.blockAtStart': string;
     'panel.blockSingle': string;
-    'panel.viewDiff': string;
+    'chip.openDefault': string;
+    'chip.reviewInPanel': string;
+    'chip.gone': string;
     'panel.fileNotPending': string;
     'panel.selectHint': string;
     'panel.searchPlaceholder': string;
@@ -305,6 +317,8 @@ export declare const en: {
     'action.gotoCancel': string;
     'panel.gotoMissing': string;
     'panel.persistFailed': string;
+    'panel.commentPersistFailed': string;
+    'panel.undoFailed': string;
     'settings.keybindings': string;
     'settings.keybindingsDesc': string;
     'settings.comment': string;
@@ -339,6 +353,20 @@ export declare const en: {
     'row.keepListed': string;
     'row.keepRemove': string;
     'row.revertRemove': string;
+    'panel.batchKeepAsk': string;
+    'panel.batchKeepRemoveAsk': string;
+    'panel.batchRevertAsk': string;
+    'panel.batchRevertRemoveAsk': string;
+    'panel.batchKeepAllAsk': string;
+    'panel.batchRevertAllAsk': string;
+    'panel.batchCloseAsk': string;
+    'panel.batchDeletes': string;
+    'panel.batchAndMore': string;
+    'panel.batchGo': string;
+    'panel.removeOneAsk': string;
+    'panel.removeCommentsOne': string;
+    'panel.removeCommentsMany': string;
+    'panel.removeCommentsMore': string;
     'row.added': string;
     'row.removed': string;
     'action.keep': string;
@@ -348,15 +376,17 @@ export declare const en: {
     'action.discussionCollapse': string;
     'discussion.placeholder': string;
     'discussion.marker': string;
+    'discussion.frameLegend': string;
+    'discussion.deletedLabel': string;
     'discussion.promptRule': string;
     'discussion.promptRuleSkill': string;
     'discussion.thinking': string;
-    'discussion.queued': string;
-    'discussion.answering': string;
     'discussion.outdatedQuote': string;
     'discussion.hidden': string;
     'discussion.failed': string;
     'discussion.stopped': string;
+    'discussion.addMissing': string;
+    'discussion.askMissing': string;
     'action.revert': string;
     'action.delete': string;
     'action.discussionEnd': string;
@@ -376,10 +406,6 @@ export declare const en: {
     'action.viewUnified': string;
     'action.viewPreview': string;
     'action.viewSource': string;
-    'action.viewBefore': string;
-    'action.viewAfter': string;
-    'action.viewSingle': string;
-    'action.viewDouble': string;
     'panel.mdPreviewFailed': string;
     'action.copyHint': string;
     'action.copied': string;
@@ -399,7 +425,6 @@ export declare const en: {
     'panel.importFailed': string;
     'action.search': string;
     'action.settings': string;
-    'action.presentation': string;
     'action.presentationCurrent': string;
     'action.cover': string;
     'cover.left': string;
@@ -424,13 +449,10 @@ export declare const en: {
     'action.summonHint': string;
     'action.retry': string;
     'panel.crashed': string;
-    'status.kept': string;
-    'status.reverted': string;
-    'status.missing': string;
     'panel.resolvedAsk': string;
     'panel.keepInList': string;
-    'panel.removalQuiet': string;
-    'panel.removalQuietHint': string;
+    'panel.keepInListQuiet': string;
+    'panel.keepInListQuietHint': string;
     'panel.fileKeptAsk': string;
     'panel.fileRevertedAsk': string;
     'panel.confirmFileRemove': string;

package/lib/types/client/panel-memory.d.ts

@@ -1,5 +1,6 @@
 /**
- * Where the review panel was left, for this page's lifetime.
+ * Where the review panel was left, and what the reader has typed but not sent, for
+ * this page's lifetime.
  *
  * Closing the panel is not "done reviewing": reopening it should put the reader
  * back on the file they were in, at the offset they were at. That is a fact about
@@ -13,52 +14,116 @@
  *
  * @module dsh-diff-approval/client/panel-memory
  */
-import type { Discussion } from './discussion.ts';
+import type { Discussion, DiscussionQuoteLine } from './discussion.ts';
 /**
- * Fired on `window` whenever a session's threads change. The threads are written by the file
- * detail (which owns them) and read by the list pane (whose comments tab shows them all), so the
- * event is what keeps the two in step: a reader who posts a comment sees it in the list at once
- * instead of on the next poll.
+ * What the panel keeps about one thread that is NOT the host's to keep: what the reader is
+ * typing, whether the block is folded, and the body height the panel last measured for it.
+ *
+ * A thread itself is the host's record now (see `pending.comments`), so nothing here may
+ * hold a turn: the panel renders the snapshot, and this only remembers the page-local
+ * presentation of it. That split is what makes a poll harmless — a poll replaces the
+ * snapshot, and a draft typed a moment ago is not in it, so keeping the draft here is what
+ * stops the poll from clearing the field under the reader's hands.
  */
-export declare const COMMENTS_CHANGED_EVENT = "dsh-diff-approval:comments-changed";
+export interface ThreadLocal {
+    /** What the reader has typed into the compose row (not sent yet). */
+    draft: string;
+    /** Folded to the one-row header. */
+    collapsed: boolean;
+    /**
+     * The last send of this thread was refused (no live agent, or the ask itself failed). It is a
+     * page-local fact because nothing was stored: the host has no question to record, so the block
+     * could only say so from here. Cleared by the next send.
+     */
+    failed?: boolean | undefined;
+    /**
+     * How many rows the block's body was last measured to occupy, or absent before the first
+     * measurement. The reservation follows the drawing rather than the other way round (see
+     * the panel's read-back), and it is kept here so a mount that comes back draws the thread
+     * at the size it was already measured at instead of reserving the model's estimate again.
+     */
+    bodyRows?: number | undefined;
+}
 /**
- * The comment threads a session's panel is holding.
+ * The page-local thread state a session's panel is holding.
  * @param sessionId - the session, if any.
- * @returns the threads by pending entry id; empty when there are none.
+ * @returns the state by comment id; empty when there is none.
  */
-export declare function rememberedDiscussions(sessionId: string | undefined): Readonly<Record<string, readonly Discussion[]>>;
+export declare function rememberedThreads(sessionId: string | undefined): Readonly<Record<string, ThreadLocal>>;
 /**
- * Remember the comment threads a session's panel holds. Called on every change, so a mount
- * that appears later (another presentation, or the panel reopened) starts from them.
+ * Remember one session's page-local thread state. Called on every change, so a mount that
+ * appears later starts from it.
  * @param sessionId - the session the panel is reviewing; nothing is recorded without one.
- * @param byFile - the threads, by pending entry id.
+ * @param threads - the state by comment id.
+ */
+export declare function rememberThreads(sessionId: string | undefined, threads: Readonly<Record<string, ThreadLocal>>): void;
+/**
+ * A block the reader has PLACED on a file's lines but has not written down yet.
+ *
+ * It has no host record (see `sendDiscussion`), so nothing else on this page holds it: without this
+ * the reader's complaint was right — placing a comment, typing a first thought and closing the panel
+ * threw the whole block away, because only the words were remembered and a comment id to hang them
+ * under does not exist until the block is sent. Everything a mount needs to draw the block again is
+ * here: the entry it was placed on, the lines, the quote it was placed against, and the words.
+ */
+export interface PlacedThread {
+    /**
+     * The block's page-local id, minted when it was placed. The comment's own id is minted when the
+     * block is written (see `sendDiscussion`), and this is only ever the name of an unsent block.
+     */
+    id: string;
+    /** The pending entry whose rows it was placed on. */
+    fileId: string;
+    /**
+     * The rows it was made on, plus the new-file lines that survive a rebuild. The lines are what a
+     * mount re-anchors from (see `remapDiscussions`); the row indices are the placement's own hint,
+     * stale the moment the model moves.
+     */
+    anchor: Discussion['anchor'];
+    /** The anchored lines as they read when it was placed (the re-anchor fingerprint). */
+    quote: string;
+    /** `quote` with one line of context on each side, as it read then. */
+    quoteContext?: string | undefined;
+    /** The gutter numbers of `quote`'s lines, in the same order. */
+    quoteLines?: readonly DiscussionQuoteLine[] | undefined;
+    /** What the reader had typed into it. */
+    draft: string;
+}
+/**
+ * The placed-but-unsent blocks a session's panel is holding.
+ * @param sessionId - the session, if any.
+ * @returns the blocks, oldest placement first; empty when there are none.
  */
-export declare function rememberDiscussions(sessionId: string | undefined, byFile: Readonly<Record<string, readonly Discussion[]>>): void;
+export declare function rememberedPlacedThreads(sessionId: string | undefined): readonly PlacedThread[];
 /**
- * End one comment, wherever it came from.
+ * Remember one session's placed-but-unsent blocks. Called on every change, so a mount that comes
+ * back draws them again on the same lines with the same words.
  *
- * The threads belong to the file detail, which writes them here on every change — but the list
- * pane shows them all and ends them too, and it has no way to reach into another component's
- * state. So the list asks the memory, and the detail reads the memory back on the event this
- * fires (see `COMMENTS_CHANGED_EVENT`): one record, one action, whichever pane the reader used.
+ * A list that says the same thing as the one held is not stored: the caller rebuilds it per render,
+ * and only a real change (a placement, a draft, a block written or discarded) may republish it.
  *
- * @param sessionId - the session the panel is reviewing; nothing is forgotten without one.
- * @param fileId - the pending entry the thread hangs in.
- * @param discussionId - the thread to end; an id that is not there changes nothing.
+ * @param sessionId - the session the panel is reviewing; nothing is recorded without one.
+ * @param threads - the placements, oldest first.
  */
-export declare function forgetDiscussion(sessionId: string | undefined, fileId: string, discussionId: string): void;
+export declare function rememberPlacedThreads(sessionId: string | undefined, threads: readonly PlacedThread[]): void;
 /**
- * Drop every thread whose file the list no longer holds.
+ * Drop every placed block whose file the list no longer holds.
+ *
+ * A placement is a block of rows in ONE file's diff, so a file the reader has kept or
+ * reverted out of the list leaves nothing for its block to hang on: it goes with the file,
+ * exactly as the comments on those rows do (the host drops those; see `comments.ts`). The
+ * entry ids ARE paths, so a path that comes back later is the same id again — without this
+ * the placement was resurrected onto a file the reader had already dealt with, and
+ * `discussionOverlapping` went on refusing a fresh comment on those very rows because a
+ * block that is not on screen still counted as theirs.
  *
- * A thread is a block of rows in one file's diff, so a file the reader has kept or reverted out of the
- * list leaves nothing for its comments to hang on: they go with it, and the same path coming back later
- * in this visit starts clean. A new object is published only when something actually goes, so the panes
- * that adopt this record (see `COMMENTS_CHANGED_EVENT`) do not re-adopt an equal object on every poll.
+ * Nothing is published when nothing is dropped, so a poll that changes no file is not a
+ * change to the page's memory (see the identity discipline in `rememberPlacedThreads`).
  *
  * @param sessionId - the session the panel is reviewing; nothing is dropped without one.
  * @param listed - the pending entry ids the list is holding.
  */
-export declare function forgetDiscussionsNotIn(sessionId: string | undefined, listed: readonly string[]): void;
+export declare function forgetPlacedThreadsNotIn(sessionId: string | undefined, listed: readonly string[]): void;
 /**
  * Whether the reader asked not to be asked again about removing one file.
  * @param sessionId - the session the panel is reviewing; nothing is quiet without one.