@goodandready/dsh-kanban 0.1.20 → 0.1.21

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'
@@ -674,6 +674,29 @@ 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') {
+            try {
+              login = await gitea.me()
+            } catch (error) {
+              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/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