@goodandready/dsh-kanban 0.1.29 → 0.1.31

package/docs/design/DESIGN.md

@@ -59,3 +59,9 @@
   - Structured Report: фиксация отчёта выполнения (summary, changedFiles, checksRun, artifacts, risks) в карточке задачи.
   - Двусторонний цикл приёмки: явные действия человека Accept (в done) и Reject (возврат в in-progress с обязательной причиной rejectReason, инжектируемой в следующую сессию агента).
   - Ветка комментариев к задаче с поддержкой ролей user и agent для сквозного диалога прямо в Task Drawer.
+
+- 2026-09-08 — Стабильность сессий и воркдеревьев (#227, #226, #213, #215):
+  - Привязка проекта сессии: при создании агента с воркдеревом поле meta.cwd сохраняет канонический projectRoot (repoDir), предотвращая выпадение сессий в категорию «Ungrouped» в сайдбаре DSH; путь воркдерева передается в meta.worktreePath и инструкциях агента (#227).
+  - Режим «↻ Продолжить» (Resume Execution): эндпоинт POST /dsh-kanban/tasks/:id/resume и кнопка в TaskDrawer; сохраняет существующее воркдерево, ветку и незакоммиченные файлы, передавая агенту преамбулу продолжения и актуальный PROGRESSDUMP (#226).
+  - Фильтрация structural noise в проверках dirtyFiles: в checkWorktreeDirty добавлен флаг --ignore-submodules=all, исключающий ложные блокировки удаления воркдеревьев из-за субмодулей и дрейфа gitlinks (#213).
+  - Фильтрация сессий субагентов: сессии с маркером subagent (origin === 'subagent' или parentId !== undefined) исключаются из liveSessions и не перезаписывают состояние корневых задач (#215).

package/docs/plans/2026-09-08-session-worktree-stability.md

@@ -0,0 +1,33 @@
+# План: Вектор 1 — Стабильность сессий и воркдеревьев (#227, #226, #213, #215)
+
+## 1. Контекст и цели
+Повышение отказоустойчивости жизненного цикла сессий задач в DSH Kanban:
+- Сохранение привязки сессии к проекту в левом сайдбаре DSH (meta.cwd = projectRoot) при изоляции в воркдереве (#227).
+- Возможность продолжить работу над задачей («↻ Продолжить / Resume») в существующем воркдереве без деструктивного сброса ветки (#226).
+- Игнорирование структурного шума субмодулей и gitlink drift в проверках чистоты воркдерева checkWorktreeDirty (#213).
+- Исключение дочерних сессий субагентов (invoke_subagent) из списка живых сессий и трекинга карточек (#215).
+
+## 2. Архитектурные изменения
+- **lib/worktree.js**:
+  - checkWorktreeDirty: вызов gitRunner(['status', '--porcelain', '--ignore-submodules=all'], worktreePath) для устранения ложного dirty-статуса от субмодулей.
+- **lib/launcher.js**:
+  - obtainAgent: meta.cwd фиксируется как projectRoot, meta.worktreePath = worktreePath.
+  - 
+  - liveSessions: фильтрация сессий с origin === 'subagent' или parentId !== undefined.
+- **lib/lifecycle.js**:
+  - handleSessionEvent: игнорирование событий от сессий-субагентов.
+- **lib/routes.js**:
+  - Эндпоинт POST /tasks/:id/resume.
+- **lib/index.js**:
+  - Регистрация маршрута /dsh-kanban/tasks/:id/resume.
+- **lib/client.js**:
+  - Кнопка «↻ Resume» в TaskDrawer.
+
+## 3. Верификация
+- Юнит-тесты: 	est/worktree-dirty.test.mjs, 	est/launcher-cwd.test.mjs, 	est/resume.test.mjs, 	est/subagent-filter.test.mjs.
+- Полный прогон 
+pm test и 
+pm run compat.
+- Проверка пакета 
+pm pack --dry-run --json.
+- Тест на MiniPC test runner через dsh-test-plugin.

package/lib/client.js

@@ -1095,6 +1095,7 @@ window.__ModuleLoader__.load({
       'panel.workDirHint': 'The project root is not set in settings, so this folder is used.',
       'panel.worktree': 'Worktree: {path}',
       'panel.cleanupWorktree': 'Clean Worktree',
+      'action.resume': '↻ Resume',
       'panel.worktreeDirty': 'Uncommitted changes in worktree: {files}',
       'panel.priority': 'Priority',
       'panel.priorityNone': 'No priority',
@@ -1416,6 +1417,7 @@ window.__ModuleLoader__.load({
       'panel.workDirHint': '设置中未指定项目根目录，因此使用此目录。',
       'panel.worktree': '工作区：{path}',
       'panel.cleanupWorktree': '清理工作区',
+      'action.resume': '↻ 继续执行',
       'panel.worktreeDirty': '工作区中存在未提交的更改：{files}',
       'panel.priority': '优先级',
       'panel.priorityNone': '无优先级',
@@ -3221,6 +3223,33 @@ window.__ModuleLoader__.load({
               }, t('panel.cleanupWorktree'))
               : null,
             ),
+            (openTask.worktree || openTask.rejectReason || (Array.isArray(openTask.runs) && openTask.runs.length > 0)) ? React.createElement('button', {
+              type: 'button',
+              className: 'dkb-secondary',
+              style: { marginRight: '8px', border: '1px solid var(--dsw-alias-border-color-primary, #444)' },
+              disabled: starting || model === '',
+              onClick: async () => {
+                setStarting(true)
+                try {
+                  rememberLaunch(board, { provider, model, agentPreset, permission })
+                  const out = await api('/task/' + encodeURIComponent(openTask.id) + '/resume', {
+                    method: 'POST',
+                    body: JSON.stringify({ provider, model, text: draftText, agentPreset, permission }),
+                  })
+                  setOpenTask(null)
+                  await load(board)
+                  if (!openSession(props.ctx, out && out.sessionId)) {
+                    setError(t('error.sessionNotOpened', { id: (out && out.sessionId) || '' }))
+                    return
+                  }
+                  if (props.onClose) props.onClose()
+                } catch (e) {
+                  setError(t(e.key || 'error.unknown'))
+                } finally {
+                  setStarting(false)
+                }
+              },
+            }, t('action.resume')) : null,
             React.createElement('button', {
               type: 'button', className: 'dkb-save', disabled: starting || model === '',
               onClick: async () => {

package/lib/index.js

@@ -268,6 +268,7 @@ export function apply(ctx, config) {
       handleSessionEvent({
         store,
         sessionId: String(session?.id ?? ''),
+        session,
         type: event?.type,
         text,
         error: event?.error,
@@ -927,6 +928,39 @@ export function apply(ctx, config) {
           reply(res, { task: updated })
           return
         }
+        if (req.method === 'POST' && action === 'resume') {
+          const task = store.getTask(id)
+          if (task === undefined) { writeJson(res, 404, { error: 'task-not-found' }); return }
+          const fallback = ctx.get('agentDefaultModel')?.currentSelection?.()
+          const picked = resolveModel({
+            requested: { model: input?.model || task.model, provider: input?.provider || task.provider },
+            fallback,
+          })
+          if (picked.error) { reply(res, picked); return }
+          const out = await runTask({
+            agents: ctx.agents,
+            store,
+            task,
+            config: getConfig(),
+            provider: picked.provider,
+            model: picked.model,
+            mintSessionId,
+            createMessage,
+            cwdOf,
+            logger: ctx.logger,
+            text: input?.text,
+            agentPreset: input?.agentPreset || task.agentPreset,
+            permission: input?.permission || task.permission,
+            permissions: ctx.permissions,
+            claimAs: req.headers['x-dsh-user'],
+            gitRunner,
+            workspaceRegistry: ctx.get('workspaceRegistry'),
+            resume: true,
+          })
+          syncPower()
+          reply(res, out)
+          return
+        }
         if (req.method === 'POST' && action === 'confirm-permission') {
           reply(res, confirmPermission({ store, id }))
           return

package/lib/launcher.js

@@ -171,6 +171,7 @@ export async function obtainAgent({
   }
 
   let cwd = resolveCwd(task, config, cwdOf)
+  const projectRoot = cwd
   let worktreePath = task?.worktree || undefined
   let branchName = task?.branch || undefined
   let workspaceId = task?.workspaceId || undefined
@@ -189,7 +190,6 @@ export async function obtainAgent({
         worktreePath = created.worktreePath
         branchName = created.branchName
         worktreeCreated = created.created === true
-        cwd = created.worktreePath
 
         if (workspaceRegistry) {
           workspaceId = await registerDshWorkspace({
@@ -202,11 +202,12 @@ export async function obtainAgent({
     } catch (err) {
       logger?.warn?.(`dsh-kanban: не удалось создать worktree для ${task.id}: ${err.message}`)
     }
-  } else if (worktreePath) {
-    cwd = worktreePath
   }
 
-  const meta = { cwd }
+  // meta.cwd сохраняет projectRoot, чтобы DSH группировал сессию внутри проекта (#227)
+  const meta = { cwd: projectRoot, projectRoot }
+  if (worktreePath) meta.worktreePath = worktreePath
+  if (branchName) meta.branch = branchName
   if (typeof agentPreset === 'string' && agentPreset !== '') meta.agentPreset = agentPreset
   if (workspaceId) meta.workspaceId = workspaceId
 
@@ -293,7 +294,7 @@ export async function runTask({
   agents, store, task, config, provider, model,
   mintSessionId, createMessage, cwdOf, logger, text,
   agentPreset, permission, permissions, claimAs,
-  gitRunner, workspaceRegistry,
+  gitRunner, workspaceRegistry, resume = false,
 }) {
   if (config?.permissionGateEnabled) {
     const perm = permission || task.permission || 'read-only'
@@ -328,8 +329,8 @@ export async function runTask({
   taken.agent.followup(createMessage({
     // Содержимое — МАССИВ блоков, а не строка: ядро перебирает его как список
     // (`content.some(...)`), и строка роняет ход целиком.
-    content: [{ type: 'text', text: messageFor(task, config, text) }],
-    source: { kind: 'plugin', plugin: 'dsh-kanban', form: 'task-start' },
+    content: [{ type: 'text', text: messageFor(task, config, text, { resume, mode: taken.mode }) }],
+    source: { kind: 'plugin', plugin: 'dsh-kanban', form: resume ? 'task-resume' : 'task-start' },
   }))
 
   const before = store.getTask(task.id)
@@ -347,6 +348,16 @@ export async function runTask({
   // остаётся чужой.
   claimOnStart({ store, task: before, login: claimAs })
 
+  if (resume) {
+    store.addTransition({
+      taskId: task.id,
+      fromCol: before.column,
+      toCol: before.column,
+      source: 'session',
+      detail: `продолжение работы над задачей (сессия ${taken.sessionId})`,
+    })
+  }
+
   // Изоляция сессии в отдельном воркдереве фиксируется в журнале.
   if (taken.worktreeCreated) {
     store.addTransition({
@@ -386,9 +397,15 @@ export async function runTask({
 }
 
 /** Текст сообщения: человеческий, если он есть, иначе встроенная заготовка. */
-function messageFor(task, config, text) {
+export function messageFor(task, config, text, options = {}) {
   const own = typeof text === 'string' ? text.trim() : ''
   let base = own !== '' ? own : buildStartMessage(task, config)
+  let resumePreamble = ''
+  if (options.resume || options.mode === 'resumed') {
+    const wt = task?.worktree ? ` в существующем воркдереве ${task.worktree}` : ''
+    const br = task?.branch ? ` на ветке ${task.branch}` : ''
+    resumePreamble = `### ↻ Режим продолжения работы над задачей\nРабота продолжается${wt}${br}. Сохраняй существующие изменения и продолжай выполнение с текущего состояния.\n\n`
+  }
   let qualityNotes = ''
   if (task?.rejectReason) {
     qualityNotes += `### ⚠️ Внимание: Задача возвращена на доработку\nЗамечания к приёмке (причина возврата):\n${task.rejectReason}\n\n`
@@ -404,6 +421,9 @@ function messageFor(task, config, text) {
   if (qualityNotes) {
     base = qualityNotes + base
   }
+  if (resumePreamble) {
+    base = resumePreamble + base
+  }
   if (config?.progressDumpEnabled && task.progressDump) {
     let parsed
     try {
@@ -590,6 +610,10 @@ export function liveSessions({ store, agents }) {
     let live
     try { live = agents.get(id) } catch { live = undefined }
     if (live === undefined) continue
+    // Исключаем дочерние сессии субагентов (#215)
+    if (live.origin === 'subagent' || live.parentId !== undefined || live.meta?.origin === 'subagent' || live.isSubagent) {
+      continue
+    }
     if (!byId.has(id)) byId.set(id, { sessionId: id, status: live.status, tasks: [] })
     byId.get(id).tasks.push({
       id: task.id, title: task.title, repo: task.repo, issueNumber: task.issueNumber,

package/lib/lifecycle.js

@@ -20,7 +20,11 @@ import { parseProgressDump, formatProgressDump } from './progress-dump.js'
  *
  * @returns {{taskId: string, waiting?: boolean}|undefined} что изменилось
  */
-export function handleSessionEvent({ store, sessionId, type, text, error }) {
+export function handleSessionEvent({ store, sessionId, session, type, text, error }) {
+  // Исключаем события дочерних сессий субагентов (#215)
+  if (session && (session.origin === 'subagent' || session.parentId !== undefined || session.meta?.origin === 'subagent' || session.isSubagent)) {
+    return undefined
+  }
   const task = store.findTaskBySession(sessionId)
   if (task === undefined) return undefined
 

package/lib/worktree.js

@@ -13,7 +13,7 @@
 
 import { execFile } from 'node:child_process'
 import { promisify } from 'node:util'
-import { existsSync } from 'node:fs'
+import { existsSync, readFileSync } from 'node:fs'
 import { isAbsolute, join, resolve, sep } from 'node:path'
 
 const execFileAsync = promisify(execFile)
@@ -202,13 +202,44 @@ export async function createTaskWorktree({
 /**
  * Проверить наличие незакоммиченных изменений в воркдереве.
  */
+function isSubmoduleDrift(worktreePath, line) {
+  try {
+    const filePath = line.slice(3).trim()
+    if (!filePath) return false
+    const fullPath = resolve(worktreePath, filePath)
+    const gitFile = join(fullPath, '.git')
+    if (existsSync(gitFile)) {
+      return true
+    }
+    const gitmodulesPath = join(worktreePath, '.gitmodules')
+    if (existsSync(gitmodulesPath)) {
+      const content = readFileSync(gitmodulesPath, 'utf8')
+      const paths = [...content.matchAll(/path\s*=\s*([^\r\n]+)/g)].map((m) => m[1].trim())
+      if (paths.includes(filePath) || paths.some((p) => filePath.startsWith(p + '/') || filePath === p)) {
+        return true
+      }
+    }
+  } catch {
+    // Игнорируем ошибки доступа к файлам при проверке
+  }
+  return false
+}
+
+/**
+ * Проверить наличие незакоммиченных изменений в воркдереве.
+ * Исключает структурный шум субмодулей и дрейф gitlinks (#213).
+ */
 export async function checkWorktreeDirty({ worktreePath, gitRunner = defaultGitRunner }) {
   if (!worktreePath || !existsSync(worktreePath)) {
     return { dirty: false, files: [] }
   }
   try {
-    const res = await gitRunner(['status', '--porcelain'], worktreePath)
-    const files = res.stdout.split('\n').map((l) => l.trim()).filter((l) => l.length > 0)
+    const res = await gitRunner(['status', '--porcelain', '--ignore-submodules=all'], worktreePath)
+    const files = res.stdout
+      .split('\n')
+      .map((l) => l.trim())
+      .filter((l) => l.length > 0)
+      .filter((line) => !isSubmoduleDrift(worktreePath, line))
     return { dirty: files.length > 0, files }
   } catch (error) {
     return { dirty: false, files: [], error: String(error?.message ?? error) }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-kanban",
-  "version": "0.1.29",
+  "version": "0.1.31",
   "description": "Visual Kanban board for DeepSeek Harness: Gitea-backed tasks, workflow columns, and a dedicated agent session per task.",
   "type": "module",
   "main": "lib/index.js",