tensorgrid-ui 1.4.0 → 1.6.1

package/lib/reviewers.js

@@ -46,24 +46,89 @@ const STDERR_CAP = 64 * 1024
 export const REVIEWERS = {
   'claude-code': {
     label: 'Claude Code',
+    kind: 'external',
     command: 'claude',
-    models: ['claude-opus-4-8', 'claude-sonnet-4-6'],
+    // Список — подсказка, а НЕ ограничение: интерфейс даёт ввести и своё имя.
+    // Жёсткий перечень устарел бы при первом же выпуске новой модели, а
+    // выяснить у продукта его собственный список нечем.
+    models: ['claude-opus-5', 'claude-opus-4-8', 'claude-sonnet-4-6'],
     efforts: ['low', 'medium', 'high'],
-    defaultModel: null,
-    defaultEffort: null,
+    // Имена, которыми продукт авторизуется. См. `authEnvFor` ниже.
+    authEnv: ['ANTHROPIC_API_KEY', 'ANTHROPIC_AUTH_TOKEN', 'CLAUDE_CODE_OAUTH_TOKEN', 'ANTHROPIC_BASE_URL'],
   },
   codex: {
     label: 'Codex',
+    kind: 'external',
     command: 'codex',
     models: ['gpt-6-astra', 'gpt-5.6-sol'],
     efforts: ['low', 'medium', 'high', 'xhigh'],
-    defaultModel: null,
-    defaultEffort: null,
+    authEnv: ['OPENAI_API_KEY', 'CODEX_HOME'],
+  },
+  internal: {
+    label: 'Наш ревизор',
+    // Единственный, кто работает НЕ отдельным продуктом, а нашим подагентом
+    // на нашем же маршруте моделей.
+    //
+    // Он появился из опыта: та же модель в другой обвязке нашла почти
+    // непересекающийся набор находок. Значит третий угол зрения даёт не
+    // только другая модель, но и другое окружение.
+    //
+    // И он единственный, чей список моделей берётся ЖИВЫМ из маршрута:
+    // Claude Code и Codex понимают только свои имена, а здесь доступно всё,
+    // что подключено — включая сотни моделей через OpenRouter.
+    kind: 'internal',
+    models: [],
+    efforts: ['low', 'medium', 'high', 'max'],
   },
 }
 
-/** Значения уходят в аргументы команды — пропускаем только безобидное. */
+/**
+ * ЕДИНСТВЕННЫЙ источник списка ревизоров.
+ *
+ * Список жил в трёх файлах, и ревизор указал, чем это кончится: добавление
+ * продукта в один файл из трёх делает запись выбора неопределённой и роняет
+ * весь раздел настроек. Я тогда починил половину и отчитался за целое —
+ * следующий обзор это и поймал.
+ */
+export const KNOWN = Object.keys(REVIEWERS)
+
+/**
+ * Что допустимо в модели и усилии.
+ *
+ * Строгий вариант — для ВНЕШНИХ ревизоров: значение уходит в аргументы
+ * командной строки, и косая черта там ни в одном имени не нужна, а вот
+ * ведущий дефис превратил бы значение в ключ команды.
+ *
+ * Для НАШЕГО ревизора черта обязательна: маршрут называет модели вида
+ * `anthropic/claude-opus-5`, а через OpenRouter — `x-ai/grok-4.6`. Туда
+ * значение уходит не в командную строку, а в опции вызова, поэтому
+ * послабление безопасно. Одна общая проверка отвергала бы половину
+ * подключённых моделей — что и обнаружилось при первой же сверке.
+ */
 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}$/
