devcontour-harness 0.1.0-alpha.5 → 0.1.0-alpha.7

package/lib/verification/gates.js

@@ -50,6 +50,26 @@ const entities = (text) => text.replace(/&(quot|apos|lt|gt|amp);/g, (_m, name) =
  * Изменённое так имя не совпадёт с названным критерием, и подтверждения
  * не будет: для секрета в имени теста это правильный исход.
  */
+/**
+ * Значение атрибута XML: парсер работает без обработки сущностей (DOCTYPE
+ * запрещён, внешних сущностей нет), поэтому предопределённые сущности и
+ * числовые ссылки раскрываются здесь — один раз, без рекурсии. Иначе testcase
+ * `a["b"]` сохранялся как `a["b"]` и не совпадал с названным
+ * testId требования или критерия. Репортер JUnit Node.js экранирует `"`
+ * дважды (`"`); оставшийся после раскрытия `"` поэтому тоже
+ * читается как `"`.
+ */
+export function xmlAttribute(value) {
+    const named = { amp: '&', lt: '<', gt: '>', quot: '"', apos: "'" };
+    return value
+        .replace(/&(#x[0-9a-fA-F]+|#[0-9]+|amp|lt|gt|quot|apos);/g, (all, ref) => {
+        if (!ref.startsWith('#'))
+            return named[ref] ?? all;
+        const code = ref.startsWith('#x') ? parseInt(ref.slice(2), 16) : parseInt(ref.slice(1), 10);
+        return code > 0 && code <= 0x10ffff ? String.fromCodePoint(code) : all;
+    })
+        .replace(/&quot;/g, '"');
+}
 export function junitSummary(xml, redact = (value) => value) {
     if (xml.includes('<!DOCTYPE') || XMLValidator.validate(xml) !== true)
         throw new Error('Некорректный JUnit XML');
@@ -75,7 +95,10 @@ export function junitSummary(xml, redact = (value) => value) {
             summary.tests++;
             if (value && typeof value === 'object') {
                 const named = value;
-                const raw = [named['@_classname'], named['@_name']].filter(Boolean).join('.');
+                const raw = [named['@_classname'], named['@_name']]
+                    .filter(Boolean)
+                    .map((v) => xmlAttribute(String(v)))
+                    .join('.');
                 const redacted = redact(raw);
                 // Длинное имя заменяется хешем: обрезка дала бы коллизии и ложные
                 // совпадения, а без замены receipt не перенёс бы манифест.
@@ -228,7 +251,14 @@ export async function prepareReportPath(root, path, escape) {
 export async function runCheck(
 // Нужна только политика изоляции: проверки ревьюера исполняются вне сервера.
 h, options) {
-    let argv = options.argv, env = options.env, started;
+    let launched = false;
+    const confirm = () => {
+        if (launched)
+            return;
+        launched = true;
+        options.onStart?.();
+    };
+    let argv = options.argv, env = options.env, started, result, attempts = 1;
     const scratch = await realpath(await mkdtemp(join(tmpdir(), 'dc-gate-')));
     // Метка старта — вне TMPDIR команды: проверка, чистящая свой временный
     // каталог, не должна стирать признак того, что она уже начиналась.
@@ -252,39 +282,73 @@ h, options) {
             argv = wrapped.argv;
             env = { ...env, ...wrapped.env };
         }
-        const execute = () => command(argv, options.cwd, {
-            signal: options.signal,
-            timeoutMs: options.timeoutMs,
-            env,
-            redact: options.redact,
-            // Отсоединившийся потомок проверки не переживает её: и при успехе,
-            // и по таймауту, отмене или падению.
-            // Каталоги проверки принадлежат только ей: процесс, запущенный в них
-            // во время проверки, — её потомок, даже без метки и родства.
-            contain: { dirs: [...options.write, scratch, marks] },
-        });
-        let result = await execute(), attempts = 1;
-        // sandbox-runtime ищет shell через `which` с таймаутом в секунду и под
-        // нагрузкой отказывает до запуска команды. Повтор допустим, только если
-        // команда точно не начиналась: метки старта нет.
-        for (; attempts < 3 && sandboxNotStarted(result, started); attempts++) {
-            await new Promise((r) => setTimeout(r, 500 * attempts));
+        const execute = () => {
+            // Уже отменённый вызов не запускается: слушатель отмены на отменённом
+            // сигнале не срабатывает, и команда шла бы до таймаута.
+            if (options.signal.aborted)
+                throw Object.assign(new Error('Проверка отменена до запуска'), { name: 'AbortError' });
+            return command(argv, options.cwd, {
+                signal: options.signal,
+                ...(started === undefined ? { onSpawn: confirm } : {}),
+                timeoutMs: options.timeoutMs,
+                env,
+                redact: options.redact,
+                // Отсоединившийся потомок проверки не переживает её: и при успехе,
+                // и по таймауту, отмене или падению.
+                // Каталоги проверки принадлежат только ей: процесс, запущенный в них
+                // во время проверки, — её потомок, даже без метки и родства.
+                contain: { dirs: [...options.write, scratch, marks] },
+            });
+        };
+        try {
             result = await execute();
+            // sandbox-runtime ищет shell через `which` с таймаутом в секунду и под
+            // нагрузкой отказывает до запуска команды. Повтор допустим, только если
+            // команда точно не начиналась: метки старта нет.
+            for (; attempts < 3 && sandboxNotStarted(result, started); attempts++) {
+                await new Promise((r) => setTimeout(r, 500 * attempts));
+                result = await execute();
+            }
+        }
+        finally {
+            // Факт старта читается до удаления метки — и при исключении после него.
+            if (started !== undefined && existsSync(started))
+                confirm();
         }
-        return { ...result, attempts };
+        // Без песочницы завершившийся процесс точно порождался.
+        if (started === undefined)
+            confirm();
+        return { ...result, attempts, started: launched };
     }
     finally {
         // Каждый каталог убирается независимо: отказ уборки временного каталога
         // проверки (она могла снять с него права) не должен оставлять и метки.
         const removed = await Promise.allSettled([
-            rm(scratch, { recursive: true, force: true }),
-            rm(marks, { recursive: true, force: true }),
+            checkCleanup.remove(scratch),
+            checkCleanup.remove(marks),
         ]);
         const failed = removed.find((r) => r.status === 'rejected');
-        if (failed)
-            throw failed.reason;
+        if (failed) {
+            // Отказ уборки после завершения не стирает исход проверки: код и
+            // вывод доступны вызывающему через ошибку (`completedCheck`).
+            const error = failed.reason instanceof Error ? failed.reason : new Error(String(failed.reason));
+            throw result
+                ? Object.assign(error, { [checkResult]: { ...result, attempts, started: launched } })
+                : error;
+        }
     }
 }
+const checkResult = Symbol('runCheck.result');
+/** Исход проверки, завершившейся до исключения (например, отказа уборки), если он есть. */
+export function completedCheck(error) {
+    return error && typeof error === 'object' && checkResult in error
+        ? error[checkResult]
+        : undefined;
+}
+/** Уборка каталогов проверки; тест подменяет её, чтобы проверить отказ после исхода. */
+export const checkCleanup = {
+    remove: (dir) => rm(dir, { recursive: true, force: true }),
+};
 /** Проверка механизма песочницы; тест подменяет её, чтобы проверить отказ. */
 export const isolationBackend = { check: isolationSupport };
 /**
@@ -305,6 +369,16 @@ export function sandboxBlockedHosts(output, allowed) {
     return [...hosts].filter((host) => host !== '127.0.0.1' &&
         !allowed.some((d) => host === d.toLowerCase() || host.endsWith('.' + d.toLowerCase())));
 }
+/**
+ * Окружение свежести test gate: те же переменные у gate движка и у gate,
+ * запущенного проверкой reviewer, — иначе probe мог бы прочитать cache там,
+ * где gate исполняет тесты заново.
+ */
+export function freshnessEnv(kind, env) {
+    return kind === 'test'
+        ? { TURBO_FORCE: 'true', GOFLAGS: [env.GOFLAGS, '-count=1'].filter(Boolean).join(' ') }
+        : {};
+}
 /**
  * Gate попытки. `test` принимается только по свежему (прежний отчёт удалён до
  * запуска) непустому JUnit без failures, errors и skips на этом SHA; exit 0
@@ -356,14 +430,22 @@ export async function runGate(run) {
             ...(corpusPath ? { DEVCONTOUR_CORPUS: corpusPath } : {}),
         }
         : {};
+    // Test gate исполняет тесты заново, через какую бы обёртку (package
+    // script, task runner) ни шла команда: Turbo не читает cache (`TURBO_FORCE`),
+    // Go не берёт результат из кэша тестов (`-count=1` в GOFLAGS, последним).
+    // Восстановленный из cache зелёный отчёт иначе выглядел бы свежим.
+    const freshness = freshnessEnv(gate.kind, run.execution.env);
     let kind = 'gate';
-    let passed = false, summary = '', log = '', exitCode = -1, manifest;
+    // Заголовок лога — фактические каталог и argv запуска: digest лога входит в
+    // evidence, и запись называет, где и чем проверка исполнялась.
+    let passed = false, summary = '', log = `# harness gate ${gate.id}: cwd=${relative(root, gateCwd) || '.'} argv=${JSON.stringify(gate.command.map(redact))}${gate.kind === 'test' ? ` fresh=${redact(JSON.stringify(freshness))}` : ''}\n`, exitCode = -1, manifest;
     try {
         const result = await runCheck({ config: run.policy }, {
             argv: gate.command,
             cwd: gateCwd,
             env: {
                 ...run.execution.env,
+                ...freshness,
                 ...(reportPath
                     ? { DEVCONTOUR_REPORT_PATH: reportPath, HARNESS_REPORT_PATH: reportPath }
                     : {}),
@@ -379,7 +461,7 @@ export async function runGate(run) {
             settingsDir: join(artifactDir, gate.id),
         });
         exitCode = result.code;
-        log = result.stdout + '\n' + result.stderr;
+        log += result.stdout + '\n' + result.stderr;
         if (result.timedOut || signal.aborted)
             throw new Error('Проверка прервана или превысила timeout');
         if (result.code !== 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devcontour-harness",
-  "version": "0.1.0-alpha.5",
+  "version": "0.1.0-alpha.7",
   "description": "Headless execution harness: work graph, isolated attempts, gates, independent reviews and accepted results.",
   "type": "module",
   "license": "MIT",
@@ -32,6 +32,26 @@
     "./testing": {
       "types": "./lib/testing/index.d.ts",
       "default": "./lib/testing/index.js"
+    },
+    "./toolchains": {
+      "types": "./lib/toolchains/index.d.ts",
+      "default": "./lib/toolchains/index.js"
+    },
+    "./integrations/connector": {
+      "types": "./lib/integrations/connector/index.d.ts",
+      "default": "./lib/integrations/connector/index.js"
+    },
+    "./integrations/relay-v1": {
+      "types": "./lib/integrations/relay-v1/index.d.ts",
+      "default": "./lib/integrations/relay-v1/index.js"
+    },
+    "./integrations/relay-v2": {
+      "types": "./lib/integrations/relay-v2/index.d.ts",
+      "default": "./lib/integrations/relay-v2/index.js"
+    },
+    "./integrations/memory": {
+      "types": "./lib/integrations/memory/index.d.ts",
+      "default": "./lib/integrations/memory/index.js"
     }
   },
   "files": [
@@ -58,6 +78,7 @@
     "schemas:check": "node --import tsx scripts/build-schemas.ts --check",
     "lint": "eslint .",
     "test": "node scripts/test.mjs",
+    "test:integration": "node --import tsx --test --test-concurrency=1 --test-timeout=1800000 tests/integration/*.test.ts",
     "test:package": "npm run build && node scripts/smoke-package.mjs",
     "check": "npm run lint && npm run typecheck && npm run schemas:check && npm run build && npm test && node scripts/smoke-package.mjs",
     "format": "prettier --write .",

package/profiles/go-gotestsum.json

@@ -0,0 +1,21 @@
+{
+  "id": "go-gotestsum",
+  "version": "0.1.0",
+  "capabilities": ["go", "unit-tests"],
+  "protectedPaths": ["go.mod", "go.sum"],
+  "toolchain": {
+    "tools": { "gotestsum": "1.12.3" }
+  },
+  "prepare": [{ "id": "install", "command": ["go", "mod", "download"], "timeoutMs": 600000 }],
+  "gates": [
+    { "id": "vet", "kind": "check", "command": ["go", "vet", "./..."], "timeoutMs": 240000 },
+    {
+      "id": "acceptance",
+      "kind": "test",
+      "command": ["gotestsum", "--junitfile", ".reports/junit.xml", "--", "-count=1", "./..."],
+      "timeoutMs": 240000,
+      "dependsOn": ["vet"],
+      "report": { "type": "junit", "path": ".reports/junit.xml" }
+    }
+  ]
+}

package/profiles/npm-ci.json

@@ -0,0 +1,22 @@
+{
+  "id": "npm-ci",
+  "version": "0.1.0",
+  "capabilities": ["node", "npm", "unit-tests"],
+  "protectedPaths": ["package.json", "package-lock.json"],
+  "toolchain": {
+    "tools": { "npm": "11.4.2" },
+    "packageManagers": { ".": "npm" }
+  },
+  "prepare": [{ "id": "install", "command": ["npm", "ci"], "timeoutMs": 600000 }],
+  "gates": [
+    { "id": "check", "kind": "check", "command": ["npm", "run", "check"], "timeoutMs": 240000 },
+    {
+      "id": "unit-tests",
+      "kind": "test",
+      "command": ["npm", "run", "test"],
+      "timeoutMs": 240000,
+      "dependsOn": ["check"],
+      "report": { "type": "junit", "path": ".reports/junit.xml" }
+    }
+  ]
+}

package/profiles/pnpm-frozen.json

@@ -0,0 +1,24 @@
+{
+  "id": "pnpm-frozen",
+  "version": "0.1.0",
+  "capabilities": ["node", "pnpm", "unit-tests"],
+  "protectedPaths": ["package.json", "pnpm-lock.yaml", "pnpm-workspace.yaml"],
+  "toolchain": {
+    "tools": { "pnpm": "10.12.3" },
+    "packageManagers": { ".": "pnpm" }
+  },
+  "prepare": [
+    { "id": "install", "command": ["pnpm", "install", "--frozen-lockfile"], "timeoutMs": 600000 }
+  ],
+  "gates": [
+    { "id": "check", "kind": "check", "command": ["pnpm", "run", "check"], "timeoutMs": 240000 },
+    {
+      "id": "unit-tests",
+      "kind": "test",
+      "command": ["pnpm", "run", "test"],
+      "timeoutMs": 240000,
+      "dependsOn": ["check"],
+      "report": { "type": "junit", "path": ".reports/junit.xml" }
+    }
+  ]
+}

package/profiles/python-uv.json

@@ -0,0 +1,26 @@
+{
+  "id": "python-uv",
+  "version": "0.1.0",
+  "capabilities": ["python", "uv", "pytest"],
+  "protectedPaths": ["pyproject.toml", "uv.lock", ".python-version"],
+  "toolchain": {
+    "tools": { "uv": "0.11.28" }
+  },
+  "prepare": [{ "id": "install", "command": ["uv", "sync", "--locked"], "timeoutMs": 600000 }],
+  "gates": [
+    {
+      "id": "lint",
+      "kind": "check",
+      "command": ["uv", "run", "--no-sync", "ruff", "check", "."],
+      "timeoutMs": 240000
+    },
+    {
+      "id": "acceptance",
+      "kind": "test",
+      "command": ["uv", "run", "--no-sync", "pytest", "--junitxml=.reports/junit.xml"],
+      "timeoutMs": 240000,
+      "dependsOn": ["lint"],
+      "report": { "type": "junit", "path": ".reports/junit.xml" }
+    }
+  ]
+}