svelte-vitals 0.49.0 → 0.50.0

package/dist/bin.js

@@ -1,12 +1,12 @@
 #!/usr/bin/env node
 import {
   runAnalyzeCliGunshi
-} from "./chunk-4FOGEXGH.js";
+} from "./chunk-QKULNWQ5.js";
 import "./chunk-DZVXCUHG.js";
 import {
   realIO
 } from "./chunk-RSSMIHIM.js";
-import "./chunk-5YVVJMZM.js";
+import "./chunk-PNHWRKRD.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-ZQZGCJ5W.js");
+      const { runCompleteCliGunshi } = await import("./complete-5VANQTDN.js");
       return { code: await runCompleteCliGunshi(argv, io), exit: "natural" };
     }
     if (argv[0] === "docs") {

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

@@ -42,6 +42,26 @@ import {
   terminalSafe
 } from "@svelte-vitals/core/internal";
 
+// src/a11y-skips.ts
+var ID_REF_RULE = "a11y/no-missing-id-ref";
+function buildIdRefSkips(a11y) {
+  return a11y.filter((r) => !r.fullyResolved).map((r) => ({ route: r.route, refs: r.idRefs.length, causes: r.unresolvedCauses ?? [] })).sort((a, b) => a.route.localeCompare(b.route));
+}
+var KIND_LABELS = [
+  ["component", "unresolved component"],
+  ["spread", "spread"],
+  ["html", "{@html}"],
+  ["dynamic-id", "dynamic id"]
+];
+function idRefSkipWarning(entries, analyzedRoutes) {
+  const parts = [];
+  for (const [kind, label] of KIND_LABELS) {
+    const n = entries.filter((e) => e.causes.some((c) => c.kind === kind)).length;
+    if (n > 0) parts.push(`${label} ${n}`);
+  }
+  return `${ID_REF_RULE} skipped ${entries.length} of ${analyzedRoutes} analyzed route(s) (${parts.join(", ")} \u2014 per-route detail in the JSON report's "skipped").`;
+}
+
 // src/runtime/node.ts
 import { readFile, access } from "fs/promises";
 import { join } from "path";
@@ -79,6 +99,7 @@ import {
   VITE_CONFIG_FILES,
   collectSuppressions,
   findMinifyDisabled,
+  lineOf,
   resolveKitAliases,
   resolveKitPathsBase
 } from "@svelte-vitals/core/internal";
@@ -168,9 +189,15 @@ function detectAppHtmlBodyTags(html) {
   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 keepNewlines = (m) => m.replace(/[^\n]/g, "");
+  const markup = html.replace(/<!--[\s\S]*?-->/g, keepNewlines).replace(/<script[\s\S]*?<\/script\s*>/gi, keepNewlines).replace(/<style[\s\S]*?<\/style\s*>/gi, keepNewlines);
   const found = markup.matchAll(/(?<![\w-])id\s*=\s*(?:"([^"]*)"|'([^']*)'|([^\s"'>{][^\s"'>]*))/gi);
