Merge branch 'browserstack:main' into main

Author: ruturaj-browserstack

.github/workflows/npm-publish.yml

@@ -8,12 +8,12 @@ permissions:
 
 jobs:
   publish:
-    name: "Publish to NPM"
     runs-on: ubuntu-latest
-
     steps:
       - name: "Checkout source code"
         uses: actions/checkout@v2
+        with:
+          fetch-depth: 0
 
       - name: "Set up Node.js"
         uses: actions/setup-node@v3
@@ -29,7 +29,63 @@ jobs:
 
       - name: "Get version from package.json"
         id: get_version
-        run: echo "version=v$(node -p "require('./package.json').version")" >> $GITHUB_OUTPUT
+        run: |
+          VERSION="v$(node -p 'require("./package.json").version')"
+          echo "version=$VERSION" >> $GITHUB_OUTPUT
+
+      - name: Get previous Git tag
+        id: get_previous_tag
+        run: |
+          VERSION="${{ steps.get_version.outputs.version }}"
+          PREV_TAG=$(git tag --sort=-creatordate | grep '^v' | grep -v "$VERSION" | head -n 1)
+          echo "previous_tag=$PREV_TAG" >> $GITHUB_OUTPUT
+
+      - name: Fetch and categorize merged PRs
+        id: fetch_prs
+        env:
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: |
+          set -e
+          PREVIOUS_TAG=${{ steps.get_previous_tag.outputs.previous_tag }}
+          if [ -z "$PREVIOUS_TAG" ]; then
+            echo "pr_list=No previous tag found to compare PRs." >> $GITHUB_OUTPUT
+            exit 0
+          fi
+          PREVIOUS_SHA=$(git rev-list -n 1 $PREVIOUS_TAG)
+          PREVIOUS_DATE=$(git show -s --format=%cI $PREVIOUS_SHA)
+          CURRENT_DATE=$(git show -s --format=%cI HEAD)
+          echo "Fetching PRs merged between $PREVIOUS_DATE and $CURRENT_DATE"
+          
+          RAW_PRS=$(gh pr list --state merged --search "merged:${PREVIOUS_DATE}..${CURRENT_DATE}" \
+            --json number,title,url \
+            --jq '.[] | "- [#\(.number)](\(.url)) \(.title)"')
+      
+          if [ -z "$RAW_PRS" ]; then
+            echo "pr_list=No pull requests were merged during this release." >> $GITHUB_OUTPUT
+            exit 0
+          fi
+      
+          ADDED=""
+          FIXED=""
+          while IFS= read -r pr; do
+            if echo "$pr" | grep -iq "fix"; then
+              FIXED+="$pr"$'\n'
+            else
+              ADDED+="$pr"$'\n'
+            fi
+          done <<< "$RAW_PRS"
+      
+          BODY=""
+          if [ -n "$ADDED" ]; then
+            BODY="$BODY### Added"$'\n'"$ADDED"
+          fi
+          if [ -n "$FIXED" ]; then
+            BODY="$BODY"$'\n'"### Fixed"$'\n'"$FIXED"
+          fi
+      
+          echo "pr_list<<EOF" >> $GITHUB_OUTPUT
+          echo "$BODY" >> $GITHUB_OUTPUT
+          echo "EOF" >> $GITHUB_OUTPUT
 
       - name: "Set Git user name and email"
         run: |
@@ -51,11 +107,10 @@ jobs:
         uses: actions/create-release@v1
         with:
           tag_name: ${{ steps.get_version.outputs.version }}
-          release_name: Release ${{ steps.get_version.outputs.version }}
+          release_name: ${{ steps.get_version.outputs.version }}
           body: |
-            ```
-            • See CHANGELOG.md for details
-            • Published by ${{ github.actor }}
-            ```
+            ${{ steps.fetch_prs.outputs.pr_list }}
+
+            Published by ${{ github.actor }}
         env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

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 };
     }