[TECH DEBT] - BlockPayload trait delete Metadata args and return vals

[ggutoski]

What is this task and why do we need to work on it?

The following methods all take an unneeded arg of type Self::Metadata:
from_bytes, builder_commitment, transactions.

The following methods all return an unneeded val of type Self::Metadata:
from_transactions, empty

Sometimes we actually clone the namespace table just to pass args or return a val that is never used!

What work will need to be done to complete this task?

No response

Are there any other details to include?

No response

What are the acceptance criteria to close this issue?

reviews from code owners

Branch work will be merged to (if not the default branch)

No response