tensorgrid-ui 1.6.1 → 2.0.0

package/lib/review-settings.js

@@ -6,120 +6,183 @@
  * значит хранилище должно быть общим для обеих сторон. Память браузера для
  * этого не годится, Host её не видит.
  *
- * Файл лежит там же, где отметка об установке, и ищется тем же подъёмом
- * вверх: так путь не зависит от того, насколько глубоко pnpm разложил пакет.
+ * ФОРМА: СПИСОК, А НЕ ФИКСИРОВАННЫЕ ГНЁЗДА
+ *
+ * Раньше ревизоры были четырьмя жёсткими гнёздами, а сверху стояли три
+ * готовых режима — быстрый, обычный, глубокий. Это оказалось не тем
+ * решением: готовый режим отвечает за человека на вопрос, на который он
+ * вполне способен ответить сам, а четыре гнезда не дают позвать, скажем,
+ * три разные модели одного семейства.
+ *
+ * Теперь это список произвольной длины. Каждая запись говорит: кем
+ * работать (нашим подагентом или чужим продуктом), какой моделью, с каким
+ * усилием, участвует ли сейчас.
  *
  * Импортируются только встроенные модули Node.
  */
-import { KNOWN } from './reviewers.js'
-import { readFile, rename, writeFile } from 'node:fs/promises'
+import { KNOWN, REVIEWERS } from './reviewers.js'
+import { randomBytes } from 'node:crypto'
+import { readFile, rename, rm, writeFile } from 'node:fs/promises'
 import { existsSync } from 'node:fs'
 import { dirname, join } from 'node:path'
 import { fileURLToPath } from 'node:url'
 
 const FILE_NAME = 'tensorgrid-review.json'
 
-/**
- * Ревизоры берутся из `reviewers.js` — единственного источника.
- * Свой литерал здесь был третьим по счёту и расходился бы молча.
- */
+/** Ревизоры берутся из `reviewers.js` — единственного источника. */
 export { KNOWN } from './reviewers.js'
 
 /**
- * Что допустимо в модели и усилии.
+ * Сколько ревизоров можно завести.
  *
- * Те же правила, что и при запуске, но применяются ЗДЕСЬ, при сохранении.
- * Раньше проверка жила только в момент вызова: строка вроде «gpt 5 (новая)»
- * спокойно сохранялась и отвергалась через двадцать минут, при следующем
- * обзоре. В интерфейсе это было невидимо — список с неизвестным значением
- * показывает пустое поле, неотличимое от «как настроено».
+ * Предел есть, потому что каждый — отдельный прогон модели: десяток
+ * означает десяток ожиданий и десяток счетов. Число намеренно щедрое:
+ * это заслон от случайной сотни, а не указание, сколько «правильно».
  */
-const SAFE_VALUE = /^[A-Za-z0-9][A-Za-z0-9._-]{0,63}$/
-const SAFE_ROUTE_VALUE = /^[A-Za-z0-9][A-Za-z0-9._\-\/]{0,95}$/
+export const MAX_REVIEWERS = 8
 
 /**
- * Режимы обзора — то, чем пользуются, не вникая.
+ * Что допустимо в провайдере, модели и усилии.
  *
- * Сотруднику незачем знать разницу между именами моделей. Ему нужно решить
- * одно: насколько тщательно проверить. Режим задаёт состав ревизоров и
- * усилие; кто хочет — открывает подробные настройки и меняет поштучно.
+ * Проверка стоит ЗДЕСЬ, при сохранении, а не в момент вызова: раньше
+ * кривое значение сохранялось молча и отвергалось через двадцать минут,
+ * при следующем обзоре, причём в интерфейсе выглядело как «как настроено».
  *
- * Модель режимы НЕ задают намеренно: у каждого продукта своя настроена, и
- * навязывать поверх значило бы ломать то, что человек уже выбрал.
+ * Строгий вариант — для внешних продуктов: значение уходит в аргументы
+ * командной строки, и косая черта там ни в одном имени не нужна.
+ * Послабленный — для наших: маршрут называет модели вида `x-ai/grok-4.6`,
+ * и общая проверка отвергала бы половину подключённых.
  */
