agent-quality-kit 0.13.0 → 0.15.0

package/tool/selfcheck/units-learn.mjs

@@ -86,3 +86,35 @@ test("typedFrom: реплика блоками собирается из тек
   });
   assert.equal(typedFrom(line)[0].text, "всегда так");
 });
+
+// ПОВТОР — СИГНАЛ, КОТОРЫЙ ДАЁТ САМ ЧЕЛОВЕК. Разбор AgentLint 2026-09-11: их SS2 сопоставляет
+// поправку с правилом свода по словам. Замер на логах владельца: из 45 поправок к записанным
+// правилам относятся от силы две — такой приём дал бы шум. Зато человек сам помечает повтор:
+// «я же говорил», «опять», «снова». Из 14 таких реплик в одном проекте настоящих норм 5–6, и
+// прежний отбор по маркерам наставления не ловил НИ ОДНОЙ.
+test("isRepeat: «я же говорил», «опять», «снова» — повтор; «опять же» и вопрос-недоумение — нет", async () => {
+  const { isRepeat } = await import("../commands/learn.mjs");
+  assert.equal(isRepeat("нет старые задачи не трогаем я же говорил только пусть новые так работают"), true);
+  assert.equal(isRepeat("стой зачем ты сумму убрал я же не просил этого ты опять не верно меня понял"), true);
+  assert.equal(isRepeat("ок давай запишем, только опять не хочу плодить файлы"), true);
+  assert.equal(isRepeat("I told you not to touch the lockfile"), true);
+  assert.equal(isRepeat("и сказать в чате, опять же, где что смотреть"), false, "«опять же» — связка, а не повтор");
+  assert.equal(isRepeat("ок го дальше"), false);
+  // Второй прогон на тех же логах: половина шума — вопросы-недоумения и вставленные цитаты агента.
+  assert.equal(isRepeat("не понял еще нужно что то опять в env добавить?"), false, "вопрос с «опять» — недоумение, а не норма");
+  assert.equal(isRepeat("⬜ Два мира ролей — всплывёт, если снова будем трогать роли. поясни"), false, "цитата ответа агента");
+  assert.equal(isRepeat("почему ты так долго тестируешь я же говорю не нужно столько тестов?"), true, "сильная пометка держится и с вопросом");
+});
+
+// Повтор, совпавший с правилом свода, — главное, что может сказать эта команда: правило
+// записано, а человек всё равно поправляет. Текстом оно не держится — ему нужен сторож.
+test("repeatedRules: повтор, совпавший с правилом свода, называет это правило", async () => {
+  const { repeatedRules } = await import("../commands/learn.mjs");
+  const entry = "# Правила\n\n- **Не плодить файлы.** Новый файл — только если без него нельзя. <!-- aqk: человек -->\n" +
+    "- **Секреты только в .env.** <!-- aqk: secrets-not-in-code -->\n- обычная строка списка без метки\n";
+  const hits = repeatedRules([{ text: "опять ты плодишь файлы, я же говорил", when: "2026-09-03" }], entry);
+  assert.equal(hits.length, 1);
+  assert.match(hits[0].rule, /Не плодить файлы/);
+  assert.equal(hits[0].arbiter, "человек");
+  assert.equal(repeatedRules([{ text: "опять не тот цвет кнопки", when: "x" }], entry).length, 0, "повтор не про правило — не сюда");
+});

package/tool/selfcheck/units-level.mjs

@@ -11,7 +11,7 @@ import test from "node:test";
 import assert from "node:assert/strict";
 import { commandFor, verdict } from "../lib/prove.mjs";
 import { assessLevel, layoutChecks, unknownKeys, KNOWN_KEYS, parseManifest, coversOf, coversUnproven, unparsedLines } from "../lib/manifest.mjs";
-import { pickLang, langFromText } from "../i18n/index.mjs";
+import { pickLang, langFromText, langFromDocs } from "../i18n/index.mjs";
 import { progress, selectGates } from "../lib/run.mjs";
 
 // --- строка «идёт» во время прогона ---------------------------------------------------
