From ee113a654e2971a6ebdb07398cc5638dbe59b48c Mon Sep 17 00:00:00 2001
From: DanConwayDev <DanConwayDev@protonmail.com>
Date: Wed, 18 Feb 2026 20:32:13 +0000
Subject: fix: replace repo_sync_index wiring with purgatory announcement sync
 timer

Instead of threading repo_sync_index through PolicyContext/builder.rs/main.rs
to handle user-submitted purgatory announcements, add a simple background
timer (run_purgatory_announcement_sync, every 5s) that scans the purgatory
for announcement entries and registers them in repo_sync_index as StateOnly.

This is simpler and covers both flows:
- Sync-path announcements: inline registration still happens during event
  processing (sync/mod.rs:1839+), timer provides a safety net
- User-submitted announcements: SelfSubscriber never sees them (rejected
  from DB), timer is the primary registration path

The timer calls sync_purgatory_announcements_to_index() which:
1. Snapshots purgatory via new announcements_for_sync() public method
2. Or_inserts StateOnly entries (never downgrades Full entries)
3. Detects newly added relay URLs and calls handle_new_sync_filters to
   connect and subscribe - fixing the failing test that expected relay
   discovery from a user-submitted purgatory announcement

Removes: repo_sync_index field from PolicyContext, set/get_repo_sync_index
methods, set_repo_sync_index on Nip34WritePolicy, wiring in main.rs, and
the inline AcceptPurgatory registration block in builder.rs.
---
 src/nostr/policy/mod.rs | 19 -------------------
 1 file changed, 19 deletions(-)

(limited to 'src/nostr/policy')

diff --git a/src/nostr/policy/mod.rs b/src/nostr/policy/mod.rs
index c958586..1566b6c 100644
--- a/src/nostr/policy/mod.rs
+++ b/src/nostr/policy/mod.rs
@@ -20,7 +20,6 @@ pub use crate::git::sync::AlignmentResult;
 
 use super::SharedDatabase;
 use crate::purgatory::Purgatory;
-use crate::sync::RepoSyncIndex;
 use nostr_relay_builder::LocalRelay;
 use std::sync::Arc;
 
@@ -35,8 +34,6 @@ pub struct PolicyContext {
     pub local_relay: Arc<std::sync::RwLock<Option<LocalRelay>>>,
     /// Configuration reference for policy settings (includes blacklists)
     pub config: crate::config::Config,
-    /// Repo sync index for registering purgatory announcements (set after SyncManager creation)
-    pub repo_sync_index: Arc<std::sync::RwLock<Option<RepoSyncIndex>>>,
 }
 
 impl PolicyContext {
@@ -54,7 +51,6 @@ impl PolicyContext {
             purgatory,
             local_relay: Arc::new(std::sync::RwLock::new(None)),
             config,
-            repo_sync_index: Arc::new(std::sync::RwLock::new(None)),
         }
     }
 
@@ -72,19 +68,4 @@ impl PolicyContext {
         let guard = self.local_relay.read().unwrap();
         guard.clone()
     }
-
-    /// Set the repo sync index after SyncManager has been created.
-    ///
-    /// This allows purgatory announcements submitted by users to be registered
-    /// in the sync index so state event sync starts promptly.
-    pub fn set_repo_sync_index(&self, index: RepoSyncIndex) {
-        let mut guard = self.repo_sync_index.write().unwrap();
-        *guard = Some(index);
-    }
-
-    /// Get a clone of the repo sync index if it has been set.
-    pub fn get_repo_sync_index(&self) -> Option<RepoSyncIndex> {
-        let guard = self.repo_sync_index.read().unwrap();
-        guard.clone()
-    }
 }
-- 
cgit v1.2.3