@7n/test 0.7.0 → 0.7.2

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [0.7.2] - 2026-06-27
+
+### Fixed
+
+- Виправлено парсинг JSON-репорту vitest 4.x: поле `name` замість `testFilePath` (breaking change у vitest JSON reporter). Наслідок: падаючі тести тепер коректно детектуються і передаються у fix-loop замість bootstrap.
+
+## [0.7.1] - 2026-06-27
+
+### Fixed
+
+- `parseFailingTests` тепер коректно обробляє module-level помилки (import/syntax errors) — тести, що падають ще до запуску (`assertionResults: []`), більше не пропускаються, що усувало нескінченну bootstrap-петлю.
+
 ## [0.7.0] - 2026-06-27
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@7n/test",
-  "version": "0.7.0",
+  "version": "0.7.2",
   "description": "CLI-утиліта @7n/test",
   "keywords": [
     "7n",

package/src/coverage-per-file.mjs

@@ -13,10 +13,7 @@ import { join, relative, dirname } from 'node:path'
 import { env } from 'node:process'
 
 const _require = createRequire(import.meta.url)
-const VITEST_BIN = join(
-  dirname(_require.resolve('vitest/package.json')),
-  'vitest.mjs'
-)
+const VITEST_BIN = join(dirname(_require.resolve('vitest/package.json')), 'vitest.mjs')
 
 const TEST_FILE_RE = /\.(test|spec)\.[^.]+$|[/\\]tests?[/\\]/
 const MAX_ERRORS_PER_FILE = 5
@@ -63,9 +60,8 @@ function parseFailingTests(jsonPath, dir) {
     const data = JSON.parse(readFileSync(jsonPath, 'utf8'))
     return (data.testResults ?? [])
       .filter(r => r.status === 'failed')
-      .map(r => ({
-        file: relative(dir, r.testFilePath),
-        errors: (r.assertionResults ?? [])
+      .map(r => {
+        const assertionErrors = (r.assertionResults ?? [])
           .filter(a => a.status === 'failed')
           .slice(0, MAX_ERRORS_PER_FILE)
           .map(a => {
@@ -73,8 +69,16 @@ function parseFailingTests(jsonPath, dir) {
             const msg = (a.failureMessages?.[0] ?? '').split('\n').slice(0, MAX_ERROR_LINES).join('\n')
             return `${name}:\n${msg}`
           })
-      }))
-      .filter(f => !f.file.startsWith('..') && f.errors.length > 0)
+        // Module-level errors (import/syntax) produce no assertionResults
+        const errors =
+          assertionErrors.length > 0
+            ? assertionErrors
+            : [
+                `Suite error: ${(r.message ?? r.failureMessage ?? 'module-level failure').split('\n').slice(0, MAX_ERROR_LINES).join('\n')}`
+              ]
+        return { file: relative(dir, r.testFilePath ?? r.name), errors }
+      })
+      .filter(f => !f.file.startsWith('..'))
   } catch {
     return []
   }
@@ -136,8 +140,19 @@ export function getUncoveredFiles(files, threshold = 80) {
 
 const SOURCE_EXT_RE = /\.(mjs|js|ts|vue|py)$/
 const IGNORE_DIRS = new Set([
-  'node_modules', 'dist', 'build', 'out', '.git', '__pycache__',
-  'coverage', '.cursor', '.claude', '.pi', 'docs', 'bin', 'reports'
+  'node_modules',
+  'dist',
+  'build',
+  'out',
+  '.git',
+  '__pycache__',
+  'coverage',
+  '.cursor',
+  '.claude',
+  '.pi',
+  'docs',
+  'bin',
+  'reports'
 ])
 
 /**

package/src/fix-tests.mjs

@@ -16,10 +16,7 @@ import { join, relative, dirname } from 'node:path'
 import { env } from 'node:process'
 
 const _require = createRequire(import.meta.url)
-const VITEST_BIN = join(
-  dirname(_require.resolve('vitest/package.json')),
-  'vitest.mjs'
-)
+const VITEST_BIN = join(dirname(_require.resolve('vitest/package.json')), 'vitest.mjs')
 
 const MODEL = env.N_CURSOR_FIX_TESTS_MODEL ?? env.N_CLOUD_MAX_MODEL ?? ''
 const MAX_ERRORS_PER_FILE = 5
@@ -35,11 +32,15 @@ export async function getFailingTests(dir) {
   const outputFile = join(tmpDir, 'results.json')
 
   try {
-    spawnSync(process.execPath, [VITEST_BIN, 'run', '--reporter=json', `--outputFile=${outputFile}`, '--passWithNoTests'], {
-      cwd: dir,
-      stdio: 'inherit',
-      env
-    })
+    spawnSync(
+      process.execPath,
+      [VITEST_BIN, 'run', '--reporter=json', `--outputFile=${outputFile}`, '--passWithNoTests'],
+      {
+        cwd: dir,
+        stdio: 'inherit',
+        env
+      }
+    )
 
     if (!existsSync(outputFile)) return []
 
@@ -52,9 +53,8 @@ export async function getFailingTests(dir) {
 
     return (data.testResults ?? [])
       .filter(r => r.status === 'failed')
-      .map(r => ({
-        file: relative(dir, r.testFilePath),
-        errors: (r.assertionResults ?? [])
+      .map(r => {
+        const assertionErrors = (r.assertionResults ?? [])
           .filter(a => a.status === 'failed')
           .slice(0, MAX_ERRORS_PER_FILE)
           .map(a => {
@@ -62,8 +62,12 @@ export async function getFailingTests(dir) {
             const msg = (a.failureMessages?.[0] ?? '').split('\n').slice(0, MAX_ERROR_LINES).join('\n')
             return `${name}:\n${msg}`
           })
-      }))
-      .filter(f => !f.file.startsWith('..') && f.errors.length > 0)
+        const errors = assertionErrors.length > 0
+          ? assertionErrors
+          : [`Suite error: ${(r.message ?? r.failureMessage ?? 'module-level failure').split('\n').slice(0, MAX_ERROR_LINES).join('\n')}`]
+        return { file: relative(dir, r.testFilePath ?? r.name), errors }
+      })
+      .filter(f => !f.file.startsWith('..'))
   } finally {
     await rm(tmpDir, { recursive: true, force: true }).catch(() => {})
   }