@@ -214,6 +214,26 @@ test("мусор в поле lang не молчит, а просто не дей
   assert.equal(pickLang({}, { lang: "клингонский" }), "en");
 });
 
+// Отзыв с живого проекта 2026-09-11: свод правил и методички на русском, машина на Windows без
+// LANG — и весь вывод английский, пока руками не впишешь `lang:`. Язык текста, который проект
+// сам о себе написал, — свойство проекта; поэтому он выше локали и ниже манифеста.
+test("язык свода проекта сильнее локали и слабее манифеста", () => {
+  const ruDoc = "# Правила\n\nПеред коммитом запусти `npm run check` и `make gates`. Схемы живут в " +
+    "`packages/contract`, их проверяет сервер.\n\n```sh\nnpm run typecheck && npm run lint\n```\n" +
+    "Любое изменение договора проходит через ревью и фиксируется в журнале решений.\n";
+  const enDoc = "# Rules\n\nBefore committing run `npm run check`. Schemas live in packages/contract " +
+    "and the server validates them. Every contract change goes through review and is logged.\n";
+  assert.equal(langFromDocs(ruDoc.repeat(2)), "ru", "русский свод с кодом внутри не узнан");
+  assert.equal(langFromDocs(enDoc.repeat(2)), "en");
+  assert.equal(langFromDocs("# X\n\n`npm test`\n"), "", "по трём словам язык не решается");
+  assert.equal(pickLang({ LANG: "en_US.UTF-8" }, null, "ru"), "ru");
+  assert.equal(pickLang({}, null, "ru"), "ru", "Windows без LANG: свод решает");
+  assert.equal(pickLang({ LANG: "ru_RU.UTF-8" }, null, "en"), "en");
+  assert.equal(pickLang({}, { lang: "en" }, "ru"), "en", "манифест сильнее свода");
+  assert.equal(pickLang({ AQK_LANG: "en" }, null, "ru"), "en", "переменная сильнее свода");
+  assert.equal(pickLang({ LANG: "ru_RU.UTF-8" }, null, ""), "ru", "свод молчит — решает локаль");
+});
+
 // Сокращённый разбор языка в i18n/index.mjs существует потому, что каталог строк нужен раньше,
 // чем кто-либо успеет прочитать манифест целиком. Два разбора одного файла — то же, что два
 // свода правил: через месяц они расходятся, и непонятно, какой настоящий. Сверяем ответы.

package/tool/selfcheck/units-prompt.mjs

