@vibecodeqa/cli 0.37.4 → 0.38.1

package/dist/cli.js

@@ -227,7 +227,7 @@ async function writeOutputs(report, outputDir, flags) {
         writeFileSync(join(outputDir, "report.sarif"), generateSARIF(report));
     }
 }
-async function printResults(report, trend, flags, outputDir) {
+async function printResults(report, trend, flags, outputDir, interactive) {
     const { score, grade, checks } = report;
     const totalIssues = checks.reduce((s, c) => s + c.issues.length, 0);
     if (flags.jsonOnly) {
@@ -248,15 +248,10 @@ async function printResults(report, trend, flags, outputDir) {
             console.log(formatTrend(trend, scores));
         }
         console.log("");
-        console.log(`  \x1b[2mReport: ${join(outputDir, "report/index.html")}\x1b[0m`);
-        console.log(`  \x1b[2mJSON:   ${join(outputDir, "report.json")}\x1b[0m`);
-        if (flags.badgeMode)
-            console.log(`  \x1b[2mBadge:  ${join(outputDir, "badge.svg")}\x1b[0m`);
-        if (flags.sarifMode)
-            console.log(`  \x1b[2mSARIF:  ${join(outputDir, "report.sarif")}\x1b[0m`);
-        console.log("");
-        // Top actionable issues
-        if (flags.topN > 0) {
+        // Top actionable issues. Explicit --top wins; otherwise show a few by default
+        // in an interactive terminal so the scan never dead-ends at a file path.
+        const effectiveTopN = flags.topN > 0 ? flags.topN : interactive ? 3 : 0;
+        if (effectiveTopN > 0) {
             const allIssues = checks.flatMap((c) => c.issues.map((iss) => ({ check: c.name, weight: getCheckMeta(c.name).weight, ...iss })));
             // Sort by: errors first, then by check weight (highest-impact first)
             allIssues.sort((a, b) => {
@@ -266,9 +261,11 @@ async function printResults(report, trend, flags, outputDir) {
                     return sevDiff;
                 return b.weight - a.weight;
             });
-            const top = allIssues.slice(0, flags.topN);
+            const top = allIssues.slice(0, effectiveTopN);
             if (top.length > 0) {
-                console.log(`  \x1b[1mTop ${top.length} issues to fix:\x1b[0m`);
+                const more = allIssues.length - top.length;
+                const moreStr = more > 0 ? ` \x1b[2m(+${more} more)\x1b[0m` : "";
+                console.log(`  \x1b[1mTop ${top.length} issues to fix:\x1b[0m${moreStr}`);
                 for (const iss of top) {
                     const sevColor = iss.severity === "error" ? "\x1b[31m" : iss.severity === "warning" ? "\x1b[33m" : "\x1b[2m";
                     const sevChar = iss.severity[0].toUpperCase();
@@ -278,6 +275,37 @@ async function printResults(report, trend, flags, outputDir) {
                 console.log("");
             }
         }
+        // Next steps: surface the weakest scored dimensions and how to dig into each.
+        // Skipped in CI (clean machine-readable-ish output) — interactive runs get the on-ramp.
+        if (!flags.ciMode) {
+            const weakest = checks
+                .filter((c) => {
+                const det = c.details;
+                return !det.skipped && !det.comingSoon && c.score < 70;
+            })
+                .sort((a, b) => a.score - b.score)
+                .slice(0, 3);
+            if (weakest.length > 0) {
+                console.log("  \x1b[1mWeakest areas:\x1b[0m");
+                for (const c of weakest) {
+                    const gc = color(c.grade);
+                    const label = getCheckMeta(c.name).label || c.name;
+                    console.log(`  ${gc}${c.grade}\x1b[0m \x1b[2m${String(c.score).padStart(3)}\x1b[0m  ${label.padEnd(18)}\x1b[2m→ vcqa explain ${c.name}\x1b[0m`);
+                }
+                console.log("");
+            }
+        }
+        // Report paths + the interactive on-ramp.
+        console.log(`  \x1b[2mReport: ${join(outputDir, "report/index.html")}\x1b[0m`);
+        console.log(`  \x1b[2mJSON:   ${join(outputDir, "report.json")}\x1b[0m`);
+        if (flags.badgeMode)
+            console.log(`  \x1b[2mBadge:  ${join(outputDir, "badge.svg")}\x1b[0m`);
+        if (flags.sarifMode)
+            console.log(`  \x1b[2mSARIF:  ${join(outputDir, "report.sarif")}\x1b[0m`);
+        if (!interactive && !flags.ciMode) {
+            console.log(`  \x1b[2mExplore: \x1b[0m\x1b[1mvcqa monitor\x1b[0m\x1b[2m — live TUI to drill into issues & copy fix-prompts\x1b[0m`);
+        }
+        console.log("");
     }
 }
 async function handleUpload(report, cwd, jsonOnly) {
@@ -866,11 +894,14 @@ async function main() {
     };
     const trend = computeTrend(report, outputDir);
     await writeOutputs(report, outputDir, flags);
+    // Interactive = a real terminal session (not piped, CI, JSON, or watch). Gates the
+    // post-scan prompt and the default top-issues view.
+    const interactive = Boolean(process.stdin.isTTY && process.stdout.isTTY) && !quietMode && !ciMode && !watchMode;
     if (flags.markdownMode) {
         console.log(generateMarkdown(report, trend));
     }
     else {
-        await printResults(report, trend, flags, outputDir);
+        await printResults(report, trend, flags, outputDir, interactive);
     }
     if (flags.annotations) {
         emitAnnotations(report);
@@ -900,7 +931,62 @@ async function main() {
     }
     if (watchMode) {
         await startWatch(cwd);
+        return;
+    }
+    // Interactive on-ramp: offer to open the live monitor or the HTML report.
+    if (interactive && !flags.uploadMode && !flags.prComment) {
+        await promptNextAction(cwd, outputDir);
+    }
+}
+/** Read a single keypress from a TTY, restoring stdin state afterward. */
+function readKey() {
+    return new Promise((resolve) => {
+        const stdin = process.stdin;
+        const wasRaw = stdin.isRaw;
+        stdin.setRawMode?.(true);
+        stdin.resume();
+        stdin.once("data", (buf) => {
+            stdin.setRawMode?.(wasRaw ?? false);
+            stdin.pause();
+            resolve(buf.toString("utf-8"));
+        });
+    });
+}
+/** Open a file/URL with the OS default handler (detached). */
+function openPath(target) {
+    const cmd = process.platform === "darwin" ? "open" : process.platform === "win32" ? "start" : "xdg-open";
+    import("node:child_process").then(({ spawn }) => {
+        try {
+            spawn(cmd, [target], { detached: true, stdio: "ignore", shell: process.platform === "win32" }).unref();
+        }
+        catch {
+            /* opening is best-effort */
+        }
+    });
+}
+/** Post-scan prompt: [m] monitor · [o] open report · anything else quits. */
+async function promptNextAction(cwd, outputDir) {
+    process.stdout.write("  \x1b[1m[m]\x1b[0m\x1b[2m monitor\x1b[0m   \x1b[1m[o]\x1b[0m\x1b[2m open report\x1b[0m   \x1b[1m[enter]\x1b[0m\x1b[2m quit\x1b[0m  ");
+    let key;
+    try {
+        key = await readKey();
+    }
+    catch {
+        process.stdout.write("\n");
+        return;
+    }
+    process.stdout.write("\n");
+    const k = key.toLowerCase();
+    if (k === "m") {
+        const { startMonitor } = await import("./monitor.js");
+        await startMonitor(cwd);
+    }
+    else if (k === "o") {
+        const reportPath = join(outputDir, "report/index.html");
+        openPath(reportPath);
+        console.log(`  \x1b[2mOpening ${reportPath}\x1b[0m`);
     }
+    // any other key (enter, q, ctrl-c, …) → quit
 }
 async function checkForUpdate(currentVersion) {
     try {

package/dist/monitor.js

@@ -425,6 +425,9 @@ function MonitorApp({ cwd }) {
         { time: ts(), text: cached ? `Loaded cached scan: ${cached.grade} ${cached.score}/100` : `Monitoring ${basename(cwd)}...`, type: cached ? "scan" : "info" },
     ]);
     const [copied, setCopied] = useState(false);
+    const [showHelp, setShowHelp] = useState(false);
+    const [search, setSearch] = useState("");
+    const [searchActive, setSearchActive] = useState(false);
     const scanningRef = useRef(false);
     const prevScoreRef = useRef(cached ? cached.score : null);
     const addLog = useCallback((text, type = "info") => {
@@ -438,7 +441,16 @@ function MonitorApp({ cwd }) {
         const o = { error: 0, warning: 1, info: 2 };
         return (o[a.severity] ?? 2) - (o[b.severity] ?? 2);
     }), [state.checks]);
-    const currentList = panel === "checks" ? activeChecks : allIssues;
+    // Issues filtered by the dashboard search query (matches message / check / file).
+    const displayIssues = useMemo(() => {
+        if (!search)
+            return allIssues;
+        const q = search.toLowerCase();
+        return allIssues.filter((i) => i.message.toLowerCase().includes(q) ||
+            i.check.toLowerCase().includes(q) ||
+            (typeof i.file === "string" && i.file.toLowerCase().includes(q)));
+    }, [allIssues, search]);
+    const currentList = panel === "checks" ? activeChecks : displayIssues;
     // Derived data for file views (memoized, used by both render + keyboard)
     const filesWithIssues = useMemo(() => {
         const map = new Map();
@@ -529,13 +541,57 @@ function MonitorApp({ cwd }) {
     }, [pendingCfg, monCfg]);
     // ── Keyboard — single handler for all views ──
     useInput((input, key) => {
+        // Ctrl-C always quits, even mid-search
+        if (key.ctrl && input === "c") {
+            exit();
+            return;
+        }
+        // Help overlay swallows all input until dismissed
+        if (showHelp) {
+            if (input === "?" || input === "q" || key.escape || key.return)
+                setShowHelp(false);
+            return;
+        }
+        // Search input mode (filters the dashboard Issues panel)
+        if (searchActive) {
+            if (key.escape) {
+                setSearch("");
+                setSearchActive(false);
+                setCursor(0);
+                return;
+            }
+            if (key.return) {
+                setSearchActive(false);
+                return;
+            }
+            if (key.backspace || key.delete) {
+                setSearch((s) => s.slice(0, -1));
+                setCursor(0);
+                return;
+            }
+            if (input && input.length === 1 && input >= " " && !key.ctrl && !key.meta) {
+                setSearch((s) => s + input);
+                setCursor(0);
+            }
+            return;
+        }
         // Quit from anywhere
-        if (input === "q" || (key.ctrl && input === "c")) {
+        if (input === "q") {
             exit();
             return;
         }
+        // Help overlay: ? opens from any view
+        if (input === "?") {
+            setShowHelp(true);
+            return;
+        }
         // Esc: drill up one level or quit
         if (key.escape) {
+            if (mode.view === "dashboard" && search) {
+                setSearch("");
+                setCursor(0);
+                return;
+            }
             if (mode.view === "config") {
                 setPendingCfg(null);
                 setMode({ view: "dashboard" });
@@ -634,6 +690,12 @@ function MonitorApp({ cwd }) {
         }
         // ── Dashboard navigation ──
         if (mode.view === "dashboard") {
+            if (input === "/") {
+                setSearchActive(true);
+                setPanel("issues");
+                setCursor(0);
+                return;
+            }
             if (key.tab) {
                 setPanel((p) => p === "checks" ? "issues" : "checks");
                 setCursor(0);
@@ -648,8 +710,8 @@ function MonitorApp({ cwd }) {
                 setCursor(0);
                 return;
             }
-            if (key.return && panel === "issues" && allIssues[cursor]) {
-                setMode({ view: "check-detail", checkName: allIssues[cursor].check });
+            if (key.return && panel === "issues" && displayIssues[cursor]) {
+                setMode({ view: "check-detail", checkName: displayIssues[cursor].check });
                 setCursor(0);
                 return;
             }
@@ -740,6 +802,9 @@ function MonitorApp({ cwd }) {
     const proj = basename(cwd);
     const p = monCfg.panels;
     // ── Render views ──
+    if (showHelp) {
+        return (_jsxs(Box, { flexDirection: "column", height: rows, children: [_jsx(Header, { proj: proj, stack: stack, workspace: workspace, state: state }), _jsx(HelpOverlay, { height: rows - 3 }), _jsx(Box, { paddingX: 1, children: _jsx(Text, { dimColor: true, children: "? or Esc to close" }) })] }));
+    }
     if (mode.view === "all-files") {
         return (_jsxs(Box, { flexDirection: "column", height: rows, children: [_jsx(Header, { proj: proj, stack: stack, workspace: workspace, state: state }), _jsx(AllFilesView, { checks: state.checks, height: rows - 3, cursor: cursor }), _jsx(Box, { paddingX: 1, children: _jsx(Text, { dimColor: true, children: "Esc back \u00B7 \u2191\u2193 select \u00B7 Enter view file issues \u00B7 q quit" }) })] }));
     }
@@ -781,10 +846,42 @@ function MonitorApp({ cwd }) {
     return (_jsxs(Box, { flexDirection: "column", height: rows, children: [_jsx(Header, { proj: proj, stack: stack, workspace: workspace, state: state }), _jsx(Box, { paddingX: 1, gap: 2, height: 1, children: state.scanCount > 0 && (_jsxs(_Fragment, { children: [_jsxs(Text, { children: [_jsx(Text, { dimColor: true, children: "E " }), _jsx(Text, { color: "red", bold: true, children: errorCount })] }), _jsxs(Text, { children: [_jsx(Text, { dimColor: true, children: "W " }), _jsx(Text, { color: "yellow", bold: true, children: warnCount })] }), _jsxs(Text, { children: [_jsx(Text, { dimColor: true, children: "checks " }), _jsx(Text, { bold: true, children: activeChecks.length })] }), _jsxs(Text, { children: [_jsx(Text, { dimColor: true, children: "scan " }), _jsxs(Text, { children: [state.duration, "ms"] })] }), state.scores.length >= 2 && _jsx(Text, { color: "cyan", children: spark(state.scores) })] })) }), _jsxs(Box, { height: bodyRows, children: [sidebarVisible && (_jsxs(Box, { flexDirection: "column", width: 26, children: [p.score && _jsx(ScorePanel, { state: state, height: scoreH }), p.checks && (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: panel === "checks" ? "magenta" : "gray", paddingX: 1, width: 24, height: checksH, overflowY: "hidden", children: [_jsxs(Text, { bold: true, color: "magenta", children: [" \u25C8 Checks ", panel === "checks" && _jsx(Text, { dimColor: true, children: "\u25C4" })] }), activeChecks.slice(0, checksH - 3).map((c, i) => {
                                         const sel = panel === "checks" && i === cursor;
                                         return (_jsxs(Text, { children: [_jsxs(Text, { color: sel ? "white" : gc(c.grade), children: [sel ? "▸" : " ", c.grade === "A" ? "●" : c.grade === "B" ? "◐" : "○", " "] }), _jsx(Text, { bold: sel, children: c.name.slice(0, 13).padEnd(13) }), _jsx(Text, { color: gc(c.grade), children: String(c.score).padStart(3) })] }, c.name));
-                                    })] }))] })), mainVisible && (_jsxs(Box, { flexDirection: "column", flexGrow: 1, children: [p.activity && _jsx(ActivityPanel, { log: log, height: activityH }), p.issues && (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: panel === "issues" ? "magenta" : "gray", paddingX: 1, height: issuesH, overflowY: "hidden", children: [_jsxs(Text, { bold: true, color: "magenta", children: [" ", "\u25C8 Issues (", allIssues.length, ") ", panel === "issues" && _jsx(Text, { dimColor: true, children: "\u25C4" })] }), allIssues.slice(0, issuesH - 3).map((iss, i) => {
+                                    })] }))] })), mainVisible && (_jsxs(Box, { flexDirection: "column", flexGrow: 1, children: [p.activity && _jsx(ActivityPanel, { log: log, height: activityH }), p.issues && (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: panel === "issues" ? "magenta" : "gray", paddingX: 1, height: issuesH, overflowY: "hidden", children: [_jsxs(Text, { bold: true, color: "magenta", children: [" ", "\u25C8 Issues (", displayIssues.length, search ? `/${allIssues.length}` : "", ") ", panel === "issues" && _jsx(Text, { dimColor: true, children: "\u25C4" })] }), (searchActive || search) && (_jsxs(Text, { children: [" ", _jsx(Text, { dimColor: true, children: "/" }), _jsx(Text, { color: searchActive ? "white" : "cyan", children: search }), searchActive && _jsx(Text, { color: "white", children: "\u258C" })] })), displayIssues.length === 0 && (_jsxs(Text, { color: search ? "yellow" : "green", children: [" ", search ? "No issues match." : state.scanCount > 0 ? "✓ No issues — clean scan!" : "Scanning…"] })), displayIssues.slice(0, issuesH - 3).map((iss, i) => {
                                         const sel = panel === "issues" && i === cursor;
                                         return (_jsxs(Text, { wrap: "truncate", children: [_jsx(Text, { color: sel ? "white" : "gray", children: sel ? "▸" : " " }), _jsxs(Text, { color: sc(iss.severity), bold: true, children: [iss.severity[0].toUpperCase(), " "] }), _jsxs(Text, { dimColor: true, children: [(iss.check || "").slice(0, 11).padEnd(11), " "] }), iss.file && _jsxs(Text, { color: "cyan", children: [basename(String(iss.file)).slice(0, 18).padEnd(18), " "] }), _jsx(Text, { children: iss.message.slice(0, 40) })] }, `${iss.check}-${iss.file || ""}-${iss.line || i}`));
-                                    }), allIssues.length > issuesH - 3 && _jsxs(Text, { dimColor: true, children: [" +", allIssues.length - (issuesH - 3), " more"] })] }))] })), !sidebarVisible && !mainVisible && (_jsx(Box, { height: bodyRows, justifyContent: "center", alignItems: "center", children: _jsx(Text, { dimColor: true, children: "All panels hidden. Press c to configure." }) }))] }), _jsx(Box, { paddingX: 1, justifyContent: "space-between", children: _jsx(Text, { dimColor: true, children: "Tab panel \u00B7 \u2191\u2193 Enter Esc \u00B7 r scan \u00B7 f files \u00B7 g git \u00B7 t trends \u00B7 c config \u00B7 q" }) })] }));
+                                    }), displayIssues.length > issuesH - 3 && _jsxs(Text, { dimColor: true, children: [" +", displayIssues.length - (issuesH - 3), " more"] })] }))] })), !sidebarVisible && !mainVisible && (_jsx(Box, { height: bodyRows, justifyContent: "center", alignItems: "center", children: _jsx(Text, { dimColor: true, children: "All panels hidden. Press c to configure." }) }))] }), _jsx(Box, { paddingX: 1, justifyContent: "space-between", children: _jsx(Text, { dimColor: true, children: "Tab \u00B7 \u2191\u2193 Enter Esc \u00B7 / search \u00B7 r scan \u00B7 f files \u00B7 g git \u00B7 t trends \u00B7 c config \u00B7 ? help \u00B7 q" }) })] }));
+}
+function HelpOverlay({ height }) {
+    const groups = [
+        {
+            title: "Navigate",
+            keys: [
+                ["↑ ↓", "move selection"],
+                ["Enter", "drill in (check → issue → source)"],
+                ["Tab", "switch Checks / Issues panel"],
+                ["Esc", "back up one level (or clear search)"],
+                ["q · Ctrl-C", "quit"],
+            ],
+        },
+        {
+            title: "Views",
+            keys: [
+                ["r", "re-scan now"],
+                ["f", "all files by issue count"],
+                ["g", "git-changed files"],
+                ["t", "score trends"],
+                ["c", "config (thresholds, panels)"],
+            ],
+        },
+        {
+            title: "Issues",
+            keys: [
+                ["/", "search / filter issues"],
+                ["y", "copy an AI fix-prompt to clipboard"],
+            ],
+        },
+    ];
+    return (_jsxs(Box, { flexDirection: "column", height: height, paddingX: 2, paddingY: 1, overflowY: "hidden", children: [_jsx(Text, { bold: true, color: "magenta", children: "\u25C8 Keyboard shortcuts" }), _jsx(Text, { children: " " }), groups.map((g) => (_jsxs(Box, { flexDirection: "column", marginBottom: 1, children: [_jsx(Text, { bold: true, color: "cyan", children: g.title }), g.keys.map(([k, desc]) => (_jsxs(Text, { children: [_jsx(Text, { color: "white", bold: true, children: `  ${k}`.padEnd(16) }), _jsx(Text, { dimColor: true, children: desc })] }, k)))] }, g.title)))] }));
 }
 function Header({ proj, stack, workspace, state }) {
     return (_jsxs(Box, { paddingX: 1, justifyContent: "space-between", children: [_jsxs(Text, { children: [_jsx(Text, { color: "magenta", bold: true, children: "vcqa monitor" }), _jsxs(Text, { dimColor: true, children: [" v", VERSION] })] }), _jsxs(Text, { children: [_jsx(Text, { bold: true, children: proj }), _jsxs(Text, { dimColor: true, children: [" ", stack.language, "/", stack.framework, workspace.isMonorepo ? ` · ${workspace.tool}` : ""] })] }), _jsxs(Text, { children: [state.score > 0 && _jsxs(Text, { color: gc(state.grade), bold: true, children: [state.grade, " ", state.score, " "] }), _jsx(Text, { dimColor: true, children: state.scanning ? "⟳ scanning" : "● watching" })] })] }));

package/dist/runners/react.js

@@ -3,6 +3,30 @@
  *  This runner catches patterns beyond what the plugin covers. */
 import { getProductionFiles, readDeps } from "../fs-utils.js";
 import { gradeFromScore } from "../types.js";
+/** True when the `.map()` callback starting at line `i` actually returns JSX.
+ *  `after` is the line text from `.map(` onward, right-trimmed. Distinguishes JSX
+ *  returns from data maps (`=> ({...})`, `=> fn(...)`), TS generics, and comparisons. */
+function mapCallbackReturnsJsx(after, lines, i) {
+    if (/=>\s*<[A-Za-z]/.test(after))
+        return true; // inline: => <Tag
+    if (/=>\s*\($/.test(after)) {
+        // multiline arrow body: => (   followed by JSX on the next non-empty line
+        return /^<[A-Za-z]/.test((lines[i + 1] || "").trim());
+    }
+    if (/=>\s*\{$/.test(after)) {
+        // block body: JSX iff it returns `<Tag` or `(` then `<Tag`
+        for (let j = i; j < Math.min(i + 12, lines.length); j++) {
+            const lt = (lines[j] || "").trim();
+            if (/return\s*<[A-Za-z]/.test(lt))
+                return true;
+            if (/return\s*\($/.test(lt))
+                return /^<[A-Za-z]/.test((lines[j + 1] || "").trim());
+            if (j > i && /^return\b/.test(lt))
+                return false; // returns a non-JSX value
+        }
+    }
+    return false;
+}
 export function runReact(cwd, stack) {
     const start = Date.now();
     if (stack.framework !== "react") {
@@ -68,11 +92,13 @@ export function runReact(cwd, stack) {
                     rule: "conditional-hook",
                 });
             }
-            // 2. Missing key in .map() returning JSX
-            if (/\.map\s*\(/.test(trimmed)) {
-                // Look ahead for JSX return without key
-                const mapBlock = lines.slice(i, Math.min(i + 10, lines.length)).join("\n");
-                if (/<\w/.test(mapBlock) && !mapBlock.includes("key=") && !mapBlock.includes("key:")) {
+            // 2. Missing key in .map() returning JSX. Only flag genuine JSX returns —
+            // not data maps, TS generics, or comparisons (see mapCallbackReturnsJsx).
+            const mapIdx = trimmed.indexOf(".map(");
+            if (mapIdx !== -1 && mapCallbackReturnsJsx(trimmed.slice(mapIdx).trimEnd(), lines, i)) {
+                // Inspect just the JSX head for a key — enough to cover the opening element.
+                const head = lines.slice(i, Math.min(i + 8, lines.length)).join("\n");
+                if (!head.includes("key=") && !head.includes("key:")) {
                     missingKeys++;
                     issues.push({ severity: "error", message: "JSX in .map() without key prop", file: f.path, line: i + 1, rule: "missing-key" });
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vibecodeqa/cli",
-	"version": "0.37.4",
+	"version": "0.38.1",
 	"description": "Code health scanner for the AI coding era. 25 checks, zero config, full report.",
 	"type": "module",
 	"bin": {