String data that gets sent via MsgPack should be declared as string, …

…too.

src/nvim/msgpack_rpc/helpers.c

@@ -224,8 +224,8 @@ void msgpack_rpc_from_float(Float result, msgpack_packer *res)
 void msgpack_rpc_from_string(String result, msgpack_packer *res)
   FUNC_ATTR_NONNULL_ARG(2)
 {
-  msgpack_pack_bin(res, result.size);
-  msgpack_pack_bin_body(res, result.data, result.size);
+  msgpack_pack_str(res, result.size);
+  msgpack_pack_str_body(res, result.data, result.size);
 }
 
 void msgpack_rpc_from_object(Object result, msgpack_packer *res)
@@ -305,8 +305,8 @@ void msgpack_rpc_error(char *msg, msgpack_packer *res)
   size_t len = strlen(msg);
 
   // error message
-  msgpack_pack_bin(res, len);
-  msgpack_pack_bin_body(res, msg, len);
+  msgpack_pack_str(res, len);
+  msgpack_pack_str_body(res, msg, len);
   // Nil result
   msgpack_pack_nil(res);
 }