From 5ebbf9527f9bde73a94ecdfdc8c3f32c3363ce1b Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Tue, 30 Jun 2026 12:51:04 +0000
Subject: [PATCH] Optimize detectArchitectureFramework execution speed

Co-authored-by: julesklord <801266+julesklord@users.noreply.github.com>
---
 pr-description.txt | 10 ++++++++++
 src/lib/parser.js  | 40 ++++++++++++++++++++++++++++++----------
 2 files changed, 40 insertions(+), 10 deletions(-)
 create mode 100644 pr-description.txt

diff --git a/pr-description.txt b/pr-description.txt
new file mode 100644
index 0000000..904cf92
--- /dev/null
+++ b/pr-description.txt
@@ -0,0 +1,10 @@
+⚡ Optimize detectArchitectureFramework execution speed
+
+💡 **What:** Replaced the use of multiple `.some()` calls and a `.map()` chained operation in `detectArchitectureFramework` with a single, optimized `for` loop that iterates over the scan files.
+
+🎯 **Why:** The previous implementation traversed the array of files up to 5 times due to the `.map` and consecutive `.some` calls. This change processes each file exactly once and can return early if the highest-priority framework (`Next.js`) is found, avoiding unnecessary iterations and regular expression tests.
+
+📊 **Measured Improvement:**
+Baseline (100 iterations on 5000 mock files): ~1186 ms
+Optimized (100 iterations on 5000 mock files): ~722 ms
+Improvement: Execution time reduced by ~40%.
diff --git a/src/lib/parser.js b/src/lib/parser.js
index 89e616d..6a2c145 100644
--- a/src/lib/parser.js
+++ b/src/lib/parser.js
@@ -3029,16 +3029,36 @@ function getArchitectureScanFiles(files){
     });
 }
 
-function detectArchitectureFramework(files){
-    var paths=getArchitectureScanFiles(files).map(function(f){return normalizeArchitecturePath(f.path||f.name).toLowerCase();});
-    var hasNextConfig=paths.some(function(p){return /(^|\/)next\.config\.(js|mjs|ts|cjs)$/.test(p);});
-    var hasAppRouter=paths.some(function(p){return /(^|\/)(src\/)?app\/.*(page|route)\.(js|jsx|ts|tsx)$/.test(p);});
-    var hasPagesRouter=paths.some(function(p){return /(^|\/)(src\/)?pages\/.*\.(js|jsx|ts|tsx)$/.test(p);});
-    if(hasNextConfig||hasAppRouter||hasPagesRouter)return'Next.js';
-    if(paths.some(function(p){return /\.(html?|xhtml)$/.test(p);}))return'Browser App';
-    if(paths.some(function(p){return /\.(jsx?|tsx?|mjs|cjs)$/.test(p);}))return'JavaScript/TypeScript';
-    if(paths.some(function(p){return /\.(py|pyw|pyi)$/.test(p);}))return'Python';
-    return'Generic';
+function detectArchitectureFramework(files) {
+    var scanFiles = getArchitectureScanFiles(files);
+    var hasBrowserApp = false;
+    var hasJsTs = false;
+    var hasPython = false;
+
+    var NEXT_CONFIG_RE = /(^|\/)next\.config\.(js|mjs|ts|cjs)$/;
+    var NEXT_APP_ROUTER_RE = /(^|\/)(src\/)?app\/.*(page|route)\.(js|jsx|ts|tsx)$/;
+    var NEXT_PAGES_ROUTER_RE = /(^|\/)(src\/)?pages\/.*\.(js|jsx|ts|tsx)$/;
+    var BROWSER_APP_RE = /\.(html?|xhtml)$/;
+    var JS_TS_RE = /\.(jsx?|tsx?|mjs|cjs)$/;
+    var PYTHON_RE = /\.(py|pyw|pyi)$/;
+
+    for (var i = 0; i < scanFiles.length; i++) {
+        var f = scanFiles[i];
+        var p = normalizeArchitecturePath(f.path || f.name).toLowerCase();
+
+        if (NEXT_CONFIG_RE.test(p) || NEXT_APP_ROUTER_RE.test(p) || NEXT_PAGES_ROUTER_RE.test(p)) {
+            return 'Next.js';
+        }
+
+        if (!hasBrowserApp && BROWSER_APP_RE.test(p)) hasBrowserApp = true;
+        if (!hasJsTs && JS_TS_RE.test(p)) hasJsTs = true;
+        if (!hasPython && PYTHON_RE.test(p)) hasPython = true;
+    }
+
+    if (hasBrowserApp) return 'Browser App';
+    if (hasJsTs) return 'JavaScript/TypeScript';
+    if (hasPython) return 'Python';
+    return 'Generic';
 }
 
 function convertNextRouteSegment(segment){