@linxin666/dsh-client-ui-task-board 0.3.24 → 0.4.1

package/src/client/sidebar-entry-core.ts

@@ -16,33 +16,6 @@
  */
 import { subscribeBodyInvalidations } from './body-mutations.ts'
 
-/**
- * One trailing action button seated at the entry row's right edge (e.g. a
- * refresh command or a collapse chevron). Rows with actions switch from the
- * classic single-button structure to a container with a main button plus the
- * action buttons (nested interactive elements are invalid HTML); rows without
- * actions keep the classic structure untouched.
- */
-export interface SidebarEntryAction {
-  /** Stable id stamped as data-dsh-entry-action="<id>" (CSS hooks and tests). */
-  id: string
-  /** Inline icon markup while the row is active (always, when inactiveIcon is unset). */
-  icon: string
-  /**
-   * Optional icon while the row is inactive. Setting it makes the action a
-   * state mirror (a collapse chevron): the core swaps the icon and stamps
-   * aria-expanded on every active-state change. Requires the active bridge.
-   */
-  inactiveIcon?: string
-  /** Localized accessible label (aria-label + title); receives the active state. */
-  label(active: boolean): string
-  /**
-   * Click handler; receives the button so the caller can drive transient
-   * affordances (a refresh cooldown) without a re-render.
-   */
-  onClick(button: HTMLButtonElement): void
-}
-
 /** Per-package configuration for one sidebar entry row. */
 export interface SidebarEntryOptions {
   /** Full attribute name identifying the injected row (idempotency key), e.g. 'data-dsh-ssh-entry'. */
@@ -74,12 +47,6 @@ export interface SidebarEntryOptions {
   refresh?: { subscribe(listener: () => void): () => void }
   /** Click action (open/toggle the owning panel). */
   onToggle(): void
-  /**
-   * Optional trailing action buttons at the row's right edge. Requires the
-   * package CSS to seat entryMain / entryAction; empty keeps the classic
-   * single-button row.
-   */
-  actions?: readonly SidebarEntryAction[]
   /** Family-block position: 'before' inserts ahead of sibling plugin rows, 'after' behind them. */
   position: 'before' | 'after'
   /**
@@ -118,11 +85,9 @@ function newSessionButton(root: HTMLElement): HTMLButtonElement | undefined {
 }
 
 /** Build the entry row (detached; inserted once the shell is up). */
-function createEntry(options: SidebarEntryOptions): { entry: HTMLElement; applyLabel: () => void; setOpen: (open: boolean) => void } {
-  const actions = options.actions ?? []
-  const composite = actions.length > 0
-  const entry = document.createElement(composite ? 'div' : 'button') as HTMLElement
-  if (!composite) (entry as HTMLButtonElement).type = 'button'
+function createEntry(options: SidebarEntryOptions): { entry: HTMLElement; applyLabel: () => void } {
+  const entry = document.createElement('button')
+  entry.type = 'button'
   entry.setAttribute(options.rowAttribute, '')
   if (options.plugin !== undefined) {
     entry.setAttribute('data-dsh-plugin', options.plugin)
@@ -134,62 +99,15 @@ function createEntry(options: SidebarEntryOptions): { entry: HTMLElement; applyL
   const iconSpan = document.createElement('span')
   iconSpan.className = options.css['entryIcon'] ?? ''
   iconSpan.innerHTML = options.icon
-  // The main hit area: the classic row IS the button; with trailing actions
-  // the row becomes a container and the main area its own button, so the row
-  // never nests interactive elements.
-  const main = composite ? document.createElement('button') : (entry as HTMLButtonElement)
-  if (composite) {
-    main.type = 'button'
-    main.className = options.css['entryMain'] ?? ''
-    main.append(iconSpan, labelSpan)
-    entry.append(main)
-  } else {
-    entry.append(iconSpan, labelSpan)
-  }
-  const actionButtons: { action: SidebarEntryAction; button: HTMLButtonElement }[] = []
-  for (const action of actions) {
-    const button = document.createElement('button')
-    button.type = 'button'
-    button.className = options.css['entryAction'] ?? ''
-    button.setAttribute('data-dsh-entry-action', action.id)
-    button.innerHTML = action.icon
-    button.addEventListener('click', () => { action.onClick(button) })
-    entry.append(button)
-    actionButtons.push({ action, button })
-  }
-  let open = false
-  const applyActions = (): void => {
-    for (const { action, button } of actionButtons) {
-      button.innerHTML = open || action.inactiveIcon === undefined ? action.icon : action.inactiveIcon
-      const text = action.label(open)
-      button.setAttribute('aria-label', text)
-      button.setAttribute('title', text)
-      if (action.inactiveIcon !== undefined) button.setAttribute('aria-expanded', String(open))
-    }
-  }
+  entry.append(iconSpan, labelSpan)
   const applyLabel = (): void => {
-    main.setAttribute('aria-label', options.label())
-    if (options.tooltip !== undefined) main.setAttribute('title', options.tooltip())
+    entry.setAttribute('aria-label', options.label())
+    if (options.tooltip !== undefined) entry.setAttribute('title', options.tooltip())
     labelSpan.textContent = options.label()
-    applyActions()
   }
   applyLabel()
-  main.addEventListener('click', options.onToggle)
-  if (composite) {
-    // The old row was clickable edge to edge; keep the padding between the
-    // main area and the action buttons a toggle target as well.
-    entry.addEventListener('click', (event) => {
-      if (event.target === entry) options.onToggle()
-    })
-  }
-  return {
-    entry,
-    applyLabel,
-    setOpen: (next: boolean) => {
-      open = next
-      applyActions()
-    },
-  }
+  entry.addEventListener('click', options.onToggle)
+  return { entry, applyLabel }
 }
 
 /** Re-insert the entry after the New Session row (before the browser region). */
@@ -231,7 +149,7 @@ export function mountSidebarEntry(options: SidebarEntryOptions): () => void {
   if (typeof document !== 'undefined' && document.querySelector(options.rowSelector) !== null) {
     return () => {}
   }
-  const { entry, applyLabel, setOpen } = createEntry(options)
+  const { entry, applyLabel } = createEntry(options)
   let root: HTMLElement | undefined
   let placed = false
   let unsubscribeRefresh: (() => void) | undefined
@@ -299,10 +217,8 @@ export function mountSidebarEntry(options: SidebarEntryOptions): () => void {
   // row permanently highlighted — delete the attribute instead.
   const unsubscribeActive = options.active === undefined ? undefined : (() => {
     const syncActive = (): void => {
-      const open = options.active!.isOpen()
-      if (open) entry.dataset.active = 'true'
+      if (options.active!.isOpen()) entry.dataset.active = 'true'
       else delete entry.dataset.active
-      setOpen(open)
     }
     const unsubscribe = options.active.subscribe(syncActive)
     syncActive()

package/src/client/sidebar-entry.ts

@@ -19,8 +19,8 @@ import { mountSidebarEntry as mountSharedSidebarEntry } from './sidebar-entry-co
 /** Stable data attribute identifying the injected entry row. */
 export const ENTRY_SELECTOR = '[data-dsh-taskboard-entry]'
 
-/** Inline icon normalized to the shell's 18px navigation glyph size. */
-const ICON = '<svg viewBox="0 0 16 16" width="18" height="18" fill="none" stroke="currentColor" stroke-width="1.3" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><rect x="2" y="2.5" width="12" height="11" rx="1.5"/><path d="M2 6.5h12M6.5 6.5v7"/></svg>'
+/** Inline icon normalized to the shell's 16px navigation glyph size. */
+const ICON = '<svg viewBox="0 0 16 16" width="16" height="16" fill="none" stroke="currentColor" stroke-width="1.3" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><rect x="2" y="2.5" width="12" height="11" rx="1.5"/><path d="M2 6.5h12M6.5 6.5v7"/></svg>'
 
 /** Locale-change subscription the shared core asks for (ctx.locale.subscribe shape). */
 export interface LocaleRefreshSource { subscribe(listener: () => void): () => void }

package/src/core/controller.ts

@@ -42,18 +42,22 @@ export interface TaskBoardTransport {
   parseDraft?(request: TaskBoardParseRequest, signal?: AbortSignal): Promise<TaskBoardParseDraft>
 }
 
-/** The sessions face the controller needs for navigation awareness. */
+/**
+ * The sessions face the controller needs for navigation awareness. The Client
+ * Session Controller carries no global selection since 0.1.6-alpha.2, so the
+ * wiring resolves the main-view Session once and exposes it as current().
+ */
 export interface SessionsControllerFace {
-  list: {
-    getSnapshot(): { current: string | undefined }
-    subscribe(fn: () => void): () => void
-  }
-  /** Select a session as current (navigates the conversation view). */
+  /** The Session the main view currently shows, when any. */
+  current(): string | undefined
+  /** Navigate the main view to a Session. */
   open(id: string): void
+  /** Subscribe to catalog changes; selection ownership rides the same snapshot. */
+  subscribe(fn: () => void): () => void
 }
 
 function currentOf(sessions: SessionsControllerFace | undefined): string | undefined {
-  return sessions?.list.getSnapshot().current
+  return sessions?.current()
 }
 
 /** Controller dependencies (all swappable in tests). */
@@ -194,7 +198,7 @@ export class BoardController {
       this.notify()
     }) : undefined
     if (unsubscribeExternal !== undefined) this.disposers.push(unsubscribeExternal)
-    this.disposers.push(this.deps.sessions.list.subscribe(() => {
+    this.disposers.push(this.deps.sessions.subscribe(() => {
       this.onSessionsChanged()
     }))
     this.notify()

package/src/http.ts

@@ -2,7 +2,9 @@
 /**
  * Shared JSON body/response helpers for the host route families: one strict
  * bounded body reader, one lenient bounded body reader, one JSON object
- * narrow, and one JSON writer. Previously these were copy-pasted across the
+ * narrow, one JSON writer, and the outbound request-init helper that keeps a
+ * host fetch body decodable (withIdentityEncoding). Previously the readers
+ * were copy-pasted across the
  * package route files (routes.ts, update-routes.ts, mobile-api.ts, and each
  * family's route module) with drifting contracts: body caps ranging 4 KiB to
  * 1 MiB and four distinct overflow behaviors (reject, undefined, null, throw).
@@ -88,6 +90,26 @@ export function asJsonObject(value: unknown): Record<string, unknown> | undefine
   return isJsonObject(value) ? value : undefined
 }
 
+/**
+ * Request init that asks a remote origin for an uncompressed body.
+ *
+ * The DSH host boots `@deepseek-ai/dsh-http-proxy`, whose top-level import of
+ * the npm `undici` copy replaces the legacy `undici.globalDispatcher.1` slot
+ * Node's built-in `fetch()` reads. That cross-major wrapper drops the
+ * `content-encoding` header and automatic decompression, so a plain fetch
+ * resolves to raw gzip/brotli/zstd bytes — compressed noise no JSON or text
+ * consumer can read. Every host fetch that parses a remote body must request
+ * identity encoding. Caller headers survive; `accept-encoding` is forced to
+ * `identity` because a caller value has no decoder behind it in this host.
+ * @param init - the caller's request init (signal, headers, method, ...).
+ * @returns a copy with `accept-encoding: identity` merged in.
+ */
+export function withIdentityEncoding(init: RequestInit = {}): RequestInit {
+  const headers = new Headers(init.headers)
+  headers.set('accept-encoding', 'identity')
+  return { ...init, headers }
+}
+
 /**
  * Write one JSON response. Default headers are the family defaults
  * (content-type and referrer-policy); caller headers are appended or

package/src/index.ts

@@ -6,11 +6,10 @@
  * a same-origin asynchronous view over that service.
  */
 
-import type { Context } from '@deepseek-ai/cordis'
+import type { Context, Volatile } from '@deepseek-ai/cordis'
 import type {} from '@deepseek-ai/dsh-agent'
 import type {} from '@deepseek-ai/dsh-commands'
-import type { SettingsNamespace } from '@deepseek-ai/dsh-settings'
-import z from 'schemastery'
+import z from '@deepseek-ai/schemastery'
 import type {} from '@deepseek-ai/dsh-system-prompt'
 import type {} from '@deepseek-ai/dsh-api-gateway'
 import type {} from '@deepseek-ai/dsh-workspace'
@@ -35,25 +34,30 @@ export const inject = ['systemPrompt', 'typertGateway', 'workspaceRegistry', 'we
 export const TASK_BOARD_GUIDANCE = '本机已安装 dsh-task-board 插件（DSH Web GUI 的任务看板）：侧边栏「任务看板」入口；在 dsh-web 插件全家桶仓库（packages/dsh-task-board）统一维护，经聚合包 web-ui-all 一键安装。能力：多列看板管理任务；Host 权威账本；关闭浏览器后仍由 Host 执行和结算；任务可钉住工作区、agent 预设和权限；支持 Host 本地时区的 5 段 cron，错过的触发点不补跑；可选且默认关闭的空闲系统睡眠保护允许屏幕熄灭，但不承诺拦截合盖、手动睡眠、休眠、关机或唤醒已睡眠机器。执行消耗 API 额度。用户提到「任务看板 / 看板 / 定时任务」时即指本插件，请据此协作。若你同时用 todo_write 维护会话顶部的可见计划列表，最终回复前必须再次调用 todo_write 收尾：没有剩余工作时不要保留 in_progress，已完成的最后一步要标为 completed。'
 
 /**
- * Settings namespace of the board's announcement capability — the section the
- * web settings surface edits. Spelled here rather than imported: the browser
- * half spells the same value and must not depend on a Host package.
+ * Plugin config, validated by the same-named schemastery schema.
+ *
+ * This schema IS the board's settings page: the 0.1.7 Host serves one
+ * configuration form per profile entry from the entry's own Config, so the
+ * three fields the browser card edits are marked volatile — the Loader commits
+ * an edit into the running fiber's references without remounting the row (no
+ * settings document of the plugin's own exists any more).
  */
-export const TASK_BOARD_SETTINGS_NAMESPACE = 'task-board' as SettingsNamespace
-
-/** Plugin config, validated by the same-named schemastery schema. */
 export interface Config {
   /**
    * When true (default), a system-prompt section announces the board to every
    * agent. Set false to keep the board silent in prompts; agents then learn
    * about it only when the user mentions it.
    */
-  announceToAgent?: boolean
+  announceToAgent?: Volatile<boolean>
   /** Master switch for the plugin (browser half + host announcement). */
-  enabled?: boolean
+  enabled?: Volatile<boolean>
   /** Prevent idle system sleep while sessions run or schedules are armed. */
-  preventIdleSleep?: boolean
-  /** Canonical reverse-proxy Host authorities admitted with a server-side token. */
+  preventIdleSleep?: Volatile<boolean>
+  /**
+   * Canonical reverse-proxy Host authorities admitted with a server-side
+   * token. Deployment-level, so it stays an ordinary field: editing it reloads
+   * the row, which re-registers the routes with the new proxy access.
+   */
   trustedProxyHosts?: string[]
   /** Environment variable whose value the authenticated proxy injects upstream. */
   proxyTokenEnv?: string
@@ -65,15 +69,54 @@ export interface Config {
   sessionDefaultPermission?: TaskPermission
 }
 
-export const Config: z<Config> = z.object({
-  announceToAgent: z.boolean().default(false),
-  enabled: z.boolean().default(true),
-  preventIdleSleep: z.boolean().default(false),
+/**
+ * The schema is left to inference rather than annotated with `z<Config>`: a
+ * volatile field's parsed output is a `Volatile` reference while its accepted
+ * input stays the plain value, so the two sides no longer share one shape and
+ * the annotation would reject the schema the Host must be given.
+ */
+export const Config = z.object({
+  announceToAgent: z.boolean().default(false).volatile(),
+  enabled: z.boolean().default(true).volatile(),
+  preventIdleSleep: z.boolean().default(false).volatile(),
   trustedProxyHosts: z.array(z.string()).default([]),
   proxyTokenEnv: z.string().min(1).default(DEFAULT_PROXY_TOKEN_ENV),
   sessionDefaultPermission: z.union(TASK_PERMISSIONS).default(DEFAULT_SESSION_PERMISSION),
 })
 
+declare module '@deepseek-ai/cordis' {
+  interface Events {
+    /**
+     * Volatile config values were committed into the running fiber without a
+     * remount; dispatched to the owning fiber only. Spelled here because the
+     * Loader package is not a dependency of this plugin, with the Loader's own
+     * shape so the two declarations merge when a Host program carries both.
+     * @param paths - changed config paths as key arrays; every value is committed before dispatch.
+     * @mode emit
+     */
+    'loader/volatile-update'(paths: readonly (readonly string[])[]): void
+  }
+}
+
+/**
+ * Read one config field's current value.
+ *
+ * The Loader hands schema-volatile fields as stable references it commits in
+ * place, so a live value must be read at use time rather than captured when
+ * the plugin activates; a plain value (a programmatic mount, or a field the
+ * schema does not mark volatile) is returned as it stands.
+ * @param field - the config field as the Loader handed it.
+ * @param fallback - value to use when the field is absent.
+ * @returns the effective field value.
+ */
+export function readConfigField<T>(field: Volatile<T> | T | undefined, fallback: T): T {
+  if (field === undefined) return fallback
+  if (typeof field === 'object' && field !== null && typeof (field as { get?: unknown }).get === 'function') {
+    return (field as Volatile<T>).get() as T
+  }
+  return field as T
+}
+
 /** Resolve proxy access without ever placing the token value in plugin config. */
 export function resolveProxyAccess(config: Config | undefined, env: NodeJS.ProcessEnv = process.env): { trustedProxyHosts: string[]; proxyToken?: string } {
   const trustedProxyHosts = config?.trustedProxyHosts ?? []
@@ -107,16 +150,28 @@ export function resolveLlmRuntime(ctx: Context): LlmRuntime | undefined {
 }
 
 /**
- * Register the board's announcement section, gated on the composition entry's
- * `announceToAgent` (and the live settings value once the web settings
- * surface is served). The section is re-registered whenever the source
- * changes, so a settings edit takes effect without a restart.
+ * Activate the board's host half: ledger, routes, cron scheduler, power
+ * inhibitor, and the model-facing announcement.
+ *
+ * The effective settings are the config the Host hands this row. The three
+ * fields the browser card edits are schema-volatile, so the Loader commits an
+ * edit into the running fiber's references instead of remounting the row —
+ * `sync` therefore reads them at use time and follows
+ * `loader/volatile-update`, the event the Loader emits once it has committed
+ * them. Every other field reloads the row, which re-runs this activation.
  * @param ctx - the plugin context (systemPrompt injected).
  * @param config - resolved plugin config (schema defaults applied by the loader).
  */
 export const apply = mountOnce('@linxin666/dsh-client-ui-task-board', applyImpl)
 
 function applyImpl(ctx: Context, config?: Config): void {
+  /** Current master switch (the browser half reads the same field from its own form). */
+  const enabled = (): boolean => readConfigField(config?.enabled, true)
+  /** Current announcement flag. */
+  const announceToAgent = (): boolean => readConfigField(config?.announceToAgent, DEFAULT_ANNOUNCE)
+  /** Current idle-sleep protection flag. */
+  const preventIdleSleep = (): boolean => readConfigField(config?.preventIdleSleep, false)
+
   const host = new TaskBoardHostService(ctx.typertGateway, {
     workspaceRegistry: ctx.workspaceRegistry,
     sessionDefaultPermission: config?.sessionDefaultPermission ?? DEFAULT_SESSION_PERMISSION,
@@ -128,7 +183,9 @@ function applyImpl(ctx: Context, config?: Config): void {
       },
     },
   })
-  host.setConfiguration(config?.enabled ?? true, config?.preventIdleSleep ?? false)
+  // Configuration before start(): a disabled row must not take the first
+  // scheduler tick, which would roll schedules the board is not running.
+  host.setConfiguration(enabled(), preventIdleSleep())
   host.start()
   ctx.effect(() => {
     const disposers: Array<() => void> = []
@@ -151,24 +208,27 @@ function applyImpl(ctx: Context, config?: Config): void {
       host.dispose()
     }
   }, 'task-board: host ledger, scheduler, and routes')
-  // The live source the announcement reads: the settings section once the web
-  // settings surface is served, the composition entry otherwise
-  // (installSection swaps it when the namespace registers).
-  let current: () => Config = () => config ?? {}
+
   let disposeSection: (() => void) | undefined
+  let applied: { enabled: boolean; announceToAgent: boolean; preventIdleSleep: boolean } | undefined
 
-  // Register (or drop) the announcement to match the current source. The
-  // section is kept under one disposer: re-registering first tears the old
-  // one down so a duplicate-name registration never throws.
+  // Apply the current values to the host service and the announcement. A
+  // commit that changes nothing visible is a no-op, so following a coarse
+  // invalidation cannot churn the system-prompt registry. The section is kept
+  // under one disposer: re-registering first tears the old one down so a
+  // duplicate-name registration never throws.
   const sync = (): void => {
+    const next = { enabled: enabled(), announceToAgent: announceToAgent(), preventIdleSleep: preventIdleSleep() }
+    if (applied !== undefined && applied.enabled === next.enabled && applied.announceToAgent === next.announceToAgent && applied.preventIdleSleep === next.preventIdleSleep) {
+      return
+    }
+    applied = next
+    host.setConfiguration(next.enabled, next.preventIdleSleep)
     if (disposeSection !== undefined) {
       disposeSection()
       disposeSection = undefined
     }
-    const active = current().enabled ?? true
-    host.setConfiguration(active, current().preventIdleSleep ?? false)
-    if (!active) return
-    if ((current().announceToAgent ?? DEFAULT_ANNOUNCE) === false) return
+    if (!next.enabled || !next.announceToAgent) return
     disposeSection = ctx.systemPrompt.section({
       name: 'plugin:task-board',
       order: SECTION_ORDER,
@@ -176,24 +236,9 @@ function applyImpl(ctx: Context, config?: Config): void {
     })
   }
 
-  ctx.inject(['settings'], (settingsCtx) => {
-    try {
-      if (typeof settingsCtx.settings?.installSection === 'function') {
-        settingsCtx.settings.installSection(ctx, TASK_BOARD_SETTINGS_NAMESPACE, Config, config ?? {}, {
-          setSource: (source) => { current = source },
-          onChange: sync,
-        })
-      } else if (typeof settingsCtx.settings?.register === 'function') {
-        const scope = settingsCtx.settings.register(TASK_BOARD_SETTINGS_NAMESPACE, Config, { base: config ?? {} })
-        current = () => scope?.get?.() ?? (config ?? {})
-        scope?.watch?.(() => { sync() })
-      }
-    } catch {
-      // Defensive fallback against settings registration differences
-    }
-  })
-
-  // Initial registration from the composition entry (covers deployments with
-  // no settings service, whose installSection never fires its hooks).
+  // A settings edit of a volatile field is committed into the references this
+  // fiber already holds, with no remount and no second call to apply: the
+  // commit event is what makes the edit take effect without a restart.
+  ctx.on('loader/volatile-update', () => { sync() })
   sync()
 }