Merge pull request #289 from stickerdaniel/03-29-feat_connect_add_more_menu_support_and_return_profile_text

feat(connect): Add More menu support and return profile text

Author: stickerdaniel

linkedin_mcp_server/scraping/extractor.py

@@ -470,6 +470,38 @@ async def _dismiss_dialog(self) -> None:
         except PlaywrightTimeoutError:
             pass
 
+    async def _open_more_menu(self) -> bool:
+        """Open the profile's More (three-dot) menu and check for Connect.
+
+        Uses ``aria-label`` to find the More button (language-independent)
+        and ``[role="menu"]`` to detect the opened menu (structural).
+        Returns True if the menu opened and contains a Connect option.
+        """
+        more_btn = self._page.locator("main button[aria-label*='More']")
+        try:
+            if await more_btn.count() == 0:
+                return False
+            await more_btn.first.click(timeout=5000)
+        except Exception:
+            logger.debug("Could not click More button", exc_info=True)
+            return False
+
+        try:
+            await self._page.wait_for_selector("[role='menu']", timeout=3000)
+        except PlaywrightTimeoutError:
+            logger.debug("More menu did not appear")
+            return False
+
+        # Check if Connect is in the menu
+        menu_connect = (
+            self._page.locator("[role='menu']")
+            .locator("button, a, li, [role='menuitem'], [role='button']")
+            .filter(has_text=re.compile(r"^Connect$"))
+        )
+        count = await menu_connect.count()
+        logger.debug("More menu Connect matches: %d", count)
+        return count > 0
+
     async def extract_page(
         self,
         url: str,
@@ -796,13 +828,20 @@ async def connect_with_person(
                 "A connection request is already pending for this profile.",
                 profile=page_text,
             )
+        via_more_menu = False
         if state == "follow_only":
-            return _connection_result(
-                url,
-                "follow_only",
-                "This profile currently exposes Follow but not Connect.",
-                profile=page_text,
-            )
+            # Connect may be hidden behind the More (three-dot) menu
+            if await self._open_more_menu():
+                state = "connectable"
+                via_more_menu = True
+            else:
+                return _connection_result(
+                    url,
+                    "follow_only",
+                    "This profile currently exposes Follow but not Connect.",
+                    profile=page_text,
+                )
+
         if state == "unavailable":
             return _connection_result(
                 url,
@@ -821,7 +860,8 @@ async def connect_with_person(
             )
 
         # Click the button (page is already loaded from scrape_person)
-        clicked = await self.click_button_by_text(button_text)
+        click_scope = "[role='menu']" if via_more_menu else "main"
+        clicked = await self.click_button_by_text(button_text, scope=click_scope)
         if not clicked:
             return _connection_result(
                 url,

tests/test_scraping.py

@@ -810,7 +810,7 @@ async def test_connectable_clicks_connect(self, mock_page):
 
         assert result["status"] == "connected"
         assert result["url"] == "https://www.linkedin.com/in/testuser/"
-        mock_click.assert_awaited_once_with("Connect")
+        mock_click.assert_awaited_once_with("Connect", scope="main")
 
     async def test_returns_already_connected(self, mock_page):
         extractor = LinkedInExtractor(mock_page)
@@ -852,7 +852,7 @@ async def test_returns_incoming_request_accepted(self, mock_page):
             result = await extractor.connect_with_person("testuser")
 
         assert result["status"] == "accepted"
-        mock_click.assert_awaited_once_with("Accept")
+        mock_click.assert_awaited_once_with("Accept", scope="main")
 
     async def test_returns_follow_only(self, mock_page):
         extractor = LinkedInExtractor(mock_page)