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

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

@@ -2,7 +2,6 @@ import type { TaskRecord } from '../../core/tasks.ts';
 /** Compact relative/absolute time label. */
 export declare function formatHostTimestamp(ms: number, timeZone?: string): string;
 export declare function formatTime(ms: number, timeZone?: string): string;
-/** One card in a column. */
 declare function TaskCardInner({ task, pending, timeZone, onClick }: {
     task: TaskRecord;
     pending: boolean;

package/lib/types/core/controller.d.ts

@@ -2,8 +2,8 @@
  * Board controller: the single owner of task-ledger state and view state.
  *
  * In production it projects the Host ledger and submits confirmed actions;
- * the legacy store seam remains for v1 migration tests. It also closes the
- * board whenever the user navigates to a session.
+ * the legacy store seam remains for v1 migration tests. The board closes
+ * only on explicit user navigation, never implicitly on session-list churn.
  * Framework-free (structural runtime faces) so the whole orchestration is
  * unit-testable with fakes.
  *
@@ -188,9 +188,17 @@ export declare class BoardController {
     runTask(id: string): Promise<boolean>;
     /** Re-run a settled task through the Host (the Host replans and executes). */
     rerunTask(id: string): Promise<void>;
-    /** Close the board when the user navigates to another session. */
+    /**
+     * Session-list notifications fire for all kinds of incidental churn
+     * (background navigation, the Host runner creating and selecting a fresh
+     * execution session, settlement, other plugins), so closing on `current`
+     * changes would evict the board without the user asking. The board closes
+     * only on explicit user navigation: a sidebar session/workspace row click
+     * (board-mount onClickSidebarRow) or the board's own actions
+     * (openSession / close). Keeping the hook preserves the subscription
+     * contract for future listeners.
+     */
     private onSessionsChanged;
-    private lastCurrent;
     private persistAndNotify;
     private commitRemote;
     private performRemote;

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.4",
+  "version": "0.3.5",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/types/index.d.ts",

package/src/client/board/TaskBoard.tsx

@@ -5,7 +5,7 @@
  */
 import { memo, useCallback, useEffect, useState } from 'react'
 import { selectedTaskOf, type BoardController } from '../../core/controller.ts'
-import { COLUMNS, type TaskRecord } from '../../core/tasks.ts'
+import { COLUMNS, canMoveManually, type TaskRecord } from '../../core/tasks.ts'
 import { t } from '../locales.ts'
 import css from '../board.module.css'
 import { NewTaskModal } from './NewTaskModal.tsx'
@@ -124,8 +124,27 @@ export function TaskBoard({ controller }: { controller: BoardController }) {
         ) : (
           COLUMNS.map(column => {
             const tasks = visible.filter(task => task.status === column.status)
+            const isManualDropTarget = column.status === 'backlog' || column.status === 'todo'
             return (
-              <section key={column.status} className={css.column} data-status={column.status} data-dsh-part="column">
+              <section
+                key={column.status}
+                className={css.column}
+                data-status={column.status}
+                data-dsh-part="column"
+                onDragOver={isManualDropTarget ? (event) => {
+                  event.preventDefault()
+                  event.dataTransfer.dropEffect = 'move'
+                } : undefined}
+                onDrop={isManualDropTarget ? (event) => {
+                  event.preventDefault()
+                  const taskId = event.dataTransfer.getData('text/plain')
+                  if (!taskId) return
+                  const dropped = snapshot.tasks.find(t => t.id === taskId)
+                  if (dropped && canMoveManually(dropped.status, column.status) && dropped.status !== column.status) {
+                    controller.moveTask(taskId, column.status)
+                  }
+                } : undefined}
+              >
                 <header className={css.columnHeader}>
                   <span className={css.statusDot} data-status={column.status} aria-hidden="true" />
                   <h3 className={css.columnTitle}>{t(STATUS_KEY[column.status])}</h3>

package/src/client/board/TaskCard.tsx

@@ -37,11 +37,12 @@ export function formatTime(ms: number, timeZone?: string): string {
   return `${date.getFullYear()}-${String(date.getMonth() + 1).padStart(2, '0')}-${String(date.getDate()).padStart(2, '0')}`
 }
 
-/** One card in a column. */
 function TaskCardInner({ task, pending, timeZone, onClick }: { task: TaskRecord; pending: boolean; timeZone?: string; onClick: () => void }) {
   const latest = task.executions[task.executions.length - 1]
   const runs = task.executions.length
   const archived = task.archivedAt !== undefined
+  const isDraggable = !archived && task.status !== 'running' && !pending
+
   return (
     <button
       type="button"
@@ -49,6 +50,11 @@ function TaskCardInner({ task, pending, timeZone, onClick }: { task: TaskRecord;
       data-status={archived ? 'archived' : task.status}
       data-dsh-part="card"
       data-pending={pending || undefined}
+      draggable={isDraggable}
+      onDragStart={isDraggable ? (event) => {
+        event.dataTransfer.setData('text/plain', task.id)
+        event.dataTransfer.effectAllowed = 'move'
+      } : undefined}
       onClick={onClick}
       title={task.description !== '' ? task.description : task.title}
     >

package/src/core/controller.ts

@@ -2,8 +2,8 @@
  * Board controller: the single owner of task-ledger state and view state.
  *
  * In production it projects the Host ledger and submits confirmed actions;
- * the legacy store seam remains for v1 migration tests. It also closes the
- * board whenever the user navigates to a session.
+ * the legacy store seam remains for v1 migration tests. The board closes
+ * only on explicit user navigation, never implicitly on session-list churn.
  * Framework-free (structural runtime faces) so the whole orchestration is
  * unit-testable with fakes.
  *
@@ -108,11 +108,6 @@ function randomUuid(): string {
   return `${hex.slice(0, 8)}-${hex.slice(8, 12)}-${hex.slice(12, 16)}-${hex.slice(16, 20)}-${hex.slice(20)}`
 }
 
-/** Read the current selection off a session-list snapshot (structural). */
-function currentOf(sessions: SessionsControllerFace): string | undefined {
-  return sessions.list.getSnapshot().current
-}
-
 function messageOf(error: unknown): string {
   return error instanceof Error ? error.message : String(error)
 }
@@ -205,10 +200,11 @@ export class BoardController {
 
   openBoard(): void {
     if (this.boardOpen) return
-    // Baseline the selection the board opened against: the board stays open
-    // until the user navigates (selection changes), never on mere status
-    // updates of the already-selected session.
-    this.lastCurrent = currentOf(this.deps.sessions)
+    // 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()
   }
@@ -428,16 +424,20 @@ export class BoardController {
 
   // --- internals ---------------------------------------------------------------
 
-  /** Close the board when the user navigates to another session. */
+  /**
+   * Session-list notifications fire for all kinds of incidental churn
+   * (background navigation, the Host runner creating and selecting a fresh
+   * execution session, settlement, other plugins), so closing on `current`
+   * changes would evict the board without the user asking. The board closes
+   * only on explicit user navigation: a sidebar session/workspace row click
+   * (board-mount onClickSidebarRow) or the board's own actions
+   * (openSession / close). Keeping the hook preserves the subscription
+   * contract for future listeners.
+   */
   private onSessionsChanged(): void {
-    if (!this.boardOpen) return
-    const current = currentOf(this.deps.sessions)
-    if (current !== this.lastCurrent) this.closeBoard()
-    this.lastCurrent = current
+    // Intentionally empty: never close the board implicitly.
   }
 
-  private lastCurrent: string | undefined = undefined
-
   private persistAndNotify(): void {
     if (this.deps.transport === undefined) this.deps.store.save(this.tasks)
     this.notify()