svelte-vitals 0.44.2 → 0.44.4

package/dist/bin.js

@@ -8,7 +8,7 @@ import {
   readPackageVersion,
   readPkg,
   run
-} from "./chunk-EWDGS52A.js";
+} from "./chunk-YY567AYV.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),
@@ -1484,15 +1486,15 @@ ${planText}` });
     }
   };
 }
-async function runInstallCli(args) {
+async function runInstallCli(args, io = consoleIO) {
   const argv = parseInstallArgs(args);
   if (argv.help) {
-    console.log(INSTALL_HELP);
+    io.log(INSTALL_HELP);
     return 0;
   }
   const { flags, warnings, errors } = resolveInstallArgs(argv);
-  for (const w of warnings) console.error(w);
-  for (const e of errors) console.error(e);
+  for (const w of warnings) io.errorLog(w);
+  for (const e of errors) io.errorLog(e);
   if (!flags) return 2;
   return runInstall(flags, realIO(), clackPrompts(), readPackageVersion());
 }
@@ -1720,7 +1722,7 @@ function runExplainCli(args, io = consoleIO) {
   return 0;
 }
 
-// src/bin.ts
+// src/cli.ts
 var HELP = `svelte-vitals \u2014 a deterministic SvelteKit code-health scanner (SEO \xB7 performance \xB7 correctness \xB7 security \xB7 architecture)
 
 Usage:
@@ -1784,45 +1786,53 @@ var VERSION = readPackageVersion();
 function selectApp(apps) {
   return selectAppPrompt(apps, "Multiple SvelteKit apps found \u2014 which one should svelte-vitals analyze?");
 }
