@svelte-vitals/core 0.23.0 → 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;

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",
@@ -2516,7 +2525,7 @@ function formatConsoleReport(results, config, options = {}) {
     lines.push("");
   }
   if (options.byRoute) lines.push(...byRouteTree(p, results, config, options.verbose ?? false));
-  if (summary.dynamic > 0) lines.push(p.dim("\u21AF = set dynamically (verified at runtime)."));
+  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.23.0",
+  "version": "0.24.0",
   "description": "Shared, runtime-agnostic core for svelte-vitals (types, rule engine, scorer, reporter).",
   "type": "module",
   "license": "MIT",