@vernikr/size-report 1.2.0 → 1.3.1

package/src/config.js

@@ -7,6 +7,7 @@ import { LOCALES } from './locales.js';
 import { METRICS, MINIFY_ENGINES } from './metrics.js';
 import { TOKEN_DEFAULTS, TOKEN_FAMILIES } from './tokens.js';
 import { CATEGORY_ORDER } from './data.js';
+import { projectConfig } from './project.js';
 
 /* Настройки проекта-потребителя: значения по умолчанию, чтение и проверка.
  * Настройки описывают проект, а не механику, поэтому проверка стоит здесь же и
@@ -70,6 +71,41 @@ export function gitRoot() {
   }
 }
 
+/* Настроек нет — их выводит сам проект (`src/project.js`), и работа начинается сразу:
+ * заводить файл ради первого запуска незачем, а `--init` закрепляет выведенное
+ * файлом, когда его хотят править. Так бывает только с умолчательным именем: файл,
+ * названный ключом `--config`, — это уже запрос про конкретный файл, и его
+ * отсутствие остаётся отказом (иначе опечатка в пути молча дала бы чужие настройки).
+ * `path` назван словами, а не путём: файла нет, и текст «правьте <путь>» привёл бы
+ * человека к тому, чего в проекте не лежит. */
+export function derivedConfig(root) {
+  const cfg = derivedProfile(root);
+  cfg.path = 'настройки, выведенные из проекта';
+  cfg.derived = true;
+  validateConfig(cfg);
+  return cfg;
+}
+
+/* Выведенное из проекта + умолчания — то, чем проект работает без файла, и то,
+ * что закрепляет `--init`. Одно место на две роли (иначе «файл» и «работа без
+ * файла» разошлись бы колонкой или числом), а сам вывод (`src/project.js`) о
+ * умолчаниях не знает: он говорит только то, что видит в проекте. */
+export function derivedProfile(root) {
+  return withDefaults(projectConfig(root));
+}
+
+/* Настройки поверх умолчаний — одним местом на два источника (файл и проект):
+ * вложенное досыпается по ключам, потому что `minify: {engine: …}` не значит «у
+ * `minify` больше нет других полей», а значило бы, что снятие балласта потеряло
+ * список расширений. */
+function withDefaults(raw) {
+  const cfg = Object.assign({}, DEFAULT_CONFIG, raw);
+  ['minify', 'tokens', 'hooks', 'links', 'rows'].forEach((key) => {
+    cfg[key] = Object.assign({}, DEFAULT_CONFIG[key], raw[key]);
+  });
+  return cfg;
+}
+
 /* Настройки читаются как есть и досыпаются значениями по умолчанию: у проекта,
  * который только подключил генератор, конфиг может быть в три строки. */
 export function loadConfig(file, root) {
@@ -78,8 +114,10 @@ export function loadConfig(file, root) {
     // черновик под умолчательным именем в корне проекта — то есть починил бы не то,
     // о чём спросили. Имя не называем ровно тогда, когда оно и так умолчательное.
     const dflt = root !== undefined && path.resolve(root, CONFIG_NAME) === path.resolve(file);
+    if (dflt) return derivedConfig(root);
     refuseCause('нет файла настроек', 'нет файла настроек ' + file
-      + '\n  создайте его: ' + cliCommand('--init' + (dflt ? '' : ' ' + advicePath(file))));
+      + '\n  создайте его: ' + cliCommand('--init ' + advicePath(file))
+      + '\n  смотрите: без «--config» настройки не нужны — они выводятся из проекта');
   }
   let raw;
   try {
@@ -88,26 +126,19 @@ export function loadConfig(file, root) {
     refuseCause('настройки не разобраны', 'не разобран ' + file + ': ' + e.message
       + '\n  починка: правьте ' + file + '; образец настроек даёт ' + cliCommand('--init') + ' в пустом каталоге');
   }
-  const cfg = Object.assign({}, DEFAULT_CONFIG, raw);
-  cfg.minify = Object.assign({}, DEFAULT_CONFIG.minify, raw.minify);
-  cfg.tokens = Object.assign({}, DEFAULT_CONFIG.tokens, raw.tokens);
-  cfg.hooks = Object.assign({}, DEFAULT_CONFIG.hooks, raw.hooks);
-  cfg.links = Object.assign({}, DEFAULT_CONFIG.links, raw.links);
-  cfg.rows = Object.assign({}, DEFAULT_CONFIG.rows, raw.rows);
+  const cfg = withDefaults(raw);
   cfg.path = file;
   validateConfig(cfg);
   return cfg;
 }
 
