FuelLabs / fuel-vm

Fuel v2 interpreter in Rust
Other
358 stars 88 forks source link

GTF support for new tx types #812

Open Dentosal opened 2 months ago

Dentosal commented 2 months ago

From Slack:

I don't see a GTF opcode defined to get the witnessesCount, witnesses, outputs, outputsCount, inputs, and inputsCount for TransactionUpgrade, TransactionUpload, and TransactionBlob in the specs as well as transaction specific types such as subsectionIndex, subsectionsNumber, root, witnessIndex, proofSetCount, and proofSet in TransactionUpload.

This is blocking https://github.com/FuelLabs/sway/pull/6500

xgreenx commented 2 months ago

Also, it seems that we have a different getter for each type of transaction. I think we just need to have one getter for all types. And if the transaction doesn't support it, return an error like in the case of Mint transaction.

image image image image