@7n/rules 1.49.21 → 1.49.23

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [1.49.23] - 2026-07-27
+
+### Fixed
+
+- Treat reconcile diffs containing only release entries as patch-equivalent
+
+## [1.49.22] - 2026-07-27
+
+### Fixed
+
+- Retry initial PR check discovery and prune rebuildable forensic dependencies
+
 ## [1.49.21] - 2026-07-27
 
 ### Fixed

package/package.json

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

package/skills/git-reconcile/js/docs/orchestrate.md

@@ -3,7 +3,7 @@ type: JS Module
 title: orchestrate.mjs
 resource: npm/skills/git-reconcile/js/orchestrate.mjs
 docgen:
-  crc: f87f22fc
+  crc: 69749e90
   model: omlx/gemma-4-e4b-it-OptiQ-4bit
   score: 100
 ---
@@ -20,11 +20,13 @@ docgen:
 
 Для корисної групи JS створює ізольований worktree від policy base, застосовує commits або stash і відсікає порожній tree diff. `captureCachedBehaviorBaseline` кешує Promise baseline за base OID, щоб concurrent PR-групи не дублювали test run. `validateBehaviorState` перевіряє Git state, scoped docs/lint, tests відносно baseline і changelog. `remediateBehaviorState` запускає canonical fixers для code та non-code directories; після remediation final gates повторюються у read-only режимі.
 
-`runAsync` запускає install, tests, lint і PR checks без блокування event loop. Після push та `gh pr create` функція `verifyPullRequestReadiness` очікує terminal checks і передає набори PR/base checks у `classifyPullRequestChecks`. Успішний outcome `pr-created` повертається лише для `ready`; regression, baseline-red, pending, timeout або unreadable checks зберігають branch, URL і worktree та блокують cleanup.
+`runAsync` запускає install, tests, lint і PR checks без блокування event loop. Після push та `gh pr create` функція `verifyPullRequestReadiness` очікує terminal checks і передає набори PR/base checks у `classifyPullRequestChecks`. Якщо initial rollup порожній, orchestration дає GitHub коротке вікно на реєстрацію checks і повторює watch. Успішний outcome `pr-created` повертається лише для `ready`; regression, baseline-red, pending, timeout або unreadable checks зберігають branch, URL і worktree та блокують cleanup.
 
 `commitPendingChanges` створює додатковий commit лише для staged remediation. Чистий index після перенесення branch-source є валідним, коли корисні commits уже присутні в `HEAD`. Порожній PR check rollup вважається непідтвердженим, а не успішним.
 
-`cleanupSource` видаляє лише точний доказово безпечний branch/stash. Cleanup review-source дозволений після `drop` або коли всі його групи завершились `pr-created` чи `patch-equivalent`; `failed` і всі `pr-checks-*` outcomes лишають source та forensic worktree.
+`hasOnlyChangeEntries` відсікає tree diff, у якому лишилися тільки `*/.changes/*.md`: такий результат вважається `patch-equivalent`, не створює PR і дозволяє cleanup source branch.
+
+`pruneForensicDependencies` прибирає лише rebuildable `node_modules` зі збереженого forensic worktree. `cleanupSource` видаляє лише точний доказово безпечний branch/stash. Cleanup review-source дозволений після `drop` або коли всі його групи завершились `pr-created` чи `patch-equivalent`; `failed` і всі `pr-checks-*` outcomes лишають source та forensic worktree.
 
 ## Публічний API
 
@@ -36,6 +38,8 @@ docgen:
 - `captureBehaviorBaseline`, `captureCachedBehaviorBaseline` — фіксують і кешують test baseline.
 - `validateBehaviorState`, `validateFinalProjectGates`, `remediateBehaviorState` — виконують behavioral та canonical gates.
 - `commitPendingChanges` — комітить лише staged remediation, не вимагаючи порожнього commit для вже перенесених commits.
+- `hasOnlyChangeEntries` — розпізнає release-metadata-only diff, який не потребує PR.
+- `pruneForensicDependencies` — звільняє rebuildable dependencies без втрати Git evidence.
 - `classifyPullRequestChecks`, `verifyPullRequestReadiness` — класифікують CI відносно base commit.
 - `formatOutcomeCounts`, `formatReport` — формують точний deterministic summary.
 - `cleanupSource`, `runGitReconcileOrchestrator` — виконують безпечний cleanup і координують повний flow.
@@ -47,6 +51,8 @@ docgen:
 - Canonical fixers охоплюють code/non-code scope, після чого gates повторюються без fix.
 - PR вважається створеним успішно лише після terminal green checks.
 - Порожній PR check rollup fail-closed зберігає forensic worktree.
+- Forensic worktree зберігає Git evidence без накопичення `node_modules`.
+- Diff лише з `.changes` не створює PR і не блокує cleanup source branch.
 - CI regression, baseline-red або непідтверджений стан завершують orchestration non-zero і зберігають forensic refs/worktree.
 - Cleanup не видаляє protected, open-PR, kept, failed або `pr-checks-*` sources.
 - `spawnSync.error`, зокрема `ENOENT`, зберігається в command diagnostics.

