Remove usage of stake pool keys as credentials

[carbolymer]

Changelog

- description: |
    Removes usage of stake pool keys as credentials
# uncomment types applicable to the change:
  type:
  # - feature        # introduces a new feature
   - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
   - improvement    # QoL changes e.g. refactoring
  # - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context

This PR removes usage of stake pool keys as credentials. This also removes coercing of the keys types in the code.

The context for the change: https://input-output-rnd.slack.com/archives/G011N23CEAE/p1698227871191609

Requires:

• Remove uses of coerceKeyRole, use asWitness / fromWitness when key role conversion is required cardano-api#341

This PR requres IntersectMBO/cardano-api#341 to compile. To be merged after merging of the required PR in API.

How to trust this PR

Highlight important bits of the PR that will make the review faster. If there are commands the reviewer can run to observe the new behavior, describe them.

Checklist

• Commit sequence broadly makes sense and commits have useful messages
• New tests are added if needed and existing tests are updated. See Running tests for more details
• Self-reviewed the diff

[Jimbo4350]

Ignore this

[carbolymer]

Superceded by: #430