@7n/rules 1.8.1 → 1.8.2

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.8.2] - 2026-07-17
+
+### Fixed
+
+- bun/licensee: розрізняти crash інструмента (stderr) від реального ліцензійного порушення (stdout, --errors-only)
+- k8s/manifests: знизити когнітивну складність ensureHasuraConfigMapRequiredEnv нижче порогу sonarjs
+
 ## [1.8.1] - 2026-07-17
 
 ### Changed

package/package.json

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

package/rules/bun/licensee/docs/main.md

@@ -3,35 +3,28 @@ type: JS Module
 title: main.mjs
 resource: npm/rules/bun/licensee/main.mjs
 docgen:
-  crc: bbaca446
-  model: omlx/gemma-4-e4b-it-OptiQ-4bit
-  score: 90
-  issues: surzhik,judge:inaccurate:0.96
-  judgeModel: openai-codex/gpt-5.4-mini
+  crc: 48c72e2b
+  model: manual
 ---
 
 ## Огляд
 
-Встановлює конфігурацію ліцензій, спираючись на файл .licensee.json, якщо він відсутній. Виконує перевірку відповідності ліцензій усіх npm-залежностей проєкту встановленим вимогам, що визначаються в конфігурації. У випадку виявлення порушень, система повідомляє про них, підтверджуючи чисту відповідність або фіксуючи розбіжності. Маркери повідомлень згадуються у Поведінці: (bun.mdc).
+Read-only detector `bun/licensee`: перевіряє ліцензії npm-залежностей проєкту через зовнішній інструмент `licensee` (`bun x licensee`), звіряючи їх з allowlist у `.licensee.json`. Розрізняє два принципово різні провали: `licensee` знайшов реально заборонену ліцензію (`license-violation`) — і `licensee` сам впав з помилкою (`licensee-crashed`), наприклад через несумісність `@npmcli/arborist` (яким він читає `node_modules`) з деревом, зібраним `bun install`.
 
 ## Поведінка
 
-Поведінка:
-
-1. Викликається `lint` для запуску перевірки ліцензій npm-залежностей.
-2. Якщо файл конфігурації `.licensee.json` відсутній:
-   a. Якщо режим читання лише (`readOnly`) увімкнено, повідомляється, що необхідно запустити `npx @7n/rules fix bun` локально для генерації (bun.mdc).
-   b. Якщо режим читання лише вимкнено, створюється `.licensee.json` з дефолтним allowlist (`blueOak: bronze`) та повідомляється про створення конфігурації.
-3. Визначається команда `bun`. Якщо `bun` не знайдена у системному шляху, повідомляється про це (bun.mdc).
-4. Виконується команда `bun x licensee --production --quiet` у корені проєкту.
-5. Якщо команда `licensee` завершується успішно (статус 0), повідомляється, що ліцензії відповідають вимогам.
-6. Якщо команда `licensee` завершується з помилкою (не 0), повідомляється про порушення ліцензій (код) (bun.mdc).
+1. Якщо `.licensee.json` відсутній — `licensee-config-missing`; генерація конфігу — окремий T0-фікс (`fix-licensee.mjs`), не тут.
+2. Якщо `bun` не знайдено в PATH — `bun-missing`.
+3. Запускає `bun x licensee --production --errors-only` (без `--quiet` — реальне порушення потребує деталі: ім'я/версія/ліцензія пакета, які `licensee` пише в stdout через `print()`).
+4. Статус 0 — без порушень.
+5. Статус ≠ 0 і є вивід у **stderr** — трактується як crash/`die()` самого `licensee` (invalid config або необроблений виняток усередині тула), НЕ як ліцензійне порушення: `licensee-crashed`, повідомлення включає stderr і підказку перевірити вручну `bunx licensee --production`.
+6. Статус ≠ 0 і stderr порожній, є вивід у **stdout** — це справжній `NOT APPROVED`-звіт від `licensee` (`--errors-only` друкує лише невідповідні пакети): `license-violation` з деталлю пакета/ліцензії у повідомленні.
 
 ## Публічний API
 
-lint — аналізує код на відповідність заданим стилістичним нормам у контексті `bun`.
-licensee — перевіряє відповідність ліцензій усіх використаних npm-залежностей файлу `.licensee.json`.
+- `lint(ctx)` — detector-контракт unified lint surface; повертає `{ violations }`.
 
 ## Гарантії поведінки
 
-- (специфічних машинно-виведених гарантій немає)
+- Read-only: не пише `.licensee.json` і не змінює `node_modules`/lockfile сам (побічний ефект `bun x` — авто-встановлення пакета `licensee`, якщо відсутній у bunx-кеші, — керується bun, не цим модулем).
+- Канал stdout/stderr — єдиний сигнал розрізнення crash vs реальне порушення; текстові евристики за вмістом повідомлення не використовуються.

package/rules/bun/licensee/main.mjs

@@ -34,11 +34,27 @@ export function lint(ctx) {
     return reporter.result()
   }
 
-  const r = spawnSync(bun, ['x', 'licensee', '--production', '--quiet'], { cwd, encoding: 'utf8', shell: false })
+  // Без --quiet: `licensee` пише реальні NOT APPROVED записи (name/version/license) у
+  // stdout через print() — деталь замість голого "код 1". Crash/die() усередині
+  // самого тула (invalid config, внутрішній виняток на кшталт "Cannot read properties
+  // of undefined (reading 'localeCompare')" — спостережено з @npmcli/arborist на
+  // bun-based node_modules) завжди йде у stderr через die(); легітимний звіт про
+  // порушення — лише у stdout. Канал розрізняє crash від реального порушення.
+  const r = spawnSync(bun, ['x', 'licensee', '--production', '--errors-only'], { cwd, encoding: 'utf8', shell: false })
   if (r.status !== 0) {
-    const out = `${r.stdout ?? ''}${r.stderr ?? ''}`.trim().slice(0, 2000)
-    const detail = out ? `\n${out}` : ''
-    fail(`lint-bun: licensee — порушення ліцензій (код ${r.status}, bun.mdc)${detail}`, 'license-violation')
+    const stderr = (r.stderr ?? '').trim().slice(0, 2000)
+    if (stderr) {
+      fail(
+        `lint-bun: licensee — інструмент завершився з помилкою, це НЕ підтверджене ліцензійне порушення ` +
+          `(код ${r.status}, bun.mdc). Ймовірна причина — несумісність @npmcli/arborist (яким licensee читає ` +
+          `node_modules) з деревом, зібраним bun install. Перевір вручну: \`bunx licensee --production\`.\n${stderr}`,
+        'licensee-crashed'
+      )
+    } else {
+      const stdout = (r.stdout ?? '').trim().slice(0, 2000)
+      const detail = stdout ? `\n${stdout}` : ''
+      fail(`lint-bun: licensee — порушення ліцензій (код ${r.status}, bun.mdc)${detail}`, 'license-violation')
+    }
   }
   return reporter.result()
 }

