The KeyNotFound and NoSuchKey code names may cause confusion to readers. Apparently, KeyNotFound refers to indexes, while NoSuchKey is used to indicate a field is not found within a BSON object. Making the names more explicit would help readability and make incorrect use less likely.

There is at least one location (storage_interface_impl.cpp) where NoSuchKey is used to indicate no documents were found.