dsh-taskboard 0.6.6 → 0.7.0

package/src/index.ts

@@ -1,7 +1,7 @@
 /**
  * Host loader entry for dsh-taskboard.
  *
- * Wiring: the ledger store (one JSON file under the DSH home), the ten
+ * Wiring: the configurable local data stores, the ten
  * `taskboard_*` agent tools, the agent workflow-protocol system-prompt
  * section, the /taskboard JSON+SSE routes (when a webServer is served),
  * the host execution service (fresh in-project sessions, pinned models), and
@@ -29,14 +29,19 @@ import { dshHomePath } from './host/sdk.ts'
 import { TaskStore } from './host/store.ts'
 import { TemplateStore } from './host/templates.ts'
 import { ExternalSessionSyncService } from './host/session-sync.ts'
-import { registerTaskboardTools, workspaceFace } from './host/tools.ts'
+import { ERR, ToolError, registerTaskboardTools, workspaceFace, type WorkspaceFace } from './host/tools.ts'
+import { AssetStore } from './host/assets.ts'
+import { STORAGE_CONFIG_FILE, StorageCoordinator } from './host/storage.ts'
 
-/** Ledger file name under the DSH home. */
+/** Ledger file name under the active taskboard data directory. */
 export const LEDGER_FILE = 'dsh-taskboard.json'
 
-/** Task-template side file name under the DSH home (0.4.0). */
+/** Task-template side file name under the active data directory. */
 export const TEMPLATES_FILE = 'dsh-taskboard-templates.json'
 
+/** Content-addressed image attachment directory under the active data directory. */
+export const ASSETS_DIR = 'dsh-taskboard-assets'
+
 /** Cordis plugin name. */
 export const name = 'dsh-taskboard'
 
@@ -48,14 +53,24 @@ export const inject = ['tools', 'systemPrompt']
  * @param ctx - the plugin context (tools + systemPrompt injected).
  */
 export function apply(ctx: Context): void {
-  const store = new TaskStore({ file: dshHomePath(LEDGER_FILE) })
-  const templates = new TemplateStore(dshHomePath(TEMPLATES_FILE))
+  const storage = new StorageCoordinator({
+    defaultDirectory: dshHomePath(),
+    configFile: dshHomePath(STORAGE_CONFIG_FILE),
+    ledgerName: LEDGER_FILE,
+    templatesName: TEMPLATES_FILE,
+    assetsName: ASSETS_DIR,
+  })
+  const store = new TaskStore({ file: storage.ledgerPath(), queue: storage.queue })
+  const templates = new TemplateStore(storage.templatesPath(), storage.queue)
+  const assets = new AssetStore(storage.assetsPath(), () => Date.now(), storage.queue)
+  storage.attach({ ledger: store, templates, assets })
   // Eager first load: the tools and most routes read snapshot()/get() without
   // triggering the lazy load, so a fresh boot used to serve an EMPTY board to
   // taskboard_list/get until the scheduler catchup tick or the first
   // GET /state happened to load the file (review P0). load() never throws —
   // a corrupt ledger is quarantined instead.
-  void store.load()
+  const storeReady = storage.ready().then(() => store.load())
+  void storeReady.then(() => assets.cleanup(JSON.stringify(store.snapshot())))
   const now = () => Date.now()
   // Global execution concurrency cap (DSH_TASKBOARD_MAX_CONCURRENT overrides).
   const maxConcurrent = Math.max(1, Number.parseInt(process.env.DSH_TASKBOARD_MAX_CONCURRENT ?? '', 10) || DEFAULT_MAX_CONCURRENT)
@@ -68,29 +83,60 @@ export function apply(ctx: Context): void {
   })
   ctx.effect(() => disposeSection, 'dsh-taskboard: protocol section')
 