@@ -0,0 +1,106 @@
+// tool/selfcheck/units-prompt.mjs — задание для агента одним текстом (`aqk prompt`).
+//
+// ЗАЧЕМ. Между диагнозом и действием не было моста: `doctor` пишет человеку, `context` — «как
+// дела» агенту, а «почини вот это, это и это» человек пересказывал сам. Идея — из разбора
+// agentlint (research/competitors/agentlint.md, «Задание для агента»): правила поведения
+// сверху, исправления по весу, в конце — как проверить. Их слабость не берём: у них «проверь» —
+// это «балл вырос», у нас — команда, которая краснеет и зеленеет.
+//
+//   node --test tool/selfcheck/units-prompt.mjs
+import test from "node:test";
+import assert from "node:assert/strict";
+import { taskText } from "../commands/prompt.mjs";
+import { CATALOGS } from "../i18n/index.mjs";
+
+const T = CATALOGS.ru.prompt;
+const base = {
+  self: "aqk", manifest: true, run: { when: "2026-09-11 12:00", red: [], stale: false },
+  blind: [], adopt: [], shim: null, start: [],
+};
+const text = (s) => taskText({ ...base, ...s }, T).join("\n");
+const items = (s) => text(s).split("\n").filter((l) => /^\d+\. /.test(l));
+
+test("правила поведения сверху и проверка в конце — всегда, даже когда пунктов нет", () => {
+  const t = text({});
+  assert.ok(t.indexOf(T.rulesTitle) < t.indexOf(T.verifyTitle), "правила идут до проверки");
+  assert.match(t, /aqk doctor --run/, "без команды проверки «готово» ничем не доказано");
+  assert.match(t, new RegExp(T.empty.slice(0, 20)), "пустое задание сказано словами");
+});
+
+test("красный гейт — пункт с командой, которая его запускает одного", () => {
+  const it = items({ run: { when: "x", red: ["lint", "units"], stale: false } });
+  assert.equal(it.length, 2);
+  assert.match(it[0], /aqk doctor --run --only lint/);
+});
+
+// Прогона не было или он старше последнего коммита: список красных — про другой код. Первый
+// пункт — прогнать; пустое задание при этом было бы утверждением «чисто», которого никто не делал.
+test("нет прогона или он устарел — первый пункт «прогони», а не «всё чисто»", () => {
+  const none = text({ run: null });
+  assert.doesNotMatch(none, new RegExp(T.empty.slice(0, 20)), "без прогона задание не пустое");
+  assert.match(items({ run: null })[0], /aqk doctor --run/);
+  const stale = items({ run: { when: "x", red: ["lint"], stale: true } });
+  assert.match(stale[0], /aqk doctor --run/, "устаревший прогон — сначала прогнать заново");
+});
+
+test("порядок по весу: манифест → красное → брак из пробы → свои проверки → свод → начало", () => {
+  const it = items({
+    manifest: false,
+    run: { when: "x", red: ["lint"], stale: false },
+    blind: [{ slug: "swallowed-error", file: "src/a.py", command: "ruff check --select BLE ." }],
+    adopt: [{ name: "test", cmd: "npm test", source: "package.json" }],
+    shim: "missing",
+    start: [{ slug: "secrets-not-in-code", intent: "ключи не в коде", command: "gitleaks dir ." }],
+  });
+  assert.match(it[0], /aqk init/);
+  assert.match(it[1], /lint/);
+  assert.match(it[2], /swallowed-error.*src\/a\.py/);
+  assert.match(it[3], /npm test/);
+  assert.match(it[4], /@AGENTS\.md/);
+  assert.equal(it.length, 5, "больше пяти за раз агент не удержит");
+  assert.match(text({
+    manifest: false, run: { when: "x", red: ["lint"], stale: false },
+    blind: [{ slug: "s", file: "f", command: "" }], adopt: [{ name: "t", cmd: "c", source: "p" }],
+    shim: "missing", start: [{ slug: "x", intent: "y", command: "" }],
+  }), new RegExp(T.more(1).slice(0, 12)), "остаток назван числом, а не выброшен молча");
+});
+
+test("класс из пробы не повторяется в «начните с»; без готовой команды — aqk add", () => {
+  const it = items({
+    blind: [{ slug: "swallowed-error", file: "src/a.py", command: "" }],
+    start: [{ slug: "swallowed-error", intent: "i", command: "" }, { slug: "todo-without-task", intent: "i", command: "" }],
+  });
+  assert.equal(it.length, 2);
+  assert.match(it[0], /aqk add swallowed-error/);
+  assert.match(it[1], /todo-without-task/);
+});
+
+// Гейт стоит, проба его гоняла — и он пропустил. Не «поставь» (стоит), а «здесь он слеп».
+test("гейт стоит, но пропустил брак из пробы — пункт «разберись», сразу после красных", () => {
+  const it = items({
+    run: { when: "x", red: ["lint"], stale: false },
+    missed: [{ slug: "swallowed-error", file: "src/a.py" }],
+    start: [{ slug: "todo-without-task", intent: "i", command: "" }],
+  });
+  assert.match(it[1], /swallowed-error.*src\/a\.py/);
+  assert.doesNotMatch(it[1], /aqk add/, "гейт уже стоит — ставить его второй раз бессмысленно");
+});
+
+test("у каждого пункта — чем доказать, что готово", () => {
+  const it = items({
+    run: { when: "x", red: ["lint"], stale: false },
+    blind: [{ slug: "swallowed-error", file: "src/a.py", command: "" }],
+    missed: [{ slug: "complexity-limit", file: "src/b.py" }],
+    start: [{ slug: "todo-without-task", intent: "i", command: "" }],
+  });
+  for (const line of it) assert.match(line, new RegExp(T.done.slice(0, 6)), `пункт без арбитра: ${line}`);
+});
+
+test("оба языка несут одни и те же ключи задания", () => {
+  const keys = (o) => Object.keys(o).sort().join(",");
+  assert.equal(keys(CATALOGS.ru.prompt), keys(CATALOGS.en.prompt));
+  // Пункты лежат глубже — пропущенный в одном языке пункт упал бы ошибкой только у того, кто
+  // на этом языке работает.
+  assert.equal(keys(CATALOGS.ru.prompt.item), keys(CATALOGS.en.prompt.item));
+  assert.equal(keys(CATALOGS.ru.prompt.item.shim), keys(CATALOGS.en.prompt.item.shim));
+});

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, isApiSpec } from "../lib/repo.mjs";
+import { triggerVerdict, recipeFor, EXT_LANG, whichSync, browserServerAdvice, MARKS, isApiSpec, claudeSeesRules } from "../lib/repo.mjs";
 import { startWith } from "../lib/advice.mjs";
 import { proposeGates } from "../lib/adopt.mjs";
 import { CATALOGS, L } from "../i18n/index.mjs";
