Summary

Related issue: #____

What changed?

• Refactored WebSocketClientHandler subscription setup/teardown into focused helpers to shrink the method and centralize cleanup.
• Introduced ZapRequestParameters and routed NIP-57 APIs/builders through it to eliminate seven-argument overloads.
• Extended the NIP-57 unit suite to cover the new parameter object while documenting test intent.

BREAKING

None.

Review focus

• Confirm the new subscription handle still closes relay resources and surfaces failures reliably.
• Ensure ZapRequestParameters preserves existing zap request scenarios.

Checklist

• Scope ≤ 300 lines (or split/stack)
• Title is verb + 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

• Blocked: https://repo.maven.apache.org/maven2/xyz/tcheeric/nostr-java-bom/1.1.1/

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