Merge bitcoindevkit/bdk#2083: fix: no `Debug` on `Display` implementations

823e4e980af03672cc6342d0e4850febcbd14c9d fix(file_store)!: remove Debug usage and implement Display for StoreError. (Dmenec)
5a85f6e25a9e14204cb66fec3f7381a59133f0be fix(chain)!: remove Debug usage and implement Display for CalculateFeeError (Dmenec)
d0889f37ae1176eb1510a8d260cde8e05fe68f76 docs(core): add detailed SyncRequest output example (Dmenec)
5d715c2e4c8076be058e342efe279a3fc3481312 fix(chain)!: remove Debug usage and implement Display for InsertDescriptorError (Dmenec)

Pull request description:

  ### Description

  Fixes https://github.com/bitcoindevkit/bdk/issues/1933, remove the usage of `Debug` on `Display` implemetations mentioned in https://github.com/bitcoindevkit/bdk/pull/1881#issuecomment-2718412512

  ### Changelog notice

  - Replaced  `Debug` usage with `Display` messages for `InsertDescriptorError`, `CalculateFeeError` and `StoreError`.
  - `InvalidMagicBytes` now displays expected and actual bytes in hexadecimal instead of using `Debug`.
  - Added a new `short_descriptor` function to shorten descriptors for concise error output.
  - Added detailed `SyncRequest` `Display` for Esplora and Electrum examples.

  ### Checklists

  #### All Submissions:
  * [x] I have signed all my commits
  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
  * [x] I ran `just check`, `just fmt` and `just test` before committing

ACKs for top commit:
  evanlinjin:
    ACK 823e4e980af03672cc6342d0e4850febcbd14c9d

Tree-SHA512: fc3d66b72dc3f7ed3dee3b017687ece73c93883d23cbb019c402e55957368a6487e02b400b092169c326a47a86403f7aa865fe7d7238ccc6eb77afef258ec1ca