@nitra/cursor 1.27.9 → 1.28.0

package/rules/test/js/vitest-config-pool-forks.mjs

@@ -0,0 +1,52 @@
+/**
+ * `vitest.config.js` має ставити `pool: 'forks'` — defense-in-depth для
+ * race-bug у `process.cwd()` (test.mdc, секція "Заборона `process.chdir` у тестах").
+ *
+ * Чому не достатньо самої заборони `process.chdir(`: third-party код у залежностях
+ * може робити chdir всередині vitest worker'а. У `pool: 'threads'` (default) усі
+ * workers ділять один процес → race на `process.cwd()` між паралельними test
+ * files. `pool: 'forks'` ізолює кожен test file у власному child-процесі.
+ *
+ * Перевірка — substring у source-тексті `vitest.config.js`. Не парсимо JS AST,
+ * бо це може бути будь-який export-формат (ESM default, named, CommonJS).
+ * Достатньо знайти `pool:` із значенням `'forks'`/`"forks"` (whitespace дозволений).
+ *
+ * Скіпи: правило не застосовне, якщо `vitest.config.js` не існує (нема vitest
+ * у проєкті) — це не помилка, лише skip. Якщо файл є — `pool: 'forks'`
+ * обов'язковий.
+ */
+import { existsSync } from 'node:fs'
+import { readFile } from 'node:fs/promises'
+import { join } from 'node:path'
+
+import { createCheckReporter } from '../../../scripts/lib/check-reporter.mjs'
+
+/** Subтring-pattern: `pool: 'forks'` або `pool: "forks"` (з опційним whitespace). */
+const POOL_FORKS_RE = /pool\s*:\s*['"]forks['"]/u
+
+/**
+ * Перевіряє, що `vitest.config.js` (якщо існує) містить `pool: 'forks'`.
+ * @param {string} [cwdParam] корінь репозиторію
+ * @returns {Promise<number>} 0 — OK або skip, 1 — config без `pool: 'forks'`
+ */
+export async function check(cwdParam = process.cwd()) {
+  const reporter = createCheckReporter()
+  const { pass, fail } = reporter
+
+  const configPath = join(cwdParam, 'vitest.config.js')
+  if (!existsSync(configPath)) {
+    pass('vitest.config.js відсутній — pool-перевірку пропущено')
+    return reporter.getExitCode()
+  }
+
+  const body = await readFile(configPath, 'utf8')
+  if (POOL_FORKS_RE.test(body)) {
+    pass("vitest.config.js містить pool: 'forks' (test.mdc)")
+  } else {
+    fail(
+      "vitest.config.js має містити pool: 'forks' — defense-in-depth для race у process.cwd() між паралельними test files (test.mdc)"
+    )
+  }
+
+  return reporter.getExitCode()
+}

package/rules/test/test.mdc

@@ -60,6 +60,23 @@ Recursive globs ловлять файли всередині `tests/` так с
 
 Пропускаються: `node_modules`, `.git`, `dist`, `build`, `.venv`, `venv`, шляхи з `.n-cursor.json:ignore`.
 
+## Заборона `process.chdir` у тестах
+
+`process.chdir(dir)` — **process-wide** мутація. Vitest за замовчуванням ставить `pool: 'threads'`, і всі workers ділять один процес: паралельний test file може перехопити cwd сусіда посеред FS- або `git`-операції. У реальному інциденті це призвело до того, що `git init`+`git commit` із tmp-фікстури потрапив у реальний робочий репозиторій і створив rogue commit з автором `test <test@test>`, знищивши `npm/package.json` / `CHANGELOG.md`.
+
+Тому:
+
+- **У тестах заборонено** `process.chdir(...)` напряму та будь-які хелпери, що його викликають (історичний `withTmpCwd` видалений у `1.28.0`).
+- Канон: `withTmpDir(async dir => { ... })` зі `scripts/utils/test-helpers.mjs` — створює tmpdir і передає **абсолютний** `dir` у callback **без** `process.chdir`.
+- Усі FS-операції у тесті — через `join(dir, …)` і `writeJson(join(dir, …), …)` / `ensureDir(join(dir, …))` (хелпери валідують `isAbsolute`).
+- Усі child-процеси — `execFile(bin, args, { cwd: dir })`, `spawnSync(bin, args, { cwd: dir })`.
+- Concern-функції правил — `await check(dir)`, `await applies(dir)`, `await fix(dir)`; усі production функції приймають перший параметр `cwd = process.cwd()` (default зберігає CLI-сумісність).
+- `vitest.config.js` додатково ставить `pool: 'forks'` як defense-in-depth: навіть якщо хтось пропустить правило вище, fork-ізоляція не дасть race у production tree.
+
+Це **обов'язково** і для тестів пакета `@nitra/cursor`, і для кожного проєкту-споживача. Перевіряє concern `no-process-chdir` (`rules/test/js/no-process-chdir.mjs`): сканує `**/*.test.{js,mjs}` і падає з ❌ на будь-яке вживання `process.chdir(`.
+
+Канон `vitest.config.js` (substring requirement `pool: 'forks'`): [vitest.config.snippet.js](./policy/vitest_config/template/vitest.config.snippet.js)
+
 ## Покриття + мутаційне тестування
 
 Канонічна команда — `n-cursor coverage`: збирає метрики покриття (`vitest run --coverage`, `cargo llvm-cov` тощо) і мутаційного тестування (Stryker з vitest-runner + `coverageAnalysis: 'perTest'`, `cargo-mutants`) з усіх активних провайдерів у `.n-cursor.json#rules` і пише `COVERAGE.md` у корінь проєкту. Лок і дедуп — `withLock('coverage', ...)`.

package/rules/text/js/forbidden-prettier.mjs

@@ -9,6 +9,7 @@
  * (https://prettier.io/docs/configuration). Якщо Prettier додасть новий формат — додай рядок.
  */
 import { existsSync } from 'node:fs'
+import { join } from 'node:path'
 
 import { createCheckReporter } from '../../../scripts/lib/check-reporter.mjs'
 
@@ -38,15 +39,16 @@ const FORBIDDEN_PRETTIER_FILES = [
 
 /**
  * Перевіряє, що жоден Prettier-конфіг чи ignore-файл не лежить у корені проєкту.
+ * @param {string} [cwd] корінь репозиторію
  * @returns {number} 0 — все OK, 1 — знайдено заборонений файл
  */
-export function check() {
+export function check(cwd = process.cwd()) {
   const reporter = createCheckReporter()
   const { pass, fail } = reporter
 
   let anyFound = false
   for (const file of FORBIDDEN_PRETTIER_FILES) {
-    if (existsSync(file)) {
+    if (existsSync(join(cwd, file))) {
       fail(`${file} заборонено — Prettier не використовуємо, перейди на oxfmt (text.mdc)`)
       anyFound = true
     }

package/rules/text/js/formatting.mjs

@@ -27,9 +27,11 @@
  *    у `devDependencies`, заборона `markdownlint-cli2` у залежностях.
  *  - `npm/policy/bun/package_json/` — у `devDependencies` лише `@nitra/*`
  *    (раніше дублювалося тут).
+ * @param {string} cwd корінь репозиторію
  */
 import { existsSync } from 'node:fs'
 import { readFile } from 'node:fs/promises'
+import { join } from 'node:path'
 
 import { createCheckReporter } from '../../../scripts/lib/check-reporter.mjs'
 import { anyRunStepIncludes, parseWorkflowYaml } from '../../../scripts/lib/gha-workflow.mjs'
@@ -65,14 +67,16 @@ function verifyUkApostropheRuleParagraph(filePath, body, failFn, passFn) {
  * Перевіряє .v8rignore.
  * @param {(msg: string) => void} passFn callback при успішній перевірці
  * @param {(msg: string) => void} failFn callback при помилці
+ * @param {string} cwd корінь репозиторію
  */
-async function checkV8rIgnore(passFn, failFn) {
+async function checkV8rIgnore(passFn, failFn, cwd) {
   const required = ['.vscode/extensions.json', '.vscode/settings.json']
-  if (!existsSync('.v8rignore')) {
+  const v8rPath = join(cwd, '.v8rignore')
+  if (!existsSync(v8rPath)) {
     failFn('.v8rignore не існує — створи згідно n-text.mdc (мінімум .vscode/extensions.json і .vscode/settings.json)')
     return
   }
-  const raw = await readFile('.v8rignore', 'utf8')
+  const raw = await readFile(v8rPath, 'utf8')
   const lines = new Set(
     raw
       .split('\n')
@@ -100,8 +104,9 @@ async function checkV8rIgnore(passFn, failFn) {
  * @param {(msg: string) => void} passFn callback при успішній перевірці
  * @param {(msg: string) => void} failFn callback при помилці
  * @returns {Promise<void>}
+ * @param {string} cwd корінь репозиторію
  */
-function checkTextConfigsExistence(passFn, failFn) {
+function checkTextConfigsExistence(passFn, failFn, cwd) {
   for (const [path, mdcRef] of [
     ['.oxfmtrc.json', 'text.oxfmtrc'],
     ['.cspell.json', 'text.cspell'],
@@ -109,7 +114,7 @@ function checkTextConfigsExistence(passFn, failFn) {
     ['.vscode/extensions.json', 'text.vscode_extensions'],
     ['.vscode/settings.json', 'text.vscode_settings']
   ]) {
-    if (existsSync(path)) {
+    if (existsSync(join(cwd, path))) {
       passFn(`${path} є (структуру перевіряє npx @nitra/cursor fix → ${mdcRef})`)
     } else {
       failFn(`${path} не існує — створи згідно n-text.mdc`)
@@ -125,14 +130,17 @@ function checkTextConfigsExistence(passFn, failFn) {
  * `@nitra/*` гейт) — у Rego (`text.package_json`, `bun.package_json`).
  * @param {(msg: string) => void} passFn callback при успішній перевірці
  * @param {(msg: string) => void} failFn callback при помилці
+ * @param {string} cwd корінь репозиторію
  */
-async function checkPackageJsonText(passFn, failFn) {
-  if (!existsSync('package.json')) return
-  const pkg = JSON.parse(await readFile('package.json', 'utf8'))
+async function checkPackageJsonText(passFn, failFn, cwd) {
+  const pkgPath = join(cwd, 'package.json')
+  if (!existsSync(pkgPath)) return
+  const pkg = JSON.parse(await readFile(pkgPath, 'utf8'))
   checkLintTextScript(pkg.scripts?.['lint-text'], passFn, failFn)
 
-  if (existsSync('.github/workflows/lint-text.yml')) {
-    const wf = await readFile('.github/workflows/lint-text.yml', 'utf8')
+  const lintTextWf = join(cwd, '.github/workflows/lint-text.yml')
+  if (existsSync(lintTextWf)) {
+    const wf = await readFile(lintTextWf, 'utf8')
     const root = parseWorkflowYaml(wf)
     const ok = root ? anyRunStepIncludes(root, 'bun run lint-text') : wf.includes('bun run lint-text')
     if (ok) {
@@ -166,27 +174,28 @@ function checkLintTextScript(lintText, passFn, failFn) {
 
 /**
  * Перевіряє відповідність проєкту правилам text.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
 
-  await checkV8rIgnore(pass, fail)
-  await checkTextConfigsExistence(pass, fail)
+  await checkV8rIgnore(pass, fail, cwd)
+  await checkTextConfigsExistence(pass, fail, cwd)
 
   // Prettier-конфіги/ignore — окремий concern `text.forbidden-prettier` (rules/text/js/forbidden-prettier.mjs).
 
-  const textRulePaths = ['.cursor/rules/n-text.mdc', 'npm/mdc/text.mdc'].filter(p => existsSync(p))
+  const textRulePaths = ['.cursor/rules/n-text.mdc', 'npm/mdc/text.mdc'].filter(p => existsSync(join(cwd, p)))
   if (textRulePaths.length === 0) {
     pass('n-text.mdc / npm/mdc/text.mdc відсутні — перевірку абзацу про апостроф пропущено')
   } else {
     for (const p of textRulePaths) {
-      verifyUkApostropheRuleParagraph(p, await readFile(p, 'utf8'), fail, pass)
+      verifyUkApostropheRuleParagraph(p, await readFile(join(cwd, p), 'utf8'), fail, pass)
     }
   }
 
-  await checkPackageJsonText(pass, fail)
+  await checkPackageJsonText(pass, fail, cwd)
 
   return reporter.getExitCode()
 }

package/rules/vue/js/packages.mjs

@@ -20,6 +20,7 @@
  * Окремо в `.vue` SFC заборонено імпорти Node-нативних модулів — `node:*` префікс або bare-ім’я
  * вбудованого модуля Node (`fs`, `path`, `timers/promises` тощо). Vue SFC виконується у браузері,
  * де Node API недоступне; такий код треба тримати у server-side утілітах.
+ * @param {string} cwd корінь репозиторію
  */
 import { existsSync } from 'node:fs'
 import { readFile } from 'node:fs/promises'
@@ -194,17 +195,18 @@ function ukFilesCountPhrase(n) {
  * @param {(msg: string) => void} passFn успіх
  * @param {(msg: string) => void} fail помилка
  * @returns {Promise<void>}
+ * @param {string} cwd корінь репозиторію
  */
-async function checkViteClientEnvAndEditorConfig(rootDir, prefix, passFn, fail) {
-  const envRel = join(rootDir, 'src/vite-env.d.ts')
-  if (!existsSync(envRel)) {
+async function checkViteClientEnvAndEditorConfig(rootDir, prefix, passFn, fail, cwd) {
+  const envAbs = join(cwd, rootDir, 'src/vite-env.d.ts')
+  if (!existsSync(envAbs)) {
     fail(
       `${prefix}немає src/vite-env.d.ts — додай файл з рядком /// <reference types="vite/client" /> ` +
         `(інакше TS/Volar не бачать типів для імпортів асетів: png, avif, css як URL).`
     )
     return
   }
-  const envContent = await readFile(envRel, 'utf8')
+  const envContent = await readFile(envAbs, 'utf8')
   if (!VITE_CLIENT_REFERENCE_RE.test(envContent)) {
     fail(
       `${prefix}src/vite-env.d.ts має містити /// <reference types="vite/client" /> ` +
@@ -214,7 +216,7 @@ async function checkViteClientEnvAndEditorConfig(rootDir, prefix, passFn, fail)
   }
   passFn(`${prefix}src/vite-env.d.ts посилається на vite/client`)
 
-  if (!existsSync(join(rootDir, 'jsconfig.json'))) {
+  if (!existsSync(join(cwd, rootDir, 'jsconfig.json'))) {
     fail(
       `${prefix}немає jsconfig.json у корені пакета — додай файл з "include": ["src/**/*"] тощо, ` +
         `щоб IDE підхопила vite-env.d.ts і .vue.`
@@ -268,15 +270,16 @@ function viteConfigHasVueInAutoImports(content) {
  * @param {(msg: string) => void} passFn callback при успішній перевірці
  * @param {(msg: string) => void} fail callback при помилці
  * @returns {Promise<{ hasVueAutoImport: boolean }>} ознака успішно сконфігурованого vue-auto-import (для checkVueImportViolations)
+ * @param {string} cwd корінь репозиторію
  */
-async function checkViteConfig(rootDir, prefix, passFn, fail) {
+async function checkViteConfig(rootDir, prefix, passFn, fail, cwd) {
   const configFiles = ['vite.config.js', 'vite.config.ts', 'vite.config.mjs']
-  const viteConfig = configFiles.find(f => existsSync(join(rootDir, f)))
+  const viteConfig = configFiles.find(f => existsSync(join(cwd, rootDir, f)))
   if (!viteConfig) {
     fail(`${prefix}немає vite.config.js|ts|mjs у каталозі пакета`)
     return { hasVueAutoImport: false }
   }
-  const content = await readFile(join(rootDir, viteConfig), 'utf8')
+  const content = await readFile(join(cwd, rootDir, viteConfig), 'utf8')
   if (ESBUILD_RE.test(content)) {
     fail(`${prefix}${viteConfig} містить 'esbuild' — заміни на 'rolldown'`)
   }
@@ -410,37 +413,39 @@ async function checkVueImportViolations(rootDir, absPackageRoot, ignorePaths, ha
  * @param {(msg: string) => void} fail функція зворотного виклику для реєстрації помилки перевірки
  * @param {(msg: string) => void} passFn успішне повідомлення (як у check-reporter)
  * @returns {Promise<void>} завершується після перевірок залежностей, `vite.config` і сканування джерел на імпорти з `vue`
+ * @param {string} cwd корінь репозиторію
  */
-async function checkVuePackage(rootDir, ignorePaths, fail, passFn) {
+async function checkVuePackage(rootDir, ignorePaths, fail, passFn, cwd) {
   const prefix = `[${packageLabel(rootDir)}] `
   passFn(`${prefix}package.json залежності перевіряє npx @nitra/cursor fix → vue.package_json`)
 
-  await checkViteClientEnvAndEditorConfig(rootDir, prefix, passFn, fail)
+  await checkViteClientEnvAndEditorConfig(rootDir, prefix, passFn, fail, cwd)
 
-  const { hasVueAutoImport } = await checkViteConfig(rootDir, prefix, passFn, fail)
+  const { hasVueAutoImport } = await checkViteConfig(rootDir, prefix, passFn, fail, cwd)
   await checkVueImportViolations(
     rootDir,
-    join(process.cwd(), rootDir),
+    join(cwd, rootDir),
     ignorePaths,
     hasVueAutoImport,
     prefix,
     passFn,
     fail
   )
-  await checkVueNodeImportViolations(rootDir, join(process.cwd(), rootDir), ignorePaths, prefix, passFn, fail)
-  await checkEsbuildMentions(rootDir, join(process.cwd(), rootDir), ignorePaths, prefix, passFn, fail)
+  await checkVueNodeImportViolations(rootDir, join(cwd, rootDir), ignorePaths, prefix, passFn, fail)
+  await checkEsbuildMentions(rootDir, join(cwd, rootDir), ignorePaths, prefix, passFn, fail)
 }
 
 /**
  * Збирає корені пакетів, у яких у `dependencies` є `vue`.
  * @param {string[]} roots усі корені пакетів monorepo
  * @returns {Promise<string[]>} перелік пакетів з vue у dependencies
+ * @param {string} cwd корінь репозиторію
  */
-async function collectVueRoots(roots) {
+async function collectVueRoots(roots, cwd) {
   /** @type {string[]} */
   const vueRoots = []
   for (const r of roots) {
-    const p = join(r, 'package.json')
+    const p = join(cwd, r, 'package.json')
     if (!existsSync(p)) continue
     const pkg = JSON.parse(await readFile(p, 'utf8'))
     if (pkg.dependencies?.vue) vueRoots.push(r)
@@ -453,13 +458,15 @@ async function collectVueRoots(roots) {
  * @param {(msg: string) => void} pass pass callback
  * @param {(msg: string) => void} fail fail callback
  * @returns {Promise<void>}
+ * @param {string} cwd корінь репозиторію
  */
-async function checkVueVolarRecommendation(pass, fail) {
-  if (!existsSync('.vscode/extensions.json')) {
+async function checkVueVolarRecommendation(pass, fail, cwd) {
+  const extPath = join(cwd, '.vscode/extensions.json')
+  if (!existsSync(extPath)) {
     fail('.vscode/extensions.json не існує (для Vue-проєкту потрібна рекомендація Vue.volar)')
     return
   }
-  const ext = JSON.parse(await readFile('.vscode/extensions.json', 'utf8'))
+  const ext = JSON.parse(await readFile(extPath, 'utf8'))
   if (ext.recommendations?.includes('Vue.volar')) {
     pass('extensions.json містить Vue.volar')
   } else {
@@ -469,14 +476,15 @@ async function checkVueVolarRecommendation(pass, fail) {
 
 /**
  * Перевіряє відповідність проєкту правилам vue.mdc (корінь і всі workspace-пакети з `vue` у dependencies).
+ * @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
 
-  const roots = await getMonorepoPackageRootDirs()
-  const vueRoots = await collectVueRoots(roots)
+  const roots = await getMonorepoPackageRootDirs(cwd)
+  const vueRoots = await collectVueRoots(roots, cwd)
 
   if (vueRoots.length === 0) {
     pass('Vue.volar: пропущено (у repo немає пакетів з vue у dependencies)')
@@ -484,11 +492,11 @@ export async function check() {
     return reporter.getExitCode()
   }
 
-  await checkVueVolarRecommendation(pass, fail)
+  await checkVueVolarRecommendation(pass, fail, cwd)
 
-  const ignorePaths = await loadCursorIgnorePaths(process.cwd())
+  const ignorePaths = await loadCursorIgnorePaths(cwd)
   for (const r of vueRoots) {
-    await checkVuePackage(r, ignorePaths, fail, pass)
+    await checkVuePackage(r, ignorePaths, fail, pass, cwd)
   }
 
   return reporter.getExitCode()