dsh-diff-approval 0.27.0 → 0.28.0

package/lib/index.js

@@ -2482,7 +2482,8 @@ function apply(ctx, config) {
 						files,
 						workspacePath: workspaceOf(sessionId)?.path,
 						redoCleared: redoCleared || void 0,
-						commentSkill: commentSkill()
+						commentSkill: commentSkill(),
+						persistError: persistFailureReported
 					}
 				};
 			}

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

@@ -155,6 +155,10 @@ export declare const SplitDiff: import("react").ForwardRefExoticComponent<{
     renderDiscussion?: (discussion: Discussion, bodyWidth: number, split: boolean) => ReactNode;
     /** The comment action for the current selection, when the panel offers one for it. */
     selectionComment?: ReactNode;
+    /** The changed-line runs the overview ruler draws, in whole-file row indices. */
+    rulerRuns: readonly RulerRun[];
+    /** The go-to popup, which this view centres on its own box (see `gotoDialog`). */
+    gotoDialog?: ReactNode;
 } & import("react").RefAttributes<SplitDiffHandle>>;
 /**
  * Reconstruct the plain text of the current selection so auto-wrap's visual
@@ -164,6 +168,12 @@ export declare const SplitDiff: import("react").ForwardRefExoticComponent<{
  * without a newline while the real newline between diff rows is kept.
  */
 export declare function selectedPlainText(): string | undefined;
+/** One maximal run of same-kind changed rows: what both views' rulers are built from. */
+interface RulerRun {
+    start: number;
+    end: number;
+    kind: 'del' | 'add';
+}
 /** Render the pending-edit review panel and its unified footer action. */
 export declare function PendingPanel({ wide, useSessions, usePending, onRefresh, onKeep, onRevert, onBlockKeep, onBlockRevert, onOpen, onPreviewImage, onPasteReference, onAskAgent, watchChat, onUndo, onRedo, onImportVcs, onRefreshVcs, onBrowse, onAddPath, onKeepAll, onRevertAll, onAckRedoCleared, collapseSidebar, t, docked, dockHost, onOpenDock, closeDock, useDock, }: PendingPanelProps): import("react").JSX.Element;
 export {};

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

@@ -16,7 +16,6 @@ export declare const zh: {
     'panel.tab.comments': string;
     'panel.commentsEmpty': string;
     'panel.commentOutdated': string;
-    'panel.commentEmptyTitle': string;
     'panel.aria': string;
     'panel.stats': string;
     'panel.blockPosition': string;
@@ -65,6 +64,7 @@ export declare const zh: {
     'action.gotoGo': string;
     'action.gotoCancel': string;
     'panel.gotoMissing': string;
+    'panel.persistFailed': string;
     'settings.keybindings': string;
     'settings.keybindingsDesc': string;
     'settings.comment': string;
@@ -248,7 +248,6 @@ export declare const en: {
     'panel.tab.comments': string;
     'panel.commentsEmpty': string;
     'panel.commentOutdated': string;
-    'panel.commentEmptyTitle': string;
     'panel.aria': string;
     'panel.stats': string;
     'panel.blockPosition': string;
@@ -297,6 +296,7 @@ export declare const en: {
     'action.gotoGo': string;
     'action.gotoCancel': string;
     'panel.gotoMissing': string;
+    'panel.persistFailed': string;
     'settings.keybindings': string;
     'settings.keybindingsDesc': string;
     'settings.comment': string;

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

@@ -47,6 +47,18 @@ export declare function rememberDiscussions(sessionId: string | undefined, byFil
  * @param discussionId - the thread to end; an id that is not there changes nothing.
  */
 export declare function forgetDiscussion(sessionId: string | undefined, fileId: string, discussionId: string): void;
+/**
+ * Drop every thread 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.
+ *
+ * @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;
 /**
  * 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.

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

@@ -24,6 +24,12 @@ export interface PendingDiffSnapshot {
      * same rules inline when it is not.
      */
     commentSkill?: string | undefined;
+    /**
+     * The last failure to write the pending state to disk, or absent while writes are working. The
+     * panel says so once: the file is what makes the list survive a restart, so a reader who is never
+     * told finds the list gone later with nothing to explain it (see the wire type).
+     */
+    persistError?: string | undefined;
     /** Latched when an external change created a fresh undo checkpoint that
      * superseded the redo history; the panel surfaces it once (deferred if the
      * panel is closed) via a bottom-right notice. */

package/lib/types/types.d.ts

@@ -66,6 +66,12 @@ export interface DiffApprovalListValue {
      * is set, and carries the same rules inline when it is not.
      */
     commentSkill?: string | undefined;
+    /**
+     * The last failure to write the pending state to disk, or absent while writes are working. The
+     * panel says so once: that file is what makes the list survive a restart, so a reader who is not
+     * told about it simply finds the list gone later, with nothing to explain it (issue #6).
+     */
+    persistError?: string | undefined;
 }
 /** What the open endpoint asks the OS to do with a file. */
 export type DiffApprovalOpenAction = 'open' | 'reveal';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-diff-approval",
-  "version": "0.27.0",
+  "version": "0.28.0",
   "description": "DeepSeek Harness plugin: pending-edit review with whole-file diff and Keep/Revert",
   "packageManager": "pnpm@11.21.0",
   "author": "Wu Zhiwei",