@7n/test 0.7.2 → 0.9.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [0.9.0] - 2026-06-28
+
+### Changed
+
+- В нормальному циклі `assessNeed` (LLM-оцінка) повністю прибрано: coverage % < порогу є достатнім сигналом для генерації тестів. У bootstrap-режимі тепер використовується тільки `quickClassify` (локально, без LLM). Таким чином нульова кількість LLM-викликів до `assessNeed` у будь-якому сценарії.
+
+## [0.8.0] - 2026-06-28
+
+### Added
+
+- `assessNeed` тепер запускає локальні JS-евристики перед LLM-викликом: файли-реекспорти (`export { } from`) класифікуються як `needsTests:false`, файли з функціями та гілками — як `needsTests:true`, без будь-яких витрат на API. LLM викликається тільки для неоднозначних файлів. Нова публічна функція `quickClassify(content)` доступна для тестування/розширення.
+
 ## [0.7.2] - 2026-06-27
 
 ### Fixed

package/package.json

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

package/src/assess-need.mjs

@@ -1,6 +1,8 @@
 /**
  * LLM assessment via pi SDK: does an uncovered file actually need unit tests?
  * Uses callText() (no tools) for structured JSON replies.
+ *
+ * Quick local heuristics run first — LLM is called only for ambiguous files.
  */
 import { existsSync, readFileSync } from 'node:fs'
 import { join } from 'node:path'
