Fix Mapping, Remove Dummy UTXOs, Support Edge Cases (current)

[p0stcapone]

Addressing: #2

Refactor of selectUtxos() and generatePsbtBuyingInscription()

-Inscriptions will now always be at satIndex 0 of the taker's receiving output
-Dummy UTXOs to assert index match requirements of sighashsingle are now unecessary

With the new PSBT construction, the inscribed output may in many cases be the last output of the transaction. In any cases where the fees for the PSBT would result in the inscribed output being less than 2k sats in value, the taker will be required to provide additional inputs to cover these fees. If the additional inputs provided are also sufficient to add padding back to the inscribed output, they will be used (up to 10k value). -Note: The taker still owns these sats, they're just preserving the padding.

This is an updated pull request reflecting the current features for openordex

I'm uncertain why the diff generated is being stubborn.

All changes refer to

selectUtxos()
generatePsbtBuyingInscription()
updatePayerAddress()

Changes can be found both in the function definitions and as they are called.

[rot13maxi]

this is showing up in GH as +1,022 −1,094. Needs a rebase maybe?

[rot13maxi]

Can you add an explanation of the proposed construction to the README? Would be great to be able to look at/talk through the PSBT construction and then compare intent to implementation.

[rot13maxi]

tried to fix the weird diff'ing issues. PR against your fork here:
p0stcapone#1

diff against upstream here:
#30

[p0stcapone]

Closing in favor of: #30

Reason: Rebase more clearly demonstrates the diff.