workflow-ai 1.0.64 → 1.0.66

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';
 
@@ -71,13 +72,25 @@ function stripLineSuffix(filePath) {
   return match ? match[1] : filePath;
 }
 
-function checkFilesExist(filePaths) {
+function checkFilesExist(filePaths, workStartTime) {
+  const ticketWorkStart = workStartTime ? new Date(workStartTime) : null;
   return filePaths.map(filePath => {
     const fullPath = path.isAbsolute(filePath) ? filePath : path.join(PROJECT_DIR, filePath);
-    return {
-      path: filePath,
-      exists: fs.existsSync(fullPath)
-    };
+    const exists = fs.existsSync(fullPath);
+
+    if (!exists) {
+      return { path: filePath, exists: false, unchanged: false };
+    }
+
+    if (!ticketWorkStart) {
+      return { path: filePath, exists: true, unchanged: false };
+    }
+
+    const stats = fs.statSync(fullPath);
+    const fileMtime = new Date(stats.mtime);
+    const unchanged = fileMtime < ticketWorkStart;
+
+    return { path: filePath, exists: true, unchanged };
   });
 }
 
@@ -145,29 +158,130 @@ 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}`);
   }
-  
+
   const content = fs.readFileSync(ticketPath, 'utf8');
   const { frontmatter, body } = parseFrontmatter(content);
-  
+
   const filePaths = parseChangedFiles(body);
-  const filesExist = checkFilesExist(filePaths);
-  
+  // Точка отсчёта для mtime — created_at: это стабильная метка, которая не
+  // перезаписывается при move-ticket / retry-циклах. updated_at мутирует на
+  // каждом перемещении (ready → in-progress → review → ready → …), поэтому
+  // в retry файлы, реально изменённые в ранней попытке, становятся формально
+  // «unchanged» относительно нового updated_at и тикет ложно блокируется.
+  const filesExist = checkFilesExist(filePaths, frontmatter.created_at || frontmatter.updated_at);
+
   const dodStats = parseDoDCompletion(body);
-  
+
   const resultStats = checkResultSection(body);
-  
+
+  const assertions = parseImplementationAssertions(body);
+
   return {
     ticket_id: frontmatter.id,
+    created_at: frontmatter.created_at,
     files_exist: filesExist,
     dod_completion_pct: dodStats.percentage,
     dod_checked: dodStats.checked,
     dod_completed: dodStats.completed,
     result_exists: resultStats.exists,
-    result_filled: resultStats.summaryFilled
+    result_filled: resultStats.summaryFilled,
+    assertions,
   };
 }
 
@@ -202,10 +316,15 @@ function formatVerdict(result) {
     .filter((f) => !f.exists)
     .map((f) => f.path);
 
+  const unchangedFiles = result.files_exist
+    .filter((f) => f.exists && f.unchanged)
+    .map((f) => f.path);
+
   // Критерии failed:
   //   - result_filled == false (секция Result пуста)
   //   - dod_completion_pct == 0 (ни один пункт DoD не отмечен)
   //   - есть отсутствующие файлы из "Изменённые файлы"
+  //   - есть неизменённые файлы (file_unchanged)
   const failReasons = [];
   const humanIssues = [];
   if (!result.result_filled) {
@@ -222,10 +341,26 @@ function formatVerdict(result) {
     failReasons.push(`missing_files=${missingFiles.join(',')}`);
     humanIssues.push(`не найдены заявленные файлы: ${missingFiles.join(', ')}`);
   }
+  if (unchangedFiles.length > 0) {
+    failReasons.push(`file_unchanged=${unchangedFiles.join(',')}`);
+    humanIssues.push(
+      `файлы не были изменены после начала выполнения тикета: ${unchangedFiles.join(', ')}`
+    );
+  }
+
+  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, failReasons, humanIssues };
+  return { status, missingFiles, unchangedFiles, failReasons, humanIssues };
 }
 
 function buildReviewRow(humanIssues) {
@@ -288,7 +423,7 @@ function appendReviewNote(ticketPath, humanIssues) {
   return true;
 }
 
-function main() {
+async function main() {
   const args = process.argv.slice(2);
 
   if (args.length === 0) {
@@ -320,6 +455,8 @@ function main() {
 
   try {
     const result = verifyTicket(ticketPath);
+    result.assertionResults = await runImplementationAssertions(result.assertions || []);
+
     const verdict = formatVerdict(result);
 
     let reviewNoteWritten = false;
@@ -327,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 || ''}`);
@@ -335,6 +475,9 @@ function main() {
     console.log(`dod_completed: ${result.dod_completed}`);
     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---

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

@@ -0,0 +1,17 @@
+Скрипт недоступен — перехожу к ручной проверке (fallback по workflow).
+Проверка завершена. Файл существует, но нарушает изоляцию. Формирую вердикт.
+Проверка тикета QA-904 завершена. DoD критерий #3 (изоляция теста) не выполнен — три нарушения:
+
+1. **Корень артефактов — рабочий репозиторий** (`example-test.mjs:16`): `join(__dirname, '..', '..')` указывает на корень проекта, а не на системный tmp.
+2. **Cleanup удаляет конкретный файл** (`:28`): `rmSync(testConfigPath)`, не весь tmp-корень — при прерывании артефакт остаётся.
+3. **Cleanup в теле test-функции** (`:35`), а не в teardown/after-hook — падение assertion до cleanup = утечка state.
+
+```
+---RESULT---
+status: failed
+issues:
+  - "DoD #3 не выполнен: тест пишет в рабочий репозиторий — example-test.mjs:16-17, testProjectRoot и testConfigPath указывают на .workflow/config/sample.yaml внутри корня репозитория, а не на tmp-корень через стандартную функцию ОС"
+  - "DoD #3 не выполнен: cleanup удаляет конкретный файл (example-test.mjs:28 — rmSync(testConfigPath)), а не весь tmp-корень рекурсивно — при прерывании до вызова cleanup файл остаётся в рабочем репозитории"
+  - "DoD #3 не выполнен: cleanup вызывается в теле test-функции (example-test.mjs:35), а не в teardown/after-hook — падение assertion до этой строки оставляет артефакт в репозитории"
+---RESULT---
+```

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

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

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

@@ -0,0 +1,31 @@
+Проверяю DoD тикета QA-904:
+
+**Пункт 1** — Файл создан: ✅ Существует
+
+**Пункт 2** — Тест проходит: ✅ Формально `node --test` success
+
+**Пункт 3** — Изоляция (tmp-корень, не рабочий репозиторий): ❌ **НАРУШЕН**
+
+Анализ теста из тикета:
+
+1. `testProjectRoot = join(__dirname, '..', '..')` — вычисляется от расположения теста, ведёт к реальному корню репозитория
+2. `writeFileSync(testConfigPath, ...)` — пишет в `.workflow/config/sample.yaml` рабочего репозитория
+3. `cleanupTestConfig()` — вызывается **внутри** test-функции, не в afterEach/teardown. При падении assertion до этой строки файл останется в рабочем репозитории
+
+Это прямое нарушение изоляции: тест пишет в `.workflow/config/`, а не в tmp-корень (напр. `os.tmpdir()`).
+
+Секции "## Ревью" в тикете нет — это первое ревью.
+
+---
+
+## Вердикт
+
+**DoD пункт 3 не выполнен:** тест записывает артефакты в рабочий репозиторий проекта, а не в tmp-корень. Cleanup находится внутри тела функции, а не в afterEach/teardown, что оставляет файл при падении.
+
+---
+
+---RESULT---
+status: failed
+issues:
+  - "DoD пункт 3 нарушен: тест пишет в рабочий репозиторий (`.workflow/config/sample.yaml`), а не в tmp-корень. cleanup вызывается в теле test-функции, а не в teardown/afterEach — при падении assertion файл останется в рабочем репозитории. Ожидалось: `os.tmpdir()` + afterEach/teardown."
+---RESULT---

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

@@ -0,0 +1,5 @@
+---RESULT---
+status: failed
+issues:
+  - "Пункт DoD #3 не выполнен: ожидался tmp-корень с изоляцией в teardown/after-hook, но тест пишет в `.workflow/config/` рабочего репозитория (`testProjectRoot = join(__dirname, '..', '..')`), cleanup вызывается в теле теста (строка 25), а не в afterEach/teardown"
+---RESULT---