https://github.com/pantsbuild/pants/issues/6898#issuecomment-446296831 describes

bazel_protos::remote_execution::Action

for entries in the

Store

, and

bazel_protos::remote_execution:Directory

and friends are what are already used there. i'm not inserting any of that anywhere else, just a compatibility layer in order to avoid extracting a

FallibleExecuteProcessResult

in two places from two different types of

ActionResult

. unless you're suggesting we use the tower `Action`/`ActionResult` as k/v in the

lmdb::Database

in the

Store

, which is great and will remove some (only a little) code, just that's not what's in the comment