bark, testing: explicitly reject arkade addresses

AcidBunny21requested to merge
invalid-ark-address-handling into master
  • check or remove this line after you've added a CHANGELOG entry or if your PR doesn't need one (they should go into CHANGELOG/unreleased/)

Summary by CodeRabbit

  • Bug Fixes
    • Sending to unsupported Arkade addresses now returns a clear “Arkade addresses are not supported” error (CLI and wallet send endpoint), instead of being treated as an unknown destination.
    • Payment-request parsing now surfaces Arkade as an explicit unsupported-address condition while keeping other destination formats working as before, and improves bitcoin:/BIP-321 URI error specificity.
    • Ark-related payment parsing was refined to better preserve valid requests and route Arkade vs Bark consistently.
  • Tests
    • Added coverage to verify Arkade rejection when sending, and correct Arkade handling when parsing BIP-321-style payment requests.
Edited by coderabbit

Merge request reports