@@ -308,3 +308,22 @@ test("совет проекту на Biome: команда Biome, а не eslint
   const none = blindAdvice({ ...entry, biome_rules: "none" }, { langs: new Set(["javascript"]), has_biome: true });
   assert.match(none.command, /^eslint /);
 });
+
+// --- свод виден Claude Code --------------------------------------------------------
+// Документация Claude Code (code.claude.com/docs/en/memory, раздел AGENTS.md, сверено
+// 2026-09-11): «Claude Code reads CLAUDE.md, not AGENTS.md». Рекомендовано: CLAUDE.md с
+// @AGENTS.md либо символическая ссылка. Проект, где Claude Code настроен, а свод лежит только
+// в AGENTS.md, пишет правила агенту, который их не читает.
+test("свод в AGENTS.md виден Claude Code только через CLAUDE.md с @AGENTS.md", () => {
+  const base = { agents: true, claude: null, claudeLink: false, dotClaude: false };
+  assert.equal(claudeSeesRules(base), null, "Claude Code не настроен — молчим: Codex и Cursor читают AGENTS.md сами");
+  assert.equal(claudeSeesRules({ ...base, dotClaude: true }), "missing");
+  assert.equal(claudeSeesRules({ ...base, claude: "# Rules\n\nSee AGENTS.md for everything.\n" }), "noImport",
+    "упоминание словами — не подключение: файл Claude Code в контекст не загрузит");
+  assert.equal(claudeSeesRules({ ...base, claude: "# CLAUDE.md\n\n@AGENTS.md\n" }), null);
+  assert.equal(claudeSeesRules({ ...base, claude: "@../AGENTS.md\n" }), null, "из .claude/CLAUDE.md свод подключают на уровень выше");
+  assert.equal(claudeSeesRules({ ...base, claude: "Правила — в `@AGENTS.md`.\n" }), "noImport",
+    "в обратных кавычках @ не подключает — так в документации");
+  assert.equal(claudeSeesRules({ ...base, claude: "x", claudeLink: true }), null, "ссылка на AGENTS.md — тот же файл");
+  assert.equal(claudeSeesRules({ ...base, agents: false, dotClaude: true }), null, "AGENTS.md нет — подключать нечего");
+});