@operstack/audit 0.16.0 → 0.16.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@operstack/audit",
-  "version": "0.16.0",
+  "version": "0.16.2",
   "description": "SEO, AEO and GEO audit generator: collect public signals from a site into audit.json, let an analyst or an agent fill the narrative, render the OperStack report as HTML and PDF.",
   "type": "module",
   "license": "MIT",

package/src/collect.mjs

@@ -448,11 +448,6 @@ export async function collect(startUrl, { pages = 20, log = () => {}, backlinks
   const msg = pagesForTrust.filter((p) => p.messengerLinks > 0);
   checks.push(row('conv-messenger', 'conversion', 'A messenger link', msg.length ? 'ok' : 'warn', msg.length ? `on ${msg.length} of ${pagesForTrust.length} sampled page(s)` : 'none on the sampled pages', msg.length ? '' : 'for an international audience reading on a phone, one messenger link is usually worth more than a form'));
 
-  const { scores, scoreBasis } = computeScores(checks);
-
-  // Один балл на весь отчёт: его же берут страница проверки и письмо, чтобы цифры не спорили.
-
-  const overall = computeOverall(checks);
   // Оценки считаются до перевода: язык на цифры не влияет.
   // Сколько текста не видно без выполнения скриптов. Меряется настоящим браузером и только по
   // просьбе: запуск браузера это десятки секунд на страницу, и еженедельному мониторингу это ни к
@@ -473,6 +468,12 @@ export async function collect(startUrl, { pages = 20, log = () => {}, backlinks
     checks.push(jsBlindnessCheck(measured, lang));
   }
 
+  // Балл считается ПОСЛЕ всех этапов, включая браузерный: раньше он снимался до него, и на прогоне
+  // с `rendered: true` сохранённое поле расходилось с тем, что печатает отчёт. Письмо говорило 61,
+  // PDF печатал 59. Одна функция, один набор проверок, один момент: вот этот.
+  const { scores, scoreBasis } = computeScores(checks);
+  const overall = computeOverall(checks);
+
   const localised = localiseChecks(checks, lang);
   const critical = checks.filter((c) => c.status === 'bad').map((c) => ({ title: c.label, text: `${c.value}${c.comment ? `. ${c.comment}` : ''}`, level: 'bad' }));
 

package/src/test-overall.mjs

@@ -5,7 +5,8 @@
  *  44 из 60, то есть 73 процента. Два разных движка в одном документе читаются как выдуманные
  *  цифры. Эти тесты держат единственный источник балла и запрет считать неизмеренное нулём. */
 import { readFileSync } from 'node:fs';
-import { computeOverall, computeScores, verifyScores, SCORE_AREAS } from './collect.mjs';
+import { createServer } from 'node:http';
+import { collect, computeOverall, computeScores, verifyScores, SCORE_AREAS } from './collect.mjs';
 import { toHtml } from './render.mjs';
 
 let bad = 0;
@@ -86,5 +87,25 @@ is('все спорные: половина', computeOverall(every('warn')).scor
   ok('разбивка по областям осталась под общим баллом', Object.keys(scores).every((k) => html.includes('/10') || scores[k] === null));
 }
 
+// Сохранённые оценки обязаны сходиться с итоговым набором проверок того же объекта.
+//
+// 14.09.2026 письмо сказало 61, а PDF напечатал 59: поле снималось до браузерного этапа, а он
+// дописывает проверки. Отчёт считал заново и получал другое число. Тест поднимает маленький сайт
+// и сверяет то, что collect положил в объект, с тем, что даёт функция по его же checks.
+{
+  const server = createServer((req, res) => {
+    if (req.url === '/robots.txt') return res.end('User-agent: *\nAllow: /\nSitemap: http://127.0.0.1:' + server.address().port + '/sitemap.xml');
+    if (req.url === '/sitemap.xml') { res.setHeader('content-type', 'application/xml'); return res.end(`<?xml version="1.0"?><urlset><url><loc>http://127.0.0.1:${server.address().port}/</loc></url></urlset>`); }
+    res.setHeader('content-type', 'text/html');
+    res.end('<!doctype html><html><head><title>A small test page for the audit collector</title><meta name="description" content="A page that exists only so the collector has something real to read during the test run of the suite."></head><body><h1>Test page</h1><p>Ten pounds a month, measured on our own invoices in 2026.</p><h2>One</h2><p>Text.</p><h2>Two</h2><p>Text.</p><h2>Three</h2><p>Text.</p><a href="mailto:a@b.c">write</a></body></html>');
+  });
+  await new Promise((r) => server.listen(0, '127.0.0.1', r));
+  const audit = await collect(`http://127.0.0.1:${server.address().port}/`, { pages: 2, log: () => {} });
+  server.close();
+  is('сохранённый общий балл равен пересчитанному по тем же проверкам', audit.overall.score, computeOverall(audit.checks).score);
+  is('сохранённые оценки областей тоже сходятся', audit.scores, computeScores(audit.checks).scores);
+  is('и собственная проверка отчёта молчит', verifyScores(audit).filter((p) => p.startsWith('overall')), []);
+}
+
 if (bad) { console.error(`\n${bad} тест(ов) упало`); process.exit(1); }
 console.log('\nвсе тесты общего балла прошли');