-export const MODES = {
-  fast: {
-    label: 'Быстрый',
-    hint: 'Один ревизор, обычное усилие. Минуты. Для рядовой работы.',
-    enabled: ['claude-code'],
-    effort: null,
-  },
-  normal: {
-    label: 'Обычный',
-    hint: 'Два разных продукта. Они находят разное — один упускает половину.',
-    enabled: ['claude-code', 'codex'],
-    effort: null,
-  },
-  deep: {
-    label: 'Глубокий',
-    hint: 'Все ревизоры на полном усилии, включая нашего на выбранной модели. Долго и дорого — для того, что увидят люди.',
-    // «Глубокий» означает ВСЕ, а не перечисленных поимённо: иначе добавление
-    // четвёртого ревизора молча оставило бы самый тщательный режим без него.
-    // Остальные режимы называют состав явно — в этом и есть их смысл.
-    enabled: [...KNOWN],
-    effort: 'high',
-  },
-}
-
-/** Режим по умолчанию: два ревизора — проверенная середина. */
-const DEFAULT_MODE = 'normal'
+const SAFE_VALUE = /^[A-Za-z0-9][A-Za-z0-9._-]{0,63}$/
+const SAFE_ROUTE_VALUE = /^[A-Za-z0-9][A-Za-z0-9._\-/]{0,95}$/
 
 /**
  * Что действует, пока пользователь ничего не выбрал.
  *
- * Два ревизора включены намеренно: они находят РАЗНОЕ, и один по умолчанию
- * означал бы, что половина находок не появится у того, кто в настройки не
- * заглядывал. Модель и усилие пустые — берутся собственные настройки
- * продукта, то есть то, чем человек уже пользуется.
+ * Два ревизора на РАЗНЫХ семействах моделей из двух подписок. Два —
+ * потому что они находят разное: один упускает половину. Разных семейств —
+ * потому что два прогона одной модели дают почти одинаковые находки.
+ *
+ * Обе модели по подписке: обзор по умолчанию не должен молча тратить
+ * деньги поштучно.
  */
 export function defaults() {
   return {
-    mode: DEFAULT_MODE,
-    enabled: [...MODES[DEFAULT_MODE].enabled],
-    // Записи строятся ПО СПИСКУ, а не перечисляются вручную: перечисление
-    // и было тем третьим источником правды, из-за которого добавление
-    // продукта оставило бы выбор неопределённым, а раздел настроек — упавшим.
-    choices: Object.fromEntries(KNOWN.map((kind) => [kind, { provider: null, model: null, effort: null }])),
+    reviewers: [
+      {
+        id: 'r1',
+        kind: 'internal',
+        provider: REVIEWERS.internal.defaultProvider,
+        model: REVIEWERS.internal.defaultModel,
+        effort: null,
+        enabled: true,
+      },
+      {
+        id: 'r2',
+        kind: 'internal',
+        provider: REVIEWERS['internal-alt'].defaultProvider,
+        model: REVIEWERS['internal-alt'].defaultModel,
+        effort: null,
+        enabled: true,
+      },
+    ],
   }
 }
 
+/** Признак записи. Короткий и без общих счётчиков: их сложно держать в файле. */
+function freshId() {
+  return `r${randomBytes(4).toString('hex')}`
+}
+
 /**
- * Применить режим, сохранив поштучный выбор моделей.
+ * Привести прочитанное к ожидаемой форме: файл правит и человек руками.
  *
- * Переключение режима меняет СОСТАВ и усилие, но не трогает выбранные
- * модели: человек, настроивший модель для Codex, не должен терять её
- * оттого, что разок проверил быстрым режимом.
+ * Понимает и ПРЕЖНЮЮ форму — `enabled` + `choices` + `mode`. Без этого
+ * переход на список молча обнулил бы настройки у всех, кто уже что-то
+ * выбрал: файл бы не разобрался, а пустой файл означает умолчания.
  */
