@safetnsr/vet 1.8.2 → 1.8.3

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/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.3",
   "description": "vet your AI-generated code — one command, one score card, one letter grade",
   "type": "module",
   "bin": {