svelte-vitals 0.48.0 → 0.49.0

package/dist/bin.js

@@ -1,12 +1,12 @@
 #!/usr/bin/env node
 import {
   runAnalyzeCliGunshi
-} from "./chunk-4LSRQXJ6.js";
+} from "./chunk-4FOGEXGH.js";
 import "./chunk-DZVXCUHG.js";
 import {
   realIO
 } from "./chunk-RSSMIHIM.js";
-import "./chunk-LJPABOWU.js";
+import "./chunk-5YVVJMZM.js";
 import "./chunk-WGFXFKBB.js";
 import {
   consoleIO
@@ -21,7 +21,7 @@ async function runCli(argv, io = consoleIO, env = process.env) {
   try {
     const locale = resolveLocale(env);
     if (argv[0] === "complete") {
-      const { runCompleteCliGunshi } = await import("./complete-IOAMJRBY.js");
+      const { runCompleteCliGunshi } = await import("./complete-ZQZGCJ5W.js");
       return { code: await runCompleteCliGunshi(argv, io), exit: "natural" };
     }
     if (argv[0] === "docs") {
@@ -29,7 +29,7 @@ async function runCli(argv, io = consoleIO, env = process.env) {
       return { code: await runDocsCliGunshi(argv.slice(1), io, locale), exit: "natural" };
     }
     if (argv[0] === "explain") {
-      const { runExplainCliGunshi } = await import("./explain-WOKUZJVY.js");
+      const { runExplainCliGunshi } = await import("./explain-PRCEATL4.js");
       return { code: await runExplainCliGunshi(argv.slice(1), io, locale), exit: "natural" };
     }
     if (argv[0] === "install") {

package/dist/{chunk-4LSRQXJ6.js → chunk-4FOGEXGH.js}

@@ -8,7 +8,7 @@ import {
 } from "./chunk-RSSMIHIM.js";
 import {
   run
-} from "./chunk-LJPABOWU.js";
+} from "./chunk-5YVVJMZM.js";
 import {
   readCoreVersion,
   readPackageVersion

package/dist/{chunk-LJPABOWU.js → chunk-5YVVJMZM.js}

@@ -34,6 +34,8 @@ import {
   selectRules,
   applyRuleSeverities,
   applyOverrides,
+  applyInlineDirectives,
+  unknownDirectiveIds,
   settingSeverity,
   withFailedRulesOff,
   formatFailedRuleWarning,
@@ -75,6 +77,7 @@ import {
   SITEMAP_SOURCE_PATHS,
   SVELTE_CONFIG_FILES,
   VITE_CONFIG_FILES,
+  collectSuppressions,
   findMinifyDisabled,
   resolveKitAliases,
   resolveKitPathsBase
@@ -152,6 +155,18 @@ function detectHtmlLang(html) {
   const value = match[1] ?? match[2] ?? match[3] ?? "";
   return { presence: "own", value: value.trim().length > 0 ? "static" : "absent" };
 }
+function detectAppHtmlBodyTags(html) {
+  let markup = html.replace(/<!--[\s\S]*?-->/g, "").replace(/<script[\s\S]*?<\/script\s*>/gi, "").replace(/<style[\s\S]*?<\/style\s*>/gi, "");
+  const innermost = /<template\b[^>]*>(?:(?!<template\b)[\s\S])*?<\/template\s*>/gi;
+  for (let prev = ""; prev !== markup; ) {
+    prev = markup;
+    markup = markup.replace(innermost, "\0");
+  }
+  markup = markup.replaceAll("\0", "<template></template>");
+  const body = /<body\b[^>]*>([\s\S]*?)(?:<\/body\s*>|$)/i.exec(markup)?.[1];
+  if (body === void 0) return [];
+  return [...new Set([...body.matchAll(/<([a-zA-Z][a-zA-Z0-9-]*)\b/g)].map((m) => m[1].toLowerCase()))];
+}
 function detectAppHtmlIds(html) {
   const markup = html.replace(/<!--[\s\S]*?-->/g, "").replace(/<script[\s\S]*?<\/script\s*>/gi, "").replace(/<style[\s\S]*?<\/style\s*>/gi, "");
   const found = markup.matchAll(/(?<![\w-])id\s*=\s*(?:"([^"]*)"|'([^']*)'|([^\s"'>{][^\s"'>]*))/gi);
@@ -172,7 +187,8 @@ async function detectAppHtmlFacts(rt, cwd) {
     // [\s\S]*? is ambiguous across iterations and backtracks exponentially on a comment run
     // with no doctype (measured: ~45 leading comments hang the process).
     appHtmlDoctype: /^\s*<!doctype\s+html/i.test(content.replace(/<!--[\s\S]*?-->/g, "")),
-    appHtmlIds: detectAppHtmlIds(content)
+    appHtmlIds: detectAppHtmlIds(content),
+    appHtmlBodyTags: detectAppHtmlBodyTags(content)
   };
 }
 async function robotsRefsSitemap(rt, cwd) {
@@ -189,8 +205,9 @@ async function detectViteMinifyDisabled(rt, cwd) {
   const file = VITE_CONFIG_FILES[exists.indexOf(true)];
   if (!file) return void 0;
   try {
-    const hit = findMinifyDisabled(await rt.readFile(rt.join(cwd, file)));
-    return hit ? { file, line: hit.line } : void 0;
+    const source = await rt.readFile(rt.join(cwd, file));
+    const hit = findMinifyDisabled(source);
+    return hit ? { file, line: hit.line, suppressions: collectSuppressions(source) } : void 0;
   } catch {
     return void 0;
   }
@@ -243,7 +260,12 @@ import "@svelte-vitals/core";
 import {
   collectComponentFacts,
   collectKitModuleFacts,
-  collectSourceFiles
+  collectSourceFiles,
+  compileOverrides,
+  ROBOTS_SOURCE_PATHS as ROBOTS_SOURCE_PATHS2,
+  SITEMAP_SOURCE_PATHS as SITEMAP_SOURCE_PATHS2,
+  SVELTE_CONFIG_FILES as SVELTE_CONFIG_FILES2,
+  VITE_CONFIG_FILES as VITE_CONFIG_FILES2
 } from "@svelte-vitals/core/internal";
 
 // src/providers/source/routes.ts
@@ -387,6 +409,7 @@ function findAdapter(info) {
 
 // src/providers/source/parse.ts
 import {
+  collectSuppressions as collectSuppressions2,
   stripTextDirective,
   parseSvelte,
   CHILD_NODE_KEYS,
@@ -605,6 +628,8 @@ function collectA11y(fragment, source) {
   let groups = 0;
   let slotInLandmark;
   let unknowableContent = false;
+  const elementTags = /* @__PURE__ */ new Set();
+  let elementsUnknowable = false;
   const emit = (ctx, node) => {
     const inLandmark = ctx.landmarks.at(-1);
     nodes.push({ ...node, repeatable: ctx.repeatable, path: ctx.path, ...inLandmark ? { inLandmark } : {} });
@@ -627,6 +652,7 @@ function collectA11y(fragment, source) {
       // head content never renders into the body
       case "HtmlTag":
         unknowableContent = true;
+        elementsUnknowable = true;
         return;
       case "IfBlock":
         walkIfChain(node, ctx, groups++, 0);
@@ -649,6 +675,8 @@ function collectA11y(fragment, source) {
       // attributes are real — dropping them would make no-missing-id-ref report phantom misses.
       case "RegularElement":
       case "SvelteElement":
+        if (node.type === "SvelteElement") elementsUnknowable = true;
+        else elementTags.add(node.name.toLowerCase());
         walkElement(node, ctx);
         return;
       case "Component":
@@ -728,7 +756,13 @@ function collectA11y(fragment, source) {
     });
   };
   walk(fragment, { path: [], repeatable: false, landmarks: [], elementDepth: 0, asideDemoting: 0 });
-  return { nodes, ...slotInLandmark ? { slotInLandmark } : {}, unknowableContent };
+  return {
+    nodes,
+    ...slotInLandmark ? { slotInLandmark } : {},
+    unknowableContent,
+    elementTags: [...elementTags],
+    elementsUnknowable
+  };
 }
 function isChildrenRender(node) {
   const call = node.expression.type === "ChainExpression" ? node.expression.expression : node.expression;
@@ -750,7 +784,8 @@ function parseFile(source, filename) {
     imports: collectImports(ast),
     images,
     headings,
-    a11y: collectA11y(ast.fragment, source)
+    a11y: collectA11y(ast.fragment, source),
+    suppressions: collectSuppressions2(source)
   };
 }
 
@@ -921,6 +956,8 @@ function offsetPath(path, base) {
 async function composeA11y(ctx, fileRel, parsed, depth, visited, chain) {
   const { rt, cwd, state } = ctx;
   if (parsed.a11y.unknowableContent) state.fullyResolved = false;
+  for (const t of parsed.a11y.elementTags) state.elementTags.add(t);
+  if (parsed.a11y.elementsUnknowable) state.elementsClosed = false;
   const base = state.nextGroup;
   state.nextGroup += groupSpan(parsed.a11y.nodes);
   const composed = [];
@@ -934,6 +971,7 @@ async function composeA11y(ctx, fileRel, parsed, depth, visited, chain) {
     const childRel = info ? resolveComponentPath(info.source, fileRel, ctx.aliases) : void 0;
     if (!childRel || depth <= 0 || visited.has(childRel) || !await rt.exists(rt.join(cwd, childRel))) {
       state.fullyResolved = false;
+      state.elementsClosed = false;
       continue;
     }
     const childParsed = await readAndParse(rt, cwd, childRel, ctx.cache);
@@ -963,7 +1001,7 @@ function representatives(nodes, chainOrder) {
     [...folded].map(([key, list]) => [key, list.sort(order).map(({ file, line }) => ({ file, line }))])
   );
 }
-async function resolveRoute(rt, cwd, pageRel, config, layouts, cache, aliases, appHtmlIds) {
+async function resolveRoute(rt, cwd, pageRel, config, layouts, cache, aliases, appHtmlIds, appHtmlBodyTags) {
   const files = chainFiles(pageRel, layouts);
   const chainOrder = new Map(files.map((f, i) => [f.rel, i]));
   const composed = /* @__PURE__ */ new Map();
@@ -973,7 +1011,14 @@ async function resolveRoute(rt, cwd, pageRel, config, layouts, cache, aliases, a
   const images = [];
   const headings = [];
   const componentHeadings = [];
-  const a11yCtx = { rt, cwd, config, cache, aliases, state: { nextGroup: 0, fullyResolved: true } };
+  const a11yCtx = {
+    rt,
+    cwd,
+    config,
+    cache,
+    aliases,
+    state: { nextGroup: 0, fullyResolved: true, elementTags: new Set(appHtmlBodyTags ?? []), elementsClosed: true }
+  };
   const a11yNodes = [];
   const nestedLandmarks = [];
   let slotLandmark;
@@ -1033,14 +1078,17 @@ async function resolveRoute(rt, cwd, pageRel, config, layouts, cache, aliases, a
       // never a missing reference (HTML's "top of the document" fragment).
       idRefs: a11yNodes.filter((n) => n.kind === "idref" && !(n.attr === "href" && isTopFragment(n.key))).map((n) => ({ id: n.key, attr: n.attr ?? "", file: n.file, line: n.line })),
       idCandidates: [.../* @__PURE__ */ new Set([...literalIds.map((n) => n.key), ...appHtmlIds ?? []])],
-      fullyResolved: a11yCtx.state.fullyResolved
+      fullyResolved: a11yCtx.state.fullyResolved,
+      elementTags: [...a11yCtx.state.elementTags],
+      elementsClosed: a11yCtx.state.elementsClosed,
+      file: pageRel
     }
   };
 }
-async function collectRoutes(rt, cwd, config = defaultConfig, cache = /* @__PURE__ */ new Map(), aliases, appHtmlIds) {
+async function collectRoutes(rt, cwd, config = defaultConfig, cache = /* @__PURE__ */ new Map(), aliases, appHtmlIds, appHtmlBodyTags) {
   const [pages, layouts] = await Promise.all([enumerateRoutePages(rt, cwd), collectLayouts(rt, cwd)]);
   const facts = await Promise.all(
-    pages.map((page) => resolveRoute(rt, cwd, page, config, layouts, cache, aliases, appHtmlIds))
+    pages.map((page) => resolveRoute(rt, cwd, page, config, layouts, cache, aliases, appHtmlIds, appHtmlBodyTags))
   );
   return {
     heads: facts.map((f) => f.head),
@@ -1059,11 +1107,15 @@ function routeMatcher(glob) {
 }
 
 // src/collect-all.ts
+function globList(globs) {
+  return globs === void 0 ? [] : Array.isArray(globs) ? globs : [globs];
+}
 async function collectAll(rt, cwd, config, opts = {}) {
   const matches = routeMatcher(opts.route);
+  const parseCache = opts.parseCache ?? /* @__PURE__ */ new Map();
   const project = await collectProjectFacts(rt, cwd);
   const [collected, components, kitModules, sourceFiles] = await Promise.all([
-    collectRoutes(rt, cwd, config, opts.parseCache, project.kitAliases, project.appHtmlIds),
+    collectRoutes(rt, cwd, config, parseCache, project.kitAliases, project.appHtmlIds, project.appHtmlBodyTags),
     // 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.
@@ -1079,7 +1131,45 @@ async function collectAll(rt, cwd, config, opts = {}) {
   const images = collected.images.filter((i) => matches(i.route));
   const headings = collected.headings.filter((h) => matches(h.route));
   const a11y = collected.a11y.filter((a) => matches(a.route));
-  return { heads, images, headings, a11y, project, components, kitModules, sourceFiles };
+  const directives = /* @__PURE__ */ new Map();
+  for (const [file, parsed] of parseCache) {
+    const suppressions = await parsed.then(
+      (parsedFile) => parsedFile.suppressions,
+      () => void 0
+    );
+    if (suppressions) directives.set(file, suppressions);
+  }
+  for (const c of components) directives.set(c.file, c.suppressions ?? []);
+  for (const m of kitModules) directives.set(m.file, m.suppressions ?? []);
+  const viteConfig = project.viteMinifyDisabled;
+  if (viteConfig?.file) directives.set(viteConfig.file, viteConfig.suppressions ?? []);
+  const routes = collected.heads.map((h) => h.route);
+  const emptySelections = [];
+  if (opts.route !== void 0 && routes.length > 0 && !routes.some(matches))
+    emptySelections.push(`--route '${opts.route}' matched none of the ${routes.length} route(s) found.`);
+  if (opts.route === void 0 && routes.length > 0) {
+    const attributable = [
+      ...directives.keys(),
+      ...ROBOTS_SOURCE_PATHS2,
+      ...SITEMAP_SOURCE_PATHS2,
+      ...VITE_CONFIG_FILES2,
+      ...SVELTE_CONFIG_FILES2
+    ];
+    const entries = config.overrides ?? [];
+    const compiled = compileOverrides(config);
+    entries.forEach((entry, i) => {
+      globList(entry.route).forEach((glob) => {
+        if (!routes.some(routeMatcher(glob)))
+          emptySelections.push(`overrides entry for route '${glob}' matched no route.`);
+      });
+      globList(entry.files).forEach((glob, j) => {
+        const pattern = compiled[i]?.files[j];
+        if (pattern && !attributable.some((f) => pattern.test(f)))
+          emptySelections.push(`overrides entry for files '${glob}' matched no file.`);
+      });
+    });
+  }
+  return { heads, images, headings, a11y, project, components, kitModules, sourceFiles, directives, emptySelections };
 }
 
 // src/changed-files.ts
@@ -1580,17 +1670,21 @@ async function analyzeProject(opts = {}) {
     ...await checkVersionFloor(rt, cwd),
     ...overridesOffWarnings(opts.allowRules, config.overrides)
   ];
-  const { heads, images, headings, a11y, project, components, kitModules, sourceFiles } = await collectAll(
-    rt,
-    cwd,
-    config,
-    {
-      route: opts.route,
-      parseCache: opts.parseCache
-    }
-  );
+  const { heads, images, headings, a11y, project, components, kitModules, sourceFiles, directives, emptySelections } = await collectAll(rt, cwd, config, {
+    route: opts.route,
+    parseCache: opts.parseCache
+  });
+  warnings.push(...emptySelections);
+  if (opts.route === void 0) warnings.push(...unknownDirectiveIds(directives, allRules2));
   const selected = selectRules(allRules2, config);
   const rules = opts.categories ? selected.filter((r) => opts.categories.includes(r.category)) : selected;
+  if (opts.route !== void 0 && opts.allowRules?.length) {
+    const starved = rules.filter((r) => opts.allowRules.includes(r.id) && r.scope !== "route").map((r) => r.id);
+    if (starved.length > 0)
+      warnings.push(
+        `--rules ${starved.map((id) => `'${id}'`).join(", ")} examined nothing: --route collects route facts only.`
+      );
+  }
   const {
     results: rawResults,
     examined,
@@ -1606,7 +1700,12 @@ async function analyzeProject(opts = {}) {
     kitModules,
     sourceFiles
   });
-  const results = applyOverrides(applyRuleSeverities(rawResults, config), config);
+  const results = applyInlineDirectives(
+    applyOverrides(applyRuleSeverities(rawResults, config), config),
+    directives,
+    rules,
+    config
+  );
   const failedRuleIds = failedRules.map((f) => f.id);
   const scoringConfig = withFailedRulesOff(config, failedRuleIds);
   return {

package/dist/{chunk-ZAHTKEJ5.js → chunk-K363W33X.js}

@@ -26,7 +26,11 @@ function describeOptions(id, options) {
     "string-map": "merged over the default entries \u2014 a new key is added, a built-in key has its value overridden"
   };
   const lines = options.map((o) => {
-    const bounds = [o.min !== void 0 ? `>= ${o.min}` : "", o.max !== void 0 ? `<= ${o.max}` : ""].filter(Boolean).join(", ");
+    const bounds = [
+      o.min !== void 0 ? `>= ${o.min}` : "",
+      o.max !== void 0 ? `<= ${o.max}` : "",
+      o.pattern ? `each entry ${o.pattern}` : ""
+    ].filter(Boolean).join(", ");
     return `- ${o.name} (${o.kind}, default ${JSON.stringify(o.default)}${bounds ? `, ${bounds}` : ""}) \u2014 ${MERGE[o.kind]}`;
   });
   return `set in svelte-vitals.config.* as \`rules: { '${id}': { options: { \u2026 } } }\`, or per path in \`overrides\`:

package/dist/{complete-IOAMJRBY.js → complete-ZQZGCJ5W.js}

@@ -8,14 +8,14 @@ import {
 import "./chunk-CTMKTFST.js";
 import {
   ROOT_ARGS
-} from "./chunk-4LSRQXJ6.js";
+} from "./chunk-4FOGEXGH.js";
 import {
   CATEGORIES,
   FAIL_ON_VALUES,
   TREAT_DYNAMIC_AS_VALUES
 } from "./chunk-DZVXCUHG.js";
 import "./chunk-RSSMIHIM.js";
-import "./chunk-LJPABOWU.js";
+import "./chunk-5YVVJMZM.js";
 import {
   REPORTER_NAMES
 } from "./chunk-WGFXFKBB.js";
@@ -27,7 +27,7 @@ import {
 } from "./chunk-72ZLERW3.js";
 import {
   EXPLAIN_ARGS
-} from "./chunk-ZAHTKEJ5.js";
+} from "./chunk-K363W33X.js";
 import {
   consoleIO
 } from "./chunk-SLUMRYUD.js";

package/dist/{explain-WOKUZJVY.js → explain-PRCEATL4.js}

@@ -1,7 +1,7 @@
 import {
   EXPLAIN_ARGS,
   runExplainCliGunshi
-} from "./chunk-ZAHTKEJ5.js";
+} from "./chunk-K363W33X.js";
 import "./chunk-SLUMRYUD.js";
 import "./chunk-NMVBVKLX.js";
 import "./chunk-AI24FJ5Q.js";

package/dist/gunshi-registry.js

@@ -4,10 +4,10 @@ import {
 import "./chunk-CTMKTFST.js";
 import {
   ROOT_ARGS
-} from "./chunk-4LSRQXJ6.js";
+} from "./chunk-4FOGEXGH.js";
 import "./chunk-DZVXCUHG.js";
 import "./chunk-RSSMIHIM.js";
-import "./chunk-LJPABOWU.js";
+import "./chunk-5YVVJMZM.js";
 import "./chunk-WGFXFKBB.js";
 import "./chunk-MLIODPHZ.js";
 import {

package/dist/index.d.ts

@@ -1,7 +1,7 @@
 import { Config, Severity, RuleSetting, Category, Result } from '@svelte-vitals/core';
 export { defineConfig } from '@svelte-vitals/core';
 import { AST } from 'svelte/compiler';
-import { HeadTag, BranchStep, RuleOptionsSpec } from '@svelte-vitals/core/internal';
+import { HeadTag, BranchStep, SuppressionDirective, RuleOptionsSpec } from '@svelte-vitals/core/internal';
 
 /** A resolved import binding: which module, and which export ('default' for default imports). */
 interface ImportInfo {
@@ -58,6 +58,10 @@ interface ParsedA11y {
     slotInLandmark?: string;
     /** file contains {@html} or a spread attribute — poisons the closed world for no-missing-id-ref */
     unknowableContent: boolean;
+    /** Distinct lowercased tag names of the body's `RegularElement`s (a11y/required-element's presence set). */
+    elementTags: string[];
+    /** file contains `{@html}` or a `<svelte:element>` — either can render an element the walk cannot see */
+    elementsUnknowable: boolean;
 }
 interface ParsedFile {
     headTags: ParsedTag[];
@@ -66,6 +70,10 @@ interface ParsedFile {
     images: ParsedImage[];
     headings: ParsedHeading[];
     a11y: ParsedA11y;
+    /** Inline `svelte-vitals-disable-next-line` directives in this file, for the central
+     *  suppression pass. Collected here because a route-scoped finding can be located in any file
+     *  the composition reads, including ones no component-fact collection visited (`--route`). */
+    suppressions: SuppressionDirective[];
 }
 
 /**

package/dist/index.js

@@ -6,7 +6,7 @@ import {
   routeMatcher,
   run,
   spinnerEnabled
-} from "./chunk-LJPABOWU.js";
+} from "./chunk-5YVVJMZM.js";
 import {
   CONFIG_FILENAMES,
   loadConfigFile

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "svelte-vitals",
-  "version": "0.48.0",
+  "version": "0.49.0",
   "description": "A deterministic SvelteKit code-health scanner (SEO, performance, correctness, security, architecture, accessibility).",
   "type": "module",
   "license": "MIT",
@@ -52,7 +52,7 @@
     "magicast": "^0.5.4",
     "svelte": "^5.56.9",
     "tinyglobby": "^0.2.17",
-    "@svelte-vitals/core": "0.44.0"
+    "@svelte-vitals/core": "0.46.0"
   },
   "devDependencies": {
     "@gunshi/docs": "0.37.1",