@nitra/cursor 1.11.1 → 1.11.3

package/CHANGELOG.md

@@ -4,6 +4,18 @@
 
 Формат — [Keep a Changelog](https://keepachangelog.com/uk/1.1.0/), нумерація — [SemVer](https://semver.org/lang/uk/).
 
+## [1.11.3] - 2026-05-15
+
+### Fixed
+
+- **`npm/bin/n-cursor.js`** — `detectAutoSkills` тепер отримує **ефективний** список правил (опт-ін вручну з `.n-cursor.json:rules` ∪ auto-detected, мінус `disable-rules`), а не лише auto-detected. Без цього скіли із залежністю на правило, додане вручну (наприклад, `adr` без `auto.md`-умови), не активувалися — у репо з `"rules": ["adr", …]` скіл `adr-normalize` залишався відсутнім, попри `[adr]` у його `skills/adr-normalize/auto.md`. Тепер `adr-normalize`, `abie-clean`, `abie-kustomize`, `taze` авто-додаються коректно як при auto-detected, так і при manual-opt-in відповідних правил.
+
+## [1.11.2] - 2026-05-15
+
+### Fixed
+
+- **`npm/scripts/auto-skills.mjs`** — джерело правди для автоактивації скілів тепер `skills/<skill>/auto.md`, а не hardcoded мапа в JS. Парсер розпізнає три формати: `завжди` (always-on), `[rule, rule, …]` (умова на правила), відсутній/нерозпізнаний файл (opt-in). Експортовані константи `AUTO_SKILL_ORDER` та `AUTO_SKILL_RULE_DEPENDENCIES` тепер похідні від сканування `npm/skills/` під час завантаження модуля (зберігаються для зворотної сумісності). Побічно виправлено пропуск `abie-clean` у hardcoded мапі попри `[abie]` у його `auto.md` — тепер скіл коректно автоактивується разом з правилом `abie`.
+
 ## [1.11.1] - 2026-05-15
 
 ### Fixed
@@ -45,9 +57,6 @@
   - **Walk-glob правила** (6): `js-mssql`, `js-bun-db`, `js-bun-redis`, `js-run` (package_json + configmap), `vue`, `image-avif` — `walkGlob: "**/package.json"` або відповідний патерн.
   - **k8s.* концерни** (8): `manifest`, `gateway`, `hpa_pdb`, `kustomization`, `svc_yaml`, `svc_hl_yaml`, `base_kustomization`, `base_manifest` — `walkGlob` по YAML під сегментом `k8s/`; `base_manifest` використовує негативний glob для виключення `kustomization.yaml`.
   - **abie концерни** (4): `clean_merged_ignore_branches` (single), `health_check_policy` (walkGlob `**/k8s/**/hc.yaml`), `http_route_base` (walkGlob `**/k8s/**/base/**/hr.yaml`), `base_deployment_preem` (walkGlob `**/k8s/**/base/**/*.{yaml,yml}` з виключенням `kustomization.yaml`).
-
-### Changed
-
 - **`capture-decisions.sh` тепер пише чернетки напряму в `docs/adr/<timestamp>-<sid>.md`** (раніше — у `docs/adr/_inbox/`). Сам каталог `_inbox/` більше не створюється, але `normalize-decisions.sh` бачить його рекурсивно — старі чернетки з `_inbox/` поступово розчищаються нормалізацією. Можна також одноразово `git mv docs/adr/_inbox/*.md docs/adr/` і прибрати порожній каталог.
 - **Правило `adr` (`npm/rules/adr/adr.mdc`)**: повне переписування під дві фази (capture + normalize). Видалено згадки `_inbox/`. Версія `version: '2.0'`.
 - **`npm/rules/adr/js/check.mjs`**: перевірка обох hook-скриптів (canonicity), обох log-файлів у `.gitignore`.

package/bin/n-cursor.js

@@ -261,9 +261,16 @@ async function readConfig(paths = {}) {
       packageJsonParsed: rootPkg,
       disableRules
     })
