tensorgrid-ui 1.10.0 → 2.1.0

package/lib/review-settings.js

@@ -6,12 +6,21 @@
  * значит хранилище должно быть общим для обеих сторон. Память браузера для
  * этого не годится, Host её не видит.
  *
- * Файл лежит там же, где отметка об установке, и ищется тем же подъёмом
- * вверх: так путь не зависит от того, насколько глубоко pnpm разложил пакет.
+ * ФОРМА: СПИСОК, А НЕ ФИКСИРОВАННЫЕ ГНЁЗДА
+ *
+ * Раньше ревизоры были четырьмя жёсткими гнёздами, а сверху стояли три
+ * готовых режима — быстрый, обычный, глубокий. Это оказалось не тем
+ * решением: готовый режим отвечает за человека на вопрос, на который он
+ * вполне способен ответить сам, а четыре гнезда не дают позвать, скажем,
+ * три разные модели одного семейства.
+ *
+ * Теперь это список произвольной длины. Каждая запись говорит: кем
+ * работать (нашим подагентом или чужим продуктом), какой моделью, с каким
+ * усилием, участвует ли сейчас.
  *
  * Импортируются только встроенные модули Node.
  */
-import { KNOWN } from './reviewers.js'
+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'
@@ -20,133 +29,160 @@ import { fileURLToPath } from 'node:url'
 
 const FILE_NAME = 'tensorgrid-review.json'
 
-/**
- * Ревизоры берутся из `reviewers.js` — единственного источника.
- * Свой литерал здесь был третьим по счёту и расходился бы молча.
- */
+/** Ревизоры берутся из `reviewers.js` — единственного источника. */
 export { KNOWN } from './reviewers.js'
