upleb.uk

Public git repos — served from a NIP-34 GRASP relay at git.upleb.uk

summaryrefslogtreecommitdiff
path: root/src/nostr
diff options
context:
space:
mode:
authorDanConwayDev <DanConwayDev@protonmail.com>2025-11-19 17:01:36 +0000
committerDanConwayDev <DanConwayDev@protonmail.com>2025-11-19 17:01:36 +0000
commitbf7f4d5381203d5c27b2811d62c5b1781533aa2b (patch)
tree26903bbf535d83abd7242370d8b6932eb80e3389 /src/nostr
parentfa065ad128882755f2a988d6203b59a2ab5e38ff (diff)
fix some clippy fmt warnings
Diffstat (limited to 'src/nostr')
-rw-r--r--src/nostr/events.rs74
1 files changed, 29 insertions, 45 deletions
diff --git a/src/nostr/events.rs b/src/nostr/events.rs
index 88aefed..21dd2dd 100644
--- a/src/nostr/events.rs
+++ b/src/nostr/events.rs
@@ -1,12 +1,11 @@
1/// NIP-34 Git Repository Event Handling 1/// NIP-34 Git Repository Event Handling
2/// 2///
3/// This module handles Git repository announcements (kind 30617) and 3/// This module handles Git repository announcements (kind 30617) and
4/// repository state announcements (kind 30618) according to NIP-34 and GRASP-01. 4/// repository state announcements (kind 30618) according to NIP-34 and GRASP-01.
5/// 5///
6/// Reference: 6/// Reference:
7/// - NIP-34: https://nips.nostr.com/34 7/// - NIP-34: https://nips.nostr.com/34
8/// - GRASP-01: https://gitworkshop.dev/danconwaydev.com/grasp/01.md 8/// - GRASP-01: https://gitworkshop.dev/danconwaydev.com/grasp/01.md
9
10use anyhow::{anyhow, Result}; 9use anyhow::{anyhow, Result};
11use nostr_sdk::{Event, Kind, TagKind, ToBech32}; 10use nostr_sdk::{Event, Kind, TagKind, ToBech32};
12 11
@@ -138,8 +137,8 @@ impl RepositoryAnnouncement {
138 } 137 }
139 138
140 /// Check if this announcement lists the service (both clone and relay) 139 /// Check if this announcement lists the service (both clone and relay)
141 /// 140 ///
142 /// GRASP-01 requirement: MUST reject announcements that do not list 141 /// GRASP-01 requirement: MUST reject announcements that do not list
143 /// the service in both `clone` and `relays` tags unless implementing GRASP-05. 142 /// the service in both `clone` and `relays` tags unless implementing GRASP-05.
144 pub fn lists_service(&self, domain: &str) -> bool { 143 pub fn lists_service(&self, domain: &str) -> bool {
145 self.has_clone_url(domain) && self.has_relay(domain) 144 self.has_clone_url(domain) && self.has_relay(domain)
@@ -278,19 +277,19 @@ impl RepositoryState {
278} 277}
279 278
280/// Validate a repository announcement according to GRASP-01 279/// Validate a repository announcement according to GRASP-01
281/// 280///
282/// Returns Ok(()) if valid, Err with reason if invalid. 281/// Returns Ok(()) if valid, Err with reason if invalid.
283pub fn validate_announcement(event: &Event, domain: &str) -> Result<()> { 282pub fn validate_announcement(event: &Event, domain: &str) -> Result<()> {
284 // Must be kind 30617 283 // Must be kind 30617
285 if event.kind != Kind::from(KIND_REPOSITORY_ANNOUNCEMENT) { 284 if event.kind != Kind::from(KIND_REPOSITORY_ANNOUNCEMENT) {
286 return Err(anyhow!("Invalid kind: expected {}", KIND_REPOSITORY_ANNOUNCEMENT)); 285 return Err(anyhow!(
286 "Invalid kind: expected {}",
287 KIND_REPOSITORY_ANNOUNCEMENT
288 ));
287 } 289 }
288 290
289 // Must have identifier 291 // Must have identifier
290 let has_identifier = event 292 let has_identifier = event.tags.iter().any(|t| t.kind() == TagKind::d());
291 .tags
292 .iter()
293 .any(|t| t.kind() == TagKind::d());
294 if !has_identifier { 293 if !has_identifier {
295 return Err(anyhow!("Missing required 'd' tag (identifier)")); 294 return Err(anyhow!("Missing required 'd' tag (identifier)"));
296 } 295 }
@@ -298,7 +297,7 @@ pub fn validate_announcement(event: &Event, domain: &str) -> Result<()> {
298 // Parse full announcement to validate structure 297 // Parse full announcement to validate structure
299 let announcement = RepositoryAnnouncement::from_event(event.clone())?; 298 let announcement = RepositoryAnnouncement::from_event(event.clone())?;
300 299
301 // GRASP-01: MUST reject announcements that do not list the service 300 // GRASP-01: MUST reject announcements that do not list the service
302 // in both `clone` and `relays` tags unless implementing GRASP-05 301 // in both `clone` and `relays` tags unless implementing GRASP-05
303 if !announcement.lists_service(domain) { 302 if !announcement.lists_service(domain) {
304 return Err(anyhow!( 303 return Err(anyhow!(
@@ -313,7 +312,7 @@ pub fn validate_announcement(event: &Event, domain: &str) -> Result<()> {
313} 312}
314 313
315/// Validate a repository state announcement according to GRASP-01 314/// Validate a repository state announcement according to GRASP-01
316/// 315///
317/// Returns Ok(()) if valid, Err with reason if invalid. 316/// Returns Ok(()) if valid, Err with reason if invalid.
318pub fn validate_state(event: &Event) -> Result<()> { 317pub fn validate_state(event: &Event) -> Result<()> {
319 // Must be kind 30618 318 // Must be kind 30618
@@ -322,10 +321,7 @@ pub fn validate_state(event: &Event) -> Result<()> {
322 } 321 }
323 322
324 // Must have identifier 323 // Must have identifier
325 let has_identifier = event 324 let has_identifier = event.tags.iter().any(|t| t.kind() == TagKind::d());
326 .tags
327 .iter()
328 .any(|t| t.kind() == TagKind::d());
329 if !has_identifier { 325 if !has_identifier {
330 return Err(anyhow!("Missing required 'd' tag (identifier)")); 326 return Err(anyhow!("Missing required 'd' tag (identifier)"));
331 } 327 }
@@ -352,7 +348,7 @@ mod tests {
352 relays: Vec<&str>, 348 relays: Vec<&str>,
353 ) -> Event { 349 ) -> Event {
354 use nostr_sdk::Tag; 350 use nostr_sdk::Tag;
355 351
356 let mut tags = vec![Tag::custom( 352 let mut tags = vec![Tag::custom(
357 nostr_sdk::TagKind::d(), 353 nostr_sdk::TagKind::d(),
358 vec![identifier.to_string()], 354 vec![identifier.to_string()],
@@ -372,18 +368,15 @@ mod tests {
372 )); 368 ));
373 } 369 }
374 370
375 EventBuilder::new( 371 EventBuilder::new(Kind::from(KIND_REPOSITORY_ANNOUNCEMENT), "Test repository")
376 Kind::from(KIND_REPOSITORY_ANNOUNCEMENT), 372 .tags(tags)
377 "Test repository", 373 .sign_with_keys(keys)
378 ) 374 .unwrap()
379 .tags(tags)
380 .sign_with_keys(keys)
381 .unwrap()
382 } 375 }
383 376
384 fn create_state_event(keys: &Keys, identifier: &str, branches: Vec<(&str, &str)>) -> Event { 377 fn create_state_event(keys: &Keys, identifier: &str, branches: Vec<(&str, &str)>) -> Event {
385 use nostr_sdk::Tag; 378 use nostr_sdk::Tag;
386 379
387 let mut tags = vec![Tag::custom( 380 let mut tags = vec![Tag::custom(
388 nostr_sdk::TagKind::d(), 381 nostr_sdk::TagKind::d(),
389 vec![identifier.to_string()], 382 vec![identifier.to_string()],
@@ -392,10 +385,7 @@ mod tests {
392 for (branch, commit) in branches { 385 for (branch, commit) in branches {
393 tags.push(Tag::custom( 386 tags.push(Tag::custom(
394 nostr_sdk::TagKind::Custom("ref".into()), 387 nostr_sdk::TagKind::Custom("ref".into()),
395 vec![ 388 vec![format!("refs/heads/{}", branch), commit.to_string()],
396 format!("refs/heads/{}", branch),
397 commit.to_string(),
398 ],
399 )); 389 ));
400 } 390 }
401 391
@@ -428,12 +418,9 @@ mod tests {
428 #[test] 418 #[test]
429 fn test_parse_announcement_missing_identifier() { 419 fn test_parse_announcement_missing_identifier() {
430 let keys = create_test_keys(); 420 let keys = create_test_keys();
431 let event = EventBuilder::new( 421 let event = EventBuilder::new(Kind::from(KIND_REPOSITORY_ANNOUNCEMENT), "Test repository")
432 Kind::from(KIND_REPOSITORY_ANNOUNCEMENT), 422 .sign_with_keys(&keys)
433 "Test repository", 423 .unwrap();
434 )
435 .sign_with_keys(&keys)
436 .unwrap();
437 424
438 let result = RepositoryAnnouncement::from_event(event); 425 let result = RepositoryAnnouncement::from_event(event);
439 assert!(result.is_err()); 426 assert!(result.is_err());
@@ -539,7 +526,7 @@ mod tests {
539 #[test] 526 #[test]
540 fn test_announcement_maintainers() { 527 fn test_announcement_maintainers() {
541 use nostr_sdk::Tag; 528 use nostr_sdk::Tag;
542 529
543 let keys = create_test_keys(); 530 let keys = create_test_keys();
544 let maintainer_keys = create_test_keys(); 531 let maintainer_keys = create_test_keys();
545 532
@@ -558,13 +545,10 @@ mod tests {
558 // Add maintainer 545 // Add maintainer
559 tags.push(Tag::public_key(maintainer_keys.public_key())); 546 tags.push(Tag::public_key(maintainer_keys.public_key()));
560 547
561 let event = EventBuilder::new( 548 let event = EventBuilder::new(Kind::from(KIND_REPOSITORY_ANNOUNCEMENT), "Test repository")
562 Kind::from(KIND_REPOSITORY_ANNOUNCEMENT), 549 .tags(tags)
563 "Test repository", 550 .sign_with_keys(&keys)
564 ) 551 .unwrap();
565 .tags(tags)
566 .sign_with_keys(&keys)
567 .unwrap();
568 552
569 let announcement = RepositoryAnnouncement::from_event(event).unwrap(); 553 let announcement = RepositoryAnnouncement::from_event(event).unwrap();
570 assert_eq!(announcement.maintainers.len(), 1); 554 assert_eq!(announcement.maintainers.len(), 1);
@@ -573,7 +557,7 @@ mod tests {
573 #[test] 557 #[test]
574 fn test_state_with_tags() { 558 fn test_state_with_tags() {
575 use nostr_sdk::Tag; 559 use nostr_sdk::Tag;
576 560
577 let keys = create_test_keys(); 561 let keys = create_test_keys();
578 let mut tags = vec![Tag::custom( 562 let mut tags = vec![Tag::custom(
579 nostr_sdk::TagKind::d(), 563 nostr_sdk::TagKind::d(),