Thank you for the sats, and you’re welcome for the review! Merry Christmas!

thanks again for your review, I'll be implementing some changes based off of your feedback over the holidays, and I've just boosted you 60k sats on this comment. many thanks and Happy Christmas!

jowo

great thanks for your feedback. I'll take a deeper look into the points you've made in the next few days and get back to you!

You've made a bunch of good point and I'll be making some amendments to the code based off the back of your feedback.

I'll post back here once I've done so and give a little more context on the changes and on your questions/points :)

status: 410, // return a 410 status so the client knows the session no longer exists

TIL, I don't think I've ever seen HTTP 410 used anywhere. 

const lnurlEncode = // @ts-ignore
    (await import("lnurl/lib/encode.js")).default;
  const encoded = lnurlEncode(callbackUrl.toString()).toUpperCase();

This is an interesting approach to importing. Is it just for smaller bundles? Isn't this code executed server-side? 

          copy.innerHTML = session.lnurl;

This seems unnecessarily risky. Any reason not to use 

bitcoin

Reviewers and testers wanted for OpenSats funded OSS Lightning project!

1. 
```
status: 410, // return a 410 status so the client knows the session no longer exists
```
TIL, I don't think I've ever seen HTTP 410 used anywhere. [Source](https://github.com/jowo-io/next-auth-lightning-provider/blob/ae14061115838c7bcde76836a4affb64ce4cc0d0/src/main/handlers/poll.ts#L34)

2. 
```
const lnurlEncode = // @ts-ignore
    (await import("lnurl/lib/encode.js")).default;
  const encoded = lnurlEncode(callbackUrl.toString()).toUpperCase();
```
This is an interesting approach to importing. Is it just for smaller bundles? Isn't this code executed server-side? [Source](https://github.com/jowo-io/next-auth-lightning-provider/blob/ae14061115838c7bcde76836a4affb64ce4cc0d0/src/main/handlers/create.ts#L43-L45)

3. 
```
          copy.innerHTML = session.lnurl;
```
This seems unnecessarily risky. Any reason not to use `textContent`? [Source](https://github.com/jowo-io/next-auth-lightning-provider/blob/ae14061115838c7bcde76836a4affb64ce4cc0d0/src/main/utils/vanilla.ts#L137)

4. Just a general comment - for being a TS project, I see a lot of `typeof` checks, `any`, type-casting, etc. I know you said no nit-picking, but I guess I can't help myself.