[BREAKING CHANGE] refactor: remove the tx parameter from the sign method in the Signer trait

[EthanYuan]

The sign method should be independent of CKB tx and should only be concerned with the signing message. Otherwise, it could lead to confusion.

pub trait Signer {
    /// `message` type is variable length, because different algorithm have
    /// different length of message:
    ///   * secp256k1 => 256bits
    ///   * RSA       => 512bits (when key size is 1024bits)
    fn sign(
        &self,
        id: &[u8],
        message: &[u8],
        recoverable: bool,
        tx: &TransactionView,
    ) -> Result<Bytes, SignerError>;
}

remove the tx parameter:

pub trait Signer {
    /// `message` type is variable length, because different algorithm have
    /// different length of message:
    ///   * secp256k1 => 256bits
    ///   * RSA       => 512bits (when key size is 1024bits)
    fn sign(&self, id: &[u8], message: &[u8], recoverable: bool) -> Result<Bytes, SignerError>;
}