-async function main() {
-  const rawArgs = process.argv.slice(2);
-  if (rawArgs[0] === "docs") {
-    const { runDocsCli } = await import("./cli-L2HHOZDN.js");
-    process.exitCode = runDocsCli(rawArgs.slice(1));
-    return;
-  }
-  if (rawArgs[0] === "explain") {
-    process.exitCode = runExplainCli(rawArgs.slice(1));
-    return;
-  }
-  if (rawArgs[0] === "install") {
-    const code2 = await runInstallCli(rawArgs.slice(1));
-    process.exit(code2);
-  }
-  if (rawArgs[0] === "ci") {
-    const code2 = await runCiCli(rawArgs.slice(1));
-    process.exit(code2);
-  }
-  const argv = parseRunArgs(rawArgs);
-  if (argv.help) {
-    console.log(HELP);
-    return;
-  }
-  if (argv.version) {
-    console.log(`${VERSION} (core ${readCoreVersion()})`);
-    console.error("svelte-vitals: run `svelte-vitals docs list` for the bundled guides.");
-    return;
-  }
-  const { options, warnings, errors, minHealth } = resolveArgs(argv);
-  for (const w of warnings) console.error(w);
-  for (const e of errors) console.error(e);
-  if (!options) process.exit(2);
+async function runCli(argv, io = consoleIO) {
+  if (argv[0] === "docs") {
+    const { runDocsCli } = await import("./cli-KH6FL5V7.js");
+    return { code: runDocsCli(argv.slice(1), io), exit: "natural" };
+  }
+  if (argv[0] === "explain") {
+    return { code: runExplainCli(argv.slice(1), io), exit: "natural" };
+  }
+  if (argv[0] === "install") {
+    return { code: await runInstallCli(argv.slice(1), io), exit: "immediate" };
+  }
+  if (argv[0] === "ci") {
+    const code2 = await runCiCli(argv.slice(1), { ...realIO(), log: io.log, errorLog: io.errorLog });
+    return { code: code2, exit: "immediate" };
+  }
+  const parsed = parseRunArgs(argv);
+  if (parsed.help) {
+    io.log(HELP);
+    return { code: 0, exit: "natural" };
+  }
+  if (parsed.version) {
+    io.log(`${VERSION} (core ${readCoreVersion()})`);
+    io.errorLog("svelte-vitals: run `svelte-vitals docs list` for the bundled guides.");
+    return { code: 0, exit: "natural" };
+  }
+  const { options, warnings, errors, minHealth } = resolveArgs(parsed);
+  for (const w of warnings) io.errorLog(w);
+  for (const e of errors) io.errorLog(e);
+  if (!options) return { code: 2, exit: "immediate" };
   const code = await run({
     ...options,
     minHealth,
-    selectApp
+    selectApp,
+    log: io.log,
+    errorLog: io.errorLog
   });
   await new Promise((resolve) => process.stdout.write("", resolve));
-  process.exit(code);
+  return { code, exit: "immediate" };
+}
+
+// src/bin.ts
+async function main() {
+  const { code, exit } = await runCli(process.argv.slice(2));
+  if (exit === "immediate") {
+    process.exit(code);
+  } else {
+    process.exitCode = code;
+  }
 }
 void main();

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

@@ -723,6 +723,7 @@ function chainFiles(pageRel, layouts) {
 async function resolveRoute(rt, cwd, pageRel, config, layouts, cache) {
   const files = chainFiles(pageRel, layouts);
   const composed = /* @__PURE__ */ new Map();
+  const jsonldTags = [];
   let broadOwn = false;
   let broadInherited = false;
   const images = [];
@@ -738,7 +739,9 @@ async function resolveRoute(rt, cwd, pageRel, config, layouts, cache) {
     }
     const resolved = await resolveFileTags(rt, cwd, rel, parsed, config, MAX_DEPTH, /* @__PURE__ */ new Set([rel]), cache);
     for (const tag of resolved.tags) {
-      composed.set(tagKey(tag), { ...tag, presence: isPage ? "own" : "inherited", file: rel });
+      const stamped = { ...tag, presence: isPage ? "own" : "inherited", file: rel };
+      if (tag.kind === "jsonld") jsonldTags.push(stamped);
+      else composed.set(tagKey(tag), stamped);
     }
     if (resolved.broad) {
       if (isPage) broadOwn = true;
@@ -755,7 +758,7 @@ async function resolveRoute(rt, cwd, pageRel, config, layouts, cache) {
   }
   const route = deriveRoute(pageRel);
   return {
-    head: { route, source: "static", tags: [...composed.values()], file: pageRel },
+    head: { route, source: "static", tags: [...composed.values(), ...jsonldTags], file: pageRel },
     images: { route, images },
     headings: { route, headings, componentHeadings }
   };
@@ -773,7 +776,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 +784,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 +990,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 +1233,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 +1562,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 +1620,7 @@ async function analyzeProject(opts = {}) {
     version: readPackageVersion(),
     ruleIds: rules.map((r) => r.id),
     examined,
-    warnings,
+    warnings: [...warnings, ...skippedFileWarnings([...components, ...kitModules])],
     loadedConfig: loaded
   };
 }

package/dist/{cli-L2HHOZDN.js → cli-KH6FL5V7.js}

@@ -39,7 +39,7 @@ var EMBEDDED_DOCS = [
     name: "scoping",
     title: "Scoping findings to a change",
     description: "Use --diff, --staged, --baseline and the suppressions file so only what a change introduced is reported, instead of a legacy backlog.",
-    body: "# Scoping findings to a change\n\nAn existing project usually has a backlog nobody is about to fix. Scope the report rather than\ndisabling rules.\n\n## Scope by file\n\n- **`--diff [ref]`** \u2014 only findings in files changed versus `ref` (default `HEAD`).\n- **`--staged`** \u2014 only findings in staged files. The pre-commit gate.\n\n```bash\nsvelte-vitals . --diff --reporter agent   # after editing: what did I just break?\nsvelte-vitals . --staged                  # before committing\n```\n\nBoth work when the project is not at the git repo root.\n\n## Scope by finding (`--baseline <ref>`)\n\nReports only findings **not already present** at `ref`. Scopes by finding identity rather than by\nfile, so a pre-existing problem in a file you touched does not fail the gate. No default ref.\n\n```bash\nsvelte-vitals --diff origin/main --baseline origin/main --fail-on warning   # PR gate\n```\n\nIt checks `ref` out into a temporary worktree and subtracts those findings. On failure (no git,\nbad ref) it warns and reports everything rather than failing the run.\n\n## Accept a backlog once (`svelte-vitals-suppressions.json`)\n\n`--baseline` handles the transient case. For a persistent ramp, record today's findings once:\n\n```bash\nsvelte-vitals --update-suppressions\ngit add svelte-vitals-suppressions.json\n```\n\nThis analyzes the whole project (`--diff`/`--staged`/`--baseline` are ignored), writes every\npenalized finding, and exits `0` without a report.\n\nThe file then applies automatically on every run, after `--diff`/`--staged` and `--baseline`.\nFixing an accepted finding leaves a **stale** entry, reported on stderr but never failing the run.\n`--no-suppressions` ignores the file for one run.\n\nA malformed suppressions file is a hard error (exit `2`), not a silent skip.\n\n## Which one\n\n| Situation                              | Use                                      |\n| -------------------------------------- | ---------------------------------------- |\n| Checking an edit you just made         | `--diff`                                 |\n| Pre-commit hook                        | `--staged`                               |\n| PR gate against a base branch          | `--diff <base> --baseline <base>`        |\n| Adopting on a legacy project, for good | `--update-suppressions`, commit the file |\n\nMatching ignores line numbers in both `--baseline` and the suppressions file, so a second\nviolation of the same rule lower in the same file does not surface as new.\n\n## Related\n\n- `svelte-vitals docs show ci` \u2014 the generated PR gate already does the `--diff`/`--baseline` pairing\n- `svelte-vitals docs show config` \u2014 turning a rule off for good, when that is genuinely right"
+    body: "# Scoping findings to a change\n\nAn existing project usually has a backlog nobody is about to fix. Scope the report rather than\ndisabling rules.\n\n## Scope by file\n\n- **`--diff [ref]`** \u2014 only findings in files changed versus `ref` (default `HEAD`).\n- **`--staged`** \u2014 only findings in staged files. The pre-commit gate.\n\n```bash\nsvelte-vitals . --diff --reporter agent   # after editing: what did I just break?\nsvelte-vitals . --staged                  # before committing\n```\n\nBoth work when the project is not at the git repo root.\n\n## Scope by finding (`--baseline <ref>`)\n\nReports only findings **not already present** at `ref`. Scopes by finding identity rather than by\nfile, so a pre-existing problem in a file you touched does not fail the gate. No default ref.\n\n```bash\nsvelte-vitals --diff origin/main --baseline origin/main --fail-on warning   # PR gate\n```\n\nIt checks `ref` out into a temporary worktree and subtracts those findings. On failure (no git,\nbad ref) it warns and reports everything rather than failing the run.\n\n## Accept a backlog once (`svelte-vitals-suppressions.json`)\n\n`--baseline` handles the transient case. For a persistent ramp, record today's findings once:\n\n```bash\nsvelte-vitals --update-suppressions\ngit add svelte-vitals-suppressions.json\n```\n\nThis analyzes the whole project (`--diff`/`--staged`/`--baseline` are ignored), writes every\npenalized finding, and exits `0` without a report.\n\nThe file then applies automatically on every run, after `--diff`/`--staged` and `--baseline`.\nFixing an accepted finding leaves a **stale** entry, reported on stderr but never failing the run.\n`--no-suppressions` ignores the file for one run.\n\nAn entry covers whatever its rule reports at that route and location, not just the message\nrecorded when written \u2014 a different finding from the same rule at the same spot still matches\nand stays suppressed (and not stale).\n\nA malformed suppressions file is a hard error (exit `2`), not a silent skip.\n\n## Which one\n\n| Situation                              | Use                                      |\n| -------------------------------------- | ---------------------------------------- |\n| Checking an edit you just made         | `--diff`                                 |\n| Pre-commit hook                        | `--staged`                               |\n| PR gate against a base branch          | `--diff <base> --baseline <base>`        |\n| Adopting on a legacy project, for good | `--update-suppressions`, commit the file |\n\nMatching ignores line numbers in both `--baseline` and the suppressions file, so a second\nviolation of the same rule lower in the same file does not surface as new.\n\n## Related\n\n- `svelte-vitals docs show ci` \u2014 the generated PR gate already does the `--diff`/`--baseline` pairing\n- `svelte-vitals docs show config` \u2014 turning a rule off for good, when that is genuinely right"
   }
 ];
 

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-YY567AYV.js";
 import {
   findUnknownRuleIds,
   knownRuleIds,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "svelte-vitals",
-  "version": "0.44.2",
+  "version": "0.44.4",
   "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.41.0"
   },
   "devDependencies": {
     "@types/estree": "^1.0.9",