add changes to disable and log reason

.github/llm-bot/issue_bot.py

@@ -5,16 +5,15 @@
 such as tagging, closing, translating, or requesting more information.
 """
 
+import json
 import os
 import sys
-import json
-from typing import List, Optional
 from enum import Enum
+from typing import List, Optional
 
-from pydantic import BaseModel, Field
 from a5.ai import LLM
-from github import Github, Auth
-
+from github import Auth, Github
+from pydantic import BaseModel, Field
 
 # ============================================================================
 # Configuration
@@ -225,16 +224,16 @@ def apply_decision(self, decision: BotDecision):
             # Add debug information section
             comment_body += self._format_debug_section(decision)
 
-            print(f"\nPosting comment...")
+            print("\nPosting comment...")
             self.issue.create_comment(comment_body)
 
         # Close issue if needed
         if decision.action == IssueAction.CLOSE:
-            print(f"\nClosing issue...")
+            print("\nClosing issue...")
             self.issue.edit(state="closed")
 
         # Always add bot-processed label to indicate the bot has taken action
-        print(f"\nAdding bot-processed label...")
+        print("\nAdding bot-processed label...")
         self.issue.add_to_labels("bot-processed")
 
         print(f"\n✅ Successfully processed issue #{self.issue_number}")

.github/llm-bot/pr_release_notes.py

@@ -7,15 +7,14 @@
 """
 
 import os
-import sys
 import re
-from typing import Optional, List, Dict, Any
+import sys
 from enum import Enum
+from typing import Any, Dict, List, Optional
 
-from pydantic import BaseModel, Field
 from a5.ai import LLM
-from github import Github, Auth
-
+from github import Auth, Github
+from pydantic import BaseModel, Field
 
 # ============================================================================
 # Configuration
@@ -476,7 +475,7 @@ def apply_decision(self, decision: BotDecision):
 """
                 comment_body += self._format_debug_section(decision)
 
-                print(f"\nPosting release note comment...")
+                print("\nPosting release note comment...")
                 self.pr.create_issue_comment(comment_body)
 
         else:  # EXCLUDE
@@ -493,7 +492,7 @@ def apply_decision(self, decision: BotDecision):
 """
             comment_body += self._format_debug_section(decision)
 
-            print(f"\nPosting exclusion comment...")
+            print("\nPosting exclusion comment...")
             self.pr.create_issue_comment(comment_body)
 
         print(f"\n✅ Successfully processed PR #{self.pr_number}")

app/backend/src/couchers/servicers/admin.py

@@ -48,6 +48,7 @@
 from couchers.servicers.api import user_model_to_pb
 from couchers.servicers.auth import create_session
 from couchers.servicers.events import generate_event_delete_notifications
+from couchers.servicers.notifications import disable_push_notifications_for_user
 from couchers.servicers.threads import unpack_thread_id
 from couchers.sql import to_bool, username_or_email_or_id
 from couchers.utils import Timestamp_from_datetime, date_to_api, now, parse_date, to_aware_datetime
@@ -372,9 +373,14 @@ def BanUser(
         user = session.execute(select(User).where(username_or_email_or_id(request.user))).scalar_one_or_none()
         if not user:
             context.abort_with_error_code(grpc.StatusCode.NOT_FOUND, "user_not_found")
+        # Ensure admin note is logged directly
+        log_admin_action(session, context, user, "ban", note=request.admin_note, level=AdminActionLevel.high)
+        disable_push_notifications_for_user(
+            user_id=user.id,
+            session=session,
+        )
         if not request.admin_note.strip():
             context.abort_with_error_code(grpc.StatusCode.INVALID_ARGUMENT, "admin_note_cant_be_empty")
-        log_admin_action(session, context, user, "ban", note=request.admin_note, level=AdminActionLevel.high)
         user.banned_at = now()
         return _user_to_details(session, user)
 

app/backend/src/couchers/servicers/notifications.py

@@ -61,6 +61,19 @@ def notification_to_pb(user: User, notification: Notification) -> notifications_
     )
 
 
+def disable_push_notifications_for_user(user_id: int, session: Session) -> None:
+    """
+    Disables push notifications for a specific user by updating their subscriptions.
+    """
+    session.execute(
+        update(PushNotificationSubscription)
+        .where(PushNotificationSubscription.user_id == user_id)
+        .where(PushNotificationSubscription.disabled_at > now())
+        .values(disabled_at=now())
+    )
+    session.commit()
+
+
 class Notifications(notifications_pb2_grpc.NotificationsServicer):
     def GetNotificationSettings(
         self, request: notifications_pb2.GetNotificationSettingsReq, context: CouchersContext, session: Session

app/media/src/tests/test_server.py

@@ -16,8 +16,8 @@
 from PIL import Image
 from PIL.JpegImagePlugin import JpegImageFile
 
-from media.server import create_app
 from media.proto import media_pb2, media_pb2_grpc
+from media.server import create_app
 
 DATADIR = Path(__file__).parent / "data"