@svelte-vitals/core 0.22.1 → 0.24.0

package/dist/index.d.ts

@@ -247,6 +247,11 @@ interface ComponentFacts {
     propCount: number;
     /** Module specifiers of every `import` in the instance + module scripts (Bundle PERF009). */
     imports: string[];
+    /** Module specifiers of every `import`, each with its source line (Bundle PERF009). */
+    importSpans: {
+        source: string;
+        line: number;
+    }[];
     /** Value `import * as X from '<bare pkg>'` namespace imports (type-only excluded) — Bundle PERF010. */
     namespaceImports: {
         source: string;
@@ -275,6 +280,10 @@ declare function parseComponentFacts(source: string, filename: string): {
     loc: number;
     propCount: number;
     imports: string[];
+    importSpans: {
+        source: string;
+        line: number;
+    }[];
     namespaceImports: {
         source: string;
         line: number;
@@ -629,6 +638,10 @@ interface ConsoleReportOptions {
     mode?: string;
     /** Color decorators; defaults to no color. */
     palette?: Palette;
+    /** Show every failing/passed/route entry uncapped and ungrouped, exactly as before this option existed. Default false (capped, grouped by rule). */
+    verbose?: boolean;
+    /** Internal: set by the CLI when it has already animated the Health header itself — skips the brand/Health lines (category score lines still print). Default false. */
+    omitHeader?: boolean;
 }
 /**
  * Render results as a console report string (design §7). Pure: returns a string,

package/dist/index.js

@@ -419,9 +419,11 @@ function countLines(source) {
   if (source.length === 0) return 0;
   return source.split("\n").length - (source.endsWith("\n") ? 1 : 0);
 }
-function collectImportSources(program, acc) {
+function collectImportSources(program, source, acc) {
   walkEstree(program, (n) => {
-    if (n.type === "ImportDeclaration" && typeof n.source?.value === "string") acc.push(n.source.value);
+    if (n.type === "ImportDeclaration" && typeof n.source?.value === "string") {
+      acc.push({ source: n.source.value, line: lineOf(source, n.start) });
+    }
   });
 }
 function isBareSpecifier(s) {
@@ -459,10 +461,10 @@ function parseComponentFacts(source, filename) {
   collectSecurityFacts(ast.fragment ?? ast, source, htmlTags, javascriptUrls);
   const loc = countLines(source);
   const suppressions = collectSuppressions(source);
-  const imports = [];
+  const importSpans = [];
   const namespaceImports = [];
   if (ast.module?.content) {
-    collectImportSources(ast.module.content, imports);
+    collectImportSources(ast.module.content, source, importSpans);
     collectNamespaceImports(ast.module.content, source, namespaceImports);
   }
   const effects = [];
@@ -471,7 +473,7 @@ function parseComponentFacts(source, filename) {
   let propCount = 0;
   const program = ast.instance?.content;
   if (program) {
-    collectImportSources(program, imports);
+    collectImportSources(program, source, importSpans);
     collectNamespaceImports(program, source, namespaceImports);
     propCount = countProps(program);
     const nonBindableProps = collectNonBindableProps(program);
@@ -509,6 +511,7 @@ function parseComponentFacts(source, filename) {
       if (!writtenOrEscaped.has(d.name)) constableStates.push(d);
     }
   }
+  const imports = importSpans.map((s) => s.source);
   return {
     eachBlocks,
     effects,
@@ -517,6 +520,7 @@ function parseComponentFacts(source, filename) {
     loc,
     propCount,
     imports,
+    importSpans,
     namespaceImports,
     constableStates,
     mutatedProps,
@@ -535,6 +539,7 @@ function emptyComponentFacts(file) {
     loc: 0,
     propCount: 0,
     imports: [],
+    importSpans: [],
     namespaceImports: [],
     constableStates: [],
     mutatedProps: [],
@@ -2143,7 +2148,7 @@ var arch002PropCount = componentRule({
   bad: (c) => c.propCount > MAX_PROPS ? [{ line: 1, message: `Component takes ${c.propCount} props (over ${MAX_PROPS})` }] : []
 });
 
-// src/rules/performance/perf009-heavy-import.ts
+// src/rules/perf/perf009-heavy-import.ts
 var HEAVY_PACKAGES = {
   lodash: "import a submodule (lodash/debounce) or use lodash-es for tree-shaking",
   moment: "use a lighter date library (date-fns or dayjs) \u2014 moment is large and not tree-shakeable"
@@ -2156,20 +2161,24 @@ var perf009HeavyImport = componentRule({
   label: "No heavy imports",
   recommendation: "Import a submodule or switch to a lighter, tree-shakeable alternative.",
   rationale: "Importing a large, non-tree-shakeable package pulls its whole weight into the bundle even when only a fraction is used, slowing load.",
-  applies: (c) => c.imports.length > 0,
+  // ComponentFacts is a public @svelte-vitals/core export — an external caller compiled
+  // against an older version may still construct one without importSpans. Fall back to the
+  // line-less `imports` (line: 0, the pre-fix behavior) instead of crashing on `undefined`.
+  applies: (c) => (c.importSpans ?? c.imports).length > 0,
   bad: (c) => {
     const seen = /* @__PURE__ */ new Set();
     const out = [];
-    for (const src of c.imports) {
+    const spans = c.importSpans ?? c.imports.map((source) => ({ source, line: 0 }));
+    for (const { source: src, line } of spans) {
       if (!Object.hasOwn(HEAVY_PACKAGES, src) || seen.has(src)) continue;
       seen.add(src);
-      out.push({ line: 0, message: `Heavy import "${src}" \u2014 ${HEAVY_PACKAGES[src]}` });
+      out.push({ line, message: `Heavy import "${src}" \u2014 ${HEAVY_PACKAGES[src]}` });
     }
     return out;
   }
 });
 
-// src/rules/performance/perf010-namespace-import.ts
+// src/rules/perf/perf010-namespace-import.ts
 var perf010NamespaceImport = componentRule({
   id: "PERF010",
   title: "Namespace import",
@@ -2399,27 +2408,53 @@ var CATEGORY_LABEL = {
   architecture: "Architecture"
 };
 var CATEGORY_ORDER = ["seo", "performance", "correctness", "security", "architecture"];
+var MAX_RULE_GROUPS_PER_BUCKET = 5;
+function groupByRule(results) {
+  const groups = /* @__PURE__ */ new Map();
+  for (const r of results) {
+    const bucket = groups.get(r.id);
+    if (bucket) bucket.push(r);
+    else groups.set(r.id, [r]);
+  }
+  return [...groups.entries()].map(([id, rs]) => ({ id, results: rs })).sort((a, b) => b.results.length - a.results.length || a.id.localeCompare(b.id));
+}
 function scoreLine(p, label, { score, scoreModel }) {
   const parts = [`route avg ${scoreModel.routeAverage}`];
   if (scoreModel.sitePenalty > 0) parts.push(`site \u2212${scoreModel.sitePenalty}`);
   if (scoreModel.criticalCap !== null) parts.push(`capped at ${scoreModel.criticalCap}: critical present`);
   return `${label} Score: ${scoreColor(p, score)(`${score}/100`)}   ${p.dim(`(${parts.join(" \xB7 ")})`)}`;
 }
-function byRouteTree(p, results, config) {
+var MAX_ROUTES_BY_ROUTE = 10;
+function byRouteTree(p, results, config, verbose) {
   const routes = /* @__PURE__ */ new Map();
   for (const r of results) {
     if (r.route === void 0) continue;
     if (!routes.has(r.route)) routes.set(r.route, []);
     routes.get(r.route).push(r);
   }
+  const scored = [...routes.entries()].map(([route, rs]) => ({
+    route,
+    rs,
+    score: computeScore(rs, config, { applyCriticalCap: false }).score
+  }));
+  scored.sort((a, b) => a.score - b.score || a.route.localeCompare(b.route));
+  const shown = verbose ? scored : scored.slice(0, MAX_ROUTES_BY_ROUTE);
   const lines = [p.bold("By route"), p.dim(RULE)];
-  for (const [route, rs] of [...routes.entries()].sort((a, b) => a[0].localeCompare(b[0]))) {
-    const { score } = computeScore(rs, config, { applyCriticalCap: false });
+  for (const { route, rs, score } of shown) {
     lines.push(`${route.padEnd(28)} ${scoreColor(p, score)(`${score}`)}`);
     for (const r of rs.filter((x) => classify(x, config) === "fail")) {
       lines.push(`    ${p.red("\u2717")} ${r.id}  ${r.message}`);
     }
   }
+  if (!verbose && scored.length > MAX_ROUTES_BY_ROUTE) {
+    const remaining = scored.slice(MAX_ROUTES_BY_ROUTE);
+    const avgScore = Math.round(remaining.reduce((sum, r) => sum + r.score, 0) / remaining.length);
+    lines.push(
+      p.dim(
+        `\u2026and ${remaining.length} more route${remaining.length > 1 ? "s" : ""} (avg score ${avgScore}) \u2014 run with --verbose to see all`
+      )
+    );
+  }
   lines.push("");
   return lines;
 }
@@ -2428,15 +2463,18 @@ function formatConsoleReport(results, config, options = {}) {
   const summary = summarize(results, config);
   const { health, categories: byCat } = computeHealth(results, config);
   const present2 = CATEGORY_ORDER.filter((c) => byCat[c] !== void 0);
-  const header = [
-    p.bold(`Svelte Vitals  \xB7  ${options.mode ?? "static mode"}`),
-    "",
-    `${p.bold("Health:")} ${scoreColor(p, health)(`${health}/100`)}`
-  ];
+  const lines = [];
+  if (!options.omitHeader) {
+    lines.push(
+      p.bold(`Svelte Vitals  \xB7  ${options.mode ?? "static mode"}`),
+      "",
+      `${p.bold("Health:")} ${scoreColor(p, health)(`${health}/100`)}`
+    );
+  }
   for (const c of present2) {
-    header.push(scoreLine(p, CATEGORY_LABEL[c] ?? c, byCat[c]));
+    lines.push(scoreLine(p, CATEGORY_LABEL[c] ?? c, byCat[c]));
   }
-  const lines = [...header, ""];
+  lines.push("");
   const SEVERITY_COLOR = {
     critical: (s) => p.red(p.bold(s)),
     warning: (s) => p.yellow(p.bold(s)),
@@ -2447,25 +2485,47 @@ function formatConsoleReport(results, config, options = {}) {
     const bucket = failures.filter((r) => effectiveSeverity(r, config) === severity);
     if (bucket.length === 0) continue;
     lines.push(SEVERITY_COLOR[severity](`${SEVERITY_TITLE[severity]} (${bucket.length})`), p.dim(RULE));
-    for (const r of bucket) {
-      lines.push(`${p.red("\u2717")} ${r.id}  ${r.message}`);
-      if (r.route) lines.push(p.dim(`            ${r.route}`));
-      if (r.location) lines.push(p.dim(`            ${r.location}${r.line ? `:${r.line}` : ""}`));
+    if (options.verbose) {
+      for (const r of bucket) {
+        lines.push(`${p.red("\u2717")} ${r.id}  ${r.message}`);
+        if (r.route) lines.push(p.dim(`            ${r.route}`));
+        if (r.location) lines.push(p.dim(`            ${r.location}${r.line ? `:${r.line}` : ""}`));
+      }
+    } else {
+      const groups = groupByRule(bucket);
+      const shownGroups = groups.slice(0, MAX_RULE_GROUPS_PER_BUCKET);
+      for (const group of shownGroups) {
+        const r = group.results[0];
+        lines.push(`${p.red("\u2717")} ${r.id}  ${r.message}`);
+        if (r.route) lines.push(p.dim(`            ${r.route}`));
+        if (r.location) lines.push(p.dim(`            ${r.location}${r.line ? `:${r.line}` : ""}`));
+        if (group.results.length > 1) {
+          lines.push(p.dim(`            \u2026and ${group.results.length - 1} more`));
+        }
+      }
+      if (groups.length > MAX_RULE_GROUPS_PER_BUCKET) {
+        const remaining = groups.length - MAX_RULE_GROUPS_PER_BUCKET;
+        lines.push(
+          p.dim(`\u2026and ${remaining} more rule${remaining > 1 ? "s" : ""} affected \u2014 run with --verbose to see all`)
+        );
+      }
     }
     lines.push("");
   }
   const passed = results.filter((r) => classify(r, config) !== "fail");
   if (passed.length > 0) {
     lines.push(p.bold(`Passed (${passed.length})`), p.dim(RULE));
-    for (const r of passed) {
-      const marker = classify(r, config) === "dynamic" ? p.cyan("  \u21AF dynamic") : "";
-      const route = r.route ? `  ${r.route}` : "";
-      lines.push(`${p.green("\u2713")} ${r.id}  ${r.message}${marker}${route}`);
+    if (options.verbose) {
+      for (const r of passed) {
+        const marker = classify(r, config) === "dynamic" ? p.cyan("  \u21AF dynamic") : "";
+        const route = r.route ? `  ${r.route}` : "";
+        lines.push(`${p.green("\u2713")} ${r.id}  ${r.message}${marker}${route}`);
+      }
     }
     lines.push("");
   }
-  if (options.byRoute) lines.push(...byRouteTree(p, results, config));
-  if (summary.dynamic > 0) lines.push(p.dim("\u21AF = set dynamically (verified at runtime)."));
+  if (options.byRoute) lines.push(...byRouteTree(p, results, config, options.verbose ?? false));
+  if (options.verbose && summary.dynamic > 0) lines.push(p.dim("\u21AF = set dynamically (verified at runtime)."));
   return lines.join("\n").replace(/\n+$/, "\n");
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@svelte-vitals/core",
-  "version": "0.22.1",
+  "version": "0.24.0",
   "description": "Shared, runtime-agnostic core for svelte-vitals (types, rule engine, scorer, reporter).",
   "type": "module",
   "license": "MIT",