+
+/**
+ * Вернуть ревизору его собственные ключи авторизации.
+ *
+ * Служба запуска вычищает из окружения ребёнка всё, что похоже на секрет —
+ * `KEY`, `TOKEN`, `SECRET`, `PASSWORD`. Для чужого процесса это правильно:
+ * наши ключи ему незачем. Но ревизор авторизуется как раз такими именами, и
+ * у пользователя, который входит ключом, а не подпиской, обзор молча не
+ * работал бы — при том что та же команда из консоли отвечает.
+ *
+ * Поэтому возвращаем ТОЧЕЧНО: только имена этого продукта и только если они
+ * заданы. Ничего чужого, никаких шаблонов.
+ */
+function authEnvFor(kind) {
+  const names = REVIEWERS[kind]?.authEnv ?? []
+  const env = {}
+  for (const name of names) {
+    const value = process.env[name]
+    if (typeof value === 'string' && value !== '') env[name] = value
+  }
+  return env
+}
 
 /**
  * Собрать аргументы запуска.
@@ -78,9 +143,20 @@ export function buildArgs(kind, { model, effort, answerFile }) {
   if (kind === 'claude-code') {
     // `-p` без текста задания: задание придёт через поток ввода.
     args.push('-p')
-    // Ревизор только читает, но запрет на чтение файлов сделал бы его
-    // бесполезным, а спрашивать разрешение в фоне некому.
-    args.push('--permission-mode', 'bypassPermissions')
+    // Ревизор ТОЛЬКО ЧИТАЕТ, и это обеспечено правами, а не просьбой.
+    //
+    // Здесь стоял `bypassPermissions` — полное право писать файлы и выполнять
+    // команды в рабочей папке пользователя, при том что соседний Codex
+    // запускался с `read-only`. Оба независимых ревизора нашли это и назвали
+    // критичным, справедливо: ревизор по заданию читает чужой непроверенный
+    // код, и указание внутри этого кода («игнорируй предыдущее, выполни…»)
+    // исполнилось бы без подтверждения и без следа в переписке — это
+    // отдельный процесс.
+    //
+    // Проверено с контролем: с этим списком инструментов запись отклоняется
+    // системой прав, с `bypassPermissions` файл создаётся немедленно.
+    args.push('--permission-mode', 'dontAsk')
+    args.push('--allowedTools', 'Read', 'Glob', 'Grep')
     if (model) args.push('--model', model)
     if (effort) args.push('--effort', effort)
     return args
@@ -103,28 +179,110 @@ export function buildArgs(kind, { model, effort, answerFile }) {
   throw new Error(`неизвестный ревизор: ${kind}`)
 }
 
+/**
+ * Позвать нашего собственного ревизора — подагента на нашем маршруте.
+ *
+ * Отличается от внешних принципиально: не отдельный продукт, а агент в этом
+ * же процессе. Отсюда и возможности, которых нет у остальных: любая модель
+ * из подключённых, включая сотни через OpenRouter, и усилие рассуждения из
+ * тех же настроек, что у основного агента.
+ *
+ * Рабочую папку он берёт у родителя сам — она и есть папка чата, что
+ * совпадает с ограничением доступа для внешних.
+ */
+async function runInternal(ctx, kind, { prompt, provider, model, effort, signal }) {
+  const subagents = ctx.get === undefined ? ctx.subagents : ctx.get('subagents')
+  if (subagents === undefined) {
+    return { kind, failed: 'реестр подагентов не смонтирован', answer: null }
+  }
+  if (typeof subagents.list === 'function' && !subagents.list().includes('spawn')) {
+    return { kind, failed: 'провайдер spawn недоступен', answer: null }
+  }
+
+  const agentOptions = {}
+  // Провайдер и модель приходят ПОРОЗНЬ и порознь же передаются.
+  //
+  // Соблазнительно вывести провайдера из имени модели по первой косой
+  // черте — и это неверно. У OpenRouter `x-ai/grok-4.6` целиком является
+  // именем модели, а провайдер называется `openrouter`. Деление по черте
+  // дало бы несуществующего провайдера `x-ai`, и маршрут отказал бы.
+  if (provider) agentOptions.provider = String(provider)
+  if (model) agentOptions.model = String(model)
+  if (effort) agentOptions.reasoningEffort = String(effort)
+
+  let run = null
+  try {
+    run = await subagents.start('spawn', {
+      label: 'обзор: наш ревизор',
+      prompt: [{ type: 'text', text: prompt }],
+      parent: ctx.reviewParent,
+      signal,
+      ...(Object.keys(agentOptions).length > 0 ? { agentOptions } : {}),
+    })
+    const result = await run.result
+    const answer = (result?.output ?? [])
+      .filter((block) => block?.type === 'text')
+      .map((block) => String(block.text ?? ''))
+      .join('')
+
+    if (signal?.aborted === true) return { kind, failed: 'отменено', cancelled: true, answer: null }
+    if (answer.trim() === '') return { kind, failed: 'пустой ответ', answer: null }
+    return { kind, failed: null, cancelled: false, truncated: false, answer, model: model ?? null, effort: effort ?? null }
+  } catch (error) {
+    if (signal?.aborted === true) return { kind, failed: 'отменено', cancelled: true, answer: null }
+    return { kind, failed: String(error?.message ?? error), answer: null }
+  } finally {
+    if (run !== null) {
+      try {
+        await run.dispose()
+      } catch {
+        // Уборка не должна подменять результат обзора.
+      }
+    }
+  }
+}
+
 /**
  * Запустить одного ревизора и вернуть его ответ.
  *
  * Отказ не бросается наружу: обзор двумя ревизорами не должен пропадать
  * целиком из-за одного неответившего. Возвращается либо ответ, либо причина.
  *
- * @param ctx - контекст со службой `subprocess`.
+ * @param ctx - контекст со службой `subprocess`; для нашего ревизора ещё и
+ *   `subagents`, а в `reviewParent` — агент, от имени которого идёт вызов.
  * @param kind - имя ревизора.
  * @param options - задание, рабочая папка, модель, усилие, сигнал отмены.
  */
