workflow-ai 1.0.65 → 1.0.66

package/src/skills/execute-task/tests/cases/TC-EXECUTE-TASK-005/current/meta.json

@@ -1,88 +1,88 @@
-{
-  "date": "2026-04-21T08:57:32.497Z",
-  "skill_sha": "1503ea1",
-  "status": "passed",
-  "duration_ms": 1,
-  "per_model": {
-    "claude-haiku": {
-      "passed": true,
-      "errored": false,
-      "pass_count": 3,
-      "error_count": 0,
-      "total": 3,
-      "threshold": 2
-    },
-    "kilo-free": {
-      "passed": true,
-      "errored": false,
-      "pass_count": 3,
-      "error_count": 0,
-      "total": 3,
-      "threshold": 2
-    },
-    "kilo-glm-air": {
-      "passed": true,
-      "errored": false,
-      "pass_count": 3,
-      "error_count": 0,
-      "total": 3,
-      "threshold": 2
-    }
-  },
-  "rubric_scores": [
-    {
-      "agentId": "claude-haiku",
-      "trial": 1,
-      "score": 5,
-      "errored": false
-    },
-    {
-      "agentId": "claude-haiku",
-      "trial": 2,
-      "score": 5,
-      "errored": false
-    },
-    {
-      "agentId": "claude-haiku",
-      "trial": 3,
-      "score": 5,
-      "errored": false
-    },
-    {
-      "agentId": "kilo-free",
-      "trial": 1,
-      "score": 5,
-      "errored": false
-    },
-    {
-      "agentId": "kilo-free",
-      "trial": 2,
-      "score": 5,
-      "errored": false
-    },
-    {
-      "agentId": "kilo-free",
-      "trial": 3,
-      "score": 5,
-      "errored": false
-    },
-    {
-      "agentId": "kilo-glm-air",
-      "trial": 1,
-      "score": 5,
-      "errored": false
-    },
-    {
-      "agentId": "kilo-glm-air",
-      "trial": 2,
-      "score": 5,
-      "errored": false
-    },
-    {
-      "agentId": "kilo-glm-air",
-      "trial": 3,
-      "score": 5,
-      "errored": false
-    }
-  ]
+{
+  "date": "2026-04-21T16:43:17.772Z",
+  "skill_sha": "1503ea1",
+  "status": "passed",
+  "duration_ms": 2,
+  "per_model": {
+    "claude-haiku": {
+      "passed": true,
+      "errored": false,
+      "pass_count": 3,
+      "error_count": 0,
+      "total": 3,
+      "threshold": 2
+    },
+    "kilo-free": {
+      "passed": true,
+      "errored": false,
+      "pass_count": 3,
+      "error_count": 0,
+      "total": 3,
+      "threshold": 2
+    },
+    "kilo-glm-air": {
+      "passed": true,
+      "errored": false,
+      "pass_count": 3,
+      "error_count": 0,
+      "total": 3,
+      "threshold": 2
+    }
+  },
+  "rubric_scores": [
+    {
+      "agentId": "claude-haiku",
+      "trial": 1,
+      "score": 5,
+      "errored": false
+    },
+    {
+      "agentId": "claude-haiku",
+      "trial": 2,
+      "score": 5,
+      "errored": false
+    },
+    {
+      "agentId": "claude-haiku",
+      "trial": 3,
+      "score": 5,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-free",
+      "trial": 1,
+      "score": 5,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-free",
+      "trial": 2,
+      "score": 5,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-free",
+      "trial": 3,
+      "score": 5,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-glm-air",
+      "trial": 1,
+      "score": 5,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-glm-air",
+      "trial": 2,
+      "score": 5,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-glm-air",
+      "trial": 3,
+      "score": 5,
+      "errored": false
+    }
+  ]
 }

package/src/skills/review-result/SKILL.md

@@ -35,6 +35,7 @@ description: >
 | Модуль | Когда загружать |
 |--------|----------------|
 | `knowledge/dod-patterns.md` | При определении типа проверки для конкретного пункта DoD |
+| `knowledge/test-hygiene.md` | Когда DoD требует создать/изменить автотест — красные и зелёные флаги изоляции, что считать нарушением и как оформить issue |
 | `../shared/*` | **ВСЕГДА** перед началом работы — общие знания проекта |
 
 ## Загрузка шаблонов

package/src/skills/review-result/knowledge/test-hygiene.md

@@ -0,0 +1,44 @@
+# Проверка изоляции тестов на ревью
+
+Когда DoD тикета требует создать/изменить автотест, ревьюер **отдельно** проверяет, что тест не пишет артефакты в рабочий репозиторий или в общие системные пути. Это не дублирует обычный «тесты проходят» — проверка изоляции ортогональна результату прогона (зелёный тест всё равно может быть токсичным).
+
+## Зачем ревьюеру проверять это отдельно
+
+Тест, который проходит в чистом прогоне, но пишет в рабочий репозиторий, создаёт отложенные инциденты:
+- в CI / на второй машине / при параллельном запуске тесты ложно падают или ложно проходят;
+- при прерывании (Ctrl+C, таймаут) оставленный state подхватывается следующим прогоном как валидный;
+- затираются файлы, нужные параллельным агентам/пайплайну;
+- ложно срабатывают механические проверки (verify-artifacts, git-snapshot).
+
+Поэтому «зелёный прогон» — необходимый, но не достаточный сигнал. Нужна **отдельная** проверка изоляции.
+
+## Что читать
+
+1. Файл(ы) теста, затронутые тикетом.
+2. Соседние test-файлы в той же директории — какой паттерн изоляции уже принят в проекте.
+
+## Красные флаги при чтении теста
+
+- Путь к тест-артефакту строится от корня репозитория или CWD (любая конструкция «директория исходника + подъём к корню», «текущий рабочий каталог», «заранее заданный projectRoot») **без** префикса стандартного системного tmp-корня языка.
+- `cleanup`-функция удаляет конкретный файл, а не весь tmp-корень. При прерывании до её вызова файл останется в рабочем репозитории.
+- Cleanup — в теле test-функции, а не в teardown/after-hook. Падение assertion до cleanup-вызова = утечка state.
+- Тест модифицирует реальный файл в config-, data-, state-, cache- или иных рабочих каталогах проекта внутри корня репозитория.
+- Глобальные константы теста указывают на рабочий репозиторий и используются более чем в одном тест-кейсе (shared mutable state).
+
+## Зелёные сигналы
+
+- Корень артефактов создаётся через стандартную tmp-функцию стандартной библиотеки языка (исполнитель выбирает конкретную по языку проекта).
+- Cleanup удаляет **весь корень** рекурсивно и идемпотентен (можно вызвать повторно без ошибки).
+- Cleanup в teardown/after-hook, гарантированно выполняется при любом исходе теста.
+- Каждый тест получает свой свежий корень — нет разделяемого изменяемого состояния между тестами.
+
+## Что делать при обнаружении нарушения
+
+- Если нарушена изоляция — `failed` с конкретным issue: какая строка файла указывает на реальный каталог проекта, почему cleanup не защищает от прерывания.
+- Не смягчай finding на основании того, что тест «проходит» — проверка изоляции ортогональна прогону.
+- В issue — процитировать фрагмент теста дословно (путь до корня проекта, функция очистки), а не пересказать.
+
+## Когда проверка не применяется
+
+- Тикет явно просит тест-генератор, который коммитит файлы в репозиторий как артефакт. В этом случае в DoD должно быть явное обоснование.
+- Тест read-only: не создаёт и не модифицирует файлы (только reads).

package/src/skills/review-result/scripts/verify-artifacts.js

@@ -27,6 +27,7 @@
 
 import fs from 'fs';
 import path from 'path';
+import { pathToFileURL } from 'url';
 import { findProjectRoot } from 'workflow-ai/lib/find-root.mjs';
 import { parseFrontmatter } from 'workflow-ai/lib/utils.mjs';
 
@@ -157,6 +158,98 @@ function checkResultSection(body) {
   };
 }
 
