tensorgrid-ui 1.4.1 → 1.10.0

package/lib/reviewers.js

@@ -44,26 +44,136 @@ const STDERR_CAP = 64 * 1024
  * список устарел бы при первом же выпуске нового имени модели.
  */
 export const REVIEWERS = {
+  // ── Наши ревизоры ────────────────────────────────────────────────────
+  //
+  // Работают не отдельными продуктами, а нашими подагентами на нашем же
+  // маршруте моделей. Их ДВА, и это не прихоть: два разных СЕМЕЙСТВА
+  // моделей и есть то разнообразие, ради которого раньше звали чужое.
+  //
+  // Чистый опыт показал, что обвязка почти ничего не решает: одна модель в
+  // Claude Code и у нас дала совпадение 13 находок из 14. Разное находят
+  // разные МОДЕЛИ, а не разные обвязки. А обе подписки — Anthropic и
+  // OpenAI — доступны прямо здесь, со своими списками моделей и без
+  // поштучной оплаты.
+  //
+  // Список моделей пуст намеренно: он приходит живым из маршрута. Внешние
+  // продукты понимают только свои имена, а здесь доступно всё подключённое,
+  // включая сотни моделей через OpenRouter.
+  internal: {
+    label: 'Наш ревизор',
+    kind: 'internal',
+    models: [],
+    efforts: ['low', 'medium', 'high', 'max'],
+    defaultProvider: 'anthropic',
+    defaultModel: 'claude-opus-5',
+  },
+  'internal-alt': {
+    label: 'Наш ревизор, вторая модель',
+    kind: 'internal',
+    models: [],
+    efforts: ['low', 'medium', 'high', 'max'],
+    // Намеренно ДРУГОЕ семейство: два прогона одной модели дадут почти
+    // одинаковые находки, и второй ревизор не окупит своего времени.
+    defaultProvider: 'openai-codex',
+    defaultModel: 'gpt-6-astra',
+  },
+
+  // ── Внешние продукты ─────────────────────────────────────────────────
+  //
+  // Оставлены, но НЕ требуются: в умолчания не входят, в установку не
+  // включены, и без них всё работает.
+  //
+  // Причина не в том, что они плохи, а в том, что не добавляют. По деньгам
+  // — подписка работает и у нас. По находкам — совпадение 13 из 14. А
+  // стоят они двух лишних установок сотруднику и почти всех дефектов,
+  // которые пришлось чинить: аргументы, поток ввода, забор ответа,
+  // обёртки Windows, ключи авторизации, отмена, тайм-аут.
+  //
+  // Кому они уже установлены — пусть включает и пользуется.
   'claude-code': {
     label: 'Claude Code',
+    kind: 'external',
+    optional: true,
     command: 'claude',
+    // Список — подсказка, а НЕ ограничение: интерфейс даёт ввести и своё имя.
+    // Жёсткий перечень устарел бы при первом же выпуске новой модели, а
+    // выяснить у продукта его собственный список нечем.
     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',
+    optional: true,
     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'],
   },
 }
 
