agent-quality-kit 0.9.0 → 0.10.1

package/tool/selfcheck/units-probe.mjs

@@ -0,0 +1,100 @@
+// Проверки разбора истории для `aqk probe`. Написаны ДО кода: без них нельзя отличить
+// «функция работает» от «функция написана».
+import test from "node:test";
+import assert from "node:assert/strict";
+import { isFix, fixHotspots, probeVerdict } from "../lib/history.mjs";
+import { scanningGates, isCode } from "../commands/probe.mjs";
+
+// Признак починки берётся из ТЕМЫ коммита, а не из тела: тема — единственное, что пишут все,
+// и единственное, что видно в `git log --oneline`. Три написания, потому что репозитории
+// бывают на двух языках и с conventional commits.
+test("темой починки считаются fix, исправ и почин — на обоих языках и с областью", () => {
+  assert.equal(isFix("fix: гейт молчал на красном"), true);
+  assert.equal(isFix("fix(prove): windows-пути"), true);
+  assert.equal(isFix("Исправлено: разбор манифеста"), true);
+  assert.equal(isFix("починка отчёта"), true);
+  assert.equal(isFix("FIX: заглавными тоже"), true);
+});
+
+// Слово «fix» внутри обычной темы починкой не делает: иначе «feat: prefix для путей» попадёт
+// в рейтинг, и рейтинг перестанет что-либо значить.
+test("не всякое упоминание fix — починка", () => {
+  assert.equal(isFix("feat: prefix для путей"), false);
+  assert.equal(isFix("docs: как чинить гейт"), false);
+  assert.equal(isFix("refactor: суффиксы"), false);
+  assert.equal(isFix(""), false);
+});
+
+// НАСТОЯЩИЙ формат `git log --format=%s --name-only`, снятый с живого репозитория 2026-09-09:
+// тема, ПУСТАЯ строка, пути, и сразу следующая тема — без пустой строки перед ней.
+// Первая версия этого теста была написана по моему представлению о формате (пустая строка
+// СЛЕДОВАЛА за файлами), код под неё разобрал историю неверно и нашёл ноль починок там, где
+// их сорок пять. Тот же класс, что и дважды сегодня: проверял замысел, а не вывод.
+const LOG = [
+  "fix: разбор манифеста",
+  "",
+  "tool/lib/manifest.mjs",
+  "tool/selfcheck/units.mjs",
+  "feat: новая запись",
+  "",
+  "kit/gates/x/check.sh",
+  "fix(manifest): кавычки",
+  "",
+  "tool/lib/manifest.mjs",
+  "README.md",
+].join("\n");
+
+test("рейтинг считает только файлы из коммитов-починок", () => {
+  const hot = fixHotspots(LOG, { isCode: () => true });
+  assert.deepEqual(hot[0], { path: "tool/lib/manifest.mjs", fixes: 2 });
+  // Файл из коммита-фичи в рейтинг не попадает вовсе.
+  assert.equal(hot.some((h) => h.path === "kit/gates/x/check.sh"), false);
+});
+
+test("рейтинг отсеивает то, что не код: документ чинят иначе, чем программу", () => {
+  const hot = fixHotspots(LOG, { isCode: (p) => p.endsWith(".mjs") });
+  assert.equal(hot.some((h) => h.path === "README.md"), false);
+  assert.equal(hot.length, 2);
+});
+
+test("пустая история — пустой рейтинг, а не падение", () => {
+  assert.deepEqual(fixHotspots("", { isCode: () => true }), []);
+});
+
+// Три состояния, и сливать их нельзя. «Не смогли проверить» — не «прикрыто»: ровно та
+// подмена, против которой написан весь комплект.
+test("вердикт пробы различает поймано, не поймано и нечем проверить", () => {
+  assert.equal(probeVerdict([{ code: 1 }, { code: 0 }]), "caught");
+  assert.equal(probeVerdict([{ code: 0 }, { code: 0 }]), "blind");
+  assert.equal(probeVerdict([{ code: 2 }, { code: 0 }]), "unknown");
+  // Поймавший гейт сильнее непроверенного: класс закрыт, даже если рядом чего-то не хватает.
+  assert.equal(probeVerdict([{ code: 2 }, { code: 1 }]), "caught");
+  assert.equal(probeVerdict([]), "unknown");
+});
+
+// Пробовать можно только те гейты, которым есть куда подставить каталог: рецепт каталога
+// кончается каталогом проверки, команда, написанная руками, — чем угодно. То же правило, по
+// которому `prove` объявляет запись недоказуемой, а не сломанной.
+test("пробуются только гейты, кончающиеся каталогом проверки", () => {
+  const man = { gates: {
+    ok: "bash gates/x/check.sh .",
+    slash: "bash gates/y/check.sh ./",
+    handmade: "eslint . --max-warnings 0",
+    empty: "",
+  } };
+  assert.deepEqual(scanningGates(man).map(([n]) => n), ["ok", "slash"]);
+});
+
+test("манифест без гейтов не роняет разбор", () => {
+  assert.deepEqual(scanningGates(null), []);
+  assert.deepEqual(scanningGates({ gates: [] }), []);
+});
+
+// Образцы каталога исключены по той же причине, по какой их исключает каждая сканирующая
+// проверка: они существуют, чтобы быть неправильными, и «горячими» быть не могут.
+test("горячим считается код, но не документ и не образец каталога", () => {
+  assert.equal(isCode("src/a.py"), true);
+  assert.equal(isCode("tool/lib/core.mjs"), true);
+  assert.equal(isCode("README.md"), false);
+  assert.equal(isCode("gates/secrets-not-in-code/red/a.py"), false);
+});

