@goodandready/dsh-kanban 0.1.20 → 0.1.22

package/lib/gitea.js

@@ -222,6 +222,34 @@ export function createGiteaClient({ getConfig, resolveToken, fetchImpl = fetch,
       return request('PATCH', repoPath(owner, repo, `/issues/${n}`), { state: 'closed' })
     },
 
+    /**
+     * Назначить ответственного за issue.
+     *
+     * Пустой список снимает назначение: Gitea понимает это как «никто», и
+     * доска обязана уметь то же самое — иначе снять с себя задачу можно было
+     * бы только руками в Gitea.
+     */
+    async setAssignees({ owner, repo, index, logins }) {
+      const n = safeSegment(index)
+      if (n === undefined) throw Object.assign(new Error('недопустимый номер issue'), { code: 'bad-segment' })
+      const list = (Array.isArray(logins) ? logins : [])
+        .filter((one) => typeof one === 'string' && one !== '')
+      return request('PATCH', repoPath(owner, repo, `/issues/${n}`), { assignees: list })
+    },
+
+    /**
+     * Кто мы для Gitea.
+     *
+     * «Взять себе» на доске значит «назначить того, чьим токеном доска ходит»:
+     * своего понятия пользователя у харнесса нет, и выдумывать его вместо
+     * ответа сервера нельзя.
+     */
+    async me() {
+      const row = await request('GET', '/user')
+      const login = row?.login
+      return typeof login === 'string' ? login : ''
+    },
+
     /** Организации, доступные токену. Нужны, чтобы не спрашивать владельца лишний раз. */
     async listOrgs() {
       const rows = await request('GET', '/user/orgs?limit=50')

package/lib/import.js

@@ -33,6 +33,22 @@ export function issueAuthor(issue) {
   return typeof login === 'string' ? login : ''
 }
 
+/**
+ * Кто отвечает за issue.
+ *
+ * Gitea отдаёт назначенного двумя способами: одиночным `assignee` и списком
+ * `assignees`. Берём первого из списка, когда одиночного нет: доска показывает
+ * ОДНОГО ответственного, потому что «отвечают все» означает «не отвечает
+ * никто».
+ */
+export function issueAssignee(issue) {
+  const one = issue?.assignee?.login
+  if (typeof one === 'string' && one !== '') return one
+  const many = Array.isArray(issue?.assignees) ? issue.assignees : []
+  const first = many.find((x) => typeof x?.login === 'string' && x.login !== '')
+  return first === undefined ? '' : first.login
+}
+
 /**
  * Поля задачи из issue.
  *
@@ -60,6 +76,7 @@ export function issueToTask(issue, { board = 'main', column = 'backlog', owner,
     issueUrl: typeof issue?.html_url === 'string' ? issue.html_url : undefined,
     createdAt: issueBornAt(issue),
     author: issueAuthor(issue),
+    assignee: issueAssignee(issue),
   }
 }
 
@@ -103,6 +120,12 @@ export function refreshPatch(task, issue) {
   if (born !== undefined) patch.createdAt = born
   const author = issueAuthor(issue)
   if (author !== '') patch.author = author
+  // Снятие ответственного — тоже факт, поэтому пустое значение пишется наравне
+  // с именем: иначе доска показывала бы вчерашнего владельца задачи, с которой
+  // его уже сняли.
+  if (issue?.assignee !== undefined || Array.isArray(issue?.assignees)) {
+    patch.assignee = issueAssignee(issue)
+  }
   return patch
 }
 

package/lib/index.js

@@ -22,7 +22,7 @@ import {
   MAX_BODY_BYTES, isTrustedRequest, parseTaskPath,
   buildBoard, applyMove, createTask, updateTask, deleteTask, appendNote, setArchived, listArchive,
   taskBody,
-  taskLog, taskBySession, exportBoard, importBoard,
+  taskLog, taskBySession, exportBoard, importBoard, assignTask,
 } from './routes.js'
 import { listImportable, importIssue, refreshTask, searchRepos, createProjectTask } from './import.js'
 import {
@@ -36,7 +36,7 @@ import { isWatched } from './intake.js'
 import { handleSessionEvent } from './lifecycle.js'
 import { boardMoveDefinition, boardPlanDefinition } from './board-tool.js'
 import { verifySignature, parseEvent } from './webhook.js'
-import { planOutbound, createOutbox } from './outbound.js'
+import { planOutbound, planAssign, createOutbox } from './outbound.js'
 import { credentialRef } from '@deepseek-ai/dsh-credentials'
 
 export { COLUMN_ORDER, wipLimitField, withDefaults } from './config.js'
@@ -387,6 +387,7 @@ export function apply(ctx, config) {
           agentPreset: typeof input.agentPreset === 'string' ? input.agentPreset : undefined,
           permission: typeof input.permission === 'string' ? input.permission : undefined,
           permissions: ctx.get('permissionPresets'),
+          claimAs: await whoAmI(),
         }))
       } catch (error) {
         writeJson(res, error?.status ?? 500, { error: 'batch-failed' })
@@ -525,6 +526,33 @@ export function apply(ctx, config) {
   // самый путь, на котором с первого дня живёт импорт issue из Gitea. Два
   // обработчика на одном пути означают, что один из них не отвечает вовсе, и
   // какой именно — решает порядок регистрации, а не мы.
+  /**
+   * Кто мы для Gitea, с памятью на время работы.
+   *
+   * Логин нужен и «взять себе», и запуску работы. Спрашивать его у Gitea
+   * каждый раз значит добавить в путь запуска ещё одну сеть, которая может не
+   * ответить; поэтому спрашиваем один раз и помним. Не ответила — работаем без
+   * назначения, а не отказываем в запуске.
+   */
+  let knownLogin
+  const whoAmI = async () => {
+    if (knownLogin !== undefined) return knownLogin
+    try {
+      knownLogin = await gitea.me()
+    } catch (error) {
+      knownLogin = ''
+    }
+    return knownLogin
+  }
+
+  ctx.effect(() => ctx.webServer.register({
+    kind: 'exact',
+    path: '/dsh-kanban/whoami',
+    // Пустой логин — законный ответ: Gitea может быть не настроен или не
+    // отвечать. Браузер по нему просто не рисует «Мои», а не показывает ноль.
+    handler: async (req, res) => { reply(res, { login: await whoAmI() }) },
+  }), 'dsh-kanban: кто мы для Gitea')
+
   ctx.effect(() => ctx.webServer.register({
     kind: 'exact',
     path: '/dsh-kanban/snapshot',
@@ -619,6 +647,7 @@ export function apply(ctx, config) {
             agentPreset: typeof input.agentPreset === 'string' ? input.agentPreset : undefined,
             permission: typeof input.permission === 'string' ? input.permission : undefined,
             permissions: ctx.get('permissionPresets'),
+            claimAs: await whoAmI(),
           }))
         } catch (error) {
           ctx.logger?.warn?.(`dsh-kanban: запуск задачи не удался: ${error?.message}`)
@@ -674,6 +703,25 @@ export function apply(ctx, config) {
           reply(res, out)
           return
         }
+        if (req.method === 'POST' && action === 'assign') {
+          const was = store.getTask(id)
+          if (was === undefined) { writeJson(res, 404, { error: 'task-not-found' }); return }
+          // Пустой логин снимает назначение; слово «me» просит того, чьим
+          // токеном ходит доска: своего понятия пользователя у харнесса нет.
+          let login = typeof input.login === 'string' ? input.login.trim() : ''
+          if (login === 'me') {
+            login = await whoAmI()
+            if (login === '') { writeJson(res, 502, { error: 'gitea-unreachable' }); return }
+          }
+          const out = assignTask({ store, id, login })
+          if (out.task !== undefined && config.pushToGitea !== false) {
+            // В Gitea уходит очередью: доска не ждёт сеть и работает, когда
+            // Gitea лежит.
+            outbox.push(id, planAssign(out.task, login))
+          }
+          reply(res, out)
+          return
+        }
         if (req.method === 'POST' && action === 'archive') {
           reply(res, setArchived({ store, id, archived: true }))
           return

package/lib/launcher.js

@@ -199,10 +199,34 @@ export async function obtainAgent({
  * @param {string} [text] сообщение человека; пусто — встроенная заготовка
  * @returns {{sessionId: string, mode: string, task: object}}
  */
+/**
+ * Взять задачу на себя при запуске.
+ *
+ * Запустить работу и оставить задачу ничьей — значит показать её свободной,
+ * когда над ней уже работают. Чужое назначение не трогаем: задачу могли отдать
+ * другому осознанно.
+ *
+ * @returns {string} логин, если назначение случилось; иначе пустая строка
+ */
+export function claimOnStart({ store, task, login }) {
+  const who = typeof login === 'string' ? login.trim() : ''
+  if (who === '') return ''
+  if (typeof task?.assignee === 'string' && task.assignee !== '') return ''
+  store.updateTask(task.id, { assignee: who })
+  store.addTransition({
+    taskId: task.id,
+    fromCol: task.column,
+    toCol: task.column,
+    source: 'manual',
+    detail: `ответственный: ${who} (взял при запуске)`,
+  })
+  return who
+}
+
 export async function runTask({
   agents, store, task, config, provider, model,
   mintSessionId, createMessage, cwdOf, logger, text,
-  agentPreset, permission, permissions,
+  agentPreset, permission, permissions, claimAs,
 }) {
   const taken = await obtainAgent({
     agents, task, config, provider, model, mintSessionId, cwdOf, logger,
@@ -220,6 +244,9 @@ export async function runTask({
   const before = store.getTask(task.id)
   const patch = { sessionId: taken.sessionId, provider, model }
   store.updateTask(task.id, patch)
+  // Запустил — значит отвечаешь: свободная задача становится твоей, чужая
+  // остаётся чужой.
+  claimOnStart({ store, task: before, login: claimAs })
 
   // Подмена чата не должна быть молчаливой: человек вернётся к задаче и не
   // поймёт, куда делась переписка.
@@ -297,7 +324,7 @@ export function buildQueuedMessage(task, config, at, total) {
 export async function runBatch({
   agents, store, tasks, config, provider, model,
   mintSessionId, createMessage, cwdOf, logger, text, now = Date.now(),
-  agentPreset, permission, permissions,
+  agentPreset, permission, permissions, claimAs,
 }) {
   const list = (tasks ?? []).filter((task) => task !== undefined)
   if (list.length === 0) return { error: 'nothing-picked', status: 400 }
@@ -329,6 +356,9 @@ export async function runBatch({
     // даже когда всё случилось в одну миллисекунду.
     const patch = { sessionId: taken.sessionId, provider, model, queuedAt: at === 0 ? 0 : now + at }
     store.updateTask(task.id, patch)
+    // Пачку берёт на себя тот, кто её запустил: пять карточек в работе без
+    // ответственного выглядят свободными.
+    claimOnStart({ store, task: before, login: claimAs })
     if (at === 0 && before.column === 'backlog') {
       store.moveTask(task.id, { column: 'in-progress' })
       store.addTransition({

package/lib/outbound.js

@@ -13,6 +13,22 @@
 // из состояния issue, ветки и pull request, а метка лишь пересказывала в Gitea
 // то, что там уже написано. Взамен доска не трогает чужую разметку вовсе.
 
+/**
+ * Что отправить в Gitea при смене ответственного.
+ *
+ * Своей задачи (без issue) это не касается: назначать там некого и негде.
+ */
+export function planAssign(task, login) {
+  if (typeof task?.issueNumber !== 'number') return []
+  return [{
+    kind: 'assign',
+    owner: task.owner,
+    repo: task.repo,
+    index: task.issueNumber,
+    login: typeof login === 'string' ? login : '',
+  }]
+}
+
 export function planOutbound(task, column, config) {
   if (typeof task?.issueNumber !== 'number') return []
   // Закрытие issue — единственное, что доска сообщает Gitea: это факт, а не
@@ -58,6 +74,12 @@ export function createOutbox({ gitea, store, logger, maxAttempts = 4 }) {
           if (item.op.kind === 'close') {
             await gitea.closeIssue({ owner: item.op.owner, repo: item.op.repo, index: item.op.index })
           }
+          if (item.op.kind === 'assign') {
+            await gitea.setAssignees({
+              owner: item.op.owner, repo: item.op.repo, index: item.op.index,
+              logins: item.op.login === '' ? [] : [item.op.login],
+            })
+          }
           sent += 1
         } catch (error) {
           item.attempts += 1