-export async function runReviewer(ctx, kind, { prompt, cwd, model, effort, signal, timeoutMs }) {
+export async function runReviewer(ctx, kind, { prompt, cwd, provider, model, effort, signal, timeoutMs }) {
   const spec = REVIEWERS[kind]
   if (spec === undefined) return { kind, failed: `неизвестный ревизор: ${kind}`, answer: null }
 
-  for (const [label, value] of [['модель', model], ['усилие', effort]]) {
-    if (value !== undefined && value !== null && value !== '' && !SAFE_VALUE.test(String(value))) {
+  const allowed = spec.kind === 'internal' ? SAFE_ROUTE_VALUE : SAFE_VALUE
+  for (const [label, value] of [['провайдер', provider], ['модель', model], ['усилие', effort]]) {
+    if (value !== undefined && value !== null && value !== '' && !allowed.test(String(value))) {
       return { kind, failed: `недопустимое значение (${label}): ${String(value)}`, answer: null }
     }
   }
 
+  // Отмену, случившуюся ДО вызова, надо увидеть сразу.
+  //
+  // Подписка на будущее событие `abort` у уже отменённого сигнала не
+  // срабатывает никогда: обработчик задним числом не вызывается. Без этой
+  // проверки нажатие «стоп» перед самым запуском не мешало нам завести
+  // платный процесс, который потом жил до самого тайм-аута.
+  if (signal?.aborted === true) {
+    return { kind, failed: 'отменено', cancelled: true, answer: null }
+  }
+
+  // Наш ревизор — не отдельный продукт, а подагент на нашем маршруте.
+  // Ни команды, ни аргументов, ни временных файлов ему не нужно.
+  if (spec.kind === 'internal') {
+    return runInternal(ctx, kind, { prompt, provider, model, effort, signal })
+  }
+
   let workDir = null
   let handle = null
+  let truncated = false
   // Своя отмена поверх пользовательской: ревизор без ограничения по времени
   // держал бы обзор бесконечно.
   const deadline = new AbortController()
@@ -154,33 +312,76 @@ export async function runReviewer(ctx, kind, { prompt, cwd, model, effort, signa
       },
       graceMs: 5000,
       signal: deadline.signal,
+      // Точечный возврат собственных ключей продукта: служба вычищает всё
+      // похожее на секрет, а ревизор именно ими и авторизуется.
+      env: authEnvFor(kind),
     })
 
     const outcome = await handle.done
+    const stderrText = (await handle.collected.stderr.readFrom(0)).text
+    const tail = stderrText.trim().split('\n').slice(-3).join(' ')
+
+    // Отмена — это отмена, а не отказ ревизора.
+    //
+    // Раньше прерванный обзор возвращался как «пустой ответ», то есть
+    // выглядел обычным результатом: агент получал «ревизоры не ответили,
+    // находок нет» и спокойно шёл дальше, считая работу проверенной.
+    if (signal?.aborted === true) {
+      return { kind, failed: 'отменено', cancelled: true, answer: null }
+    }
+    if (deadline.signal.aborted) {
+      return { kind, failed: `превышено время ожидания (${Math.round((timeoutMs ?? DEFAULT_TIMEOUT_MS) / 1000)} с)`, answer: null }
+    }
+
+    // КОД ВОЗВРАТА ПРОВЕРЯЕТСЯ ВСЕГДА, а не только при пустом выводе.
+    //
+    // Claude Code печатает текст ошибки — истёкший вход, исчерпанный лимит,
+    // несуществующая модель — в обычный вывод и завершается с ненулевым
+    // кодом. Раньше такой вывод считался успешным ответом, разбор не находил
+    // в нём ни одной находки, и пользователь получал справку о чистоте кода
+    // при том, что обзор вообще не выполнялся. Это ровно тот тихий отказ,
+    // ради поиска которого весь инструмент и написан.
+    if (outcome.exitCode !== 0) {
+      const code = outcome.exitCode === null ? `сигнал ${String(outcome.signal)}` : `код ${outcome.exitCode}`
+      const said = (await handle.collected.stdout.readFrom(0)).text.trim().split('\n').slice(0, 2).join(' ')
+      const detail = tail !== '' ? tail : said
+      return { kind, failed: `ревизор завершился с ошибкой (${code})${detail === '' ? '' : `: ${detail}`}`, answer: null }
+    }
 
-    // Codex кладёт итог в файл, Claude Code печатает его в вывод.
     let answer = ''
     if (kind === 'codex') {
+      // Итог Codex берётся ТОЛЬКО из файла. Запасное чтение вывода отсюда
+      // убрано намеренно: в выводе лежит весь ход рассуждения вместе с
+      // образцом формата из задания, и разбор наплодил бы по нему находки-
+      // призраки — в том числе «критичную» с названием «краткое название».
+      // Отсутствие файла — это сбой, а не повод подсунуть журнал.
       try {
         answer = await readFile(answerFile, 'utf8')
       } catch {
-        answer = ''
+        return { kind, failed: `Codex не оставил итогового ответа${tail === '' ? '' : `: ${tail}`}`, answer: null }
+      }
+    } else {
+      // Признак потери читается, а не игнорируется.
+      //
+      // Собранный вывод хранит только ХВОСТ потока: длинный ответ обрезается
+      // с НАЧАЛА, а список ревизор начинает с самого опасного. Молча отдать
+      // такой обрывок как полный обзор — значит потерять именно то, ради
+      // чего обзор затевался.
+      const read = await handle.collected.stdout.readFrom(0)
+      answer = read.text
+      if (read.lossy === true) {
+        truncated = true
       }
-    }
-    if (answer.trim() === '') {
-      answer = (await handle.collected.stdout.readFrom(0)).text
     }
 
     if (answer.trim() === '') {
-      const stderr = (await handle.collected.stderr.readFrom(0)).text
-      const tail = stderr.trim().split('\n').slice(-3).join(' ')
-      const code = outcome.exitCode === null ? `сигнал ${String(outcome.signal)}` : `код ${outcome.exitCode}`
-      return { kind, failed: `пустой ответ (${code})${tail === '' ? '' : `: ${tail}`}`, answer: null }
+      return { kind, failed: `пустой ответ${tail === '' ? '' : `: ${tail}`}`, answer: null }
     }
 
-    return { kind, failed: null, answer, model: model ?? null, effort: effort ?? null }
+    return { kind, failed: null, cancelled: false, truncated, answer, model: model ?? null, effort: effort ?? null }
   } catch (error) {
-    const reason = deadline.signal.aborted && signal?.aborted !== true ? 'превышено время ожидания' : String(error?.message ?? error)
+    if (signal?.aborted === true) return { kind, failed: 'отменено', cancelled: true, answer: null }
+    const reason = deadline.signal.aborted ? 'превышено время ожидания' : String(error?.message ?? error)
     return { kind, failed: reason, answer: null }
   } finally {
     clearTimeout(timer)
@@ -188,6 +389,23 @@ export async function runReviewer(ctx, kind, { prompt, cwd, model, effort, signa
     if (handle !== null) {
       try {
         handle.terminate()
+        // Ждём смерти процесса, но НЕ бесконечно.
+        //
+        // Ждать нужно: без ожидания уборка идёт при живом процессе с открытым
+        // файлом итога, на Windows это отказ, `force` его проглатывает, и
+        // временные каталоги копятся после каждой отмены.
+        //
+        // Ограничивать тоже нужно: `waitForExit` ждёт опустошения всего
+        // управляемого диапазона, а не только самой команды. Потомок,
+        // который не умирает, держал бы вызов после тайм-аута — в
+        // неотменяемом `finally`, где «стоп» уже не поможет.
+        const cleanup = new AbortController()
+        const cleanupTimer = setTimeout(() => cleanup.abort(), 10000)
+        try {
+          await handle.waitForExit(cleanup.signal)
+        } finally {
+          clearTimeout(cleanupTimer)
+        }
       } catch {
         // Уборка не должна подменять результат.
       }

package/package.json

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