-/** Значения уходят в аргументы команды — пропускаем только безобидное. */
+/**
+ * ЕДИНСТВЕННЫЙ источник списка ревизоров.
+ *
+ * Список жил в трёх файлах, и ревизор указал, чем это кончится: добавление
+ * продукта в один файл из трёх делает запись выбора неопределённой и роняет
+ * весь раздел настроек. Я тогда починил половину и отчитался за целое —
+ * следующий обзор это и поймал.
+ */
+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}$/
+
+/**
+ * Что разрешено НАШЕМУ ревизору.
+ *
+ * Список РАЗРЕШЕНИЙ, а не запретов, и это принципиально. Запрет пришлось бы
+ * дополнять при каждом новом инструменте харнесса: забыли один раз — и
+ * ревизор снова умеет писать. Разрешение ошибается в безопасную сторону:
+ * незнакомый инструмент по умолчанию недоступен.
+ *
+ * Те же три возможности, что у Claude Code: читать файл, искать по именам,
+ * искать по содержимому. Больше обзору не нужно ничего.
+ */
+const REVIEW_TOOLS = { allow: ['Read', 'Glob', 'Grep'] }
+
+/**
+ * Вернуть ревизору его собственные ключи авторизации.
+ *
+ * Служба запуска вычищает из окружения ребёнка всё, что похоже на секрет —
+ * `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
+}
 
 /**
  * Собрать аргументы запуска.
@@ -114,22 +224,99 @@ 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,
+      // ТОЛЬКО ЧТЕНИЕ — так же, как у внешних ревизоров.
+      //
+      // Без этого ревизор наследовал бы весь набор инструментов родителя,
+      // включая запись файлов и запуск команд. Ровно та дыра, которую оба
+      // внешних ревизора назвали критичной у Claude Code: я убрал её там и
+      // немедленно завёл здесь, просто не передав ограничения.
+      //
+      // Асимметрия недопустима: обзор, который «ничего не меняет», не должен
+      // иметь права изменить всё — тем более что ревизор по заданию читает
+      // чужой непроверенный код и указание внутри этого кода исполнилось бы
+      // без подтверждения.
+      toolFilter: REVIEW_TOOLS,
+      ...(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 }
     }
   }
@@ -144,8 +331,15 @@ export async function runReviewer(ctx, kind, { prompt, cwd, model, effort, signa
     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()
@@ -175,6 +369,9 @@ export async function runReviewer(ctx, kind, { prompt, cwd, model, effort, signa
       },
       graceMs: 5000,
       signal: deadline.signal,
+      // Точечный возврат собственных ключей продукта: служба вычищает всё
+      // похожее на секрет, а ревизор именно ими и авторизуется.
+      env: authEnvFor(kind),
     })
 
     const outcome = await handle.done
@@ -221,14 +418,24 @@ export async function runReviewer(ctx, kind, { prompt, cwd, model, effort, signa
         return { kind, failed: `Codex не оставил итогового ответа${tail === '' ? '' : `: ${tail}`}`, answer: null }
       }
     } else {
-      answer = (await handle.collected.stdout.readFrom(0)).text
+      // Признак потери читается, а не игнорируется.
+      //
+      // Собранный вывод хранит только ХВОСТ потока: длинный ответ обрезается
+      // с НАЧАЛА, а список ревизор начинает с самого опасного. Молча отдать
+      // такой обрывок как полный обзор — значит потерять именно то, ради
+      // чего обзор затевался.
+      const read = await handle.collected.stdout.readFrom(0)
+      answer = read.text
+      if (read.lossy === true) {
+        truncated = true
+      }
     }
 
     if (answer.trim() === '') {
       return { kind, failed: `пустой ответ${tail === '' ? '' : `: ${tail}`}`, answer: null }
     }
 
-    return { kind, failed: null, cancelled: false, answer, model: model ?? null, effort: effort ?? null }
+    return { kind, failed: null, cancelled: false, truncated, answer, model: model ?? null, effort: effort ?? null }
   } catch (error) {
     if (signal?.aborted === true) return { kind, failed: 'отменено', cancelled: true, answer: null }
     const reason = deadline.signal.aborted ? 'превышено время ожидания' : String(error?.message ?? error)
@@ -239,12 +446,23 @@ export async function runReviewer(ctx, kind, { prompt, cwd, model, effort, signa
     if (handle !== null) {
       try {
         handle.terminate()
-        // Ждём, пока процесс действительно умрёт, прежде чем убирать за ним.
-        // Без ожидания удаление шло при живом процессе, у которого есть
-        // отпущенное время на завершение и открытый файл итога: на Windows
-        // это даёт отказ, `force` его проглатывает, и временные каталоги
-        // копятся после каждой отмены.
-        await handle.waitForExit()
+        // Ждём смерти процесса, но НЕ бесконечно.
+        //
+        // Ждать нужно: без ожидания уборка идёт при живом процессе с открытым
+        // файлом итога, на 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.1",
+  "version": "1.10.0",
   "description": "TENSOR GRID — фирменный интерфейс поверх DeepSeek Harness: палитра, живой ambient-слой, айдентика и русский язык-пакет",
   "type": "module",
   "main": "lib/index.js",
@@ -16,8 +16,18 @@
     },
     "./package.json": "./package.json"
   },
+  "bin": {
+    "tensorgrid-ui": "lib/cli.mjs"
+  },
   "files": [
-    "lib"
+    "lib",
+    "install.ps1",
+    "preset",
+    "profile",
+    "brand/tensorgrid.ico",
+    "tools/round-ico-corners.mjs",
+    "env/snapshot.json",
+    "LICENSE"
   ],
   "peerDependencies": {
     "@deepseek-ai/dsh-tools": "*"