@nitra/cursor 12.16.1 → 12.17.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [12.17.0] - 2026-06-27
+
+### Changed
+
+- Перехід на globby для обходу файлової системи з урахуванням .gitignore
+
+## [12.16.2] - 2026-06-27
+
+### Changed
+
+- feat(orchestrator): skip non-actionable violation (не годувати агента tool-crash); feat(pi-agent-fix): логувати вхід LLM у trace (violation + розмір промпта)
+
 ## [12.16.1] - 2026-06-26
 
 ### Changed

package/lib/pi-agent-fix.mjs

@@ -213,12 +213,11 @@ export async function runPiAgentFix(ruleId, violation, cwd, opts = {}) {
     }
   })
 
+  const fixPrompt = buildFixPrompt({ ruleId, violation, ruleText, feedback })
   const startedAt = clock()
   let error = null
   try {
-    await withTimeout(session.prompt(buildFixPrompt({ ruleId, violation, ruleText, feedback })), timeoutMs, () =>
-      session.abort?.()
-    )
+    await withTimeout(session.prompt(fixPrompt), timeoutMs, () => session.abort?.())
   } catch (e) {
     error = e.message
   }
@@ -244,9 +243,17 @@ export async function runPiAgentFix(ruleId, violation, cwd, opts = {}) {
     rung: tier,
     model: modelSpec,
     cwd,
+    // ВХІД LLM (щоб «що подали» було видно у trace, без розкопок escalation-log):
+    // violation — обрізаний (може бути великим); promptChars — повний розмір промпта.
+    violation: typeof violation === 'string' ? violation.slice(0, 4000) : null,
+    violationChars: typeof violation === 'string' ? violation.length : 0,
+    promptChars: fixPrompt.length,
+    // вихід:
     turnCount,
     toolCallCount,
+    touchedFiles,
     backstopHit,
+    wallMs: clock() - startedAt,
     error
   })
   return { applied: touchedFiles.length > 0, touchedFiles, telemetry, error, rollback: guard.rollback }

package/package.json

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

package/scripts/auto-rules.mjs

@@ -16,10 +16,13 @@
  * їх у конфіг із поправкою на legacy-id (`migrateRuleIds`).
  */
 import { readdirSync } from 'node:fs'
-import { readdir, readFile } from 'node:fs/promises'
+import { readFile } from 'node:fs/promises'
 import { basename, dirname, join, relative } from 'node:path'
 import { fileURLToPath } from 'node:url'
 
