fix(rpc/trace): wrong calculate of block ommer rewards

[jsvisa]

In the current implementation of trace_block rpc, we return block base reward as block reward, and the uncle inclusion rewards as uncle reward.

But in OpenEthereum/Erigon's implementation, they return block base + uncle inclusion as block reward, and each uncle reward as uncle reward, so the actual uncle rewards were missed.

IMHO, we should keep the same as oe/erigon, because the results will be more inaccurate, similar to etherscan's.

Eg the reward of https://etherscan.io/block/1378035

Reth's response:

{
    "jsonrpc": "2.0",
    "result": [
        {
            "action": {
                "author": "0xea674fdde714fd979de3edf0f56aa9716b898ec8",
                "rewardType": "block",
                "value": "0x4563918244f40000" // 5ETH
            },
            "blockHash": "0xd6940190d24aa1c2e8aa70fb2847aba6c4461679753a7546daf79e6295a9e1e2",
            "blockNumber": 1378035,
            "subtraces": 0,
            "traceAddress": [],
            "type": "reward"
        },
        {
            "action": {
                "author": "0xea674fdde714fd979de3edf0f56aa9716b898ec8",
                "rewardType": "uncle",
                "value": "0x22b1c8c1227a000" // 0.15625ETH
            },
            "blockHash": "0xd6940190d24aa1c2e8aa70fb2847aba6c4461679753a7546daf79e6295a9e1e2",
            "blockNumber": 1378035,
            "subtraces": 0,
            "traceAddress": [],
            "type": "reward"
        }
    ],
    "id": "reth"
}

Erigon's response

{
    "jsonrpc": "2.0",
    "id": "erigon",
    "result": [
        {
            "action": {
                "author": "0xea674fdde714fd979de3edf0f56aa9716b898ec8",
                "rewardType": "block",
                "value": "0x478eae0e571ba000" // 5.15625ETH
            },
            "blockHash": "0xd6940190d24aa1c2e8aa70fb2847aba6c4461679753a7546daf79e6295a9e1e2",
            "blockNumber": 1378035,
            "result": null,
            "subtraces": 0,
            "traceAddress": [],
            "type": "reward"
        },
        {
            "action": {
                "author": "0xea674fdde714fd979de3edf0f56aa9716b898ec8",
                "rewardType": "uncle",
                "value": "0x3cb71f51fc558000" // 4.375ETH
            },
            "blockHash": "0xd6940190d24aa1c2e8aa70fb2847aba6c4461679753a7546daf79e6295a9e1e2",
            "blockNumber": 1378035,
            "result": null,
            "subtraces": 0,
            "traceAddress": [],
            "type": "reward"
        }
    ]
}

Ref https://github.com/ledgerwatch/erigon/blob/b4c9f22923cd7156cd26be91a8a83b131eb3cc73/consensus/ethash/consensus.go#L624-L668

[mattsse]

makes sense