Merge pull request 'fix(nostr): log publish skips at WARNING instead of silently' (#54) from fix/publish-skip-logging into main

Reviewed-on: #54

nostr_hooks.py

@@ -32,9 +32,19 @@ async def publish_or_delete_nostr_event(event: Event, *, delete: bool = False) -
         if signer is None:
             # Wallet missing, account missing, unclassified row, or
             # ClientSideOnlySigner account (server can't sign for them).
-            # Soft-fail: skip the publish silently. The user can still
+            # Soft-fail: the HTTP / payment flow that triggered this must
-            # publish kind-31922/31923 events client-side once we have
+            # not break. The user can still publish kind-31922/31923
-            # that path.
+            # events client-side once we have that path.
+            #
+            # Logged at WARNING, not debug: skipping the publish means the
+            # relay keeps serving whatever inventory it last saw, so the
+            # public ticket count silently stops tracking the DB. That has
+            # twice been discovered only by a human noticing a wrong number
+            # on a public page (aiolabs/events#35, #51).
+            logger.warning(
+                f"[EVENTS] No signer for wallet {event.wallet}, skipping "
+                f"NIP-52 {'delete' if delete else 'publish'} for event {event.id}"
+            )
             return
 
         nostr_event = await publish_event_to_nostr(

nostr_publisher.py

@@ -172,7 +172,14 @@ async def publish_event_to_nostr(
     Returns the published NostrEvent for metadata storage, or None on failure.
     """
     if not nostr_client:
-        logger.debug("[EVENTS] No NostrClient available, skipping publish")
+        # WARNING, not debug: with no client the event is never queued, so
+        # the relay keeps serving stale inventory and nothing downstream
+        # can tell. At debug this skip is invisible at the INFO level
+        # instances actually run at (aiolabs/events#35, #51).
+        logger.warning(
+            "[EVENTS] No NostrClient, skipping NIP-52 "
+            f"{'delete' if delete else 'publish'} for event {event.id}"
+        )
         return None
 
     try: