Remove GetCBOR wrapper from QueryLedgerPeerSnapshot

[Jimbo4350]

Changelog

- description: |
    Remove GetCBOR wrapper from QueryLedgerPeerSnapshot query. Return
    LedgerPeerSnapshot directly instead of Serialised LedgerPeerSnapshot.
# uncomment types applicable to the change:
  type:
   - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  # - refactoring    # QoL changes
  # - 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...
# uncomment at least one main project this PR is associated with
  projects:
   - cardano-api
  # - cardano-api-gen
  # - cardano-rpc
  # - cardano-wasm

Context

There is no reason to keep the LedgerPeerSnapshot result serialized — callers always need the deserialized value. This removes the GetCBOR wrapper so the query returns LedgerPeerSnapshot directly instead of Serialised LedgerPeerSnapshot.

Replaces #930.

How to trust this PR

2-file, 5-line change. The GetCBOR wrapper is removed from:

• The QueryLedgerPeerSnapshot result type
• The consensus query construction in toConsensusQueryShelleyBased
• The result pattern matches in fromConsensusQueryResultShelleyBased
• The queryLedgerPeerSnapshot type signature in Expr.hs

Builds cleanly (lib + gen + both test suites).

Checklist

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

[Copilot]

Pull request overview

Removes the GetCBOR wrapper from the QueryLedgerPeerSnapshot query so callers receive a LedgerPeerSnapshot directly (instead of a Serialised LedgerPeerSnapshot), reflecting the fact that consumers always need the deserialized value.

Changes:

• Adjust QueryLedgerPeerSnapshot result type to return LedgerPeerSnapshot directly.
• Update consensus query construction to remove Consensus.GetCBOR.
• Update query result pattern matches and the queryLedgerPeerSnapshot type signature accordingly.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
cardano-api/src/Cardano/Api/Query/Internal/Type/QueryInMode.hs	Removes GetCBOR from the query type/result mapping and consensus-query construction/result handling.
cardano-api/src/Cardano/Api/Query/Internal/Expr.hs	Updates the public query helper’s type signature to return LedgerPeerSnapshot directly.

[palas]

Looks good. One thought: I am not sure of how users use this, but it may make sense to do one of two things:

• Either deprecating/removing decodeLedgerPeerSnapshot
• Or adding an encodeLedgerPeerSnapshot function