@nitra/cursor 1.27.9 → 1.28.1

package/CHANGELOG.md

@@ -4,6 +4,31 @@
 
 Формат — [Keep a Changelog](https://keepachangelog.com/uk/1.1.0/), нумерація — [SemVer](https://semver.org/lang/uk/).
 
+## [1.28.0] - 2026-05-27
+
+### BREAKING
+
+- **`scripts/utils/test-helpers.mjs#withTmpCwd` видалено**. Замість нього — `withTmpDir(async dir => …)`, що **НЕ** мутує `process.cwd()`, а передає абсолютний шлях `dir` у callback. Усі callers (43 тестових файли у пакеті) переписано: `await writeJson(join(dir, …), …)`, `await ensureDir(join(dir, …))`, `execFile('git', […], { cwd: dir })`, `await check(dir)`. `writeJson` і `ensureDir` тепер вимагають абсолютних шляхів (кидають помилку на relative). Споживачі пакета — мігруйте за конвенцією з `test.mdc` (секція "Заборона `process.chdir` у тестах").
+- **Production-API: 24 `check()`/`applies()` JS concerns приймають `cwd = process.cwd()` параметром** у `rules/{abie,adr,bun,capacitor,changelog,docker,ga,graphql,hasura,image-avif,image-compress,js-bun-db,js-lint,js-run,k8s,nginx-default-tpl,npm-module,rego,rust,security,style-lint,test,text,vue}/js/*.mjs`. Default зберігає CLI-сумісність (runner викликає без аргументів — default `process.cwd()` спрацює).
+
+### Fixed
+
+- **Race у `process.cwd()` між паралельними vitest workers** (root cause). У default `pool: 'threads'` усі workers ділять один процес. Паралельні `withTmpCwd` ламали один одному cwd: `git init`+`git commit` із фікстури `rules/changelog/.../check.test.mjs` (з `user.name=test`, `user.email=test@test`, `-m 'init'`) потрапляли в реальний робочий репозиторій, де відбувався vitest run, і знищували `npm/package.json` (зменшували до `{"name":"mono"}`) + `npm/CHANGELOG.md` (зрізали до плейсхолдера). Race множив rogue commits/branches (`feat/x`, `feat/docs`, `feat/sync`) щоразу при запуску `bun run coverage`/Stryker. Усунено повним переписуванням `withTmpCwd` → `withTmpDir` (без `chdir`), `cwd` параметром у production функціях і defense-in-depth `pool: 'forks'`.
+- **`rules/test/coverage/coverage.mjs:192`** — dynamic import шлях `../../scripts/coverage-fix.mjs` резолвив у неіснуючий `npm/rules/scripts/coverage-fix.mjs`. Виправлено на `../../../scripts/coverage-fix.mjs` (реальний файл — у `npm/scripts/`). Усуває ERR_MODULE_NOT_FOUND у `n-cursor coverage --fix` і прибирає потребу у stub-стратегії, що створювала race-небезпечні fs-артефакти у production tree `rules/scripts/`.
+
+### Added
+
+- **`rules/test/js/no-process-chdir.mjs`** — JS concern: сканує `**/*.test.{js,mjs}` і падає на `process.chdir(`. Token-based regex (`/process\.chdir\s*\(/u`) — не зачіпає згадки у JSDoc. 8 unit-тестів.
+- **`rules/test/js/no-relative-fs-path.mjs`** — AST-сканер (`oxc-parser`) для `**/*.test.{js,mjs}`: знаходить виклики FS-функцій з `node:fs`/`node:fs/promises` (`writeFile`, `copyFile`, `mkdir`, `readFile`, `existsSync`, `rename`, `symlink`, `cp`, `*Sync`-варіанти + `writeJson`/`ensureDir`), де path-аргумент — string literal без префікса `/`, `\`, `file:`, `http(s):`, `data:`, чи Windows-disk-letter. Покриває обидва path-arg позиції у `copyFile`/`rename`/`symlink`/`link`/`cp`. Виловив би інцидент 1.28.0 (`tests/check-rule-fixtures.test.mjs`): `copyFile(src, 'default.conf.template')` / `copyFile(src, 'values-dev.ini')` зливали fixture у production tree `npm/`. 17 unit-тестів; на власному репо знайдено 0 порушень серед 106 test files.
+- **`rules/test/js/vitest-config-pool-forks.mjs`** — JS concern: substring-перевірка `pool: 'forks'` у `vitest.config.js`. Defense-in-depth. 6 unit-тестів.
+- **`rules/test/js/data/vitest_config/vitest.config.baseline.js`** — canonical baseline тепер містить `pool: 'forks'` з обґрунтуванням race-bug у docstring.
+- **`rules/test/test.mdc` — секція "Заборона `process.chdir` у тестах"** із описом інциденту, контрактом `withTmpDir`, посиланнями на нові concern'и.
+
+### Changed
+
+- **`scripts/utils/test-helpers.mjs`** — `withTmpDir(fn)` без `chdir`; `writeJson(absPath, data)` і `ensureDir(absPath)` валідують `isAbsolute`. Docstring описує інцидент.
+- **`vitest.config.js`** — `pool: 'forks'` як defense-in-depth з повним коментарем.
+
 ## [1.27.9] - 2026-05-27
 
 ### Added

package/package.json

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

package/rules/abie/js/applies.mjs

@@ -8,10 +8,11 @@ import { createCheckReporter } from '../../../scripts/lib/check-reporter.mjs'
 import { isAbieRuleEnabled } from '../lib/enabled.mjs'
 
 /**
+ * @param {string} [cwd] корінь репозиторію
  * @returns {Promise<boolean>} `true` — правило застосовне; `false` — пропустити
  */
-export function applies() {
-  return isAbieRuleEnabled(process.cwd())
+export function applies(cwd = process.cwd()) {
+  return isAbieRuleEnabled(cwd)
 }
 
 /**

package/rules/abie/js/env_dns.mjs

@@ -5,6 +5,7 @@
  *   - `ua.env`  → `abie-ua.internal` + `ua-*` namespace
  *
  * Файл `.env` без імені (локальний для розробника) — виключено.
+ * @param {string} [cwd] корінь репозиторію
  */
 import { readFile } from 'node:fs/promises'
 import { basename, relative } from 'node:path'
@@ -16,11 +17,12 @@ import { abieEnvNameFromBasename, collectAbieEnvFiles, validateAbieEnvInternalUr
 
 /**
  * @returns {Promise<number>} результат
+ * @param {string} [cwd] корінь репозиторію
  */
-export async function check() {
+export async function check(cwd = process.cwd()) {
   const reporter = createCheckReporter()
   const { pass, fail } = reporter
-  const root = process.cwd()
+  const root = cwd
 
   const ignorePaths = await loadCursorIgnorePaths(root)
   const envFiles = await collectAbieEnvFiles(root, ignorePaths)

package/rules/abie/js/firebase_hosting.mjs

@@ -12,12 +12,13 @@ import { createCheckReporter } from '../../../scripts/lib/check-reporter.mjs'
 const SKIP_TOP_DIR_NAMES = new Set(['.git', 'node_modules'])
 
 /**
+ * @param {string} [cwd] корінь репозиторію
  * @returns {Promise<number>} результат
  */
-export async function check() {
+export async function check(cwd = process.cwd()) {
   const reporter = createCheckReporter()
   const { pass, fail } = reporter
-  const root = process.cwd()
+  const root = cwd
 
   let entries
   try {

package/rules/abie/js/hc_pairing.mjs

@@ -17,12 +17,13 @@ import { validateAbieHcModeline } from '../lib/hc-yaml.mjs'
 import { collectDeploymentDirs, findK8sYamlFiles } from '../lib/k8s-tree.mjs'
 
 /**
+ * @param {string} [cwd] корінь репозиторію
  * @returns {Promise<number>} результат
  */
-export async function check() {
+export async function check(cwd = process.cwd()) {
   const reporter = createCheckReporter()
   const { pass, fail } = reporter
-  const root = process.cwd()
+  const root = cwd
 
   const ignorePaths = await loadCursorIgnorePaths(root)
   const yamls = await findK8sYamlFiles(root, ignorePaths)

package/rules/abie/js/ua_http_route.mjs

@@ -6,6 +6,7 @@
  * Для спільних сервісів (`auth-run-hl`, `file-link-hl`) у base-HTTPRoute пакета — кожен `backendRef`
  * має `namespace: dev`; в overlay patch — JSON6902 на `/spec/rules/…/backendRefs/…/namespace` зі
  * `value: ua`. Кількість patch-ів = кількість таких посилань у base.
+ * @param {string} [cwd] корінь репозиторію
  */
 import { readFile } from 'node:fs/promises'
 import { relative } from 'node:path'
@@ -27,11 +28,12 @@ import {
 
 /**
  * @returns {Promise<number>} результат
+ * @param {string} [cwd] корінь репозиторію
  */
-export async function check() {
+export async function check(cwd = process.cwd()) {
   const reporter = createCheckReporter()
   const { pass, fail } = reporter
-  const root = process.cwd()
+  const root = cwd
 
   const ignorePaths = await loadCursorIgnorePaths(root)
   const yamls = await findK8sYamlFiles(root, ignorePaths)

package/rules/abie/js/ua_node_selector.mjs

@@ -4,6 +4,7 @@
  *
  * Структурні обмеження JSON6902 (заборона `remove + add` на той самий path) перевіряє k8s.mdc /
  * `k8s.kustomization` rego — тут лише abie-специфічне.
+ * @param {string} [cwd] корінь репозиторію
  */
 import { readFile } from 'node:fs/promises'
 import { relative } from 'node:path'
@@ -17,11 +18,12 @@ import { abieOverlayK8sTreeHasDeployment, isUaKustomizationPath } from '../lib/o
 
 /**
  * @returns {Promise<number>} результат
+ * @param {string} [cwd] корінь репозиторію
  */
-export async function check() {
+export async function check(cwd = process.cwd()) {
   const reporter = createCheckReporter()
   const { pass, fail } = reporter
-  const root = process.cwd()
+  const root = cwd
 
   const ignorePaths = await loadCursorIgnorePaths(root)
   const yamls = await findK8sYamlFiles(root, ignorePaths)

package/rules/adr/js/hooks.mjs

@@ -32,8 +32,8 @@ const HOOK_ARTIFACTS = /** @type {const} */ ([
   { scriptName: 'normalize-decisions.sh', logName: 'normalize-decisions.log' }
 ])
 
-const PROJECT_SETTINGS_PATH = '.claude/settings.json'
-const CURSOR_HOOKS_PATH = '.cursor/hooks.json'
+const PROJECT_SETTINGS_REL = '.claude/settings.json'
+const CURSOR_HOOKS_REL = '.cursor/hooks.json'
 const EOL_RE = /\r?\n/u
 
 const here = dirname(fileURLToPath(import.meta.url))
@@ -83,26 +83,28 @@ function gitignoreLineCoversHookLog(line, logPath) {
 /**
  * Перевіряє наявність і канонічність одного hook-скрипта.
  * @param {import('../../../scripts/lib/check-reporter.mjs').CheckReporter} reporter репортер для збору результатів
+ * @param {string} cwd корінь репозиторію
  * @param {string} scriptName базове ім'я скрипта (наприклад `capture-decisions.sh`)
  * @returns {Promise<void>}
  */
-async function checkHookScript(reporter, scriptName) {
+async function checkHookScript(reporter, cwd, scriptName) {
   const { pass, fail } = reporter
-  const projectPath = projectHookPath(scriptName)
+  const projectRel = projectHookPath(scriptName)
+  const projectAbs = join(cwd, projectRel)
   const bundledPath = join(BUNDLED_HOOKS_DIR, scriptName)
-  if (!existsSync(projectPath)) {
-    fail(`${projectPath} не існує — запусти \`npx @nitra/cursor\` (правило adr копіює канонічний скрипт)`)
+  if (!existsSync(projectAbs)) {
+    fail(`${projectRel} не існує — запусти \`npx @nitra/cursor\` (правило adr копіює канонічний скрипт)`)
     return
   }
   if (!existsSync(bundledPath)) {
     fail(`канонічний скрипт у пакеті не знайдено: ${bundledPath} — перевстанови @nitra/cursor`)
     return
   }
