svelte-vitals 0.44.1 → 0.44.3

package/README.md

@@ -31,14 +31,11 @@ Health: 76/100
 SEO Score: 51/100   (route avg 61 · site −10)
 Architecture Score: 100/100   (route avg 100)
 
-Critical (2)
+Critical (1)
 ────────────────────────
 ✗ seo/title-presence  Missing <title>
             /
             src/routes/+page.svelte
-✗ seo/description-presence  Missing <meta name="description">
-            /
-            src/routes/+page.svelte
 
 Warnings (10)
 ────────────────────────

package/dist/bin.js

@@ -2,20 +2,22 @@
 import {
   CONFIG_FILENAMES,
   discoverApps,
-  findUnknownRuleIds,
   hasDep,
   isReporterName,
-  knownRuleIds,
   readCoreVersion,
   readPackageVersion,
   readPkg,
   run
-} from "./chunk-4MC7STFI.js";
+} from "./chunk-NIUYLAJK.js";
 import {
   consoleIO,
   parseCliArgs,
   toList
 } from "./chunk-5Q2PT47V.js";
+import {
+  findUnknownRuleIds,
+  knownRuleIds
+} from "./chunk-TFBLQUAC.js";
 
 // src/resolve-args.ts
 var CATEGORIES = ["seo", "performance", "correctness", "security", "architecture"];
