feat(ui): add XOR validation for social platform/URL entries

Author: zodiacfireworks

src/edit_python_pe/screens/member_form.py

@@ -250,8 +250,16 @@ def save_member(self) -> None:
             has_errors = True
 
         for se in self.social_entries:
+            plat = se.select.value
             urlval = se.url_input.value.strip()
-            if urlval and not URL_REGEX.match(urlval):
+
+            # XOR: if one is set but not the other
+            if bool(plat) != bool(urlval):
+                se.show_error(
+                    _("Both platform and URL must be provided if either is set.")
+                )
+                has_errors = True
+            elif urlval and not URL_REGEX.match(urlval):
                 se.show_error(_("Invalid URL format for social network."))
                 has_errors = True
 

tests/e2e/test_app_e2e.py

@@ -64,7 +64,7 @@ async def test_full_app_flow(
             assert app.forked_repo is not None
 
             # 4. Dashboard -> Add Member
-            await pilot.click("#dash-add")
+            app.screen.query_one("#dash-add").press()
             await pilot.pause()
             assert isinstance(app.screen, MemberFormScreen)
 
@@ -80,6 +80,7 @@ async def test_full_app_flow(
             screen.homepage_input.value = "not_a_url"
             if screen.social_entries:
                 screen.social_entries[-1].url_input.value = "not_a_social_url"
+                screen.social_entries[-1].select.value = "github"
 
             screen.action_save()
             await pilot.pause()
@@ -96,11 +97,13 @@ async def test_full_app_flow(
             screen.homepage_input.value = "https://example.com"
             if screen.social_entries:
                 screen.social_entries[-1].url_input.value = "https://github.com/john"
+                screen.social_entries[-1].select.value = "github"
+            if screen.alias_entries:
+                screen.alias_entries[-1].alias_input.value = "john-gh"
 
             await pilot.pause()
             screen.action_save()
 
-
             # 6. Save Loading Screen — give up to 5 s for the background work
             for _ in range(50):
                 await pilot.pause(0.1)