@7n/rules 1.18.1 → 1.19.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [1.19.0] - 2026-07-18
+
+### Added
+
+- js/eslint T0-патерн js-eslint-mechanical-text-fix — текстові заміни для suggestion-only правил поза покриттям oxlint/eslint --fix (наразі unicorn/prefer-number-is-safe-integer); зменшує обсяг порушень, що йдуть у LLM-ладдер
+
 ## [1.18.1] - 2026-07-18
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@7n/rules",
-  "version": "1.18.1",
+  "version": "1.19.0",
   "description": "CLI еталонних правил і skills (префікс n-): синк у репозиторій, дельта-lint, конформність",
   "keywords": [
     "cli",

package/rules/js/eslint/docs/fix-eslint.md

@@ -3,30 +3,48 @@ type: JS Module
 title: fix-eslint.mjs
 resource: npm/rules/js/eslint/fix-eslint.mjs
 docgen:
-  crc: e354d939
+  crc: 31afb96e
+  model: manual
 ---
 
 ## Огляд
 
-T0-autofix для `js/eslint`: детермінований прогін лінтерів у fix-режимі
-(`oxlint --fix` + `eslint --fix`) по файлах, де детектор знайшов порушення. Виправляє лише
-авто-fixable правила; усе інше лишається детектору на повторну перевірку, а далі — LLM-ладдеру.
-Запис незворотний (поза rollback).
+T0-autofix для `js/eslint`, два патерни в порядку застосування. `js-eslint-autofix`:
+детермінований прогін лінтерів у fix-режимі (`oxlint --fix` + `eslint --fix`) по файлах, де
+детектор знайшов порушення — виправляє лише авто-fixable правила. `js-eslint-mechanical-text-fix`:
+для правил, які `--fix` НЕ покриває (suggestion-only у власній реалізації інструментів, перевірено
+емпірично на реальних lint-прогонах), але заміна на позначеному `data.line` текстуально однозначна
+й безпечна без AST-парсингу. Усе, що лишається непокритим — детектору на повторну перевірку, а
+далі LLM-ладдеру. Запис незворотний (поза rollback).
 
 ## Поведінка
 
+`js-eslint-autofix`:
 - Збирає унікальні файли з порушень концерну, лишає лише js-сімейство (`.mjs/.cjs/.js/.jsx/.ts/.tsx/.vue`).
 - Послідовно застосовує `oxlint --fix` (CLI) і `eslint --fix` (через API з `outputFixes`).
 - До списку змінених потрапляють лише файли, чий вміст фактично змінився (порівняння до/після).
 
+`js-eslint-mechanical-text-fix`:
+- `MECHANICAL_TEXT_FIXES` — реєстр `{ reasons, replace }`: `reasons` покриває обидва формати
+  `violation.reason` (eslint `"plugin/rule"`, oxlint `"plugin(rule)"` — той самий rule різні тули
+  віддають по-різному); `replace(line)` — чиста функція рядка, `null` якщо шаблон не знайдено.
+- Групує порушення за файлом, для кожного цільового `data.line` (1-indexed) застосовує
+  відповідну `replace`; рядок без очікуваного шаблону (файл змінився з моменту detect-у) —
+  пропускається без гадання.
+- Наразі покриває `unicorn/prefer-number-is-safe-integer` (`Number.isInteger` → `Number.isSafeInteger`).
+
 ## Публічний API
 
-- `patterns` — масив T0-патернів (`id` / `test` / `apply`), що його споживає центральний fix-pipeline.
-  Єдиний патерн `js-eslint-autofix` спрацьовує, якщо є порушення з прив'язкою до файлу.
+- `patterns` — масив T0-патернів (`id` / `test` / `apply`), що його споживає центральний fix-pipeline:
+  `js-eslint-autofix` (лінтери в fix-режимі), `js-eslint-mechanical-text-fix` (текстові заміни для
+  suggestion-only правил).
 
 ## Гарантії поведінки
 
-- Якщо серед порушень немає js-файлів — лінтери не запускаються, змін немає.
-- Повертаються лише фактично змінені файли; перед записом кожен реєструється через `recordWrite`.
+- `js-eslint-autofix`: якщо серед порушень немає js-файлів — лінтери не запускаються, змін немає.
+- `js-eslint-mechanical-text-fix`: пише лише файли з реальною зміною рядка (шаблон збігся); не
+  чіпає рядки поза `MECHANICAL_TEXT_FIXES`-реєстром.
+- Обидва патерни повертають лише фактично змінені файли; перед записом кожен реєструється через
+  `recordWrite`.
 - Межа CI: модуль належить fix-фазі; у `--no-fix` (CI) не запускається — узгоджено із
   забороною `oxlint --fix`/`eslint --fix` у CI.

package/rules/js/eslint/fix-eslint.mjs

@@ -11,7 +11,7 @@
  * узгоджено із забороною `oxlint --fix`/`eslint --fix` у CI (js.mdc / lint_js_yml).
  */
 import { spawnSync } from 'node:child_process'
-import { readFileSync } from 'node:fs'
+import { readFileSync, writeFileSync } from 'node:fs'
 import { resolve } from 'node:path'
 
 import { ESLint } from 'eslint'
@@ -58,6 +58,59 @@ async function runLinterFix(jsFiles, cwd) {
   return touched
 }
 
+/**
+ * Реєстр "механічних" правил `js/eslint`: `oxlint --fix`/`eslint --fix` (T0
+ * `js-eslint-autofix` вище) їх НЕ покриває (suggestion-only у власній реалізації
+ * інструментів — перевірено емпірично: лишаються порушеними після `--fix` на
+ * реальних прогонах), але текстуальна заміна на позначеному рядку однозначна й
+ * безпечна без повного AST-парсингу — проста заміна ідентифікатора/API-виклику.
+ * `reasons` — обидва формати `reason` з `main.mjs` (`f.rule`): eslint (`ruleId`,
+ * `"plugin/rule"`) і oxlint (`d.code`, `"plugin(rule)"`) — те саме правило різні
+ * тули віддають по-різному.
+ * @type {Array<{ reasons: string[], replace: (line: string) => string|null }>}
+ */
+const MECHANICAL_TEXT_FIXES = [
+  {
+    // Number.isInteger(x) не ловить x поза Number.MIN/MAX_SAFE_INTEGER — сама заміна
+    // імені методу семантично точна для рядка, що ЛИШЕ так і викликає isInteger.
+    reasons: ['unicorn/prefer-number-is-safe-integer', 'unicorn(prefer-number-is-safe-integer)'],
+    replace: line =>
+      line.includes('Number.isInteger') ? line.replaceAll('Number.isInteger', 'Number.isSafeInteger') : null
+  }
+]
+
+/**
+ * Знаходить механічний фікс для reason-у порушення (обидва формати tool-у).
+ * @param {string} reason `violation.reason`
+ * @returns {((line: string) => string|null)|null} replace-функція або null
+ */
+function mechanicalFixFor(reason) {
+  return MECHANICAL_TEXT_FIXES.find(f => f.reasons.includes(reason))?.replace ?? null
+}
+
+/**
+ * Застосовує механічні текстові заміни по конкретних рядках (`data.line`, 1-indexed)
+ * файлу. Рядок без збігу очікуваного шаблону (файл змінився з моменту detect-у) —
+ * пропускається, не гадаємо.
+ * @param {string} content вміст файлу
+ * @param {Array<{ line: number, replace: (line: string) => string|null }>} targets рядки й replace-функції
+ * @returns {string|null} новий вміст або null, якщо жоден рядок не змінився
+ */
+function applyMechanicalLineFixes(content, targets) {
+  const lines = content.split('\n')
+  let changed = false
+  for (const { line, replace } of targets) {
+    const idx = line - 1
+    if (idx < 0 || idx >= lines.length) continue
+    const next = replace(lines[idx])
+    if (next !== null && next !== lines[idx]) {
+      lines[idx] = next
+      changed = true
+    }
+  }
+  return changed ? lines.join('\n') : null
+}
+
 /** @type {import('../../../scripts/lib/lint-surface/types.mjs').T0Pattern[]} */
 export const patterns = [
   {
@@ -73,5 +126,35 @@ export const patterns = [
         ? { touchedFiles, message: `oxlint/eslint --fix: ${touchedFiles.length} файл(ів)` }
         : { touchedFiles: [] }
     }
+  },
+  {
+    id: 'js-eslint-mechanical-text-fix',
+    test: violations => violations.some(v => v.file && v.data?.line && mechanicalFixFor(v.reason)),
+    apply: (violations, ctx) => {
+      /** @type {Map<string, Array<{ line: number, replace: (line: string) => string|null }>>} */
+      const byFile = new Map()
+      for (const v of violations) {
+        const replace = v.file && v.data?.line ? mechanicalFixFor(v.reason) : null
+        if (!replace) continue
+        const arr = byFile.get(v.file)
+        const target = { line: v.data.line, replace }
+        if (arr) arr.push(target)
+        else byFile.set(v.file, [target])
+      }
+      const touchedFiles = []
+      for (const [rel, targets] of byFile) {
+        const abs = resolve(ctx.cwd, rel)
+        const content = readOrNull(abs)
+        if (content === null) continue
+        const next = applyMechanicalLineFixes(content, targets)
+        if (next === null) continue
+        ctx.recordWrite?.(abs)
+        writeFileSync(abs, next)
+        touchedFiles.push(abs)
+      }
+      return touchedFiles.length > 0
+        ? { touchedFiles, message: `механічні текстові заміни: ${touchedFiles.length} файл(ів)` }
+        : { touchedFiles: [] }
+    }
   }
 ]