Multiple definitions of `response_handler_args_t` in src/or/directory.h are breaking clang builds

Travis caught a warning from clang that we defined multiple response_handler_args_t in src/or/directory.h. This appears to stem from commit 6eb9de1b8c from #17242 (moved), where we made the type public, but never removed the old typedef.