@@ -1427,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),
@@ -1785,7 +1789,7 @@ function selectApp(apps) {
 async function main() {
   const rawArgs = process.argv.slice(2);
   if (rawArgs[0] === "docs") {
-    const { runDocsCli } = await import("./cli-LGD6NDAO.js");
+    const { runDocsCli } = await import("./cli-L2HHOZDN.js");
     process.exitCode = runDocsCli(rawArgs.slice(1));
     return;
   }

package/dist/{chunk-4MC7STFI.js → chunk-NIUYLAJK.js}

@@ -1,8 +1,14 @@
+import {
+  findUnknownRuleIds,
+  knownRuleIds,
+  ruleOptionsSpec
+} from "./chunk-TFBLQUAC.js";
+
 // src/index.ts
 import { mkdirSync, writeFileSync as writeFileSync2 } from "fs";
 import { dirname as dirname2, join as join7 } from "path";
 import {
-  allRules as allRules3,
+  allRules as allRules2,
   runRules,
   formatConsoleReport,
   formatJsonReport,
@@ -392,6 +398,29 @@ function collectSvelteHeads(node, acc) {
     if (key in node) collectSvelteHeads(childOf(node, key), acc);
   }
 }
+var JS_MIME_TYPES = /* @__PURE__ */ new Set([
+  "application/ecmascript",
+  "application/javascript",
+  "application/x-ecmascript",
+  "application/x-javascript",
+  "text/ecmascript",
+  "text/javascript",
+  "text/javascript1.0",
+  "text/javascript1.1",
+  "text/javascript1.2",
+  "text/javascript1.3",
+  "text/javascript1.4",
+  "text/javascript1.5",
+  "text/jscript",
+  "text/livescript",
+  "text/x-ecmascript",
+  "text/x-javascript"
+]);
+function isClassicScriptType(type) {
+  if (type === void 0) return true;
+  const normalized = type.trim().toLowerCase();
+  return normalized === "" || JS_MIME_TYPES.has(normalized);
+}
 function tagsFromHead(head) {
   const tags = [];
   const children = head.fragment.nodes;
@@ -451,7 +480,7 @@ function tagsFromHead(head) {
       } else {
         const src = attrText(attributes, "src");
         if (src) {
-          const blocking = findAttr3(attributes, "defer") === void 0 && findAttr3(attributes, "async") === void 0 && type !== "module";
+          const blocking = isClassicScriptType(type) && findAttr3(attributes, "defer") === void 0 && findAttr3(attributes, "async") === void 0;
           tags.push({ kind: "script", value: "static", href: src, ...blocking ? { blocking: true } : {} });
         }
       }
@@ -593,6 +622,7 @@ function resolveComponentPath(source, fromFileRel) {
 }
 async function resolveFileTags(rt, cwd, fileRel, parsed, config, depth, visited, cache = /* @__PURE__ */ new Map()) {
   const tags = [...parsed.headTags];
+  const headings = [];
   let broad = false;
   for (const use of parsed.components) {
     const info = parsed.imports.get(use.name);
@@ -616,10 +646,12 @@ async function resolveFileTags(rt, cwd, fileRel, parsed, config, depth, visited,
         const child = await resolveFileTags(rt, cwd, childRel, childParsed, config, depth - 1, childVisited, cache);
         tags.push(...child.tags);
         broad = broad || child.broad;
+        for (const h of childParsed.headings) headings.push({ ...h, file: childRel });
+        headings.push(...child.headings);
       }
     }
   }
-  return { tags, broad };
+  return { tags, broad, headings };
 }
 
 // src/providers/source/routes.ts
@@ -695,6 +727,7 @@ async function resolveRoute(rt, cwd, pageRel, config, layouts, cache) {
   let broadInherited = false;
   const images = [];
   const headings = [];
+  const componentHeadings = [];
   for (const { rel, isPage } of files) {
     const parsed = await readAndParse(rt, cwd, rel, cache);
     for (const img of parsed.images) {
@@ -711,6 +744,7 @@ async function resolveRoute(rt, cwd, pageRel, config, layouts, cache) {
       if (isPage) broadOwn = true;
       else broadInherited = true;
     }
+    componentHeadings.push(...resolved.headings);
   }
   if (broadOwn || broadInherited) {
     const presence = broadOwn ? "own" : "inherited";
@@ -723,7 +757,7 @@ async function resolveRoute(rt, cwd, pageRel, config, layouts, cache) {
   return {
     head: { route, source: "static", tags: [...composed.values()], file: pageRel },
     images: { route, images },
-    headings: { route, headings }
+    headings: { route, headings, componentHeadings }
   };
 }
 async function collectRoutes(rt, cwd, config = defaultConfig, cache = /* @__PURE__ */ new Map()) {
@@ -739,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(/^\//, ""));
 }
@@ -747,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 };
 }
 
@@ -944,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";
 
@@ -959,22 +1002,6 @@ import {
   shouldSkipRangeCheck,
   validateRuleSetting
 } from "@svelte-vitals/core";
-
-// src/rules-config.ts
-import { allRules } from "@svelte-vitals/core";
-var KNOWN_IDS = new Set(allRules.map((r) => r.id));
-var RULE_BY_ID = new Map(allRules.map((r) => [r.id, r]));
-function findUnknownRuleIds(ids) {
-  return [...new Set(ids.filter((id) => !KNOWN_IDS.has(id)))];
-}
-function knownRuleIds() {
-  return [...KNOWN_IDS].sort();
-}
-function ruleOptionsSpec(id) {
-  return RULE_BY_ID.get(id)?.options;
-}
-
-// src/config-file.ts
 var CONFIG_FILENAMES = ["svelte-vitals.config.mjs", "svelte-vitals.config.js", "svelte-vitals.config.ts"];
 var TREAT_DYNAMIC_AS_VALUES = ["pass", "warn", "fail"];
 var FAIL_ON_VALUES = ["critical", "warning", "info"];
@@ -1203,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) {
@@ -1472,13 +1509,13 @@ function scoreAnimationEnabled(opts) {
 }
 
 // src/rule-selection.ts
-import { allRules as allRules2 } from "@svelte-vitals/core";
+import { allRules } from "@svelte-vitals/core";
 function resolveRuleSelection(input) {
   const out = { ...input.rules ?? input.fileRules };
   const allow = input.allowRules ?? [];
   if (allow.length > 0) {
     const allowed = new Set(allow);
-    for (const rule of allRules2) if (!allowed.has(rule.id)) out[rule.id] = "off";
+    for (const rule of allRules) if (!allowed.has(rule.id)) out[rule.id] = "off";
     for (const id of allowed) {
       const setting = out[id];
       if (setting === void 0) continue;
@@ -1522,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();
@@ -1551,7 +1598,7 @@ async function analyzeProject(opts = {}) {
     route: opts.route,
     parseCache: opts.parseCache
   });
-  const selected = selectRules(allRules3, config);
+  const selected = selectRules(allRules2, config);
   const rules = opts.categories ? selected.filter((r) => opts.categories.includes(r.category)) : selected;
   const { results: rawResults, examined } = await runRules(rules, {
     heads,
@@ -1570,7 +1617,7 @@ async function analyzeProject(opts = {}) {
     version: readPackageVersion(),
     ruleIds: rules.map((r) => r.id),
     examined,
-    warnings,
+    warnings: [...warnings, ...skippedFileWarnings([...components, ...kitModules])],
     loadedConfig: loaded
   };
 }
@@ -1828,9 +1875,6 @@ export {
   readPackageVersion,
   readCoreVersion,
   isReporterName,
-  findUnknownRuleIds,
-  knownRuleIds,
-  ruleOptionsSpec,
   CONFIG_FILENAMES,
   loadConfigFile,
   spinnerEnabled,

package/dist/chunk-TFBLQUAC.js

@@ -0,0 +1,19 @@
+// src/rules-config.ts
+import { allRules } from "@svelte-vitals/core";
+var KNOWN_IDS = new Set(allRules.map((r) => r.id));
+var RULE_BY_ID = new Map(allRules.map((r) => [r.id, r]));
+function findUnknownRuleIds(ids) {
+  return [...new Set(ids.filter((id) => !KNOWN_IDS.has(id)))];
+}
+function knownRuleIds() {
+  return [...KNOWN_IDS].sort();
+}
+function ruleOptionsSpec(id) {
+  return RULE_BY_ID.get(id)?.options;
+}
+
+export {
+  findUnknownRuleIds,
+  knownRuleIds,
+  ruleOptionsSpec
+};

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

@@ -2,6 +2,12 @@ import {
   consoleIO,
   parseCliArgs
 } from "./chunk-5Q2PT47V.js";
+import {
+  knownRuleIds
+} from "./chunk-TFBLQUAC.js";
+
+// src/docs/cli.ts
+import { docsUrlFor } from "@svelte-vitals/core";
 
 // src/docs/generated.ts
 var EMBEDDED_DOCS = [
@@ -100,9 +106,16 @@ function runDocsCli(args, io = consoleIO) {
       io.errorLog(`svelte-vitals: known topics: ${knownTopicNames()}.`);
       return 2;
     }
-    const doc = EMBEDDED_DOCS.find((d) => d.name === rest[0]);
+    const name = rest[0];
+    const doc = EMBEDDED_DOCS.find((d) => d.name === name);
     if (!doc) {
-      io.errorLog(`svelte-vitals: unknown docs topic '${rest[0]}'.`);
+      const ruleId = name.startsWith("rules/") ? name.slice("rules/".length) : name;
+      if (knownRuleIds().includes(ruleId)) {
+        io.errorLog(`svelte-vitals: '${ruleId}' is a rule, not a docs topic.`);
+        io.errorLog(`svelte-vitals: rule detail: \`svelte-vitals explain ${ruleId}\`; web: ${docsUrlFor(ruleId)}`);
+        return 2;
+      }
+      io.errorLog(`svelte-vitals: unknown docs topic '${name}'.`);
       io.errorLog(`svelte-vitals: known topics: ${knownTopicNames()}.`);
       return 2;
     }

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

@@ -3,14 +3,16 @@ import {
   analyzeProject,
   applyScope,
   defineConfig,
-  findUnknownRuleIds,
-  knownRuleIds,
   loadConfigFile,
   routeMatcher,
-  ruleOptionsSpec,
   run,
   spinnerEnabled
-} from "./chunk-4MC7STFI.js";
+} from "./chunk-NIUYLAJK.js";
+import {
+  findUnknownRuleIds,
+  knownRuleIds,
+  ruleOptionsSpec
+} from "./chunk-TFBLQUAC.js";
 export {
   ProjectError,
   analyzeProject,

package/package.json

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