@vernikr/size-report 2.8.3 → 2.8.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vernikr/size-report",
-  "version": "2.8.3",
+  "version": "2.8.5",
   "author": "vernikr",
   "repository": {
     "type": "git",
@@ -65,7 +65,7 @@
     "report"
   ],
   "devDependencies": {
-    "@vernikr/size-report": "2.8.2",
+    "@vernikr/size-report": "2.8.4",
     "c8": "10",
     "dependency-cruiser": "17",
     "eslint": "^9.18.0",

package/src/config.js

@@ -76,7 +76,7 @@ export function gitRoot() {
  * request for that very file, so its absence stays a refusal (otherwise a typo in the path would
  * silently yield someone else's settings). `path` is given in words rather than as a path: there is no
  * file, and "edit <path>" would lead the reader to something the project does not have. */
-export function derivedConfig(root) {
+function derivedConfig(root) {
   const cfg = derivedProfile(root);
   cfg.path = 'derived from the project';
   cfg.derived = true;

package/src/hook.js

@@ -2,10 +2,9 @@ import fs from 'fs';
 import path from 'path';
 import { fileURLToPath } from 'url';
 import { EXIT, Refusal, cliCommand, refuseCause } from './refusal.js';
-import { installSpec } from './tool.js';
+import { TOOL_PKG, installSpec } from './tool.js';
 import { git, gitTry } from './git.js';
 import { loadConfig } from './config.js';
-import { TOOL_PKG } from './tool.js';
 import { rebuild } from './artifact.js';
 
 /* The `post-commit` and `post-merge` hooks: after every commit and merge the report rebuilds itself and, when it is tracked
@@ -120,6 +119,16 @@ function script(entry) {
     + 'exec node ' + entry.quoted + ' hook-run\n';
 }
 
+/* Writing the hook files, executable: the named command and the silent install write the same bytes, since a difference
+ * between them would surface only after somebody's commit. */
+function installFiles(hooks, entry, files) {
+  fs.mkdirSync(hooks.dir, { recursive: true });
+  files.forEach((file) => {
+    fs.writeFileSync(file, script(entry));
+    fs.chmodSync(file, 0o755);
+  });
+}
+
 /* The hook's state for `size doctor`: whether it is installed and how its last run ended. It counts nothing and commits
  * to nothing. */
 export function hookStatus(root) {
@@ -180,11 +189,7 @@ export function installHook(root, cfg) {
       '  to remove it: ' + cliCommand('uninstall-hook')
     ] };
   }
-  fs.mkdirSync(hooks.dir, { recursive: true });
-  files.forEach((file) => {
-    fs.writeFileSync(file, script(entry));
-    fs.chmodSync(file, 0o755);
-  });
+  installFiles(hooks, entry, files);
 
   const lines = [
     '✓ hook: ' + rels.join(', '),
@@ -219,11 +224,7 @@ export function autoInstall(root, cfg) {
     const files = HOOKS.map((name) => path.join(hooks.dir, name));
     if (files.some((f) => fs.existsSync(f) && !isOurs(f))) return null;
     if (files.every(isOurs)) return null;
-    fs.mkdirSync(hooks.dir, { recursive: true });
-    files.forEach((file) => {
-      fs.writeFileSync(file, script(entry));
-      fs.chmodSync(file, 0o755);
-    });
+    installFiles(hooks, entry, files);
     return files.map((f) => path.relative(root, f));
   } catch (_e) {
     /* Not a git repository, no rights on `.git`, a foreign format — all of it means one thing: there will be no automation

package/src/locales.js

@@ -48,9 +48,6 @@ export const LOCALES = {
         + 'что выбрано в этом браузере.',
       linkExtra: 'Часть ссылки пропущена: в этом отчёте нет {n} названных имён. '
         + 'Остальное применено.',
-      /* Данные отчёта лежат в файле сжатыми, и распаковывает их браузер. Текст называет причину
-       * (браузера нет в списке тех, что умеют `DecompressionStream`) и что с этим делать — пустая
-       * таблица не сказала бы читателю ничего. */
       unpack: 'Данные отчёта сжаты, а браузер не умеет их распаковать: в нём нет DecompressionStream. '
         + 'Обновите браузер — таблицу показывать не из чего.',
       note: 'Абсолютные размеры стоят один раз — в строке «{now}». Строк: {rows}: коммиты без '

package/src/metrics.js

@@ -58,9 +58,6 @@ const STYLES = {
   }
 };
 
-/* Метрика объявляет, нужен ли ей текст блоба: `needsText: false` вместе с
- * `fromSize: true` означает «хватит размера объекта», и тогда содержимое не
- * читается вовсе (`git cat-file --batch` не вызывается). */
 export const METRICS = {
   raw: {
     label: 'raw',
@@ -208,7 +205,7 @@ function esbuildLoader(file, cfg) {
 /* The engine actually in force: the requested one may be unavailable, in which case the metric
  * counts another way — and that is announced (`sensorGaps`), or a different count would travel
  * into CI as the requested one. */
-export function minEngine(cfg) {
+function minEngine(cfg) {
   if (cfg.minify.engine !== 'esbuild') return 'strip';
   return minifier().tool === null ? 'strip' : 'esbuild';
 }

package/src/page/table.js

@@ -8,8 +8,8 @@ import { appData, appUi, appView } from './state.js';
  *
  * **Why not a `<table>`.** Measured on this repository's own report (238 500 cells = 370 758 nodes, a table 71 712 ×
  * 5 982 px): about 1.4 GB of a browser's memory, of which roughly half the nodes and half the painted area, and a
- * browser's relayout of it costs close to a second on any switch (`probes/step-12-columns.mjs`). `content-visibility:
- * auto`, the cheap way out, is ignored on a table row by Chrome 153 (`probes/step-10-tables.mjs`), so the answer is
+ * browser's relayout of it costs close to a second on any switch (`probes/archive/step-12-columns.mjs`). `content-visibility:
+ * auto`, the cheap way out, is ignored on a table row by Chrome 153 (`probes/archive/step-10-tables.mjs`), so the answer is
  * to build less rather than to promise the browser will skip it. A grid of `position: absolute` rows has no layout to
  * be redone: a row is placed by its `top`, a column by the `left` of the group of cells that starts it, and the
  * browser never measures a cell to decide a width — every column is `--col` wide (70px), which is what the numbers
@@ -36,7 +36,7 @@ import { appData, appUi, appView } from './state.js';
  * numbers, so a step sideways makes the columns that entered and drops the ones that left (`appStrip`) — a cell per
  * row, the header's two lines included. A row that left the window vertically is dropped and made again when the
  * reader comes back to it, which is what makes a jump down the table cost the same as a step. Building the rows again
- * on every sideways scroll — the first version of this step — is what `probes/step-12-window.mjs` measures against the
+ * on every sideways scroll — the first version of this step — is what `probes/archive/step-12-window.mjs` measures against the
  * splice (`worklog/0211-sideways-step.md`).
  *
  * The choice is applied by building the window again: the columns of a switched-off file are simply not among the

package/src/project.js

@@ -204,7 +204,7 @@ function columnsOf(files, journal) {
 }
 
 export function projectConfig(root) {
-  const output = outputOf(root);
+  const output = outputOf();
   const journal = journalOf(root);
   const files = allPaths(root);
   /* Only what git tracks becomes a column: a file absent at HEAD has nothing to measure (it would be

package/src/refusal.js

@@ -28,8 +28,8 @@ export function refuse(code, message) {
 }
 
 /* The causes of a code-2 refusal in one list, and this is the only place where they are
- * spelled out: the help prints them from it, the code table in `README.md` is checked
- * against it (`test/docs-commands.test.js`), and `refuseCause` lets no refusal through
+ * spelled out: the help prints them from it, the code table in `docs/archive/README_old.md` is
+ * checked against it (`test/docs-commands.test.js`), and `refuseCause` lets no refusal through
  * without a named cause. So "the documentation says less than happens" cannot pass here
  * silently. The groups say where a cause comes from: the command line, settings and the
  * project, history, the hook, measurement. */
@@ -133,5 +133,5 @@ export const USAGE = [
   'internal error. doctor has an order of its own: 2, 3, 1, 4 — by the weight of a finding',
   'rather than by what came first.',
   '',
-  'Causes of a code-2 refusal (the code table in README.md names them too):'
+  'Causes of a code-2 refusal:'
 ].concat(CAUSE_LINES, ['']).join('\n');

package/src/strip/js.js

@@ -59,16 +59,21 @@ function regexAllowed(last, word) {
   return '([{,;:=!&|?+-*%~^<>'.indexOf(last) !== -1;
 }
 
-function takeRegex(s) {
-  if (s.src[s.i] !== '/' || !regexAllowed(s.last, s.word)) return false;
-  const end = endOfRegex(s.src, s.i);
+/* A token taken whole: it goes into the output and the pass state is advanced over it — the token itself, not the word
+ * before it, decides where a regex may stand next, and the word being read ends with it. */
+function take(s, end, last) {
   s.out += s.src.slice(s.i, end);
   s.i = end;
-  s.last = '/';
+  s.last = last;
   s.word = '';
   return true;
 }
 
+function takeRegex(s) {
+  if (s.src[s.i] !== '/' || !regexAllowed(s.last, s.word)) return false;
+  return take(s, endOfRegex(s.src, s.i), '/');
+}
+
 function endOfRegex(src, start) {
   let i = start + 1;
   let inClass = false;
@@ -88,12 +93,7 @@ function endOfRegex(src, start) {
 function takeString(s) {
   const quote = s.src[s.i];
   if (quote !== '"' && quote !== "'" && quote !== '`') return false;
-  const end = endOfString(s.src, s.i, quote);
-  s.out += s.src.slice(s.i, end);
-  s.i = end;
-  s.last = quote;
-  s.word = '';
-  return true;
+  return take(s, endOfString(s.src, s.i, quote), quote);
 }
 
 function endOfString(src, start, quote) {

package/src/tokens.js

@@ -28,7 +28,7 @@ export const CHARS_PER_TOKEN = 3;
  * and the tokenizer would split them into anything at all, giving a random number. The list
  * exists so that the metric says this in words instead of passing such a count off as counted.
  * SVG is deliberately not here: it is text, and its tokens are meaningful. */
-export const BINARY_EXTS = [
+const BINARY_EXTS = [
   '.png', '.jpg', '.jpeg', '.gif', '.webp', '.ico', '.avif',
   '.woff', '.woff2', '.ttf', '.otf', '.eot',
   '.pdf', '.zip', '.gz', '.tar', '.mp4', '.mp3', '.mov'

package/templates/README.md

@@ -83,5 +83,5 @@ well. About a single commit `pnpm exec size explain <sha>` answers.
 
 A block for the project's own agent files (`AGENTS.md` and the like) is deliberately absent here: the
 requirements do not ask for such a file, and inventing the format of someone else's repository is not
-something the tool should do. What an agent needs to know it will take from `size --help` and `size --data` —
-the commands and the data are described in the package's `README.md`, section "For an AI agent".
+something the tool should do. What an agent needs to know it will take from `size --help` and
+`size --data` — the commands and the data are described there rather than in a file of the tool's own.