Beef up config validator

It was performing relatively unsophisticated checks. Make it so that it
can catch more subtle issues with timestamp channel configurations.

Change-Id: Ie42fba108c6bcb0d1cad2484dc435bac10a003ba
Signed-off-by: James Kuszmaul <jabukuszmaul+collab@gmail.com>
diff --git a/aos/util/simulation_logger.cc b/aos/util/simulation_logger.cc
index e459978..1f55ece 100644
--- a/aos/util/simulation_logger.cc
+++ b/aos/util/simulation_logger.cc
@@ -1,11 +1,12 @@
 #include "aos/util/simulation_logger.h"
+#include "aos/events/logging/logfile_utils.h"
 
 namespace aos::util {
 LoggerState::LoggerState(aos::SimulatedEventLoopFactory *factory,
                          const aos::Node *node, std::string_view output_folder)
     : event_loop_(factory->MakeEventLoop("logger", node)),
       namer_(std::make_unique<aos::logger::MultiNodeFilesLogNamer>(
-          absl::StrCat(output_folder, "/", node->name()->string_view(), "/"),
+          absl::StrCat(output_folder, "/", logger::MaybeNodeName(node), "/"),
           event_loop_.get())),
       logger_(std::make_unique<aos::logger::Logger>(event_loop_.get())) {
   event_loop_->SkipTimingReport();