@7n/rules 1.18.0 → 1.19.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # 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
+
+- `resolveChangedBase` стійкий до stale-ref: merge-base рахується для обох кандидатів (`origin/main` і локальна `main`) і береться новіший (descendant) — у git-worktree застрягла локальна `main` більше не дає застарілу базу з фантомними «зміненими» файлами давно влитих PR (T0-фіксер створював фейкові change-фрагменти для незачеплених воркспейсів); фолбек на єдиний доступний ref офлайн/без remote збережено
+
 ## [1.18.0] - 2026-07-18
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@7n/rules",
-  "version": "1.18.0",
+  "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: [] }
+    }
   }
 ]

package/scripts/lib/changed-files.mjs

@@ -47,21 +47,32 @@ function dropWorktreeCheckouts(paths) {
 
 /**
  * Визначає git base для scoped-перевірок без зовнішнього runtime-стану.
- * Пріоритет: локальна `main`, потім `origin/main`; якщо обидві відсутні,
- * повертає null і caller порівнює лише робоче дерево з HEAD.
- * Явний `baseRef` (CI: `--base origin/main` після fetch) вимикає каскад —
+ * Кандидати — `origin/main` і локальна `main`: рахуємо merge-base з HEAD для обох
+ * і беремо **новіший** (descendant) з двох. Це захищає від stale-ref з будь-якого
+ * боку: у git-worktree локальна `main` закріплена за іншим деревом і часто відстає
+ * (застаріла база → фантомні «змінені» файли з давно влитих PR), а без свіжого
+ * fetch може відставати вже `origin/main`. Якщо доступний лише один ref (офлайн,
+ * без remote) — його merge-base; якщо жодного — null, і caller порівнює лише
+ * робоче дерево з HEAD. Повернений sha завжди досяжний (це merge-base існуючого
+ * ref), тож fail-closed перевірка в `collectChangedFilesSince` не спрацює хибно.
+ * Явний `baseRef` (CI: `--base origin/main` після fetch) вимикає вибір —
  * merge-base рахується лише проти нього.
  * @param {string} [cwd] корінь репо
- * @param {string|null} [baseRef] явний ref бази замість каскаду main→origin/main
+ * @param {string|null} [baseRef] явний ref бази замість вибору origin/main|main
  * @returns {string|null} merge-base commit або null
  */
 export function resolveChangedBase(cwd = process.cwd(), baseRef = null) {
-  for (const ref of baseRef ? [baseRef] : ['main', 'origin/main']) {
+  const mergeBaseWith = ref => {
     const result = spawnSync('git', ['merge-base', 'HEAD', ref], { cwd, encoding: 'utf8' })
-    const base = result.status === 0 && !result.error ? result.stdout.trim() : ''
-    if (base) return base
+    return result.status === 0 && !result.error ? result.stdout.trim() : ''
   }
-  return null
+  if (baseRef) return mergeBaseWith(baseRef) || null
+  const [primary, secondary] = ['origin/main', 'main'].map(ref => mergeBaseWith(ref)).filter(Boolean)
+  if (!primary || !secondary || primary === secondary) return primary ?? null
+  // Обидва ref-и дали різні merge-base: новіший — той, що є нащадком іншого.
+  // Якщо гілки merge-base розійшлися (екзотика), лишаємо пріоритет origin/main.
+  const ancestry = spawnSync('git', ['merge-base', '--is-ancestor', primary, secondary], { cwd })
+  return ancestry.status === 0 && !ancestry.error ? secondary : primary
 }
 
 /**

package/scripts/lib/docs/changed-files.md

@@ -3,7 +3,7 @@ type: JS Module
 title: changed-files.mjs
 resource: npm/scripts/lib/changed-files.mjs
 docgen:
-  crc: 254d852a
+  crc: 1c7d9027
   score: 100
 ---
 
@@ -19,8 +19,9 @@ collectChangedFilesSince Збирає змінені та незакомічен
 
 - collectChangedFiles: Збирає список змінених та невідстежених файлів з робочого дерева відносно робочого дерева.
 - resolveChangedBase: Визначає базовий комміт для обмежених перевірок без використання зовнішнього стану.
-  - Пріоритет: локальна `main`, потім `origin/main`; явний другий аргумент `baseRef` (CI: `--base origin/main`) вимикає каскад — merge-base рахується лише проти нього.
-  - Якщо ref-и відсутні, повертає null, і викликаючий порівнює лише робоче дерево з HEAD.
+  - Кандидати — `origin/main` і локальна `main`: merge-base з HEAD рахується для обох, береться новіший (descendant). Це захищає від stale-ref: у git-worktree локальна `main` часто відстає, без свіжого fetch може відставати `origin/main`.
+  - Явний другий аргумент `baseRef` (CI: `--base origin/main`) вимикає вибір — merge-base рахується лише проти нього.
+  - Якщо доступний лише один ref (офлайн/без remote) — його merge-base; якщо ref-и відсутні, повертає null, і викликаючий порівнює лише робоче дерево з HEAD.
 - collectChangedFilesSince: Збирає список змінених та невідстежених файлів відносно базового комміту.
 - git diff <base>: Порівнює комміт `base` з поточним робочим деревом (включаючи закомічені та незакомічені модифікації).
   - Якщо `base` відсутній, використовується `collectChangedFiles`.