Merge pull request #9 from Zacgoose/copilot/fix-guard-evaluation-issue

fix: remove redundant type guard and fix unused matchDetails assignment

Author: Zacgoose

scripts/content.js

@@ -3255,7 +3255,7 @@ if (window.checkExtensionLoaded) {
       let cleanedSourceLength = null;
       if (typeof arguments[1] === "object" && arguments[1]?.scanCleaned) {
         // If scanCleaned is true, get cleaned page source length
-        const cleanedSource = typeof getPageSource === "function" ? getPageSource(true) : null;
+        const cleanedSource = getCleanPageSource();
         cleanedSourceLength = cleanedSource ? cleanedSource.length : null;
         logger.log(
           `📄 Page info: ${document.querySelectorAll("*").length} elements, ${

scripts/modules/rules-engine-core.js

@@ -361,7 +361,7 @@ function processPhishingIndicators(indicators, pageSource, pageText, url, option
         category: ind.category || "general",
         confidence: ind.confidence ?? 0.5,
         description: ind.description || "",
-        matchDetails: snippet || matchedFrom || "",
+        matchDetails: snippet || matchedFrom || matchDetails || "",
       };
       threats.push(threat);