lmdb: consolidate deletion logic and improve transactional consistency

- Added `DeletionInfo` to resolve borrow checker limitations when using LMDB transactions.
- Remove separate read transaction usage, ensuring that events in the same batch maintain consistency

Pull-Request:#1010
Signed-off-by: Yuki Kishimoto <[email protected]>

Author: dcadenas

database/nostr-lmdb/CHANGELOG.md

@@ -23,6 +23,12 @@
 
 -->
 
+## Unreleased
+
+### Changed
+
+- Consolidate deletion logic and improve transactional consistency (https://github.com/rust-nostr/nostr/pull/1010)
+
 ## v0.43.0 - 2025/07/28
 
 ### Changed

database/nostr-lmdb/src/store/ingester.rs

@@ -94,23 +94,14 @@ impl Ingester {
         &self,
         event: Event,
         fbb: &mut FlatBufferBuilder,
-    ) -> nostr::Result<SaveEventStatus, Error> {
-        let read_txn = self.db.read_txn()?;
+    ) -> Result<SaveEventStatus, Error> {
         let mut write_txn = self.db.write_txn()?;
 
-        let status: SaveEventStatus =
-            self.db
-                .save_event_with_txn(&read_txn, &mut write_txn, fbb, &event)?;
+        let status: SaveEventStatus = self.db.save_event_with_txn(&mut write_txn, fbb, &event)?;
 
         match &status {
-            SaveEventStatus::Success => {
-                write_txn.commit()?;
-                read_txn.commit()?;
-            }
-            SaveEventStatus::Rejected(_) => {
-                write_txn.abort();
-                read_txn.commit()?;
-            }
+            SaveEventStatus::Success => write_txn.commit()?,
+            SaveEventStatus::Rejected(_) => write_txn.abort(),
         }
 
         Ok(status)