Fix NWC support detection

[ekzyis]

It worked with Mutiny because Mutiny only supports pay_invoice so the resulting array only contained a single element with value pay_invoice.

With wallets that supported multiple methods, the single element no longer matched pay_invoice.

2024-02-17.05-30-13.mp4

I think that's what every stacker encountered with red indicators but no error message since there is indeed no toast if pay_invoice support was not detected.

So I believe this fixes #821. I think the errors in the console are unrelated to NWC and that they are just "CSP WTF reports".

[sudocarlos]

Woot. Lfg

[benthecarman]

This did not fix for me when using https://github.com/benthecarman/nostr-wallet-connect-lnd

It seems you are separating by comma, but the nip defines it as space separated

[sudocarlos]

So Alby implemented it wrong. Could just parse with split(' ') after parsing with split(,) and then both would work, right? Go easy on me - am a noob.

[benthecarman]

Have a fix for alby: getAlby/nostr-wallet-connect#247

[ekzyis]

Could just parse with split(' ') after parsing with split(,) and then both would work, right?

Yes that's kind of what ben did in #855.

But you can't do it exactly like you described because split returns an array and you can't split an array:

> "foo,bar".split(",")
[ 'foo', 'bar' ]
> "foo,bar".split(",").split(" ")
Uncaught TypeError: "foo,bar".split(...).split is not a function