@vibecodeqa/cli 0.16.0 → 0.18.0

package/dist/runners/standards.js

@@ -1,6 +1,7 @@
 /** Code standards check — naming conventions, anti-patterns, config hygiene. */
-import { readdirSync, readFileSync, statSync } from "node:fs";
+import { readFileSync } from "node:fs";
 import { basename, extname, join } from "node:path";
+import { getProductionFiles, readDeps } from "../fs-utils.js";
 import { gradeFromScore } from "../types.js";
 const CODE_SMELLS = [
     {
@@ -40,16 +41,7 @@ export function runStandards(cwd, stack) {
     const start = Date.now();
     const issues = [];
     // Collect source files
-    const files = [];
-    const dirs = ["src", "web/src"];
-    for (const dir of dirs) {
-        try {
-            collectFiles(join(cwd, dir), cwd, files);
-        }
-        catch {
-            /* dir doesn't exist */
-        }
-    }
+    const files = getProductionFiles(cwd);
     // ── File naming conventions ──
     let namingViolations = 0;
     for (const f of files) {
@@ -170,28 +162,3 @@ export function runStandards(cwd, stack) {
         duration: Date.now() - start,
     };
 }
-function collectFiles(dir, cwd, out) {
-    for (const entry of readdirSync(dir)) {
-        if (entry === "node_modules" || entry === "dist" || entry === ".git")
-            continue;
-        const full = join(dir, entry);
-        if (statSync(full).isDirectory()) {
-            collectFiles(full, cwd, out);
-        }
-        else {
-            const ext = extname(entry);
-            if ([".ts", ".tsx", ".js", ".jsx"].includes(ext) && !entry.includes(".test.") && !entry.includes(".spec.")) {
-                out.push({ path: full.replace(`${cwd}/`, ""), content: readFileSync(full, "utf-8") });
-            }
-        }
-    }
-}
-function readDeps(cwd) {
-    try {
-        const pkg = JSON.parse(readFileSync(join(cwd, "package.json"), "utf-8"));
-        return { ...pkg.dependencies, ...pkg.devDependencies };
-    }
-    catch {
-        return {};
-    }
-}

package/dist/runners/structure.js

@@ -1,19 +1,29 @@
 /** Project structure check — does the repo have standard files and conventions? */
-import { existsSync, readdirSync, readFileSync, statSync } from "node:fs";
-import { extname, join } from "node:path";
+import { existsSync, readFileSync } from "node:fs";
+import { join } from "node:path";
+import { collectSourceFiles } from "../fs-utils.js";
 import { gradeFromScore } from "../types.js";
-const EXPECTED_FILES = [
+const NODE_FILES = [
     { name: "package.json", path: "package.json", required: true, description: "Package manifest" },
     { name: "tsconfig.json", path: "tsconfig.json", required: false, description: "TypeScript configuration" },
     { name: "LICENSE", path: "LICENSE", required: true, description: "Open source license" },
     { name: ".gitignore", path: ".gitignore", required: true, description: "Git ignore rules" },
     { name: "README.md", path: "README.md", required: false, description: "Project documentation" },
 ];
+const DART_FILES = [
+    { name: "pubspec.yaml", path: "pubspec.yaml", required: true, description: "Dart package manifest" },
+    { name: "analysis_options.yaml", path: "analysis_options.yaml", required: true, description: "Dart analysis options" },
+    { name: "LICENSE", path: "LICENSE", required: true, description: "Open source license" },
+    { name: ".gitignore", path: ".gitignore", required: true, description: "Git ignore rules" },
+    { name: "README.md", path: "README.md", required: false, description: "Project documentation" },
+];
 export function runStructure(cwd, stack) {
     const start = Date.now();
     const issues = [];
     const found = [];
     const missing = [];
+    const isDart = stack.language === "dart";
+    const EXPECTED_FILES = isDart ? DART_FILES : NODE_FILES;
     // Check standard files
     for (const fc of EXPECTED_FILES) {
         // tsconfig is required only for TS projects
@@ -31,24 +41,24 @@ export function runStructure(cwd, stack) {
         }
     }
     // Check for lockfile
-    const hasLock = ["pnpm-lock.yaml", "package-lock.json", "yarn.lock", "bun.lockb"].some((f) => existsSync(join(cwd, f)));
+    const lockfiles = isDart ? ["pubspec.lock"] : ["pnpm-lock.yaml", "package-lock.json", "yarn.lock", "bun.lockb"];
+    const hasLock = lockfiles.some((f) => existsSync(join(cwd, f)));
     if (hasLock) {
         found.push("lockfile");
     }
     else {
         issues.push({ severity: "warning", message: "No lockfile found — builds may not be reproducible", rule: "missing-lockfile" });
     }
-    // Check for src directory
-    const hasSrc = existsSync(join(cwd, "src")) || existsSync(join(cwd, "web/src"));
+    // Check for source directory
+    const srcDirs = isDart ? ["lib"] : ["src", "web/src"];
+    const hasSrc = srcDirs.some((d) => existsSync(join(cwd, d)));
     if (!hasSrc) {
-        issues.push({ severity: "error", message: "No src/ directory found", rule: "no-src" });
+        issues.push({ severity: "error", message: `No ${srcDirs[0]}/ directory found`, rule: "no-src" });
     }
     // Count source vs test files
-    const srcFiles = [];
-    const testFiles = [];
-    collectAll(cwd, srcFiles, testFiles);
-    const srcCount = srcFiles.length;
-    const testCount = testFiles.length;
+    const allFiles = collectSourceFiles(cwd, { includeTests: true });
+    const srcCount = allFiles.filter((f) => !f.isTest).length;
+    const testCount = allFiles.filter((f) => f.isTest).length;
     const testRatio = srcCount > 0 ? testCount / srcCount : 0;
     if (testCount === 0 && srcCount > 0) {
         issues.push({ severity: "error", message: `No test files found (${srcCount} source files with zero tests)`, rule: "no-tests" });
@@ -60,17 +70,19 @@ export function runStructure(cwd, stack) {
             rule: "low-test-ratio",
         });
     }
-    // Check package.json has essential scripts
-    try {
-        const pkg = JSON.parse(readFileSync(join(cwd, "package.json"), "utf-8"));
-        const scripts = pkg.scripts || {};
-        if (!scripts.test)
-            issues.push({ severity: "warning", message: "No 'test' script in package.json", rule: "no-test-script" });
-        if (!scripts.build && !scripts.dev)
-            issues.push({ severity: "info", message: "No 'build' or 'dev' script in package.json", rule: "no-build-script" });
-    }
-    catch {
-        /* no package.json or parse error */
+    // Check manifest has essential config
+    if (!isDart) {
+        try {
+            const pkg = JSON.parse(readFileSync(join(cwd, "package.json"), "utf-8"));
+            const scripts = pkg.scripts || {};
+            if (!scripts.test)
+                issues.push({ severity: "warning", message: "No 'test' script in package.json", rule: "no-test-script" });
+            if (!scripts.build && !scripts.dev)
+                issues.push({ severity: "info", message: "No 'build' or 'dev' script in package.json", rule: "no-build-script" });
+        }
+        catch {
+            /* no package.json or parse error */
+        }
     }
     const errors = issues.filter((i) => i.severity === "error").length;
     const warnings = issues.filter((i) => i.severity === "warning").length;
@@ -84,35 +96,3 @@ export function runStructure(cwd, stack) {
         duration: Date.now() - start,
     };
 }
-function collectAll(cwd, src, test) {
-    const dirs = ["src", "web/src"];
-    for (const dir of dirs) {
-        try {
-            walk(join(cwd, dir), src, test);
-        }
-        catch {
-            /* dir doesn't exist */
-        }
-    }
-}
-function walk(dir, src, test) {
-    for (const entry of readdirSync(dir)) {
-        if (entry === "node_modules" || entry === "dist")
-            continue;
-        const full = join(dir, entry);
-        if (statSync(full).isDirectory()) {
-            walk(full, src, test);
-        }
-        else {
-            const ext = extname(entry);
-            if ([".ts", ".tsx", ".js", ".jsx"].includes(ext)) {
-                if (entry.includes(".test.") || entry.includes(".spec.")) {
-                    test.push(full);
-                }
-                else {
-                    src.push(full);
-                }
-            }
-        }
-    }
-}

package/dist/runners/testing.js

@@ -10,6 +10,7 @@
  */
 import { existsSync, readdirSync, readFileSync, statSync } from "node:fs";
 import { basename, extname, join } from "node:path";
+import { getProductionFiles, readDeps } from "../fs-utils.js";
 import { gradeFromScore } from "../types.js";
 import { run } from "./exec.js";
 // ── Classification rules ──
@@ -88,33 +89,7 @@ function walkTests(dir, cwd, out) {
     }
 }
 function findSourceFiles(cwd) {
-    const files = [];
-    const dirs = ["src", "web/src"];
-    for (const dir of dirs) {
-        try {
-            walkSource(join(cwd, dir), cwd, files);
-        }
-        catch {
-            /* dir doesn't exist */
-        }
-    }
-    return files;
-}
-function walkSource(dir, cwd, out) {
-    for (const entry of readdirSync(dir)) {
-        if (entry === "node_modules" || entry === "dist")
-            continue;
-        const full = join(dir, entry);
-        if (statSync(full).isDirectory()) {
-            walkSource(full, cwd, out);
-        }
-        else {
-            const ext = extname(entry);
-            if ([".ts", ".tsx", ".js", ".jsx"].includes(ext) && !entry.includes(".test.") && !entry.includes(".spec.")) {
-                out.push(full.replace(`${cwd}/`, ""));
-            }
-        }
-    }
+    return getProductionFiles(cwd).map((f) => f.path);
 }
 // ── Pairing analysis ──
 function computePairing(srcFiles, testFiles) {
@@ -151,15 +126,6 @@ function detectE2E(cwd) {
     }
     return { tool: "none", configured: false };
 }
-function readDeps(cwd) {
-    try {
-        const pkg = JSON.parse(readFileSync(join(cwd, "package.json"), "utf-8"));
-        return { ...pkg.dependencies, ...pkg.devDependencies };
-    }
-    catch {
-        return {};
-    }
-}
 // ── Coverage collection ──
 function collectCoverage(cwd, stack) {
     if (stack.testRunner === "none")

package/dist/runners/type-safety.d.ts

@@ -1,3 +1,3 @@
 /** Type safety check — count unsafe patterns: `as any`, explicit `any`, non-null assertions. */
 import type { CheckResult } from "../types.js";
-export declare function runTypeSafety(cwd: string): CheckResult;
+export declare function runTypeSafety(cwd: string, isDart?: boolean): CheckResult;

package/dist/runners/type-safety.js

@@ -1,8 +1,8 @@
 /** Type safety check — count unsafe patterns: `as any`, explicit `any`, non-null assertions. */
-import { readdirSync, readFileSync, statSync } from "node:fs";
-import { extname, join } from "node:path";
+import { getProductionFiles } from "../fs-utils.js";
 import { gradeFromScore } from "../types.js";
-const PATTERNS = [
+// TypeScript unsafe patterns
+const TS_PATTERNS = [
     { name: "as any", pattern: /\bas any\b/g, severity: "warning", weight: 2 },
     { name: ": any", pattern: /:\s*any\b/g, severity: "warning", weight: 1 },
     { name: "non-null assertion (!.)", pattern: /\w+!\./g, severity: "info", weight: 0.5 },
@@ -10,22 +10,22 @@ const PATTERNS = [
     { name: "@ts-expect-error", pattern: /@ts-expect-error/g, severity: "warning", weight: 2 },
     { name: "@ts-nocheck", pattern: /@ts-nocheck/g, severity: "error", weight: 10 },
 ];
-export function runTypeSafety(cwd) {
+// Dart unsafe patterns
+const DART_PATTERNS = [
+    { name: "dynamic type", pattern: /\bdynamic\b/g, severity: "warning", weight: 1 },
+    { name: "as dynamic", pattern: /\bas dynamic\b/g, severity: "warning", weight: 2 },
+    { name: "// ignore:", pattern: /\/\/\s*ignore:/g, severity: "error", weight: 5 },
+    { name: "// ignore_for_file:", pattern: /\/\/\s*ignore_for_file:/g, severity: "error", weight: 10 },
+    { name: "late keyword", pattern: /\blate\s+(?!final)/g, severity: "info", weight: 0.5 },
+];
+export function runTypeSafety(cwd, isDart = false) {
     const start = Date.now();
     const issues = [];
     const counts = {};
     let totalPenalty = 0;
-    const files = [];
-    const dirs = ["src", "web/src"];
-    for (const dir of dirs) {
-        try {
-            collectFiles(join(cwd, dir), files);
-        }
-        catch {
-            /* dir doesn't exist */
-        }
-    }
-    if (files.length === 0) {
+    const PATTERNS = isDart ? DART_PATTERNS : TS_PATTERNS;
+    const sourceFiles = getProductionFiles(cwd);
+    if (sourceFiles.length === 0) {
         return {
             name: "type-safety",
             score: 100,
@@ -35,10 +35,8 @@ export function runTypeSafety(cwd) {
             duration: Date.now() - start,
         };
     }
-    for (const file of files) {
-        const content = readFileSync(file, "utf-8");
-        const relPath = file.replace(`${cwd}/`, "");
-        const lines = content.split("\n");
+    for (const sf of sourceFiles) {
+        const lines = sf.content.split("\n");
         for (let i = 0; i < lines.length; i++) {
             const line = lines[i];
             const trimmed = line.trim();
@@ -53,7 +51,7 @@ export function runTypeSafety(cwd) {
                     counts[p.name] = (counts[p.name] || 0) + matches.length;
                     totalPenalty += p.weight * matches.length;
                     for (const _m of matches) {
-                        issues.push({ severity: p.severity, message: p.name, file: relPath, line: i + 1, rule: "unsafe-type" });
+                        issues.push({ severity: p.severity, message: p.name, file: sf.path, line: i + 1, rule: "unsafe-type" });
                     }
                 }
             }
@@ -64,24 +62,8 @@ export function runTypeSafety(cwd) {
         name: "type-safety",
         score,
         grade: gradeFromScore(score),
-        details: { ...counts, filesScanned: files.length, totalUnsafe: issues.length },
+        details: { ...counts, filesScanned: sourceFiles.length, totalUnsafe: issues.length },
         issues,
         duration: Date.now() - start,
     };
 }
-function collectFiles(dir, out) {
-    for (const entry of readdirSync(dir)) {
-        if (entry === "node_modules" || entry === "dist")
-            continue;
-        const full = join(dir, entry);
-        if (statSync(full).isDirectory()) {
-            collectFiles(full, out);
-        }
-        else {
-            const ext = extname(entry);
-            if ((ext === ".ts" || ext === ".tsx") && !entry.includes(".test.") && !entry.includes(".spec.")) {
-                out.push(full);
-            }
-        }
-    }
-}

package/dist/runners/types-check.d.ts

@@ -1,3 +1,3 @@
 /** TypeScript type checking runner. */
 import type { CheckResult } from "../types.js";
-export declare function runTypeCheck(cwd: string): CheckResult;
+export declare function runTypeCheck(cwd: string, isDart?: boolean): CheckResult;

package/dist/runners/types-check.js

@@ -3,33 +3,51 @@ import { existsSync } from "node:fs";
 import { join } from "node:path";
 import { gradeFromScore } from "../types.js";
 import { run } from "./exec.js";
-export function runTypeCheck(cwd) {
+export function runTypeCheck(cwd, isDart = false) {
     const start = Date.now();
     const issues = [];
-    if (!existsSync(join(cwd, "tsconfig.json")) && !existsSync(join(cwd, "tsconfig.app.json"))) {
-        return {
-            name: "types",
-            score: 0,
-            grade: "F",
-            details: { skipped: true, reason: "no tsconfig.json" },
-            issues: [],
-            duration: Date.now() - start,
-        };
-    }
-    const { stdout } = run("npx tsc --noEmit 2>&1 || true", cwd, 30_000);
-    const lines = stdout.split("\n");
-    for (const line of lines) {
-        const match = line.match(/^(.+)\((\d+),\d+\): error (TS\d+): (.+)/);
-        if (match) {
+    if (isDart) {
+        // Dart uses dart analyze for type checking — errors are type errors
+        const { stdout } = run("dart analyze --format=machine 2>/dev/null || true", cwd, 30_000);
+        for (const line of stdout.split("\n")) {
+            const parts = line.split("|");
+            if (parts.length < 8 || parts[0] !== "ERROR")
+                continue;
             issues.push({
                 severity: "error",
-                file: match[1],
-                line: parseInt(match[2], 10),
-                rule: match[3],
-                message: match[4],
+                file: parts[3],
+                line: parseInt(parts[4], 10) || undefined,
+                rule: parts[2],
+                message: parts[7],
             });
         }
     }
+    else {
+        if (!existsSync(join(cwd, "tsconfig.json")) && !existsSync(join(cwd, "tsconfig.app.json"))) {
+            return {
+                name: "types",
+                score: 0,
+                grade: "F",
+                details: { skipped: true, reason: "no tsconfig.json" },
+                issues: [],
+                duration: Date.now() - start,
+            };
+        }
+        const { stdout } = run("npx tsc --noEmit 2>&1 || true", cwd, 30_000);
+        const lines = stdout.split("\n");
+        for (const line of lines) {
+            const match = line.match(/^(.+)\((\d+),\d+\): error (TS\d+): (.+)/);
+            if (match) {
+                issues.push({
+                    severity: "error",
+                    file: match[1],
+                    line: parseInt(match[2], 10),
+                    rule: match[3],
+                    message: match[4],
+                });
+            }
+        }
+    }
     const errorCount = issues.length;
     const score = errorCount === 0 ? 100 : Math.max(0, 100 - errorCount * 5);
     return {

package/dist/types.d.ts

@@ -30,11 +30,11 @@ export interface VibeReport {
     };
 }
 export interface StackInfo {
-    language: "typescript" | "javascript" | "unknown";
-    framework: "react" | "vue" | "svelte" | "none" | "unknown";
+    language: "typescript" | "javascript" | "dart" | "unknown";
+    framework: "react" | "vue" | "svelte" | "flutter" | "none" | "unknown";
     bundler: "vite" | "webpack" | "esbuild" | "none" | "unknown";
-    testRunner: "vitest" | "jest" | "none" | "unknown";
-    linter: "biome" | "eslint" | "none" | "unknown";
-    packageManager: "pnpm" | "npm" | "yarn" | "bun" | "unknown";
+    testRunner: "vitest" | "jest" | "flutter_test" | "dart_test" | "none" | "unknown";
+    linter: "biome" | "eslint" | "dart_analyze" | "none" | "unknown";
+    packageManager: "pnpm" | "npm" | "yarn" | "bun" | "pub" | "unknown";
 }
 export declare function gradeFromScore(score: number): "A" | "B" | "C" | "D" | "F";

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@vibecodeqa/cli",
-  "version": "0.16.0",
-  "description": "Code health scanner for the AI coding era. 15 checks, zero config, full report.",
+  "version": "0.18.0",
+  "description": "Code health scanner for the AI coding era. 21 checks, zero config, full report.",
   "type": "module",
   "bin": {
     "vcqa": "./dist/cli.js",
@@ -12,6 +12,13 @@
     "LICENSE",
     "README.md"
   ],
+  "scripts": {
+    "build": "tsc",
+    "dev": "tsc --watch",
+    "test": "vitest run",
+    "lint": "biome check src/",
+    "prepublishOnly": "tsc && chmod +x dist/cli.js"
+  },
   "keywords": [
     "code-quality",
     "code-health",
@@ -32,7 +39,7 @@
   "license": "MIT",
   "repository": {
     "type": "git",
-    "url": "https://github.com/freeappstore-online/vibe-check"
+    "url": "https://github.com/vibecodeqa/cli"
   },
   "homepage": "https://vibecodeqa.online",
   "engines": {
@@ -43,11 +50,5 @@
     "@types/node": "^25.8.0",
     "typescript": "^5.8.3",
     "vitest": "^4.1.6"
-  },
-  "scripts": {
-    "build": "tsc",
-    "dev": "tsc --watch",
-    "test": "vitest run",
-    "lint": "biome check src/"
   }
-}
+}