@7n/test 0.7.2 → 0.8.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [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.8.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]