package/rules/k8s/manifests/docs/fix-manifests.md

@@ -3,7 +3,7 @@ type: JS Module
 title: fix-manifests.mjs
 resource: npm/rules/k8s/manifests/fix-manifests.mjs
 docgen:
-  crc: 7245cdbe
+  crc: e0c44174
 ---
 
 ## Огляд

package/rules/k8s/manifests/fix-manifests.mjs

@@ -157,6 +157,22 @@ function isFalsyBool(v) {
   return typeof v === 'string' && v.trim().toLowerCase() === 'false'
 }
 
+/**
+ * Визначає значення для одного `HASURA_GRAPHQL_*` ключа за очікуванням із
+ * `HASURA_REQUIRED_ENV_VALUES`: `null` — довільне значення, проставляється лише
+ * якщо ключ відсутній (дефолт `'true'`); `'true'`/`'false'` — булеве порівняння
+ * (case-insensitive, `isTruthyBool`/`isFalsyBool`); інакше — точний рядок.
+ * @param {string|null} expected очікуване значення з `HASURA_REQUIRED_ENV_VALUES`
+ * @param {unknown} current поточне значення ключа в `data` (може бути `undefined`)
+ * @returns {string|null} значення для `setIn`, або `null`, якщо змін не потрібно
+ */
+function resolveHasuraEnvValue(expected, current) {
+  if (expected === null) return current === undefined ? 'true' : null
+  if (expected === 'true') return isTruthyBool(current) ? null : 'true'
+  if (expected === 'false') return isFalsyBool(current) ? null : 'false'
+  return current === expected ? null : expected
+}
+
 /**
  * Проставляє обов'язкові `HASURA_GRAPHQL_*` env-ключі (`HASURA_REQUIRED_ENV_VALUES`,
  * дзеркалить `k8s.hasura_configmap.rego`) у `data` документа `kind: ConfigMap`. Ключ з
@@ -177,32 +193,10 @@ export function ensureHasuraConfigMapRequiredEnv(content) {
 
   let changed = false
   for (const [key, expected] of Object.entries(HASURA_REQUIRED_ENV_VALUES)) {
-    const current = doc.getIn(['data', key])
-    if (expected === null) {
-      if (current === undefined) {
-        doc.setIn(['data', key], 'true')
-        changed = true
-      }
-      continue
-    }
-    if (expected === 'true') {
-      if (!isTruthyBool(current)) {
-        doc.setIn(['data', key], 'true')
-        changed = true
-      }
-      continue
-    }
-    if (expected === 'false') {
-      if (!isFalsyBool(current)) {
-        doc.setIn(['data', key], 'false')
-        changed = true
-      }
-      continue
-    }
-    if (current !== expected) {
-      doc.setIn(['data', key], expected)
-      changed = true
-    }
+    const value = resolveHasuraEnvValue(expected, doc.getIn(['data', key]))
+    if (value === null) continue
+    doc.setIn(['data', key], value)
+    changed = true
   }
   if (!changed) return null
   return doc.toString()