@linxin666/dsh-client-ui-task-board 0.3.5 → 0.3.6

package/lib/index.js

@@ -1620,6 +1620,13 @@ function isErrorTurnEnd(data) {
 var HostExecutionRunner = class {
 	api;
 	commands;
+	/**
+	* Newest scanned event seq per session whose complete backward scan found
+	* no matching turn/end. While a session's newest seq is unchanged a
+	* re-scan cannot change the outcome, so only a one-message probe runs per
+	* poll tick instead of up to 100 history pages.
+	*/
+	scanMemos = /* @__PURE__ */ new Map();
 	constructor(api, commands) {
 		this.api = api;
 		this.commands = commands;
@@ -1696,11 +1703,24 @@ var HostExecutionRunner = class {
 			items = response.result.value.items;
 		}
 		const summary = items.find((item) => item.sessionId === sessionId);
-		if (summary === void 0) return {
-			outcome: "cancelled",
-			error: "execution session no longer exists"
-		};
+		if (summary === void 0) {
+			this.scanMemos.delete(sessionId);
+			return {
+				outcome: "cancelled",
+				error: "execution session no longer exists"
+			};
+		}
 		if (summary.running) return { outcome: "pending" };
+		const probe = await this.api.sessions.history(request({
+			sessionId: summary.sessionId,
+			maxMessages: 1
+		}));
+		if (!probe.result.ok) return { outcome: "pending" };
+		const newestSeq = probe.result.value.events.reduce((newest, entry) => {
+			const seq = entry.event.seq;
+			return typeof seq !== "number" ? newest : newest === void 0 ? seq : Math.max(newest, seq);
+		}, void 0);
+		if (newestSeq !== void 0 && this.scanMemos.get(sessionId) === newestSeq) return { outcome: "pending" };
 		const events = [];
 		let beforeSeq;
 		let reachedExecutionBoundary = false;
@@ -1729,7 +1749,11 @@ var HostExecutionRunner = class {
 		}
 		if (!reachedExecutionBoundary) return { outcome: "pending" };
 		const turnEnd = events.filter((entry) => entry.event.type === "turn/end" && (startedAt <= 0 || typeof entry.event.time === "number" && entry.event.time >= startedAt)).sort((a, b) => (a.event.seq ?? Number.MAX_SAFE_INTEGER) - (b.event.seq ?? Number.MAX_SAFE_INTEGER))[0];
-		if (turnEnd === void 0) return { outcome: "pending" };
+		if (turnEnd === void 0) {
+			if (newestSeq !== void 0) this.scanMemos.set(sessionId, newestSeq);
+			return { outcome: "pending" };
+		}
+		this.scanMemos.delete(sessionId);
 		return isErrorTurnEnd(turnEnd.event.data) ? {
 			outcome: "failed",
 			error: "agent turn ended with an error"

package/lib/types/client/board/TaskForm.d.ts

@@ -0,0 +1,28 @@
+/**
+ * Shared task-modal pieces: the overlay shell (backdrop, form, title, error,
+ * footer) and the title/description/prompt field trio used by both the
+ * NewTaskModal and the EditTaskModal. State stays in the owning modal; these
+ * are controlled components.
+ */
+import type { ReactNode } from 'react';
+/** Modal overlay: closes on backdrop press, submits through the form. */
+export declare function ModalShell({ ariaLabel, title, error, pending, submitLabel, onSubmit, onClose, children, }: {
+    ariaLabel: string;
+    title: string;
+    error: string | undefined;
+    pending: boolean;
+    submitLabel: string;
+    onSubmit: () => void;
+    onClose: () => void;
+    children: ReactNode;
+}): import("react").JSX.Element;
+/** Title + description + prompt fields shared by the new and edit task forms. */
+export declare function TaskContentFields({ title, description, prompt, onTitleChange, onDescriptionChange, onPromptChange, }: {
+    title: string;
+    description: string;
+    prompt: string;
+    /** Receives the new title; the owner also clears its error state. */
+    onTitleChange: (value: string) => void;
+    onDescriptionChange: (value: string) => void;
+    onPromptChange: (value: string) => void;
+}): import("react").JSX.Element;

package/lib/types/host-runner.d.ts

@@ -30,6 +30,13 @@ export declare class SessionLaunchError extends Error {
 export declare class HostExecutionRunner {
     private readonly api;
     private readonly commands?;
+    /**
+     * Newest scanned event seq per session whose complete backward scan found
+     * no matching turn/end. While a session's newest seq is unchanged a
+     * re-scan cannot change the outcome, so only a one-message probe runs per
+     * poll tick instead of up to 100 history pages.
+     */
+    private readonly scanMemos;
     constructor(api: ApiProxy, commands?: SessionCommandDispatcher | undefined);
     launch(task: TaskRecord): Promise<string>;
     listRunning(): Promise<{

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@linxin666/dsh-client-ui-task-board",
   "description": "Host-authoritative task board for the DSH Web GUI with real session execution, Host cron scheduling, and optional cross-platform idle-sleep protection; mounted without DSH source changes.",
-  "version": "0.3.5",
+  "version": "0.3.6",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/types/index.d.ts",

package/src/client/board/EditTaskModal.tsx

@@ -9,7 +9,7 @@ import { useState } from 'react'
 import type { BoardController } from '../../core/controller.ts'
 import type { TaskRecord } from '../../core/tasks.ts'
 import { t } from '../locales.ts'
-import css from '../board.module.css'
+import { ModalShell, TaskContentFields } from './TaskForm.tsx'
 
 /** Edit-task form overlay. */
 export function EditTaskModal({ controller, task, onClose }: { controller: BoardController; task: TaskRecord; onClose: () => void }) {
@@ -36,59 +36,23 @@ export function EditTaskModal({ controller, task, onClose }: { controller: Board
   }
 
   return (
-    <div className={css.modalBackdrop} onMouseDown={event => { if (event.target === event.currentTarget) onClose() }}>
-      <form
-        className={css.modal}
-        role="dialog"
-        aria-label={t('edit.title')}
-        onSubmit={event => { event.preventDefault(); void submit() }}
-      >
-        <h2 className={css.modalTitle}>{t('edit.title')}</h2>
-
-        <label className={css.field}>
-          <span className={css.fieldLabel}>{t('new.title')}</span>
-          <input
-            className={css.input}
-            value={title}
-            autoFocus
-            placeholder={t('new.titlePlaceholder')}
-            onChange={event => { setTitle(event.target.value); setError(undefined) }}
-          />
-        </label>
-
-        <label className={css.field}>
-          <span className={css.fieldLabel}>{t('new.description')}</span>
-          <textarea
-            className={css.input}
-            rows={3}
-            value={description}
-            placeholder={t('new.descriptionPlaceholder')}
-            onChange={event => { setDescription(event.target.value) }}
-          />
-        </label>
-
-        <label className={css.field}>
-          <span className={css.fieldLabel}>{t('new.prompt')}</span>
-          <textarea
-            className={css.input}
-            rows={4}
-            value={prompt}
-            placeholder={t('new.promptPlaceholder')}
-            onChange={event => { setPrompt(event.target.value) }}
-          />
-        </label>
-
-        {error !== undefined && <p className={css.formError}>{error}</p>}
-
-        <footer className={css.modalFooter}>
-          <button type="button" className={css.ghostButton} onClick={onClose}>
-            {t('new.cancel')}
-          </button>
-          <button type="submit" className={css.primaryButton} disabled={pending}>
-            {t('edit.save')}
-          </button>
-        </footer>
-      </form>
-    </div>
+    <ModalShell
+      ariaLabel={t('edit.title')}
+      title={t('edit.title')}
+      error={error}
+      pending={pending}
+      submitLabel={t('edit.save')}
+      onSubmit={() => { void submit() }}
+      onClose={onClose}
+    >
+      <TaskContentFields
+        title={title}
+        description={description}
+        prompt={prompt}
+        onTitleChange={value => { setTitle(value); setError(undefined) }}
+        onDescriptionChange={setDescription}
+        onPromptChange={setPrompt}
+      />
+    </ModalShell>
   )
 }

package/src/client/board/NewTaskModal.tsx

@@ -8,6 +8,7 @@ import { isValidCron, nextRunAtMs } from '../../core/schedule.ts'
 import { TASK_PERMISSIONS, type TaskPermission } from '../../core/tasks.ts'
 import { t, type TaskBoardKey } from '../locales.ts'
 import { SCHEDULE_PRESETS } from '../schedule-presets.ts'
+import { ModalShell, TaskContentFields } from './TaskForm.tsx'
 import css from '../board.module.css'
 
 /** New-task form overlay. */
@@ -64,47 +65,23 @@ export function NewTaskModal({ controller, onClose }: { controller: BoardControl
     : undefined
 
   return (
-    <div className={css.modalBackdrop} onMouseDown={event => { if (event.target === event.currentTarget) onClose() }}>
-      <form
-        className={css.modal}
-        role="dialog"
-        aria-label={t('board.new')}
-        onSubmit={event => { event.preventDefault(); void submit() }}
-      >
-        <h2 className={css.modalTitle}>{t('board.new')}</h2>
-
-        <label className={css.field}>
-          <span className={css.fieldLabel}>{t('new.title')}</span>
-          <input
-            className={css.input}
-            value={title}
-            autoFocus
-            placeholder={t('new.titlePlaceholder')}
-            onChange={event => { setTitle(event.target.value); setError(undefined) }}
-          />
-        </label>
-
-        <label className={css.field}>
-          <span className={css.fieldLabel}>{t('new.description')}</span>
-          <textarea
-            className={css.input}
-            rows={3}
-            value={description}
-            placeholder={t('new.descriptionPlaceholder')}
-            onChange={event => { setDescription(event.target.value) }}
-          />
-        </label>
-
-        <label className={css.field}>
-          <span className={css.fieldLabel}>{t('new.prompt')}</span>
-          <textarea
-            className={css.input}
-            rows={4}
-            value={prompt}
-            placeholder={t('new.promptPlaceholder')}
-            onChange={event => { setPrompt(event.target.value) }}
-          />
-        </label>
+    <ModalShell
+      ariaLabel={t('board.new')}
+      title={t('board.new')}
+      error={error}
+      pending={pending}
+      submitLabel={t('new.submit')}
+      onSubmit={() => { void submit() }}
+      onClose={onClose}
+    >
+      <TaskContentFields
+        title={title}
+        description={description}
+        prompt={prompt}
+        onTitleChange={value => { setTitle(value); setError(undefined) }}
+        onDescriptionChange={setDescription}
+        onPromptChange={setPrompt}
+      />
 
         <label className={css.field}>
           <span className={css.fieldLabel}>{t('new.workspace')}</span>
@@ -201,18 +178,6 @@ export function NewTaskModal({ controller, onClose }: { controller: BoardControl
             </>
           )}
         </section>
-
-        {error !== undefined && <p className={css.formError}>{error}</p>}
-
-        <footer className={css.modalFooter}>
-          <button type="button" className={css.ghostButton} onClick={onClose}>
-            {t('new.cancel')}
-          </button>
-          <button type="submit" className={css.primaryButton} disabled={pending}>
-            {t('new.submit')}
-          </button>
-        </footer>
-      </form>
-    </div>
+    </ModalShell>
   )
 }

package/src/client/board/TaskForm.tsx

@@ -0,0 +1,111 @@
+/**
+ * Shared task-modal pieces: the overlay shell (backdrop, form, title, error,
+ * footer) and the title/description/prompt field trio used by both the
+ * NewTaskModal and the EditTaskModal. State stays in the owning modal; these
+ * are controlled components.
+ */
+import type { ReactNode } from 'react'
+import { t } from '../locales.ts'
+import css from '../board.module.css'
+
+/** Modal overlay: closes on backdrop press, submits through the form. */
+export function ModalShell({
+  ariaLabel,
+  title,
+  error,
+  pending,
+  submitLabel,
+  onSubmit,
+  onClose,
+  children,
+}: {
+  ariaLabel: string
+  title: string
+  error: string | undefined
+  pending: boolean
+  submitLabel: string
+  onSubmit: () => void
+  onClose: () => void
+  children: ReactNode
+}) {
+  return (
+    <div className={css.modalBackdrop} onMouseDown={event => { if (event.target === event.currentTarget) onClose() }}>
+      <form
+        className={css.modal}
+        role="dialog"
+        aria-label={ariaLabel}
+        onSubmit={event => { event.preventDefault(); onSubmit() }}
+      >
+        <h2 className={css.modalTitle}>{title}</h2>
+
+        {children}
+
+        {error !== undefined && <p className={css.formError}>{error}</p>}
+
+        <footer className={css.modalFooter}>
+          <button type="button" className={css.ghostButton} onClick={onClose}>
+            {t('new.cancel')}
+          </button>
+          <button type="submit" className={css.primaryButton} disabled={pending}>
+            {submitLabel}
+          </button>
+        </footer>
+      </form>
+    </div>
+  )
+}
+
+/** Title + description + prompt fields shared by the new and edit task forms. */
+export function TaskContentFields({
+  title,
+  description,
+  prompt,
+  onTitleChange,
+  onDescriptionChange,
+  onPromptChange,
+}: {
+  title: string
+  description: string
+  prompt: string
+  /** Receives the new title; the owner also clears its error state. */
+  onTitleChange: (value: string) => void
+  onDescriptionChange: (value: string) => void
+  onPromptChange: (value: string) => void
+}) {
+  return (
+    <>
+      <label className={css.field}>
+        <span className={css.fieldLabel}>{t('new.title')}</span>
+        <input
+          className={css.input}
+          value={title}
+          autoFocus
+          placeholder={t('new.titlePlaceholder')}
+          onChange={event => onTitleChange(event.target.value)}
+        />
+      </label>
+
+      <label className={css.field}>
+        <span className={css.fieldLabel}>{t('new.description')}</span>
+        <textarea
+          className={css.input}
+          rows={3}
+          value={description}
+          placeholder={t('new.descriptionPlaceholder')}
+          onChange={event => onDescriptionChange(event.target.value)}
+        />
+      </label>
+
+      <label className={css.field}>
+        <span className={css.fieldLabel}>{t('new.prompt')}</span>
+        <textarea
+          className={css.input}
+          rows={4}
+          value={prompt}
+          placeholder={t('new.promptPlaceholder')}
+          onChange={event => onPromptChange(event.target.value)}
+        />
+      </label>
+    </>
+  )
+}

package/src/client/board-mount.tsx

@@ -44,7 +44,13 @@ export function mountBoard(controller: BoardController): () => void {
   let container: HTMLDivElement | undefined
 
   const ensure = (): void => {
-    if (container !== undefined) return
+    if (container !== undefined) {
+      if (container.isConnected) return
+      root?.unmount()
+      root = undefined
+      container.remove()
+      container = undefined
+    }
     const column = conversationColumn()
     if (column === undefined) return
     container = document.createElement('div')

package/src/core/controller.ts

@@ -200,17 +200,11 @@ export class BoardController {
 
   openBoard(): void {
     if (this.boardOpen) return
-    // The board stays open until the user explicitly closes it: by clicking
-    // the sidebar entry again, a sidebar session/workspace row, or a board
-    // action (openSession / close). Session-list churn (background
-    // navigation, the Host runner selecting a fresh execution session,
-    // settlement, other plugins) never evicts the board.
     this.boardOpen = true
     this.notify()
   }
 
   closeBoard(): void {
-    if (!this.boardOpen) return
     this.boardOpen = false
     this.notify()
   }
@@ -396,6 +390,7 @@ export class BoardController {
    * @param sessionId - the execution session to open.
    */
   openSession(sessionId: string): void {
+    this.closeBoard()
     this.deps.sessions.open(sessionId)
   }
 

package/src/host-runner.ts

@@ -50,6 +50,14 @@ function isErrorTurnEnd(data: unknown): boolean {
 }
 
 export class HostExecutionRunner {
+  /**
+   * Newest scanned event seq per session whose complete backward scan found
+   * no matching turn/end. While a session's newest seq is unchanged a
+   * re-scan cannot change the outcome, so only a one-message probe runs per
+   * poll tick instead of up to 100 history pages.
+   */
+  private readonly scanMemos = new Map<string, number>()
+
   constructor(
     private readonly api: ApiProxy,
     private readonly commands?: SessionCommandDispatcher,
@@ -124,8 +132,21 @@ export class HostExecutionRunner {
       items = response.result.value.items
     }
     const summary = items.find(item => item.sessionId === sessionId)
-    if (summary === undefined) return { outcome: 'cancelled', error: 'execution session no longer exists' }
+    if (summary === undefined) {
+      this.scanMemos.delete(sessionId)
+      return { outcome: 'cancelled', error: 'execution session no longer exists' }
+    }
     if (summary.running) return { outcome: 'pending' }
+    // Probe the newest event before paging: SessionSummary.updatedAt tracks
+    // human prompts, not event appends, so only the history head proves
+    // whether a re-scan could find anything new.
+    const probe = await this.api.sessions.history(request({ sessionId: summary.sessionId, maxMessages: 1 }))
+    if (!probe.result.ok) return { outcome: 'pending' }
+    const newestSeq = probe.result.value.events.reduce<number | undefined>((newest, entry) => {
+      const seq = entry.event.seq
+      return typeof seq !== 'number' ? newest : newest === undefined ? seq : Math.max(newest, seq)
+    }, undefined)
+    if (newestSeq !== undefined && this.scanMemos.get(sessionId) === newestSeq) return { outcome: 'pending' }
     const events: Array<{ event: { type: string; seq?: number; time?: number; data: unknown } }> = []
     let beforeSeq: number | undefined
     let reachedExecutionBoundary = false
@@ -158,7 +179,12 @@ export class HostExecutionRunner {
         startedAt <= 0 || (typeof entry.event.time === 'number' && entry.event.time >= startedAt)
       ))
       .sort((a, b) => (a.event.seq ?? Number.MAX_SAFE_INTEGER) - (b.event.seq ?? Number.MAX_SAFE_INTEGER))[0]
-    if (turnEnd === undefined) return { outcome: 'pending' }
+    if (turnEnd === undefined) {
+      // Complete scan, no match: remember the head so later ticks probe only.
+      if (newestSeq !== undefined) this.scanMemos.set(sessionId, newestSeq)
+      return { outcome: 'pending' }
+    }
+    this.scanMemos.delete(sessionId)
     return isErrorTurnEnd(turnEnd.event.data)
       ? { outcome: 'failed', error: 'agent turn ended with an error' }
       : { outcome: 'succeeded' }