Should this be merged with InboundKeyInfoResponse in keyserver/addons/rust-node-addon/rust-bindings-types.js (and put in lib-types-identity-service-types)?

Let's keep the naming consistent (please also change the name of the _helper function)

We should probably add a comment like for OutboundKeyInfoResponse to keep types in sync.

It would be good to merge this with InboundKeyInfoResponse from keyserver/addons/rust-node-addon/src/identity_client/mod.rs, but it might be harder than merging the JS equivalents.
The keyserver struct has #[napi(object)] attribute which (I assume, cc @varun) would have to be applied in the same crate that this struct is defined in. This would probably mean moving it to comm-lib and adding a feature for napi and applying this attribute conditionally (e.g. #[cfg_attr(feature = "napi", napi(object))]). Not sure if it's worth it.

as @tomek pointed out in D10449, we aren't actually doing anything with error here (or any of the other methods in this file, for that matter).

I think we need something like this after the try-block:

if (!error.empty()) {
  this->jsInvoker_->invokeAsync([error, promise]() {
    promise->reject(error);
  });
}