@nitra/cursor 12.16.2 → 12.17.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [12.17.0] - 2026-06-27
+
+### Changed
+
+- Перехід на globby для обходу файлової системи з урахуванням .gitignore
+
 ## [12.16.2] - 2026-06-27
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nitra/cursor",
-  "version": "12.16.2",
+  "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/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 (включно з вкладеними).