tensorgrid-ui 1.2.0 → 1.3.0

package/lib/auth-bridge.js

@@ -120,6 +120,14 @@ export function begin(authorization, key, method) {
         if (current.notices.length > MAX_NOTICES) current.notices.shift()
       },
       prompt(prompt) {
+        // Поток задал новый вопрос, не дождавшись ответа на прежний. Прежнее
+        // обещание нужно закрыть, иначе оно повисает навсегда, а вместе с ним
+        // и вся попытка: ключ занят, отвечать уже некому.
+        if (current.resolvePrompt !== null) {
+          const orphan = current.resolvePrompt
+          current.resolvePrompt = null
+          orphan('')
+        }
         current.prompt = prompt
         return new Promise((resolve) => { current.resolvePrompt = resolve })
       },
@@ -143,31 +151,59 @@ export function begin(authorization, key, method) {
   return { attempt: view() }
 }
 
-/** Отвечает на вопрос, который задал поток. */
+/**
+ * Отвечает на вопрос, который задал поток.
+ *
+ * Значение приводится к строке ДО того, как вопрос снят. Обратный порядок
+ * подвешивал разговор: тело вида `{"value":{"toString":null}}` роняло
+ * приведение уже после очистки, обещание оставалось без ответа, а функция
+ * его разрешения терялась — попытка числилась активной, но вопрос исчезал.
+ */
 export function answer(value) {
   if (attempt === null || attempt.resolvePrompt === null) {
     return { problem: 'поток ничего не спрашивает', attempt: view() }
   }
+
+  let text
+  try {
+    text = typeof value === 'string' ? value : String(value ?? '')
+  } catch {
+    return { problem: 'ответ не приводится к тексту', attempt: view() }
+  }
+
   const resolve = attempt.resolvePrompt
   attempt.resolvePrompt = null
   attempt.prompt = null
-  resolve(String(value))
+  resolve(text)
   return { attempt: view() }
 }
 
 /**
- * Снимает попытку. Нужно не только по желанию пользователя: стойка держит
- * ключ занятым, и осиротевшая попытка иначе блокирует повторный вход до
- * перезапуска.
+ * Снимает текущую попытку. Нужно не только по желанию пользователя: стойка
+ * держит ключ занятым, и осиротевшая попытка иначе блокирует повторный вход
+ * до перезапуска.
+ *
+ * Ключ из запроса НЕ принимается: снимать можно только то, что идёт сейчас.
+ * Иначе вторая вкладка — или посторонний запрос — снимала бы чужую попытку
+ * по произвольному имени.
  */
 export function cancel(authorization, key) {
-  const target = key ?? attempt?.key
-  if (typeof target !== 'string') return { problem: 'нечего отменять', attempt: view() }
+  if (attempt === null) return { problem: 'нечего отменять', attempt: view() }
+
+  const target = attempt.key
+  if (typeof key === 'string' && key !== '' && key !== target) {
+    return { problem: 'идёт попытка для другого провайдера', attempt: view() }
+  }
+
   authorization.cancel(target)
-  if (attempt !== null && attempt.key === target) {
-    attempt.problem = attempt.problem ?? 'отменено'
-    attempt.prompt = null
+  attempt.problem = attempt.problem ?? 'отменено'
+  attempt.prompt = null
+  if (attempt.resolvePrompt !== null) {
+    // Ожидающее обещание закрываем сами: иначе поток останется висеть на
+    // вопросе, на который уже никто не ответит.
+    const pending = attempt.resolvePrompt
     attempt.resolvePrompt = null
+    pending('')
   }
   return { attempt: view() }
 }

package/lib/client.js

@@ -1947,10 +1947,41 @@ window.__ModuleLoader__.load({
     // браузер начинает попытку и затем опрашивает состояние.
     const AUTH_PATH = '/api/tensorgrid.auth'
 
+    /**
+     * Пропустить только настоящий веб-адрес.
+     *
+     * Ссылка приходит от стойки входа, то есть в конечном счёте от внешнего
+     * провайдера. Подставлять её в `href` как есть нельзя: схема
+     * `javascript:` превратила бы «Открыть страницу входа» в запуск чужого
+     * кода прямо в приложении. Разрешаем http и https, остальное прячем.
+     *
+     * @param value - адрес из уведомления.
+     * @returns адрес, пригодный для ссылки, или null.
+     */
+    function safeUrl(value) {
+      if (typeof value !== 'string' || value === '') return null
+      try {
+        const parsed = new URL(value)
+        return parsed.protocol === 'http:' || parsed.protocol === 'https:' ? value : null
+      } catch {
+        return null
+      }
+    }
+
     function AuthRow({ t }) {
       const [state, setState] = React.useState(null)
+      // Занятость от ДЕЙСТВИЯ пользователя и фоновый опрос — разные вещи.
+      //
+      // Раньше опрос тоже поднимал этот признак, а поле ввода и кнопки на
+      // нём завязаны: каждые две секунды поле гасло прямо под руками. Это
+      // ломало ровно тот случай, ради которого опрос и заведён, — ввод кода.
       const [busy, setBusy] = React.useState(false)
       const [draft, setDraft] = React.useState('')
+      // Ответы приходят не в том порядке, в каком ушли: медленный опрос
+      // может вернуться после быстрого ответа и откатить состояние назад.
+      // Применяем только тот ответ, который свежее уже показанного.
+      const seq = React.useRef(0)
+      const applied = React.useRef(0)
 
       // Функция перевода приходит от рантайма НОВОЙ при каждом рендере —
       // в поставочном коде это обычная стрелка, не мемоизированная. Если
@@ -1991,14 +2022,35 @@ window.__ModuleLoader__.load({
         return response.json()
       }, [])
 
-      const refresh = React.useCallback(async (body) => {
-        setBusy(true)
+      /**
+       * Спросить Host и показать ответ.
+       *
+       * @param body - тело действия; без него идёт GET за списком.
+       * @param background - фоновый опрос: не блокирует поля и не стирает
+       *   показанное состояние, если сеть отвалилась. Пользователь в этот
+       *   момент вводит код, и мигание под руками недопустимо.
+       */
+      const refresh = React.useCallback(async (body, background) => {
+        const ticket = ++seq.current
+        if (!background) setBusy(true)
         try {
-          setState(await send(body))
+          const next = await send(body)
+          // Обгонять уже применённый ответ нельзя: иначе запоздавший опрос
+          // вернёт вопрос, на который только что ответили.
+          if (ticket > applied.current) {
+            applied.current = ticket
+            setState(next)
+          }
         } catch (error) {
-          setState({ available: true, problem: String(error && error.message ? error.message : error) })
+          // Сбой фонового опроса состояние НЕ трогает. Раньше он затирал его
+          // объектом без попытки, из-за чего опрос останавливался навсегда:
+          // интернет возвращался, а строка молчала до перезагрузки.
+          if (!background && ticket > applied.current) {
+            applied.current = ticket
+            setState({ available: true, problem: String(error && error.message ? error.message : error) })
+          }
         } finally {
-          setBusy(false)
+          if (!background) setBusy(false)
         }
       }, [send])
 
@@ -2009,7 +2061,7 @@ window.__ModuleLoader__.load({
       const active = state !== null && state.attempt !== undefined && state.attempt !== null && state.attempt.active === true
       React.useEffect(() => {
         if (!active) return undefined
-        const id = setInterval(() => { refresh({ action: 'poll' }) }, 2000)
+        const id = setInterval(() => { refresh({ action: 'poll' }, true) }, 2000)
         return () => clearInterval(id)
       }, [active, refresh])
 
@@ -2038,9 +2090,9 @@ window.__ModuleLoader__.load({
             className: 'dsx-auth__notice',
             children: [
               jsx('div', { children: notice.message }),
-              notice.url === null ? null : jsx('a', {
+              safeUrl(notice.url) === null ? null : jsx('a', {
                 className: 'dsx-auth__link',
-                href: notice.url,
+                href: safeUrl(notice.url),
                 target: '_blank',
                 rel: 'noreferrer',
                 children: t('auth.openLink'),

package/lib/http-runtime.js

@@ -27,30 +27,55 @@ function reply(res, code, value) {
  * Принять маленькое JSON-тело.
  *
  * Здесь всегда только действие, ключ провайдера или ответ на вопрос.
- * Ограничение 16 КБ не даёт случайному клиенту держать в памяти тело без
- * практической причины.
+ *
+ * Куски накапливаются БУФЕРАМИ, а не строкой. Склейка строкой вызывает
+ * `toString()` на каждом куске, и символ UTF-8, разорванный по границе
+ * сетевого пакета, превращается в «замену» — ответ с кириллицей изредка
+ * приезжал бы битым. Предел считается в БАЙТАХ по той же причине: длина
+ * строки и размер тела — разные величины.
+ *
+ * Тайм-аут обязателен: клиент, открывший запрос и не закрывший его, иначе
+ * держал бы обработчик и сокет бесконечно.
  */
-function readJsonBody(req) {
+function readJsonBody(req, limitBytes = 16384, timeoutMs = 15000) {
   return new Promise((resolve) => {
-    let text = ''
-    let tooBig = false
+    const chunks = []
+    let size = 0
+    let settled = false
+
+    const finish = (value) => {
+      if (settled) return
+      settled = true
+      clearTimeout(timer)
+      resolve(value)
+    }
+
+    const timer = setTimeout(() => {
+      req.destroy()
+      finish(null)
+    }, timeoutMs)
+
     req.on('data', (chunk) => {
-      if (tooBig) return
-      text += chunk
-      if (text.length > 16384) {
-        tooBig = true
-        resolve(null)
+      if (settled) return
+      const buffer = Buffer.isBuffer(chunk) ? chunk : Buffer.from(chunk)
+      size += buffer.length
+      if (size > limitBytes) {
+        req.destroy()
+        finish(null)
+        return
       }
+      chunks.push(buffer)
     })
     req.on('end', () => {
-      if (tooBig) return
+      if (settled) return
+      const text = Buffer.concat(chunks).toString('utf8')
       try {
-        resolve(JSON.parse(text === '' ? '{}' : text))
+        finish(JSON.parse(text === '' ? '{}' : text))
       } catch {
-        resolve(null)
+        finish(null)
       }
     })
-    req.on('error', () => resolve(null))
+    req.on('error', () => finish(null))
   })
 }
 
@@ -59,6 +84,95 @@ function acceptsJson(req) {
   return String(req.headers['content-type'] ?? '').includes('application/json')
 }
 
+/**
+ * Запрос пришёл со своей же страницы?
+ *
+ * Межсайтовый POST с нашим типом содержимого браузер и так не пустит без
+ * предварительной проверки CORS, которую мы не проходим. Но держать защиту
+ * только на чужом поведении неправильно: явная сверка источника дешева и
+ * делает намерение видимым в коде.
+ *
+ * Отсутствие заголовка допускается намеренно: его не шлют не-браузерные
+ * клиенты, а локальный процесс и без того читает файл с учётными данными
+ * напрямую — запрет ему ничего бы не добавил.
+ */
+function sameOrigin(req) {
+  const origin = req.headers.origin
+  if (typeof origin !== 'string' || origin === '') return true
+  const host = String(req.headers.host ?? '')
+  if (host === '') return false
+  try {
+    return new URL(origin).host === host
+  } catch {
+    return false
+  }
+}
+
+/**
+ * Найти поставочный файл фронтенда.
+ *
+ * Профили делят общий `node_modules`, поэтому от нашего кода до него —
+ * подъём на несколько уровней. Ищем перебором вверх, а не по заданному
+ * пути: глубина вложения зависит от того, как pnpm разложил пакеты.
+ *
+ * @param fromDir - каталог, от которого подниматься.
+ * @param relative - путь внутри `node_modules`.
+ * @returns полный путь или null, если не нашли.
+ */
+async function findShipped(fromDir, relative) {
+  const { existsSync } = await import('node:fs')
+  const { dirname, join } = await import('node:path')
+  let dir = fromDir
+  for (let depth = 0; depth < 8; depth += 1) {
+    const candidate = join(dir, 'node_modules', relative)
+    if (existsSync(candidate)) return candidate
+    const parent = dirname(dir)
+    if (parent === dir) break
+    dir = parent
+  }
+  return null
+}
+
+/**
+ * Отдать веб-манифест: поставочный, но с нашим именем и иконкой.
+ *
+ * Полная подмена была бы заморозкой чужого файла на сегодняшней версии:
+ * dsh добавит поле — ярлыки, снимки экрана, обработчики ссылок, — а мы
+ * продолжим отдавать форму без него, и установленное приложение молча
+ * потеряет возможности.
+ *
+ * Поэтому поставочный манифест читается и ДОПОЛНЯЕТСЯ. Своими остаются
+ * ровно те поля, ради которых всё затевалось: имя и иконка. Всё прочее —
+ * `id`, `start_url`, `scope`, `display` и любое будущее — проходит как есть.
+ *
+ * Если файл не найден, отдаётся наш запасной: лучше своё имя без чужих
+ * полей, чем отказ.
+ *
+ * @param baseDir - каталог нашего модуля, от него ищется поставочный файл.
+ * @param overrides - наши поля.
+ * @param fallback - что отдать, если поставочного файла нет.
+ */
+export async function handleManifest(req, res, baseDir, overrides, fallback) {
+  let shipped = null
+  try {
+    const path = await findShipped(baseDir, '@deepseek-ai/dsh-web-frontend/dist/manifest.webmanifest')
+    if (path !== null) {
+      const { readFile } = await import('node:fs/promises')
+      shipped = JSON.parse(await readFile(path, 'utf8'))
+    }
+  } catch {
+    // Испорченный или недоступный файл — не повод отказывать.
+    shipped = null
+  }
+
+  const value = shipped === null ? fallback : { ...shipped, ...overrides }
+  res.writeHead(200, {
+    'content-type': 'application/manifest+json; charset=utf-8',
+    'cache-control': 'no-cache',
+  })
+  res.end(JSON.stringify(value, null, 2))
+}
+
 /** Обновить кеш состояния версий. Вызывается и фоновым таймером. */
 export async function refreshUpdate(state, loadModule) {
   const module = await loadModule('./update-check.js')
@@ -84,6 +198,10 @@ export async function handleUpdate(req, res, state, loadModule) {
         reply(res, 415, { ok: false, problem: 'ожидается content-type: application/json' })
         return
       }
+      if (!sameOrigin(req)) {
+        reply(res, 403, { ok: false, problem: 'запрос с чужой страницы' })
+        return
+      }
       const module = await loadModule('./update-check.js')
       const result = await module.applyUpdate()
       state.cached = result.status ?? state.cached
@@ -125,6 +243,11 @@ export async function handleAuth(req, res, services, loadModule) {
       return
     }
 
+    if (!sameOrigin(req)) {
+      reply(res, 403, { problem: 'запрос с чужой страницы' })
+      return
+    }
+
     const body = await readJsonBody(req)
     if (body === null) {
       reply(res, 400, { problem: 'тело запроса не разобрано' })
@@ -145,7 +268,15 @@ export async function handleAuth(req, res, services, loadModule) {
       return
     }
 
-    reply(res, 200, { available: true, ...result })
+    // Список провайдеров прикладывается к КАЖДОМУ ответу, а не только к GET.
+    //
+    // Браузер заменяет состояние ответом целиком, и ответ без списка обнулял
+    // его: после первого же нажатия «Войти» кнопки провайдеров исчезали и не
+    // возвращались до перезагрузки страницы. Отметка «подключено» вдобавок
+    // меняется как раз по итогу входа — значит список и должен приходить
+    // вместе с ним.
+    const listing = await bridge.listProviders(services.authorization, services.credentials)
+    reply(res, 200, { available: true, ...listing, ...result })
   } catch (error) {
     reply(res, 500, { problem: String(error?.message ?? error) })
   }

package/lib/index.js

@@ -69,33 +69,42 @@ const APP_ICON = (() => {
 const APP_ICON_PATH = '/tensorgrid-icon.svg'
 
 /**
- * Веб-манифест. Он задаёт не только иконку, но и ИМЯ установленного
- * приложения — то, что видно в панели задач, меню «Пуск» и заголовке окна.
- * Поставочный называет продукт чужим именем, поэтому манифест свой.
+ * Наши поля веб-манифеста — и только они.
  *
- * Поля id, start_url, scope и display повторяют поставочные: они определяют
- * поведение установленного приложения, и менять их значило бы чинить не то.
+ * Манифест задаёт не только иконку, но и ИМЯ установленного приложения: то,
+ * что видно в панели задач, меню «Пуск» и заголовке окна. Поставочный
+ * называет продукт чужим именем, поэтому эти поля мы переопределяем.
+ *
+ * Остальное НЕ трогаем. Поставочный манифест читается из файла фронтенда и
+ * дополняется этими полями, так что `id`, `start_url`, `scope`, `display` и
+ * всё, что dsh добавит завтра, проходят как есть. Полная подмена заморозила
+ * бы чужой файл на сегодняшней версии.
  */
-const MANIFEST = JSON.stringify(
-  {
-    id: '/',
-    name: 'TENSOR GRID',
-    short_name: 'TENSOR GRID',
-    start_url: '/',
-    scope: '/',
-    display: 'fullscreen',
-    background_color: '#0B0D12',
-    theme_color: '#0B0D12',
-    icons: [
-      { src: APP_ICON_PATH, sizes: 'any', type: 'image/svg+xml', purpose: 'any' },
-      // maskable разрешает системе обрезать иконку под свою форму. Знак
-      // отступает от краёв на 16%, поэтому обрезка его не заденет.
-      { src: APP_ICON_PATH, sizes: 'any', type: 'image/svg+xml', purpose: 'maskable' },
-    ],
-  },
-  null,
-  2,
-)
+const MANIFEST_OVERRIDES = {
+  name: 'TENSOR GRID',
+  short_name: 'TENSOR GRID',
+  background_color: '#0B0D12',
+  theme_color: '#0B0D12',
+  icons: [
+    { src: APP_ICON_PATH, sizes: 'any', type: 'image/svg+xml', purpose: 'any' },
+    // maskable разрешает системе обрезать иконку под свою форму. Знак
+    // отступает от краёв на 16%, поэтому обрезка его не заденет.
+    { src: APP_ICON_PATH, sizes: 'any', type: 'image/svg+xml', purpose: 'maskable' },
+  ],
+}
+
+/**
+ * Запасной манифест — на случай, если поставочного файла не нашлось.
+ * Повторяет поля, которые поставка задавала на момент проверки: лучше своё
+ * имя без чужих новшеств, чем отказ отдать манифест вовсе.
+ */
+const MANIFEST_FALLBACK = {
+  id: '/',
+  start_url: '/',
+  scope: '/',
+  display: 'fullscreen',
+  ...MANIFEST_OVERRIDES,
+}
 
 export const name = 'tensorgrid-ui'
 
@@ -135,20 +144,6 @@ export function apply(ctx) {
       }),
     )
 
-    web.effect(() =>
-      web.webServer.register({
-        kind: 'exact',
-        path: '/manifest.webmanifest',
-        handler(_req, res) {
-          res.writeHead(200, {
-            'content-type': 'application/manifest+json; charset=utf-8',
-            'cache-control': 'no-cache',
-          })
-          res.end(MANIFEST)
-        },
-      }),
-    )
-
     // Стабильная оболочка маршрутов. Вся бизнес-логика живёт в
     // `http-runtime.js`, который перечитывается по времени правки файла.
     // Поэтому исправление обработчика доезжает со следующим запросом — без
@@ -168,6 +163,19 @@ export function apply(ctx) {
       return import(`${path.href}${version}`)
     }
 
+    /**
+     * Превратить file-адрес в путь файловой системы без импорта модуля.
+     *
+     * `node:url` импортировать здесь нельзя: строка композиции грузится
+     * раньше, чем о пакете что-либо известно, и host-половина обязана
+     * оставаться без импортов. На Windows адрес выглядит как
+     * `file:///C:/…`, поэтому ведущая косая убирается.
+     */
+    function fileURLToPathSafe(url) {
+      const decoded = decodeURIComponent(String(url.pathname))
+      return /^\/[A-Za-z]:/.test(decoded) ? decoded.slice(1) : decoded
+    }
+
     /** Минимальный запасной ответ, если горячий модуль не загрузился вовсе. */
     function fail(res, error) {
       const body = JSON.stringify({ problem: `обработчик не загрузился: ${String(error?.message ?? error)}` })
@@ -212,6 +220,26 @@ export function apply(ctx) {
       })
     })
 
+    // Манифест собирается в горячем модуле: он ЧИТАЕТ поставочный файл и
+    // дополняет его нашими полями, а значит зависит от чужой формы, которая
+    // может поменяться. Такую логику правильнее держать там, где её можно
+    // поправить без перезапуска.
+    web.effect(() =>
+      web.webServer.register({
+        kind: 'exact',
+        path: '/manifest.webmanifest',
+        async handler(req, res) {
+          try {
+            const runtime = await loadModule('./http-runtime.js')
+            const here = new URL('.', import.meta.url)
+            await runtime.handleManifest(req, res, fileURLToPathSafe(here), MANIFEST_OVERRIDES, MANIFEST_FALLBACK)
+          } catch (error) {
+            fail(res, error)
+          }
+        },
+      }),
+    )
+
     web.effect(() =>
       web.webServer.register({
         kind: 'exact',

package/lib/review.js

@@ -0,0 +1,329 @@
+/**
+ * «Второе мнение» — независимый обзор работы чужими исполнителями.
+ *
+ * Это строка АГЕНТ-ПЛОСКОСТИ: инструмент регистрируется в реестр пресета, а
+ * не профиля. Провайдеры подагентов живут на Host и монтируются отдельно;
+ * наличие провайдера само по себе прав агенту не даёт, их даёт эта строка.
+ *
+ * Зачем это нужно, коротко. Ошибки бывают громкие и тихие. Громкие падают с
+ * криком, и их находит первый же пользователь. Тихие проявляются только в
+ * определённой последовательности действий — их не находит никто, включая
+ * автора, который перечитал свой код и уверен в нём.
+ *
+ * Проверено на себе: два независимых ревизора нашли восемь настоящих
+ * дефектов в коде, который был написан, вычитан и выпущен как готовый. Ни
+ * одного из них не нашёл ни автор, ни пользователь.
+ *
+ * Три решения, взятые из того опыта и не подлежащие упрощению:
+ *
+ *   1. РЕВИЗОРОВ ДВОЕ. Они находят РАЗНОЕ, а не подстраховывают друг друга:
+ *      два самых опасных дефекта нашли разные исполнители, по одному у
+ *      каждого. Один ревизор — гарантированно упущенная половина.
+ *
+ *   2. ФОРМАТ СТРОГИЙ. Без него приходит поток рассуждений, по которому
+ *      нельзя работать. С ним — список, где у каждой находки есть место,
+ *      последствие и способ воспроизвести.
+ *
+ *   3. НЕУВЕРЕННОСТЬ ЦЕННА. Ревизор, который пишет «возможно, мне не дали
+ *      этот файл — стоит подтвердить», полезнее уверенного: в том опыте он
+ *      был прав, файла ему действительно не дали.
+ *
+ * Зависимость объявлена как peerDependency: так пакет пользуется той же
+ * копией `dsh-tools`, что и работающий dsh, а не своей собственной. Импорт
+ * ЛЕНИВЫЙ, внутри `apply`, по двум причинам: модуль можно загрузить и
+ * проверить там, где dsh не установлен, а отсутствие пакета даёт внятную
+ * ошибку одной строки вместо обрушения всего ряда.
+ */
+
+/** Провайдеры, которых зовём. Порядок влияет только на вывод. */
+const REVIEWERS = ['claude-code', 'codex']
+
+/**
+ * Задание ревизору.
+ *
+ * Оно самодостаточно: одноразовые провайдеры не видят нашей переписки, не
+ * знают персоны и не помнят прошлого вызова. Всё, что нужно для суждения,
+ * должно быть здесь.
+ *
+ * Формат ответа задан жёстко и с примером. Просьба «найди проблемы» без
+ * формата даёт эссе; с форматом — список, пригодный к работе.
+ */
+function buildPrompt({ request, paths, focus }) {
+  const lines = []
+
+  lines.push('Ты — независимый ревизор. Работу делал другой исполнитель, ты её раньше не видел.')
+  lines.push('Твоя задача — НАЙТИ ПРОБЛЕМЫ. Не переписывать, не хвалить, не предлагать улучшений стиля.')
+  lines.push('')
+  lines.push('ЧТО ТРЕБОВАЛОСЬ СДЕЛАТЬ')
+  lines.push('')
+  lines.push(String(request).trim())
+  lines.push('')
+
+  if (Array.isArray(paths) && paths.length > 0) {
+    lines.push('ЧТО СМОТРЕТЬ')
+    lines.push('')
+    for (const path of paths) lines.push(`  ${String(path)}`)
+    lines.push('')
+    lines.push('Пути указаны от рабочей папки, в которой ты запущен.')
+    lines.push('')
+  }
+
+  if (typeof focus === 'string' && focus.trim() !== '') {
+    lines.push('НА ЧТО ОБРАТИТЬ ОСОБОЕ ВНИМАНИЕ')
+    lines.push('')
+    lines.push(focus.trim())
+    lines.push('')
+  }
+
+  lines.push('ЧТО ИСКАТЬ В ЛЮБОМ СЛУЧАЕ')
+  lines.push('')
+  lines.push('  1. Ошибки, из-за которых что-то сломается, зависнет или потеряется')
+  lines.push('  2. Необработанные состояния: гонки, утечки, брошенные обещания, отказы сети')
+  lines.push('  3. Несоответствие тому, что требовалось')
+  lines.push('  4. Безопасность: что можно сделать со стороны, чего не следует')
+  lines.push('  5. Случаи, о которых автор скорее всего не подумал')
+  lines.push('')
+  lines.push('ФОРМАТ ОТВЕТА — строго такой, без вступления и без заключения.')
+  lines.push('Для каждой находки:')
+  lines.push('')
+  lines.push('### краткое название')
+  lines.push('ВАЖНОСТЬ: критично | серьёзно | мелочь')
+  lines.push('ГДЕ: файл и строка либо имя функции')
+  lines.push('ЧТО НЕ ТАК: одно-два предложения')
+  lines.push('ПОЧЕМУ ЭТО ВАЖНО: что произойдёт у пользователя')
+  lines.push('КАК ПРОВЕРИТЬ: конкретные действия, по которым это воспроизводится')
+  lines.push('')
+  lines.push('Если находок нет — напиши ровно: НАХОДОК НЕТ')
+  lines.push('')
+  lines.push('Если каких-то данных тебе не дали и суждение из-за этого неполное — скажи')
+  lines.push('об этом прямо в находке, вместо того чтобы догадываться. Честная')
+  lines.push('неуверенность полезнее уверенной ошибки.')
+
+  return lines.join('\n')
+}
+
+/** Уровни важности в порядке убывания — по нему сортируется итог. */
+const SEVERITY_ORDER = ['критично', 'серьёзно', 'мелочь']
+
+/**
+ * Разобрать ответ ревизора в список находок.
+ *
+ * Разбор намеренно снисходительный: ревизор — языковая модель, и точного
+ * соблюдения формата от неё ждать нельзя. Не разобранное не выбрасывается,
+ * а попадает в `raw`: потерять находку из-за лишнего пробела хуже, чем
+ * показать её неразмеченной.
+ */
+export function parseFindings(text, reviewer) {
+  const source = String(text ?? '')
+  if (/^\s*НАХОДОК НЕТ\s*$/im.test(source) && !source.includes('###')) return []
+
+  const findings = []
+  const blocks = source.split(/^###\s+/m).slice(1)
+
+  for (const block of blocks) {
+    const [titleLine, ...rest] = block.split('\n')
+    const body = rest.join('\n')
+    // Поле тянется до следующего заголовка или до конца блока.
+    //
+    // Флаг `m` здесь недопустим: с ним `$` означает конец СТРОКИ, и
+    // объяснение обрывалось на первой же — а ревизор объясняет развёрнуто,
+    // и вторая строка обычно и есть самое полезное. Начало строки
+    // обеспечивается явным `(?:^|\n)`, конец блока — `$` без `m`.
+    const field = (label) => {
+      const match = body.match(new RegExp(`(?:^|\\n)${label}:\\s*([\\s\\S]+?)(?=\\n[А-ЯЁ ]{3,}:|$)`))
+      return match === null ? null : match[1].trim()
+    }
+
+    const severityRaw = (field('ВАЖНОСТЬ') ?? '').toLowerCase()
+    const severity = SEVERITY_ORDER.find((level) => severityRaw.includes(level)) ?? 'мелочь'
+
+    findings.push({
+      reviewer,
+      title: titleLine.trim(),
+      severity,
+      where: field('ГДЕ'),
+      what: field('ЧТО НЕ ТАК'),
+      why: field('ПОЧЕМУ ЭТО ВАЖНО'),
+      howToCheck: field('КАК ПРОВЕРИТЬ'),
+    })
+  }
+
+  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: [] }
+  }
+
+  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 {
+      // Неудача уборки не должна подменять результат обзора.
+    }
+  }
+}
+
+/** Отсортировать находки: сначала опасные, внутри уровня — по ревизору. */
+function ordered(findings) {
+  return [...findings].sort((a, b) => {
+    const bySeverity = SEVERITY_ORDER.indexOf(a.severity) - SEVERITY_ORDER.indexOf(b.severity)
+    return bySeverity !== 0 ? bySeverity : a.reviewer.localeCompare(b.reviewer)
+  })
+}
+
+/** Человекочитаемый вид итога. */
+export function render(value) {
+  const lines = []
+
+  if (value.reviewers.length === 0) {
+    return 'Ни один ревизор не доступен: провайдеры подагентов не смонтированы.'
+  }
+
+  const failed = value.reviewers.filter((r) => r.failed !== null)
+  const worked = value.reviewers.filter((r) => r.failed === null)
+
+  lines.push(`Ревизоров опрошено: ${worked.length} из ${value.reviewers.length}. Находок: ${value.findings.length}.`)
+  for (const entry of failed) lines.push(`  ✗ ${entry.provider} — ${entry.failed}`)
+  lines.push('')
+
+  if (value.findings.length === 0) {
+    lines.push(worked.length === 0 ? 'Судить не по чему: не ответил ни один ревизор.' : 'Находок нет.')
+    return lines.join('\n')
+  }
+
+  // Сортируем здесь, а не полагаемся на вызывающего: опасное должно быть
+  // сверху при любом способе получить эти данные.
+  for (const finding of ordered(value.findings)) {
+    lines.push(`### ${finding.title}   [${finding.severity}] — ${finding.reviewer}`)
+    if (finding.where) lines.push(`ГДЕ: ${finding.where}`)
+    if (finding.what) lines.push(`ЧТО НЕ ТАК: ${finding.what}`)
+    if (finding.why) lines.push(`ПОЧЕМУ ЭТО ВАЖНО: ${finding.why}`)
+    if (finding.howToCheck) lines.push(`КАК ПРОВЕРИТЬ: ${finding.howToCheck}`)
+    lines.push('')
+  }
+
+  // Совпадение двух независимых ревизоров — сильный признак: они друг друга
+  // не видели. Расхождение тоже полезно, но его надо проверять самому.
+  const byReviewer = new Map()
+  for (const finding of value.findings) {
+    byReviewer.set(finding.reviewer, (byReviewer.get(finding.reviewer) ?? 0) + 1)
+  }
+  lines.push(`По ревизорам: ${[...byReviewer].map(([name, count]) => `${name} — ${count}`).join(', ')}.`)
+  lines.push('Находки не проверены: это мнение со стороны, а не приговор. Подтвердите каждую прежде чем чинить.')
+
+  return lines.join('\n')
+}
+
+export const name = 'tensorgrid-second-opinion'
+export const inject = ['tools', 'subagents']
+
+/**
+ * @param ctx - контекст агент-плоскости с реестрами `tools` и `subagents`.
+ */
+export async function apply(ctx) {
+  const { defineTool } = await import('@deepseek-ai/dsh-tools')
+
+  ctx.tools.register(
+    defineTool({
+      name: 'second_opinion',
+      description:
+        'Ask independent reviewers (Claude Code and Codex) to find problems in work that was just done. ' +
+        'They run as separate products in the same workspace, do not see this conversation, and return only findings — ' +
+        'severity, location, what is wrong, why it matters, how to reproduce. ' +
+        'Use after non-trivial work, before declaring it finished: quiet defects that only appear in a specific sequence ' +
+        'are exactly what the author cannot see. Do not use for trivial edits — each review costs a full run of another product.',
+      parameters: {
+        request: {
+          type: 'string',
+          required: true,
+          description:
+            'What was originally asked for, restated so someone who never saw this conversation can judge whether it was met.',
+        },
+        paths: {
+          type: 'array',
+          items: { type: 'string' },
+          description: 'Files or directories to examine, relative to the workspace.',
+        },
+        focus: {
+          type: 'string',
+          description: 'Optional: what deserves particular attention, such as a risky area or a known uncertainty.',
+        },
+        reviewers: {
+          type: 'array',
+          items: { type: 'string', enum: REVIEWERS },
+          description:
+            'Which reviewers to ask. Defaults to both — they find different things, and asking one guarantees missing half.',
+        },
+      },
+      output: {
+        schema: {
+          type: 'object',
+          additionalProperties: true,
+          properties: {
+            findings: { type: 'array', items: { type: 'object', additionalProperties: true } },
+            reviewers: { type: 'array', items: { type: 'object', additionalProperties: true } },
+          },
+        },
+        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 prompt = buildPrompt(args)
+
+        // Спрашиваем ОБОИХ разом: они независимы, и последовательный опрос
+        // удвоил бы ожидание без всякой пользы.
+        const answers = await Promise.all(
+          usable.map((provider) => askReviewer(ctx, provider, prompt, parent, exec.signal)),
+        )
+
+        return {
+          findings: ordered(answers.flatMap((entry) => entry.findings)),
+          reviewers: answers.map((entry) => ({ provider: entry.provider, failed: entry.failed, found: entry.findings.length })),
+        }
+      },
+    }),
+  )
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tensorgrid-ui",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "description": "TENSOR GRID — фирменный интерфейс поверх DeepSeek Harness: палитра, живой ambient-слой, айдентика и русский язык-пакет",
   "type": "module",
   "main": "lib/index.js",
@@ -11,11 +11,22 @@
     "./client": {
       "default": "./lib/client.js"
     },
+    "./review": {
+      "default": "./lib/review.js"
+    },
     "./package.json": "./package.json"
   },
   "files": [
     "lib"
   ],
+  "peerDependencies": {
+    "@deepseek-ai/dsh-tools": "*"
+  },
+  "peerDependenciesMeta": {
+    "@deepseek-ai/dsh-tools": {
+      "optional": true
+    }
+  },
   "publishConfig": {
     "registry": "https://registry.npmjs.org"
   },