dsh-diff-approval 0.29.1 → 0.31.0

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

@@ -20,14 +20,15 @@
 import type { ReactNode } from 'react';
 import type { HostObservable, InjectFace, PropsLocale } from '@deepseek-ai/dsh-client-ui-slots';
 import type { SessionId } from '@deepseek-ai/dsh-client-connection/client';
-import type { PendingDiffSnapshot } from './slots.ts';
+import type { PendingDiffSnapshot, PendingViewHooks } from './slots.ts';
 import type { DockSnapshot } from './dock.tsx';
-/** What the header entry reads from this plugin's own face: the two observables.
- *  Everything else it needs (the button's own click) goes through the window
- *  events, so the entry never holds a copy of the panel's state. */
-export interface HeaderEntryFace {
+/** What the header entry reads from this plugin's own face: the two observables, and the per-session
+ *  view reader (`pendingView`) that keeps its count on ITS session. Everything else it needs (the
+ *  button's own click) goes through the window events, so the entry never holds a copy of the panel's
+ *  state. */
+export interface HeaderEntryFace extends PendingViewHooks {
     hooks: {
-        /** Live pending-diff snapshot: the count on the button. */
+        /** Live pending-diff snapshot: the fallback count for a face with no per-session reader. */
         pending: HostObservable<PendingDiffSnapshot>;
         /** The dock's state, absent in a build without a right sidebar. */
         dock?: HostObservable<DockSnapshot>;
@@ -38,15 +39,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 +72,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, pendingView, 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,25 +9,35 @@ 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;
     'panel.tab.comments': string;
     'panel.commentsEmpty': string;
-    'panel.commentOutdated': string;
     'panel.aria': string;
     'panel.stats': string;
     'panel.blockPosition': string;
     'panel.blockAtEnd': string;
     'panel.blockAtStart': string;
     'panel.blockSingle': string;
-    'panel.viewDiff': string;
+    'chip.openDefault': string;
+    'chip.reviewInPanel': string;
+    'chip.copyPath': string;
+    'chip.about': string;
+    'chip.about.title': string;
+    'panel.confirmFileTitle': string;
+    'panel.batchConfirmTitle': string;
+    'chip.about.body': string;
+    'chip.about.ok': string;
+    'update.chip': string;
+    'update.body': string;
+    'update.npm': string;
+    'update.github': string;
+    'update.dismiss': string;
+    'chip.gone': string;
     'panel.fileNotPending': string;
     'panel.selectHint': string;
     'panel.searchPlaceholder': string;
-    'panel.missing': string;
-    'panel.missingHint': string;
     'panel.externalChanged': string;
     'panel.dismiss': string;
     'panel.pasteOnCopy': string;
@@ -67,12 +77,16 @@ 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;
     'settings.commentDesc': string;
     'settings.diffView': string;
     'settings.diffViewDesc': string;
+    'settings.chipMenu': string;
+    'settings.chipMenu.help': string;
     'settings.diffPreview': string;
     'panel.keyDesc': string;
     'panel.shortcutNone': string;
@@ -95,12 +109,29 @@ export declare const zh: {
     'panel.key.coverRight': string;
     'panel.key.coverTop': string;
     'settings.tabLabel': string;
-    'row.create': string;
     'row.failed': string;
+    'panel.unseen': string;
+    'panel.commentCategory': string;
     'row.dismiss': string;
     'row.keepListed': string;
     'row.keepRemove': string;
     'row.revertRemove': string;
+    'row.deleteRemove': string;
+    'panel.batchKeepAsk': string;
+    'panel.batchKeepRemoveAsk': string;
+    'panel.batchRevertAsk': string;
+    'panel.batchRevertRemoveAsk': string;
+    'panel.batchKeepAllAsk': string;
+    'panel.batchRevertAllAsk': string;
+    'panel.batchRevertAllDeletedAsk': string;
+    'panel.batchCloseAsk': string;
+    'panel.batchDeletes': string;
+    'panel.batchDeletesList': string;
+    'panel.batchGo': string;
+    'panel.removeOneAsk': string;
+    'panel.removeCommentsOne': string;
+    'panel.removeCommentsMany': string;
+    'panel.batchCommentsList': string;
     'row.added': string;
     'row.removed': string;
     'action.keep': string;
@@ -110,15 +141,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 +171,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 +190,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 +214,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;
@@ -221,6 +246,7 @@ export declare const zh: {
     'panel.addNoEntries': string;
     'panel.addMissing': string;
     'panel.addOutside': string;
+    'panel.addOtherSession': string;
     'panel.addNotAFile': string;
     'panel.addFailed': string;
     'action.cancel': string;
@@ -247,25 +273,35 @@ 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;
     'panel.tab.comments': string;
     'panel.commentsEmpty': string;
-    'panel.commentOutdated': string;
     'panel.aria': string;
     'panel.stats': string;
     'panel.blockPosition': string;
     'panel.blockAtEnd': string;
     'panel.blockAtStart': string;
     'panel.blockSingle': string;
-    'panel.viewDiff': string;
+    'chip.openDefault': string;
+    'chip.reviewInPanel': string;
+    'chip.copyPath': string;
+    'chip.about': string;
+    'chip.about.title': string;
+    'panel.confirmFileTitle': string;
+    'panel.batchConfirmTitle': string;
+    'chip.about.body': string;
+    'chip.about.ok': string;
+    'update.chip': string;
+    'update.body': string;
+    'update.npm': string;
+    'update.github': string;
+    'update.dismiss': string;
+    'chip.gone': string;
     'panel.fileNotPending': string;
     'panel.selectHint': string;
     'panel.searchPlaceholder': string;
-    'panel.missing': string;
-    'panel.missingHint': string;
     'panel.externalChanged': string;
     'panel.dismiss': string;
     'panel.pasteOnCopy': string;
@@ -305,12 +341,16 @@ 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;
     'settings.commentDesc': string;
     'settings.diffView': string;
     'settings.diffViewDesc': string;
+    'settings.chipMenu': string;
+    'settings.chipMenu.help': string;
     'settings.diffPreview': string;
     'panel.keyDesc': string;
     'panel.shortcutNone': string;
@@ -333,12 +373,29 @@ export declare const en: {
     'panel.key.coverRight': string;
     'panel.key.coverTop': string;
     'settings.tabLabel': string;
-    'row.create': string;
     'row.failed': string;
+    'panel.unseen': string;
+    'panel.commentCategory': string;
     'row.dismiss': string;
     'row.keepListed': string;
     'row.keepRemove': string;
     'row.revertRemove': string;
+    'row.deleteRemove': string;
+    'panel.batchKeepAsk': string;
+    'panel.batchKeepRemoveAsk': string;
+    'panel.batchRevertAsk': string;
+    'panel.batchRevertRemoveAsk': string;
+    'panel.batchKeepAllAsk': string;
+    'panel.batchRevertAllAsk': string;
+    'panel.batchRevertAllDeletedAsk': string;
+    'panel.batchCloseAsk': string;
+    'panel.batchDeletes': string;
+    'panel.batchDeletesList': string;
+    'panel.batchGo': string;
+    'panel.removeOneAsk': string;
+    'panel.removeCommentsOne': string;
+    'panel.removeCommentsMany': string;
+    'panel.batchCommentsList': string;
     'row.added': string;
     'row.removed': string;
     'action.keep': string;
@@ -348,15 +405,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 +435,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 +454,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 +478,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;
@@ -459,6 +510,7 @@ export declare const en: {
     'panel.addNoEntries': string;
     'panel.addMissing': string;
     'panel.addOutside': string;
+    'panel.addOtherSession': string;
     'panel.addNotAFile': string;
     'panel.addFailed': string;
     'action.cancel': 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,65 +14,147 @@
  *
  * @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 rememberDiscussions(sessionId: string | undefined, byFile: Readonly<Record<string, readonly Discussion[]>>): void;
+export declare function rememberThreads(sessionId: string | undefined, threads: Readonly<Record<string, ThreadLocal>>): void;
 /**
- * End one comment, wherever it came from.
+ * A block the reader has PLACED on a file's lines but has not written down yet.
  *
- * 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.
+ * 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 rememberedPlacedThreads(sessionId: string | undefined): readonly PlacedThread[];
+/**
+ * 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.
  *
- * @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.
+ * 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 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 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.
+ * 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.
+ *
+ * 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.
+ * Whether the reader asked not to be asked again about rows leaving THIS session's list.
  * @param sessionId - the session the panel is reviewing; nothing is quiet without one.
- * @param fileId - the pending entry the question was about.
  * @returns true when the action should run without the question.
  */
-export declare function removalAskQuiet(sessionId: string | undefined, fileId: string): boolean;
+export declare function removalAskQuiet(sessionId: string | undefined): boolean;
 /**
- * Stop asking whether one file should leave the list, for the rest of this page.
+ * Stop asking whether rows should leave the list, for the rest of this page: the whole session, every
+ * file in it.
  * @param sessionId - the session the panel is reviewing; nothing is recorded without one.
- * @param fileId - the pending entry the question was about.
  */
-export declare function quietenRemovalAsk(sessionId: string | undefined, fileId: string): void;
+export declare function quietenRemovalAsk(sessionId: string | undefined): void;
+/**
+ * Window event a dismissal dispatches, so every mounted seat hears it.
+ *
+ * The notice's chip lives in the open file's status bar, and the panel that draws it can be mounted more than
+ * once (the footer seat and the right sidebar's docked tab are separate mounts): the state above is the fact,
+ * and this event is how the OTHER mount learns about it NOW rather than at its next mount. One state, one
+ * event — so whichever seat is on screen cannot disagree about whether the reader has said "got it".
+ */
+export declare const UPDATE_DISMISSED_EVENT = "diff-approval:update-dismissed";
+/**
+ * The release whose notice the reader dismissed on this page.
+ * @returns the dismissed version, or `''` when none has been dismissed here.
+ */
+export declare function dismissedUpdateVersion(): string;
+/**
+ * Remember that the reader dismissed the notice for one version on this page, and tell the other mounts.
+ * @param version - the version that notice was about.
+ */
+export declare function dismissUpdate(version: string): void;
 /**
  * Remember that a session's panel was left showing one file, and where in it.
  * @param sessionId - the session the panel was reviewing; nothing is recorded

package/lib/types/client/path-match.d.ts

@@ -0,0 +1,50 @@
+/**
+ * Path comparison for the review's own references: is the path a press named the same file as one the
+ * pending list holds?
+ *
+ * Its own module because three places ask it and they must not disagree — the panel's jump to a file, the
+ * plugin's produced-file press bridge, and the store's "does the page hold this path" reader — while the
+ * panel itself is a React component module that a data-layer reader should not have to import.
+ *
+ * @module dsh-diff-approval/client/path-match
+ */
+/** Normalize a path for comparison: forward slashes, no trailing slash. */
+export declare function normalizeDiffPath(p: string): string;
+/**
+ * The directory a file path lives in, as the path was spelled.
+ *
+ * @param p - the file's own path.
+ * @returns the directory, or undefined when the path names no directory (empty, or a bare file name).
+ */
+export declare function directoryOfPath(p: string | undefined): string | undefined;
+/**
+ * The absolute path a Markdown link names, resolved the way Markdown means it: against the directory of
+ * the file the preview is showing.
+ *
+ * A link in a document is relative to THAT DOCUMENT — `../src/foo.ts` in `docs/a.md` is `src/foo.ts` at the
+ * workspace root, not `../src/foo.ts` of the page's own URL — which is why the base is the previewed file's
+ * own directory and not the workspace root. The href is percent-decoded, its `#line`/`?query` suffix is
+ * dropped, `.`/`..` are resolved, and both separators are accepted. An href that is already absolute
+ * (drive letter or a leading slash) needs no base.
+ *
+ * Exported so the panel's preview click and its tests share one implementation and cannot disagree about
+ * what a link points at.
+ *
+ * @param baseDirectory - the directory of the file being previewed, or the workspace root when that file's
+ *   own path is unknown.
+ * @param href - the anchor's href, as written in the Markdown.
+ * @returns the absolute path, or undefined when it cannot be named (an empty href, or a relative href with
+ *   no base to resolve against).
+ */
+export declare function resolveMarkdownHref(baseDirectory: string | undefined, href: string): string | undefined;
+/**
+ * Whether a produced-file chip path and a pending file path refer to the same file, tolerant of separator
+ * style (\\ vs /) and of a workspace-relative vs absolute form. `chipPath` is typically the harness's
+ * workspace-relative forward-slash path; `filePath` is the host's absolute native-separator path. Matching
+ * is case-insensitive so a Windows drive/segment case difference does not miss the file the user clicked.
+ * @param chipPath - the path a press (or a reference) named.
+ * @param filePath - the path the pending list holds.
+ * @param workspacePath - the session's workspace root, when it is known.
+ * @returns whether the two name one file.
+ */
+export declare function diffPathsMatch(chipPath: string, filePath: string, workspacePath: string | undefined): boolean;