@linxin666/dsh-client-ui-task-board 0.4.3 → 0.4.4

package/src/client/index.ts

@@ -372,8 +372,12 @@ export function apply(ctx: ClientContext): void {
         console.error('[dsh-task-board] model options read failed', error)
       }
     }
+    void pushPresetOptions()
     void pushModelOptions()
-    disposers.push(ctx.on('connection/reset', () => { void pushModelOptions() }))
+    disposers.push(ctx.on('connection/reset', () => {
+      void pushPresetOptions()
+      void pushModelOptions()
+    }))
     // Native panel surfaces: one sidebar row and one center-column page, both
     // through the official slot seats the shell itself renders. The layout's
     // panel selection is reconciled back into the controller so the board's own

package/src/client/locales.ts

@@ -29,8 +29,11 @@ export const zh = {
   'board.updated': '更新于',
   'board.created': '创建于',
   'board.hostError': 'Host 操作失败：{error}',
-  'board.hostError.notMounted': '任务看板的后台接口没有挂载：宿主可能没有加载该插件，也可能后台数据正被另一个 DSH 实例占用。重启 DSH 服务后再试',
+  'board.hostError.notMounted': '任务看板的后台接口没有挂载：宿主没有加载这个插件（可能未安装、未启用，或该行启动失败）。重启 DSH 服务后再试',
+  'board.hostError.ledgerLocked': '任务看板的账本已被另一个 DSH 进程占用（占用者 pid {pid}），本实例没有注册后台接口：关闭那个 DSH 实例，或等它退出后再点重试——反复重启本实例无效（宿主原因：{detail}）',
+  'board.hostError.degraded': '任务看板这一行在启动时降级，后台接口没有注册（宿主原因：{detail}）',
   'board.hostError.unauthorized': '登录状态已失效，请刷新页面后重试',
+  'board.hostError.forbidden': '当前环境未通过同源校验：桌面外壳或反向代理可能未传递浏览器标记，也没有携带宿主认证凭据',
   'board.hostError.locked': '任务看板的后台数据被占用：{detail}',
   'board.hostError.timeout': '后台 {seconds} 秒内没有响应，请稍后重试',
   'board.hostError.unreachable': '连接不上本机 DSH 服务，请确认它仍在运行',
@@ -55,6 +58,20 @@ export const zh = {
   'new.aiParseFailed': '解析失败：{error}',
   'board.retryHost': '重试连接 Host',
   'board.hostMeta': 'Host 时区 {timeZone} · revision {revision}',
+  // 新建/编辑对话框的区域分组：折叠后每个区域仍以一行摘要显示当前配置。
+  'new.section.parse': 'AI 解析',
+  'new.section.content': '任务内容',
+  'new.section.labels': '标签',
+  'new.section.execution': '执行设置',
+  'new.section.run': '运行方式',
+  'new.section.handover': '交接与续跑',
+  'new.section.schedule': '定时运行',
+  'new.summary.none': '未设置',
+  'new.summary.labelCount': '{count} 个标签',
+  'new.summary.filled': '已填写',
+  'new.summary.scheduleOff': '未启用',
+  'new.summary.multiRound': '多轮 /goal',
+  'new.summary.singleRound': '单轮',
   'new.title': '标题',
   'new.titlePlaceholder': '一句话描述要做什么',
   'new.description': '描述',
@@ -146,13 +163,31 @@ export const zh = {
   'detail.schedule.invalid': 'Cron 表达式无效',
   'detail.schedule.notScheduled': '尚未排程',
   'detail.schedule.dueSoon': '即将运行',
+  'detail.schedule.timeZone': '时区',
+  'detail.schedule.timeZoneHost': 'Host 时区（{timeZone}）',
+  'detail.schedule.timeZoneHostUnknown': 'Host 时区（未知）',
+  'detail.schedule.timeZoneHint': 'Cron 的墙上时间按此时区解读；夏令时跳变会跳过不存在的时刻，重复的时刻只触发一次。',
+  'detail.schedule.timeZoneSearch': '搜索时区…',
+  'detail.schedule.timeZoneNone': '没有匹配的时区',
+  'detail.schedule.nextRunOverdue': '已逾期 {duration}',
+  'detail.schedule.duration.days': '{count} 天',
+  'detail.schedule.duration.hours': '{count} 小时',
+  'detail.schedule.duration.minutes': '{count} 分钟',
+  'detail.schedule.duration.seconds': '{count} 秒',
   'card.scheduled': '定时',
   'new.workspace': '工作区',
-  'new.mode': '模式',
+  'new.agentPreset': 'Agent 预设',
   'new.permission': '权限',
   'new.model': '模型',
   'exec.workspace.recent': '最近使用（默认）',
-  'exec.mode.default': '部署默认',
+  'exec.mode.inherit': '继承（部署默认）',
+  'exec.mode.inheritWith': '继承（部署默认：{preset}）',
+  'exec.mode.builtinGroup': '内置预设',
+  'exec.mode.customGroup': '自定义预设',
+  'preset.builtin.standard': '标准',
+  'preset.builtin.ptc': 'PTC（程序化工具调用）',
+  'preset.builtin.minimal': '极简',
+  'preset.builtin.cordis': 'Cordis',
   'exec.mode.defaultSuffix': '（默认）',
   'exec.mode.brokenSuffix': '（不可用）',
   'exec.mode.removed': '（已移除）',
@@ -168,8 +203,10 @@ export const zh = {
   'exec.teamRunHint': '开启后：执行本任务只启动一个 Leader 会话，每个子任务由它派生为 teammate 并在该会话内并行工作（子任务自身的权限钉住不适用）。关闭则每个成员各开独立会话。',
   'exec.teamRunUnavailable': '当前部署未提供 Agent Teams 服务，无法启用。',
   'exec.reuseSessionHint': '开启后，本任务的后续执行在上一次会话里继续（该会话空闲且仍存在时），不再每次新建对话；每次复用时都会重新应用上面钉住的权限与模型。',
+  'exec.goalRun': '以 dsh 内置的 /goal 开始执行任务',
+  'exec.goalRunHint': '默认开启：执行时把任务目标作为持久目标交给 dsh 内置的 /goal，会话会自动续跑多轮，直到 agent 标记目标完成；看板在该目标真正结束后才结算本次执行。关闭则只执行一轮普通对话。',
   'detail.executionSettings': '执行设置',
-  'exec.hint': '执行时生效：工作区决定执行会话落在哪个工作区；模式决定会话的 agent 预设；权限经 /permission 命令应用到会话。留空则使用运行时默认。',
+  'exec.hint': '执行时生效：工作区决定执行会话落在哪个工作区；Agent 预设决定会话的组合（工具面与提示词段）；权限经 /permission 命令应用到会话。留空则继承部署默认。',
   // 插件设置卡片（settings.plugin.item 席位）。
   'settings.title': '任务看板',
   'settings.description': '控制 Host 任务看板、agent 播报与运行期间的系统空闲睡眠保护。',
@@ -254,8 +291,11 @@ export const en: Record<keyof typeof zh, string> = {
   'board.updated': 'Updated',
   'board.created': 'Created',
   'board.hostError': 'Host action failed: {error}',
-  'board.hostError.notMounted': 'The task board Host API is not mounted: the Host may not have loaded the plugin, or another DSH instance holds the ledger. Restart the DSH service and try again',
+  'board.hostError.notMounted': 'The task board Host API is not mounted: the Host did not load this plugin (not installed, disabled, or its row failed to start). Restart the DSH service and try again',
+  'board.hostError.ledgerLocked': 'Another DSH process (pid {pid}) holds the task board ledger, so this instance registered no Host API: close that DSH instance, or wait for it to exit and retry. Restarting this instance cannot help (reason: {detail})',
+  'board.hostError.degraded': 'The task board row degraded during start, so its Host API was not registered: {detail}',
   'board.hostError.unauthorized': 'This session is no longer signed in; reload the page and try again',
+  'board.hostError.forbidden': 'The request did not pass the same-origin fence: the desktop shell or reverse proxy may not have forwarded browser credentials',
   'board.hostError.locked': 'The task board ledger is locked: {detail}',
   'board.hostError.timeout': 'The Host did not answer within {seconds}s; try again shortly',
   'board.hostError.unreachable': 'Cannot reach the local DSH service; make sure it is still running',
@@ -280,6 +320,20 @@ export const en: Record<keyof typeof zh, string> = {
   'new.aiParseFailed': 'Parsing failed: {error}',
   'board.retryHost': 'Retry Host connection',
   'board.hostMeta': 'Host time zone {timeZone} · revision {revision}',
+  // Create/edit dialog regions: a collapsed region still shows a one-line summary of its values.
+  'new.section.parse': 'AI Parse',
+  'new.section.content': 'Task Content',
+  'new.section.labels': 'Tags',
+  'new.section.execution': 'Execution Settings',
+  'new.section.run': 'Run Mode',
+  'new.section.handover': 'Handover',
+  'new.section.schedule': 'Scheduled Runs',
+  'new.summary.none': 'Not set',
+  'new.summary.labelCount': '{count} tags',
+  'new.summary.filled': 'Filled',
+  'new.summary.scheduleOff': 'Off',
+  'new.summary.multiRound': 'multi-round /goal',
+  'new.summary.singleRound': 'single round',
   'new.title': 'Title',
   'new.titlePlaceholder': 'What should be done, in one line',
   'new.description': 'Description',
@@ -371,13 +425,31 @@ export const en: Record<keyof typeof zh, string> = {
   'detail.schedule.invalid': 'Invalid cron expression',
   'detail.schedule.notScheduled': 'Not scheduled yet',
   'detail.schedule.dueSoon': 'Due soon',
+  'detail.schedule.timeZone': 'Time zone',
+  'detail.schedule.timeZoneHost': 'Host time zone ({timeZone})',
+  'detail.schedule.timeZoneHostUnknown': 'Host time zone (unknown)',
+  'detail.schedule.timeZoneHint': 'The cron wall clock is read in this zone; daylight-saving gaps are skipped and a repeated time fires once.',
+  'detail.schedule.timeZoneSearch': 'Search time zones…',
+  'detail.schedule.timeZoneNone': 'No matching time zone',
+  'detail.schedule.nextRunOverdue': 'Overdue by {duration}',
+  'detail.schedule.duration.days': '{count}d',
+  'detail.schedule.duration.hours': '{count}h',
+  'detail.schedule.duration.minutes': '{count}m',
+  'detail.schedule.duration.seconds': '{count}s',
   'card.scheduled': 'scheduled',
   'new.workspace': 'Workspace',
-  'new.mode': 'Mode',
+  'new.agentPreset': 'Agent preset',
   'new.permission': 'Permission',
   'new.model': 'Model',
   'exec.workspace.recent': 'Most recent (default)',
-  'exec.mode.default': 'Deployment default',
+  'exec.mode.inherit': 'Inherit (deployment default)',
+  'exec.mode.inheritWith': 'Inherit (deployment default: {preset})',
+  'exec.mode.builtinGroup': 'Built-in presets',
+  'exec.mode.customGroup': 'Custom presets',
+  'preset.builtin.standard': 'Standard',
+  'preset.builtin.ptc': 'PTC (programmatic tool calling)',
+  'preset.builtin.minimal': 'Minimal',
+  'preset.builtin.cordis': 'Cordis',
   'exec.mode.defaultSuffix': ' (default)',
   'exec.mode.brokenSuffix': ' (unavailable)',
   'exec.mode.removed': ' (removed)',
@@ -393,8 +465,10 @@ export const en: Record<keyof typeof zh, string> = {
   'exec.teamRunHint': 'On: running this task starts one Lead session and spawns a teammate per subtask inside it, working in parallel (a subtask permission pin does not apply). Off: every member gets its own independent session.',
   'exec.teamRunUnavailable': 'This deployment serves no Agent Teams service, so team runs cannot be enabled.',
   'exec.reuseSessionHint': 'When on, later runs continue in the previous session (when that session is idle and still exists) instead of starting a new conversation each time; the pinned permission and model above are re-applied on every reuse.',
+  'exec.goalRun': "Start the run with dsh's built-in /goal",
+  'exec.goalRunHint': 'On by default: the run arms dsh built-in /goal with the task objective, so the session keeps working automatic continuation rounds until the agent marks the goal complete, and the board settles the execution only when that goal really ends. Off: one plain turn.',
   'detail.executionSettings': 'Execution Settings',
-  'exec.hint': 'Applied when the task runs: the workspace decides where the execution session lands; the mode composes the session\'s agent preset; the permission is applied through the /permission command. Blank = runtime default.',
+  'exec.hint': 'Applied when the task runs: the workspace decides where the execution session lands; the agent preset composes the session (tool face and prompt sections); the permission is applied through the /permission command. Blank inherits the deployment default.',
   // Plugin settings card (the `settings.plugin.item` seat).
   'settings.title': 'Task Board',
   'settings.description': 'Configure the Host task board, agent announcement, and idle-system-sleep protection while work is pending.',

package/src/client/schedule-zone.ts

@@ -0,0 +1,135 @@
+/**
+ * Shared scheduling helpers for the board's schedule editors (the new-task
+ * dialog and the task detail panel): the time-zone picker inventory, the
+ * relative next-run wording, and the combined absolute + relative label.
+ *
+ * Everything here is browser-side presentation built on the same pure engine
+ * the Host schedules with (`core/schedule.ts`), so the preview a user sees is
+ * computed by the code that will actually arm the rule.
+ */
+import { isValidTimeZone } from '../core/schedule.ts'
+import { t } from './locales.ts'
+
+/** One selectable zone, in the order the picker shows it. */
+export interface ZoneChoice {
+  /** Canonical IANA id, or `''` for the Host-zone entry. */
+  id: string
+  /** Display label. */
+  label: string
+}
+
+/**
+ * The runtime's zone inventory, read once. `Intl.supportedValuesOf` is not
+ * available in every engine, so a missing inventory degrades to `UTC` rather
+ * than offering a list that cannot resolve.
+ *
+ * The inventory is deliberately NOT truncated. A capped list silently omits
+ * whole regions, and because a `<select>` whose value matches no option falls
+ * back to its first option, an omitted zone would make the control show — and
+ * then save — a different zone than the rule actually stores.
+ */
+let inventoryCache: readonly string[] | undefined
+
+function zoneInventory(): readonly string[] {
+  if (inventoryCache !== undefined) return inventoryCache
+  let inventory: readonly string[] = []
+  try {
+    inventory = typeof Intl.supportedValuesOf === 'function' ? Intl.supportedValuesOf('timeZone') : []
+  } catch {
+    inventory = []
+  }
+  inventoryCache = inventory.length > 0 ? inventory : ['UTC']
+  return inventoryCache
+}
+
+/**
+ * Zones offered by the picker: the Host zone first (the default, and the entry
+ * a rule with no stored zone follows), then the runtime's own inventory.
+ *
+ * The Host entry is always present, even before the Host snapshot reports a
+ * zone: it is the entry that stores no zone, so dropping it would leave a rule
+ * unable to express "follow the Host" and would let the control silently
+ * display an unrelated zone.
+ *
+ * A stored zone the inventory does not list (a rule written on another machine
+ * or under a different ICU build) is appended, so opening the editor can never
+ * silently rewrite the rule's zone.
+ * @param hostTimeZone - zone the Host reported for this deployment, when known.
+ * @param storedTimeZone - zone the edited rule already stores, when any.
+ * @returns the de-duplicated choice list.
+ */
+export function zoneChoices(hostTimeZone: string | undefined, storedTimeZone?: string): ZoneChoice[] {
+  const host = hostTimeZone !== undefined && isValidTimeZone(hostTimeZone) ? hostTimeZone : undefined
+  const choices: ZoneChoice[] = []
+  const seen = new Set<string>()
+  choices.push({
+    id: '',
+    label: host === undefined
+      ? t('detail.schedule.timeZoneHostUnknown')
+      : t('detail.schedule.timeZoneHost', { timeZone: host }),
+  })
+  if (host !== undefined) seen.add(host)
+  for (const zone of zoneInventory()) {
+    if (seen.has(zone) || !isValidTimeZone(zone)) continue
+    seen.add(zone)
+    choices.push({ id: zone, label: zone })
+  }
+  if (storedTimeZone !== undefined && !seen.has(storedTimeZone) && isValidTimeZone(storedTimeZone)) {
+    choices.push({ id: storedTimeZone, label: storedTimeZone })
+  }
+  return choices
+}
+
+/** Compact duration parts, largest unit first, with at most two units kept. */
+function durationParts(ms: number): string {
+  const total = Math.max(0, Math.floor(ms / 1000))
+  const days = Math.floor(total / 86400)
+  const hours = Math.floor((total % 86400) / 3600)
+  const minutes = Math.floor((total % 3600) / 60)
+  const seconds = total % 60
+  const parts: string[] = []
+  if (days > 0) parts.push(t('detail.schedule.duration.days', { count: String(days) }))
+  if (hours > 0) parts.push(t('detail.schedule.duration.hours', { count: String(hours) }))
+  if (minutes > 0) parts.push(t('detail.schedule.duration.minutes', { count: String(minutes) }))
+  if (parts.length < 2 && (seconds > 0 || parts.length === 0)) {
+    parts.push(t('detail.schedule.duration.seconds', { count: String(seconds) }))
+  }
+  return parts.slice(0, 2).join(' ')
+}
+
+/**
+ * The relative half of a next-run label: how far away the instant is, or how
+ * far past it the board is running. `now` is passed in rather than sampled so
+ * callers can share one clock reading across a row, and tests stay
+ * deterministic.
+ * @param targetMs - the instant being described.
+ * @param now - the reference instant.
+ * @returns localized relative wording.
+ */
+export function relativeTimeLabel(targetMs: number, now: number): string {
+  const delta = targetMs - now
+  if (delta < 0) return t('detail.schedule.nextRunOverdue', { duration: durationParts(-delta) })
+  return `${durationParts(delta)}`
+}
+
+/**
+ * The full next-run label: the absolute wall clock in the schedule's own zone,
+ * plus the relative distance in parentheses. Showing both matters because a
+ * rule's zone can differ from the reader's, so the wall clock alone is
+ * ambiguous and the distance alone loses the actual moment.
+ * @param targetMs - the instant being described.
+ * @param timeZone - zone to render the absolute half in.
+ * @param formatAbsolute - formatter for the absolute half (keeps Intl caching in the caller).
+ * @param now - the reference instant.
+ * @param translate - translate function, bound so this module stays dependency-light.
+ * @returns the combined label.
+ */
+export function nextRunLabel(
+  targetMs: number,
+  timeZone: string | undefined,
+  formatAbsolute: (ms: number, timeZone?: string) => string,
+  now: number,
+): string {
+  const absolute = formatAbsolute(targetMs, timeZone)
+  return `${absolute} (${relativeTimeLabel(targetMs, now)})`
+}

package/src/core/controller.ts

@@ -17,6 +17,7 @@ import { applyArchiveTask, applyRestoreTask } from './use-cases/task-archive.ts'
 import { applyCreateTask } from './use-cases/task-create.ts'
 import { applyDeleteTask } from './use-cases/task-delete.ts'
 import { applyScheduleNextRun as applyScheduleRollForward, applySetSchedule } from './use-cases/task-schedule.ts'
+import { resolveHostTimeZone } from './schedule.ts'
 import { applySetParent } from './use-cases/task-parent.ts'
 import { applyUpdateTask, type TaskUpdatePatch } from './use-cases/task-update.ts'
 import { DEFAULT_SUBTASK_DEPTH } from './subtask.ts'
@@ -517,8 +518,12 @@ export class BoardController {
    * @param patch - fields to change (absent fields keep their current value).
    * @returns true when applied, false when rejected (invalid cron / unknown task).
    */
-  setSchedule(id: string, patch: { enabled?: boolean; cron?: string }): boolean {
-    const { tasks, applied } = applySetSchedule(this.tasks, id, patch, this.now())
+  setSchedule(id: string, patch: { enabled?: boolean; cron?: string; timeZone?: string | null }): boolean {
+    // The Host zone is the fallback a rule with no stored zone follows: the
+    // mirrored snapshot carries it, and a controller without a Host mirror
+    // (pure client tests) falls back to the process zone.
+    const hostTimeZone = this.hostState?.scheduler.timeZone ?? resolveHostTimeZone()
+    const { tasks, applied } = applySetSchedule(this.tasks, id, patch, this.now(), hostTimeZone)
     if (!applied) return false
     if (this.deps.transport !== undefined) {
       void this.commitRemote({ kind: 'set-schedule', taskId: id, patch }, id)

package/src/core/degraded-reason.ts

@@ -0,0 +1,81 @@
+/**
+ * The family health ledger's degraded-record contract (issues #1528, #1730).
+ *
+ * A family plugin row that fails during apply is recorded by the dsh-web-all
+ * shell as a degraded record; the shell serves the ledger through its
+ * loopback-only `GET /api/dsh-web-all/degraded` route because the failed
+ * plugin could not register any route of its own — the task board whose ledger
+ * lock is held by another DSH process is exactly that case, and without this
+ * the panel could only report "the Host API is not mounted".
+ *
+ * Both halves define this shape: the shell WRITES it (packages/dsh-web-all,
+ * src/degraded.ts — the two packages have no import edge, one is bundled into
+ * the other's profile) and a plugin's browser half PARSES it. The edits move
+ * together.
+ */
+
+/** Where a degraded row failed: module import, plugin shape, or fiber start. */
+export type DegradedStage = 'import' | 'shape' | 'start'
+
+/** One degraded family row as the shell's health route serves it. */
+export interface DegradedRow {
+  /** Real plugin package name from the shell row config. */
+  plugin: string
+  /** Where the failure happened. */
+  stage: DegradedStage
+  /** The shell's one-line reason, for display. */
+  reason: string
+}
+
+/**
+ * The one degraded record naming `plugin`, or undefined when that row is not
+ * degraded (or the payload does not come from the shell's own shape). A
+ * plugin may appear once: the shell keys the ledger by package name and
+ * refreshes the record on every new failure.
+ * @param body - parsed JSON body of the health route.
+ * @param plugin - the real plugin package name to look for.
+ * @returns the degraded row, or undefined when the ledger does not name it.
+ */
+export function degradedRowOf(body: unknown, plugin: string): DegradedRow | undefined {
+  if (typeof body !== 'object' || body === null) return undefined
+  const ledger = (body as { ok?: unknown; degraded?: unknown })
+  if (ledger.ok !== true || !Array.isArray(ledger.degraded)) return undefined
+  for (const entry of ledger.degraded) {
+    if (typeof entry !== 'object' || entry === null) continue
+    const record = entry as { plugin?: unknown; stage?: unknown; reason?: unknown }
+    if (record.plugin !== plugin) continue
+    const stage = record.stage === 'import' || record.stage === 'shape' || record.stage === 'start' ? record.stage : undefined
+    if (stage === undefined) continue
+    return { plugin, stage, reason: typeof record.reason === 'string' ? record.reason : '' }
+  }
+  return undefined
+}
+
+/** The ledger owner pid a task-board lock reason names, when it names one. */
+const OWNED_BY_PROCESS_RE = /already owned by process\s+(\d+)/
+
+/**
+ * The marker the ledger appends when it could NOT confirm the recorded PID
+ * belongs to the lock's real owner: the PID was reused by an unrelated process
+ * after a crash, and the reason then carries the manual-removal step instead.
+ * That reason must be rendered verbatim — telling the user to close "the other
+ * DSH instance" would send them after a process that never touched the board.
+ */
+const UNCONFIRMED_OWNER_MARKER = 'reused after a crash'
+
+/**
+ * Extract the pid of the LIVE process a ledger-lock failure blames, when the
+ * reason identifies one. The task board's lock refusal is the one degraded
+ * reason that carries an owner the user can act on, so the panel renders the
+ * pid in its own sentence and adds the guidance that actually applies.
+ * @param reason - the shell's one-line failure reason.
+ * @returns the confirmed owner pid, or undefined when the reason names none or
+ *   warns that the PID may have been reused (its own text owns the recovery).
+ */
+export function ledgerOwnerPid(reason: string): number | undefined {
+  if (reason.includes(UNCONFIRMED_OWNER_MARKER)) return undefined
+  const match = OWNED_BY_PROCESS_RE.exec(reason)
+  if (match === null) return undefined
+  const pid = Number(match[1])
+  return Number.isSafeInteger(pid) && pid > 0 ? pid : undefined
+}

package/src/core/package-name.ts

@@ -0,0 +1,9 @@
+/**
+ * npm identity of this plugin. The dsh-web-all degraded ledger is keyed by the
+ * real plugin package name of a family row, and this plugin's browser half
+ * must name its OWN row there to learn why its Host API is missing (issue
+ * #1730): the served bundle cannot derive the name, because a family row
+ * mounts the package from a versioned profile path. The host half spells the
+ * same name in its single-mount guard (src/index.ts); the two move together.
+ */
+export const TASK_BOARD_PACKAGE = '@linxin666/dsh-client-ui-task-board'