package/skills/git-reconcile/js/orchestrate.mjs

@@ -1,10 +1,11 @@
 /** @see ./docs/orchestrate.md */
 import { spawn, spawnSync } from 'node:child_process'
 import { once } from 'node:events'
-import { appendFileSync, existsSync, readFileSync } from 'node:fs'
+import { appendFileSync, existsSync, readFileSync, rmSync } from 'node:fs'
 import { dirname, isAbsolute, join } from 'node:path'
 import { performance } from 'node:perf_hooks'
 import { env } from 'node:process'
+import { setTimeout as delay } from 'node:timers/promises'
 
 import { renderProgressLine } from '../../../scripts/lib/lint-surface/progress.mjs'
 import { readGitPolicy } from '../../../scripts/lib/git-policy.mjs'
@@ -24,6 +25,7 @@ const REF_HEADS_RE = /^refs\/heads\//
 const REF_ORIGIN_RE = /^refs\/remotes\/origin\//
 const RENAME_DELETE_CONFLICT_RE = /^CONFLICT \(rename\/delete\): .+? renamed to (.+?) in .+?, but deleted/
 const SOURCE_CODE_RE = /\.(?:js|mjs|ts|vue|rs|py)$/
+const CHANGE_ENTRY_RE = /(^|\/)\.changes\/[^/]+\.md$/
 const WHITESPACE_RE = /\s+/
 const ACP_PROGRESS_ENV = 'N_LLM_ACP_PROGRESS'
 const REF_INVENTORY_FORMAT = ['%(refname)', '%00', '%(object', 'name)', '%00', '%(committer', 'date:iso-strict)'].join(
@@ -1007,6 +1009,16 @@ export function sourceDirectories(paths) {
   return [...new Set(paths.filter(path => SOURCE_CODE_RE.test(path)).map(path => dirname(path)))].toSorted()
 }
 
+/**
+ * Визначає технічний залишок, який містить лише release entries. Такі файли
+ * не доводять окремої корисної поведінки й не мають породжувати PR.
+ * @param {string[]} paths змінені шляхи відносно policy base
+ * @returns {boolean} чи всі зміни лежать безпосередньо у `.changes/`
+ */
+export function hasOnlyChangeEntries(paths) {
+  return paths.length > 0 && paths.every(path => CHANGE_ENTRY_RE.test(path))
+}
+
 /**
  * Збирає tracked і untracked code directories відносно policy base ref.
  * @param {string} cwd worktree
@@ -1021,6 +1033,40 @@ function changedPaths(cwd, spawnFn) {
   return [...new Set([...tracked, ...untracked])]
 }
 
+/**
+ * @param {string} cwd worktree
+ * @param {typeof spawnSync} spawnFn інжект
+ * @returns {boolean} чи tree diff складається лише з release entries
+ */
+function hasOnlyChangeEntriesFromBase(cwd, spawnFn) {
+  return hasOnlyChangeEntries(changedPaths(cwd, spawnFn))
+}
+
+/**
+ * Прибирає no-op або change-only worktree до дорогих behavioral/CI gates.
+ * @param {object} args контекст materialized worktree
+ * @returns {{status:string,branch:string,rationale?:string}|null} terminal outcome
+ */
+function discardPatchEquivalentWorktree(args) {
+  const { worktree, rootCwd, spawnFn, onProgress, validated = false } = args
+  if (!hasChangesFromBase(worktree.cwd, spawnFn)) {
+    onProgress('remove no-op worktree')
+    removeReconcileWorktree(worktree, rootCwd, spawnFn)
+    return { status: 'patch-equivalent', branch: worktree.branch }
+  }
+  if (!hasOnlyChangeEntriesFromBase(worktree.cwd, spawnFn)) return null
+
+  onProgress('remove change-only worktree')
+  removeReconcileWorktree(worktree, rootCwd, spawnFn)
+  return {
+    status: 'patch-equivalent',
+    branch: worktree.branch,
+    rationale: validated
+      ? 'Після Git validation лишилися тільки release entries у .changes/'
+      : 'Після перенесення лишилися тільки release entries у .changes/'
+  }
+}
+
 /**
  * Повертає директорії зміненого коду.
  * @param {string} cwd worktree
@@ -1456,19 +1502,36 @@ export function classifyPullRequestChecks(prChecks, baseChecks) {
   }
 }
 
+/**
+ * Видаляє лише rebuildable dependencies зі збереженого forensic worktree.
+ * Git metadata, commits і tracked/untracked зміни не зачіпаються.
+ * @param {string} worktreeCwd шлях worktree
+ * @returns {boolean} чи було що прибрати
+ */
+export function pruneForensicDependencies(worktreeCwd) {
+  const dependencies = join(worktreeCwd, 'node_modules')
+  if (!existsSync(dependencies)) return false
+  try {
+    rmSync(dependencies, { recursive: true, force: true })
+    return true
+  } catch {
+    return false
+  }
+}
+
 /**
  * Чекає terminal CI state й порівнює failed checks з base commit.
  * @param {object} args PR context
  * @returns {Promise<{status:string,error?:string}>} readiness
  */
 export async function verifyPullRequestReadiness(args) {
-  const { url, cwd, spawnFn = spawnSync, asyncSpawnFn } = args
+  const { url, cwd, spawnFn = spawnSync, asyncSpawnFn, delayFn = delay } = args
   const longRunner = resolveAsyncSpawn(spawnFn, asyncSpawnFn)
   await runAsync('gh', ['pr', 'checks', url, '--watch', '--interval', '10'], cwd, longRunner, {
     allowFailure: true,
     timeoutMs: PR_CHECK_TIMEOUT_MS
   })
-  const view = await runAsync(
+  let view = await runAsync(
     'gh',
     ['pr', 'view', url, '--json', 'statusCheckRollup,baseRefOid'],
     cwd,
@@ -1478,7 +1541,28 @@ export async function verifyPullRequestReadiness(args) {
   if (view.status !== 0) {
     return { status: 'pr-checks-unverified', error: `Не вдалося прочитати PR checks: ${view.stderr || view.error}` }
   }
-  const pr = parseJson(view.stdout, null)
+  let pr = parseJson(view.stdout, null)
+  if (pr && Array.isArray(pr.statusCheckRollup) && pr.statusCheckRollup.length === 0) {
+    await delayFn(10_000)
+    await runAsync('gh', ['pr', 'checks', url, '--watch', '--interval', '10'], cwd, longRunner, {
+      allowFailure: true,
+      timeoutMs: PR_CHECK_TIMEOUT_MS
+    })
+    view = await runAsync(
+      'gh',
+      ['pr', 'view', url, '--json', 'statusCheckRollup,baseRefOid'],
+      cwd,
+      longRunner,
+      { allowFailure: true }
+    )
+    if (view.status !== 0) {
+      return {
+        status: 'pr-checks-unverified',
+        error: `Не вдалося повторно прочитати PR checks: ${view.stderr || view.error}`
+      }
+    }
+    pr = parseJson(view.stdout, null)
+  }
   if (!pr || !Array.isArray(pr.statusCheckRollup) || !pr.baseRefOid) {
     return { status: 'pr-checks-unverified', error: 'GitHub повернув неповний PR check rollup' }
   }
@@ -1596,11 +1680,8 @@ async function createPullRequest(args) {
       log,
       onProgress
     })
-    if (!hasChangesFromBase(worktree.cwd, spawnFn)) {
-      onProgress('remove no-op worktree')
-      removeReconcileWorktree(worktree, rootCwd, spawnFn)
-      return { status: 'patch-equivalent', branch: worktree.branch }
-    }
+    const appliedOutcome = discardPatchEquivalentWorktree({ worktree, rootCwd, spawnFn, onProgress })
+    if (appliedOutcome) return appliedOutcome
     const verification =
       changedSourceDirectories(worktree.cwd, spawnFn).length > 0
         ? await finalizeBehavior({
@@ -1620,11 +1701,14 @@ async function createPullRequest(args) {
     if (unresolved.length > 0) throw new Error(`Нерозв'язані конфлікти: ${unresolved.join(', ')}`)
     git(['diff', '--check'], worktree.cwd, spawnFn)
     git(['add', '-A'], worktree.cwd, spawnFn)
-    if (!hasChangesFromBase(worktree.cwd, spawnFn)) {
-      onProgress('remove no-op worktree')
-      removeReconcileWorktree(worktree, rootCwd, spawnFn)
-      return { status: 'patch-equivalent', branch: worktree.branch }
-    }
+    const validatedOutcome = discardPatchEquivalentWorktree({
+      worktree,
+      rootCwd,
+      spawnFn,
+      onProgress,
+      validated: true
+    })
+    if (validatedOutcome) return validatedOutcome
     onProgress('delta lint')
     const finalGates = await passFinalProjectGates({
       cwd: worktree.cwd,
@@ -1669,6 +1753,8 @@ async function createPullRequest(args) {
       asyncSpawnFn: deps.asyncSpawnFn
     })
     if (readiness.status !== 'ready') {
+      onProgress('prune forensic dependencies')
+      pruneForensicDependencies(worktree.cwd)
       return {
         status: readiness.status,
         error: readiness.error,
@@ -1682,6 +1768,10 @@ async function createPullRequest(args) {
     return { status: 'pr-created', url: createdPr, branch: worktree.branch }
   } catch (error) {
     const setup = /** @type {{branch?:string,worktree?:string}} */ (error)
+    if (worktree?.cwd) {
+      onProgress('prune forensic dependencies')
+      pruneForensicDependencies(worktree.cwd)
+    }
     return {
       status: 'failed',
       error: error instanceof Error ? error.message : String(error),