tensorgrid-ui 1.2.2 → 1.3.0

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.2",
+  "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"
   },