From 2a9160836bb87fdea3ae891563b0169c68d1c2ab Mon Sep 17 00:00:00 2001
From: DanConwayDev <DanConwayDev@protonmail.com>
Date: Thu, 11 Dec 2025 16:53:03 +0000
Subject: fix: resolve all fmt and clippy warnings

Main lib (src/):
- Add #[allow(dead_code)] for build_info field (stored to prevent Prometheus unregistration)
- Add #[allow(dead_code)] for first_seen field (reserved for future rate limiting)
- Replace .or_insert_with(RelaySyncNeeds::default) with .or_default()
- Replace manual div_ceil implementations with .div_ceil(100)

Test code (tests/):
- Replace .expect(&format!(...)) with .unwrap_or_else(|_| panic!(...))
- Remove needless borrows in fetch_metrics() calls
- Add #[allow(dead_code)] and #[allow(unused_imports)] to test helpers module

grasp-audit:
- Apply cargo fmt to fix formatting
---
 src/nostr/builder.rs | 22 ++++++++--------------
 1 file changed, 8 insertions(+), 14 deletions(-)

(limited to 'src/nostr/builder.rs')

diff --git a/src/nostr/builder.rs b/src/nostr/builder.rs
index 2284c18..c9bd1e1 100644
--- a/src/nostr/builder.rs
+++ b/src/nostr/builder.rs
@@ -16,8 +16,8 @@ use crate::nostr::events::{
     KIND_REPOSITORY_STATE,
 };
 use crate::nostr::policy::{
-    AnnouncementPolicy, AnnouncementResult, PolicyContext, PrEventPolicy, RelatedEventPolicy,
-    ReferenceResult, StatePolicy, StateResult,
+    AnnouncementPolicy, AnnouncementResult, PolicyContext, PrEventPolicy, ReferenceResult,
+    RelatedEventPolicy, StatePolicy, StateResult,
 };
 
 /// Type alias for the shared database used by the relay
@@ -77,7 +77,9 @@ impl Nip34WritePolicy {
                 match RepositoryAnnouncement::from_event(event.clone()) {
                     Ok(announcement) => {
                         // Try to create bare repository if it doesn't exist
-                        if let Err(e) = self.announcement_policy.ensure_bare_repository(&announcement)
+                        if let Err(e) = self
+                            .announcement_policy
+                            .ensure_bare_repository(&announcement)
                         {
                             tracing::warn!(
                                 "Failed to create bare repository for {}: {}",
@@ -145,22 +147,14 @@ impl Nip34WritePolicy {
                     Ok(_state) => {
                         // Process state alignment asynchronously
                         if let Err(e) = self.state_policy.process_state_event(event).await {
-                            tracing::warn!(
-                                "Failed to process state event {}: {}",
-                                event_id_str,
-                                e
-                            );
+                            tracing::warn!("Failed to process state event {}: {}", event_id_str, e);
                         }
 
                         tracing::debug!("Accepted repository state: {}", event_id_str);
                         PolicyResult::Accept
                     }
                     Err(e) => {
-                        tracing::warn!(
-                            "Failed to parse repository state {}: {}",
-                            event_id_str,
-                            e
-                        );
+                        tracing::warn!("Failed to parse repository state {}: {}", event_id_str, e);
                         // Still accept the event even if we can't parse it
                         // The validation passed, so it's structurally valid
                         PolicyResult::Accept
@@ -348,4 +342,4 @@ pub fn create_relay(config: &Config) -> Result<RelayWithDatabase> {
         database,
         write_policy,
     })
-}
\ No newline at end of file
+}
-- 
cgit v1.2.3