Summary

Related issue: #____

Responds to code review Finding 1.2 by removing Lombok's@SneakyThrows in favor of explicit exception handling across the event and API layers, so that JSON/URL failures surface as meaningful domain errors.

What changed?

• Wrapped Jackson parsing in marketplace, merchant, and identity-related events withEventEncodingException so invalid content fails fast without hidden checked exceptions.
• Updated API builders (NIP-05/25/57/60/61) andBaseTagFactory to convert checked exceptions into descriptiveIllegalArgumentException/EventEncodingException instances instead of relying on@SneakyThrows.
• Revised affected integration/unit tests to document intent, remove@SneakyThrows, and assert JSON parsing success explicitly.

BREAKING

None.

Review focus

• Confirm the new exception messages and types provide adequate signal for callers consuming the updated APIs.

Checklist

• Scope ≤ 300 lines (or split/stack)
• Title isverb + object (e.g., “Refactor auth middleware to async”)
• Description links the issue and answers “why now?”
• BREAKING flagged if needed
• Tests/docs updated (if relevant)

Testing

• mvn -q verify(fails: xyz.tcheeric:nostr-java-bom:1.1.1 missing from Maven Central)

Network Access

• Attempted to downloadhttps://repo.maven.apache.org/maven2/xyz/tcheeric/nostr-java-bom/1.1.1/nostr-java-bom-1.1.1.pom (artifact not found).

https://chatgpt.com/codex/tasks/task_b_68e337e2acfc83319e62250f0db94bc8