@safetnsr/vet 1.8.2 → 1.8.4

package/dist/checks/config.js

@@ -61,6 +61,10 @@ function analyzeConfig(cwd, configFile, agentName, files) {
     if (completenessChecks > 0) {
         completenessScore = Math.round((completenessHits / completenessChecks) * 10);
     }
+    else {
+        // No framework dependencies detected — completeness is not applicable, don't penalize
+        completenessScore = 10;
+    }
     // Consistency: cross-reference with actual project config
     let consistencyScore = 10;
     const tsconfig = readFile(join(cwd, 'tsconfig.json'));
@@ -80,7 +84,9 @@ function analyzeConfig(cwd, configFile, agentName, files) {
         catch { /* */ }
     }
     // Check if config mentions testing but no test framework installed
-    if ((contentLower.includes('test') || contentLower.includes('spec')) && !deps.vitest && !deps.jest && !deps.mocha && !deps.ava) {
+    // Also check if using Node's built-in test runner (node:test)
+    const usesNodeTest = contentLower.includes('node:test') || contentLower.includes('node test runner') || contentLower.includes('node built-in test');
+    if ((contentLower.includes('test') || contentLower.includes('spec')) && !deps.vitest && !deps.jest && !deps.mocha && !deps.ava && !usesNodeTest) {
         consistencyScore -= 2;
         suggestions.push('config mentions tests but no test framework in dependencies');
     }

package/dist/checks/debt.js

@@ -128,6 +128,18 @@ function extractFunctions(source, file) {
     return fns;
 }
 // ── A) Near-duplicate detection ──────────────────────────────────────────────
+/** Check if functions are in a numbered spec implementation pattern (e.g. asi01, asi02...) */
+function isSpecPattern(group) {
+    if (group.length < 3)
+        return false;
+    const dirs = new Set(group.map(f => f.file.substring(0, f.file.lastIndexOf('/'))));
+    if (dirs.size !== 1)
+        return false; // must be same directory
+    // Check if filenames follow a numbered pattern
+    const bases = group.map(f => f.file.substring(f.file.lastIndexOf('/') + 1));
+    const numbered = bases.filter(b => /\d{2}/.test(b));
+    return numbered.length >= 3;
+}
 function findDuplicates(allFuncs) {
     const issues = [];
     const groups = new Map();
@@ -138,15 +150,21 @@ function findDuplicates(allFuncs) {
         groups.set(fn.hash, existing);
     }
     const reported = new Set();
-    // Exact duplicates
+    // Exact duplicates (only flag if normalized body is substantial)
     for (const [, group] of groups) {
         if (group.length < 2)
             continue;
+        // Skip if the normalized body is too generic (short functions normalize to same hash easily)
+        if (group[0].normalized.length < 65)
+            continue;
         // Deduplicate by name+file
         const key = group.map(f => `${f.file}:${f.name}`).sort().join('|');
         if (reported.has(key))
             continue;
         reported.add(key);
+        // Skip groups that follow a numbered spec pattern (e.g., ASI01-ASI10 checks)
+        if (isSpecPattern(group))
+            continue;
         const locations = group.map(f => `${f.name} (${f.file}:${f.line})`).join(', ');
         issues.push({
             severity: 'warning',
@@ -170,7 +188,7 @@ function findDuplicates(allFuncs) {
             if (a.normalized.length < 30 || b.normalized.length < 30)
                 continue;
             const sim = similarity(a.normalized, b.normalized);
-            if (sim > 0.85) {
+            if (sim > 0.92) {
                 const key = [a.file + ':' + a.name, b.file + ':' + b.name].sort().join('|');
                 if (reported.has(key))
                     continue;
@@ -358,9 +376,12 @@ export async function checkDebt(cwd, ignore) {
     issues.push(...driftIssues);
     // ── Scoring ──────────────────────────────────────────────────────────────
     const dupPenalty = Math.min(50, dupIssues.length * 8);
-    const orphanPenalty = Math.min(30, orphanIssues.length * 5);
-    const wrapperPenalty = Math.min(15, wrapperIssues.length * 3);
-    const driftPenalty = Math.min(10, driftIssues.length * 2);
+    const orphanWarnings = orphanIssues.filter(i => i.severity === 'warning');
+    const orphanPenalty = Math.min(30, orphanWarnings.length * 5);
+    const wrapperWarnings = wrapperIssues.filter(i => i.severity === 'warning');
+    const driftWarnings = driftIssues.filter(i => i.severity === 'warning');
+    const wrapperPenalty = Math.min(15, wrapperWarnings.length * 3);
+    const driftPenalty = Math.min(10, driftWarnings.length * 2);
     const rawScore = 100 - dupPenalty - orphanPenalty - wrapperPenalty - driftPenalty;
     const finalScore = Math.max(0, Math.round(rawScore));
     // ── Summary ──────────────────────────────────────────────────────────────

package/dist/checks/diff.js

@@ -124,7 +124,7 @@ export function checkDiff(cwd, opts = {}) {
             // Extract imported name
             const nameMatch = imp.text.match(/import\s+(?:\{([^}]+)\}|(\w+))/);
             if (nameMatch) {
-                const names = (nameMatch[1] || nameMatch[2] || '').split(',').map(n => n.trim().split(' as ').pop()?.trim()).filter(Boolean);
+                const names = (nameMatch[1] || nameMatch[2] || '').split(',').map(n => n.trim().replace(/^type\s+/, '').split(' as ').pop()?.trim()).filter(Boolean);
                 for (const name of names) {
                     if (!name || name.length < 2)
                         continue;

package/dist/checks/history.js

@@ -66,7 +66,7 @@ export function checkHistory(cwd) {
     const aiPct = commits.length > 0 ? Math.round((aiCommits / commits.length) * 100) : 0;
     const infos = issues.filter(i => i.severity === 'info').length;
     const warnings = issues.filter(i => i.severity === 'warning').length;
-    const score = Math.max(0, Math.min(100, 100 - warnings * 10 - infos * 2));
+    const score = Math.max(0, Math.min(100, 100 - warnings * 10));
     return {
         name: 'history',
         score: Math.round(score),

package/dist/checks/integrity.js

@@ -304,8 +304,14 @@ function isNextjsServerComponent(file) {
     // Next.js app directory server components
     if (NEXTJS_SERVER_FILES.test(base))
         return true;
-    // Next.js API route handlers (app/api/)
-    if (normalized.includes('app/api/') && /^route\.[jt]sx?$/.test(base))
+    // Next.js route handlers (route.ts/js/tsx/jsx) anywhere in app/
+    if (/^route\.[jt]sx?$/.test(base))
+        return true;
+    // Any file in app/api/ directory
+    if (normalized.includes('app/api/'))
+        return true;
+    // Next.js middleware
+    if (/^middleware\.[jt]s$/.test(base))
         return true;
     return false;
 }

package/dist/checks/memory.js

@@ -193,6 +193,9 @@ export function checkMemory(cwd) {
         // 2. Broken path references
         const pathRefs = extractPaths(content);
         for (const { path: p, line } of pathRefs) {
+            // Skip ../  references — they point to sibling repos and can't be validated locally
+            if (p.startsWith('../'))
+                continue;
             const resolved = p.startsWith('/') ? p : resolve(cwd, p);
             if (!existsSync(resolved)) {
                 issues.push({

package/dist/checks/verify.js

@@ -147,8 +147,30 @@ function isPythonProject(cwd) {
 }
 /** Directories where small files are expected (examples, demos, docs) */
 const SMALL_FILE_DIRS = ['examples/', 'example/', 'demos/', 'demo/', 'docs/'];
+/** Next.js app router files that are designed to be small wrappers */
+const NEXTJS_APP_FILES = new Set([
+    'page.tsx', 'page.jsx', 'page.ts', 'page.js',
+    'layout.tsx', 'layout.jsx',
+    'loading.tsx', 'loading.jsx',
+    'not-found.tsx', 'not-found.jsx',
+    'error.tsx', 'error.jsx',
+    'template.tsx', 'template.jsx',
+]);
+function isNextjsAppFile(filePath) {
+    return NEXTJS_APP_FILES.has(basename(filePath));
+}
+/** Config files should never be flagged as test files */
+function isConfigFile(filePath) {
+    const base = basename(filePath);
+    return /\.config\.[a-z]+$/i.test(base);
+}
+/** Python pattern directories where small files are expected */
+const PYTHON_PATTERN_DIRS = ['profiles/', 'providers/', 'configs/', 'config/', 'tests/', 'test/'];
+/** Python pattern file names that are expected to be small */
+const PYTHON_PATTERN_NAMES = new Set(['version.py', '__version__.py', 'conftest.py']);
 function isPythonBoilerplate(filePath) {
     const base = basename(filePath);
+    const normalized = filePath.replace(/\\/g, '/');
     if (base === '__init__.py')
         return true;
     if (base === '__main__.py')
@@ -157,7 +179,20 @@ function isPythonBoilerplate(filePath) {
         return true;
     if (filePath.endsWith('.pyi'))
         return true;
-    if (filePath.replace(/\\/g, '/').includes('__pycache__/'))
+    if (normalized.includes('__pycache__/'))
+        return true;
+    // Pattern-based small Python files
+    if (PYTHON_PATTERN_NAMES.has(base))
+        return true;
+    if (base.endsWith('_utils.py'))
+        return true;
+    // Test files in test directories (test_*.py, *_test.py)
+    if (/^test_.*\.py$/.test(base) || /^.*_test\.py$/.test(base)) {
+        if (PYTHON_PATTERN_DIRS.some(d => normalized.includes(d)))
+            return true;
+    }
+    // Files in pattern directories (profiles/, providers/, configs/, config/)
+    if (base.endsWith('.py') && PYTHON_PATTERN_DIRS.some(d => normalized.includes(d)))
         return true;
     return false;
 }
@@ -263,6 +298,11 @@ export function checkVerify(cwd, since) {
             verified++;
             continue;
         }
+        // Skip thin file check for Next.js app router files (designed as small wrappers)
+        if (isNextjsAppFile(relPath)) {
+            verified++;
+            continue;
+        }
         if (lineCount < 10 && lineCount > 0) {
             issues.push({
                 severity: 'warning',
@@ -287,8 +327,8 @@ export function checkVerify(cwd, since) {
             failed++;
             continue;
         }
-        // 3. Test files must have actual assertions
-        if (isTestFile(relPath)) {
+        // 3. Test files must have actual assertions (but not config files)
+        if (isTestFile(relPath) && !isConfigFile(relPath)) {
             if (!hasAssertions(content)) {
                 issues.push({
                     severity: 'error',

package/dist/cli.js

@@ -250,13 +250,13 @@ if (isBadge && !isWatch) {
 }
 // --watch mode
 if (isWatch) {
-    console.clear();
-    let result = await runChecks();
-    console.log(reportPretty(result));
-    console.log(`  ${c.dim}watching for changes... (ctrl+c to stop)${c.reset}\n`);
-    let debounce = null;
-    const { watch } = await import('node:fs');
     try {
+        console.clear();
+        let result = await runChecks();
+        console.log(reportPretty(result));
+        console.log(`  ${c.dim}watching for changes... (ctrl+c to stop)${c.reset}\n`);
+        let debounce = null;
+        const { watch } = await import('node:fs');
         const watcher = watch(cwd, { recursive: true }, (event, filename) => {
             if (!filename)
                 return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@safetnsr/vet",
-  "version": "1.8.2",
+  "version": "1.8.4",
   "description": "vet your AI-generated code — one command, one score card, one letter grade",
   "type": "module",
   "bin": {