chore: improve logging format; set workflow output level to critical

Signed-off-by: Will Killian <[email protected]>

Author: willkill07

src/nat/cli/entrypoint.py

@@ -52,7 +52,11 @@
 def setup_logging(log_level: str):
     """Configure logging with the specified level"""
     numeric_level = LOG_LEVELS.get(log_level.upper(), logging.INFO)
-    logging.basicConfig(level=numeric_level, format='%(asctime)s - %(name)s - %(levelname)s - %(message)s')
+    logging.basicConfig(
+        level=numeric_level,
+        format="%(asctime)s - %(levelname)-8s - %(name)s:%(lineno)d - %(message)s",
+        datefmt="%Y-%m-%d %H:%M:%S",
+    )
     return numeric_level
 
 

src/nat/front_ends/console/console_front_end_plugin.py

@@ -95,16 +95,6 @@ async def run_single_query(query):
         else:
             assert False, "Should not reach here. Should have been caught by pre_run"
 
-        # Always log the result at INFO level
+        # Always log the result at CRITICAL level so it is always visible
         result_message = f"\n{'-' * 50}\n{Fore.GREEN}Workflow Result:\n%s{Fore.RESET}\n{'-' * 50}"
-        logger.info(result_message, runner_outputs)
-
-        # Additionally print to stdout if console handlers are set to a level higher than INFO
-        # This ensures workflow results are always visible regardless of logging configuration
-        root_logger = logging.getLogger()
-        console_level_too_high = all(
-            type(handler) is not logging.StreamHandler or handler.level > logging.INFO
-            for handler in root_logger.handlers)
-
-        if console_level_too_high:
-            print(f"\n{'-' * 50}\n{Fore.GREEN}Workflow Result:\n{runner_outputs}{Fore.RESET}\n{'-' * 50}")
+        logger.critical(result_message, runner_outputs)

src/nat/observability/register.py

@@ -77,6 +77,14 @@ async def console_logging_method(config: ConsoleLoggingMethodConfig, builder: Bu
     level = getattr(logging, config.level.upper(), logging.INFO)
     handler = logging.StreamHandler(stream=sys.stdout)
     handler.setLevel(level)
+
+    # Set formatter to match the default CLI format
+    formatter = logging.Formatter(
+        fmt="%(asctime)s - %(levelname)-8s - %(name)s:%(lineno)d - %(message)s",
+        datefmt="%Y-%m-%d %H:%M:%S",
+    )
+    handler.setFormatter(formatter)
+
     yield handler
 
 
@@ -95,4 +103,12 @@ async def file_logging_method(config: FileLoggingMethod, builder: Builder):
     level = getattr(logging, config.level.upper(), logging.INFO)
     handler = logging.FileHandler(filename=config.path, mode="a", encoding="utf-8")
     handler.setLevel(level)
+
+    # Set formatter to match the default CLI format
+    formatter = logging.Formatter(
+        fmt="%(asctime)s - %(levelname)-8s - %(name)s:%(lineno)d - %(message)s",
+        datefmt="%Y-%m-%d %H:%M:%S",
+    )
+    handler.setFormatter(formatter)
+
     yield handler