chore: bump rust-nosrt v0.37.0

use RelayUrl in repo_ref which I had resisted as it mutates relay
urls when printed to append a slash

15 files changed, 199 insertions, 210 deletions

diff --git a/Cargo.lock b/Cargo.lock
index 4fca6a4..819a249 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -1802,18 +1802,6 @@ dependencies = [
 ]
 
 [[package]]
-name = "lnurl-pay"
-version = "0.6.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "536e7c782167a2d48346ca0b2677fad19eaef20f19a4ab868e4d5b96ca879def"
-dependencies = [
- "bech32",
- "reqwest",
- "serde",
- "serde_json",
-]
-
-[[package]]
 name = "lock_api"
 version = "0.4.12"
 source = "registry+https://github.com/rust-lang/crates.io-index"
@@ -2006,9 +1994,9 @@ checksum = "61807f77802ff30975e01f4f071c8ba10c022052f98b3294119f3e615d13e5be"
 
 [[package]]
 name = "nostr"
-version = "0.36.0"
+version = "0.37.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "14ad56c1d9a59f4edc46b17bc64a217b38b99baefddc0080f85ad98a0855336d"
+checksum = "8aad4b767bbed24ac5eb4465bfb83bc1210522eb99d67cf4e547ec2ec7e47786"
 dependencies = [
  "aes",
  "async-trait",
@@ -2021,7 +2009,6 @@ dependencies = [
  "chacha20poly1305",
  "getrandom",
  "instant",
- "js-sys",
  "negentropy 0.3.1",
  "negentropy 0.4.3",
  "once_cell",
@@ -2031,16 +2018,13 @@ dependencies = [
  "serde_json",
  "unicode-normalization",
  "url",
- "wasm-bindgen",
- "wasm-bindgen-futures",
- "web-sys",
 ]
 
 [[package]]
 name = "nostr-connect"
-version = "0.36.0"
+version = "0.37.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "29d2a145eecf951c5461fdb72a064142b422f882760542c855bc1eca02516afb"
+checksum = "5b1bf2a6324d2c1739be462e83d27caa2c025fd7ca14b401b4bc4b7cac11d382"
 dependencies = [
  "async-trait",
  "async-utility",
@@ -2053,9 +2037,9 @@ dependencies = [
 
 [[package]]
 name = "nostr-database"
-version = "0.36.0"
+version = "0.37.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1859abebf78d7d9e945b20c8faaf710c9db905adeb148035b803ae45792dbebe"
+checksum = "23696338d51e45cd44e061823847f4b0d1d362eca80d5033facf9c184149f72f"
 dependencies = [
  "async-trait",
  "flatbuffers",
@@ -2068,9 +2052,9 @@ dependencies = [
 
 [[package]]
 name = "nostr-lmdb"
-version = "0.36.0"
+version = "0.37.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "964cd83b0fa821cd09e3cf02287a3581d7deb26c9cb6d03990936b3f0228ba84"
+checksum = "70c97ff4e384735527af2dc74b6c8f0042f02daea476bad6c7bb56be32739250"
 dependencies = [
  "heed",
  "nostr",
@@ -2082,9 +2066,9 @@ dependencies = [
 
 [[package]]
 name = "nostr-relay-pool"
-version = "0.36.0"
+version = "0.37.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e39cfcb30cab86b30ca9acba89f5ccb25a4142a5dc5fcfbf3edf34b204ddd7c7"
+checksum = "15fcc6e3f0ca54d0fc779009bc5f2684cea9147be3b6aa68a7d301ea590f95f5"
 dependencies = [
  "async-utility",
  "async-wsocket",
@@ -2101,35 +2085,21 @@ dependencies = [
 
 [[package]]
 name = "nostr-sdk"
-version = "0.36.0"
+version = "0.37.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e4739ed15ff81a0e474d79b38c3eb481ff5f968c1865f38ba46852daf6f6495e"
+checksum = "491221fc89b1aa189a0de640127127d68b4e7c5c1d44371b04d9a6d10694b5af"
 dependencies = [
  "async-utility",
  "atomic-destructor",
- "lnurl-pay",
  "nostr",
  "nostr-database",
  "nostr-relay-pool",
- "nostr-zapper",
- "nwc",
  "thiserror 1.0.69",
  "tokio",
  "tracing",
 ]
 
 [[package]]
-name = "nostr-zapper"
-version = "0.36.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9d9709ecf8050bbe4ecf0e5efda2f25b690bb1761fc504e05654621ba9e568a8"
-dependencies = [
- "async-trait",
- "nostr",
- "thiserror 1.0.69",
-]
-
-[[package]]
 name = "num"
 version = "0.4.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
@@ -2209,21 +2179,6 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "830b246a0e5f20af87141b25c173cd1b609bd7779a4617d6ec582abaf90870f3"
 
 [[package]]
-name = "nwc"
-version = "0.36.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1b5f98bcaf232b3ec48e018792ca7bc2b90e7520d001a07b8218a9e76a03fda2"
-dependencies = [
- "async-trait",
- "async-utility",
- "nostr",
- "nostr-relay-pool",
- "nostr-zapper",
- "thiserror 1.0.69",
- "tracing",
-]
-
-[[package]]
 name = "object"
 version = "0.36.5"
 source = "registry+https://github.com/rust-lang/crates.io-index"
@@ -2646,7 +2601,7 @@ dependencies = [
  "once_cell",
  "socket2 0.5.7",
  "tracing",
- "windows-sys 0.59.0",
+ "windows-sys 0.52.0",
 ]
 
 [[package]]
diff --git a/Cargo.toml b/Cargo.toml
index 8ed3741..4af74d0 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -24,10 +24,11 @@ futures = "0.3.28"
 git2 = "0.19.0"
 indicatif = "0.17.7"
 keyring = "2.0.5"
-nostr = "0.36.0"
-nostr-database = "0.36.0"
-nostr-sdk = "0.36.0"
-nostr-lmdb = "0.36.0"
+nostr = { version = "0.37.0", features = ["nip05", "nip49"] }
+nostr-connect = "0.37.0"
+nostr-database = "0.37.0"
+nostr-lmdb = "0.37.0"
+nostr-sdk = "0.37.0"
 passwords = "3.1.13"
 qrcode = { version = "0.14.1", default-features = false }
 scrypt = "0.11.0"
@@ -37,7 +38,6 @@ serde_yaml = "0.9.27"
 tokio = { version = "1.40.0", features = ["full"] }
 urlencoding = "2.1.3"
 zeroize = "1.6.0"
-nostr-connect = "0.36.0"
 
 [dev-dependencies]
 assert_cmd = "2.0.12"
diff --git a/src/bin/git_remote_nostr/push.rs b/src/bin/git_remote_nostr/push.rs
index 381a45e..d8ba65a 100644
--- a/src/bin/git_remote_nostr/push.rs
+++ b/src/bin/git_remote_nostr/push.rs
@@ -961,9 +961,7 @@ async fn create_merge_status(
         public_keys.insert(revision.pubkey);
     }
     sign_event(
-        EventBuilder::new(
-            nostr::event::Kind::GitStatusApplied,
-            String::new(),
+        EventBuilder::new(nostr::event::Kind::GitStatusApplied, String::new()).tags(
             [
                 vec![
                     Tag::custom(
@@ -972,23 +970,26 @@ async fn create_merge_status(
                     ),
                     Tag::from_standardized(nostr::TagStandard::Event {
                         event_id: proposal.id,
-                        relay_url: repo_ref.relays.first().map(nostr::UncheckedUrl::new),
+                        relay_url: repo_ref.relays.first().cloned(),
                         marker: Some(Marker::Root),
                         public_key: None,
+                        uppercase: false,
                     }),
                     Tag::from_standardized(nostr::TagStandard::Event {
                         event_id: merged_patch,
-                        relay_url: repo_ref.relays.first().map(nostr::UncheckedUrl::new),
+                        relay_url: repo_ref.relays.first().cloned(),
                         marker: Some(Marker::Mention),
                         public_key: None,
+                        uppercase: false,
                     }),
                 ],
                 if let Some(revision) = revision {
                     vec![Tag::from_standardized(nostr::TagStandard::Event {
                         event_id: revision.id,
-                        relay_url: repo_ref.relays.first().map(nostr::UncheckedUrl::new),
+                        relay_url: repo_ref.relays.first().cloned(),
                         marker: Some(Marker::Root),
                         public_key: None,
+                        uppercase: false,
                     })]
                 } else {
                     vec![]
@@ -1202,7 +1203,7 @@ impl BuildRepoState for RepoState {
                 vec![value.clone()],
             ));
         }
-        let event = sign_event(EventBuilder::new(STATE_KIND, "", tags), signer).await?;
+        let event = sign_event(EventBuilder::new(STATE_KIND, "").tags(tags), signer).await?;
         Ok(RepoState {
             identifier,
             state,
diff --git a/src/bin/ngit/sub_commands/init.rs b/src/bin/ngit/sub_commands/init.rs
index 2af6aef..0d3e344 100644
--- a/src/bin/ngit/sub_commands/init.rs
+++ b/src/bin/ngit/sub_commands/init.rs
@@ -3,7 +3,7 @@ use std::collections::HashMap;
 use anyhow::{Context, Result};
 use ngit::cli_interactor::PromptConfirmParms;
 use nostr::{nips::nip01::Coordinate, FromBech32, PublicKey, ToBech32};
-use nostr_sdk::Kind;
+use nostr_sdk::{Kind, RelayUrl};
 
 use crate::{
     cli::{extract_signer_cli_arguments, Cli},
@@ -323,24 +323,45 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
 
     // TODO: check if relays are free to post to so contributors can submit patches
     // TODO: recommend some reliable free ones
-    let relays: Vec<String> = if args.relays.is_empty() {
-        Interactor::default()
-            .input(
-                PromptInputParms::default()
-                    .with_prompt("relays")
-                    .with_default(if let Ok(config) = &repo_config_result {
-                        config.relays.clone().join(" ")
-                    } else if let Some(repo_ref) = &repo_ref {
-                        repo_ref.relays.clone().join(" ")
-                    } else {
-                        user_ref.relays.write().join(" ")
-                    }),
-            )?
-            .split(' ')
-            .map(std::string::ToString::to_string)
-            .collect()
-    } else {
-        args.relays.clone()
+    let relays: Vec<RelayUrl> = {
+        let mut default = if let Ok(config) = &repo_config_result {
+            config.relays.clone().join(" ")
+        } else if let Some(repo_ref) = &repo_ref {
+            repo_ref
+                .relays
+                .iter()
+                .map(std::string::ToString::to_string)
+                .collect::<Vec<String>>()
+                .join(" ")
+        } else {
+            user_ref.relays.write().join(" ")
+        };
+        'outer: loop {
+            let relays: Vec<String> = if args.relays.is_empty() {
+                Interactor::default()
+                    .input(
+                        PromptInputParms::default()
+                            .with_prompt("relays")
+                            .with_default(default),
+                    )?
+                    .split(' ')
+                    .map(std::string::ToString::to_string)
+                    .collect()
+            } else {
+                args.relays.clone()
+            };
+            let mut relay_urls = vec![];
+            for r in &relays {
+                if let Ok(r) = RelayUrl::parse(r) {
+                    relay_urls.push(r);
+                } else {
+                    eprintln!("{r} is not a valid relay url");
+                    default = relays.join(" ");
+                    continue 'outer;
+                }
+            }
+            break relay_urls;
+        }
     };
 
     let earliest_unique_commit = match &args.earliest_unique_commit {
@@ -415,6 +436,10 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
         false,
     )?;
 
+    let relays = relays
+        .iter()
+        .map(std::string::ToString::to_string)
+        .collect::<Vec<String>>();
     // if yaml file doesnt exist or needs updating
     if match &repo_config_result {
         Ok(config) => {
diff --git a/src/bin/ngit/sub_commands/send.rs b/src/bin/ngit/sub_commands/send.rs
index 1965875..6c9d8eb 100644
--- a/src/bin/ngit/sub_commands/send.rs
+++ b/src/bin/ngit/sub_commands/send.rs
@@ -237,7 +237,7 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs, no_fetch: bool) -> Re
     if root_proposal_id.is_none() {
         if let Some(event) = events.first() {
             let event_bech32 = if let Some(relay) = repo_ref.relays.first() {
-                Nip19Event::new(event.id, vec![relay]).to_bech32()?
+                Nip19Event::new(event.id, vec![relay.to_string()]).to_bech32()?
             } else {
                 event.id.to_bech32()?
             };
@@ -366,6 +366,7 @@ async fn get_root_proposal_id_and_mentions_from_in_reply_to(
                 relay_url: _,
                 marker: _,
                 public_key: _,
+                uppercase: false,
             }) => {
                 let events = get_events_from_local_cache(
                     git_repo_path,
diff --git a/src/lib/client.rs b/src/lib/client.rs
index 7093dd5..051aa3d 100644
--- a/src/lib/client.rs
+++ b/src/lib/client.rs
@@ -30,11 +30,11 @@ use indicatif::{MultiProgress, ProgressBar, ProgressDrawTarget, ProgressState, P
 #[cfg(test)]
 use mockall::*;
 use nostr::{nips::nip01::Coordinate, Event};
-use nostr_database::NostrDatabase;
+use nostr_database::NostrEventsDatabase;
 use nostr_lmdb::NostrLMDB;
 use nostr_sdk::{
-    prelude::RelayLimits, EventBuilder, EventId, Kind, NostrSigner, Options, PublicKey,
-    SingleLetterTag, Timestamp, Url,
+    prelude::RelayLimits, EventBuilder, EventId, Kind, NostrSigner, Options, PublicKey, RelayUrl,
+    SingleLetterTag, Timestamp,
 };
 
 use crate::{
@@ -63,7 +63,7 @@ pub trait Connect {
     fn default() -> Self;
     fn new(opts: Params) -> Self;
     async fn set_signer(&mut self, signer: Arc<dyn NostrSigner>);
-    async fn connect(&self, relay_url: &Url) -> Result<()>;
+    async fn connect(&self, relay_url: &RelayUrl) -> Result<()>;
     async fn disconnect(&self) -> Result<()>;
     fn get_fallback_relays(&self) -> &Vec<String>;
     fn get_more_fallback_relays(&self) -> &Vec<String>;
@@ -82,7 +82,7 @@ pub trait Connect {
     ) -> Result<Vec<nostr::Event>>;
     async fn get_events_per_relay(
         &self,
-        relays: Vec<Url>,
+        relays: Vec<RelayUrl>,
         filters: Vec<nostr::Filter>,
         progress_reporter: MultiProgress,
     ) -> Result<(Vec<Result<Vec<nostr::Event>>>, MultiProgress)>;
@@ -172,7 +172,7 @@ impl Connect for Client {
         self.client.set_signer(signer).await;
     }
 
-    async fn connect(&self, relay_url: &Url) -> Result<()> {
+    async fn connect(&self, relay_url: &RelayUrl) -> Result<()> {
         self.client
             .add_relay(relay_url)
             .await
@@ -244,7 +244,7 @@ impl Connect for Client {
     ) -> Result<Vec<nostr::Event>> {
         let (relay_results, _) = self
             .get_events_per_relay(
-                relays.iter().map(|r| Url::parse(r).unwrap()).collect(),
+                relays.iter().map(|r| RelayUrl::parse(r).unwrap()).collect(),
                 filters,
                 MultiProgress::new(),
             )
@@ -254,7 +254,7 @@ impl Connect for Client {
 
     async fn get_events_per_relay(
         &self,
-        relays: Vec<Url>,
+        relays: Vec<RelayUrl>,
         filters: Vec<nostr::Filter>,
         progress_reporter: MultiProgress,
     ) -> Result<(Vec<Result<Vec<nostr::Event>>>, MultiProgress)> {
@@ -335,8 +335,8 @@ impl Connect for Client {
         let fallback_relays = &self
             .fallback_relays
             .iter()
-            .filter_map(|r| Url::parse(r).ok())
-            .collect::<HashSet<Url>>();
+            .filter_map(|r| RelayUrl::parse(r).ok())
+            .collect::<HashSet<RelayUrl>>();
 
         let mut request = create_relays_request(
             git_repo_path,
@@ -359,17 +359,17 @@ impl Connect for Client {
                 // don't look for events on blaster
                 .filter(|&r| !r.as_str().contains("nostr.mutinywallet.com"))
                 .cloned()
-                .collect::<HashSet<Url>>()
+                .collect::<HashSet<RelayUrl>>()
                 .difference(&processed_relays)
                 .cloned()
-                .collect::<HashSet<Url>>();
+                .collect::<HashSet<RelayUrl>>();
             if relays.is_empty() {
                 break;
             }
             let profile_relays_only = request
                 .user_relays_for_profiles
                 .difference(&request.repo_relays)
-                .collect::<HashSet<&Url>>();
+                .collect::<HashSet<&RelayUrl>>();
             for relay in &request.repo_relays {
                 self.client
                     .add_relay(relay.as_str())
@@ -469,11 +469,7 @@ impl Connect for Client {
             processed_relays.extend(relays.clone());
 
             if let Ok(repo_ref) = get_repo_ref_from_cache(git_repo_path, repo_coordinates).await {
-                request.repo_relays = repo_ref
-                    .relays
-                    .iter()
-                    .filter_map(|r| Url::parse(r).ok())
-                    .collect();
+                request.repo_relays = repo_ref.relays.iter().cloned().collect();
             }
 
             request.user_relays_for_profiles = {
@@ -486,7 +482,7 @@ impl Connect for Client {
                 {
                     if let Ok(user_ref) = get_user_ref_from_cache(git_repo_path, user).await {
                         for r in user_ref.relays.write() {
-                            if let Ok(url) = Url::parse(&r) {
+                            if let Ok(url) = RelayUrl::parse(&r) {
                                 set.insert(url);
                             }
                         }
@@ -905,7 +901,7 @@ async fn create_relays_request(
     git_repo_path: Option<&Path>,
     repo_coordinates: &HashSet<Coordinate>,
     user_profiles: &HashSet<PublicKey>,
-    fallback_relays: HashSet<Url>,
+    fallback_relays: HashSet<RelayUrl>,
 ) -> Result<FetchRequest> {
     let repo_ref = get_repo_ref_from_cache(git_repo_path, repo_coordinates).await;
 
@@ -1026,7 +1022,7 @@ async fn create_relays_request(
         {
             if let Ok(user_ref) = get_user_ref_from_cache(git_repo_path, user).await {
                 for r in user_ref.relays.write() {
-                    if let Ok(url) = Url::parse(&r) {
+                    if let Ok(url) = RelayUrl::parse(&r) {
                         set.insert(url);
                     }
                 }
@@ -1068,17 +1064,13 @@ async fn create_relays_request(
     let relays = {
         let mut relays = fallback_relays;
         if let Ok(repo_ref) = &repo_ref {
-            for r in &repo_ref.relays {
-                if let Ok(url) = Url::parse(r) {
-                    relays.insert(url);
-                }
+            for r in repo_ref.relays.clone() {
+                relays.insert(r);
             }
         }
         for c in repo_coordinates {
             for r in &c.relays {
-                if let Ok(url) = Url::parse(r) {
-                    relays.insert(url);
-                }
+                relays.insert(r.clone());
             }
         }
         relays
@@ -1500,8 +1492,8 @@ impl Display for FetchReport {
 
 #[derive(Default, Clone)]
 pub struct FetchRequest {
-    repo_relays: HashSet<Url>,
-    selected_relay: Option<Url>,
+    repo_relays: HashSet<RelayUrl>,
+    selected_relay: Option<RelayUrl>,
     relay_column_width: usize,
     repo_coordinates_without_relays: Vec<(Coordinate, Option<Timestamp>)>,
     state: Option<(Timestamp, EventId)>,
@@ -1510,7 +1502,7 @@ pub struct FetchRequest {
     missing_contributor_profiles: HashSet<PublicKey>,
     existing_events: HashSet<EventId>,
     profiles_to_fetch_from_user_relays: HashMap<PublicKey, (Timestamp, Timestamp)>,
-    user_relays_for_profiles: HashSet<Url>,
+    user_relays_for_profiles: HashSet<RelayUrl>,
 }
 
 pub async fn fetching_with_report(
@@ -1646,7 +1638,7 @@ pub async fn send_events(
     git_repo_path: Option<&Path>,
     events: Vec<nostr::Event>,
     my_write_relays: Vec<String>,
-    repo_read_relays: Vec<String>,
+    repo_read_relays: Vec<RelayUrl>,
     animate: bool,
     silent: bool,
 ) -> Result<()> {
@@ -1659,7 +1651,12 @@ pub async fn send_events(
         },
     ]
     .concat();
-    let mut relays: Vec<&String> = vec![];
+    let mut relays: Vec<&str> = vec![];
+
+    let repo_read_relays = repo_read_relays
+        .iter()
+        .map(|r| r.to_string())
+        .collect::<Vec<String>>();
 
     let all = &[
         repo_read_relays.clone(),
@@ -1722,7 +1719,7 @@ pub async fn send_events(
 
     #[allow(clippy::borrow_deref_ref)]
     join_all(relays.iter().map(|&relay| async {
-        let relay_clean = remove_trailing_slash(&*relay);
+        let relay_clean = remove_trailing_slash(relay);
         let details = format!(
             "{}{}{} {}",
             if my_write_relays
@@ -1735,7 +1732,7 @@ pub async fn send_events(
             },
             if repo_read_relays
                 .iter()
-                .any(|r| relay_clean.eq(&remove_trailing_slash(r)))
+                .any(|r| relay_clean.eq(&remove_trailing_slash(&r.to_string())))
             {
                 " [repo-relay]"
             } else {
@@ -1763,7 +1760,7 @@ pub async fn send_events(
         let mut failed = false;
         for event in &events {
             match client
-                .send_event_to(git_repo_path, relay.as_str(), event.clone())
+                .send_event_to(git_repo_path, relay, event.clone())
                 .await
             {
                 Ok(_) => pb.inc(1),
@@ -1793,8 +1790,8 @@ pub async fn send_events(
     Ok(())
 }
 
-fn remove_trailing_slash(s: &String) -> String {
-    match s.as_str().strip_suffix('/') {
+fn remove_trailing_slash(s: &str) -> String {
+    match s.strip_suffix('/') {
         Some(s) => s,
         None => s,
     }
diff --git a/src/lib/git/nostr_url.rs b/src/lib/git/nostr_url.rs
index b310782..e4b6825 100644
--- a/src/lib/git/nostr_url.rs
+++ b/src/lib/git/nostr_url.rs
@@ -3,7 +3,7 @@ use std::{collections::HashSet, str::FromStr};
 
 use anyhow::{anyhow, bail, Context, Error, Result};
 use nostr::nips::nip01::Coordinate;
-use nostr_sdk::{PublicKey, Url};
+use nostr_sdk::{PublicKey, RelayUrl, Url};
 
 #[derive(Debug, PartialEq, Default, Clone)]
 pub enum ServerProtocol {
@@ -85,8 +85,8 @@ impl std::str::FromStr for NostrUrlDecoded {
                     decoded = format!("wss://{decoded}");
                 }
                 let url =
-                    Url::parse(&decoded).context("could not parse relays in nostr git url")?;
-                relays.push(url.to_string());
+                    RelayUrl::parse(&decoded).context("could not parse relays in nostr git url")?;
+                relays.push(url);
             } else if name == "protocol" {
                 protocol = match value.as_ref() {
                     "ssh" => Some(ServerProtocol::Ssh),
@@ -151,8 +151,8 @@ impl std::str::FromStr for NostrUrlDecoded {
                     decoded = format!("wss://{decoded}");
                 }
                 let url =
-                    Url::parse(&decoded).context("could not parse relays in nostr git url")?;
-                relays.push(url.to_string());
+                    RelayUrl::parse(&decoded).context("could not parse relays in nostr git url")?;
+                relays.push(url);
             }
             coordinates.insert(Coordinate {
                 identifier,
@@ -852,7 +852,7 @@ mod tests {
                 .unwrap(),
                 kind: nostr_sdk::Kind::GitRepoAnnouncement,
                 relays: if relays {
-                    vec!["wss://nos.lol/".to_string()]
+                    vec![RelayUrl::parse("wss://nos.lol").unwrap()]
                 } else {
                     vec![]
                 },
@@ -873,7 +873,8 @@ mod tests {
                         )
                         .unwrap(),
                         kind: nostr_sdk::Kind::GitRepoAnnouncement,
-                        relays: vec!["wss://nos.lol".to_string()], // wont add the slash
+                        relays: vec![RelayUrl::parse("wss://nos.lol").unwrap()], /* wont add the
+                                                                                  * slash */
                     }]),
                     protocol: None,
                     user: None,
@@ -942,8 +943,8 @@ mod tests {
                 fn with_multiple_encoded_relays() -> Result<()> {
                     let url = format!(
                         "nostr://npub15qydau2hjma6ngxkl2cyar74wzyjshvl65za5k5rl69264ar2exs5cyejr/ngit?relay={}&relay1={}",
-                        urlencoding::encode("wss://nos.lol"),
-                        urlencoding::encode("wss://relay.damus.io"),
+                        urlencoding::encode("wss://nos.lol/"),
+                        urlencoding::encode("wss://relay.damus.io/"),
                     );
                     assert_eq!(
                     NostrUrlDecoded::from_str(&url)?,
@@ -957,8 +958,8 @@ mod tests {
                             .unwrap(),
                             kind: nostr_sdk::Kind::GitRepoAnnouncement,
                             relays: vec![
-                                "wss://nos.lol/".to_string(),
-                                "wss://relay.damus.io/".to_string(),
+                                RelayUrl::parse("wss://nos.lol/").unwrap(),
+                                RelayUrl::parse("wss://relay.damus.io/").unwrap(),
                             ],
                         }]),
                         protocol: None,
@@ -1039,8 +1040,8 @@ mod tests {
                 fn with_multiple_encoded_relays() -> Result<()> {
                     let url = format!(
                         "nostr://npub15qydau2hjma6ngxkl2cyar74wzyjshvl65za5k5rl69264ar2exs5cyejr/{}/{}/ngit",
-                        urlencoding::encode("wss://nos.lol"),
-                        urlencoding::encode("wss://relay.damus.io"),
+                        urlencoding::encode("wss://nos.lol/"),
+                        urlencoding::encode("wss://relay.damus.io/"),
                     );
                     assert_eq!(
                     NostrUrlDecoded::from_str(&url)?,
@@ -1054,8 +1055,8 @@ mod tests {
                             .unwrap(),
                             kind: nostr_sdk::Kind::GitRepoAnnouncement,
                             relays: vec![
-                                "wss://nos.lol/".to_string(),
-                                "wss://relay.damus.io/".to_string(),
+                                RelayUrl::parse("wss://nos.lol/").unwrap(),
+                                RelayUrl::parse("wss://relay.damus.io/").unwrap(),
                             ],
                         }]),
                         protocol: None,
diff --git a/src/lib/git_events.rs b/src/lib/git_events.rs
index bfe5b30..c4d6770 100644
--- a/src/lib/git_events.rs
+++ b/src/lib/git_events.rs
@@ -4,7 +4,7 @@ use anyhow::{bail, Context, Result};
 use nostr::nips::{nip01::Coordinate, nip10::Marker, nip19::Nip19};
 use nostr_sdk::{
     hashes::sha1::Hash as Sha1Hash, Event, EventBuilder, EventId, FromBech32, Kind, NostrSigner,
-    PublicKey, Tag, TagKind, TagStandard, UncheckedUrl,
+    PublicKey, RelayUrl, Tag, TagKind, TagStandard,
 };
 
 use crate::{
@@ -96,7 +96,7 @@ pub async fn generate_patch_event(
     let commit_parent = git_repo
         .get_commit_parent(commit)
         .context("failed to get parent commit")?;
-    let relay_hint = repo_ref.relays.first().map(nostr::UncheckedUrl::from);
+    let relay_hint = repo_ref.relays.first().cloned();
 
     sign_event(
         EventBuilder::new(
@@ -104,6 +104,8 @@ pub async fn generate_patch_event(
             git_repo
                 .make_patch_from_commit(commit, &series_count)
                 .context(format!("failed to make patch for commit {commit}"))?,
+        )
+        .tags(
             [
                 repo_ref
                     .maintainers
@@ -141,6 +143,7 @@ pub async fn generate_patch_event(
                         relay_url: relay_hint.clone(),
                         marker: Some(Marker::Root),
                         public_key: None,
+                        uppercase: false,
                     })]
                 } else if let Some(event_ref) = root_proposal_id.clone() {
                     vec![
@@ -165,6 +168,7 @@ pub async fn generate_patch_event(
                         relay_url: relay_hint.clone(),
                         marker: Some(Marker::Reply),
                         public_key: None,
+                        uppercase: false,
                     })]
                 } else {
                     vec![]
@@ -256,9 +260,10 @@ pub fn event_tag_from_nip19_or_hex(
                 Nip19::Event(n) => {
                     break Ok(Tag::from_standardized(nostr_sdk::TagStandard::Event {
                         event_id: n.event_id,
-                        relay_url: n.relays.first().map(UncheckedUrl::new),
+                        relay_url: n.relays.first().and_then(|url| RelayUrl::parse(url).ok()),
                         marker: Some(marker),
                         public_key: None,
+                        uppercase: false,
                     }));
                 }
                 Nip19::EventId(id) => {
@@ -267,6 +272,7 @@ pub fn event_tag_from_nip19_or_hex(
                         relay_url: None,
                         marker: Some(marker),
                         public_key: None,
+                        uppercase: false,
                     }));
                 }
                 Nip19::Coordinate(coordinate) => {
@@ -291,6 +297,7 @@ pub fn event_tag_from_nip19_or_hex(
                 relay_url: None,
                 marker: Some(marker),
                 public_key: None,
+                uppercase: false,
             }));
         }
         if prompt_for_correction {
@@ -326,7 +333,8 @@ pub async fn generate_cover_letter_and_patch_events(
             "From {} Mon Sep 17 00:00:00 2001\nSubject: [PATCH 0/{}] {title}\n\n{description}",
             commits.last().unwrap(),
             commits.len()
-        ),
+        ))
+        .tags(
         [
             repo_ref.maintainers.iter().map(|m| Tag::coordinate(Coordinate {
                 kind: nostr::Kind::GitRepoAnnouncement,
@@ -618,7 +626,8 @@ mod tests {
             Ok(nostr::event::EventBuilder::new(
                 nostr::event::Kind::GitPatch,
                 format!("From ea897e987ea9a7a98e7a987e97987ea98e7a3334 Mon Sep 17 00:00:00 2001\nSubject: [PATCH 0/2] {title}\n\n{description}"),
-                [
+                )
+            .tags([
                     Tag::hashtag("cover-letter"),
                     Tag::hashtag("root"),
                 ],
diff --git a/src/lib/login/fresh.rs b/src/lib/login/fresh.rs
index 23c4bdc..615c0a6 100644
--- a/src/lib/login/fresh.rs
+++ b/src/lib/login/fresh.rs
@@ -5,7 +5,7 @@ use console::Style;
 use dialoguer::theme::{ColorfulTheme, Theme};
 use nostr::nips::{nip05, nip46::NostrConnectURI};
 use nostr_connect::client::NostrConnect;
-use nostr_sdk::{EventBuilder, Keys, Metadata, NostrSigner, PublicKey, ToBech32, Url};
+use nostr_sdk::{EventBuilder, Keys, Metadata, NostrSigner, PublicKey, RelayUrl, ToBech32};
 use qrcode::QrCode;
 use tokio::{signal, sync::Mutex};
 
@@ -370,8 +370,8 @@ pub fn generate_nostr_connect_app(
         client
             .get_fallback_signer_relays()
             .iter()
-            .flat_map(|s| Url::parse(s))
-            .collect::<Vec<Url>>()
+            .flat_map(RelayUrl::parse)
+            .collect::<Vec<RelayUrl>>()
     } else {
         vec![]
     };
@@ -438,8 +438,8 @@ pub async fn listen_for_remote_signer(
         let bunker_url = NostrConnectURI::Bunker {
             // TODO the remote signer pubkey may not be the user pubkey
             remote_signer_public_key: public_key,
-            relays: nostr_connect_url.relays(),
-            secret: nostr_connect_url.secret(),
+            relays: nostr_connect_url.relays().to_vec(),
+            secret: nostr_connect_url.secret().map(String::from),
         };
         Ok((signer, public_key, bunker_url))
     } else {
@@ -727,7 +727,7 @@ async fn signup(
                 client
                     .get_fallback_relays()
                     .iter()
-                    .map(|s| (Url::parse(s).unwrap(), None)),
+                    .map(|s| (RelayUrl::parse(s).unwrap(), None)),
             )
             .sign_with_keys(&keys)?;
             eprintln!("publishing user profile to relays");
diff --git a/src/lib/repo_ref.rs b/src/lib/repo_ref.rs
index 05234e2..8319c78 100644
--- a/src/lib/repo_ref.rs
+++ b/src/lib/repo_ref.rs
@@ -9,7 +9,7 @@ use std::{
 use anyhow::{bail, Context, Result};
 use console::Style;
 use nostr::{nips::nip01::Coordinate, FromBech32, PublicKey, Tag, TagStandard, ToBech32};
-use nostr_sdk::{Kind, NostrSigner, Timestamp};
+use nostr_sdk::{Kind, NostrSigner, RelayUrl, Timestamp};
 use serde::{Deserialize, Serialize};
 
 #[cfg(not(test))]
@@ -28,7 +28,7 @@ pub struct RepoRef {
     pub root_commit: String,
     pub git_server: Vec<String>,
     pub web: Vec<String>,
-    pub relays: Vec<String>,
+    pub relays: Vec<RelayUrl>,
     pub maintainers: Vec<PublicKey>,
     pub events: HashMap<Coordinate, nostr::Event>,
     // code languages and hashtags
@@ -78,8 +78,11 @@ impl TryFrom<nostr::Event> for RepoRef {
         }
 
         if let Some(t) = event.tags.iter().find(|t| t.as_slice()[0].eq("relays")) {
-            r.relays = t.clone().to_vec();
-            r.relays.remove(0);
+            for relay in t.clone().to_vec() {
+                if let Ok(relay) = RelayUrl::parse(relay) {
+                    r.relays.push(relay);
+                }
+            }
         }
 
         if let Some(t) = event
@@ -119,9 +122,7 @@ impl TryFrom<nostr::Event> for RepoRef {
 impl RepoRef {
     pub async fn to_event(&self, signer: &Arc<dyn NostrSigner>) -> Result<nostr::Event> {
         sign_event(
-            nostr_sdk::EventBuilder::new(
-                nostr::event::Kind::GitRepoAnnouncement,
-                "",
+            nostr_sdk::EventBuilder::new(nostr::event::Kind::GitRepoAnnouncement, "").tags(
                 [
                     vec![
                         Tag::identifier(if self.identifier.to_string().is_empty() {
@@ -158,7 +159,7 @@ impl RepoRef {
                         ),
                         Tag::custom(
                             nostr::TagKind::Custom(std::borrow::Cow::Borrowed("relays")),
-                            self.relays.clone(),
+                            self.relays.iter().map(|r| r.to_string()),
                         ),
                         Tag::custom(
                             nostr::TagKind::Custom(std::borrow::Cow::Borrowed("maintainers")),
@@ -206,7 +207,7 @@ impl RepoRef {
                 .unwrap(),
             identifier: self.identifier.clone(),
             relays: if let Some(relay) = self.relays.first() {
-                vec![relay.to_string()]
+                vec![relay.clone()]
             } else {
                 vec![]
             },
@@ -481,7 +482,10 @@ mod tests {
                 "https://exampleproject.xyz".to_string(),
                 "https://gitworkshop.dev/123".to_string(),
             ],
-            relays: vec!["ws://relay1.io".to_string(), "ws://relay2.io".to_string()],
+            relays: vec![
+                RelayUrl::parse("ws://relay1.io").unwrap(),
+                RelayUrl::parse("ws://relay2.io").unwrap(),
+            ],
             maintainers: vec![TEST_KEY_1_KEYS.public_key(), TEST_KEY_2_KEYS.public_key()],
             events: HashMap::new(),
         }
@@ -592,7 +596,10 @@ mod tests {
         async fn relays() {
             assert_eq!(
                 RepoRef::try_from(create().await).unwrap().relays,
-                vec!["ws://relay1.io".to_string(), "ws://relay2.io".to_string()],
+                vec![
+                    RelayUrl::parse("ws://relay1.io").unwrap(),
+                    RelayUrl::parse("ws://relay2.io").unwrap(),
+                ],
             )
         }
 
diff --git a/test_utils/Cargo.toml b/test_utils/Cargo.toml
index d27ecc0..672dcdb 100644
--- a/test_utils/Cargo.toml
+++ b/test_utils/Cargo.toml
@@ -10,10 +10,10 @@ dialoguer = "0.10.4"
 directories = "5.0.1"
 futures = "0.3.28"
 git2 = "0.19.0"
-nostr = "0.36.0"
-nostr-database = "0.36.0"
-nostr-sdk = "0.36.0"
-nostr-lmdb = "0.36.0"
+nostr = "0.37.0"
+nostr-database = "0.37.0"
+nostr-lmdb = "0.37.0"
+nostr-sdk = "0.37.0"
 once_cell = "1.18.0"
 rand = "0.8"
 rexpect = { git = "https://github.com/rust-cli/rexpect.git", rev = "9eb61dd" }
diff --git a/test_utils/src/git.rs b/test_utils/src/git.rs
index 2562799..474fc59 100644
--- a/test_utils/src/git.rs
+++ b/test_utils/src/git.rs
@@ -10,7 +10,7 @@ use std::{
 use anyhow::{Context, Result};
 use git2::{Branch, Oid, RepositoryInitOptions, Signature, Time};
 use nostr::nips::nip01::Coordinate;
-use nostr_sdk::{Kind, ToBech32};
+use nostr_sdk::{Kind, RelayUrl, ToBech32};
 
 use crate::generate_repo_ref_event;
 
@@ -28,8 +28,8 @@ impl Default for GitTestRepo {
             public_key: repo_event.pubkey,
             identifier: repo_event.tags.identifier().unwrap().to_string(),
             relays: vec![
-                "ws://localhost:8055".to_string(),
-                "ws://localhost:8056".to_string(),
+                RelayUrl::parse("ws://localhost:8055").unwrap(),
+                RelayUrl::parse("ws://localhost:8056").unwrap(),
             ],
         };
 
diff --git a/test_utils/src/lib.rs b/test_utils/src/lib.rs
index 6708c81..0c5de73 100644
--- a/test_utils/src/lib.rs
+++ b/test_utils/src/lib.rs
@@ -11,7 +11,7 @@ use dialoguer::theme::{ColorfulTheme, Theme};
 use futures::executor::block_on;
 use git::GitTestRepo;
 use nostr::{self, nips::nip65::RelayMetadata, Kind, Tag};
-use nostr_database::NostrDatabase;
+use nostr_database::NostrEventsDatabase;
 use nostr_lmdb::NostrLMDB;
 use nostr_sdk::{serde_json, Client, NostrSigner, TagStandard};
 use once_cell::sync::Lazy;
@@ -58,51 +58,46 @@ pub fn generate_test_key_1_metadata_event_old(name: &str) -> nostr::Event {
 }
 
 pub fn generate_test_key_1_kind_event(kind: Kind) -> nostr::Event {
-    nostr::event::EventBuilder::new(kind, "", [])
+    nostr::event::EventBuilder::new(kind, "")
+        .tags([])
         .sign_with_keys(&TEST_KEY_1_KEYS)
         .unwrap()
 }
 
 pub fn generate_test_key_1_relay_list_event() -> nostr::Event {
-    nostr::event::EventBuilder::new(
-        nostr::Kind::RelayList,
-        "",
-        [
+    nostr::event::EventBuilder::new(nostr::Kind::RelayList, "")
+        .tags([
             nostr::Tag::from_standardized(nostr::TagStandard::RelayMetadata {
-                relay_url: nostr::Url::from_str("ws://localhost:8053").unwrap(),
+                relay_url: nostr::RelayUrl::from_str("ws://localhost:8053").unwrap(),
                 metadata: Some(RelayMetadata::Write),
             }),
             nostr::Tag::from_standardized(nostr::TagStandard::RelayMetadata {
-                relay_url: nostr::Url::from_str("ws://localhost:8054").unwrap(),
+                relay_url: nostr::RelayUrl::from_str("ws://localhost:8054").unwrap(),
                 metadata: Some(RelayMetadata::Read),
             }),
             nostr::Tag::from_standardized(nostr::TagStandard::RelayMetadata {
-                relay_url: nostr::Url::from_str("ws://localhost:8055").unwrap(),
+                relay_url: nostr::RelayUrl::from_str("ws://localhost:8055").unwrap(),
                 metadata: None,
             }),
-        ],
-    )
-    .sign_with_keys(&TEST_KEY_1_KEYS)
-    .unwrap()
+        ])
+        .sign_with_keys(&TEST_KEY_1_KEYS)
+        .unwrap()
 }
 
 pub fn generate_test_key_1_relay_list_event_same_as_fallback() -> nostr::Event {
-    nostr::event::EventBuilder::new(
-        nostr::Kind::RelayList,
-        "",
-        [
+    nostr::event::EventBuilder::new(nostr::Kind::RelayList, "")
+        .tags([
             nostr::Tag::from_standardized(nostr::TagStandard::RelayMetadata {
-                relay_url: nostr::Url::from_str("ws://localhost:8051").unwrap(),
+                relay_url: nostr::RelayUrl::from_str("ws://localhost:8051").unwrap(),
                 metadata: Some(RelayMetadata::Write),
             }),
             nostr::Tag::from_standardized(nostr::TagStandard::RelayMetadata {
-                relay_url: nostr::Url::from_str("ws://localhost:8052").unwrap(),
+                relay_url: nostr::RelayUrl::from_str("ws://localhost:8052").unwrap(),
                 metadata: Some(RelayMetadata::Write),
             }),
-        ],
-    )
-    .sign_with_keys(&TEST_KEY_1_KEYS)
-    .unwrap()
+        ])
+        .sign_with_keys(&TEST_KEY_1_KEYS)
+        .unwrap()
 }
 
 pub static TEST_KEY_2_NSEC: &str =
@@ -133,9 +128,9 @@ pub fn make_event_old_or_change_user(
     keys: &nostr::Keys,
     how_old_in_secs: u64,
 ) -> nostr::Event {
-    let mut unsigned =
-        nostr::event::EventBuilder::new(event.kind, event.content.clone(), event.tags.clone())
-            .build(keys.public_key());
+    let mut unsigned = nostr::event::EventBuilder::new(event.kind, event.content.clone())
+        .tags(event.tags.clone())
+        .build(keys.public_key());
 
     unsigned.created_at =
         nostr::types::Timestamp::from(nostr::types::Timestamp::now().as_u64() - how_old_in_secs);
@@ -159,10 +154,8 @@ pub fn generate_repo_ref_event_with_git_server(git_servers: Vec<String>) -> nost
     // TODO - this may not be consistant across computers as it might take the
     // author and committer from global git config
     let root_commit = "9ee507fc4357d7ee16a5d8901bedcd103f23c17d";
-    nostr::event::EventBuilder::new(
-        nostr::Kind::GitRepoAnnouncement,
-        "",
-        [
+    nostr::event::EventBuilder::new(nostr::Kind::GitRepoAnnouncement, "")
+        .tags([
             Tag::identifier(
                 // root_commit.to_string()
                 format!("{}-consider-it-random", root_commit),
@@ -195,10 +188,9 @@ pub fn generate_repo_ref_event_with_git_server(git_servers: Vec<String>) -> nost
                     TEST_KEY_2_KEYS.public_key().to_string(),
                 ],
             ),
-        ],
-    )
-    .sign_with_keys(&TEST_KEY_1_KEYS)
-    .unwrap()
+        ])
+        .sign_with_keys(&TEST_KEY_1_KEYS)
+        .unwrap()
 }
 
 /// enough to fool event_is_patch_set_root
diff --git a/tests/git_remote_nostr/main.rs b/tests/git_remote_nostr/main.rs
index 84e66bd..5a1d416 100644
--- a/tests/git_remote_nostr/main.rs
+++ b/tests/git_remote_nostr/main.rs
@@ -4,7 +4,7 @@ use anyhow::{Context, Result};
 use futures::join;
 use git2::Oid;
 use nostr::nips::nip01::Coordinate;
-use nostr_sdk::{secp256k1::rand, Event, JsonUtil, Kind, ToBech32};
+use nostr_sdk::{secp256k1::rand, Event, JsonUtil, Kind, RelayUrl, ToBech32};
 use relay::Relay;
 use serial_test::serial;
 use test_utils::{git::GitTestRepo, *};
@@ -23,8 +23,8 @@ fn get_nostr_remote_url() -> Result<String> {
         public_key: repo_event.pubkey,
         identifier: repo_event.tags.identifier().unwrap().to_string(),
         relays: vec![
-            "ws://localhost:8055".to_string(),
-            "ws://localhost:8056".to_string(),
+            RelayUrl::parse("ws://localhost:8055").unwrap(),
+            RelayUrl::parse("ws://localhost:8056").unwrap(),
         ],
     }
     .to_bech32()?;
diff --git a/tests/ngit_list.rs b/tests/ngit_list.rs
index a6b1374..ab6ba22 100644
--- a/tests/ngit_list.rs
+++ b/tests/ngit_list.rs
@@ -50,6 +50,7 @@ mod cannot_find_repo_event {
     use super::*;
     mod cli_prompts {
         use nostr::{nips::nip01::Coordinate, ToBech32};
+        use nostr_sdk::RelayUrl;
 
         use super::*;
         async fn run_async_repo_event_ref_needed(invalid_input: bool, naddr: bool) -> Result<()> {
@@ -91,7 +92,7 @@ mod cannot_find_repo_event {
                             kind: nostr::Kind::GitRepoAnnouncement,
                             public_key: TEST_KEY_1_KEYS.public_key(),
                             identifier: repo_event.tags.identifier().unwrap().to_string(),
-                            relays: vec!["ws://localhost:8056".to_string()],
+                            relays: vec![RelayUrl::parse("ws://localhost:8056").unwrap()],
                         }
                         .to_bech32()?,
                     )?;