-export function applyMode(settings, mode) {
-  const spec = MODES[mode]
-  if (spec === undefined) return settings
+export function normalise(raw) {
+  if (raw === null || typeof raw !== 'object') return defaults()
 
-  const choices = {}
-  for (const kind of KNOWN) {
-    const previous = settings.choices?.[kind] ?? {}
-    choices[kind] = {
-      provider: previous.provider ?? null,
-      model: previous.model ?? null,
-      effort: spec.effort,
+  const source = Array.isArray(raw.reviewers) ? raw.reviewers : migrate(raw)
+  if (source === null) return defaults()
+
+  const seen = new Set()
+  const reviewers = []
+
+  for (const entry of source) {
+    if (reviewers.length >= MAX_REVIEWERS) break
+    if (entry === null || typeof entry !== 'object') continue
+
+    const kind = KNOWN.includes(entry.kind) ? entry.kind : null
+    if (kind === null) continue
+
+    const allowed = REVIEWERS[kind].kind === 'internal' ? SAFE_ROUTE_VALUE : SAFE_VALUE
+    const pick = (value) => {
+      if (typeof value !== 'string') return null
+      const trimmed = value.trim()
+      return trimmed !== '' && allowed.test(trimmed) ? trimmed : null
     }
+
+    // Признак обязан быть уникальным: по нему интерфейс отличает строки, и
+    // совпадение слило бы две записи в одну при первой же правке.
+    let id = typeof entry.id === 'string' && /^[A-Za-z0-9_-]{1,32}$/.test(entry.id) ? entry.id : freshId()
+    while (seen.has(id)) id = freshId()
+    seen.add(id)
+
+    reviewers.push({
+      id,
+      kind,
+      // Провайдер имеет смысл только у наших: внешний продукт ходит своим
+      // путём, и наш маршрут ему не указ.
+      provider: REVIEWERS[kind].kind === 'internal' ? pick(entry.provider) : null,
+      model: pick(entry.model),
+      effort: pick(entry.effort),
+      enabled: entry.enabled !== false,
+    })
   }
-  return { mode, enabled: [...spec.enabled], choices }
+
+  // Пустой список допустим: это осознанный выбор «никого не звать», и
+  // подменять его умолчаниями значило бы звать тех, кого выключили.
+  return { reviewers }
 }
 
 /**
- * Каталог профиля: поднимаемся, пока не найдём отметку об установке.
- * @param startDir - откуда начинать подъём.
+ * Перевести настройки прежней формы в список.
+ *
+ * Прежде было: `enabled` — какие гнёзда включены, `choices` — настройка
+ * каждого гнезда. Переносим один к одному, сохраняя и выключенные: человек
+ * их настраивал, и терять эту работу при обновлении незачем.
+ *
+ * @returns записи в новой форме либо null, если прежней формы не видно.
  */
+function migrate(raw) {
+  const hasOld = Array.isArray(raw.enabled) || (raw.choices !== null && typeof raw.choices === 'object')
+  if (!hasOld) return null
+
+  const enabled = new Set(Array.isArray(raw.enabled) ? raw.enabled : [])
+  const entries = []
+
+  for (const kind of KNOWN) {
+    const choice = raw.choices?.[kind]
+    const configured = choice !== undefined && choice !== null && typeof choice === 'object'
+    // Гнездо, которое не включали и не настраивали, переносить незачем:
+    // иначе список сразу открывается четырьмя пустыми строками.
+    if (!enabled.has(kind) && !configured) continue
+    entries.push({
+      id: freshId(),
+      kind,
+      provider: configured ? choice.provider : null,
+      model: configured ? choice.model : null,
+      effort: configured ? choice.effort : null,
+      enabled: enabled.has(kind),
+    })
+  }
+
+  return entries
+}
+
+/** Каталог профиля: поднимаемся, пока не найдём отметку об установке. */
 function findProfileRoot(startDir) {
   let dir = startDir
   for (let depth = 0; depth < 10; depth += 1) {
@@ -134,80 +197,66 @@ function findProfileRoot(startDir) {
 /**
  * Где лежит файл настроек.
  *
- * Сначала ищем от собственного расположения — так работает установленный
- * пакет. Если отметки нет (запуск из репозитория, проверка), кладём рядом с
- * рабочей папкой, чтобы проверяемый код вёл себя как настоящий.
+ * Каталог ищется ТОЛЬКО от собственного расположения модуля. Запасной
+ * подъём от рабочей папки убран намеренно: подложенная в папку чата отметка
+ * об установке увела бы настройки в чужое дерево — в тот самый
+ * непроверенный код, который отдан на обзор. Две половины одного продукта
+ * не могут искать свой файл по разным правилам.
  *
- * @param cwd - рабочая папка сессии, запасной ориентир.
+ * @param cwd - оставлен ради совместимости вызовов; не используется.
+ * @param root - явный каталог; только для проверок.
  */
-export function settingsPath(cwd) {
-  const here = dirname(fileURLToPath(import.meta.url))
-  const root = findProfileRoot(here) ?? (typeof cwd === 'string' && cwd !== '' ? findProfileRoot(cwd) : null)
-  return root === null ? null : join(root, FILE_NAME)
+export function settingsPath(cwd, root = null) {
+  if (root !== null) return join(root, FILE_NAME)
+  const found = findProfileRoot(dirname(fileURLToPath(import.meta.url)))
+  return found === null ? null : join(found, FILE_NAME)
 }
 
-/** Привести прочитанное к ожидаемой форме: файл правит человек, и правит с ошибками. */
-export function normalise(raw) {
-  const base = defaults()
-  if (raw === null || typeof raw !== 'object') return base
-
-  // Повторы отбрасываются: список запускает по процессу на элемент, и сотня
-  // одинаковых имён означала бы сотню платных обзоров разом.
-  const enabled = Array.isArray(raw.enabled)
-    ? [...new Set(raw.enabled.filter((kind) => KNOWN.includes(kind)))]
-    : base.enabled
-
-  const choices = {}
-  for (const kind of KNOWN) {
-    const source = raw.choices !== null && typeof raw.choices === 'object' ? raw.choices[kind] : undefined
-    // Недопустимое значение отбрасывается молча и превращается в «как
-    // настроено»: сохранить его значило бы выключить обзор до тех пор, пока
-    // кто-нибудь не догадается открыть файл.
-    //
-    // Наш ревизор ходит по маршруту, где имена содержат косую черту
-    // (`x-ai/grok-4.6`), внешние — по командной строке, где её быть не
-    // должно. Одна общая проверка отвергала бы половину моделей.
-    const allowed = kind === 'internal' ? SAFE_ROUTE_VALUE : SAFE_VALUE
-    const pick = (value) => {
-      if (typeof value !== 'string') return null
-      const trimmed = value.trim()
-      return trimmed !== '' && allowed.test(trimmed) ? trimmed : null
-    }
-    choices[kind] = {
-      provider: pick(source?.provider),
-      model: pick(source?.model),
-      effort: pick(source?.effort),
-    }
-  }
-
-  // Режим — подсказка для интерфейса, а не источник состава: состав уже
-  // записан в `enabled`. Неизвестное имя превращается в «свой набор»,
-  // потому что именно им оно и является.
-  const mode = typeof raw.mode === 'string' && MODES[raw.mode] !== undefined ? raw.mode : null
+/**
+ * Очередь записи на каждый файл.
+ *
+ * Запись идёт по схеме «прочитать целиком → изменить → записать целиком»,
+ * и без очереди одновременные вызовы читают ОДНО состояние, а пишут каждый
+ * своё: побеждает последний, остальные исчезают бесследно. Проверено —
+ * двенадцать одновременных записей давали одну.
+ *
+ * Очередь внутрипроцессная, и этого достаточно: оба пути — Host и браузер —
+ * проходят через один процесс.
+ */
+const queues = new Map()
 
-  // Пустой список допустим только как осознанный выбор «никого не звать»;
-  // отличить его от испорченного файла нельзя, поэтому возвращаем как есть.
-  return { mode: mode ?? matchMode(enabled), enabled, choices }
+function serialise(path, work) {
+  const previous = queues.get(path) ?? Promise.resolve()
+  const next = previous.then(work, work)
+  // В очереди держим только факт завершения: иначе одна неудачная запись
+  // уронила бы все следующие.
+  queues.set(path, next.then(() => undefined, () => undefined))
+  return next
 }
 
 /**
- * Узнать режим по составу ревизоров.
+ * Переименовать с повтором.
  *
- * Нужно, когда состав правили поштучно: режим в файле мог остаться от
- * прошлого выбора и врал бы в интерфейсе. Не совпал ни с одним — значит
- * набор свой, так и покажем.
+ * На Windows одновременное переименование двух временных файлов в одну цель
+ * изредка отказывает с EPERM или EBUSY: файл в этот момент держит чужая
+ * операция. Это не ошибка вызывающего и не повод терять запись.
  */
-function matchMode(enabled) {
-  const same = (a, b) => a.length === b.length && a.every((item) => b.includes(item))
-  for (const [name, spec] of Object.entries(MODES)) {
-    if (same(spec.enabled, enabled)) return name
+async function renameWithRetry(from, to, attempts = 5) {
+  for (let attempt = 1; ; attempt += 1) {
+    try {
+      await rename(from, to)
+      return
+    } catch (error) {
+      const retriable = error?.code === 'EPERM' || error?.code === 'EBUSY' || error?.code === 'EACCES'
+      if (!retriable || attempt >= attempts) throw error
+      await new Promise((resolve) => setTimeout(resolve, attempt * 15))
+    }
   }
-  return null
 }
 
 /** Прочитать настройки. Отсутствие файла — не ошибка, а «ничего не выбрано». */
-export async function readSettings(cwd) {
-  const path = settingsPath(cwd)
+export async function readSettings(cwd, root = null) {
+  const path = settingsPath(cwd, root)
   if (path === null) return defaults()
   try {
     const text = await readFile(path, 'utf8')
@@ -220,17 +269,29 @@ export async function readSettings(cwd) {
 /**
  * Записать настройки. Возвращает то, что действительно сохранено.
  *
- * Запись идёт через временный файл с переименованием. Прямая перезапись
- * сначала обрезает файл, и чтение, пришедшееся на этот момент, видит пустоту
- * или обрывок — а пустота у нас означает «включить обоих ревизоров», то есть
- * запустить продукт, который человек только что выключил.
+ * Присланное ЗАМЕЩАЕТ список целиком, и это правильно именно для списка:
+ * удаление записи иначе было бы невыразимо — слияние всегда возвращало бы
+ * удалённое обратно. Ответственность за целость списка лежит на том, кто
+ * его прислал, а интерфейс всегда шлёт его полностью.
  */
-export async function writeSettings(cwd, value) {
-  const path = settingsPath(cwd)
+export async function writeSettings(cwd, value, root = null) {
+  return serialise(settingsPath(cwd, root) ?? 'нет-пути', () => writeSettingsNow(cwd, value, root))
+}
+
+async function writeSettingsNow(cwd, value, root) {
+  const path = settingsPath(cwd, root)
   if (path === null) throw new Error('не найден каталог профиля — некуда сохранить настройки обзора')
+
   const normalised = normalise(value)
-  const temporary = `${path}.${process.pid}.tmp`
-  await writeFile(temporary, `${JSON.stringify(normalised, null, 2)}\n`, 'utf8')
-  await rename(temporary, path)
+  // Уникальное имя временного файла: у двух одновременных записей прежнее
+  // совпадало, и вторая получала отказ на переименовании.
+  const temporary = `${path}.${process.pid}.${randomBytes(6).toString('hex')}.tmp`
+  try {
+    await writeFile(temporary, `${JSON.stringify(normalised, null, 2)}\n`, 'utf8')
+    await renameWithRetry(temporary, path)
+  } catch (error) {
+    await rm(temporary, { force: true }).catch(() => {})
+    throw error
+  }
   return normalised
 }

package/lib/review.js

@@ -292,17 +292,17 @@ export function parseFindings(text, reviewer) {
  * Отказ одного НЕ отменяет обзор: половина ответа полезнее отказа, и
  * пользователь должен видеть, кто именно не ответил.
  */
-async function askReviewer(ctx, runReviewer, kind, prompt, cwd, choice, signal) {
+async function askReviewer(ctx, runReviewer, kind, prompt, cwd, entry, signal) {
   const outcome = await runReviewer(ctx, kind, {
     prompt,
     cwd,
-    provider: choice?.provider ?? null,
-    model: choice?.model ?? null,
-    effort: choice?.effort ?? null,
+    provider: entry?.provider ?? null,
+    model: entry?.model ?? null,
+    effort: entry?.effort ?? null,
     signal,
   })
 
-  const used = { model: choice?.model ?? null, effort: choice?.effort ?? null }
+  const used = { model: entry?.model ?? null, effort: entry?.effort ?? null }
 
   if (outcome.failed !== null) {
     return { provider: kind, failed: outcome.failed, cancelled: outcome.cancelled === true, findings: [], ...used }
@@ -458,47 +458,57 @@ export async function apply(ctx) {
           }
         }
 
-        // Выбор пользователя из настроек; вызов может его перекрыть.
+        // Состав целиком из настроек: список, который человек собрал сам.
         const settings = await readSettings(cwd)
-        const asked = Array.isArray(args.reviewers) && args.reviewers.length > 0
-          ? args.reviewers
-          : settings.enabled
-
-        // Повторы отбрасываются, и это не придирка: `reviewers` заполняет
-        // модель, а список из ста одинаковых имён запустил бы сто платных
-        // процессов разом — по одному на каждый элемент.
-        const wanted = [...new Set(asked)].filter((kind) => KNOWN_REVIEWERS.includes(kind))
-
-        if (wanted.length === 0) {
-          const problem = asked.length === 0
-            ? 'не выбран ни один ревизор — включите их в настройках'
-            : `неизвестные ревизоры: ${[...new Set(asked)].join(', ')}`
-          return { findings: [], reviewers: [], problem }
+        let chosen = settings.reviewers.filter((entry) => entry.enabled)
+
+        // Вызов может сузить состав по виду ревизора — например «только
+        // Codex». Это именно СУЖЕНИЕ настроенного, а не свой набор: модель
+        // и усилие всё равно берутся из настроек, потому что их выбирал
+        // человек, а не тот, кто зовёт инструмент.
+        if (Array.isArray(args.reviewers) && args.reviewers.length > 0) {
+          const kinds = new Set(args.reviewers.filter((kind) => KNOWN_REVIEWERS.includes(kind)))
+          if (kinds.size > 0) chosen = chosen.filter((entry) => kinds.has(entry.kind))
+        }
+
+        if (chosen.length === 0) {
+          const anyConfigured = settings.reviewers.length > 0
+          return {
+            findings: [],
+            reviewers: [],
+            problem: anyConfigured
+              ? 'ни один ревизор не включён — включите нужных в настройках'
+              : 'ревизоры не настроены — добавьте хотя бы одного в настройках',
+          }
         }
 
         const prompt = buildPrompt(args)
 
-        // Спрашиваем ОБОИХ разом: они независимы, и последовательный опрос
-        // удвоил бы ожидание без всякой пользы.
+        // Спрашиваем ВСЕХ разом: они независимы, и последовательный опрос
+        // умножил бы ожидание без всякой пользы.
         const answers = await Promise.all(
-          wanted.map((kind) =>
+          chosen.map((entry) =>
             askReviewer(
               // Наш ревизор заводит подагента, и тому нужен родитель —
               // агент, от имени которого идёт вызов. Внешним он не нужен.
               { ...ctx, get: (name) => ctx.get(name), reviewParent: exec.agent },
               runReviewer,
-              kind,
+              entry.kind,
               prompt,
               cwd,
-              settings.choices[kind],
+              entry,
               exec.signal,
             ),
           ),
         )
 
         const findings = ordered(answers.flatMap((entry) => entry.findings))
-        const reviewers = answers.map((entry) => ({
+        const reviewers = answers.map((entry, index) => ({
           provider: entry.provider,
+          // Признак записи сохраняется вместе с итогом: в списке может быть
+          // несколько ревизоров одного вида на разных моделях, и без него
+          // статистика слила бы их в одну строку.
+          slot: chosen[index].id,
           failed: entry.failed,
           cancelled: entry.cancelled === true,
           found: entry.findings.length,