feat(barretenberg): more debugging tweaks

Author: ludamad

include/msgpack/v1/adaptor/detail/cpp11_define_map.hpp

@@ -39,7 +39,7 @@ struct define_map_imp {
             convert_helper(*it->second, std::get<N-1>(t));
         } else {
             // <barretenberg>
-            throw msgpack::unpack_error(std::string("Missing field " + std::get<N-2>(t)));
+            throw msgpack::unpack_error(std::string("Missing field ") + std::get<N-2>(t));
             // </barretenberg>
         }
     }

include/msgpack/v1/object.hpp

@@ -466,8 +466,14 @@ struct object_stringize_visitor {
         m_os << '"';
         return true;
     }
-    bool visit_bin(const char* /*v*/, uint32_t size) {
-        m_os << "\"BIN(size:" << size << ")\"";
+    bool visit_bin(const char* v, uint32_t size) {
+        // <barretenberg> print binary strings in hex
+        m_os << "\"BIN(size:" << size << "): 0x";
+        for (uint32_t i = 0; i < size; ++i) {
+            m_os << std::hex << std::setw(2) << std::setfill('0') << static_cast<int>(static_cast<unsigned char>(v[i]));
+        }
+        m_os << "\"";
+        // </barrenteberg>
         return true;
     }
     bool visit_ext(const char* v, uint32_t size) {
@@ -633,6 +639,8 @@ namespace detail {
 template <typename Stream, typename T>
 struct packer_serializer {
     static msgpack::packer<Stream>& pack(msgpack::packer<Stream>& o, const T& v) {
+        // <barretenberg>Note, if this is failing, the associated type probably wants
+        // an auto msgpack(){ar(NVP(...fields...));} method</barretenberg>
         v.msgpack_pack(o);
         return o;
     }