vue-hook-optimizer 0.0.57 → 0.0.59

package/dist/index.mjs

@@ -1979,23 +1979,72 @@ function noIndegreeFilter(graph) {
 function findLinearPaths(graph) {
   const linearPaths = [];
   const visitedNodes = /* @__PURE__ */ new Set();
+  const nodeInDegrees = /* @__PURE__ */ new Map();
   for (const [node, edges] of graph.entries()) {
-    if (edges.size === 1 && !visitedNodes.has(node)) {
-      const path = [node];
-      let nextNode = Array.from(edges)[0];
-      visitedNodes.add(node);
-      while (graph.get(nextNode)?.size === 1) {
-        if (visitedNodes.has(nextNode)) {
+    if (!nodeInDegrees.has(node)) {
+      nodeInDegrees.set(node, 0);
+    }
+    for (const edge of edges) {
+      const inDegree = nodeInDegrees.get(edge) || 0;
+      nodeInDegrees.set(edge, inDegree + 1);
+    }
+  }
+  function dfs2(node, path) {
+    if (visitedNodes.has(node)) {
+      return;
+    }
+    path.push(node);
+    visitedNodes.add(node);
+    const edges = graph.get(node) || /* @__PURE__ */ new Set();
+    if (edges.size === 0 || edges.size > 1) {
+      if (path.length > 1) {
+        addOrUpdatePath([...path]);
+      }
+    } else {
+      const nextNode = Array.from(edges)[0];
+      const nextNodeInDegree = nodeInDegrees.get(nextNode) || 0;
+      if (nextNodeInDegree === 1) {
+        dfs2(nextNode, path);
+      }
+    }
+    path.pop();
+    visitedNodes.delete(node);
+  }
+  function addOrUpdatePath(newPath) {
+    let shouldAddNewPath = true;
+    for (let i = linearPaths.length - 1; i >= 0; i--) {
+      const existingPath = linearPaths[i];
+      if (isSubpath(existingPath, newPath)) {
+        linearPaths.splice(i, 1);
+      } else if (isSubpath(newPath, existingPath)) {
+        shouldAddNewPath = false;
+        break;
+      }
+    }
+    if (shouldAddNewPath && newPath.length > 2) {
+      linearPaths.push(newPath);
+    }
+  }
+  function isSubpath(shortPath, longPath) {
+    if (shortPath.length >= longPath.length) {
+      return false;
+    }
+    for (let i = 0; i <= longPath.length - shortPath.length; i++) {
+      let isSub = true;
+      for (let j = 0; j < shortPath.length; j++) {
+        if (shortPath[j] !== longPath[i + j]) {
+          isSub = false;
           break;
         }
-        path.push(nextNode);
-        visitedNodes.add(nextNode);
-        nextNode = Array.from(graph.get(nextNode))[0];
       }
-      if (path.length > 1) {
-        linearPaths.push(path);
+      if (isSub) {
+        return true;
       }
     }
+    return false;
+  }
+  for (const node of graph.keys()) {
+    dfs2(node, []);
   }
   return linearPaths;
 }
@@ -2112,11 +2161,16 @@ function gen(graph, nodesUsedInTemplate, nodesUsedInStyle = /* @__PURE__ */ new
     }
     const paths = findLinearPaths(g);
     paths.forEach((path) => {
-      suggestions.push({
-        type: "warning" /* warning */,
-        message: `Nodes [${path.length > 10 ? `${path.slice(0, 10).map((node) => node.label).join(",")}...(${path.length})` : path.map((node) => node.label).join(",")}] are have function chain calls, perhaps you can refactor it.`,
-        nodeInfo: path
-      });
+      const firstUsedNodeIndex = path.findIndex((node) => usedNodes.has(node.label));
+      const reverseLastNotUsedNodeIndex = path.slice().reverse().findIndex((node) => !usedNodes.has(node.label));
+      const lastNotUsedNodeIndex = reverseLastNotUsedNodeIndex !== -1 ? path.length - 1 - reverseLastNotUsedNodeIndex : -1;
+      if (firstUsedNodeIndex > -1 && firstUsedNodeIndex < lastNotUsedNodeIndex) {
+        suggestions.push({
+          type: "warning" /* warning */,
+          message: `Nodes [${path.length > 10 ? `${path.slice(0, 10).map((node) => node.label).join(",")}...(${path.length})` : path.map((node) => node.label).join(",")}] are have function chain calls, perhaps you can refactor it.`,
+          nodeInfo: path
+        });
+      }
     });
     if (g.size > 5) {
       const ap = findArticulationPoints(g);