nostr: rewrite tag

* Rename `Tag` enum to `TagStandard`
* Add `Tag` struct
* Remove `GenericTagValue`

Signed-off-by: Yuki Kishimoto <yukikishimoto@protonmail.com>

CHANGELOG.md

@@ -20,6 +20,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * nostr: adj. kind to be `u16` instead of `u64` according to NIP01 ([Yuki Kishimoto])
 * nostr: improve NIP19 serialization performance ([Yuki Kishimoto])
 * nostr: improve `EventId::from_hex` performance ([Yuki Kishimoto])
+* nostr: rename `Tag` enum to `TagStandard` ([Yuki Kishimoto])
 * database: small improvements to flatbuffers `Event::encode` ([Yuki Kishimoto])
 * pool: inline `RelayPool` methods ([Yuki Kishimoto])
 * sdk: inline `Client`, `ClientBuilder` and `Options` methods ([Yuki Kishimoto])
@@ -40,6 +41,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * nostr: add tags indexes to `Event` ([Yuki Kishimoto])
 * nostr: add `hex::decode_to_slice` ([Yuki Kishimoto])
 * nostr: add `SecretKey::generate` ([Yuki Kishimoto])
+* nostr: add `Tag` struct ([Yuki Kishimoto])
 * pool: add `RelayPool::start` ([Yuki Kishimoto])
 * pool: add `NegentropyDirection` default ([Yuki Kishimoto])
 * sdk: add `Client::builder()` ([Yuki Kishimoto])
@@ -56,6 +58,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Removed
 
+* nostr: remove `GenericTagValue` ([Yuki Kishimoto])
 * ffi(nostr): remove `Kind::match*` methods ([Yuki Kishimoto])
 
 ## [v0.30.0]

bindings/nostr-ffi/src/event/builder.rs

@@ -95,13 +95,16 @@ impl EventBuilder {
     }
 
     #[uniffi::constructor]
-    pub fn relay_list(list: HashMap<String, Option<RelayMetadata>>) -> Self {
-        let iter = list
-            .into_iter()
-            .map(|(url, r)| (UncheckedUrl::from(url), r.map(|r| r.into())));
-        Self {
-            inner: nostr::EventBuilder::relay_list(iter),
+    pub fn relay_list(map: HashMap<String, Option<RelayMetadata>>) -> Result<Self> {
+        let mut list = Vec::with_capacity(map.len());
+        for (url, metadata) in map.into_iter() {
+            let relay_url: Url = Url::parse(&url)?;
+            let metadata = metadata.map(|m| m.into());
+            list.push((relay_url, metadata))
         }
+        Ok(Self {
+            inner: nostr::EventBuilder::relay_list(list),
+        })
     }
 
     #[uniffi::constructor]

bindings/nostr-ffi/src/event/mod.rs

@@ -18,7 +18,7 @@ pub mod unsigned;
 pub use self::builder::EventBuilder;
 pub use self::id::EventId;
 pub use self::kind::{Kind, KindEnum};
-pub use self::tag::{RelayMetadata, Tag, TagEnum, TagKind};
+pub use self::tag::{RelayMetadata, Tag, TagKind};
 pub use self::unsigned::UnsignedEvent;
 use crate::error::Result;
 use crate::nips::nip01::Coordinate;