-import { REVIEWERS } 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: ['internal'],
-    effort: null,
-  },
-  normal: {
-    label: 'Обычный',
-    hint: 'Две разные модели. Они находят разное — одна упускает половину.',
-    // Два НАШИХ ревизора на разных семействах моделей.
-    //
-    // Раньше здесь стояли два чужих продукта. Чистый опыт показал, что
-    // обвязка почти ничего не решает — совпадение 13 находок из 14, — а
-    // разное находят разные модели. Обе подписки работают и у нас, так что
-    // тот же результат достаётся без двух лишних установок сотруднику.
-    enabled: ['internal', 'internal-alt'],
-    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: REVIEWERS[kind]?.defaultProvider ?? null,
-      model: REVIEWERS[kind]?.defaultModel ?? 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) {
-  // Принадлежность проверяется по СОБСТВЕННЫМ ключам.
-  //
-  // `MODES[mode] !== undefined` пропускало унаследованные имена: у обычного
-  // объекта есть `constructor`, `toString` и прочие. Строка `"mode":
-  // "constructor"` в правленом файле принималась как допустимый режим, а
-  // нажатие любой кнопки уходило сюда и роняло маршрут: `spec.enabled` у
-  // функции-конструктора не перебирается.
-  const spec = Object.hasOwn(MODES, mode) ? MODES[mode] : undefined
-  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) {
@@ -161,90 +197,21 @@ function findProfileRoot(startDir) {
 /**
  * Где лежит файл настроек.
  *
- * Сначала ищем от собственного расположения — так работает установленный
- * пакет. Если отметки нет (запуск из репозитория, проверка), кладём рядом с
- * рабочей папкой, чтобы проверяемый код вёл себя как настоящий.
+ * Каталог ищется ТОЛЬКО от собственного расположения модуля. Запасной
+ * подъём от рабочей папки убран намеренно: подложенная в папку чата отметка
+ * об установке увела бы настройки в чужое дерево — в тот самый
+ * непроверенный код, который отдан на обзор. Две половины одного продукта
+ * не могут искать свой файл по разным правилам.
  *
- * @param cwd - рабочая папка сессии, запасной ориентир.
+ * @param cwd - оставлен ради совместимости вызовов; не используется.
+ * @param root - явный каталог; только для проверок.
  */
 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`, хотя соседний комментарий обещал ровно эту сверку. После
-  // ручной правки состава человек видел подсвеченным «Быстрый — один
-  // ревизор, минуты», а запускались трое на полном усилии: счёт и время
-  // втрое против показанного. Состав — факт, режим — его имя; имя не может
-  // противоречить факту.
-  return { mode: matchMode(enabled), enabled, choices }
-}
-
-/**
- * Узнать режим по составу ревизоров.
- *
- * Нужно, когда состав правили поштучно: режим в файле мог остаться от
- * прошлого выбора и врал бы в интерфейсе. Не совпал ни с одним — значит
- * набор свой, так и покажем.
- */
-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
-  }
-  return null
-}
-
-/** Есть ли такой режим. Только собственные ключи — см. `applyMode`. */
-export function isMode(name) {
-  return typeof name === 'string' && Object.hasOwn(MODES, name)
-}
-
 /**
  * Очередь записи на каждый файл.
  *
@@ -253,35 +220,26 @@ export function isMode(name) {
  * своё: побеждает последний, остальные исчезают бесследно. Проверено —
  * двенадцать одновременных записей давали одну.
  *
- * Случай не выдуманный: журнал пишут две стороны — Host в конце обзора и
- * браузер по нажатию в панели. Человек подтверждает находку ровно в тот
- * момент, когда завершается второй обзор.
- *
- * Очередь внутрипроцессная, и этого достаточно: оба пути проходят через
- * один Host. От двух ОДНОВРЕМЕННО ЗАПУЩЕННЫХ приложений она не спасает —
- * такой случай потребовал бы файловой блокировки, а он у нас невозможен:
- * второй экземпляр не поднимется на занятом порту.
+ * Очередь внутрипроцессная, и этого достаточно: оба пути — Host и браузер —
+ * проходят через один процесс.
  */
 const queues = new Map()
 
 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: файл в этот момент держит чужая
- * операция. Это не ошибка вызывающего и не повод терять запись — достаточно
- * повторить через несколько миллисекунд.
- *
- * Повторяются только эти два кода. Всё прочее — настоящая ошибка (нет прав
- * на каталог, кончилось место), и глотать её нельзя.
+ * операция. Это не ошибка вызывающего и не повод терять запись.
  */
 async function renameWithRetry(from, to, attempts = 5) {
   for (let attempt = 1; ; attempt += 1) {
@@ -295,6 +253,7 @@ async function renameWithRetry(from, to, attempts = 5) {
     }
   }
 }
+
 /** Прочитать настройки. Отсутствие файла — не ошибка, а «ничего не выбрано». */
 export async function readSettings(cwd, root = null) {
   const path = settingsPath(cwd, root)
@@ -310,10 +269,10 @@ export async function readSettings(cwd, root = null) {
 /**
  * Записать настройки. Возвращает то, что действительно сохранено.
  *
- * Запись идёт через временный файл с переименованием. Прямая перезапись
- * сначала обрезает файл, и чтение, пришедшееся на этот момент, видит пустоту
- * или обрывок — а пустота у нас означает «включить обоих ревизоров», то есть
- * запустить продукт, который человек только что выключил.
+ * Присланное ЗАМЕЩАЕТ список целиком, и это правильно именно для списка:
+ * удаление записи иначе было бы невыразимо — слияние всегда возвращало бы
+ * удалённое обратно. Ответственность за целость списка лежит на том, кто
+ * его прислал, а интерфейс всегда шлёт его полностью.
  */
 export async function writeSettings(cwd, value, root = null) {
   return serialise(settingsPath(cwd, root) ?? 'нет-пути', () => writeSettingsNow(cwd, value, root))
@@ -323,36 +282,9 @@ async function writeSettingsNow(cwd, value, root) {
   const path = settingsPath(cwd, root)
   if (path === null) throw new Error('не найден каталог профиля — некуда сохранить настройки обзора')
 
-  // Присланное СЛИВАЕТСЯ с тем, что на диске, а не кладётся поверх.
-  //
-  // Прежде маршрут писал объект целиком. Две открытые вкладки настроек — и
-  // та, что открыта дольше, одним нажатием галочки возвращала старые модели
-  // и усилия: она шлёт свой устаревший снимок целиком. Любой отправитель,
-  // приславший только `enabled`, стирал ВЕСЬ выбор моделей без единого слова.
-  //
-  // Сливаем поштучно: пришло поле — берём пришедшее, не пришло — оставляем
-  // сохранённое.
-  const current = await readSettings(cwd, root)
-  const merged = {
-    enabled: Array.isArray(value?.enabled) ? value.enabled : current.enabled,
-    choices: Object.fromEntries(
-      KNOWN.map((kind) => {
-        const was = current.choices[kind] ?? { provider: null, model: null, effort: null }
-        const came = value?.choices?.[kind]
-        if (came === undefined || came === null || typeof came !== 'object') return [kind, was]
-        return [kind, {
-          provider: 'provider' in came ? came.provider : was.provider,
-          model: 'model' in came ? came.model : was.model,
-          effort: 'effort' in came ? came.effort : was.effort,
-        }]
-      }),
-    ),
-  }
-
-  const normalised = normalise(merged)
+  const normalised = normalise(value)
   // Уникальное имя временного файла: у двух одновременных записей прежнее
-  // совпадало, и вторая получала отказ на переименовании — первая уже
-  // унесла общий файл.
+  // совпадало, и вторая получала отказ на переименовании.
   const temporary = `${path}.${process.pid}.${randomBytes(6).toString('hex')}.tmp`
   try {
     await writeFile(temporary, `${JSON.stringify(normalised, null, 2)}\n`, 'utf8')

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,

package/lib/reviewers.js

@@ -63,7 +63,12 @@ export const REVIEWERS = {
     label: 'Наш ревизор',
     kind: 'internal',
     models: [],
-    efforts: ['low', 'medium', 'high', 'max'],
+    // Уровни усилия НЕ перечисляются: у каждой модели они свои, и зашитый
+    // список врал почти везде. Проверено на живых данных: у claude-opus-5
+    // их шесть, у deepseek-v4-pro — три, а одна и та же модель через
+    // подписку и через OpenRouter даёт РАЗНЫЕ наборы. Их спрашивают у
+    // маршрута по выбранной модели.
+    efforts: [],
     defaultProvider: 'anthropic',
     defaultModel: 'claude-opus-5',
   },
@@ -71,7 +76,7 @@ export const REVIEWERS = {
     label: 'Наш ревизор, вторая модель',
     kind: 'internal',
     models: [],
-    efforts: ['low', 'medium', 'high', 'max'],
+    efforts: [],
     // Намеренно ДРУГОЕ семейство: два прогона одной модели дадут почти
     // одинаковые находки, и второй ревизор не окупит своего времени.
     defaultProvider: 'openai-codex',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tensorgrid-ui",
-  "version": "1.10.0",
+  "version": "2.1.0",
   "description": "TENSOR GRID — фирменный интерфейс поверх DeepSeek Harness: палитра, живой ambient-слой, айдентика и русский язык-пакет",
   "type": "module",
   "main": "lib/index.js",