svelte-shaker 0.10.0 → 0.10.2

package/dist/analyze.js

@@ -116,6 +116,12 @@ export async function buildAnalyzeInput(entries, resolve, readFile, parseCache,
         const instance = ast.instance;
         if (!instance)
             continue;
+        // The bare component tags this file renders (`<Local …>`). Resolving a barrel
+        // (a `.js`/`.ts` re-export) means READING and PARSING the target module to
+        // chase the export, so we do it ONLY for named imports actually rendered as a
+        // component here — a named import used as a value (a helper / type) can never
+        // be a `<Local>` call site, so chasing it is pure waste.
+        const renderedTags = renderedComponentTagNames(ast);
         // Resolve this file's imports into the three attributable edge kinds.  Direct
         // default `.svelte` and simple barrel/named imports bind a bare local; a
         // namespace import (`import * as ns`) binds no single component, so it is
@@ -136,6 +142,9 @@ export async function buildAnalyzeInput(entries, resolve, readFile, parseCache,
                 }
                 continue;
             }
+            // Not rendered as `<imp.local>` -> not a call site -> skip the costly barrel read.
+            if (!renderedTags.has(imp.local))
+                continue;
             const childId = await resolveThroughBarrel(imp.value, imp.imported, id, resolve, readFile);
             if (childId) {
                 edges.push({ from: id, local: imp.local, to: childId, kind: 'barrel' });
@@ -163,9 +172,9 @@ export async function buildAnalyzeInput(entries, resolve, readFile, parseCache,
             }
         }
         // Enqueue every child this file renders: direct `.svelte`, the barrel children
-        // it renders, and the namespace members it renders.
-        const rendered = collectBarrelChildIds(ast, barrelLocals);
-        for (const childId of [...directChildren, ...rendered, ...nsChildren]) {
+        // it renders (`barrelLocals` already holds only rendered locals), and the
+        // namespace members it renders.
+        for (const childId of [...directChildren, ...barrelLocals.values(), ...nsChildren]) {
             if (!seen.has(childId)) {
                 seen.add(childId);
                 queue.push(childId);
@@ -194,6 +203,10 @@ export function buildAnalyzeInputSync(entries, resolve, readFile, parseCache, pa
         const instance = ast.instance;
         if (!instance)
             continue;
+        // See {@link buildAnalyzeInput}: resolve a barrel only for named imports
+        // actually rendered as a `<Local>` component here, to avoid reading+parsing
+        // modules behind value-only named imports.
+        const renderedTags = renderedComponentTagNames(ast);
         const barrelLocals = new Map();
         const namespaceSources = new Map();
         const directChildren = [];
@@ -210,6 +223,8 @@ export function buildAnalyzeInputSync(entries, resolve, readFile, parseCache, pa
                 }
                 continue;
             }
+            if (!renderedTags.has(imp.local))
+                continue;
             const childId = resolveThroughBarrelSync(imp.value, imp.imported, id, resolve, readFile);
             if (childId) {
                 edges.push({ from: id, local: imp.local, to: childId, kind: 'barrel' });
@@ -230,8 +245,7 @@ export function buildAnalyzeInputSync(entries, resolve, readFile, parseCache, pa
                 }
             }
         }
-        const rendered = collectBarrelChildIds(ast, barrelLocals);
-        for (const childId of [...directChildren, ...rendered, ...nsChildren]) {
+        for (const childId of [...directChildren, ...barrelLocals.values(), ...nsChildren]) {
             if (!seen.has(childId)) {
                 seen.add(childId);
                 queue.push(childId);
@@ -786,24 +800,25 @@ function collectChildCalls(ast, imports) {
     return calls;
 }
 /**
- * The set of barrel-resolved children this file actually RENDERS as `<Comp/>`.
- * Used by the Shell crawl to enqueue only the barrel children a file renders — an
- * unused barrel import resolves to a component nothing instantiates, so following
- * it would pull a never-rendered file into the program for no reason.
+ * The bare component tag names this file RENDERS (`<Local/>`, excluding dotted
+ * `<ns.X/>` member tags). The Shell crawl uses this to resolve a barrel (a
+ * `.js`/`.ts` re-export, which costs a module read+parse) only for named imports
+ * actually rendered as a component — a value-only named import (a helper / type)
+ * is never a `<Local>` call site, so following it would read+parse a module for
+ * nothing. Skipping it only ever drops a non-call-site, so attribution (and the
+ * resulting models) are unchanged.
  */
-function collectBarrelChildIds(ast, barrelLocals) {
-    const ids = new Set();
-    if (barrelLocals.size === 0)
-        return ids;
+function renderedComponentTagNames(ast) {
+    const names = new Set();
     walk(ast.fragment, null, {
         Component(node, { next }) {
-            const childId = node.name ? barrelLocals.get(node.name) : undefined;
-            if (childId)
-                ids.add(childId);
+            if (typeof node.name === 'string' && node.name !== '' && !node.name.includes('.')) {
+                names.add(node.name);
+            }
             next();
         },
     });
-    return ids;
+    return names;
 }
 /**
  * Every dotted component tag a file renders (`<ns.Child/>` -> `"ns.Child"`).  The

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "svelte-shaker",
-  "version": "0.10.0",
+  "version": "0.10.2",
   "description": "Tree shaking for Svelte components",
   "keywords": [
     "dead-code-elimination",
@@ -30,15 +30,18 @@
   "exports": {
     ".": {
       "types": "./dist/index.d.ts",
-      "import": "./dist/index.js"
+      "import": "./dist/index.js",
+      "default": "./dist/index.js"
     },
     "./vite": {
       "types": "./dist/vite.d.ts",
-      "import": "./dist/vite.js"
+      "import": "./dist/vite.js",
+      "default": "./dist/vite.js"
     },
     "./node": {
       "types": "./dist/scan.d.ts",
-      "import": "./dist/scan.js"
+      "import": "./dist/scan.js",
+      "default": "./dist/scan.js"
     },
     "./package.json": "./package.json"
   },