@goodandready/dsh-kanban 0.2.19 → 0.2.21

package/lib/intake.js

@@ -1,11 +1,9 @@
-// Архив выполненного и подхват новых задач из Gitea.
+// Archival of completed tasks and ingestion of new tasks from Gitea.
 //
-// Две противоположные заботы в одном месте: одна убирает с доски отработавшее,
-// другая приносит новое. Обе решают, ЧТО должно быть на доске, и держать их
-// рядом дешевле, чем искать по разным углам, почему карточка появилась или
-// пропала.
+// Two complementary concerns in one module: one clears finished tasks from board,
+// the other ingests new ones. Both govern WHAT belongs on the board.
 
-/** Разбор списка репозиториев из настройки: через запятую, пробелы не в счёт. */
+/** Parse repository list from config: comma-separated, ignoring whitespace. */
 export function parseWatchList(value) {
   return String(value ?? '')
     .split(',')
@@ -14,11 +12,9 @@ export function parseWatchList(value) {
 }
 
 /**
- * Репозитории, за которыми следим, если их назвали поимённо.
+ * Repositories watched explicitly when named.
  *
- * Пустой список означает «все», и разбирается это не здесь, а в проходе
- * подхвата: там есть Gitea, у которого можно спросить, какие репозитории вообще
- * существуют.
+ * Empty list means 'all', handled in intake pass where Gitea query lists all repos.
  *
  * @returns {Array<{owner: string, repo: string}>}
  */
@@ -26,8 +22,7 @@ export function watchedRepos({ config, owner }) {
   const out = []
   const seen = new Set()
   for (const name of parseWatchList(config?.watchRepos)) {
-    // Имя допускается и полное, и короткое: `владелец/репо` либо просто
-    // `репо` — тогда владелец берётся общий.
+    // Name can be full or short: `owner/repo` or just `repo` using default owner.
     const at = name.indexOf('/')
     const pair = at > 0
       ? { owner: name.slice(0, at), repo: name.slice(at + 1) }
@@ -42,16 +37,15 @@ export function watchedRepos({ config, owner }) {
 }
 
 /**
- * Стоит ли заводить карточку под этот issue.
+ * Whether to create a card for this issue.
  *
- * @returns {{take: boolean, why: string}} `why` называет причину отказа —
- *   иначе «почему этой задачи нет на доске» превращается в гадание.
+ * @returns {{take: boolean, why: string}} `why` states reason on rejection.
  */
 export function shouldTake({ store, owner, repo, issue }) {
   const number = issue?.number
   if (typeof number !== 'number') return { take: false, why: 'missing-number' }
-  // Pull request в Gitea — тоже issue, но работой на доске он не является:
-  // он ход работы, а не сама работа.
+  // Pull request in Gitea is also an issue, but not a standalone board card:
+  // it reflects execution progress, not the task definition.
   if (issue?.pull_request !== undefined && issue?.pull_request !== null) {
     return { take: false, why: 'pull-request-ignored' }
   }
@@ -66,13 +60,12 @@ export function shouldTake({ store, owner, repo, issue }) {
 }
 
 /**
- * Пора ли задаче в архив.
+ * Whether task is ready for archival.
  *
- * Архивируем ТОЛЬКО из `done`. Задача, застрявшая в ревью на месяц, — это
- * сигнал, а не мусор, и прятать её нельзя.
+ * Archived ONLY from `done`. A task stuck in review for a month is an alert,
+ * not clutter, and must not be hidden.
  *
- * Ноль в настройке отключает самоархивацию: это осознанное «не убирать»,
- * а не пропущенное значение.
+ * Zero in settings disables auto-archival.
  */
 export function archiveBefore({ now, afterDays }) {
   const days = Number(afterDays)
@@ -81,11 +74,10 @@ export function archiveBefore({ now, afterDays }) {
 }
 
 /**
- * Следим ли за этим репозиторием.
+ * Whether given repository is watched.
  *
- * Событие из репозитория вне списка подхвата не заслуживает: по таймеру мы
- * туда не ходим, и через вебхук ходить тоже не должны — иначе список перестаёт
- * что-либо решать.
+ * Events outside watched list are ignored: timer does not poll them,
+ * so webhooks should not process them either.
  */
 export function isWatched({ config, owner, defaultOwner, repo }) {
   return watchedRepos({ config, owner: defaultOwner }).some(