• const auto & where appropriate
• fix payouts.reserve() sizing

In D16695#378856, @Fabien wrote:

This memory is important so that nodes can reply to stake contender avalanche requests of recent past blocks (we cannot guarantee the entire network has finalized the same block in lock step).

Is it really ? If a block has finalized for some nodes but not others, what is the point of voting on past block rewards ? If the node votes idk the block will eventually be finalized by the poller anyway so the outcome of this stake vote doesn't matter. The only case I think it is helpful is if you get a bunch of blocks in a row, faster than finalization time. But as soon as a block is final you can forget about winners for it and past blocks.

• DEFAULT -> UNKNOWN
• fix score type

use named arg per feedback

Add hash.h

• cleanups
• strongly type StakeContenderId
• switch to blockhash + proofid instead of blockhash + payoutscript since this could be more versatile down the road
• streamlined implementation a bit, which eliminated stakecontender.cpp

Fix lock annotations

Throw an error instead of silently deduping

Not blocking on this feedback but the keys should ideally be the same and the help text should match

Cargo.lock file was already updated in this patch

One missing case in rpc_packages.py:47

Fixup generated file paths