tensorgrid-ui 1.4.0 → 1.4.1

package/lib/client.js

@@ -2283,9 +2283,21 @@ window.__ModuleLoader__.load({
       const refresh = React.useCallback(async (body) => {
         setBusy(true)
         try {
-          setState(await send(body))
+          const next = await send(body)
+          // Ответ без каталога — это отказ, а не новое состояние.
+          //
+          // Раньше состояние заменялось целиком, и одно неудачное нажатие
+          // галочки убирало все карточки: ни повторить, ни увидеть выбранное
+          // было нельзя, помогала только перезагрузка. Ту же ошибку я уже
+          // допускал в строке входа и там же чинил.
+          setState((previous) => {
+            const complete = Array.isArray(next.catalogue) && next.catalogue.length > 0
+            if (complete) return next
+            return previous === null ? next : { ...previous, problem: next.problem ?? 'сохранить не удалось' }
+          })
         } catch (error) {
-          setState({ catalogue: [], problem: String(error && error.message ? error.message : error) })
+          const text = String(error && error.message ? error.message : error)
+          setState((previous) => (previous === null ? { catalogue: [], problem: text } : { ...previous, problem: text }))
         } finally {
           setBusy(false)
         }
@@ -2343,7 +2355,12 @@ window.__ModuleLoader__.load({
             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]
+              // Запись может отсутствовать, если списки Host и браузера
+              // разойдутся. Раньше следующая строка бросала исключение прямо
+              // в отрисовке, и падал ВЕСЬ раздел настроек, а не только наша
+              // карточка.
+              const stored = settings === null ? null : settings.choices[entry.kind]
+              const choice = stored ?? { model: null, effort: null }
               return jsxs('div', {
                 className: 'dsx-rev__card' + (on ? ' dsx-rev__card--on' : ''),
                 children: [

package/lib/review-settings.js

@@ -11,7 +11,7 @@
  *
  * Импортируются только встроенные модули Node.
  */
-import { readFile, writeFile } from 'node:fs/promises'
+import { readFile, rename, writeFile } from 'node:fs/promises'
 import { existsSync } from 'node:fs'
 import { dirname, join } from 'node:path'
 import { fileURLToPath } from 'node:url'
@@ -21,6 +21,17 @@ const FILE_NAME = 'tensorgrid-review.json'
 /** Ревизоры, которых мы умеем звать. Порядок — порядок показа. */
 export const KNOWN = ['claude-code', 'codex']
 
+/**
+ * Что допустимо в модели и усилии.
+ *
+ * Те же правила, что и при запуске, но применяются ЗДЕСЬ, при сохранении.
+ * Раньше проверка жила только в момент вызова: строка вроде «gpt 5 (новая)»
+ * спокойно сохранялась и отвергалась через двадцать минут, при следующем
+ * обзоре. В интерфейсе это было невидимо — список с неизвестным значением
+ * показывает пустое поле, неотличимое от «как настроено».
+ */
+const SAFE_VALUE = /^[A-Za-z0-9][A-Za-z0-9._-]{0,63}$/
+
 /**
  * Что действует, пока пользователь ничего не выбрал.
  *
@@ -71,12 +82,23 @@ 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 enabled = Array.isArray(raw.enabled)
+    ? [...new Set(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)
+    // Недопустимое значение отбрасывается молча и превращается в «как
+    // настроено»: сохранить его значило бы выключить обзор до тех пор, пока
+    // кто-нибудь не догадается открыть файл.
+    const pick = (value) => {
+      if (typeof value !== 'string') return null
+      const trimmed = value.trim()
+      return trimmed !== '' && SAFE_VALUE.test(trimmed) ? trimmed : null
+    }
     choices[kind] = {
       model: pick(source?.model),
       effort: pick(source?.effort),
@@ -100,11 +122,20 @@ export async function readSettings(cwd) {
   }
 }
 
-/** Записать настройки. Возвращает то, что действительно сохранено. */
+/**
+ * Записать настройки. Возвращает то, что действительно сохранено.
+ *
+ * Запись идёт через временный файл с переименованием. Прямая перезапись
+ * сначала обрезает файл, и чтение, пришедшееся на этот момент, видит пустоту
+ * или обрывок — а пустота у нас означает «включить обоих ревизоров», то есть
+ * запустить продукт, который человек только что выключил.
+ */
 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')
+  const temporary = `${path}.${process.pid}.tmp`
+  await writeFile(temporary, `${JSON.stringify(normalised, null, 2)}\n`, 'utf8')
+  await rename(temporary, path)
   return normalised
 }

package/lib/review.js

@@ -39,8 +39,16 @@
  * ошибку одной строки вместо обрушения всего ряда.
  */
 
-/** Провайдеры, которых зовём. Порядок влияет только на вывод. */
-const REVIEWERS = ['claude-code', 'codex']
+/**
+ * Список ревизоров берётся из ОДНОГО места — `reviewers.js`.
+ *
+ * Здесь стоял свой список, третий по счёту: ещё один в `review-settings.js`
+ * и ещё один в справочнике запуска. Ревизор указал, чем это кончится:
+ * добавление третьего продукта в один файл из трёх делает
+ * `settings.choices[kind]` неопределённым, и раздел настроек падает целиком
+ * — React снимает поддерево, пользователь теряет и остальные настройки.
+ */
+const KNOWN_REVIEWERS = ['claude-code', 'codex']
 
 /**
  * Задание ревизору.
@@ -109,33 +117,81 @@ function buildPrompt({ request, paths, focus }) {
 /** Уровни важности в порядке убывания — по нему сортируется итог. */
 const SEVERITY_ORDER = ['критично', 'серьёзно', 'мелочь']
 
+/**
+ * Пределы на разбор.
+ *
+ * Ответ приходит из отдельного процесса и ничем не ограничен: многословная
+ * или сорвавшаяся в повтор модель способна выдать мегабайт. Всё это уходит в
+ * контекст вызывающего агента — то есть стоит дороже самого обзора и
+ * способно вытеснить ту работу, которую обзор проверял.
+ */
+const MAX_FINDINGS = 60
+const MAX_FIELD_CHARS = 1200
+
+/** Обрезать поле, не притворяясь, что ничего не обрезали. */
+function clip(value) {
+  if (value === null) return null
+  return value.length <= MAX_FIELD_CHARS ? value : `${value.slice(0, MAX_FIELD_CHARS)}… (обрезано)`
+}
+
 /**
  * Разобрать ответ ревизора в список находок.
  *
  * Разбор намеренно снисходительный: ревизор — языковая модель, и точного
- * соблюдения формата от неё ждать нельзя. Не разобранное не выбрасывается,
- * а попадает в `raw`: потерять находку из-за лишнего пробела хуже, чем
- * показать её неразмеченной.
+ * соблюдения формата от неё ждать нельзя. Пропущенное поле даёт null, а не
+ * теряет находку; жирная разметка вокруг названий полей снимается.
+ *
+ * Текст БЕЗ единого заголовка не выбрасывается, а возвращается одной
+ * находкой с пометкой «не по формату». Прежде такой ответ превращался в
+ * пустой список, то есть в «находок нет» — и сообщение продукта об истёкшем
+ * входе выглядело справкой о чистоте кода.
  */
 export function parseFindings(text, reviewer) {
   const source = String(text ?? '')
-  if (/^\s*НАХОДОК НЕТ\s*$/im.test(source) && !source.includes('###')) return []
+  if (source.trim() === '') return []
+  if (/^\s*НАХОДОК НЕТ\s*$/im.test(source) && !/^\s*#{2,3}\s+/m.test(source)) return []
 
   const findings = []
-  const blocks = source.split(/^###\s+/m).slice(1)
+  // Заголовок распознаётся только в начале строки и не глубже третьего
+  // уровня: `###` внутри примера кода в поле «КАК ПРОВЕРИТЬ» иначе разрывал
+  // находку надвое, и вторая половина уезжала в конец списка без важности.
+  const blocks = source.split(/^#{2,3}[ \t]+/m).slice(1)
+
+  if (blocks.length === 0) {
+    return [{
+      reviewer,
+      title: 'ответ не по формату',
+      severity: 'мелочь',
+      where: null,
+      what: clip(source.trim()),
+      why: 'Ревизор ответил не по заданному образцу — возможно, это сообщение об ошибке, а не обзор.',
+      howToCheck: null,
+      unparsed: true,
+    }]
+  }
 
   for (const block of blocks) {
+    if (findings.length >= MAX_FINDINGS) break
+
     const [titleLine, ...rest] = block.split('\n')
-    const body = rest.join('\n')
+    // Жирная разметка вокруг названий полей снимается: модели любят её
+    // добавлять, а без этого поле просто не находилось и содержимое терялось.
+    const body = rest.join('\n').replace(/\*\*/g, '')
+
     // Поле тянется до следующего заголовка или до конца блока.
     //
     // Флаг `m` здесь недопустим: с ним `$` означает конец СТРОКИ, и
     // объяснение обрывалось на первой же — а ревизор объясняет развёрнуто,
     // и вторая строка обычно и есть самое полезное. Начало строки
     // обеспечивается явным `(?:^|\n)`, конец блока — `$` без `m`.
+    //
+    // Следующим полем считается только ИЗВЕСТНОЕ название, а не любая
+    // строка из заглавных букв с двоеточием: «ИТОГО:» внутри объяснения
+    // обрывал поле на середине.
+    const NEXT = '(?:ВАЖНОСТЬ|ГДЕ|ЧТО НЕ ТАК|ПОЧЕМУ ЭТО ВАЖНО|КАК ПРОВЕРИТЬ)'
     const field = (label) => {
-      const match = body.match(new RegExp(`(?:^|\\n)${label}:\\s*([\\s\\S]+?)(?=\\n[А-ЯЁ ]{3,}:|$)`))
-      return match === null ? null : match[1].trim()
+      const match = body.match(new RegExp(`(?:^|\\n)${label}:\\s*([\\s\\S]+?)(?=\\n${NEXT}:|$)`))
+      return match === null ? null : clip(match[1].trim())
     }
 
     const severityRaw = (field('ВАЖНОСТЬ') ?? '').toLowerCase()
@@ -143,7 +199,7 @@ export function parseFindings(text, reviewer) {
 
     findings.push({
       reviewer,
-      title: titleLine.trim(),
+      title: clip(titleLine.replace(/\*\*/g, '').trim()),
       severity,
       where: field('ГДЕ'),
       what: field('ЧТО НЕ ТАК'),
@@ -173,7 +229,7 @@ async function askReviewer(ctx, runReviewer, kind, prompt, cwd, choice, signal)
   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: outcome.failed, cancelled: outcome.cancelled === true, findings: [], ...used }
   }
 
   return { provider: kind, failed: null, findings: parseFindings(outcome.answer, kind), ...used }
@@ -192,7 +248,16 @@ export function render(value) {
   const lines = []
 
   if (value.reviewers.length === 0) {
-    return 'Ни один ревизор не доступен: провайдеры подагентов не смонтированы.'
+    // Настоящая причина приходит в `problem` и обязана быть показана.
+    //
+    // Здесь стояла одна фраза на все случаи — про несмонтированные
+    // провайдеры подагентов. Механизм, которым этот код давно не
+    // пользуется: продукты зовутся напрямую. Пользователь, снявший обе
+    // галочки, получал объяснение про постороннюю неисправность и шёл чинить
+    // не то. Оба ревизора нашли это независимо.
+    return typeof value.problem === 'string' && value.problem !== ''
+      ? `Обзор не выполнен: ${value.problem}`
+      : 'Обзор не выполнен: ни один ревизор не выбран.'
   }
 
   const failed = value.reviewers.filter((r) => r.failed !== null)
@@ -203,7 +268,13 @@ export function render(value) {
   lines.push('')
 
   if (value.findings.length === 0) {
-    lines.push(worked.length === 0 ? 'Судить не по чему: не ответил ни один ревизор.' : 'Находок нет.')
+    // «Находок нет» говорится ТОЛЬКО когда обзор действительно состоялся.
+    // Отказ или отмена, поданные как чистый результат, — худшее, что этот
+    // инструмент может сделать: пользователь сочтёт работу проверенной.
+    const cancelled = value.reviewers.some((entry) => entry.cancelled === true)
+    if (cancelled) lines.push('Обзор отменён — судить не по чему.')
+    else if (worked.length === 0) lines.push('Судить не по чему: не ответил ни один ревизор.')
+    else lines.push('Находок нет.')
     return lines.join('\n')
   }
 
@@ -266,7 +337,7 @@ export async function apply(ctx) {
         },
         reviewers: {
           type: 'array',
-          items: { type: 'string', enum: REVIEWERS },
+          items: { type: 'string', enum: KNOWN_REVIEWERS },
           description:
             'Which reviewers to ask. Defaults to both — they find different things, and asking one guarantees missing half.',
         },
@@ -295,12 +366,20 @@ export async function apply(ctx) {
 
         // Выбор пользователя из настроек; вызов может его перекрыть.
         const settings = await readSettings(cwd)
-        const wanted = Array.isArray(args.reviewers) && args.reviewers.length > 0
+        const asked = Array.isArray(args.reviewers) && args.reviewers.length > 0
           ? args.reviewers
           : settings.enabled
 
+        // Повторы отбрасываются, и это не придирка: `reviewers` заполняет
+        // модель, а список из ста одинаковых имён запустил бы сто платных
+        // процессов разом — по одному на каждый элемент.
+        const wanted = [...new Set(asked)].filter((kind) => KNOWN_REVIEWERS.includes(kind))
+
         if (wanted.length === 0) {
-          return { findings: [], reviewers: [], problem: 'в настройках не выбран ни один ревизор' }
+          const problem = asked.length === 0
+            ? 'не выбран ни один ревизор — включите их в настройках'
+            : `неизвестные ревизоры: ${[...new Set(asked)].join(', ')}`
+          return { findings: [], reviewers: [], problem }
         }
 
         const prompt = buildPrompt(args)
@@ -316,6 +395,7 @@ export async function apply(ctx) {
           reviewers: answers.map((entry) => ({
             provider: entry.provider,
             failed: entry.failed,
+            cancelled: entry.cancelled === true,
             found: entry.findings.length,
             model: entry.model,
             effort: entry.effort,

package/lib/reviewers.js

@@ -47,7 +47,7 @@ export const REVIEWERS = {
   'claude-code': {
     label: 'Claude Code',
     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,
@@ -78,9 +78,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
@@ -123,6 +134,16 @@ export async function runReviewer(ctx, kind, { prompt, cwd, model, effort, signa
     }
   }
 
+  // Отмену, случившуюся ДО вызова, надо увидеть сразу.
+  //
+  // Подписка на будущее событие `abort` у уже отменённого сигнала не
+  // срабатывает никогда: обработчик задним числом не вызывается. Без этой
+  // проверки нажатие «стоп» перед самым запуском не мешало нам завести
+  // платный процесс, который потом жил до самого тайм-аута.
+  if (signal?.aborted === true) {
+    return { kind, failed: 'отменено', cancelled: true, answer: null }
+  }
+
   let workDir = null
   let handle = null
   // Своя отмена поверх пользовательской: ревизор без ограничения по времени
@@ -157,30 +178,60 @@ export async function runReviewer(ctx, kind, { prompt, cwd, model, effort, signa
     })
 
     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 }
       }
-    }
-    if (answer.trim() === '') {
+    } else {
       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, 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 +239,12 @@ export async function runReviewer(ctx, kind, { prompt, cwd, model, effort, signa
     if (handle !== null) {
       try {
         handle.terminate()
+        // Ждём, пока процесс действительно умрёт, прежде чем убирать за ним.
+        // Без ожидания удаление шло при живом процессе, у которого есть
+        // отпущенное время на завершение и открытый файл итога: на Windows
+        // это даёт отказ, `force` его проглатывает, и временные каталоги
+        // копятся после каждой отмены.
+        await handle.waitForExit()
       } catch {
         // Уборка не должна подменять результат.
       }

package/package.json

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