+    // Skills залежать від ефективного списку правил, який буде у конфізі після merge:
+    // вже існуючі (опт-ін вручну) + auto-detected, мінус `disable-rules`. Без цього
+    // правило, додане вручну (напр. `adr` без auto.md-умови), не активувало б залежні
+    // скіли (`adr-normalize`).
+    const disableRulesSet = new Set(disableRules)
+    const effectiveRulesForSkills = [...new Set([...normalizeIdList(parsedConfig.rules), ...autoDetectedRules.rules])]
+      .filter(id => !disableRulesSet.has(id))
     const autoDetectedSkills = detectAutoSkills({
       availableSkills,
-      detectedRules: autoDetectedRules.rules,
+      detectedRules: effectiveRulesForSkills,
       disableSkills
     })
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nitra/cursor",
-  "version": "1.11.1",
+  "version": "1.11.3",
   "description": "CLI для завантаження cursor-правил (префікс n-) у локальний репозиторій",
   "keywords": [
     "cli",

package/scripts/auto-skills.mjs

@@ -1,81 +1,122 @@
 /**
- * Автовизначення skills для `.n-cursor.json` за умовами з `npm/skills/<skill>/auto.md`.
+ * Автовизначення skills для `.n-cursor.json` за умовами зі `npm/skills/<skill>/auto.md`.
  *
- * Скіли автододаються залежно від уже виявлених правил (auto-rules) — щоб не дублювати
- * умови, які вже формалізовані для відповідного правила. Наприклад:
+ * `auto.md` — джерело правди (а не hardcoded мапа). Підтримуються три варіанти:
  *
- * - `abie-kustomize - [abie]` — додається разом з правилом `abie`
- * - `taze - [bun]` — додається разом з правилом `bun`
+ *  - `завжди` — скіл активується незалежно від інших правил
+ *    (приклади: `fix`, `lint`, `llm-patch`, `publish-telegram`).
+ *  - `[rule, rule, …]` — скіл активується, якщо ВСІ перелічені правила вже виявлені
+ *    auto-rules (приклади: `abie-clean - [abie]`, `taze - [bun]`).
+ *  - файл відсутній або формат не розпізнано — скіл opt-in лише через `.n-cursor.json:skills`.
  *
- * Скіли без секції `[rules]` у `skills/<skill>/auto.md` (`fix`, `lint`, `llm-patch`, `publish-telegram`)
- * додаються завжди, якщо доступні в пакеті й не у `disable-skills`.
+ * Сканування `npm/skills/` — sync під час завантаження модуля (детермінізм + sync API
+ * `auto-rules.mjs`-сусіда). Кеш на час процесу.
  */
+import { existsSync, readdirSync, readFileSync } from 'node:fs'
+import { dirname, join } from 'node:path'
+import { fileURLToPath } from 'node:url'
 
-/** Порядок автододавання skills відповідно до `skills/<skill>/auto.md`. */
-export const AUTO_SKILL_ORDER = Object.freeze([
-  'abie-kustomize',
-  'adr-normalize',
-  'fix',
-  'lint',
-  'llm-patch',
-  'publish-telegram',
-  'taze'
-])
+const PACKAGE_ROOT = dirname(dirname(fileURLToPath(import.meta.url)))
+const SKILLS_DIR = join(PACKAGE_ROOT, 'skills')
+
+const ALWAYS_LITERAL = 'завжди'
+const BRACKET_LIST_RE = /^\[([^\]]+)\]$/u
 
 /**
- * Залежність скілів від правил (`skills/<skill>/auto.md` синтаксис `skill - [rules]`).
- * Ключ варто автододати, коли всі правила-залежності вже додані до конфігу автодетектом.
+ * @typedef {{ always: true } | { rules: readonly string[] }} SkillAutoSpec
  */