@@ -25,17 +27,60 @@ needsTests: true when:
 - Business logic with conditions or non-trivial contracts
 - Pure functions that can be unit-tested cheaply`
 
+// Lines that are purely module wiring — no testable logic
+const WIRING_RE = /^(import\b|export\s+(?:\{[^}]*\}|\*|type\b|interface\b|enum\b))/
+
+/**
+ * Strip JS/TS comments for heuristic analysis.
+ * @param {string} src
+ * @returns {string}
+ */
+function stripComments(src) {
+  return src.replace(/\/\*[\s\S]*?\*\//g, ' ').replace(/\/\/[^\n]*/g, '')
+}
+
+/**
+ * Fast local classifier — no I/O, no LLM.
+ * Returns a result object for obvious cases, or null when ambiguous.
+ *
+ * @param {string} content file source
+ * @returns {{ needsTests: boolean, reason: string } | null}
+ */
+export function quickClassify(content) {
+  const stripped = stripComments(content)
+  const lines = stripped.split('\n').map(l => l.trim()).filter(Boolean)
+
+  // All lines are imports/re-exports → no testable logic
+  if (lines.length > 0 && lines.every(l => WIRING_RE.test(l))) {
+    return { needsTests: false, reason: 'лише імпорти/реекспорти без логіки' }
+  }
+
+  // Has control flow AND function bodies → obviously needs tests
+  const hasBranches = /\bif\s*\(|\bswitch\s*\(/.test(stripped)
+  const hasFunctions = /\bfunction\s*\w*\s*\(|=>\s*\{/.test(stripped)
+  if (hasBranches && hasFunctions) {
+    return { needsTests: true, reason: 'містить функції з розгалуженнями' }
+  }
+
+  return null
+}
+
 /**
  * @param {{file: string, pct: number}} fileInfo
  * @param {string} dir project root
- * @param {Function} [callTextFn] injectable for tests
+ * @param {Function} callTextFn
  * @returns {Promise<{file: string, pct: number, needsTests: boolean, reason: string}>}
  */
 async function assessOne(fileInfo, dir, callTextFn) {
   const absPath = join(dir, fileInfo.file)
   if (!existsSync(absPath)) return { ...fileInfo, needsTests: false, reason: 'файл недоступний' }
 
-  let content = readFileSync(absPath, 'utf8')
+  const rawContent = readFileSync(absPath, 'utf8')
+
+  const quick = quickClassify(rawContent)
+  if (quick !== null) return { ...fileInfo, ...quick }
+
+  let content = rawContent
   if (content.length > MAX_CONTENT_BYTES) content = content.slice(0, MAX_CONTENT_BYTES) + '\n...(truncated)'
 
   const prompt =
@@ -48,23 +93,20 @@ async function assessOne(fileInfo, dir, callTextFn) {
     const match = text.match(/\{[\s\S]*?"needsTests"[\s\S]*?\}/)
     const parsed = JSON.parse(match?.[0] ?? '{}')
     return {
-      file: fileInfo.file,
-      pct: fileInfo.pct,
+      ...fileInfo,
       needsTests: parsed.needsTests !== false,
       reason: typeof parsed.reason === 'string' ? parsed.reason : ''
     }
   } catch {
-    return {
-      file: fileInfo.file,
-      pct: fileInfo.pct,
-      needsTests: true,
-      reason: 'оцінка не вдалась — вважаємо що потрібні тести'
-    }
+    return { ...fileInfo, needsTests: true, reason: 'оцінка не вдалась — вважаємо що потрібні тести' }
   }
 }
 
 /**
  * Assess a list of uncovered files: do they need tests?
+ * Obvious cases (re-exports, functions-with-branches) are resolved locally.
+ * Only ambiguous files trigger an LLM call.
+ *
  * @param {Array<{file: string, pct: number}>} files
  * @param {string} dir project root
  * @param {{ callText?: Function }} [opts]

package/src/run.mjs

@@ -1,10 +1,10 @@
 /**
- * Main auto-test loop: coverage → assess → gen-tests → loop until max → mutation + fix.
+ * Main auto-test loop: coverage → gen-tests → loop until max → mutation + fix.
  *
  * Phase 1 (loop, max 5 iterations):
  *   1. Measure per-file line coverage via vitest+lcov.
- *   2. pi CLI assesses which uncovered files actually need tests.
- *   3. pi agent generates tests for those files.
+ *   2. Generate tests for all files below threshold — coverage % is the signal.
+ *   3. Bootstrap: when no tests exist, scan sources and quickClassify locally.
  *   4. Repeat until coverage maxes out or no improvement.
  *
  * Phase 2:
@@ -12,11 +12,13 @@
  *   6. Auto-fix survived mutants via pi agent.
  */
 import { measureCoveragePerFile, getUncoveredFiles, findSourceFiles } from './coverage-per-file.mjs'
-import { assessNeed } from './assess-need.mjs'
+import { quickClassify } from './assess-need.mjs'
 import { generateTests } from './gen-tests.mjs'
 import { fixFailingTests } from './fix-tests.mjs'
 import { runCoverageSteps } from './coverage/coverage.mjs'
 import { withLock } from './scripts/utils/with-lock.mjs'
+import { readFileSync } from 'node:fs'
+import { join } from 'node:path'
 
 const MAX_ITERATIONS = 5
 const COVERAGE_THRESHOLD = 80
@@ -53,17 +55,26 @@ export async function runAutoTest(dir, opts = {}) {
         console.log('⚠ Vitest coverage не повернула даних — перевір налаштування vitest.')
         break
       }
-      console.log(`\n── Bootstrap: ${sourceFiles.length} файлів без тестів — оцінюю потребу ──\n`)
-      const bootstrapFiles = sourceFiles.map(f => ({ file: f, pct: 0 }))
-      const assessed = await assessNeed(bootstrapFiles, dir)
-      const needsTests = assessed.filter(f => f.needsTests)
-      if (needsTests.length === 0) {
-        console.log('✓ LLM вирішила: жоден файл не потребує unit-тестів.')
+      console.log(`\n── Bootstrap: ${sourceFiles.length} джерел — класифікую локально ──\n`)
+
+      const bootstrapFiles = sourceFiles
+        .map(f => {
+          try {
+            const q = quickClassify(readFileSync(join(dir, f), 'utf8'))
+            if (q?.needsTests === false) return null
+          } catch { /* include on read error */ }
+          return { file: f, pct: 0 }
+        })
+        .filter(Boolean)
+
+      if (bootstrapFiles.length === 0) {
+        console.log('✓ Жоден файл не потребує unit-тестів (реекспорти/типи).')
         break
       }
-      console.log(`\n→ Bootstrap-тести для (${needsTests.length}):`)
-      for (const f of needsTests) console.log(`  • ${f.file} — ${f.reason}`)
-      await generateTests(needsTests, dir)
+
+      console.log(`\n→ Bootstrap-тести для (${bootstrapFiles.length}):`)
+      for (const f of bootstrapFiles) console.log(`  • ${f.file}`)
+      await generateTests(bootstrapFiles, dir)
       continue
     }
 
@@ -84,21 +95,12 @@ export async function runAutoTest(dir, opts = {}) {
     }
     prevUncoveredCount = uncovered.length
 
-    console.log(`\n── Оцінюю ${uncovered.length} непокритих файлів (LLM) ──\n`)
-    const assessed = await assessNeed(uncovered, dir)
-    const needsTests = assessed.filter(f => f.needsTests)
-
-    if (needsTests.length === 0) {
-      console.log('✓ LLM вирішила: жоден непокритий файл не потребує unit-тестів.')
-      break
-    }
-
-    console.log(`\n→ Потребують тестів (${needsTests.length}):`)
-    for (const f of needsTests) {
-      console.log(`  • ${f.file} (${f.pct.toFixed(1)}%) — ${f.reason}`)
+    console.log(`\n→ Генерую тести для ${uncovered.length} файлів:`)
+    for (const f of uncovered) {
+      console.log(`  • ${f.file} (${f.pct.toFixed(1)}%)`)
     }
 
-    await generateTests(needsTests, dir)
+    await generateTests(uncovered, dir)
   }
 
   if (opts.noMutation) {