-  const [project, bundled] = await Promise.all([readFile(projectPath, 'utf8'), readFile(bundledPath, 'utf8')])
+  const [project, bundled] = await Promise.all([readFile(projectAbs, 'utf8'), readFile(bundledPath, 'utf8')])
   if (project === bundled) {
-    pass(`${projectPath} збігається з канонічним`)
+    pass(`${projectRel} збігається з канонічним`)
   } else {
-    fail(`${projectPath} відрізняється від канонічного — запусти \`npx @nitra/cursor\` для повторного синку`)
+    fail(`${projectRel} відрізняється від канонічного — запусти \`npx @nitra/cursor\` для повторного синку`)
   }
 }
 
@@ -112,13 +114,14 @@ async function checkHookScript(reporter, scriptName) {
  * `capture-decisions.sh`; `settings.local.json` не дублює) валідують
  * `npm/policy/adr/settings_json/` і `npm/policy/adr/settings_local_json/`.
  * @param {import('../../../scripts/lib/check-reporter.mjs').CheckReporter} reporter репортер
+ * @param {string} cwd корінь репозиторію
  */
-function checkProjectSettings(reporter) {
+function checkProjectSettings(reporter, cwd) {
   const { pass, fail } = reporter
-  if (existsSync(PROJECT_SETTINGS_PATH)) {
-    pass(`${PROJECT_SETTINGS_PATH} є (Stop-hook перевіряє npx @nitra/cursor fix → adr.settings_json)`)
+  if (existsSync(join(cwd, PROJECT_SETTINGS_REL))) {
+    pass(`${PROJECT_SETTINGS_REL} є (Stop-hook перевіряє npx @nitra/cursor fix → adr.settings_json)`)
   } else {
-    fail(`${PROJECT_SETTINGS_PATH} не існує — запусти \`npx @nitra/cursor\``)
+    fail(`${PROJECT_SETTINGS_REL} не існує — запусти \`npx @nitra/cursor\``)
   }
 }
 
@@ -165,25 +168,27 @@ function cursorConfigHasStopHook(config, marker) {
 /**
  * Перевіряє project-level Cursor hooks config для ADR stop-hooks.
  * @param {import('../../../scripts/lib/check-reporter.mjs').CheckReporter} reporter репортер
+ * @param {string} cwd корінь репозиторію
  * @returns {Promise<void>}
  */
-async function checkCursorHooks(reporter) {
+async function checkCursorHooks(reporter, cwd) {
   const { pass, fail } = reporter
-  if (!existsSync(CURSOR_HOOKS_PATH)) {
-    fail(`${CURSOR_HOOKS_PATH} не існує — запусти \`npx @nitra/cursor\``)
+  const cursorHooksAbs = join(cwd, CURSOR_HOOKS_REL)
+  if (!existsSync(cursorHooksAbs)) {
+    fail(`${CURSOR_HOOKS_REL} не існує — запусти \`npx @nitra/cursor\``)
     return
   }
-  const config = await readJsonSafe(CURSOR_HOOKS_PATH)
+  const config = await readJsonSafe(cursorHooksAbs)
   if (config === null) {
-    fail(`${CURSOR_HOOKS_PATH} не парситься як JSON — запусти \`npx @nitra/cursor\` або виправ файл`)
+    fail(`${CURSOR_HOOKS_REL} не парситься як JSON — запусти \`npx @nitra/cursor\` або виправ файл`)
     return
   }
   for (const { scriptName } of HOOK_ARTIFACTS) {
     const marker = projectHookPath(scriptName)
     if (cursorConfigHasStopHook(config, marker)) {
-      pass(`${CURSOR_HOOKS_PATH} має stop-hook для ${marker}`)
+      pass(`${CURSOR_HOOKS_REL} має stop-hook для ${marker}`)
     } else {
-      fail(`${CURSOR_HOOKS_PATH}: відсутній stop-hook для \`${marker}\` (adr.mdc)`)
+      fail(`${CURSOR_HOOKS_REL}: відсутній stop-hook для \`${marker}\` (adr.mdc)`)
     }
   }
 }
@@ -212,17 +217,19 @@ function checkGitignoreForLog(reporter, logName, gitignoreContent) {
 /**
  * Перевіряє `.gitignore` для всіх hook-логів одним проходом.
  * @param {import('../../../scripts/lib/check-reporter.mjs').CheckReporter} reporter репортер для збору результатів
+ * @param {string} cwd корінь репозиторію
  * @returns {Promise<void>}
  */
-async function checkGitignore(reporter) {
+async function checkGitignore(reporter, cwd) {
   const { fail } = reporter
-  if (!existsSync('.gitignore')) {
+  const gitignoreAbs = join(cwd, '.gitignore')
+  if (!existsSync(gitignoreAbs)) {
     for (const { logName } of HOOK_ARTIFACTS) {
       fail(`.gitignore не існує — додай рядок \`${projectLogPath(logName)}\``)
     }
     return
   }
-  const content = await readFile('.gitignore', 'utf8')
+  const content = await readFile(gitignoreAbs, 'utf8')
   for (const { logName } of HOOK_ARTIFACTS) {
     checkGitignoreForLog(reporter, logName, content)
   }
@@ -273,16 +280,17 @@ function checkLlmCliAvailable(reporter) {
 
 /**
  * Перевіряє відповідність проєкту правилам adr.mdc.
+ * @param {string} [cwd] корінь репозиторію
  * @returns {Promise<number>} 0 — все OK, 1 — є проблеми
  */
-export async function check() {
+export async function check(cwd = process.cwd()) {
   const reporter = createCheckReporter()
   for (const { scriptName } of HOOK_ARTIFACTS) {
-    await checkHookScript(reporter, scriptName)
+    await checkHookScript(reporter, cwd, scriptName)
   }
-  checkProjectSettings(reporter)
-  await checkCursorHooks(reporter)
-  await checkGitignore(reporter)
+  checkProjectSettings(reporter, cwd)
+  await checkCursorHooks(reporter, cwd)
+  await checkGitignore(reporter, cwd)
   checkLlmCliAvailable(reporter)
   return reporter.getExitCode()
 }

package/rules/bun/js/layout.mjs

@@ -19,6 +19,7 @@
  */
 import { existsSync } from 'node:fs'
 import { readFile } from 'node:fs/promises'
+import { join } from 'node:path'
 
 import { createCheckReporter } from '../../../scripts/lib/check-reporter.mjs'
 
@@ -31,13 +32,15 @@ const WHITESPACE_RE = /\s+/u
 
 /**
  * Зчитує `rules` та `disable-rules` з `.n-cursor.json`.
+ * @param {string} cwd корінь репозиторію
  * @returns {Promise<{ rules: Set<string>, disabled: Set<string> }>} активні правила і явно вимкнені
  */
-async function loadNCursorRules() {
+async function loadNCursorRules(cwd) {
   const empty = { rules: new Set(), disabled: new Set() }
-  if (!existsSync('.n-cursor.json')) return empty
+  const cfgPath = join(cwd, '.n-cursor.json')
+  if (!existsSync(cfgPath)) return empty
   try {
-    const raw = JSON.parse(await readFile('.n-cursor.json', 'utf8'))
+    const raw = JSON.parse(await readFile(cfgPath, 'utf8'))
     const list = Array.isArray(raw?.rules) ? raw.rules.map(String) : []
     const disabled = Array.isArray(raw?.['disable-rules']) ? raw['disable-rules'].map(String) : []
     return { rules: new Set(list), disabled: new Set(disabled) }
@@ -157,45 +160,47 @@ function checkCursorRuleScripts(reporter, scripts, cursorRules) {
 
 /**
  * Перевіряє відповідність проєкту правилам bun.mdc
+ * @param {string} [cwd] корінь репозиторію
  * @returns {Promise<number>} 0 — все OK, 1 — є проблеми
  */
-export async function check() {
+export async function check(cwd = process.cwd()) {
   const reporter = createCheckReporter()
   const { pass, fail } = reporter
 
   for (const f of ['package-lock.json', 'yarn.lock', 'pnpm-lock.yaml', '.yarnrc.yml']) {
-    if (existsSync(f)) {
+    if (existsSync(join(cwd, f))) {
       fail(`Знайдено заборонений файл: ${f} — видали його`)
     } else {
       pass(`Немає ${f}`)
     }
   }
 
-  if (existsSync('.yarn')) {
+  if (existsSync(join(cwd, '.yarn'))) {
     fail('Знайдено директорію .yarn — видали її')
   } else {
     pass('Немає .yarn/')
   }
-  if (existsSync('bun.lock')) {
+  if (existsSync(join(cwd, 'bun.lock'))) {
     pass('bun.lock є')
   } else {
     fail('Відсутній bun.lock — запусти bun i')
   }
 
-  if (existsSync('bunfig.toml')) {
+  if (existsSync(join(cwd, 'bunfig.toml'))) {
     pass('bunfig.toml є (структуру перевіряє npx @nitra/cursor fix → bun.bunfig)')
   } else {
     fail('Відсутній bunfig.toml — створи з [install] linker = "hoisted" (bun.mdc)')
   }
 
-  const cursorRules = await loadNCursorRules()
+  const cursorRules = await loadNCursorRules(cwd)
 
-  if (!existsSync('package.json')) {
+  const pkgPath = join(cwd, 'package.json')
+  if (!existsSync(pkgPath)) {
     fail('Відсутній package.json у корені')
     return reporter.getExitCode()
   }
 
-  const pkg = JSON.parse(await readFile('package.json', 'utf8'))
+  const pkg = JSON.parse(await readFile(pkgPath, 'utf8'))
   const scripts = pkg.scripts && typeof pkg.scripts === 'object' ? pkg.scripts : {}
   checkCursorRuleScripts(reporter, scripts, cursorRules)
 

package/rules/capacitor/js/platforms.mjs

@@ -429,12 +429,13 @@ async function isIosCocoaPodsExemptByNitraConfig(root) {
 }
 
 /**
+ * @param {string} [cwd] корінь репозиторію
  * @returns {Promise<number>} **0** — **ok**; **1** — **fail** (див. **capacitor.mdc**)
  */
-export async function check() {
+export async function check(cwd = process.cwd()) {
   const reporter = createCheckReporter()
   const { pass, fail, getExitCode } = reporter
-  const root = process.cwd()
+  const root = cwd
 
   const acc = { byPath: new Map(), anyCapacitor: false }
   await collectCapacitorDataFromAllPackageJson(root, acc)