-export const AUTO_SKILL_RULE_DEPENDENCIES = Object.freeze(
-  /** @type {Record<string, readonly string[]>} */ ({
-    'abie-kustomize': Object.freeze(['abie']),
-    'adr-normalize': Object.freeze(['adr']),
-    taze: Object.freeze(['bun'])
-  })
+
+/**
+ * Парсить тіло `auto.md` одного скіла.
+ * @param {string} text вміст файла (без `trim`)
+ * @returns {SkillAutoSpec | null} `null` — формат не розпізнано (= opt-in)
+ */
+function parseSkillAutoSpec(text) {
+  const trimmed = text.trim()
+  if (trimmed === ALWAYS_LITERAL) {
+    return { always: true }
+  }
+  const m = trimmed.match(BRACKET_LIST_RE)
+  if (m) {
+    const rules = m[1]
+      .split(',')
+      .map(s => s.trim())
+      .filter(s => s.length > 0)
+    if (rules.length === 0) return null
+    return { rules: Object.freeze(rules) }
+  }
+  return null
+}
+
+/**
+ * Сканує `npm/skills/<id>/auto.md`. Скіли без `auto.md` або з нерозпізнаним
+ * вмістом не потрапляють у результат — їх можна вмикати лише вручну в конфізі.
+ * @param {string} [skillsDir] override для тестів
+ * @returns {Record<string, SkillAutoSpec>}
+ */
+export function discoverSkillAutoActivation(skillsDir = SKILLS_DIR) {
+  if (!existsSync(skillsDir)) return {}
+  /** @type {Record<string, SkillAutoSpec>} */
+  const out = {}
+  for (const entry of readdirSync(skillsDir, { withFileTypes: true })) {
+    if (!entry.isDirectory() || entry.name.startsWith('.')) continue
+    const autoMdPath = join(skillsDir, entry.name, 'auto.md')
+    if (!existsSync(autoMdPath)) continue
+    const spec = parseSkillAutoSpec(readFileSync(autoMdPath, 'utf8'))
+    if (spec) out[entry.name] = spec
+  }
+  return out
+}
+
+/** Cache на час процесу: один скан `npm/skills/` дає всю автоактивацію. */
+const SKILL_AUTO_ACTIVATION = discoverSkillAutoActivation()
+
+/**
+ * Стабільний алфавітний порядок скілів з автоактивацією. Експортовано для зворотної
+ * сумісності (попередня версія мала жорстко прописаний `AUTO_SKILL_ORDER`).
+ */
+export const AUTO_SKILL_ORDER = Object.freeze(
+  Object.keys(SKILL_AUTO_ACTIVATION).toSorted((a, b) => a.localeCompare(b))
 )
 
-/** Скіли без залежностей — додаються завжди (рядок «завжди» в `skills/<skill>/auto.md`). */
-const ALWAYS_ON_SKILLS = Object.freeze(['fix', 'lint', 'llm-patch', 'publish-telegram'])
+/**
+ * Похідна view на `SKILL_AUTO_ACTIVATION`: лише скіли з rule-залежностями.
+ * Експортовано для зворотної сумісності та автодоку.
+ */
+export const AUTO_SKILL_RULE_DEPENDENCIES = Object.freeze(
+  Object.fromEntries(
+    Object.entries(SKILL_AUTO_ACTIVATION)
+      .filter(([, spec]) => 'rules' in spec)
+      .map(([id, spec]) => [id, /** @type {{ rules: readonly string[] }} */ (spec).rules])
+  )
+)
 
 const DEFAULT_DISABLED_LIST = Object.freeze([])
 
 /**
- * Визначає авто-skills згідно з `skills/<skill>/auto.md`.
+ * Визначає авто-skills згідно з вмістом `skills/<skill>/auto.md`.
  * @param {object} params параметри
  * @param {string[]} params.availableSkills перелік доступних skills із пакету (id без префікса n-)
  * @param {string[]} params.detectedRules id правил, виявлених auto-rules (вхідні залежності)
  * @param {string[]} [params.disableSkills] список `disable-skills` з конфігу
- * @returns {{ skills: string[] }} список id у стабільному порядку (за `AUTO_SKILL_ORDER`)
+ * @returns {{ skills: string[] }} список id у стабільному алфавітному порядку
  */
 export function detectAutoSkills({ availableSkills, detectedRules, disableSkills = DEFAULT_DISABLED_LIST }) {
   const normalizedSkills = new Set(availableSkills.map(s => s.trim().toLowerCase()))
   const disableSkillsSet = new Set(disableSkills)
   const detectedRulesSet = new Set(detectedRules)
 
-  /** @type {string[]} */
-  const detected = []
-
-  /**
-   * Додає skill до результату, якщо він доступний і не в disable-списку.
-   * @param {string} skillId id skill
-   * @returns {void}
-   */
-  function addSkill(skillId) {
-    if (!normalizedSkills.has(skillId) || disableSkillsSet.has(skillId) || detected.includes(skillId)) {
-      return
-    }
-    detected.push(skillId)
-  }
-
-  for (const skillId of ALWAYS_ON_SKILLS) {
-    addSkill(skillId)
-  }
+  /** @type {Set<string>} */
+  const detected = new Set()
 
-  for (const [skillId, deps] of Object.entries(AUTO_SKILL_RULE_DEPENDENCIES)) {
-    if (deps.every(d => detectedRulesSet.has(d))) {
-      addSkill(skillId)
+  for (const [skillId, spec] of Object.entries(SKILL_AUTO_ACTIVATION)) {
+    if (!normalizedSkills.has(skillId) || disableSkillsSet.has(skillId)) continue
+    if ('always' in spec) {
+      detected.add(skillId)
+    } else if (spec.rules.every(d => detectedRulesSet.has(d))) {
+      detected.add(skillId)
     }
   }
 
-  return { skills: AUTO_SKILL_ORDER.filter(id => detected.includes(id)) }
+  return { skills: AUTO_SKILL_ORDER.filter(id => detected.has(id)) }
 }