Fix local urls support in a11y scan (#52)

* feat: enhance accessibility scan report with issue count and pagination details

* Refactor: Improve formatting and error logging in AccessibilityScanner

* Refactor: Replace hardcoded local IP with a set of local hostnames in AccessibilityScanner

* Fix: Add '0.0.0.0' to local hosts in AccessibilityScanner

* Bump version to 1.1.4 in package.json and package-lock.json

* Fix: Improve formatting of localHosts declaration in AccessibilityScanner

Author: ruturaj-browserstack

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@browserstack/mcp-server",
-  "version": "1.1.3",
+  "version": "1.1.4",
   "description": "BrowserStack's Official MCP Server",
   "main": "dist/index.js",
   "repository": {

package.json

@@ -73,7 +73,9 @@ export async function killExistingBrowserStackLocalProcesses() {
   }
 }
 
-export async function ensureLocalBinarySetup(localIdentifier?: string): Promise<void> {
+export async function ensureLocalBinarySetup(
+  localIdentifier?: string,
+): Promise<void> {
   logger.info(
     "Ensuring local binary setup as it is required for private URLs...",
   );
@@ -87,33 +89,30 @@ export async function ensureLocalBinarySetup(localIdentifier?: string): Promise<
     localIdentifier?: string;
   } = {
     key: config.browserstackAccessKey,
-    username: config.browserstackUsername
+    username: config.browserstackUsername,
   };
 
   if (localIdentifier) {
     requestBody.localIdentifier = localIdentifier;
   }
 
   return await new Promise((resolve, reject) => {
-    localBinary.start(
-      requestBody,
-      (error?: Error) => {
-        if (error) {
-          logger.error(
-            `Unable to start BrowserStack Local... please check your credentials and try again. Error: ${error}`,
-          );
+    localBinary.start(requestBody, (error?: Error) => {
+      if (error) {
+        logger.error(
+          `Unable to start BrowserStack Local... please check your credentials and try again. Error: ${error}`,
+        );
 
-          reject(
-            new Error(
-              `Unable to configure local tunnel binary, please check your credentials and try again. Error: ${error}`,
-            ),
-          );
-        } else {
-          logger.info("Successfully started BrowserStack Local");
-          resolve();
-        }
-      },
-    );
+        reject(
+          new Error(
+            `Unable to configure local tunnel binary, please check your credentials and try again. Error: ${error}`,
+          ),
+        );
+      } else {
+        logger.info("Successfully started BrowserStack Local");
+        resolve();
+      }
+    });
   });
 }
 

src/lib/local.ts

@@ -29,11 +29,10 @@ export class AccessibilityScanner {
     name: string,
     urlList: string[],
   ): Promise<AccessibilityScanResponse> {
-
     // Check if any URL is local
     const hasLocal = urlList.some(isLocalURL);
     const localIdentifier = crypto.randomUUID();
-    const LOCAL_IP = "127.0.0.1";
+    const localHosts = new Set(["127.0.0.1", "localhost", "0.0.0.0"]);
     const BS_LOCAL_DOMAIN = "bs-local.com";
 
     if (hasLocal) {
@@ -45,13 +44,13 @@ export class AccessibilityScanner {
     const transformedUrlList = urlList.map((url) => {
       try {
         const parsed = new URL(url);
-        if (parsed.hostname === LOCAL_IP) {
+        if (localHosts.has(parsed.hostname)) {
           parsed.hostname = BS_LOCAL_DOMAIN;
           return parsed.toString();
         }
         return url;
       } catch (e) {
-        logger.warn(`[AccessibilityScan] Invalid URL skipped: ${url}`);
+        logger.warn(`[AccessibilityScan] Invalid URL skipped: ${e}`);
         return url;
       }
     });
@@ -68,7 +67,7 @@ export class AccessibilityScanner {
         localTestingInfo: {
           localIdentifier,
           localEnabled: true,
-        }
+        },
       };
       requestBody = { ...baseRequestBody, ...localConfig };
     }