-export function validateConfig(cfg) {
-  const fail = (msg) => refuseCause('настройки неверны',
-    'конфиг ' + cfg.path + ': ' + msg + '\n  починка: правьте ' + cfg.path);
+/* Колонки: метка и пути — имена, а не что попало: путь числом или объектом молча не
+ * совпадает ни с чем, и колонка отчитывается нулём строк за успех. Отказ обязан
+ * случиться здесь, а не превратиться в пустой отчёт. */
+function checkColumns(cfg, fail) {
   if (!cfg.columns || cfg.columns.length === 0) fail('не задано ни одной колонки (columns)');
   const labels = new Set();
   cfg.columns.forEach((c, i) => {
-    // Метка и пути — имена, а не что попало: путь числом или объектом молча не
-    // совпадает ни с чем, и колонка отчитывается нулём строк за успех. Отказ
-    // обязан случиться здесь, а не превратиться в пустой отчёт.
     const pathsAreNames = c && Array.isArray(c.paths)
       && c.paths.length > 0 && c.paths.every((p) => typeof p === 'string' && p !== '');
     if (!c || typeof c.label !== 'string' || c.label === '' || !pathsAreNames) {
@@ -122,13 +153,22 @@ export function validateConfig(cfg) {
     }
     labels.add(c.label);
   });
+}
+
+function checkMetrics(cfg, fail) {
   if (!Array.isArray(cfg.metrics) || cfg.metrics.length === 0) fail('не заданы метрики (metrics)');
   cfg.metrics.forEach((m) => {
     if (!METRICS[m]) fail('неизвестная метрика «' + m + '» (есть: ' + Object.keys(METRICS).join(', ') + ')');
   });
+}
+
+function checkMinify(cfg, fail) {
   if (MINIFY_ENGINES.indexOf(cfg.minify.engine) < 0) {
     fail('неизвестный способ минификации «' + cfg.minify.engine + '» (есть: ' + MINIFY_ENGINES.join(', ') + ')');
   }
+}
+
+function checkTokens(cfg, fail) {
   const family = TOKEN_FAMILIES[cfg.tokens.family];
   if (family === undefined) {
     fail('неизвестное семейство токенизатора «' + cfg.tokens.family + '» (есть: '
@@ -138,21 +178,60 @@ export function validateConfig(cfg) {
     fail('неизвестная кодировка токенизатора «' + cfg.tokens.encoding + '» у семейства '
       + cfg.tokens.family + ' (есть: ' + family.encodings.join(', ') + ')');
   }
+}
+
+/* Файл таблицы не может быть её колонкой: размер артефакта зависит от числа строк,
+ * то есть от самого себя. */
+function checkOutput(cfg, fail) {
+  cfg.columns.forEach((c) => {
+    if (c.paths.indexOf(cfg.output) >= 0) fail('файл таблицы (' + cfg.output + ') не может быть колонкой');
+  });
+  if (!cfg.output) fail('не задан output');
+}
+
+function checkJournal(cfg, fail) {
+  if (!cfg.journal) return;
+  if (!cfg.journal.path) fail('journal.path не задан');
+  if (!cfg.journal.pattern) fail('journal.pattern не задан');
+  try { new RegExp(cfg.journal.pattern); } catch (e) { fail('journal.pattern не компилируется: ' + e.message); }
+}
+
+/* Что настройки говорят про путь: `columns` — его отслеживает колонка, `excluded` — он
+ * объявлен исключением (`skip` и сам файл отчёта), `outside` — мимо того и другого.
+ * Суждение одно на два ответа: `check` спрашивает его про всю историю, `explain` — про
+ * один коммит. Знакомство считается по колонкам целиком, а не по метке: у колонки путей
+ * может быть несколько (переименование), и любой из них — она сама. */
+export function pathRoles(cfg) {
+  const tracked = new Set();
+  cfg.columns.forEach((col) => col.paths.forEach((p) => tracked.add(p)));
+  const excluded = new Set([cfg.output].concat(cfg.skip || []));
+  return (file) => {
+    if (tracked.has(file)) return 'columns';
+    return excluded.has(file) ? 'excluded' : 'outside';
+  };
+}
+
+/* Починка «мимо колонок» — один текст на два ответа, и он называет пути: команда без
+ * имён не команда. Текст собирается из имён, а не приписывает их по ветке, поэтому и
+ * предусматривать тут нечего: коммит без файлов вовсе починки не получает — на пустом
+ * списке её не зовут (см. `fixFor` в `src/explain.js`). */
+export function outsideFix(paths) {
+  return 'допишите эти пути колонкой или в «skip» файла ' + CONFIG_NAME + ': ' + paths.join(', ');
+}
+
+export function validateConfig(cfg) {
+  const fail = (msg) => refuseCause('настройки неверны',
+    'конфиг ' + cfg.path + ': ' + msg + '\n  починка: правьте ' + cfg.path);
+  checkColumns(cfg, fail);
+  checkMetrics(cfg, fail);
+  checkMinify(cfg, fail);
+  checkTokens(cfg, fail);
   if (!LOCALES[cfg.locale]) fail('неизвестная локаль «' + cfg.locale + '» (есть: ' + Object.keys(LOCALES).join(', ') + ')');
   // Выключатель хука — «да/нет», а не «правда/ложь»: `false` от случайной строки
   // отличать обязан инструмент, иначе выключенная автоматика осталась бы включённой.
   if (typeof cfg.hooks.enabled !== 'boolean') {
     fail('hooks.enabled — не «да/нет»: ' + JSON.stringify(cfg.hooks.enabled));
   }
-  // Файл таблицы не может быть её колонкой: размер артефакта зависит от числа
-  // строк, то есть от самого себя.
-  cfg.columns.forEach((c) => {
-    if (c.paths.indexOf(cfg.output) >= 0) fail('файл таблицы (' + cfg.output + ') не может быть колонкой');
-  });
-  if (!cfg.output) fail('не задан output');
-  if (cfg.journal) {
-    if (!cfg.journal.path) fail('journal.path не задан');
-    if (!cfg.journal.pattern) fail('journal.pattern не задан');
-    try { new RegExp(cfg.journal.pattern); } catch (e) { fail('journal.pattern не компилируется: ' + e.message); }
-  }
+  checkOutput(cfg, fail);
+  checkJournal(cfg, fail);
 }

package/src/data.js

@@ -66,14 +66,8 @@ export function reportData(cfg, root) {
     categories: CATEGORY_ORDER.filter((key) => files.some((f) => f.category === key))
       .map((key) => ({ key: key, label: loc.categories[key] })),
     files: files,
-    rows: rows.map((r) => ({
-      sha: r.sha,
-      when: r.when,
-      subject: r.subject,
-      section: r.section === null ? null : { id: r.section.id, head: r.section.head, added: r.section.added },
-      href: rowHref(r.section, r.sha, cfg),
-      values: r.cells
-    })),
+    rows: rows.map((r) => Object.assign(rowShape(r),
+      { href: rowHref(r.section, r.sha, cfg), values: r.cells })),
     now: state.map((s) => (s === null ? null : s.cells)),
     approx: approxMarks(rows, state, cfg),
     skipped: dropped.map(skipLine)
@@ -99,7 +93,20 @@ function approxMarks(rows, state, cfg) {
   return out;
 }
 
+/* Общая часть строки ответа: она есть и у контракта страницы (`--data`), и у
+ * прежней формы `--json` (она заморожена эталоном паритета). Одно место — потому
+ * что разойтись эти два ответа могут ровно здесь, а **порядок полей и есть байты
+ * ответа**: они в объекте в том же порядке, в каком их печатает замороженная форма. */
+export function rowShape(r) {
+  return {
+    sha: r.sha,
+    when: r.when,
+    subject: r.subject,
+    section: r.section === null ? null : { id: r.section.id, head: r.section.head, added: r.section.added }
+  };
+}
+
 /* Производные величины живут в `src/derived.js`: их считает и артефакт (импорт
  * ниже), и страница (получает тот же файл текстом). Второго расчёта той же
  * таблицы нет вовсе, поэтому разойтись молча двум отчётам нечем — это стережёт
- * `test/contract.test.js`. */
+ * `test/contract-derived.test.js`. */

package/src/derived.js

@@ -12,8 +12,9 @@
  *      непонятая строка не должна молча попасть в страницу (`pageScript`).
  *
  * Расхождение двух отчётов возможно только здесь, поэтому и стеречь его надо
- * здесь: `test/contract.test.js` сверяет числа страницы с числами артефакта и
- * следит, чтобы у страницы не появилось своего расчёта. */
+ * здесь: `test/contract-derived.test.js` сверяет числа страницы с числами
+ * артефакта, а `test/page-view.test.js` следит, чтобы у страницы не появилось
+ * своего расчёта. */
 
 // Разряды тонкими пробелами: toLocaleString зависит от ICU сборки Node, а строка
 // таблицы обязана совпадать побайтово на любой машине.

package/src/doctor.js

@@ -4,6 +4,7 @@ import { loadConfig } from './config.js';
 import { TOOL_PKG } from './tool.js';
 import { coverage, coverageText } from './check.js';
 import { hookStatus } from './hook.js';
+import { derivedSummary } from './project.js';
 import { minifier } from './minify.js';
 import { tokenizer } from './tokens.js';
 
@@ -15,13 +16,10 @@ import { tokenizer } from './tokens.js';
  *
  * Правило ответа: `ok` значит «делать нечего», а у находки назван уровень.
  * `action` — что-то надо сделать (и, где возможно, названа команда починки);
- * `note` — наблюдение: знать полезно, делать нечего. Код выхода — первый по
- * важности, а не «всё хорошо»: 2 — настройки нечитаемы (читать больше нечего),
- * 3 — история обрезана, 1 — покрытие неполно, 4 — число приближённо. Порядок
- * именно такой: сначала то, что мешает считать, потом то, что требует починки,
- * потом честная оговорка о счёте. Хук в этот порядок не входит: отчёт собирается
- * и без него, поэтому сломанный хук — находка без своего кода (вердикт `ok` при
- * этом всё равно «есть дело»).
+ * `note` — наблюдение: знать полезно, делать нечего. Код выхода считает один
+ * `verdictOf` в конце: шаги чтения только называют вид обстоятельства (`troubles`),
+ * а и порядок видов, и код каждого — один список `WEIGHT`. Своего кода у шага нет,
+ * поэтому разойтись эти два ответа не могут.
  *
  * Чего ответ не делает: не говорит, «правильно» ли выбраны колонки (это знает
  * проект), и не угадывает там, где данных нет, — отсутствие ответа называется
@@ -81,85 +79,142 @@ function dependencies(cfg) {
   ];
 }
 
-export function doctor(root, configFile) {
-  const rep = {
-    schema: 1,
-    ok: false,
-    tool: { name: TOOL_PKG.name, version: TOOL_PKG.version },
-    environment: environment(root),
-    config: null,
-    dependencies: null,
-    hooks: null,
-    coverage: null,
-    findings: [],
-    exit: EXIT.OK
-  };
-  let cfg = null;
+/* Настройки: при нечитаемых ответ честно неполон (покрытие считать нечем), а
+ * причина — не отказ, а находка: диагностика затем и нужна, чтобы назвать причину
+ * и починку, — их и несёт текст отказа. Вес обстоятельства шаг только **называет**
+ * (`troubles`), а важнее оно или нет — не его дело: решает `verdictOf`.
+ *
+ * Настроек, выведенных из проекта, тут не обстоятельство, а наблюдение: проект
+ * работает, но числа его отчёта зависят от того, что инструмент о нём угадал, —
+ * поэтому находка уровня `note` (код выхода она не несёт) и поле `derived` в ответе. */
+function readConfig(root, configFile) {
   try {
-    cfg = loadConfig(configFile, root);
-    rep.config = { file: configFile, ok: true, columns: cfg.columns.length, metrics: cfg.metrics };
+    const cfg = loadConfig(configFile, root);
+    const derived = cfg.derived === true;
+    return {
+      cfg: cfg,
+      report: {
+        file: configFile, ok: true, derived: derived,
+        columns: cfg.columns.length, metrics: cfg.metrics
+      },
+      findings: derived
+        ? [{ level: 'note', what: derivedSummary(cfg), fix: 'закрепите их файлом: ' + cliCommand('--init') }]
+        : [],
+      troubles: []
+    };
   } catch (e) {
-    // Отказ настроек здесь не отказ, а находка: диагностика затем и нужна, чтобы
-    // назвать причину и починку, — их и несёт текст отказа.
     if (!(e instanceof Refusal)) throw e;
-    rep.config = { file: configFile, ok: false, problem: e.message };
-    rep.findings.push({ level: 'action', what: e.message });
-    rep.exit = e.code;
+    return {
+      cfg: null,
+      report: { file: configFile, ok: false, problem: e.message },
+      findings: [{ level: 'action', what: e.message }],
+      troubles: ['config']
+    };
   }
-  rep.dependencies = dependencies(cfg);
-  rep.hooks = hooksReport(root, cfg);
-  if (rep.hooks.installed && rep.hooks.enabled === false) {
-    rep.findings.push({
+}
+
+/* Хук: две находки, у каждой своя починка. Веса у них нет и своего кода выхода тоже —
+ * отчёт собирается и без хука, поэтому сломанный хук меняет только вердикт `ok`. */
+function hookFindings(hooks) {
+  const found = [];
+  if (!hooks.installed) return found;
+  if (hooks.enabled === false) {
+    found.push({
       level: 'action',
       what: 'хук установлен, но автоматика выключена настройкой hooks.enabled: отчёт обновляется руками',
       fix: 'верните «"hooks": {"enabled": true}» в файл настроек или снимите хук: ' + cliCommand('uninstall-hook')
     });
   }
-  if (rep.hooks.installed && rep.hooks.last !== null && HOOK_BAD.indexOf(rep.hooks.last.result) >= 0) {
-    rep.findings.push({
+  if (hooks.last !== null && HOOK_BAD.indexOf(hooks.last.result) >= 0) {
+    found.push({
       level: 'action',
-      what: 'хук: последний запуск не пересобрал отчёт — ' + rep.hooks.last.why,
+      what: 'хук: последний запуск не пересобрал отчёт — ' + hooks.last.why,
       fix: 'починьте то, на что жалуется причина, и пересоберите отчёт: ' + cliCommand('--write')
     });
   }
+  return found;
+}
+
+/* Покрытие — тот же ответ, что даёт `size check`, плюс вид обстоятельства, если оно есть:
+ * неполнота пути или приближение датчика (вес у видов разный — `WEIGHT`); неполнота
+ * старше, потому что без неё чисел нет вовсе.
+ * Что попадает в находки, а что нет: в отчёте целиком стоит блок покрытия (тот же текст,
+ * что у `size check`), поэтому неполнота второй раз не пересказывается — она весит. А по
+ * датчикам находка есть: `size check` печатает их строкой `!`, здесь они часть ответа. */
+function readCoverage(cfg, root, configFile) {
   if (cfg === null) {
-    rep.findings.push({
-      level: 'note',
-      what: 'покрытие не считалось: настройки нечитаемы — почините их и спросите снова'
-    });
-  } else {
-    try {
-      rep.coverage = coverage(cfg, root, configFile);
-    } catch (e) {
-      if (!(e instanceof Refusal)) throw e;
-      rep.findings.push({ level: 'action', what: e.message });
-      rep.exit = e.code;
-    }
+    return {
+      report: null,
+      findings: [{
+        level: 'note',
+        what: 'покрытие не считалось: настройки нечитаемы — почините их и спросите снова'
+      }],
+      troubles: []
+    };
   }
-
-  /* Что попадает в находки, а что нет: в отчёте уже целиком стоит блок покрытия
-   * (тот же текст, что у `size check`), поэтому неполнота здесь второй раз не
-   * пересказывается — она меняет вердикт и код выхода. Находкой становится то,
-   * чего в блоке покрытия нет: нечитаемые настройки, обрезанная история, а по
-   * датчикам — их причина и починка (их `size check` печатает отдельной строкой
-   * `!`, а здесь они часть того же ответа). Неполнота старше приближения: из
-   * двух причин починки код выхода несёт ту, без которой чисел нет вовсе. */
-  if (rep.coverage !== null) {
-    rep.coverage.sensors.forEach((gap) => {
-      rep.findings.push({ level: 'action', what: gap.why, fix: gap.fix });
-    });
-    if (!rep.coverage.ok) rep.exit = EXIT.VIOLATION;
-    else if (rep.coverage.sensors.length > 0) rep.exit = EXIT.SENSOR;
+  try {
+    const report = coverage(cfg, root, configFile);
+    return {
+      report: report,
+      findings: report.sensors.map((gap) => ({ level: 'action', what: gap.why, fix: gap.fix })),
+      troubles: report.ok ? (report.sensors.length > 0 ? ['sensor'] : []) : ['coverage']
+    };
+  } catch (e) {
+    if (!(e instanceof Refusal)) throw e;
+    /* Внутри покрытия отказывают двое, и род у них разный: обрезанной истории —
+     * свой вид (`assertFullHistory`), а неразобранному файлу — настройки: числа
+     * нет из-за них, и починка у него настройками же. Вид выбирается по коду
+     * отказа — эти два кода и есть весь выбор. */
+    return {
+      report: null,
+      findings: [{ level: 'action', what: e.message }],
+      troubles: [e.code === EXIT.SHALLOW ? 'history' : 'config']
+    };
   }
+}
 
-  /* «Делать нечего»: ни одной находки-действия и покрытие сосчитано и полно.
-   * Покрытие спрашивается отдельно, потому что его неполнота говорится не находкой,
-   * а блоком покрытия (см. выше), — а вердикт она менять обязана. */
+/* Вес обстоятельств — вот и весь порядок важности, и он один на весь модуль: ключи идут
+ * по важности, значения — код каждого вида. Код выхода берётся у самого важного из
+ * найденного, а не у того, что нашлось позже: сначала чем считать нечем (настройки,
+ * история), потом неполное покрытие, потом оговорка о счёте.
+ * Хук в список не входит: отчёт собирается и без него (см. `hookFindings`). */
+const WEIGHT = {
+  config: EXIT.CONFIG,
+  history: EXIT.SHALLOW,
+  coverage: EXIT.VIOLATION,
+  sensor: EXIT.SENSOR
+};
+
+/* Вердикт — одно место, где обстоятельства превращаются в код выхода и в `ok`.
+ * «Делать нечего» — это ни одной находки-действия и сосчитанное полное покрытие: без
+ * покрытия вердикта нет, потому что считать больше нечего (см. заметку в `readCoverage`). */
+function verdictOf(rep, troubles) {
+  const found = Object.keys(WEIGHT).filter((kind) => troubles.indexOf(kind) >= 0);
+  rep.exit = found.length === 0 ? EXIT.OK : WEIGHT[found[0]];
   rep.ok = rep.coverage !== null && rep.coverage.ok
     && !rep.findings.some((f) => f.level === 'action');
   return rep;
 }
 
+export function doctor(root, configFile) {
+  const config = readConfig(root, configFile);
+  const hooks = hooksReport(root, config.cfg);
+  const cov = readCoverage(config.cfg, root, configFile);
+  const rep = {
+    schema: 1,
+    ok: false,
+    tool: { name: TOOL_PKG.name, version: TOOL_PKG.version },
+    environment: environment(root),
+    config: config.report,
+    dependencies: dependencies(config.cfg),
+    hooks: hooks,
+    coverage: cov.report,
+    findings: config.findings.concat(hookFindings(hooks), cov.findings),
+    exit: EXIT.OK
+  };
+  return verdictOf(rep, config.troubles.concat(cov.troubles));
+}
+
 /* Итог последнего запуска хука словами: по нему человек понимает, что произошло
  * после коммита, не заглядывая в `.git`. */
 const HOOK_RESULT = {
@@ -210,7 +265,8 @@ export function doctorText(rep) {
   lines.push('  git читается с закреплениями: ' + env.pins.join(', ') + '; локаль ' + env.locale
     + ' (настройки машины на числа не влияют)');
   lines.push('  настройки: ' + (rep.config.ok
-    ? rep.config.file + ' — ' + rep.config.columns + ' колонок, метрики ' + rep.config.metrics.join(' ')
+    ? (rep.config.derived ? 'выводятся из проекта (файла нет)' : rep.config.file)
+      + ' — ' + rep.config.columns + ' колонок, метрики ' + rep.config.metrics.join(' ')
     : rep.config.file + ' — нечитаемы'));
   lines.push('  зависимости: ' + rep.dependencies.map((d) => d.name
     + (d.present === null ? ' — ' + d.note : d.present ? ' ' + d.version + ' есть' : ' нет')

package/src/explain.js

@@ -1,7 +1,7 @@
 import { assertFullHistory, readHistory, resolveCommit } from './git.js';
 import { measureHistory } from './history.js';
 import { cliCommand, refuseCause } from './refusal.js';
-import { CONFIG_NAME } from './config.js';
+import { outsideFix, pathRoles } from './config.js';
 
 /* Почему у коммита нет строки — ответ на конкретный вопрос про конкретный коммит.
  *
@@ -21,12 +21,10 @@ const REASON_TEXT = {
   flat: 'числа не сдвинулись: файлы колонок тронуты, а объём не изменился'
 };
 
-export function explainCommit(cfg, root, target) {
-  assertFullHistory(root);
-  const commits = readHistory(root);
-  /* Имя ревизии разрешает git, и только если имени нет — ищем начало sha по
-   * списку коммитов: так у неоднозначного префикса остаётся человеческий отказ
-   * со списком подходящих, а у имени — правила git, а не наши. */
+/* Коммит по названию. Имя ревизии разрешает git, и только если имени нет — ищем
+ * начало sha по списку коммитов: так у неоднозначного префикса остаётся
+ * человеческий отказ со списком подходящих, а у имени — правила git, а не наши. */
+function lookup(root, commits, target) {
   const resolved = resolveCommit(root, String(target));
   const needle = String(target).toLowerCase();
   const found = resolved === null
@@ -50,33 +48,52 @@ export function explainCommit(cfg, root, target) {
       + '\n  ' + found.slice(0, 5).map((c) => c.sha.slice(0, 7) + ' ' + c.subject).join('\n  ')
       + '\n  починка: назовите больше знаков');
   }
-  const c = found[0];
-  const measured = measureHistory(cfg, root, commits);
-  const row = measured.rows.findIndex((r) => r.sha === c.sha);
-  const dropped = measured.dropped.find((d) => d.sha === c.sha);
+  return found[0];
+}
 
-  const tracked = new Set();
-  cfg.columns.forEach((col) => col.paths.forEach((p) => tracked.add(p)));
-  const excluded = new Set([cfg.output].concat(cfg.skip || []));
+/* Улики: что коммит тронул — колонки, исключённое, мимо колонок. Суждение о роли пути
+ * — одно и живёт в настройках (`pathRoles`), здесь только раскладка его ответа. */
+function touchedOf(cfg, files) {
+  const role = pathRoles(cfg);
   const touched = { columns: [], excluded: [], untracked: [] };
-  c.files.forEach((f) => {
-    if (tracked.has(f)) { if (touched.columns.indexOf(f) < 0) touched.columns.push(f); return; }
-    if (excluded.has(f)) { if (touched.excluded.indexOf(f) < 0) touched.excluded.push(f); return; }
-    if (touched.untracked.indexOf(f) < 0) touched.untracked.push(f);
+  const into = { columns: touched.columns, excluded: touched.excluded, outside: touched.untracked };
+  files.forEach((f) => {
+    const bucket = into[role(f)];
+    if (bucket.indexOf(f) < 0) bucket.push(f);
   });
+  return touched;
+}
+
+const FIX = {
+  merge: 'включите строки слияний: "rows": { "merges": true }',
+  report: 'не требуется: строка про коммит не может лежать внутри самого коммита — обновляйте отчёт отдельным коммитом',
+  flat: 'не требуется: числа не изменились — строка без единого числа читалась бы как поломка'
+};
+
+/* Починка по причине. У «мимо колонок» она одна на два ответа (`outsideFix`) и называет
+ * пути. У коммита без файлов (`--allow-empty`) называть нечего — значит, починки нет
+ * вовсе, а не команда без имён; этот случай и держит ветка. */
+function fixFor(reason, touched) {
+  if (reason === 'outside') {
+    return touched.untracked.length === 0 ? null : outsideFix(touched.untracked);
+  }
+  return FIX[reason];
+}
+
+export function explainCommit(cfg, root, target) {
+  assertFullHistory(root);
+  const commits = readHistory(root);
+  const c = lookup(root, commits, target);
+  const measured = measureHistory(cfg, root, commits);
+  const row = measured.rows.findIndex((r) => r.sha === c.sha);
+  const dropped = measured.dropped.find((d) => d.sha === c.sha);
+  const touched = touchedOf(cfg, c.files);
 
   /* Разница, которой нет в строке отчёта: «без изменения объёма» у коммита мимо
    * колонок означает не то же самое, что у коммита, тронувшего колонку. */
   let reason = row >= 0 ? null : dropped.reason;
   if (reason === 'flat' && touched.columns.length === 0) reason = 'outside';
-
-  const fix = {
-    merge: 'включите строки слияний: "rows": { "merges": true }',
-    report: 'не требуется: строка про коммит не может лежать внутри самого коммита — обновляйте отчёт отдельным коммитом',
-    outside: 'допишите ' + (touched.untracked.length > 0 ? 'эти пути' : 'тронутые файлы')
-      + ' колонкой или в «skip» файла ' + CONFIG_NAME,
-    flat: 'не требуется: числа не изменились — строка без единого числа читалась бы как поломка'
-  }[reason];
+  const fix = fixFor(reason, touched);
 
   return {
     schema: 1,