refactor(grasp-audit): introduce SpecRef enum for type-safe spec references

Replace string-based spec references with typed SpecRef enum for
compile-time validation and better IDE support. TestResult::new() now
accepts SpecRef enum plus a requirement description string for
test-specific context.

1 files changed, 23 insertions, 22 deletions

diff --git a/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs b/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs
index 5b697d8..8259283 100644
--- a/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs
+++ b/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs
@@ -92,6 +92,7 @@
 //! - Transitive tests verify multi-hop acceptance chains
 
 use crate::fixtures::{send_and_verify_accepted, send_and_verify_rejected};
+use crate::specs::grasp01::SpecRef;
 use crate::{AuditClient, AuditResult, FixtureKind, TestContext, TestResult};
 use nostr_sdk::{Event, Filter, Kind, Tag, TagKind, Timestamp, ToBech32};
 use std::time::Duration;
@@ -148,8 +149,8 @@ impl EventAcceptancePolicyTests {
     pub async fn test_accept_valid_repo_announcement(client: &AuditClient) -> TestResult {
         TestResult::new(
             "accept_valid_repo_announcement",
-            "GRASP-01:nostr-relay:7",
-            "Accept valid repository announcements with service in clone and relays tags",
+            SpecRef::NostrRelayNip01Compliant,
+            "MUST accept repo announcements listing service in clone & relays tags",
         )
         .run(|| async {
             // Create TestContext for mode-aware fixture management
@@ -253,8 +254,8 @@ impl EventAcceptancePolicyTests {
     ) -> TestResult {
         TestResult::new(
             "reject_repo_announcement_missing_clone_tag",
-            "GRASP-01:nostr-relay:9",
-            "Reject repository announcements without service in clone tag",
+            SpecRef::NostrRelayRejectMissingCloneRelays,
+            "MUST reject announcements not listing service in clone tag",
         )
         .run(|| async {
             // Get relay URL from client
@@ -329,8 +330,8 @@ impl EventAcceptancePolicyTests {
     ) -> TestResult {
         TestResult::new(
             "reject_repo_announcement_missing_relays_tag",
-            "GRASP-01:nostr-relay:9",
-            "Reject repository announcements without service in relays tag",
+            SpecRef::NostrRelayRejectMissingCloneRelays,
+            "MUST reject announcements not listing service in relays tag",
         )
         .run(|| async {
             // Get relay URL from client
@@ -425,8 +426,8 @@ impl EventAcceptancePolicyTests {
     ) -> TestResult {
         TestResult::new(
             "accept_recursive_maintainer_announcement_without_service",
-            "GRASP-01:nostr-relay:9",
-            "Accept recursive maintainer announcement for chain discovery (even without GRASP server in clone)",
+            SpecRef::NostrRelayRejectMissingCloneRelays,
+            "MUST accept recursive maintainer announcements for chain discovery",
         )
         .run(|| async {
             // Create TestContext for mode-aware fixture management
@@ -593,7 +594,7 @@ impl EventAcceptancePolicyTests {
     pub async fn test_accept_issue_via_a_tag(client: &AuditClient) -> TestResult {
         TestResult::new(
             "accept_issue_via_a_tag",
-            "GRASP-01:nostr-relay:13",
+            SpecRef::NostrRelayMustAcceptTaggedEvents,
             "Accept issue referencing repo via 'a' tag",
         )
         .run(|| async {
@@ -628,7 +629,7 @@ impl EventAcceptancePolicyTests {
     pub async fn test_accept_comment_via_capital_a_tag(client: &AuditClient) -> TestResult {
         TestResult::new(
             "accept_comment_via_A_tag",
-            "GRASP-01:nostr-relay:13",
+            SpecRef::NostrRelayMustAcceptTaggedEvents,
             "Accept NIP-22 comment with root 'A' tag referencing repo",
         )
         .run(|| async {
@@ -681,8 +682,8 @@ impl EventAcceptancePolicyTests {
     pub async fn test_accept_kind1_via_q_tag(client: &AuditClient) -> TestResult {
         TestResult::new(
             "accept_kind1_via_q_tag",
-            "GRASP-01:nostr-relay:13",
-            "Accept kind 1 note quoting repo via 'q' tag",
+            SpecRef::NostrRelayMustAcceptTaggedEvents,
+            "Accept kind 1 text note quoting repo via 'q' tag",
         )
         .run(|| async {
             // Create TestContext
@@ -731,8 +732,8 @@ impl EventAcceptancePolicyTests {
     pub async fn test_accept_issue_quoting_issue_via_q(client: &AuditClient) -> TestResult {
         TestResult::new(
             "accept_issue_quoting_issue_via_q",
-            "GRASP-01:nostr-relay:13",
-            "Accept issue quoting accepted issue (transitive)",
+            SpecRef::NostrRelayMustAcceptTaggedEvents,
+            "Accept issue quoting another accepted issue (transitive)",
         )
         .run(|| async {
             // Create TestContext
@@ -777,7 +778,7 @@ impl EventAcceptancePolicyTests {
     pub async fn test_accept_comment_via_capital_e_tag(client: &AuditClient) -> TestResult {
         TestResult::new(
             "accept_comment_via_E_tag",
-            "GRASP-01:nostr-relay:13",
+            SpecRef::NostrRelayMustAcceptTaggedEvents,
             "Accept NIP-22 comment with root 'E' tag to accepted issue",
         )
         .run(|| async {
@@ -816,7 +817,7 @@ impl EventAcceptancePolicyTests {
     pub async fn test_accept_kind1_via_e_tag(client: &AuditClient) -> TestResult {
         TestResult::new(
             "accept_kind1_via_e_tag",
-            "GRASP-01:nostr-relay:13",
+            SpecRef::NostrRelayMustAcceptTaggedEvents,
             "Accept kind 1 reply via 'e' tag to accepted kind 1",
         )
         .run(|| async {
@@ -872,7 +873,7 @@ impl EventAcceptancePolicyTests {
     pub async fn test_accept_kind1_referenced_in_issue(client: &AuditClient) -> TestResult {
         TestResult::new(
             "accept_kind1_referenced_in_issue",
-            "GRASP-01:nostr-relay:13",
+            SpecRef::NostrRelayMustAcceptTaggedEvents,
             "Accept kind 1 referenced in accepted issue (forward ref)",
         )
         .run(|| async {
@@ -964,7 +965,7 @@ impl EventAcceptancePolicyTests {
     pub async fn test_accept_comment_referenced_in_comment(client: &AuditClient) -> TestResult {
         TestResult::new(
             "accept_comment_referenced_in_comment",
-            "GRASP-01:nostr-relay:13",
+            SpecRef::NostrRelayMustAcceptTaggedEvents,
             "Accept comment referenced in another accepted comment (forward ref)",
         )
         .run(|| async {
@@ -1025,7 +1026,7 @@ impl EventAcceptancePolicyTests {
     pub async fn test_accept_kind1_referenced_in_kind1(client: &AuditClient) -> TestResult {
         TestResult::new(
             "accept_kind1_referenced_in_kind1",
-            "GRASP-01:nostr-relay:13",
+            SpecRef::NostrRelayMustAcceptTaggedEvents,
             "Accept kind 1 referenced in another accepted kind 1 (forward ref)",
         )
         .run(|| async {
@@ -1083,7 +1084,7 @@ impl EventAcceptancePolicyTests {
     pub async fn test_reject_orphan_issue(client: &AuditClient) -> TestResult {
         TestResult::new(
             "reject_orphan_issue",
-            "GRASP-01:nostr-relay:18",
+            SpecRef::NostrRelayMayRejectSpamCuration,
             "Reject issue referencing unaccepted repo",
         )
         .run(|| async {
@@ -1110,7 +1111,7 @@ impl EventAcceptancePolicyTests {
     pub async fn test_reject_orphan_kind1(client: &AuditClient) -> TestResult {
         TestResult::new(
             "reject_orphan_kind1",
-            "GRASP-01:nostr-relay:18",
+            SpecRef::NostrRelayMayRejectSpamCuration,
             "Reject kind 1 with no repo references",
         )
         .run(|| async {
@@ -1139,7 +1140,7 @@ impl EventAcceptancePolicyTests {
     pub async fn test_reject_comment_quoting_other_repo(client: &AuditClient) -> TestResult {
         TestResult::new(
             "reject_comment_quoting_other_repo",
-            "GRASP-01:nostr-relay:18",
+            SpecRef::NostrRelayMayRejectSpamCuration,
             "Reject comment quoting unaccepted repo",
         )
         .run(|| async {