svelte-shaker 0.10.1 → 0.11.0

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/dist/svelte_shaker_engine.js

@@ -51,6 +51,27 @@ function analyze_program(input_json) {
 }
 exports.analyze_program = analyze_program;
 
+/**
+ * JSON-string wrapper of {@link find_never_passed_props} for the WASM boundary.
+ * @param {string} input_json
+ * @returns {string}
+ */
+function find_never_passed_props_json(input_json) {
+    let deferred2_0;
+    let deferred2_1;
+    try {
+        const ptr0 = passStringToWasm0(input_json, wasm.__wbindgen_malloc, wasm.__wbindgen_realloc);
+        const len0 = WASM_VECTOR_LEN;
+        const ret = wasm.find_never_passed_props_json(ptr0, len0);
+        deferred2_0 = ret[0];
+        deferred2_1 = ret[1];
+        return getStringFromWasm0(ret[0], ret[1]);
+    } finally {
+        wasm.__wbindgen_free(deferred2_0, deferred2_1, 1);
+    }
+}
+exports.find_never_passed_props_json = find_never_passed_props_json;
+
 /**
  * Whole-program shake: analyze + transform.  `input` is `{ files: [{id, ast,
  * code}], edges, entries }`.  Returns `{ id: slimmedSource }` for every file —

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "svelte-shaker",
-  "version": "0.10.1",
+  "version": "0.11.0",
   "description": "Tree shaking for Svelte components",
   "keywords": [
     "dead-code-elimination",