tensorgrid-ui 1.4.1 → 1.10.0

package/lib/review-settings.js

@@ -11,15 +11,21 @@
  *
  * Импортируются только встроенные модули Node.
  */
-import { readFile, rename, writeFile } from 'node:fs/promises'
+import { KNOWN } 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'
 
-/** Ревизоры, которых мы умеем звать. Порядок — порядок показа. */
-export const KNOWN = ['claude-code', 'codex']
+/**
+ * Ревизоры берутся из `reviewers.js` — единственного источника.
+ * Свой литерал здесь был третьим по счёту и расходился бы молча.
+ */
+export { KNOWN } from './reviewers.js'
+import { REVIEWERS } from './reviewers.js'
 
 /**
  * Что допустимо в модели и усилии.
@@ -31,20 +37,110 @@ export const KNOWN = ['claude-code', 'codex']
  * показывает пустое поле, неотличимое от «как настроено».
  */
 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 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'
 
 /**
  * Что действует, пока пользователь ничего не выбрал.
  *
- * Оба ревизора включены намеренно: они находят РАЗНОЕ, и один по умолчанию
+ * Два ревизора включены намеренно: они находят РАЗНОЕ, и один по умолчанию
  * означал бы, что половина находок не появится у того, кто в настройки не
  * заглядывал. Модель и усилие пустые — берутся собственные настройки
  * продукта, то есть то, чем человек уже пользуется.
  */
 export function defaults() {
   return {
-    enabled: [...KNOWN],
-    choices: { 'claude-code': { model: null, effort: null }, codex: { model: null, effort: null } },
+    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,
+    }])),
+  }
+}
+
+/**
+ * Применить режим, сохранив поштучный выбор моделей.
+ *
+ * Переключение режима меняет СОСТАВ и усилие, но не трогает выбранные
+ * модели: человек, настроивший модель для Codex, не должен терять её
+ * оттого, что разок проверил быстрым режимом.
+ */
+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
+
+  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,
+    }
   }
+  return { mode, enabled: [...spec.enabled], choices }
 }
 
 /**
@@ -71,10 +167,17 @@ function findProfileRoot(startDir) {
  *
  * @param cwd - рабочая папка сессии, запасной ориентир.
  */
-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)
 }
 
 /** Привести прочитанное к ожидаемой форме: файл правит человек, и правит с ошибками. */
@@ -94,25 +197,107 @@ export function normalise(raw) {
     // Недопустимое значение отбрасывается молча и превращается в «как
     // настроено»: сохранить его значило бы выключить обзор до тех пор, пока
     // кто-нибудь не догадается открыть файл.
+    //
+    // Наш ревизор ходит по маршруту, где имена содержат косую черту
+    // (`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 !== '' && SAFE_VALUE.test(trimmed) ? trimmed : null
+      return trimmed !== '' && allowed.test(trimmed) ? trimmed : null
     }
     choices[kind] = {
+      provider: pick(source?.provider),
       model: pick(source?.model),
       effort: pick(source?.effort),
     }
   }
 
-  // Пустой список допустим только как осознанный выбор «никого не звать»;
-  // отличить его от испорченного файла нельзя, поэтому возвращаем как есть.
-  return { enabled, choices }
+  // Режим ВСЕГДА выводится из состава, а не берётся из файла.
+  //
+  // Прежде сохранённое имя принималось на веру и никогда не сверялось с
+  // `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)
 }
 
+/**
+ * Очередь записи на каждый файл.
+ *
+ * Запись идёт по схеме «прочитать целиком → изменить → записать целиком»,
+ * и без очереди одновременные вызовы читают ОДНО состояние, а пишут каждый
+ * своё: побеждает последний, остальные исчезают бесследно. Проверено —
+ * двенадцать одновременных записей давали одну.
+ *
+ * Случай не выдуманный: журнал пишут две стороны — 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) {
+    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))
+    }
+  }
+}
 /** Прочитать настройки. Отсутствие файла — не ошибка, а «ничего не выбрано». */
-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')
@@ -130,12 +315,51 @@ 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)
+
+  // Присланное СЛИВАЕТСЯ с тем, что на диске, а не кладётся поверх.
+  //
+  // Прежде маршрут писал объект целиком. Две открытые вкладки настроек — и
+  // та, что открыта дольше, одним нажатием галочки возвращала старые модели
+  // и усилия: она шлёт свой устаревший снимок целиком. Любой отправитель,
+  // приславший только `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 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
 }