-  // Tools, routes, execution, and the scheduler all come up with the
-  // workspace registry (claim boundary + project execution need it).
-  ctx.inject(['workspaceRegistry'], (wsCtx: Context) => {
-    const disposers: Array<() => void> = []
-
-    // Registered model provider routes (from the host llm runtime), read
-    // lazily at call time so late availability still applies; undefined when
-    // the runtime is absent → only structural model validation runs.
-    const modelProviders = (): string[] | undefined => {
-      try {
-        const llm = wsCtx.get('llm') as { listProviders?: () => Array<{ id: string }> } | undefined
-        return llm === undefined || typeof llm.listProviders !== 'function'
-          ? undefined
-          : llm.listProviders().map(p => p.id)
-      } catch { return undefined }
+  // Register the complete tool schema in the same synchronous mount as the
+  // protocol. Keeping schemas stable from the first request preserves the
+  // provider's prefix cache; calls use the live workspace service below.
+  let activeWorkspaces: WorkspaceFace | undefined
+  let activeWorkspaceContext: Context | undefined
+  const requireWorkspaces = (): WorkspaceFace => {
+    if (activeWorkspaces === undefined) {
+      throw new ToolError(ERR.notReady, 'workspace service is not ready; retry after host startup completes')
     }
+    return activeWorkspaces
+  }
+  const workspaces: WorkspaceFace = {
+    resolveByPath: path => requireWorkspaces().resolveByPath(path),
+    get: id => requireWorkspaces().get(id),
+    list: () => requireWorkspaces().list(),
+  }
+  // Preserve the optional archive capability without replacing the stable
+  // facade captured by the tool definitions.
+  Object.defineProperty(workspaces, 'archiveSession', {
+    enumerable: true,
+    get: () => activeWorkspaces?.archiveSession === undefined
+      ? undefined
+      : (sessionId: string) => requireWorkspaces().archiveSession!(sessionId),
+  })
+  const modelProviders = (): string[] | undefined => {
+    try {
+      const llm = activeWorkspaceContext?.get('llm') as { listProviders?: () => Array<{ id: string }> } | undefined
+      return llm === undefined || typeof llm.listProviders !== 'function'
+        ? undefined
+        : llm.listProviders().map(p => p.id)
+    } catch { return undefined }
+  }
+  const disposeTools = registerTaskboardTools(ctx, {
+    store,
+    workspaces,
+    now,
+    modelProviders,
+    ready: async () => {
+      if (activeWorkspaces === undefined) {
+        throw new ToolError(ERR.notReady, 'workspace service is not ready; retry after host startup completes')
+      }
+      await storeReady
+    },
+  })
+  ctx.effect(() => () => {
+    for (const dispose of disposeTools.splice(0)) dispose()
+  }, 'dsh-taskboard: tools')
 
-    disposers.push(...registerTaskboardTools(wsCtx, {
-      store,
-      workspaces: workspaceFace(wsCtx.workspaceRegistry),
-      now,
-      modelProviders,
-    }))
+  // Runtime services come and go with the workspace registry. Tool schemas
+  // remain mounted and resolve this current service only when called.
+  ctx.inject(['workspaceRegistry'], (wsCtx: Context) => {
+    const workspaceDisposers: Array<() => void> = []
+    activeWorkspaces = workspaceFace(wsCtx.workspaceRegistry)
+    activeWorkspaceContext = wsCtx
 
     // Settlement listener over the session event bus.
     const events: EventsFace = {
@@ -122,7 +168,7 @@ export function apply(ctx: Context): void {
       },
       now,
     })
-    disposers.push(() => sessionSync.dispose())
+    workspaceDisposers.push(() => sessionSync.dispose())
 
     // The narrow git face shared by execution (worktree isolation) and the
     // routes (merge / remove / workspace detection), plus the shared
@@ -131,6 +177,7 @@ export function apply(ctx: Context): void {
     const scanner = createRepoScanner()
 
     wsCtx.inject(['agents'], (agentCtx: Context) => {
+      const agentDisposers: Array<() => void> = []
       agentSessions = agentCtx.get('sessions') as { get?: (id: string) => unknown; list?: () => unknown[] } | undefined
       const execution = new ExecutionService({
         store,
@@ -207,6 +254,9 @@ export function apply(ctx: Context): void {
           git,
           scanner,
           templates,
+          assets,
+          storage,
+          ready: async () => { await storeReady },
           promptCompletions: async () => {
             try {
               const skillsService = agentCtx.get('skills') as { list?(options?: unknown): Promise<Array<{ name: string; description?: string }>> } | undefined
@@ -307,19 +357,24 @@ export function apply(ctx: Context): void {
       // browser open. Shares the execution concurrency cap.
       const scheduler = new SchedulerService({ store, execution, now, maxConcurrent })
       scheduler.start()
-      disposers.push(() => scheduler.dispose())
+      agentDisposers.push(() => scheduler.dispose())
       // Detach the settlement listener with the plugin — a hot reload must
       // not leave stale services reacting to turn/end errors (review P1).
-      disposers.push(() => execution.dispose())
+      agentDisposers.push(() => execution.dispose())
 
       return () => {
         disposeRoutes?.()
-        for (const dispose of disposers.splice(0)) dispose()
+        agentSessions = undefined
+        for (const dispose of agentDisposers.splice(0)) dispose()
       }
     })
 
     return () => {
-      for (const dispose of disposers.splice(0)) dispose()
+      if (activeWorkspaceContext === wsCtx) {
+        activeWorkspaceContext = undefined
+        activeWorkspaces = undefined
+      }
+      for (const dispose of workspaceDisposers.splice(0)) dispose()
     }
   })
 }

package/src/shared/api.ts

@@ -38,7 +38,7 @@ export type ApiResult<T> = ApiOk<T> | ApiFail
 // ---------------------------------------------------------------------------
 
 /** Full-state response (the reconnect baseline after an SSE gap). */
-export type StateResponse = TaskLedger
+export type StateResponse = TaskLedger & { capabilities?: { archiveSessions: boolean } }
 
 /**
  * Workspace listing for the UI pickers. `repoCount` (0.6.3): how many repos a
@@ -89,7 +89,10 @@ export type UpdateTaskBody = {
 }
 
 /** Move-task request body (ifVersion mandatory; the user MAY move to done). */
-export type MoveTaskBody = { ifVersion: number; status: string }
+export type MoveTaskBody = { ifVersion: number; status: string; archiveSessions?: boolean }
+
+export type SessionArchiveResult = { archived: string[]; failed: Array<{ sessionId: string; error: string }>; unsupported: string[] }
+export type MoveTaskResponse = TaskSummary & { sessionArchive?: SessionArchiveResult }
 
 /**
  * Quick-reject request body (card ✗ button): move back to todo plus an
@@ -101,6 +104,16 @@ export type RejectTaskBody = { ifVersion: number; body?: string }
 /** Comment request body. */
 export type CommentBody = { body: string }
 
+/** One content-addressed image uploaded outside the ledger. */
+export type AttachmentUpload = {
+  id: string
+  name: string
+  size: number
+  url: string
+  extension: 'png' | 'jpg' | 'gif' | 'webp'
+  mime: 'image/png' | 'image/jpeg' | 'image/gif' | 'image/webp'
+}
+
 /** Delete request body (purge=true physically removes a trashed task). */
 export type DeleteTaskBody = { ifVersion?: number; purge?: boolean }
 
@@ -184,6 +197,22 @@ export type TemplatesResponse = { templates: TaskTemplate[] }
 /** Board-settings response (0.5.0; absent fields follow factory defaults). */
 export type SettingsResponse = BoardSettings
 
+/** Current host-side location of all durable taskboard data. */
+export type StorageStatus = {
+  currentDirectory: string
+  defaultDirectory: string
+  isDefault: boolean
+  configured: boolean
+  writable: boolean
+  assetCount: number
+  assetBytes: number
+  checkedDirectory?: string
+  error?: string
+}
+
+/** Completed storage relocation, including non-fatal old-file cleanup failures. */
+export type StorageMigrationResult = StorageStatus & { migrated: boolean; warnings: string[] }
+
 /** Update-board-settings request body (0.5.0; whole-object replace semantics). */
 export type UpdateSettingsBody = {
   /** Default code isolation for NEW tasks ('worktree' | 'none'). */
@@ -270,4 +299,4 @@ export type ChangeEvent = {
   revision: number
   kind: 'task-created' | 'task-updated' | 'task-moved' | 'task-deleted' | 'comment-added' | 'execution-recorded' | 'settings-updated' | 'ledger-replaced'
   tasks: TaskSummary[]
-}
+}

package/src/shared/builtin-templates.ts

@@ -0,0 +1,153 @@
+/**
+ * Built-in task-template content (0.6.4): the four factory templates seeded
+ * into the template side file, in both shipped locales. The host seeds the zh
+ * copy as the persisted fallback (the side file is plain data); the client
+ * resolves the ACTIVE locale's copy at render / prefill time via
+ * {@link builtinTemplateContent}, so an English GUI shows English template
+ * text without ever re-seeding the side file.
+ *
+ * Kept in shared/ (not the client i18n dictionaries) because the content is
+ * structured task DATA — a checklist is an array and the prompt is multi-line
+ * — rather than UI-chrome strings, and because the host needs the same seed
+ * content to write the side file.
+ *
+ * @module dsh-taskboard/shared/builtin-templates
+ */
+import type { TaskTemplateSpec } from './api.ts'
+
+/** Shipped built-in template locales (mirrors the client i18n LocaleId). */
+export type BuiltinTemplateLocale = 'zh' | 'en'
+
+/** One built-in template's localized name + task spec. */
+export interface BuiltinTemplateContent {
+  name: string
+  task: TaskTemplateSpec
+}
+
+/** Stable ids of the factory templates (persisted in the side file). */
+export const BUILTIN_TEMPLATE_IDS = ['tpl-feature', 'tpl-bugfix', 'tpl-release', 'tpl-patrol'] as const
+
+export type BuiltinTemplateId = (typeof BUILTIN_TEMPLATE_IDS)[number]
+
+/** Built-in template content per locale (name + prefilled task spec). */
+export const BUILTIN_TEMPLATE_CONTENT: Readonly<Record<BuiltinTemplateLocale, Readonly<Record<BuiltinTemplateId, BuiltinTemplateContent>>>> = {
+  zh: {
+    'tpl-feature': {
+      name: '新增功能',
+      task: {
+        title: '新增：',
+        prompt: [
+          '实现以上新功能并按序交接：',
+          '1. 明确需求边界与验收标准，列出实现要点',
+          '2. 实现功能（含类型定义与错误处理）',
+          '3. 补充测试（单测/回归）',
+          '4. 运行相关测试套件确认通过',
+        ].join('\n'),
+        urgency: 'normal',
+        checklist: ['实现要点已明确（需求边界与验收标准）', '功能已实现并补充测试', '相关测试套件通过'],
+      },
+    },
+    'tpl-bugfix': {
+      name: 'Bug 修复',
+      task: {
+        title: '修复：',
+        prompt: [
+          '修复以上问题并按序交接：',
+          '1. 复现问题（写最小复现步骤或测试）',
+          '2. 定位根因，说明为什么会发生',
+          '3. 修复并补回归测试',
+          '4. 运行相关测试套件确认无回归',
+        ].join('\n'),
+        urgency: 'urgent',
+        checklist: ['已复现并定位根因', '修复已提交到任务分支', '回归测试通过'],
+      },
+    },
+    'tpl-release': {
+      name: '发布检查',
+      task: {
+        title: '发布：',
+        prompt: '执行发布流程：版本号更新、构建、测试、变更记录，完成后按序交接（不要实际推送/发布，等用户确认）。',
+        urgency: 'normal',
+        checklist: ['版本号已更新（package.json 与版本常量同步）', '构建通过', '全部测试通过', '变更记录已写'],
+      },
+    },
+    'tpl-patrol': {
+      name: '例行巡检',
+      task: {
+        title: '巡检：',
+        prompt: [
+          '例行巡检：检查依赖更新、失败测试、明显代码问题与未处理的告警。',
+          '发现的问题逐条列出（严重度/位置/建议），小问题直接修复，大问题只报告不动手。',
+          '输出巡检摘要（用 {{lastComments}} 可回看上次巡检结论）。',
+        ].join('\n'),
+        urgency: 'relaxed',
+        execution: { mode: 'scheduled', cron: '0 9 * * 1' },
+      },
+    },
+  },
+  en: {
+    'tpl-feature': {
+      name: 'New feature',
+      task: {
+        title: 'New feature:',
+        prompt: [
+          'Implement the new feature above and hand off in order:',
+          '1. Clarify the requirement boundaries and acceptance criteria; list the implementation points',
+          '2. Implement the feature (including type definitions and error handling)',
+          '3. Add tests (unit / regression)',
+          '4. Run the relevant test suites and confirm they pass',
+        ].join('\n'),
+        urgency: 'normal',
+        checklist: ['Implementation points clarified (requirement boundaries and acceptance criteria)', 'Feature implemented with tests added', 'Relevant test suites pass'],
+      },
+    },
+    'tpl-bugfix': {
+      name: 'Bug fix',
+      task: {
+        title: 'Fix:',
+        prompt: [
+          'Fix the issue above and hand off in order:',
+          '1. Reproduce the issue (write minimal reproduction steps or a test)',
+          '2. Locate the root cause and explain why it happens',
+          '3. Fix it and add regression tests',
+          '4. Run the relevant test suites and confirm there are no regressions',
+        ].join('\n'),
+        urgency: 'urgent',
+        checklist: ['Issue reproduced and root cause located', 'Fix committed to the task branch', 'Regression tests pass'],
+      },
+    },
+    'tpl-release': {
+      name: 'Release check',
+      task: {
+        title: 'Release:',
+        prompt: 'Run the release process: bump the version, build, test, and update the changelog, then hand off in order (do not actually push / publish — wait for user confirmation).',
+        urgency: 'normal',
+        checklist: ['Version bumped (package.json and version constants in sync)', 'Build passes', 'All tests pass', 'Changelog written'],
+      },
+    },
+    'tpl-patrol': {
+      name: 'Routine patrol',
+      task: {
+        title: 'Patrol:',
+        prompt: [
+          'Routine patrol: check dependency updates, failing tests, obvious code issues, and unhandled alerts.',
+          'List each finding (severity / location / suggestion); fix small issues directly, and only report (do not touch) large ones.',
+          'Output a patrol summary (use {{lastComments}} to review the last patrol\u2019s conclusions).',
+        ].join('\n'),
+        urgency: 'relaxed',
+        execution: { mode: 'scheduled', cron: '0 9 * * 1' },
+      },
+    },
+  },
+}
+
+/**
+ * Resolve one built-in template's localized content.
+ * @param id - the template id.
+ * @param locale - the requested locale.
+ * @returns the content, or undefined when `id` is not a built-in template id.
+ */
+export function builtinTemplateContent(id: string, locale: BuiltinTemplateLocale): BuiltinTemplateContent | undefined {
+  if (!(BUILTIN_TEMPLATE_IDS as readonly string[]).includes(id)) return undefined
+  return BUILTIN_TEMPLATE_CONTENT[locale][id as BuiltinTemplateId]
+}

package/src/shared/protocol.ts

@@ -335,6 +335,15 @@ export type Actor =
   | { kind: 'agent'; sessionId: string }
   | { kind: 'system' }
 
+/** Structured row of a multi-repo merge system comment (0.6.4). */
+export type SystemCommentRow = {
+  /** Repo path relative to the workspace ('' = the workspace root repo). */
+  repo: string
+  outcome: 'merged' | 'noop' | 'failed'
+  /** Failure reason (verbatim) when outcome = 'failed'. */
+  error?: string
+}
+
 /** A progress/report comment on a task. */
 export type CommentRecord = {
   id: string
@@ -345,6 +354,16 @@ export type CommentRecord = {
   createdAt: number
   /** The session that wrote this comment; absent for user-written ones. */
   threadId?: string
+  /**
+   * i18n key of a host-generated system message (0.6.4). The GUI localizes it
+   * at render time; `body` stays a zh fallback for agent tools / CSV / raw
+   * JSON views.
+   */
+  systemKey?: string
+  /** Flat {name} interpolation params for the system message. */
+  systemParams?: Record<string, string>
+  /** Structured per-repo rows for the multi-repo merge summary (0.6.4). */
+  systemRows?: SystemCommentRow[]
 }
 
 /** One commit produced by an isolated execution (hash + subject). */
@@ -756,6 +775,36 @@ export function syncClaim(task: TaskRecord, to: TaskStatus, now: number, holder?
   }
 }
 
+/**
+ * Collect unique execution session IDs associated with a task:
+ * - executions with a non-empty `sessionId`
+ * Creator and claim sessions may serve other tasks and are never included.
+ * @param task - the task record to inspect.
+ * @returns an array of distinct session IDs in stable discovery order.
+ */
+export function taskAssociatedSessionIds(task: TaskRecord): string[] {
+  const seen = new Set<string>()
+  const result: string[] = []
+  const push = (raw: unknown) => {
+    if (typeof raw === 'string') {
+      const trimmed = raw.trim()
+      if (trimmed.length > 0 && !seen.has(trimmed)) {
+        seen.add(trimmed)
+        result.push(trimmed)
+      }
+    }
+  }
+
+  if (Array.isArray(task.executions)) {
+    for (const ex of task.executions) {
+      if (ex !== null && typeof ex === 'object') {
+        push((ex as { sessionId?: unknown }).sessionId)
+      }
+    }
+  }
+  return result
+}
+
 /**
  * Validate and normalize a pinned model: `{ provider, model, reasoningEffort? }`,
  * provider and model must be non-empty trimmed strings.
@@ -1020,6 +1069,21 @@ export function validateImportedTask(raw: unknown, now: number): { ok: true; tas
           version: numOr(ce, 'version', 1),
           createdAt: numOr(ce, 'createdAt', now),
           ...(typeof ce.threadId === 'string' ? { threadId: ce.threadId } : {}),
+          ...(typeof ce.systemKey === 'string' && /^sys\.[A-Za-z0-9]+$/.test(ce.systemKey) && ce.systemKey.length <= 100
+            ? {
+                systemKey: ce.systemKey,
+                ...(typeof ce.systemParams === 'object' && ce.systemParams !== null && !Array.isArray(ce.systemParams)
+                  ? { systemParams: Object.fromEntries(Object.entries(ce.systemParams).filter(([key, value]) => key.length <= 100 && typeof value === 'string' && value.length <= 4000).slice(0, 20)) as Record<string, string> }
+                  : {}),
+                ...(Array.isArray(ce.systemRows)
+                  ? { systemRows: ce.systemRows.filter((row): row is SystemCommentRow => typeof row === 'object' && row !== null
+                      && typeof row.repo === 'string' && (row.repo === '' || isValidRelRepoPath(row.repo))
+                      && ['merged', 'noop', 'failed'].includes(row.outcome)
+                      && (row.error === undefined || typeof row.error === 'string'))
+                    .slice(0, MAX_MIRROR_REPOS).map(row => ({ repo: row.repo, outcome: row.outcome, ...(row.error !== undefined ? { error: row.error.slice(0, 4000) } : {}) })) }
+                  : {}),
+              }
+            : {}),
         })
       }
     } else return fail('comments must be an array')

package/src/shared/version.ts

@@ -1,9 +1,9 @@
-/**
- * The plugin package version shown in the board UI. Kept in sync with
- * package.json by a regression test (tests lock drift).
- *
- * @module dsh-taskboard/shared/version
- */
-
-/** The package version (must equal package.json "version"). */
-export const PLUGIN_VERSION = '0.6.6'
+/**
+ * The plugin package version shown in the board UI. Kept in sync with
+ * package.json by a regression test (tests lock drift).
+ *
+ * @module dsh-taskboard/shared/version
+ */
+
+/** The package version (must equal package.json "version"). */
+export const PLUGIN_VERSION = '0.7.0'