@dcoder-x/plugin-shared 0.1.14 → 0.1.15

package/dist/extractors/ComponentExtractor.js

@@ -14,17 +14,21 @@ const HtmlTagGuards_1 = require("../injection/HtmlTagGuards");
 const IdStrategy_1 = require("../injection/IdStrategy");
 /**
  * A route is a usable module-graph entry point only when its filePath
- * points to an actual page component, not a layout/guard/wrapper.
- * Score 0 = structural wrapper → skip as entry point.
+ * points to an actual page component, not a layout/guard/wrapper/router file.
  */
 function isPageRoute(route) {
     if (route.isLayoutRoute)
         return false;
     const p = route.filePath.replace(/\\/g, '/').toLowerCase();
+    const base = path_1.default.basename(p).replace(/\.(tsx?|jsx?)$/, '');
     if (p.includes('/pages/') || p.includes('/views/') || p.includes('/screens/'))
         return true;
-    if (/\/(contexts?|providers?|guards?|layouts?|wrappers?|hoc|utils?|helpers?)\//.test(p) ||
-        /(provider|context|guard|layout|redirect|catchall|catch-all)/.test(path_1.default.basename(p)))
+    // Directory-based structural patterns
+    if (/\/(contexts?|providers?|guards?|layouts?|wrappers?|hoc|utils?|helpers?)\//.test(p))
+        return false;
+    // Basename-based structural patterns: wrapper, router config, and provider files
+    if (/(provider|context|guard|layout|redirect|catchall|catch-all)/.test(base) ||
+        /^(router|routes|routerconfig|approuter|routesconfig|routingconfig|routeconfig|approviders|appwrapper)$/i.test(base))
         return false;
     return true;
 }
@@ -413,10 +417,21 @@ function extractStaticProps(attributes) {
             continue;
         if (attr.value?.type === 'StringLiteral') {
             props[key] = attr.value.value;
+            continue;
         }
-        if (attr.value?.type === 'JSXExpressionContainer' &&
-            attr.value.expression?.type === 'StringLiteral') {
-            props[key] = attr.value.expression.value;
+        if (attr.value?.type === 'JSXExpressionContainer') {
+            const expr = attr.value.expression;
+            if (expr?.type === 'StringLiteral') {
+                props[key] = expr.value;
+            }
+            else if (expr?.type === 'TemplateLiteral' &&
+                expr.quasis?.length === 1 &&
+                expr.expressions?.length === 0) {
+                // Pure template literal with no interpolations: `to="/some/path"`
+                const raw = expr.quasis[0].value.cooked ?? expr.quasis[0].value.raw;
+                if (raw)
+                    props[key] = raw;
+            }
         }
     }
     return props;

package/dist/extractors/FlowInferrer.d.ts

@@ -14,6 +14,12 @@ export declare class FlowInferrer {
      */
     private isCoherentChain;
     private detectInteractionFlows;
+    /**
+     * Deduplicate interaction flows by (page, trigger event, effect target).
+     * Shared components used on multiple routes produce one flow per component
+     * interaction. After dedup, each distinct interaction appears once.
+     */
+    private deduplicateInteractionFlows;
     private buildChain;
     private generateIntentPatterns;
     /**

package/dist/extractors/FlowInferrer.js

@@ -45,10 +45,9 @@ class FlowInferrer {
     infer() {
         const edges = this.buildEdgeGraph();
         const routeFlows = this.detectLinearFlows(edges);
-        const interactionFlows = this.detectInteractionFlows();
+        const interactionFlows = this.deduplicateInteractionFlows(this.detectInteractionFlows());
         const combined = [...routeFlows, ...interactionFlows];
-        // Fix 4: remove patterns that appear in too many flows — they are structural
-        // navigation chrome (sidebars, footers) rather than user intent signals
+        // Remove patterns that appear in too many flows — structural nav chrome
         return this.deduplicateCrossFlowNoise(combined);
     }
     // ---------------------------------------------------------------------------
@@ -56,13 +55,21 @@ class FlowInferrer {
     // ---------------------------------------------------------------------------
     buildEdgeGraph() {
         const edges = [];
+        // Index routes both with and without leading slash to handle mixed conventions
         const routePaths = new Set(this.routes.map((r) => r.path));
+        const normalizeHref = (href) => href.replace(/^\//, '');
         for (const el of this.elements) {
             const isNav = el.tag === 'a' || el.isNavigationLink === true;
-            const href = el.staticProps['href'] || el.staticProps['to'];
-            if (!isNav || !href)
+            const rawHref = el.staticProps['href'] || el.staticProps['to'];
+            if (!isNav || !rawHref)
                 continue;
-            if (!routePaths.has(href))
+            // Match against both the raw href and the normalized (no leading slash) form
+            const href = routePaths.has(rawHref)
+                ? rawHref
+                : routePaths.has(normalizeHref(rawHref))
+                    ? normalizeHref(rawHref)
+                    : null;
+            if (!href)
                 continue;
             const triggerText = el.staticProps['aria-label'] ||
                 el.label ||
@@ -162,6 +169,26 @@ class FlowInferrer {
         return flows;
     }
     // ---------------------------------------------------------------------------
+    // Interaction flow deduplication
+    // ---------------------------------------------------------------------------
+    /**
+     * Deduplicate interaction flows by (page, trigger event, effect target).
+     * Shared components used on multiple routes produce one flow per component
+     * interaction. After dedup, each distinct interaction appears once.
+     */
+    deduplicateInteractionFlows(flows) {
+        const seen = new Set();
+        return flows.filter((f) => {
+            const trigger = f.edges[0]?.trigger ?? '';
+            const step = f.steps[0] ?? '';
+            const sig = `${f.page}::${trigger}::${step}`;
+            if (seen.has(sig))
+                return false;
+            seen.add(sig);
+            return true;
+        });
+    }
+    // ---------------------------------------------------------------------------
     // Chain builder
     // ---------------------------------------------------------------------------
     buildChain(start, adjacency, seen) {

package/dist/extractors/RouteExtractor.js

@@ -18,16 +18,20 @@ const traverse = typeof traverse_1.default === 'function' ? traverse_1.default :
 /**
  * Score a resolved file path: higher = more likely to be an actual page component.
  * 3 = pages/views/screens directory → definitely a page
- * 2 = named after a recognisable page pattern but not in a pages dir
  * 1 = unknown / ambiguous
- * 0 = structural wrapper (provider, guard, layout, context, …)
+ * 0 = structural file (provider, guard, layout, context, router config, …)
  */
 function scoreFilePath(filePath) {
     const p = filePath.replace(/\\/g, '/').toLowerCase();
+    const base = path_1.default.basename(p).replace(/\.(tsx?|jsx?)$/, '');
     if (p.includes('/pages/') || p.includes('/views/') || p.includes('/screens/'))
         return 3;
-    if (/\/(contexts?|providers?|guards?|layouts?|wrappers?|hoc|utils?|helpers?)\//.test(p) ||
-        /(provider|context|guard|layout|redirect|catchall|catch-all)/.test(path_1.default.basename(p)))
+    // Directory-based structural patterns
+    if (/\/(contexts?|providers?|guards?|layouts?|wrappers?|hoc|utils?|helpers?)\//.test(p))
+        return 0;
+    // Basename-based structural patterns: wrapper/config/router files
+    if (/(provider|context|guard|layout|redirect|catchall|catch-all)/.test(base) ||
+        /^(router|routes|routerconfig|approuter|routesconfig|routingconfig|routeconfig|approviders|appwrapper)$/i.test(base))
         return 0;
     return 1;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dcoder-x/plugin-shared",
-  "version": "0.1.14",
+  "version": "0.1.15",
   "private": false,
   "main": "dist/index.js",
   "types": "dist/index.d.ts",