package/tool/selfcheck/units-repo.mjs

@@ -8,7 +8,7 @@
 //   node --test tool/selfcheck/units-repo.mjs
 import test from "node:test";
 import assert from "node:assert/strict";
-import { triggerVerdict, recipeFor, EXT_LANG, whichSync, browserServerAdvice, MARKS } from "../lib/repo.mjs";
+import { triggerVerdict, recipeFor, EXT_LANG, whichSync, browserServerAdvice, MARKS, isApiSpec } from "../lib/repo.mjs";
 import { CATALOGS, L } from "../i18n/index.mjs";
 import { dirname } from "node:path";
 
@@ -115,6 +115,36 @@ test("сервер уже объявлен — совета нет", () => {
   assert.equal(browserServerAdvice({ has_ui: true }, pw), null);
 });
 
+// Признак «здесь есть договор с чужим кодом»: спецификация API. Считается обходом дерева, а не
+// списком путей, — файл кладут где угодно (`openapi.yaml`, `docs/api/openapi.json`, `schema.yml`
+// рядом с приложением), и фиксированный список промахнулся бы на большинстве проектов.
+//
+// ПОЧЕМУ ПРИЗНАК ОТДЕЛЬНЫЙ, а не часть has_deps. Запись про арбитра контракта не касается
+// библиотеки, консольной программы и монолита без внешнего интерфейса — а таких большинство.
+// Показанной не тому записи не верят, и каталог теряет доверие целиком, а не одной строкой.
+test("спецификация API опознаётся по имени файла, а не по расположению", () => {
+  assert.equal(isApiSpec("openapi.yaml"), true);
+  assert.equal(isApiSpec("openapi.json"), true);
+  assert.equal(isApiSpec("swagger.yml"), true);
+  assert.equal(isApiSpec("asyncapi.yaml"), true);
+  assert.equal(isApiSpec("openapi-v2.yaml"), true);
+  // Имя без расширения спецификации — не она: договор лежит в разбираемом формате.
+  assert.equal(isApiSpec("openapi.md"), false);
+  // Соседи по алфавиту, которые не договор: обычные файлы проекта.
+  assert.equal(isApiSpec("api.yaml"), false);
+  assert.equal(isApiSpec("schema.sql"), false);
+  assert.equal(isApiSpec("package.json"), false);
+});
+
+// Объяснение обязано быть на обоих языках. Признак без объяснения выключает запись НАВСЕГДА и
+// молча — тот же класс, что поймал has_mcp ниже, поэтому сторожится отдельной строкой: в MARKS
+// этого признака нет, он считается обходом, и общая проверка его не увидит.
+test("у признака спецификации API есть объяснение на обоих языках", () => {
+  for (const lang of ["ru", "en"]) {
+    assert.ok(CATALOGS[lang].trigger.flags.has_api_spec, `${lang}: нет объяснения has_api_spec`);
+  }
+});
+
 // Признак репозитория без объяснения — это запись каталога, ВЫКЛЮЧЕННАЯ НАВСЕГДА. Триггер
 // с неизвестным ключом даёт «условие программа не умеет считать», и запись не показывается
 // никому и никогда. Поймано на себе 2026-09-08: завёл has_mcp в списке признаков, объяснение

package/tool/selfcheck/units-vitals.mjs

@@ -52,6 +52,25 @@ test("вердикт краснеет от отказов, но не от нез
   assert.equal(vitalsVerdict(vitalsRows({ ...ok, tools: [{ gate: "g", prog: "x", found: false }] })), 1);
 });
 
+// НАЙДЕНО АУДИТОМ ФИЧ 2026-09-09. Реестр ответил ошибкой (не упал запрос, а вернул не-200) —
+// `latest` остаётся пустым, и строка печатала «0.9.0, свежая». То есть посмотреть не смогли, а
+// сказали «всё хорошо»: ровно тот грех, против которого написан весь комплект, у него самого.
+// Три состояния, а не два: свежая · старая · посмотреть не смогли.
+test("не дозвонились до реестра — так и говорим, а не «свежая»", () => {
+  const unknown = vitalsRows({ ...ok, version: { current: "0.9.0", latest: "" } });
+  const u = unknown.find((r) => r.key === "version");
+  assert.equal(u.ok, null);
+  assert.match(u.detail, /0\.9\.0/);
+  // Утверждение по СМЫСЛУ, а не по слову: «свежая ли, неизвестно» тоже содержит корень «свеж».
+  // Первая версия этого теста ловила именно слово и краснела на честном тексте.
+  assert.match(u.detail, /неизвестн|unknown/i, `незнание не названо: ${u.detail}`);
+
+  const fresh = vitalsRows({ ...ok, version: { current: "0.9.0", latest: "0.9.0" } });
+  const fd = fresh.find((r) => r.key === "version").detail;
+  assert.doesNotMatch(fd, /неизвестн|unknown/i);
+  assert.notEqual(fd, u.detail, "«свежая» и «не знаем» неразличимы по выводу");
+});
+
 // Устаревшая версия — не отказ: человек мог закрепить её сознательно.
 test("старая версия сообщается, но не роняет", () => {
   const rows = vitalsRows({ ...ok, version: { current: "0.8.0", latest: "0.9.0" } });