+import { globby } from 'globby'
+
+import { ALWAYS_IGNORE } from './utils/walkDir.mjs'
 import { globToRegex } from '../rules/npm-module/js/package_structure.mjs'
 import { textHasBunSqlImport } from '../rules/js-bun-db/lib/bun-sql-scan.mjs'
 import {
@@ -84,9 +87,30 @@ export const AUTO_RULE_DEPENDENCIES = Object.freeze(
 
 const HASURA_CONFIG_MARKER = 'metadata_directory: metadata'
 const REGO_RE = /\.rego$/iu
-const IGNORED_DIR_NAMES = new Set(['node_modules', '.git', '.next', '.turbo'])
 const DEFAULT_DISABLED_LIST = Object.freeze([])
 
+/**
+ * Збирає relative-posix шляхи дерева (файли + директорії), **поважаючи `.gitignore`** —
+ * через той самий `globby`-канон, що й `walkDir` (звідси `ALWAYS_IGNORE`). Спільне джерело
+ * для `collectRepoPaths` (Type A glob-матчинг) і `collectAutoRuleFacts` (content-факти).
+ * Раніше тут був ручний `readdir`-обхід із хардкод skip-набором, який ігнорував `.gitignore`
+ * і помилково активував правила на згенерованих артефактах (`coverage/*.png` → image-compress).
+ * @param {string} root абсолютний шлях кореня репозиторію
+ * @returns {Promise<{ files: string[], dirs: string[] }>} relative-posix шляхи файлів і директорій
+ */
+async function collectTreePaths(root) {
+  const opts = { cwd: root, gitignore: true, dot: true, ignore: ALWAYS_IGNORE }
+  try {
+    const [files, dirs] = await Promise.all([
+      globby('**/*', { ...opts, onlyFiles: true }),
+      globby('**/*', { ...opts, onlyDirectories: true })
+    ])
+    return { files, dirs }
+  } catch {
+    return { files: [], dirs: [] }
+  }
+}
+
 /**
  * Чи містить текст джерела імпорт імені `sql` або `SQL` з `"bun"` (після витягування `<script>` у `.vue`).
  * @param {string} content вміст файлу
@@ -224,35 +248,13 @@ export async function collectAutoRuleFacts(root) {
     hasTempoDir: false
   }
 
-  /**
-   * Рекурсивний обхід каталогу з пропуском службових директорій.
-   * @param {string} dir абсолютний шлях каталогу
-   * @returns {Promise<void>}
-   */
-  async function walk(dir) {
-    let entries
-    try {
-      entries = await readdir(dir, { withFileTypes: true })
-    } catch {
-      return
-    }
-
-    for (const entry of entries) {
-      const absPath = join(dir, entry.name)
-      if (entry.isDirectory()) {
-        if (!IGNORED_DIR_NAMES.has(entry.name)) {
-          if (entry.name === 'tempo') {
-            facts.hasTempoDir = true
-          }
-          await walk(absPath)
-        }
-      } else if (entry.isFile()) {
-        await processFileEntry(absPath, root, facts)
-      }
-    }
+  const { files, dirs } = await collectTreePaths(root)
+  if (dirs.some(d => basename(d) === 'tempo')) {
+    facts.hasTempoDir = true
+  }
+  for (const rel of files) {
+    await processFileEntry(join(root, rel), root, facts)
   }
-
-  await walk(root)
   return facts
 }
 
@@ -266,34 +268,8 @@ export async function collectAutoRuleFacts(root) {
  * @returns {Promise<string[]>} шляхи відносно root у posix-форматі
  */
 async function collectRepoPaths(root) {
-  /** @type {string[]} */
-  const out = []
-  /**
-   * Рекурсивний обхід каталогу з пропуском службових директорій.
-   * @param {string} dir каталог
-   * @returns {Promise<void>}
-   */
-  async function walk(dir) {
-    let entries
-    try {
-      entries = await readdir(dir, { withFileTypes: true })
-    } catch {
-      return
-    }
-    for (const entry of entries) {
-      const abs = join(dir, entry.name)
-      if (entry.isDirectory()) {
-        if (!IGNORED_DIR_NAMES.has(entry.name)) {
-          out.push(relative(root, abs).split('\\').join('/'))
-          await walk(abs)
-        }
-      } else if (entry.isFile()) {
-        out.push(relative(root, abs).split('\\').join('/'))
-      }
-    }
-  }
-  await walk(root)
-  return out
+  const { files, dirs } = await collectTreePaths(root)
+  return [...dirs, ...files]
 }
 
 /**

package/scripts/lib/fix/orchestrator.mjs

@@ -51,6 +51,19 @@ export function classifyFixError(error) {
   return 'quality'
 }
 
+/**
+ * Чи violation придатний для LLM-фіксу: містить хоч одне actionable `❌`-порушення
+ * (формат конформ-чека `❌ <file>: <інструкція>`). Без жодного ❌ — це не список
+ * фіксабельних порушень, а шум/збій тула (Usage, «лок взято», порожньо). Годувати
+ * таким агента = марні turns/timeout (вимір 7n-test: doc-files surface-ив Usage
+ * downstream-тула → агент флоундерив 4 рунги). Тоді LLM-фікс пропускаємо.
+ * @param {string|null|undefined} output violation-вивід правила
+ * @returns {boolean} true — є що фіксити
+ */
+export function hasActionableViolation(output) {
+  return /❌/u.test(output ?? '')
+}
+
 /**
  * Будує драбину ескалації за наявними тирами (спека 2026-06-19-fix-escalation-cascade):
  *  1. `local-min`       — `N_LOCAL_MIN_MODEL`, перший прохід;
@@ -124,6 +137,27 @@ export async function escalateRule(rule, cwd, deps) {
   const skipModels = new Set()
   let avgUsed = 0
 
+  // §2-профілактика: violation без actionable ❌ (tool-crash/Usage/шум) → не годуємо
+  // агента (інакше флоундерить рунги до timeout). Рапортуємо як non-actionable, не фіксимо.
+  if (!hasActionableViolation(rule.output)) {
+    record({
+      rung: -1,
+      tier: 'skip',
+      model: '',
+      withFeedback: false,
+      callOk: false,
+      callError: 'non-actionable violation (нема ❌ — ймовірно check-error/tool-crash)',
+      recheckOk: false,
+      remainingViolation: rule.output,
+      diagnosis: null,
+      ms: 0
+    })
+    log(
+      `  ⏭️  ${rule.ruleId}: LLM-фікс пропущено — у violation немає ❌-порушень (check-error/tool-crash, не фіксабельне)`
+    )
+    return { resolved: false, avgUsed: 0 }
+  }
+
   for (const [idx, rung] of ladder.entries()) {
     if (skipModels.has(rung.model)) continue
 

package/scripts/lib/rule-predicates.mjs

@@ -8,27 +8,23 @@
  * Сигнатури неоднорідні (одні беруть `facts`, інші — `cwd`/`packageJson`), бо предикати
  * читають різні джерела; виклик диспетчиться в `auto-rules.mjs` за іменем предиката.
  */
-import { readdir, readFile } from 'node:fs/promises'
+import { readFile } from 'node:fs/promises'
 import { join } from 'node:path'
 
+import { findAllPackageJsonPaths } from '../utils/find-package-json-paths.mjs'
 import { getRepositoryUrl } from './rule-meta-helpers.mjs'
 
-const IGNORED_DIR_NAMES = new Set(['node_modules', '.git', '.next', '.turbo'])
-
 /**
  * Чи package.json дерева містить будь-який із зазначених пакетів у dependencies.
+ * Обхід — через `findAllPackageJsonPaths` (на `walkDir`/`globby`), тож **поважає `.gitignore`**
+ * і не зчитує package.json з ігнорованих каталогів (build-артефакти, vendored-копії).
  * @param {string} root корінь репо
  * @param {string[]} keys імена пакетів
  * @returns {Promise<boolean>} true, якщо знайдено хоч один
  */
-function anyDepInTree(root, keys) {
+async function anyDepInTree(root, keys) {
   const wanted = new Set(keys)
-  /**
-   * Чи package.json за `abs` оголошує будь-який пакет із `wanted` у `dependencies`.
-   * @param {string} abs шлях до package.json
-   * @returns {Promise<boolean>} true, якщо знайдено хоч один
-   */
-  async function pkgDeclaresWanted(abs) {
+  for (const abs of await findAllPackageJsonPaths(root, [])) {
     try {
       const deps = JSON.parse(await readFile(abs, 'utf8'))?.dependencies
       if (deps && typeof deps === 'object' && !Array.isArray(deps)) {
@@ -37,70 +33,29 @@ function anyDepInTree(root, keys) {
     } catch {
       /* ігноруємо пошкоджені package.json */
     }
-    return false
   }
-  /**
-   * @param {string} dir каталог обходу
-   * @returns {Promise<boolean>} true, якщо знайдено хоч один пакет
-   */
-  async function walk(dir) {
-    let entries
-    try {
-      entries = await readdir(dir, { withFileTypes: true })
-    } catch {
-      return false
-    }
-    for (const entry of entries) {
-      const abs = join(dir, entry.name)
-      if (entry.isDirectory()) {
-        if (!IGNORED_DIR_NAMES.has(entry.name) && (await walk(abs))) return true
-      } else if (entry.isFile() && entry.name === 'package.json' && (await pkgDeclaresWanted(abs))) {
-        return true
-      }
-    }
-    return false
-  }
-  return walk(root)
+  return false
 }
 
 /**
  * Чи існує вкладений (не кореневий) package.json без `vite` у devDependencies.
+ * Обхід — `findAllPackageJsonPaths` (gitignore-aware), як у `anyDepInTree`.
  * @param {string} root корінь репо
  * @returns {Promise<boolean>} true, якщо знайдено
  */
 async function nestedWithoutVite(root) {
   const rootPkg = join(root, 'package.json')
-  let result = false
-  /**
-   * @param {string} dir каталог
-   * @returns {Promise<void>}
-   */
-  async function walk(dir) {
-    if (result) return
-    let entries
+  for (const abs of await findAllPackageJsonPaths(root, [])) {
+    if (abs === rootPkg) continue
     try {
-      entries = await readdir(dir, { withFileTypes: true })
+      const dev = JSON.parse(await readFile(abs, 'utf8'))?.devDependencies
+      const hasVite = dev && typeof dev === 'object' && !Array.isArray(dev) && Object.hasOwn(dev, 'vite')
+      if (!hasVite) return true
     } catch {
-      return
-    }
-    for (const entry of entries) {
-      if (result) return
-      const abs = join(dir, entry.name)
-      if (entry.isDirectory()) {
-        if (!IGNORED_DIR_NAMES.has(entry.name)) await walk(abs)
-      } else if (entry.isFile() && entry.name === 'package.json' && abs !== rootPkg) {
-        try {
-          const dev = JSON.parse(await readFile(abs, 'utf8'))?.devDependencies
-          const hasVite = dev && typeof dev === 'object' && !Array.isArray(dev) && Object.hasOwn(dev, 'vite')
-          if (!hasVite) result = true
-        } catch {
-          /* пошкоджений package.json не вважаємо vite-проєктом */
-        }
-      }
+      /* пошкоджений package.json не вважаємо vite-проєктом */
     }
   }
-  await walk(root)
-  return result
+  return false
 }
 
 /** Реєстр предикатів: імʼя → реалізація. Виклик за `meta.json.auto.predicate`. */

package/scripts/utils/walkDir.mjs

@@ -4,7 +4,7 @@ import { globby } from 'globby'
 
 // .git ніколи не потрапляє в .gitignore — пропускаємо завжди.
 // node_modules — safety net: проєкт може не мати .gitignore або запускатись поза git-репо.
-const ALWAYS_IGNORE = ['.git/**', 'node_modules/**']
+export const ALWAYS_IGNORE = ['.git/**', 'node_modules/**']
 
 /**
  * Рекурсивно обходить каталог, поважаючи .gitignore (включно з вкладеними).