@7n/rules 1.32.0 → 1.32.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [1.32.1] - 2026-07-20
+
+### Changed
+
+- fix(n-rules/n-taze): не забруднювати дерево перед worktree-гейтом і не губити untracked-директорії при поверненні змін (#160)
+
 ## [1.32.0] - 2026-07-19
 
 ### Changed

package/bin/n-rules.js

@@ -101,7 +101,7 @@ import { syncClaudeConfig } from '../scripts/sync-claude-config.mjs'
 import { syncGitignoreWorktree } from '../scripts/lib/sync-gitignore-worktree.mjs'
 import { upgradeNRulesToLatestAndBunInstall } from '../scripts/upgrade-n-rules-and-install.mjs'
 import { runRenameYamlExtensionsCli } from './rename-yaml-extensions.mjs'
-import { runSkillsCli } from '../scripts/skills-cli.mjs'
+import { isTazeOrchestratorSkillArgs, runSkillsCli } from '../scripts/skills-cli.mjs'
 import { syncSetupBunDepsAction } from '../scripts/sync-setup-bun-deps-action.mjs'
 
 const PACKAGE_NAME = '@7n/rules'
@@ -1729,7 +1729,14 @@ try {
     }
     // `ci` (plan) — read-only гейт-команда для CI-джоб: не мутує package.json
     // (ensure дописав би devDependency прямо в чекауті pipeline-агента).
-    if (command !== 'ci') await ensureNRulesInRootDevDependencies(effectiveRoot)
+    // `skill <runner> taze` — JS-оркестрований worktree-only шлях
+    // (skills/taze/js/orchestrate.mjs): сам створює worktree і гейтить на
+    // чистоту дерева ДО checkout. Мутація тут забруднила б дерево прямо
+    // перед тим гейтом і провалювала б auto-create на інакше чистому дереві —
+    // оркестратор сам робить self-upgrade devDependency вже ВСЕРЕДИНІ
+    // щойно створеного worktree, після власного гейту чистоти.
+    const skipDevDepsEnsure = command === 'skill' && isTazeOrchestratorSkillArgs(args)
+    if (command !== 'ci' && !skipDevDepsEnsure) await ensureNRulesInRootDevDependencies(effectiveRoot)
     // Підкоманди-оркестратори (hook/lint/skill/adr-normalize-local/taze/release тощо)
     // можуть спавнити внутрішню agent/LLM-сесію — ADR Stop-hooks (capture/normalize)
     // мають пропустити її як технічний шум, не людську думку (spec 2026-06-30).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@7n/rules",
-  "version": "1.32.0",
+  "version": "1.32.1",
   "description": "CLI еталонних правил і skills (префікс n-): синк у репозиторій, дельта-lint, конформність",
   "keywords": [
     "cli",

package/scripts/docs/skills-cli.md

@@ -3,7 +3,7 @@ type: JS Module
 title: skills-cli.mjs
 resource: npm/scripts/skills-cli.mjs
 docgen:
-  crc: 12d3bf6d
+  crc: 3e5781e6
   model: omlx/gemma-4-e4b-it-OptiQ-4bit
 ---
 

package/scripts/lib/auto-worktree.mjs

@@ -1,10 +1,11 @@
 /** @see ./docs/auto-worktree.md */
-import { existsSync } from 'node:fs'
-import { copyFile, mkdir, rm } from 'node:fs/promises'
-import { dirname, join } from 'node:path'
+import { existsSync, statSync } from 'node:fs'
+import { copyFile, mkdir, readdir, rm } from 'node:fs/promises'
+import { dirname, join, relative } from 'node:path'
 import { createInterface } from 'node:readline/promises'
 
 const YES_RE = /^y(es)?$/i
+const TRAILING_SLASH_RE = /\/$/
 
 /**
  * Питає y/N у терміналі. Поза TTY (CI, неінтерактивний виклик) — одразу `false`,
@@ -112,6 +113,31 @@ export async function ensureRunningInWorktree(
   return { cwd: newCwd, autoCreated: true, branchArg }
 }
 
+/**
+ * Рекурсивно копіює вміст директорії (лише файли-листки; підкаталоги
+ * створюються по дорозі через `makeDir`). `listDir` — інжектований `readdir`
+ * з `{ recursive: true, withFileTypes: true }` (Node ≥20.1): кожен `Dirent`
+ * несе `parentPath`/`path` — реальний каталог, у якому файл лежить.
+ * @param {string} srcDir директорія-джерело (у worktree)
+ * @param {string} destDir директорія-призначення (у оригінальному дереві)
+ * @param {{ copy: (src: string, dest: string) => Promise<void>, makeDir: (path: string, opts?: object) => Promise<void>, listDir: (path: string, opts?: object) => Promise<Array<{ name: string, parentPath?: string, path?: string, isFile: () => boolean }>> }} io інжекти
+ * @returns {Promise<string[]>} шляхи скопійованих файлів відносно `srcDir` (forward-slash)
+ */
+async function copyDirectoryRecursive(srcDir, destDir, { copy, makeDir, listDir }) {
+  const entries = await listDir(srcDir, { recursive: true, withFileTypes: true })
+  const copiedRelPaths = []
+  for (const entry of entries) {
+    if (!entry.isFile()) continue
+    const entryDir = entry.parentPath ?? entry.path ?? srcDir
+    const relFromSrc = relative(srcDir, join(entryDir, entry.name)).replaceAll('\\', '/')
+    const dest = join(destDir, relFromSrc)
+    await makeDir(dirname(dest), { recursive: true })
+    await copy(join(entryDir, entry.name), dest)
+    copiedRelPaths.push(relFromSrc)
+  }
+  return copiedRelPaths
+}
+
 /**
  * Переносить зміни з автоствореного worktree назад у вихідне дерево як
  * **untracked/незакомічені** правки (просте копіювання файлів, без git
@@ -122,48 +148,79 @@ export async function ensureRunningInWorktree(
  * лише нову назву — стара лишається як була, прийнятний компроміс для
  * інструментів, що самі файли не перейменовують (ефект можливий лише як
  * побічний результат LLM-рефакторингу чи форматера).
+ *
+ * **Untracked-директорія цілком.** Git схлопує щойно створену untracked
+ * директорію в один porcelain-рядок із `/` на кінці (якщо в ній нема жодного
+ * файлу, вже відомого git) — `copyFile` на такий шлях впав би (`EISDIR`/`ENOENT`)
+ * і обривав би цикл, гублячи все, що йшло по порядку ітерації ДАЛІ. Такий
+ * рядок (суфікс `/` або реальний каталог на диску) копіюється рекурсивно
+ * (`copyDirectoryRecursive`) — весь вміст, а не один `copyFile`.
  * @param {string} worktreeCwd автостворений worktree, з якого переносимо
  * @param {string} originalCwd вихідне дерево, куди переносимо
  * @param {typeof import('node:child_process').spawnSync} spawnFn інжект для тестів
  * @param {(line: string) => void} log колбек прогресу
- * @param {{ copyFile?: (src: string, dest: string) => Promise<void>, rm?: (path: string, opts?: object) => Promise<void>, mkdir?: (path: string, opts?: object) => Promise<void> }} [deps] інжекти для тестів
- * @returns {Promise<string[]>} відносні шляхи перенесених файлів
+ * @param {{ copyFile?: (src: string, dest: string) => Promise<void>, rm?: (path: string, opts?: object) => Promise<void>, mkdir?: (path: string, opts?: object) => Promise<void>, readdir?: (path: string, opts?: object) => Promise<Array<object>> }} [deps] інжекти для тестів
+ * @returns {Promise<{ brought: string[], failed: boolean }>} відносні шляхи перенесених файлів і чи стався провал (частковий чи повний)
  */
 export async function bringChangesBackToOriginal(worktreeCwd, originalCwd, spawnFn, log, deps = {}) {
   const copy = deps.copyFile ?? copyFile
   const removeFile = deps.rm ?? rm
   const makeDir = deps.mkdir ?? mkdir
+  const listDir = deps.readdir ?? readdir
 
   const statusResult = spawnFn('git', ['status', '--porcelain'], { cwd: worktreeCwd, encoding: 'utf8' })
   if (statusResult.status !== 0) {
     log(
       `⚠️ Не вдалось прочитати git status у "${worktreeCwd}" — зміни НЕ перенесені назад, worktree лишиться для ручного розбору.`
     )
-    return []
+    return { brought: [], failed: true }
   }
 
   const lines = statusResult.stdout.split('\n').filter(Boolean)
   if (lines.length === 0) {
     log('ℹ️ Worktree без змін — нічого переносити назад.')
-    return []
+    return { brought: [], failed: false }
   }
 
   const brought = []
+  let failed = false
   for (const line of lines) {
     const rest = line.slice(3)
     const relPath = rest.includes(' -> ') ? rest.split(' -> ', 2)[1] : rest
     const srcPath = join(worktreeCwd, relPath)
     const destPath = join(originalCwd, relPath)
-    if (existsSync(srcPath)) {
-      await makeDir(dirname(destPath), { recursive: true })
-      await copy(srcPath, destPath)
-    } else {
-      await removeFile(destPath, { force: true })
+    try {
+      if (!existsSync(srcPath)) {
+        await removeFile(destPath, { force: true, recursive: true })
+        brought.push(relPath)
+        continue
+      }
+
+      const isDir = relPath.endsWith('/') || statSync(srcPath).isDirectory()
+      if (isDir) {
+        const trimmedRelPath = relPath.replace(TRAILING_SLASH_RE, '')
+        const copiedRelPaths = await copyDirectoryRecursive(srcPath, destPath, { copy, makeDir, listDir })
+        for (const nestedRelPath of copiedRelPaths) {
+          brought.push(`${trimmedRelPath}/${nestedRelPath}`)
+        }
+      } else {
+        await makeDir(dirname(destPath), { recursive: true })
+        await copy(srcPath, destPath)
+        brought.push(relPath)
+      }
+    } catch (error) {
+      failed = true
+      log(
+        `⚠️ Не вдалось перенести "${relPath}" назад у "${originalCwd}" — ${error instanceof Error ? error.message : String(error)}`
+      )
     }
-    brought.push(relPath)
   }
-  log(`📤 Перенесено назад у "${originalCwd}" як untracked: ${brought.join(', ')}`)
-  return brought
+  log(
+    failed
+      ? `⚠️ Перенесення назад у "${originalCwd}" частково провалилось — перенесено: ${brought.join(', ') || '(нічого)'}`
+      : `📤 Перенесено назад у "${originalCwd}" як untracked: ${brought.join(', ')}`
+  )
+  return { brought, failed }
 }
 
 /**

package/scripts/lib/docs/auto-worktree.md

@@ -3,7 +3,7 @@ type: JS Module
 title: auto-worktree.mjs
 resource: npm/scripts/lib/auto-worktree.mjs
 docgen:
-  crc: 82081402
+  crc: bae09068
   model: openai-codex/gpt-5.4-mini
   score: 100
   issues: judge:inaccurate:0.98

package/scripts/skills-cli.mjs

@@ -244,6 +244,23 @@ export function resolveBundledPackageRoot(fromModuleUrl = import.meta.url) {
   return join(dirname(fileURLToPath(fromModuleUrl)), '..')
 }
 
+/**
+ * Чи `argv` (аргументи після `skill`) резолвиться в JS-оркестрований
+ * worktree-only `taze`-шлях (`runTazeOrchestratorCli`) — той самий критерій,
+ * що й нижче в `runSkillsCli`. Використовується `n-rules.js`, щоб не мутувати
+ * root `package.json` (self-upgrade `@7n/rules`) ДО власного worktree-гейту
+ * оркестратора: той сам створює worktree і перевіряє чистоту дерева
+ * (`ensureRunningInWorktree`, `requireCleanTree: true`) — мутація package.json
+ * прямо перед цим викликом примусово провалила б auto-create там, де дерево
+ * інакше було б чисте.
+ * @param {string[]} argv аргументи після `skill`
+ * @returns {boolean} `true`, якщо запуск піде через `runTazeOrchestratorCli`
+ */
+export function isTazeOrchestratorSkillArgs(argv) {
+  const [first, second] = argv
+  return RUNNERS.has(first) && first !== 'claude' && normalizeSkillId(second) === 'taze'
+}
+
 /**
  * @param {string[]} argv аргументи після `skill` у `n-rules`
  * @param {{ packageRoot?: string, projectDir?: string, log?: (line: string) => void, logError?: (line: string) => void, deps?: { runPiAgentSkill?: (prompt: string, opts?: object) => Promise<{ ok: boolean, error: string|null }> } }} [options] перевизначення кореня пакета, каталогу проєкту, функцій виводу та інжектів (для тестів)

package/skills/taze/js/docs/orchestrate.md

@@ -3,7 +3,7 @@ type: JS Module
 title: orchestrate.mjs
 resource: npm/skills/taze/js/orchestrate.mjs
 docgen:
-  crc: 7bf5d0c0
+  crc: 63cee070
   model: openai-codex/gpt-5.4-mini
   score: 100
   issues: judge:inaccurate:0.98

package/skills/taze/js/orchestrate.mjs

@@ -2,6 +2,7 @@
 import { spawnSync } from 'node:child_process'
 import { pathToFileURL } from 'node:url'
 
+import { ensureNRulesInRootDevDependencies } from '../../../scripts/ensure-n-rules-dev-dependencies.mjs'
 import {
   bringChangesBackToOriginal,
   ensureRunningInWorktree,
@@ -138,7 +139,13 @@ async function runEcosystem(provider, { cwd, runner, log, deps, spawnFn, call })
       eco.results.push({ ...entry, ...outcome })
       log(outcome.ok ? `  ✅ ${entry.pkg}` : `  ❌ ${entry.pkg}: ${outcome.error}`)
       if (outcome.ok && outcome.text) {
-        await writeCache(entry.pkg, entry.from, entry.to, { notes: outcome.text, sourceRepo: cwd, updatedAt: new Date().toISOString() }, deps)
+        await writeCache(
+          entry.pkg,
+          entry.from,
+          entry.to,
+          { notes: outcome.text, sourceRepo: cwd, updatedAt: new Date().toISOString() },
+          deps
+        )
       }
     }
 
@@ -241,22 +248,50 @@ export async function runTazeOrchestrator(options = {}) {
   })
   const cwd = worktree.cwd
 
+  // Self-upgrade root package.json devDependency (`@7n/rules`) — навмисно
+  // ВСЕРЕДИНІ worktree, ПІСЛЯ гейту `ensureRunningInWorktree` на чистоту
+  // дерева: та ж мутація у `originalCwd` ДО цього гейту забруднила б дерево
+  // прямо перед перевіркою, яку сам гейт щойно пройшов (n-rules.js навмисно
+  // пропускає її для цього шляху). Провал — не критичний для taze-прогону,
+  // лише лог.
+  try {
+    const ensureDevDeps = deps.ensureNRulesInRootDevDependencies ?? ensureNRulesInRootDevDependencies
+    await ensureDevDeps(cwd, { silent: true })
+  } catch (error) {
+    log(
+      `⚠️ Self-upgrade @7n/rules у "${cwd}" провалився — пропускаю (${error instanceof Error ? error.message : String(error)})`
+    )
+  }
+
   let cleanedUp = false
   /**
    * Переносить зміни назад і прибирає автостворений worktree — не більше
    * одного разу (idempotent), щоб і сигнальний обробник, і `finally` могли
    * безпечно кликати те саме без подвійного `bringChangesBackToOriginal`/
    * `removeAutoCreatedWorktree`.
+   *
+   * **Захист від втрати даних.** Якщо перенесення провалилось (частково чи
+   * повністю — `bringChangesBackToOriginal` кидає або повертає
+   * `failed: true`), worktree НЕ прибирається: джерело ще не перенесених
+   * файлів лишається на диску для ручного відновлення замість того, щоб
+   * зникнути разом з `npx \@7n/mt worktree remove`.
    * @returns {Promise<void>}
    */
   const cleanupAutoCreatedWorktree = async () => {
     if (cleanedUp) return
     cleanedUp = true
+    let failed = true
     try {
-      await bringChangesBackToOriginal(cwd, originalCwd, spawnFn, log, deps)
+      ;({ failed } = await bringChangesBackToOriginal(cwd, originalCwd, spawnFn, log, deps))
     } catch (error) {
       log(`⚠️ Перенесення змін назад провалилось: ${error instanceof Error ? error.message : String(error)}`)
     }
+    if (failed) {
+      log(
+        `⚠️ Перенесення назад не вдалось повністю — worktree "${cwd}" (гілка "${worktree.branchArg}") НЕ прибирається, розберіться вручну.`
+      )
+      return
+    }
     removeAutoCreatedWorktree(worktree.branchArg, originalCwd, spawnFn, log)
   }