tensorgrid-ui 1.3.0 → 1.4.0

package/lib/client.js

@@ -177,6 +177,14 @@ window.__ModuleLoader__.load({
       'intensity.vivid': 'Vivid',
       'accent.title': 'Obsidian / Ion accent',
       'accent.hint': 'Leading tone of the interface and the glow',
+      'review.nav': 'Reviewers',
+      'review.title': 'Second opinion',
+      'review.hint': 'Independent reviewers examine finished work and report what is wrong. They run as separate products and do not see this conversation.',
+      'review.model': 'Model',
+      'review.effort': 'Effort',
+      'review.asConfigured': 'as configured',
+      'review.none': 'No reviewer is selected — a review will find nothing.',
+      'review.note': 'Two reviewers are enabled by default because they find different things: asking one guarantees missing half. Leaving a model empty uses whatever that product is already set up with.',
       'auth.nav': 'Subscriptions',
       'auth.title': 'Provider sign-in',
       'auth.others': 'Only providers offering a subscription sign-in are listed. The rest — Z.AI among them — offer an API key only; configure those under Models.',
@@ -224,6 +232,14 @@ window.__ModuleLoader__.load({
       'intensity.vivid': '强烈',
       'accent.title': 'Obsidian / Ion 主色',
       'accent.hint': '界面与光晕的主色调',
+      'review.nav': '审阅者',
+      'review.title': '第二意见',
+      'review.hint': '独立审阅者检查已完成的工作并报告问题。它们作为独立产品运行，看不到本次对话。',
+      'review.model': '模型',
+      'review.effort': '推理强度',
+      'review.asConfigured': '按产品设置',
+      'review.none': '未选择任何审阅者——审阅将一无所获。',
+      'review.note': '默认启用两位审阅者，因为他们发现的问题不同：只用一位必然漏掉一半。模型留空则沿用该产品自身的设置。',
       'auth.nav': '订阅',
       'auth.title': '登录服务商',
       'auth.others': '这里只列出提供订阅登录的服务商。其余（包括 Z.AI）只提供 API 密钥，请在「模型」中配置。',
@@ -1568,6 +1584,14 @@ window.__ModuleLoader__.load({
       'intensity.vivid': 'Ярко',
       'accent.title': 'Акцент Obsidian / Ion',
       'accent.hint': 'Ведущий тон интерфейса и свечения',
+      'review.nav': 'Ревизоры',
+      'review.title': 'Второе мнение',
+      'review.hint': 'Независимые ревизоры смотрят готовую работу и сообщают, что в ней не так. Работают как отдельные продукты и нашей переписки не видят.',
+      'review.model': 'Модель',
+      'review.effort': 'Усилие',
+      'review.asConfigured': 'как настроено',
+      'review.none': 'Не выбран ни один ревизор — обзор ничего не найдёт.',
+      'review.note': 'По умолчанию включены оба: они находят разное, и один гарантированно упускает половину. Пустая модель означает «как настроено у самого продукта».',
       'auth.nav': 'Подписки',
       'auth.title': 'Вход к провайдерам',
       'auth.others': 'Здесь только провайдеры, у которых есть вход по подписке. Остальные — в том числе Z.AI — предлагают лишь ключ API; их место в разделе «Модели».',
@@ -1716,7 +1740,7 @@ window.__ModuleLoader__.load({
       // поэтому строка остаётся согласованной при любой палитре.
       '.dsx-setting{display:flex;align-items:center;justify-content:space-between;gap:16px;width:100%;}',
       '.dsx-setting__title{color:var(--dsw-alias-label-primary);font-size:13px;font-weight:500;}',
-      '.dsx-subs{display:flex;flex-direction:column;gap:14px;padding:4px 0}.dsx-subs__head{display:flex;flex-direction:column;gap:4px}.dsx-subs__title{font-weight:600;font-size:15px;color:var(--dsw-alias-label-primary)}.dsx-subs__note{padding-top:6px;border-top:1px solid var(--dsw-alias-border-l1);color:var(--dsw-alias-label-tertiary);font-size:12px;line-height:1.5}.dsx-setting--stack{flex-direction:column;align-items:stretch;gap:10px}.dsx-auth__list{display:flex;flex-direction:column;gap:6px}.dsx-auth__row{display:flex;align-items:center;justify-content:space-between;gap:12px}.dsx-auth__name{color:var(--dsw-alias-label-primary)}.dsx-auth__badge{margin-left:8px;padding:1px 6px;border-radius:999px;font-size:11px;color:var(--dsw-alias-state-success-primary);border:1px solid var(--dsw-alias-border-l2)}.dsx-auth__panel{display:flex;flex-direction:column;gap:8px;padding:10px;border-radius:8px;background:var(--dsw-alias-bg-layer-2);border:1px solid var(--dsw-alias-border-l1)}.dsx-auth__notice,.dsx-auth__prompt{display:flex;flex-direction:column;gap:6px;color:var(--dsw-alias-label-secondary)}.dsx-auth__link{color:var(--dsw-alias-brand-primary);word-break:break-all}.dsx-auth__code{font-family:ui-monospace,monospace;letter-spacing:.08em;color:var(--dsw-alias-label-primary)}.dsx-auth__input{flex:1;min-width:0;padding:4px 8px;border-radius:6px;border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-layer-1);color:var(--dsw-alias-label-primary)}.dsx-setting__hint--strong{color:var(--dsw-alias-label-primary)}.dsx-setting__hint--warn{color:var(--dsw-alias-state-warn-primary)}.dsx-setting__hint{color:var(--dsw-alias-label-secondary);font-size:12px;margin-top:2px;}',
+      '.dsx-rev__list{display:flex;flex-direction:column;gap:8px}.dsx-rev__card{padding:10px 12px;border-radius:8px;border:1px solid var(--dsw-alias-border-l1);background:var(--dsw-alias-bg-layer-2);opacity:.6;transition:opacity .15s ease,border-color .15s ease}.dsx-rev__card--on{opacity:1;border-color:var(--dsw-alias-border-l2)}.dsx-rev__head{display:flex;align-items:center;gap:8px;cursor:pointer}.dsx-rev__name{font-weight:600;color:var(--dsw-alias-label-primary)}.dsx-rev__fields{display:flex;flex-wrap:wrap;gap:12px;margin-top:8px;padding-left:22px}.dsx-rev__field{display:flex;align-items:center;gap:6px;color:var(--dsw-alias-label-secondary);font-size:12px}.dsx-rev__select{padding:3px 6px;border-radius:6px;border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-layer-1);color:var(--dsw-alias-label-primary);font-size:12px}.dsx-subs{display:flex;flex-direction:column;gap:14px;padding:4px 0}.dsx-subs__head{display:flex;flex-direction:column;gap:4px}.dsx-subs__title{font-weight:600;font-size:15px;color:var(--dsw-alias-label-primary)}.dsx-subs__note{padding-top:6px;border-top:1px solid var(--dsw-alias-border-l1);color:var(--dsw-alias-label-tertiary);font-size:12px;line-height:1.5}.dsx-setting--stack{flex-direction:column;align-items:stretch;gap:10px}.dsx-auth__list{display:flex;flex-direction:column;gap:6px}.dsx-auth__row{display:flex;align-items:center;justify-content:space-between;gap:12px}.dsx-auth__name{color:var(--dsw-alias-label-primary)}.dsx-auth__badge{margin-left:8px;padding:1px 6px;border-radius:999px;font-size:11px;color:var(--dsw-alias-state-success-primary);border:1px solid var(--dsw-alias-border-l2)}.dsx-auth__panel{display:flex;flex-direction:column;gap:8px;padding:10px;border-radius:8px;background:var(--dsw-alias-bg-layer-2);border:1px solid var(--dsw-alias-border-l1)}.dsx-auth__notice,.dsx-auth__prompt{display:flex;flex-direction:column;gap:6px;color:var(--dsw-alias-label-secondary)}.dsx-auth__link{color:var(--dsw-alias-brand-primary);word-break:break-all}.dsx-auth__code{font-family:ui-monospace,monospace;letter-spacing:.08em;color:var(--dsw-alias-label-primary)}.dsx-auth__input{flex:1;min-width:0;padding:4px 8px;border-radius:6px;border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-layer-1);color:var(--dsw-alias-label-primary)}.dsx-setting__hint--strong{color:var(--dsw-alias-label-primary)}.dsx-setting__hint--warn{color:var(--dsw-alias-state-warn-primary)}.dsx-setting__hint{color:var(--dsw-alias-label-secondary);font-size:12px;margin-top:2px;}',
       '.dsx-setting__control{display:inline-flex;padding:2px;gap:2px;border:1px solid var(--dsw-alias-border-l1);border-radius:8px;background:var(--dsw-alias-bg-layer-2);flex:none;}',
       '.dsx-seg{appearance:none;border:0;cursor:pointer;padding:4px 10px;border-radius:6px;font-size:12px;line-height:18px;background:transparent;color:var(--dsw-alias-label-secondary);transition:background .16s ease,color .16s ease;}',
       '.dsx-seg:hover{color:var(--dsw-alias-label-primary);}',
@@ -2217,6 +2241,154 @@ window.__ModuleLoader__.load({
       })
     }
 
+    // ── Ревизоры ─────────────────────────────────────────────────────────
+    //
+    // Раздел настроек, где выбирают, кто смотрит работу вторым мнением,
+    // какой моделью и с каким усилием.
+    //
+    // Справочник моделей приходит с Host, а не зашит здесь: иначе список
+    // расширят в одной половине и забудут в другой. Пустой выбор модели
+    // означает «как настроено у самого продукта» — человек уже настроил
+    // свой Claude Code и Codex, и навязывать поверх нечего.
+    const REVIEW_PATH = '/api/tensorgrid.review'
+
+    function ReviewRow({ t }) {
+      const [state, setState] = React.useState(null)
+      const [busy, setBusy] = React.useState(false)
+
+      const translate = React.useRef(t)
+      translate.current = t
+
+      const send = React.useCallback(async (body) => {
+        const response = body === undefined
+          ? await fetch(REVIEW_PATH)
+          : await fetch(REVIEW_PATH, {
+              method: 'POST',
+              headers: { 'content-type': 'application/json' },
+              body: JSON.stringify(body),
+            })
+        const type = response.headers.get('content-type') || ''
+        if (!type.includes('application/json')) {
+          const say = translate.current
+          return {
+            catalogue: [],
+            problem: response.status === 401 || response.status === 404
+              ? say('auth.needsRestart')
+              : say('auth.badResponse').replace('{status}', String(response.status)),
+          }
+        }
+        return response.json()
+      }, [])
+
+      const refresh = React.useCallback(async (body) => {
+        setBusy(true)
+        try {
+          setState(await send(body))
+        } catch (error) {
+          setState({ catalogue: [], problem: String(error && error.message ? error.message : error) })
+        } finally {
+          setBusy(false)
+        }
+      }, [send])
+
+      React.useEffect(() => { refresh() }, [refresh])
+
+      const catalogue = state !== null && Array.isArray(state.catalogue) ? state.catalogue : []
+      const settings = state !== null && state.settings ? state.settings : null
+
+      // Изменение любого поля сразу сохраняется: отдельная кнопка «применить»
+      // добавляет состояние «изменено, но не сохранено», в котором легко
+      // забыть нажать и потом удивляться.
+      const save = React.useCallback((next) => { refresh(next) }, [refresh])
+
+      const toggle = (kind) => {
+        if (settings === null) return
+        const on = settings.enabled.includes(kind)
+        const enabled = on ? settings.enabled.filter((k) => k !== kind) : [...settings.enabled, kind]
+        save({ ...settings, enabled })
+      }
+
+      const pick = (kind, field, value) => {
+        if (settings === null) return
+        const choices = { ...settings.choices, [kind]: { ...settings.choices[kind], [field]: value } }
+        save({ ...settings, choices })
+      }
+
+      const selector = (kind, field, options, current) =>
+        jsxs('select', {
+          className: 'dsx-rev__select',
+          value: current === null ? '' : current,
+          disabled: busy,
+          onChange: (event) => pick(kind, field, event.target.value === '' ? null : event.target.value),
+          children: [
+            jsx('option', { value: '', children: t('review.asConfigured') }, ''),
+            ...options.map((option) => jsx('option', { value: option, children: option }, option)),
+          ],
+        })
+
+      return jsxs('div', {
+        className: 'dsx-subs',
+        children: [
+          jsxs('div', {
+            className: 'dsx-subs__head',
+            children: [
+              jsx('div', { className: 'dsx-subs__title', children: t('review.title') }),
+              jsx('div', { className: 'dsx-setting__hint', children: t('review.hint') }),
+            ],
+          }),
+          state !== null && state.problem
+            ? jsx('div', { className: 'dsx-setting__hint dsx-setting__hint--warn', children: state.problem })
+            : null,
+          jsx('div', {
+            className: 'dsx-rev__list',
+            children: catalogue.map((entry) => {
+              const on = settings !== null && settings.enabled.includes(entry.kind)
+              const choice = settings === null ? { model: null, effort: null } : settings.choices[entry.kind]
+              return jsxs('div', {
+                className: 'dsx-rev__card' + (on ? ' dsx-rev__card--on' : ''),
+                children: [
+                  jsxs('label', {
+                    className: 'dsx-rev__head',
+                    children: [
+                      jsx('input', {
+                        type: 'checkbox',
+                        checked: on,
+                        disabled: busy,
+                        onChange: () => toggle(entry.kind),
+                      }),
+                      jsx('span', { className: 'dsx-rev__name', children: entry.label }),
+                    ],
+                  }),
+                  jsxs('div', {
+                    className: 'dsx-rev__fields',
+                    children: [
+                      jsxs('label', {
+                        className: 'dsx-rev__field',
+                        children: [
+                          jsx('span', { children: t('review.model') }),
+                          selector(entry.kind, 'model', entry.models, choice.model),
+                        ],
+                      }),
+                      jsxs('label', {
+                        className: 'dsx-rev__field',
+                        children: [
+                          jsx('span', { children: t('review.effort') }),
+                          selector(entry.kind, 'effort', entry.efforts, choice.effort),
+                        ],
+                      }),
+                    ],
+                  }),
+                ],
+              }, entry.kind)
+            }),
+          }),
+          settings !== null && settings.enabled.length === 0
+            ? jsx('div', { className: 'dsx-setting__hint dsx-setting__hint--warn', children: t('review.none') })
+            : null,
+          jsx('div', { className: 'dsx-subs__note', children: t('review.note') }),
+        ],
+      })
+    }
     // ── Обновления ───────────────────────────────────────────────────────
     //
     // Маршрут поднимает host-половина пакета. Путь относительный, токен не
@@ -2444,6 +2616,21 @@ window.__ModuleLoader__.load({
         ),
       )
 
+      // Ревизоры — соседний раздел: и подписки, и обзор про то, чем и как
+      // работает агент, а не про внешний вид.
+      ctx.slots.inject('settings.section', () =>
+        ctx.slots.register(
+          {
+            name: 'settings.section',
+            id: 'tensorgrid-review',
+            order: 13,
+            locale: LOCALE_NS,
+            label: () => ctx.locale.translate(LOCALE_NS, 'review.nav'),
+          },
+          ReviewRow,
+        ),
+      )
+
       ctx.slots.inject('settings.general.item', () =>
         ctx.slots.register(
           { name: 'settings.general.item', id: 'obsidian-ion-update', order: 11, locale: LOCALE_NS },

package/lib/http-runtime.js

@@ -215,6 +215,60 @@ export async function handleUpdate(req, res, state, loadModule) {
   }
 }
 
+/**
+ * Обслужить `/api/tensorgrid.review` — настройки обзора.
+ *
+ * GET отдаёт выбранное и справочник для показа: кого можно звать, какие
+ * модели и усилия у каждого. Справочник приходит с Host, а не зашит в
+ * браузер: иначе список расширят в одной половине и забудут в другой.
+ *
+ * POST сохраняет выбор. Форма проверяется на Host — файл правит и человек
+ * руками, и надеяться, что дошло только отправленное браузером, нельзя.
+ */
+export async function handleReview(req, res, loadModule) {
+  try {
+    const settings = await loadModule('./review-settings.js')
+    const { REVIEWERS } = await loadModule('./reviewers.js')
+
+    const catalogue = Object.entries(REVIEWERS).map(([kind, spec]) => ({
+      kind,
+      label: spec.label,
+      models: spec.models,
+      efforts: spec.efforts,
+    }))
+
+    if (req.method === 'GET') {
+      reply(res, 200, { catalogue, settings: await settings.readSettings(null) })
+      return
+    }
+
+    if (req.method !== 'POST') {
+      reply(res, 405, { problem: 'поддерживаются GET и POST' })
+      return
+    }
+
+    if (!acceptsJson(req)) {
+      reply(res, 415, { problem: 'ожидается content-type: application/json' })
+      return
+    }
+
+    if (!sameOrigin(req)) {
+      reply(res, 403, { problem: 'запрос с чужой страницы' })
+      return
+    }
+
+    const body = await readJsonBody(req)
+    if (body === null) {
+      reply(res, 400, { problem: 'тело запроса не разобрано' })
+      return
+    }
+
+    reply(res, 200, { catalogue, settings: await settings.writeSettings(null, body) })
+  } catch (error) {
+    reply(res, 500, { problem: String(error?.message ?? error) })
+  }
+}
+
 /** Обслужить `/api/tensorgrid.auth`. */
 export async function handleAuth(req, res, services, loadModule) {
   try {

package/lib/index.js

@@ -240,6 +240,21 @@ export function apply(ctx) {
       }),
     )
 
+    web.effect(() =>
+      web.webServer.register({
+        kind: 'exact',
+        path: '/api/tensorgrid.review',
+        async handler(req, res) {
+          try {
+            const runtime = await loadModule('./http-runtime.js')
+            await runtime.handleReview(req, res, loadModule)
+          } catch (error) {
+            fail(res, error)
+          }
+        },
+      }),
+    )
+
     web.effect(() =>
       web.webServer.register({
         kind: 'exact',

package/lib/review-settings.js

@@ -0,0 +1,110 @@
+/**
+ * Настройки обзора: кого звать, какой моделью и с каким усилием.
+ *
+ * Хранятся файлом рядом с профилем, а не в браузере. Причина простая:
+ * выбирает человек в интерфейсе, а читает Host в момент вызова инструмента —
+ * значит хранилище должно быть общим для обеих сторон. Память браузера для
+ * этого не годится, Host её не видит.
+ *
+ * Файл лежит там же, где отметка об установке, и ищется тем же подъёмом
+ * вверх: так путь не зависит от того, насколько глубоко pnpm разложил пакет.
+ *
+ * Импортируются только встроенные модули Node.
+ */
+import { readFile, 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']
+
+/**
+ * Что действует, пока пользователь ничего не выбрал.
+ *
+ * Оба ревизора включены намеренно: они находят РАЗНОЕ, и один по умолчанию
+ * означал бы, что половина находок не появится у того, кто в настройки не
+ * заглядывал. Модель и усилие пустые — берутся собственные настройки
+ * продукта, то есть то, чем человек уже пользуется.
+ */
+export function defaults() {
+  return {
+    enabled: [...KNOWN],
+    choices: { 'claude-code': { model: null, effort: null }, codex: { model: null, effort: null } },
+  }
+}
+
+/**
+ * Каталог профиля: поднимаемся, пока не найдём отметку об установке.
+ * @param startDir - откуда начинать подъём.
+ */
+function findProfileRoot(startDir) {
+  let dir = startDir
+  for (let depth = 0; depth < 10; depth += 1) {
+    if (existsSync(join(dir, 'tensorgrid-installed.json'))) return dir
+    const parent = dirname(dir)
+    if (parent === dir) break
+    dir = parent
+  }
+  return null
+}
+
+/**
+ * Где лежит файл настроек.
+ *
+ * Сначала ищем от собственного расположения — так работает установленный
+ * пакет. Если отметки нет (запуск из репозитория, проверка), кладём рядом с
+ * рабочей папкой, чтобы проверяемый код вёл себя как настоящий.
+ *
+ * @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 normalise(raw) {
+  const base = defaults()
+  if (raw === null || typeof raw !== 'object') return base
+
+  const enabled = Array.isArray(raw.enabled) ? 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
+    const pick = (value) => (typeof value === 'string' && value.trim() !== '' ? value.trim() : null)
+    choices[kind] = {
+      model: pick(source?.model),
+      effort: pick(source?.effort),
+    }
+  }
+
+  // Пустой список допустим только как осознанный выбор «никого не звать»;
+  // отличить его от испорченного файла нельзя, поэтому возвращаем как есть.
+  return { enabled, choices }
+}
+
+/** Прочитать настройки. Отсутствие файла — не ошибка, а «ничего не выбрано». */
+export async function readSettings(cwd) {
+  const path = settingsPath(cwd)
+  if (path === null) return defaults()
+  try {
+    const text = await readFile(path, 'utf8')
+    return normalise(JSON.parse(text.replace(/^\uFEFF/, '')))
+  } catch {
+    return defaults()
+  }
+}
+
+/** Записать настройки. Возвращает то, что действительно сохранено. */
+export async function writeSettings(cwd, value) {
+  const path = settingsPath(cwd)
+  if (path === null) throw new Error('не найден каталог профиля — некуда сохранить настройки обзора')
+  const normalised = normalise(value)
+  await writeFile(path, `${JSON.stringify(normalised, null, 2)}\n`, 'utf8')
+  return normalised
+}

package/lib/review.js

@@ -2,8 +2,12 @@
  * «Второе мнение» — независимый обзор работы чужими исполнителями.
  *
  * Это строка АГЕНТ-ПЛОСКОСТИ: инструмент регистрируется в реестр пресета, а
- * не профиля. Провайдеры подагентов живут на Host и монтируются отдельно;
- * наличие провайдера само по себе прав агенту не даёт, их даёт эта строка.
+ * не профиля.
+ *
+ * Продукты вызываются НАПРЯМУЮ через `ctx.subprocess`, а не через готовые
+ * провайдеры подагентов. Причина в `lib/reviewers.js`: провайдеры не
+ * передают дочернему продукту модель и усилие, а без них выбор из
+ * интерфейса невозможен.
  *
  * Зачем это нужно, коротко. Ошибки бывают громкие и тихие. Громкие падают с
  * криком, и их находит первый же пользователь. Тихие проявляются только в
@@ -151,47 +155,28 @@ export function parseFindings(text, reviewer) {
   return findings
 }
 
-/** Вытащить текст из блоков ответа подагента. */
-function textOf(output) {
-  if (!Array.isArray(output)) return ''
-  return output.filter((block) => block?.type === 'text').map((block) => String(block.text ?? '')).join('')
-}
-
 /**
- * Позвать одного ревизора и вернуть его находки.
+ * Позвать одного ревизора и разобрать его находки.
  *
  * Отказ одного НЕ отменяет обзор: половина ответа полезнее отказа, и
  * пользователь должен видеть, кто именно не ответил.
  */
-async function askReviewer(ctx, provider, prompt, parent, signal) {
-  let run
-  try {
-    run = await ctx.subagents.start(provider, {
-      label: `обзор: ${provider}`,
-      prompt: [{ type: 'text', text: prompt }],
-      parent,
-      signal,
-    })
-  } catch (error) {
-    return { provider, failed: String(error?.message ?? error), findings: [] }
-  }
+async function askReviewer(ctx, runReviewer, kind, prompt, cwd, choice, signal) {
+  const outcome = await runReviewer(ctx, kind, {
+    prompt,
+    cwd,
+    model: choice?.model ?? null,
+    effort: choice?.effort ?? null,
+    signal,
+  })
 
-  try {
-    const result = await run.result
-    const answer = textOf(result?.output)
-    if (answer.trim() === '') {
-      return { provider, failed: 'пустой ответ', findings: [] }
-    }
-    return { provider, failed: null, findings: parseFindings(answer, provider), answer }
-  } catch (error) {
-    return { provider, failed: String(error?.message ?? error), findings: [] }
-  } finally {
-    try {
-      await run.dispose()
-    } catch {
-      // Неудача уборки не должна подменять результат обзора.
-    }
+  const used = { model: choice?.model ?? null, effort: choice?.effort ?? null }
+
+  if (outcome.failed !== null) {
+    return { provider: kind, failed: outcome.failed, findings: [], ...used }
   }
+
+  return { provider: kind, failed: null, findings: parseFindings(outcome.answer, kind), ...used }
 }
 
 /** Отсортировать находки: сначала опасные, внутри уровня — по ревизору. */
@@ -246,10 +231,10 @@ export function render(value) {
 }
 
 export const name = 'tensorgrid-second-opinion'
-export const inject = ['tools', 'subagents']
+export const inject = ['tools', 'subprocess']
 
 /**
- * @param ctx - контекст агент-плоскости с реестрами `tools` и `subagents`.
+ * @param ctx - контекст агент-плоскости со службами `tools` и `subprocess`.
  */
 export async function apply(ctx) {
   const { defineTool } = await import('@deepseek-ai/dsh-tools')
@@ -298,17 +283,24 @@ export async function apply(ctx) {
         render: (_args, value) => [{ type: 'text', text: render(value) }],
       },
       async execute(args, exec) {
-        const parent = exec.agent
-        const available = ctx.subagents.list()
-        const wanted = Array.isArray(args.reviewers) && args.reviewers.length > 0 ? args.reviewers : REVIEWERS
-        const usable = wanted.filter((provider) => available.includes(provider))
-
-        if (usable.length === 0) {
-          return {
-            findings: [],
-            reviewers: [],
-            problem: `нет доступных ревизоров; провайдеры на Host: ${available.join(', ') || 'нет'}`,
-          }
+        const { runReviewer } = await import('./reviewers.js')
+        const { readSettings } = await import('./review-settings.js')
+
+        // Рабочая папка берётся у сессии, как это делают поставочные
+        // провайдеры: ревизор должен смотреть тот же код, что и автор.
+        const cwd = exec.agent?.session?.header?.cwd
+        if (typeof cwd !== 'string' || cwd === '') {
+          return { findings: [], reviewers: [], problem: 'у сессии нет рабочей папки — обзору нечего смотреть' }
+        }
+
+        // Выбор пользователя из настроек; вызов может его перекрыть.
+        const settings = await readSettings(cwd)
+        const wanted = Array.isArray(args.reviewers) && args.reviewers.length > 0
+          ? args.reviewers
+          : settings.enabled
+
+        if (wanted.length === 0) {
+          return { findings: [], reviewers: [], problem: 'в настройках не выбран ни один ревизор' }
         }
 
         const prompt = buildPrompt(args)
@@ -316,12 +308,18 @@ export async function apply(ctx) {
         // Спрашиваем ОБОИХ разом: они независимы, и последовательный опрос
         // удвоил бы ожидание без всякой пользы.
         const answers = await Promise.all(
-          usable.map((provider) => askReviewer(ctx, provider, prompt, parent, exec.signal)),
+          wanted.map((kind) => askReviewer(ctx, runReviewer, kind, prompt, cwd, settings.choices[kind], exec.signal)),
         )
 
         return {
           findings: ordered(answers.flatMap((entry) => entry.findings)),
-          reviewers: answers.map((entry) => ({ provider: entry.provider, failed: entry.failed, found: entry.findings.length })),
+          reviewers: answers.map((entry) => ({
+            provider: entry.provider,
+            failed: entry.failed,
+            found: entry.findings.length,
+            model: entry.model,
+            effort: entry.effort,
+          })),
         }
       },
     }),

package/lib/reviewers.js

@@ -0,0 +1,199 @@
+/**
+ * Запуск ревизоров — Claude Code и Codex — с выбором модели и усилия.
+ *
+ * ПОЧЕМУ НЕ ЧЕРЕЗ ПРОВАЙДЕРОВ ПОДАГЕНТОВ
+ *
+ * Готовые провайдеры `codex` и `claude-code` запускают те же продукты, но
+ * берут из вызова только задание, отмену и родителя. Модель и усилие они не
+ * передают: у них это задаётся окружением в строке композиции, то есть один
+ * раз при старте. Выбор из интерфейса так не сделать — пришлось бы монтировать
+ * отдельного провайдера на каждое сочетание и перезапускать приложение.
+ *
+ * Поэтому продукты вызываются напрямую, через `ctx.subprocess` — ту же службу,
+ * которой пользуются сами провайдеры. Она даёт поиск команды в PATH, очистку
+ * окружения от чужих учётных данных, управление деревом процессов и
+ * остановку по сигналу. Своего здесь — только сборка аргументов.
+ *
+ * ДВЕ ТОНКОСТИ, СТОИВШИЕ ВРЕМЕНИ
+ *
+ * Задание передаётся через ПОТОК ВВОДА, а не аргументом. Обе команды так
+ * умеют, а длинное задание с кавычками в аргументе командной строки ломается:
+ * сегодня это уже случилось — кавычки в тексте разорвали разбор аргументов.
+ *
+ * Codex печатает в вывод весь ход рассуждения — в одном из прогонов вышло
+ * 138 КБ на страницу ответа. Итог забирается отдельным файлом через
+ * `--output-last-message`. Claude Code в режиме `-p` и так печатает только
+ * ответ.
+ */
+import { mkdtemp, readFile, rm } from 'node:fs/promises'
+import { tmpdir } from 'node:os'
+import { join } from 'node:path'
+
+/** Сколько ждать ревизора. Обзор большого кода у Codex занимал 450 секунд. */
+const DEFAULT_TIMEOUT_MS = 900000
+
+/** Предел собираемого вывода: ответ ревизора — текст, а не поток данных. */
+const STDOUT_CAP = 1024 * 1024
+const STDERR_CAP = 64 * 1024
+
+/**
+ * Что известно про каждого ревизора.
+ *
+ * `models` и `efforts` — не ограничение, а подсказка для интерфейса: список
+ * показывается пользователю, но передать можно и не перечисленное. Жёсткий
+ * список устарел бы при первом же выпуске нового имени модели.
+ */
+export const REVIEWERS = {
+  'claude-code': {
+    label: 'Claude Code',
+    command: 'claude',
+    models: ['claude-opus-4-8', 'claude-sonnet-4-6'],
+    efforts: ['low', 'medium', 'high'],
+    defaultModel: null,
+    defaultEffort: null,
+  },
+  codex: {
+    label: 'Codex',
+    command: 'codex',
+    models: ['gpt-6-astra', 'gpt-5.6-sol'],
+    efforts: ['low', 'medium', 'high', 'xhigh'],
+    defaultModel: null,
+    defaultEffort: null,
+  },
+}
+
+/** Значения уходят в аргументы команды — пропускаем только безобидное. */
+const SAFE_VALUE = /^[A-Za-z0-9][A-Za-z0-9._-]{0,63}$/
+
+/**
+ * Собрать аргументы запуска.
+ *
+ * @param kind - имя ревизора.
+ * @param options - модель, усилие, файл для итога.
+ * @returns аргументы без имени программы.
+ */
+export function buildArgs(kind, { model, effort, answerFile }) {
+  const args = []
+
+  if (kind === 'claude-code') {
+    // `-p` без текста задания: задание придёт через поток ввода.
+    args.push('-p')
+    // Ревизор только читает, но запрет на чтение файлов сделал бы его
+    // бесполезным, а спрашивать разрешение в фоне некому.
+    args.push('--permission-mode', 'bypassPermissions')
+    if (model) args.push('--model', model)
+    if (effort) args.push('--effort', effort)
+    return args
+  }
+
+  if (kind === 'codex') {
+    args.push('exec')
+    // Обзор ничего не меняет: только чтение. Это и защита, и честность —
+    // ревизор не должен «починить» по дороге то, что взялся оценивать.
+    args.push('--sandbox', 'read-only')
+    // Рабочая папка может не быть репозиторием, и это не повод отказываться.
+    args.push('--skip-git-repo-check')
+    args.push('--output-last-message', answerFile)
+    if (model) args.push('-m', model)
+    // Значение читается как TOML, поэтому строка в кавычках.
+    if (effort) args.push('-c', `model_reasoning_effort="${effort}"`)
+    return args
+  }
+
+  throw new Error(`неизвестный ревизор: ${kind}`)
+}
+
+/**
+ * Запустить одного ревизора и вернуть его ответ.
+ *
+ * Отказ не бросается наружу: обзор двумя ревизорами не должен пропадать
+ * целиком из-за одного неответившего. Возвращается либо ответ, либо причина.
+ *
+ * @param ctx - контекст со службой `subprocess`.
+ * @param kind - имя ревизора.
+ * @param options - задание, рабочая папка, модель, усилие, сигнал отмены.
+ */
+export async function runReviewer(ctx, kind, { prompt, cwd, 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))) {
+      return { kind, failed: `недопустимое значение (${label}): ${String(value)}`, answer: null }
+    }
+  }
+
+  let workDir = null
+  let handle = null
+  // Своя отмена поверх пользовательской: ревизор без ограничения по времени
+  // держал бы обзор бесконечно.
+  const deadline = new AbortController()
+  const timer = setTimeout(() => deadline.abort(), timeoutMs ?? DEFAULT_TIMEOUT_MS)
+  const onOuterAbort = () => deadline.abort()
+  if (signal !== undefined) signal.addEventListener('abort', onOuterAbort, { once: true })
+
+  try {
+    workDir = await mkdtemp(join(tmpdir(), 'tensorgrid-review-'))
+    const answerFile = join(workDir, 'answer.txt')
+
+    let executable
+    try {
+      executable = await ctx.subprocess.resolveExecutable(spec.command, undefined, deadline.signal)
+    } catch (error) {
+      return { kind, failed: `${spec.command} не найден: ${String(error?.message ?? error)}`, answer: null }
+    }
+
+    handle = ctx.subprocess.spawn({
+      argv: [executable, ...buildArgs(kind, { model, effort, answerFile })],
+      cwd,
+      stdio: {
+        // Задание через поток ввода: кавычки и переводы строк проходят как есть.
+        stdin: { data: prompt },
+        stdout: { maxBytes: STDOUT_CAP },
+        stderr: { maxBytes: STDERR_CAP },
+      },
+      graceMs: 5000,
+      signal: deadline.signal,
+    })
+
+    const outcome = await handle.done
+
+    // Codex кладёт итог в файл, Claude Code печатает его в вывод.
+    let answer = ''
+    if (kind === 'codex') {
+      try {
+        answer = await readFile(answerFile, 'utf8')
+      } catch {
+        answer = ''
+      }
+    }
+    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: null, answer, model: model ?? null, effort: effort ?? null }
+  } catch (error) {
+    const reason = deadline.signal.aborted && signal?.aborted !== true ? 'превышено время ожидания' : String(error?.message ?? error)
+    return { kind, failed: reason, answer: null }
+  } finally {
+    clearTimeout(timer)
+    if (signal !== undefined) signal.removeEventListener('abort', onOuterAbort)
+    if (handle !== null) {
+      try {
+        handle.terminate()
+      } catch {
+        // Уборка не должна подменять результат.
+      }
+    }
+    if (workDir !== null) {
+      await rm(workDir, { recursive: true, force: true }).catch(() => {})
+    }
+  }
+}

package/package.json

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