-  return [...new Set([...found].map((m) => m[1] ?? m[2] ?? m[3] ?? "").filter(Boolean))];
+  const out = /* @__PURE__ */ new Map();
+  for (const m of found) {
+    const id = m[1] ?? m[2] ?? m[3] ?? "";
+    if (id && !out.has(id)) out.set(id, lineOf(markup, m.index));
+  }
+  return [...out].map(([id, line]) => ({ id, line }));
 }
 async function detectAppHtmlFacts(rt, cwd) {
   const appHtmlPath = rt.join(cwd, "src/app.html");
@@ -413,7 +440,7 @@ import {
   stripTextDirective,
   parseSvelte,
   CHILD_NODE_KEYS,
-  lineOf,
+  lineOf as lineOf2,
   findAttr as findAttr3,
   valueFromNodes,
   textFromNodes,
@@ -591,7 +618,7 @@ function collectImages(node, source, acc) {
       // A literal loading="lazy" only — a spread or dynamic loading={…} must not be flagged.
       lazy: attrText(attrs, "loading") === "lazy",
       hasSrcset: hasSpread || Boolean(findAttr3(attrs, "srcset")),
-      line: lineOf(source, node.start)
+      line: lineOf2(source, node.start)
     });
   }
   for (const key of CHILD_NODE_KEYS) {
@@ -606,7 +633,7 @@ function collectHeadings(node, source, acc) {
   if (!node || typeof node !== "object") return;
   if (node.type === "SvelteHead") return;
   if (node.type === "RegularElement" && /^h[1-6]$/.test(node.name)) {
-    acc.push({ level: Number(node.name[1]), line: lineOf(source, node.start) });
+    acc.push({ level: Number(node.name[1]), line: lineOf2(source, node.start) });
   }
   for (const key of CHILD_NODE_KEYS) {
     if (key in node) collectHeadings(childOf(node, key), source, acc);
@@ -627,7 +654,7 @@ function collectA11y(fragment, source) {
   const nodes = [];
   let groups = 0;
   let slotInLandmark;
-  let unknowableContent = false;
+  const unknowable = [];
   const elementTags = /* @__PURE__ */ new Set();
   let elementsUnknowable = false;
   const emit = (ctx, node) => {
@@ -636,9 +663,9 @@ function collectA11y(fragment, source) {
   };
   const noteSpread = (node) => {
     const attributes = node.attributes;
-    if (Array.isArray(attributes) && attributes.some((a) => a.type === "SpreadAttribute")) {
-      unknowableContent = true;
-    }
+    if (!Array.isArray(attributes)) return;
+    const spread = attributes.find((a) => a.type === "SpreadAttribute");
+    if (spread) unknowable.push({ kind: "spread", line: lineOf2(source, spread.start) });
   };
   const walk = (node, ctx) => {
     if (Array.isArray(node)) {
@@ -651,7 +678,7 @@ function collectA11y(fragment, source) {
         return;
       // head content never renders into the body
       case "HtmlTag":
-        unknowableContent = true;
+        unknowable.push({ kind: "html", line: lineOf2(source, node.start) });
         elementsUnknowable = true;
         return;
       case "IfBlock":
@@ -683,7 +710,7 @@ function collectA11y(fragment, source) {
       case "SvelteComponent":
       case "SvelteSelf":
         noteSpread(node);
-        emit(ctx, { kind: "component", key: node.name, line: lineOf(source, node.start) });
+        emit(ctx, { kind: "component", key: node.name, line: lineOf2(source, node.start) });
         walk(node.fragment, { ...ctx, elementDepth: ctx.elementDepth + 1 });
         return;
       case "SlotElement":
@@ -711,7 +738,7 @@ function collectA11y(fragment, source) {
   };
   const walkElement = (node, ctx) => {
     noteSpread(node);
-    const line = lineOf(source, node.start);
+    const line = lineOf2(source, node.start);
     const attrs = node.attributes;
     const roleAttr = findAttr3(attrs, "role");
     const role = roleAttr ? splitTokens(attrTextOf3(roleAttr))[0] : void 0;
@@ -759,7 +786,7 @@ function collectA11y(fragment, source) {
   return {
     nodes,
     ...slotInLandmark ? { slotInLandmark } : {},
-    unknowableContent,
+    unknowable,
     elementTags: [...elementTags],
     elementsUnknowable
   };
@@ -950,12 +977,23 @@ function groupSpan(nodes) {
   }
   return max + 1;
 }
+function dedupeCauses(causes) {
+  const seen = /* @__PURE__ */ new Map();
+  for (const c of causes) {
+    const key = `${c.kind}::${c.file}::${c.detail ?? ""}`;
+    if (!seen.has(key)) seen.set(key, c);
+  }
+  return [...seen.values()];
+}
 function offsetPath(path, base) {
   return base === 0 ? path : path.map((step) => ({ group: step.group + base, branch: step.branch }));
 }
 async function composeA11y(ctx, fileRel, parsed, depth, visited, chain) {
   const { rt, cwd, state } = ctx;
-  if (parsed.a11y.unknowableContent) state.fullyResolved = false;
+  if (parsed.a11y.unknowable.length > 0) {
+    state.fullyResolved = false;
+    for (const u of parsed.a11y.unknowable) state.causes.push({ ...u, file: fileRel });
+  }
   for (const t of parsed.a11y.elementTags) state.elementTags.add(t);
   if (parsed.a11y.elementsUnknowable) state.elementsClosed = false;
   const base = state.nextGroup;
@@ -971,6 +1009,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.causes.push({ kind: "component", detail: node.key, file: fileRel, line: node.line });
       state.elementsClosed = false;
       continue;
     }
@@ -1017,7 +1056,13 @@ async function resolveRoute(rt, cwd, pageRel, config, layouts, cache, aliases, a
     config,
     cache,
     aliases,
-    state: { nextGroup: 0, fullyResolved: true, elementTags: new Set(appHtmlBodyTags ?? []), elementsClosed: true }
+    state: {
+      nextGroup: 0,
+      fullyResolved: true,
+      causes: [],
+      elementTags: new Set(appHtmlBodyTags ?? []),
+      elementsClosed: true
+    }
   };
   const a11yNodes = [];
   const nestedLandmarks = [];
@@ -1059,8 +1104,20 @@ async function resolveRoute(rt, cwd, pageRel, config, layouts, cache, aliases, a
     }
   }
   const idNodes = a11yNodes.filter((n) => n.kind === "id");
-  if (idNodes.some((n) => n.key === "")) a11yCtx.state.fullyResolved = false;
+  for (const n of idNodes) {
+    if (n.key !== "") continue;
+    a11yCtx.state.fullyResolved = false;
+    a11yCtx.state.causes.push({ kind: "dynamic-id", file: n.file, line: n.line });
+  }
   const literalIds = idNodes.filter((n) => n.key !== "");
+  const ids = representatives(literalIds, chainOrder);
+  if (appHtmlIds) {
+    const shell = new Map(appHtmlIds.map((s) => [s.id, s.line]));
+    for (const key of Object.keys(ids)) {
+      const line = shell.get(key);
+      if (line !== void 0) ids[key] = [{ file: "src/app.html", line }, ...ids[key]];
+    }
+  }
   const route = deriveRoute(pageRel);
   return {
     head: { route, source: "static", tags: [...composed.values(), ...additiveTags], file: pageRel },
@@ -1073,12 +1130,13 @@ async function resolveRoute(rt, cwd, pageRel, config, layouts, cache, aliases, a
         chainOrder
       ),
       nestedLandmarks,
-      ids: representatives(literalIds, chainOrder),
+      ids,
       // `href="#top"` scrolls to the document top with no element of that id, so it is
       // 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 ?? []])],
+      idCandidates: [.../* @__PURE__ */ new Set([...literalIds.map((n) => n.key), ...(appHtmlIds ?? []).map((s) => s.id)])],
       fullyResolved: a11yCtx.state.fullyResolved,
+      ...a11yCtx.state.causes.length > 0 ? { unresolvedCauses: dedupeCauses(a11yCtx.state.causes) } : {},
       elementTags: [...a11yCtx.state.elementTags],
       elementsClosed: a11yCtx.state.elementsClosed,
       file: pageRel
@@ -1610,6 +1668,11 @@ function resolveRuleSelection(input) {
         else out[id] = rest;
       }
     }
+    for (const id of allowed) {
+      if (out[id] !== void 0) continue;
+      const rule = allRules.find((r) => r.id === id);
+      if (rule?.defaultOff) out[id] = rule.severity;
+    }
   }
   for (const id of input.ignoreRules ?? []) out[id] = "off";
   return out;
@@ -1678,6 +1741,8 @@ async function analyzeProject(opts = {}) {
   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;
+  const idRefSkips = rules.some((r) => r.id === ID_REF_RULE) ? buildIdRefSkips(a11y) : [];
+  if (idRefSkips.length > 0) warnings.push(idRefSkipWarning(idRefSkips, a11y.length));
   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)
@@ -1714,6 +1779,7 @@ async function analyzeProject(opts = {}) {
     version: readPackageVersion(),
     ruleIds: rules.map((r) => r.id),
     examined,
+    ...idRefSkips.length > 0 ? { skipped: { [ID_REF_RULE]: idRefSkips } } : {},
     failedRuleIds,
     warnings: [...warnings, ...skippedFileWarnings([...components, ...kitModules]), ...failedRuleWarnings(failedRules)],
     loadedConfig: loaded
@@ -1904,7 +1970,7 @@ async function run(opts = {}) {
         );
       }
       if (reporter === "json") {
-        log(formatJsonReport(results, config, { version }, analysis.ruleIds, analysis.examined));
+        log(formatJsonReport(results, config, { version }, analysis.ruleIds, analysis.examined, analysis.skipped));
       } else if (reporter === "agent") {
         log(formatAgentReport(results, config));
       } else if (reporter === "sarif") {

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

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

package/dist/{complete-ZQZGCJ5W.js → complete-5VANQTDN.js}

@@ -8,14 +8,14 @@ import {
 import "./chunk-CTMKTFST.js";
 import {
   ROOT_ARGS
-} from "./chunk-4FOGEXGH.js";
+} from "./chunk-QKULNWQ5.js";
 import {
   CATEGORIES,
   FAIL_ON_VALUES,
   TREAT_DYNAMIC_AS_VALUES
 } from "./chunk-DZVXCUHG.js";
 import "./chunk-RSSMIHIM.js";
-import "./chunk-5YVVJMZM.js";
+import "./chunk-PNHWRKRD.js";
 import {
   REPORTER_NAMES
 } from "./chunk-WGFXFKBB.js";

package/dist/gunshi-registry.js

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

package/dist/index.d.ts

@@ -1,4 +1,4 @@
-import { Config, Severity, RuleSetting, Category, Result } from '@svelte-vitals/core';
+import { Config, Severity, RuleSetting, Category, Result, JsonReport } from '@svelte-vitals/core';
 export { defineConfig } from '@svelte-vitals/core';
 import { AST } from 'svelte/compiler';
 import { HeadTag, BranchStep, SuppressionDirective, RuleOptionsSpec } from '@svelte-vitals/core/internal';
@@ -56,8 +56,11 @@ interface ParsedA11y {
     nodes: A11yNode[];
     /** landmark ancestor of this file's <slot>/{@render children()} position, if any */
     slotInLandmark?: string;
-    /** file contains {@html} or a spread attribute — poisons the closed world for no-missing-id-ref */
-    unknowableContent: boolean;
+    /** {@html} tags and spread attributes, located — each poisons the closed world for no-missing-id-ref */
+    unknowable: {
+        kind: 'spread' | 'html';
+        line: number;
+    }[];
     /** 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 */
@@ -276,9 +279,11 @@ interface AnalyzeResult {
     ruleIds: string[];
     /** Per-rule, per-declaration counts of places examined, unfiltered by `--diff`/`--baseline`/suppressions. */
     examined: Record<string, Record<string, number>>;
+    /** Routes a closed-world rule skipped, keyed by rule id — the analysis-side companion to `examined`; unfiltered by `--diff`/`--baseline`/suppressions. Absent when no analyzed route was skipped or the rule was not selected. */
+    skipped?: JsonReport['skipped'];
     /** Ids of rules `runRules` caught throwing — already folded into `config` via `withFailedRulesOff`; exposed separately so a caller with its own base config (the vite dev dashboard) can apply the same correction without adopting this call's `config`. */
     failedRuleIds: string[];
-    /** 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. */
+    /** Non-fatal issues surfaced during analysis: config-file problems (unknown top-level keys, invalid enum values), version-floor notices, `--rules`/overrides conflicts, closed-world skip notices (`a11y/no-missing-id-ref`), 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

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "svelte-vitals",
-  "version": "0.49.0",
+  "version": "0.50.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.46.0"
+    "@svelte-vitals/core": "0.47.0"
   },
   "devDependencies": {
     "@gunshi/docs": "0.37.1",