Wrong tweaking in the new musig2 crate

[ceyhunsen]

Proposal Description

Secp256k1's musig implementation's tweaking methods doesn't act the same with the previous version. It might be because there are differences inside the library or wrong usage of the library.

WIP PR: #405

Looks like the new module sorts public keys regardless. See: rust-bitcoin/rust-secp256k1#716 (comment)

[ozankaymak]

#[derive(Debug, Clone)]
pub enum MuSig2Mode {
  OnlyKeySpend(XOnlyPublicKey),
  OnlyScriptSpend(TapNodeHash),
  BothKeyAndScriptSpend(XOnlyPublicKey, TapNodeHash),
  None,
}

It makes sense to use something like this now since we have new use cases.

[ceyhunsen]

#[derive(Debug, Clone)]
pub enum MuSig2Mode {
  OnlyKeySpend(XOnlyPublicKey),
  OnlyScriptSpend(TapNodeHash),
  BothKeyAndScriptSpend(XOnlyPublicKey, TapNodeHash),
  None,
}

It makes sense to use something like this now since we have new use cases.

Added to the PR.

[ozankaymak]

Closing this because #405 is merged. Check #427 .