svelte-vitals 0.44.2 → 0.44.3

package/dist/bin.js

@@ -8,7 +8,7 @@ import {
   readPackageVersion,
   readPkg,
   run
-} from "./chunk-EWDGS52A.js";
+} from "./chunk-NIUYLAJK.js";
 import {
   consoleIO,
   parseCliArgs,
@@ -1429,7 +1429,9 @@ function realIO() {
       writeFileSync(path, content);
     },
     cwd: process.cwd(),
-    isTTY: Boolean(process.stdout.isTTY),
+    // clack reads from stdin and renders to stdout, so both must be interactive —
+    // a piped/redirected stdin would leave the prompt hanging for input that never comes.
+    isTTY: Boolean(process.stdin.isTTY) && Boolean(process.stdout.isTTY),
     nodeVersion: process.version,
     log: (line) => console.log(line),
     errorLog: (line) => console.error(line),

package/dist/{chunk-EWDGS52A.js → chunk-NIUYLAJK.js}

@@ -773,7 +773,7 @@ async function collectRoutes(rt, cwd, config = defaultConfig, cache = /* @__PURE
 // src/route-matcher.ts
 function routeMatcher(glob2) {
   if (!glob2) return () => true;
-  const body = glob2.replace(/[.+^${}()|[\]\\]/g, "\\$&").replace(/\*\*/g, " ").replace(/\*/g, "[^/]*").replace(/\/ $/g, "(?:/.*)?").replace(/^ \//g, "(?:.*/)?").replace(/ \//g, "(?:.*/)?").replace(/\/ /g, "(?:/.*)?").replace(/ /g, ".*");
+  const body = glob2.replace(/[.+^${}()|[\]\\]/g, "\\$&").replace(/\*\*/g, "\0").replace(/\*/g, "[^/]*").replace(/\/\0$/g, "(?:/.*)?").replace(/^\0\//g, "(?:.*/)?").replace(/\0\//g, "(?:.*/)?").replace(/\/\0/g, "(?:/.*)?").replace(/\0/g, ".*");
   const re = new RegExp(`^${body}$`);
   return (route) => re.test(route.replace(/^\//, ""));
 }
@@ -781,14 +781,23 @@ function routeMatcher(glob2) {
 // src/collect-all.ts
 async function collectAll(rt, cwd, config, opts = {}) {
   const matches = routeMatcher(opts.route);
-  const collected = await collectRoutes(rt, cwd, config, opts.parseCache);
+  const project = await collectProjectFacts(rt, cwd);
+  const [collected, components, kitModules, sourceFiles] = await Promise.all([
+    collectRoutes(rt, cwd, config, opts.parseCache),
+    // Component (Correctness) facts are file-scoped with no route attribution yet, so a
+    // route-filtered run skips them rather than reporting unrelated components (#68 review);
+    // kitModules is skipped for the same reason.
+    opts.route ? [] : collectComponentFacts(rt, cwd),
+    opts.route ? [] : collectKitModuleFacts(rt, cwd, project.kitAliases),
+    // Unlike its two neighbours above, the --route branch gets `undefined` here, not `[]`: an empty
+    // inventory would tell architecture/unit-entry-file that the declared unit directories truly do
+    // not exist, so it would report every declaration as inert, whereas `undefined` means the mode
+    // never collected the fact at all, and the rule stays silent instead of raising a false alarm.
+    opts.route ? void 0 : collectSourceFiles(rt, cwd)
+  ]);
   const heads = collected.heads.filter((h) => matches(h.route));
   const images = collected.images.filter((i) => matches(i.route));
   const headings = collected.headings.filter((h) => matches(h.route));
-  const project = await collectProjectFacts(rt, cwd);
-  const components = opts.route ? [] : await collectComponentFacts(rt, cwd);
-  const kitModules = opts.route ? [] : await collectKitModuleFacts(rt, cwd, project.kitAliases);
-  const sourceFiles = opts.route ? void 0 : await collectSourceFiles(rt, cwd);
   return { heads, images, headings, project, components, kitModules, sourceFiles };
 }
 
@@ -978,7 +987,7 @@ function filterToNewFindings(results, baselineResults, config = defaultConfig3)
 }
 
 // src/suppressions.ts
-import { readFileSync as readFileSync2, writeFileSync } from "fs";
+import { readFileSync as readFileSync2, renameSync, unlinkSync, writeFileSync } from "fs";
 import { join as join6 } from "path";
 import { isPenalized as isPenalized3 } from "@svelte-vitals/core";
 
@@ -1221,7 +1230,17 @@ function writeSuppressions(cwd, results, config) {
   }
   entries.sort(compareEntries);
   const path = join6(cwd, SUPPRESSIONS_FILE);
-  writeFileSync(path, JSON.stringify({ version: 1, suppressions: entries }, null, 2) + "\n");
+  const tmpPath = `${path}.tmp`;
+  try {
+    writeFileSync(tmpPath, JSON.stringify({ version: 1, suppressions: entries }, null, 2) + "\n");
+    renameSync(tmpPath, path);
+  } catch (err) {
+    try {
+      unlinkSync(tmpPath);
+    } catch {
+    }
+    throw err;
+  }
   return entries.length;
 }
 function applySuppressions(results, entries, config, allResults) {
@@ -1540,6 +1559,16 @@ function overridesOffWarnings(allowRules, overrides) {
   }
   return warnings;
 }
+function skippedFileWarnings(facts) {
+  const files = [...new Set(facts.filter((f) => f.parseFailed).map((f) => f.file))].sort();
+  if (files.length === 0) return [];
+  const shown = files.slice(0, 10);
+  const list = files.length > shown.length ? `${shown.join(", ")}, \u2026 and ${files.length - shown.length} more` : shown.join(", ");
+  return [
+    `skipped ${files.length} file(s) that could not be parsed: ${list}`,
+    "findings for these files are unavailable until they parse."
+  ];
+}
 async function analyzeProject(opts = {}) {
   const cwd = opts.cwd ?? process.cwd();
   const rt = createNodeRuntime();
@@ -1588,7 +1617,7 @@ async function analyzeProject(opts = {}) {
     version: readPackageVersion(),
     ruleIds: rules.map((r) => r.id),
     examined,
-    warnings,
+    warnings: [...warnings, ...skippedFileWarnings([...components, ...kitModules])],
     loadedConfig: loaded
   };
 }

package/dist/index.d.ts

@@ -231,7 +231,7 @@ interface AnalyzeResult {
     ruleIds: string[];
     /** Per-rule, per-declaration counts of places examined, unfiltered by `--diff`/`--baseline`/suppressions. */
     examined: Record<string, Record<string, number>>;
-    /** Non-fatal config-file issues (unknown top-level keys, invalid enum values). Empty when no config file or none found. */
+    /** Non-fatal issues surfaced during analysis: config-file problems (unknown top-level keys, invalid enum values), version-floor notices, `--rules`/overrides conflicts, and skipped-file notices. Empty when none apply. */
     warnings: string[];
     /**
      * This analysis's config-file load result (`undefined` when no config file exists at its

package/dist/index.js

@@ -7,7 +7,7 @@ import {
   routeMatcher,
   run,
   spinnerEnabled
-} from "./chunk-EWDGS52A.js";
+} from "./chunk-NIUYLAJK.js";
 import {
   findUnknownRuleIds,
   knownRuleIds,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "svelte-vitals",
-  "version": "0.44.2",
+  "version": "0.44.3",
   "description": "A deterministic SvelteKit code-health scanner (SEO, performance, correctness, security, architecture) — not a runtime Web Vitals reporter.",
   "type": "module",
   "license": "MIT",
@@ -48,7 +48,7 @@
     "magicast": "^0.5.4",
     "svelte": "^5.56.8",
     "tinyglobby": "^0.2.17",
-    "@svelte-vitals/core": "0.40.0"
+    "@svelte-vitals/core": "0.40.1"
   },
   "devDependencies": {
     "@types/estree": "^1.0.9",