+/**
+ * E2E assertion против ghost-execution: парсит секцию `### Implementation assertions`
+ * в теле тикета и возвращает список проверок вида
+ *   { module, export, method?, type? }
+ *
+ * Формат каждой строки-bullet'а (значения в backticks, ключи через запятую):
+ *   - module: `path/to/file.mjs`, export: `ClassName`, method: `methodName`
+ *   - module: `path/to/file.mjs`, export: `namedExport`
+ *   - module: `path/to/file.mjs`, export: `helper`, type: `function`
+ *
+ * Секция необязательна. Если её нет — возвращаем пустой массив
+ * (тикеты без исполняемых артефактов — например, DOCS/HUMAN — проходят без гейта).
+ */
+function parseImplementationAssertions(body) {
+  const headerRegex = /^###\s*(?:Implementation assertions|E2E assertions)\s*$/m;
+  const match = headerRegex.exec(body);
+  if (!match) return [];
+
+  const startIdx = match.index + match[0].length;
+  const nextH3 = body.indexOf('\n### ', startIdx);
+  const nextH2 = body.indexOf('\n## ', startIdx);
+  const candidates = [nextH3, nextH2].filter(i => i !== -1);
+  const sectionEnd = candidates.length > 0 ? Math.min(...candidates) : body.length;
+  const sectionContent = body.substring(startIdx, sectionEnd);
+
+  const assertions = [];
+  const bulletRegex = /^[-*]\s+(.+)$/gm;
+  let bulletMatch;
+  while ((bulletMatch = bulletRegex.exec(sectionContent)) !== null) {
+    const line = bulletMatch[1];
+    const kv = {};
+    const pairRegex = /(\w+)\s*:\s*`([^`]+)`/g;
+    let pair;
+    while ((pair = pairRegex.exec(line)) !== null) {
+      kv[pair[1]] = pair[2];
+    }
+    if (kv.module && kv.export) {
+      assertions.push({
+        module: kv.module,
+        export: kv.export,
+        method: kv.method || null,
+        type: kv.type || 'function',
+      });
+    }
+  }
+  return assertions;
+}
+
+/**
+ * Выполняет список implementation assertions против живого кода.
+ * Для каждой: dynamic import модуля → достаём export → проверяем method/type.
+ * Возвращает массив { ...assertion, ok, reason? }.
+ */
+async function runImplementationAssertions(assertions) {
+  const results = [];
+  for (const a of assertions) {
+    const absPath = path.isAbsolute(a.module) ? a.module : path.join(PROJECT_DIR, a.module);
+    if (!fs.existsSync(absPath)) {
+      results.push({ ...a, ok: false, reason: `module_not_found: ${a.module}` });
+      continue;
+    }
+    let mod;
+    try {
+      mod = await import(pathToFileURL(absPath).href);
+    } catch (err) {
+      results.push({ ...a, ok: false, reason: `import_failed: ${err.message.replace(/\n/g, ' ')}` });
+      continue;
+    }
+    const exported = mod[a.export];
+    if (exported === undefined) {
+      results.push({ ...a, ok: false, reason: `export_not_found: ${a.export}` });
+      continue;
+    }
+    if (a.method) {
+      // Класс → метод на prototype; объект → метод как свойство
+      const target = typeof exported === 'function' ? exported.prototype : exported;
+      const fn = target ? target[a.method] : undefined;
+      if (typeof fn !== 'function') {
+        results.push({ ...a, ok: false, reason: `method_not_function: ${a.export}.${a.method}` });
+        continue;
+      }
+    } else {
+      if (typeof exported !== a.type) {
+        results.push({ ...a, ok: false, reason: `type_mismatch: expected ${a.type}, got ${typeof exported}` });
+        continue;
+      }
+    }
+    results.push({ ...a, ok: true });
+  }
+  return results;
+}
+
 function verifyTicket(ticketPath) {
   if (!fs.existsSync(ticketPath)) {
     throw new Error(`Ticket file not found: ${ticketPath}`);
@@ -177,6 +270,8 @@ function verifyTicket(ticketPath) {
 
   const resultStats = checkResultSection(body);
 
+  const assertions = parseImplementationAssertions(body);
+
   return {
     ticket_id: frontmatter.id,
     created_at: frontmatter.created_at,
@@ -185,7 +280,8 @@ function verifyTicket(ticketPath) {
     dod_checked: dodStats.checked,
     dod_completed: dodStats.completed,
     result_exists: resultStats.exists,
-    result_filled: resultStats.summaryFilled
+    result_filled: resultStats.summaryFilled,
+    assertions,
   };
 }
 
@@ -252,6 +348,16 @@ function formatVerdict(result) {
     );
   }
 
+  const assertionFailures = (result.assertionResults || []).filter(r => !r.ok);
+  if (assertionFailures.length > 0) {
+    const descriptions = assertionFailures.map(r => {
+      const target = r.method ? `${r.export}.${r.method}` : r.export;
+      return `${target} в ${r.module} (${r.reason})`;
+    });
+    failReasons.push(`assertion_failed=${descriptions.length}`);
+    humanIssues.push(`не прошли E2E-assertions (ghost execution): ${descriptions.join('; ')}`);
+  }
+
   const status = failReasons.length === 0 ? 'passed' : 'failed';
 
   return { status, missingFiles, unchangedFiles, failReasons, humanIssues };
@@ -317,7 +423,7 @@ function appendReviewNote(ticketPath, humanIssues) {
   return true;
 }
 
-function main() {
+async function main() {
   const args = process.argv.slice(2);
 
   if (args.length === 0) {
@@ -349,6 +455,8 @@ function main() {
 
   try {
     const result = verifyTicket(ticketPath);
+    result.assertionResults = await runImplementationAssertions(result.assertions || []);
+
     const verdict = formatVerdict(result);
 
     let reviewNoteWritten = false;
@@ -356,6 +464,9 @@ function main() {
       reviewNoteWritten = appendReviewNote(ticketPath, verdict.humanIssues);
     }
 
+    const assertionsTotal = result.assertionResults.length;
+    const assertionsFailed = result.assertionResults.filter(r => !r.ok).length;
+
     console.log('---RESULT---');
     console.log(`status: ${verdict.status}`);
     console.log(`ticket_id: ${result.ticket_id || ''}`);
@@ -365,6 +476,8 @@ function main() {
     console.log(`result_filled: ${result.result_filled}`);
     console.log(`missing_files: ${verdict.missingFiles.join(',')}`);
     console.log(`unchanged_files: ${verdict.unchangedFiles.join(',')}`);
+    console.log(`assertions_total: ${assertionsTotal}`);
+    console.log(`assertions_failed: ${assertionsFailed}`);
     if (verdict.failReasons.length > 0) {
       console.log(`fail_reasons: ${verdict.failReasons.join('; ')}`);
       console.log(`issues: ${verdict.humanIssues.join('; ')}`);

package/src/skills/review-result/tests/cases/TC-REVIEW-RESULT-003/current/claude-sonnet/trial-1.md

@@ -0,0 +1,7 @@
+---RESULT---
+status: failed
+issues:
+  - "Пункт DoD #3 не выполнен: тест пишет в рабочий репозиторий — `const testProjectRoot = join(__dirname, '..', '..')` (example-test.mjs:17) строит корень артефактов от __dirname внутри репозитория, а не от системного tmp-корня"
+  - "Пункт DoD #3 не выполнен: cleanup удаляет конкретный файл `rmSync(testConfigPath)` (example-test.mjs:32), а не весь tmp-корень — при прерывании до cleanup файл `.workflow/config/sample.yaml` остаётся в рабочем репозитории"
+  - "Пункт DoD #3 не выполнен: cleanup вызывается в теле test-функции (example-test.mjs:35), а не в teardown/after-hook — падение assertion до cleanup = утечка state"
+---RESULT---

package/src/skills/review-result/tests/cases/TC-REVIEW-RESULT-003/current/claude-sonnet/trial-2.md

@@ -0,0 +1,7 @@
+---RESULT---
+status: failed
+issues:
+  - "DoD пункт 3 не выполнен: тест пишет в рабочий репозиторий, а не в системный tmp-корень. Строка `const testProjectRoot = join(__dirname, '..', '..')` привязывает корень артефактов к реальному корню проекта; запись идёт в `.workflow/config/sample.yaml` внутри рабочего репозитория. Ожидалось: использование `os.tmpdir()` / `fs.mkdtempSync` как корня."
+  - "DoD пункт 3 не выполнен: cleanup (`rmSync(testConfigPath)`) удаляет конкретный файл, а не весь tmp-корень рекурсивно. При прерывании до вызова cleanup файл останется в рабочем репозитории."
+  - "DoD пункт 3 не выполнен: `cleanupTestConfig()` вызывается в теле тест-функции, а не в teardown/after-hook. Падение `assert` до этой строки — cleanup не выполняется, state утекает в репозиторий."
+---RESULT---

package/src/skills/review-result/tests/cases/TC-REVIEW-RESULT-003/current/claude-sonnet/trial-3.md

@@ -0,0 +1,7 @@
+---RESULT---
+status: failed
+issues:
+  - "Пункт DoD 3 не выполнен: `const testProjectRoot = join(__dirname, '..', '..')` привязывает корень артефактов к реальному корню репозитория, а не к системному tmp. Тест пишет в `.workflow/config/sample.yaml` рабочего репозитория (example-test.mjs:18-21)."
+  - "Пункт DoD 3 не выполнен: cleanup (`rmSync(testConfigPath)`) удаляет конкретный файл, а не весь tmp-корень — при прерывании до вызова файл остаётся в рабочем репозитории (example-test.mjs:24-26)."
+  - "Пункт DoD 3 не выполнен: cleanup вызывается в теле test-функции, а не в teardown/after-hook — падение assertion до cleanup = утечка state в рабочем репозитории (example-test.mjs:29-34)."
+---RESULT---

package/src/skills/review-result/tests/cases/TC-REVIEW-RESULT-003/current/judge.json

@@ -0,0 +1,163 @@
+{
+  "per_model": {
+    "claude-sonnet": {
+      "pass_count": 3,
+      "total": 3,
+      "trials": [
+        {
+          "trial": 1,
+          "score": 5,
+          "passed": true
+        },
+        {
+          "trial": 2,
+          "score": 5,
+          "passed": true
+        },
+        {
+          "trial": 3,
+          "score": 4,
+          "passed": true
+        }
+      ]
+    },
+    "kilo-glm": {
+      "pass_count": 3,
+      "total": 3,
+      "trials": [
+        {
+          "trial": 1,
+          "score": 4,
+          "passed": true
+        },
+        {
+          "trial": 2,
+          "score": 4,
+          "passed": true
+        },
+        {
+          "trial": 3,
+          "score": 4,
+          "passed": true
+        }
+      ]
+    },
+    "kilo-minimax": {
+      "pass_count": 3,
+      "total": 3,
+      "trials": [
+        {
+          "trial": 1,
+          "score": 4,
+          "passed": true
+        },
+        {
+          "trial": 2,
+          "score": 4,
+          "passed": true
+        },
+        {
+          "trial": 3,
+          "score": 4,
+          "passed": true
+        }
+      ]
+    },
+    "kilo-deepseek": {
+      "pass_count": 2,
+      "total": 3,
+      "trials": [
+        {
+          "trial": 1,
+          "score": 3,
+          "passed": false
+        },
+        {
+          "trial": 2,
+          "score": 4,
+          "passed": true
+        },
+        {
+          "trial": 3,
+          "score": 4,
+          "passed": true
+        }
+      ]
+    }
+  },
+  "rubric_scores": [
+    {
+      "agentId": "claude-sonnet",
+      "trial": 1,
+      "score": 5,
+      "errored": false
+    },
+    {
+      "agentId": "claude-sonnet",
+      "trial": 2,
+      "score": 5,
+      "errored": false
+    },
+    {
+      "agentId": "claude-sonnet",
+      "trial": 3,
+      "score": 4,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-deepseek",
+      "trial": 1,
+      "score": 3,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-deepseek",
+      "trial": 2,
+      "score": 4,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-deepseek",
+      "trial": 3,
+      "score": 4,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-glm",
+      "trial": 1,
+      "score": 4,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-glm",
+      "trial": 2,
+      "score": 4,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-glm",
+      "trial": 3,
+      "score": 4,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-minimax",
+      "trial": 1,
+      "score": 4,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-minimax",
+      "trial": 2,
+      "score": 4,
+      "errored": false
+    },
+    {
+      "agentId": "kilo-minimax",
+      "trial": 3,
+      "score": 4,
+      "errored": false
+    }
+  ],
+  "timestamp": "2026-04-21T14:36:41.299Z"
+}

package/src/skills/review-result/tests/cases/TC-REVIEW-RESULT-003/current/kilo-deepseek/trial-1.md

@@ -0,0 +1,5 @@
+---RESULT---
+status: failed
+issues:
+  - "Пункт DoD #3 не выполнен: тест нарушает изоляцию — пишет в рабочий репозиторий проекта, cleanup не защищает от прерывания"
+---RESULT---

package/src/skills/review-result/tests/cases/TC-REVIEW-RESULT-003/current/kilo-deepseek/trial-2.md

@@ -0,0 +1,5 @@
+---RESULT---
+status: failed
+issues:
+  - "Пункт DoD 3 не выполнен: ожидалось создание tmp-корня и изоляция в teardown/after-hook, получено: корень артефактов привязан к реальному корню репозитория (строка `const testProjectRoot = join(__dirname, '..', '..');`), cleanup удаляет конкретный файл, cleanup вызывается в теле test-функции"
+---RESULT---

package/src/skills/review-result/tests/cases/TC-REVIEW-RESULT-003/current/kilo-deepseek/trial-3.md

@@ -0,0 +1,11 @@
+Проверка тикета QA-904 выявила следующие проблемы:
+
+1. **Несоответствие путей** — в DoD указан файл `tests/fixtures/QA‑904‑test‑isolation‑violation/example‑test.mjs`, но фактический путь `.workflow/src/skills/review‑result/tests/fixtures/QA‑904‑test‑isolation‑violation/example‑test.mjs`.
+2. **Нарушение изоляции теста** — тест пишет артефакты в рабочий репозиторий (строка `const testProjectRoot = join(__dirname, '..', '..');`), cleanup удаляет только конкретный файл, а не весь tmp‑корень, и находится в теле теста, а не в teardown/after‑hook.
+
+---RESULT---
+status: failed
+issues:
+  - "Пункт DoD #1 не выполнен: ожидался файл tests/fixtures/QA-904-test-isolation-violation/example-test.mjs, фактический путь .workflow/src/skills/review-result/tests/fixtures/QA-904-test-isolation-violation/example-test.mjs"
+  - "Пункт DoD #3 не выполнен: тест нарушает изоляцию — корень артефактов привязан к рабочему репозиторию (строка 10), cleanup удаляет конкретный файл, а не tmp-корень, cleanup в теле теста, а не в teardown/after-hook"
+---RESULT---

package/src/skills/review-result/tests/cases/TC-REVIEW-RESULT-003/current/kilo-glm/trial-1.md

@@ -0,0 +1,16 @@
+Вердикт: **failed**.
+
+Проверка DoD-критериев:
+
+1. **Файл создан** — подтверждено.
+2. **Тест проходит** — принято по условию.
+3. **Изоляция** — **нарушена** по трём причинам:
+   - `testProjectRoot` указывает на корень репозитория (`join(__dirname, '..', '..')`), а не на системный tmp-каталог.
+   - `writeFileSync(testConfigPath, ...)` пишет в `.workflow/config/sample.yaml` внутри рабочего репозитория.
+   - `cleanupTestConfig()` вызывается в теле test-функции, а не в `after()`/`teardown`; при падении assertion файл остаётся в репозитории.
+
+---RESULT---
+status: failed
+issues:
+  - "Пункт DoD 3 не выполнен: тест пишет в рабочий репозиторий (testProjectRoot = join(__dirname, '..', '..') → корень репозитория), а не в tmp-корень; cleanup вызывается в теле test-функции, а не в teardown/after-hook; при падении assertion артефакт остаётся в репозитории"
+---RESULT---

package/src/skills/review-result/tests/cases/TC-REVIEW-RESULT-003/current/kilo-glm/trial-2.md

@@ -0,0 +1,18 @@
+Тикет прочитан. Три критерия DoD. Критерий 3 (изоляция) нарушен — анализ ниже.
+
+**DoD-проверка:**
+
+1. **Файл создан** — подтверждено, файл существует.
+2. **Тест проходит** — формально `node --test` success, принимается.
+3. **Изоляция** — НАРУШЕНА:
+   - `testProjectRoot = join(__dirname, '..', '..')` — указывает на корень репозитория, а не на системный `tmp`.
+   - `createTestConfig` пишет в `.workflow/config/sample.yaml` внутри рабочего репозитория.
+   - `cleanupTestConfig` удаляет конкретный файл, не tmp-корень целиком.
+   - Cleanup вызывается в теле test-функции, а не в `teardown`/`after`-хуке — при падении assertion до строки cleanup файл остаётся в репозитории.
+
+Вердикт: **failed**.
+---RESULT---
+status: failed
+issues:
+  - "Пункт DoD 3 не выполнен: ожидалось что тест создаёт и уничтожает tmp-корень, не пишет в рабочий репозиторий (изоляция в teardown/after-hook); получено: testProjectRoot привязан к реальному корню репозитория (join(__dirname, '..', '..')), createTestConfig пишет в .workflow/config/sample.yaml рабочего репозитория, cleanupTestConfig удаляет конкретный файл а не tmp-корень, cleanup вызывается в теле test-функции а не в teardown — файл: src/skills/review-result/tests/fixtures/QA-904-test-isolation-violation/example-test.mjs:12-22"
+---RESULT---