refactor/remove common::parsing

[0xaatif]

I don't think we should be writing a parsing library like this when nom or winnow exist

Originally posted by @0xaatif in 0xPolygonZero/zero-bin#90 (comment)

---

Imagine if 0xPolygonZero/zero-bin#90 looked like this:

use alloy_primitives::{FixedBytes, B256};
use winnow::{ascii::dec_uint, combinator::{alt, preceded, terminated}, error::ContextError, token::take, PResult, Parser};

pub enum BlockSlice {
    Just(BlockId),
    Range(BlockId, BlockId),
    From(BlockId),
}

pub enum BlockId {
    Hash(B256),
    Number(u64),
}

pub fn block_slice(input: &mut &str) -> PResult<BlockSlice> {
    alt((
        range.map(|(a, b)| BlockSlice::Range(a, b)),
        from.map(BlockSlice::From),
        block_id.map(BlockSlice::Just),
    ))
    .parse_next(input)
}

fn from(input: &mut &str) -> PResult<BlockId> {
    terminated(block_id, "..").parse_next(input)
}

fn range(input: &mut &str) -> PResult<(BlockId, BlockId)> {
    (block_id, preceded("..", block_id)).parse_next(input)
}

pub fn block_id(input: &mut &str) -> PResult<BlockId> {
    alt((b256.map(BlockId::Hash), dec_uint.map(BlockId::Number))).parse_next(input)
}

pub fn b256(input: &mut &str) -> PResult<B256> {
    preceded("0x", fixed_bytes).parse_next(input)
}

fn fixed_bytes<const N: usize>(input: &mut &str) -> PResult<FixedBytes<N>> {
    take(N * 2).try_map(FixedBytes::from_str).parse_next(input)
}

Instead of this:
https://github.com/0xPolygonZero/zero-bin/blob/c090919cadba56a09f10dcaf975527426ed1ec78/common/src/block_interval.rs#L62-L101
https://github.com/0xPolygonZero/zero-bin/blob/c090919cadba56a09f10dcaf975527426ed1ec78/common/src/parsing.rs#L34-L90

[0xaatif]

I'll implement the above at some point, happy for someone to beat me to the punch with Co-Authored-By

[0xaatif]

#728 removed most of this code :)