style: fix ruff and ty linting errors

Author: zodiacfireworks

src/edit_python_pe/main.py

@@ -96,7 +96,7 @@ def value(self):
     @value.setter
     def value(self, v: str):
         self._value = v
-        self.query_one(selector="#input").value = v
+        self.query_one(selector="#input").value = v  # type: ignore
 
 
 class SocialEntry(Horizontal):
@@ -160,7 +160,7 @@ def compose(self) -> ComposeResult:
 
 
 class MemberApp(App):
-    """Single app that toggles between a file list and a form while connected to a GitHub fork+push flow."""
+    """Single app that toggles between a file list and a form while connected to a GitHub fork+push flow."""  # noqa: E501
 
     def __init__(
         self,
@@ -176,7 +176,7 @@ def __init__(
         self.repo_path = repo_path
 
     def compose(self) -> ComposeResult:
-        # Two main containers: self.list_container for the file list, self.form_container for the form.
+        # Two main containers: self.list_container for the file list, self.form_container for the form.  # noqa: E501
         self.list_container = Vertical()
         yield self.list_container
 
@@ -314,9 +314,9 @@ def clear_form(self) -> None:
         self.alias_container.remove_children()
 
     def on_list_view_selected(self, event: ListView.Selected) -> None:
-        """User clicked on a file in the list. Parse it into the form fields."""
+        """User clicked on a file in the list. Parse it into the form fields."""  # noqa: E501
         item_text_widget = event.item.children[0]
-        filename = item_text_widget.content
+        filename = item_text_widget.content  # type: ignore
         self.current_file = filename
 
         self.clear_form()
@@ -351,9 +351,10 @@ def on_button_pressed(self, event: Button.Pressed) -> None:
             self.remove_alias_entry(index)
 
     def add_social_entry(
-        self, value: str | NoSelection = Select.BLANK
+        self,
+        value: str | NoSelection = Select.BLANK,  # type: ignore
     ) -> None:
-        new_entry = SocialEntry(self.social_index, value)
+        new_entry = SocialEntry(self.social_index, value)  # type: ignore
         self.social_index += 1
         self.social_entries.append(new_entry)
         self.social_container.mount(new_entry)
@@ -407,7 +408,7 @@ def save_member(self) -> None:
             plat = se.select.value
             urlval = se.url_input.value.strip()
             if plat and urlval:
-                socials.append((plat, urlval))
+                socials.append((str(plat), urlval))
 
         # Build the markdown doc as per the provided guide
         md_content = build_md_content(

src/edit_python_pe/strings.py

@@ -84,10 +84,10 @@
     "yaml_excerpt": "excerpt: 1",
     "yaml_end": "---",
     "header_name": "# {name}",
-    "gravatar_block": '```{{gravatar}} {email}\n---\nwidth: 200\nclass: "member-gravatar"\n---\n```',
+    "gravatar_block": '```{{gravatar}} {email}\n---\nwidth: 200\nclass: "member-gravatar"\n---\n```',  # noqa: E501
     "social_block_start": "```{{raw}} html",
     "social_ul_start": '<ul class="social-media profile">',
-    "social_li": '    <li>\n        <a class="external reference" href="{url}">\n            <iconify-icon icon="simple-icons:{platform}" style="font-size:2em"></iconify-icon>\n        </a>\n    </li>',
+    "social_li": '    <li>\n        <a class="external reference" href="{url}">\n            <iconify-icon icon="simple-icons:{platform}" style="font-size:2em"></iconify-icon>\n        </a>\n    </li>',  # noqa: E501
     "social_ul_end": "</ul>",
     "social_block_end": "```",
     "aliases": ":Aliases: {aliases}",
@@ -97,5 +97,5 @@
     "section_who": "### ¿Quién eres y a qué te dedicas?",
     "section_python": "### ¿Cómo programas en Python?",
     "section_contrib": "### ¿Tienes algún aporte a la comunidad de Python?",
-    "section_avail": "### ¿Estás disponible para hacer mentoring, consultorías, charlas?",
+    "section_avail": "### ¿Estás disponible para hacer mentoring, consultorías, charlas?",  # noqa: E501
 }

tests/test_member_app.py

@@ -1,7 +1,8 @@
 import os
 import sys
 import unittest
-from unittest.mock import MagicMock, patch
+import unittest.mock
+from unittest.mock import ANY, MagicMock, patch
 
 sys.path.insert(
     0, os.path.abspath(os.path.join(os.path.dirname(__file__), "../src"))
@@ -42,14 +43,14 @@ class StubTextArea:
             def __init__(self):
                 self.text = ""
 
-        self.app.name_input = StubInput()
-        self.app.email_input = StubInput()
-        self.app.city_input = StubInput()
-        self.app.homepage_input = StubInput()
-        self.app.who_area = StubTextArea()
-        self.app.python_area = StubTextArea()
-        self.app.contributions_area = StubTextArea()
-        self.app.availability_area = StubTextArea()
+        self.app.name_input = StubInput()  # type: ignore
+        self.app.email_input = StubInput()  # type: ignore
+        self.app.city_input = StubInput()  # type: ignore
+        self.app.homepage_input = StubInput()  # type: ignore
+        self.app.who_area = StubTextArea()  # type: ignore
+        self.app.python_area = StubTextArea()  # type: ignore
+        self.app.contributions_area = StubTextArea()  # type: ignore
+        self.app.availability_area = StubTextArea()  # type: ignore
 
         # Patch remove method for entries to avoid Textual lifecycle errors
         # Use stub classes for entries with .remove() method
@@ -76,18 +77,18 @@ def remove(self):
     def test_add_social_entry(self):
         # Patch add_social_entry to use stub
         self.app.social_entries = []
-        self.app.social_container.mount = MagicMock()
+        self.app.social_container.mount = MagicMock()  # type: ignore
         # Patch mount to accept any object
         self.app.social_container.mount = lambda x: None  # type: ignore
 
         def stub_add_social_entry(value):
             entry = self.StubSocialEntry()
             entry.index = self.app.social_index
             self.app.social_index += 1
-            self.app.social_entries.append(entry)
-            self.app.social_container.mount(entry)
+            self.app.social_entries.append(entry)  # type: ignore
+            self.app.social_container.mount(entry)  # type: ignore
 
-        self.app.add_social_entry = stub_add_social_entry
+        self.app.add_social_entry = stub_add_social_entry  # type: ignore
         initial_count = len(self.app.social_entries)
         self.app.add_social_entry("")
         self.assertEqual(len(self.app.social_entries), initial_count + 1)
@@ -104,8 +105,8 @@ def test_add_list_button_clears_form(self):
         self.app.contributions_area.text = "Filled Contributions"
         self.app.availability_area.text = "Filled Available"
         # Add social and alias entries
-        self.app.social_entries = [self.StubSocialEntry()]
-        self.app.alias_entries = [self.StubAliasEntry()]
+        self.app.social_entries = [self.StubSocialEntry()]  # type: ignore
+        self.app.alias_entries = [self.StubAliasEntry()]  # type: ignore
 
         # Simulate pressing the 'Añadir' button on the list screen
         class DummyButton:
@@ -114,7 +115,7 @@ class DummyButton:
         class DummyEvent:
             button = DummyButton()
 
-        self.app.on_button_pressed(DummyEvent())
+        self.app.on_button_pressed(DummyEvent())  # type: ignore
         # After pressing, form should be cleared and current_file should be None
         self.assertEqual(self.app.name_input.value, "")
         self.assertEqual(self.app.email_input.value, "")
@@ -137,18 +138,18 @@ class DummyEvent:
     def test_add_alias_entry(self):
         # Patch add_alias_entry to use stub
         self.app.alias_entries = []
-        self.app.alias_container.mount = MagicMock()
+        self.app.alias_container.mount = MagicMock()  # type: ignore
         # Patch mount to accept any object
         self.app.alias_container.mount = lambda x: None  # type: ignore
 
         def stub_add_alias_entry():
             entry = self.StubAliasEntry()
             entry.index = self.app.alias_index
             self.app.alias_index += 1
-            self.app.alias_entries.append(entry)
-            self.app.alias_container.mount(entry)
+            self.app.alias_entries.append(entry)  # type: ignore
+            self.app.alias_container.mount(entry)  # type: ignore
 
-        self.app.add_alias_entry = stub_add_alias_entry
+        self.app.add_alias_entry = stub_add_alias_entry  # type: ignore
         initial_count = len(self.app.alias_entries)
         self.app.add_alias_entry()
         self.assertEqual(len(self.app.alias_entries), initial_count + 1)
@@ -354,18 +355,18 @@ def stub_add_social_entry(value):
             entry = self.StubSocialEntry()
             entry.index = self.app.social_index
             self.app.social_index += 1
-            self.app.social_entries.append(entry)
-            self.app.social_container.mount(entry)
+            self.app.social_entries.append(entry)  # type: ignore
+            self.app.social_container.mount(entry)  # type: ignore
 
         def stub_add_alias_entry():
             entry = self.StubAliasEntry()
             entry.index = self.app.alias_index
             self.app.alias_index += 1
-            self.app.alias_entries.append(entry)
-            self.app.alias_container.mount(entry)
+            self.app.alias_entries.append(entry)  # type: ignore
+            self.app.alias_container.mount(entry)  # type: ignore
 
-        self.app.add_social_entry = stub_add_social_entry
-        self.app.add_alias_entry = stub_add_alias_entry
+        self.app.add_social_entry = stub_add_social_entry  # type: ignore
+        self.app.add_alias_entry = stub_add_alias_entry  # type: ignore
         self.app.add_social_entry("")
         self.app.add_alias_entry()
         self.app.clear_form()
@@ -411,18 +412,18 @@ def stub_add_social_entry(value):
             entry = self.StubSocialEntry()
             entry.index = self.app.social_index
             self.app.social_index += 1
-            self.app.social_entries.append(entry)
-            self.app.social_container.mount(entry)
+            self.app.social_entries.append(entry)  # type: ignore
+            self.app.social_container.mount(entry)  # type: ignore
 
         def stub_add_alias_entry():
             entry = self.StubAliasEntry()
             entry.index = self.app.alias_index
             self.app.alias_index += 1
-            self.app.alias_entries.append(entry)
-            self.app.alias_container.mount(entry)
+            self.app.alias_entries.append(entry)  # type: ignore
+            self.app.alias_container.mount(entry)  # type: ignore
 
-        self.app.add_social_entry = stub_add_social_entry
-        self.app.add_alias_entry = stub_add_alias_entry
+        self.app.add_social_entry = stub_add_social_entry  # type: ignore
+        self.app.add_alias_entry = stub_add_alias_entry  # type: ignore
         # Patch clear_form to avoid resetting stubs
         self.app.clear_form = lambda: None  # type: ignore
         load_file_into_form(self.app, "fake.md")
@@ -468,9 +469,9 @@ def test_main_runs_app(
         mock_get_repo.assert_called_once()
         mock_fork_repo.assert_called_once()
         mock_member_app.assert_called_once_with(
-            unittest.mock.ANY,
-            unittest.mock.ANY,
-            unittest.mock.ANY,
-            unittest.mock.ANY,
+            ANY,
+            ANY,
+            ANY,
+            ANY,
         )
         mock_app_instance.run.assert_called_once()