CBOR wrappers in cose::edit and historical wrappers

[maxtropets]

Nit. Changed to shared_ptr to make it easy to read one CBOR and produce another from the same data. There was no particular reason for Value to be unique_ptr, I defaulted to it because we needed a pointer to keep a tree of Values.

[Copilot]

Pull request overview

This pull request refactors COSE encoding/editing and historical query code to use new CBOR wrapper functions instead of direct QCBOR library calls. The change simplifies the code by introducing higher-level abstraction functions for building CBOR values.

Changes:

• Added new CBOR helper functions (make_simple, make_tagged, make_array, make_map, boolean_to_simple) to simplify CBOR value construction
• Changed Value type from unique_ptr to shared_ptr to enable easier value sharing
• Refactored set_unprotected_header in cose.cpp to use new CBOR wrappers instead of raw QCBOR calls
• Refactored Merkle proof encoding in historical_queries_adapter.cpp to use new CBOR wrappers
• Updated tests to use new CBOR parsing and comparison approach

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/crypto/cbor.h	Added declarations for new helper functions and changed Value from unique_ptr to shared_ptr
src/crypto/cbor.cpp	Implemented new helper functions and updated all Value allocations to use make_shared
src/crypto/cose.cpp	Refactored COSE editing to use new CBOR wrappers instead of direct QCBOR calls
src/node/historical_queries_adapter.cpp	Refactored Merkle proof encoding to use new CBOR wrappers
src/crypto/test/cose.cpp	Updated tests to use CBOR parsing and string comparison instead of manual QCBOR decoding