feat: remove debug mode

Author: Gusarich

src/cli.py

@@ -360,7 +360,6 @@ def create_parser(cls) -> argparse.ArgumentParser:
 
         # Diagnostics group
         g_diag = parser.add_argument_group("Diagnostics")
-        g_diag.add_argument("--debug", action="store_true", help="Enable debug logging")
         g_diag.add_argument(
             "--yes", action="store_true", help="Assume 'yes' for prompts"
         )
@@ -1034,8 +1033,7 @@ async def run_list_runs(self, args: argparse.Namespace) -> int:
 
         except (OSError, json.JSONDecodeError) as e:
             self.console.print(f"[red]Error listing runs: {e}[/red]")
-            if args.debug:
-                self.console.print_exception()
+            self.console.print_exception()
             return 1
 
     async def run_show_run(self, args: argparse.Namespace) -> int:
@@ -1140,8 +1138,7 @@ async def run_show_run(self, args: argparse.Namespace) -> int:
 
         except (OSError, json.JSONDecodeError) as e:
             self.console.print(f"[red]Error showing run: {e}[/red]")
-            if args.debug:
-                self.console.print_exception()
+            self.console.print_exception()
             return 1
 
     async def run_prune(self, args: argparse.Namespace) -> int:
@@ -1238,8 +1235,7 @@ async def run_prune(self, args: argparse.Namespace) -> int:
             return 0
         except Exception as e:
             self.console.print(f"[red]Prune failed: {e}[/red]")
-            if args.debug:
-                self.console.print_exception()
+            self.console.print_exception()
             return 1
 
     def _display_detailed_results(
@@ -1481,7 +1477,7 @@ async def run_orchestrator(self, args: argparse.Namespace) -> int:
         setup_structured_logging(
             logs_dir=args.logs_dir,
             run_id=run_id,
-            debug=args.debug,
+            debug=True,
             quiet=args.no_tui and args.output == "quiet",
             no_tui=args.no_tui,
         )
@@ -1638,8 +1634,7 @@ async def run_orchestrator(self, args: argparse.Namespace) -> int:
         if args.output:
             cli_config["output"] = args.output
         # Server extension support removed
-        if args.debug:
-            cli_config.setdefault("logging", {})["level"] = "DEBUG"
+        # Always use full logging; no debug toggle
         # Add CLI auth args
         if hasattr(args, "oauth_token") and args.oauth_token:
             cli_config.setdefault("runner", {})["oauth_token"] = args.oauth_token
@@ -1829,17 +1824,15 @@ def _red(k, v):
             "logs_dir"
         ) or full_config.get("logs_dir", Path("./logs"))
 
-        # Log configuration sources (only in debug mode)
-        if full_config.get("debug") or args.debug:
-            self.console.print("[dim]Configuration loaded from:[/dim]")
-            if cli_config:
-                self.console.print("  - Command line arguments")
-            # No Pitaya-specific environment variables are used
-            if dotenv_config:
-                self.console.print("  - .env file")
-            if config:
-                self.console.print(f"  - Config file: {args.config or 'pitaya.yaml'}")
-            self.console.print("  - Built-in defaults")
+        # Log configuration sources unconditionally
+        self.console.print("[dim]Configuration loaded from:[/dim]")
+        if cli_config:
+            self.console.print("  - Command line arguments")
+        if dotenv_config:
+            self.console.print("  - .env file")
+        if config:
+            self.console.print(f"  - Config file: {args.config or 'pitaya.yaml'}")
+        self.console.print("  - Built-in defaults")
 
         # Respect global session volume consent by setting env for runner
         allow_global_session = bool(getattr(args, "allow_global_session_volume", False))
@@ -2218,8 +2211,7 @@ def emit_json(ev: dict):
             return 2
         except (OrchestratorError, DockerError, ValidationError) as e:
             self.console.print(f"[red]Error: {e}[/red]")
-            if args.debug:
-                self.console.print_exception()
+            self.console.print_exception()
             # Shutdown orchestrator
             await self.orchestrator.shutdown()
             return 1
@@ -2650,8 +2642,7 @@ async def monitor_shutdown():
             return 2
         except (OrchestratorError, DockerError, ValidationError) as e:
             self.console.print(f"[red]Error: {e}[/red]")
-            if args.debug:
-                self.console.print_exception()
+            self.console.print_exception()
             # Shutdown orchestrator
             await self.orchestrator.shutdown()
             return 1

src/config.py

@@ -245,7 +245,7 @@ def get_default_config() -> Dict[str, Any]:
         "output": "tui",
         "state_dir": Path("./pitaya_state"),
         "logs_dir": Path("./logs"),
-        "debug": False,
+        # Debug mode removed; logs are verbose by default
         # Normative defaults per spec §6.1.1 (subset applied where supported)
         "import_policy": "auto",  # auto|never|always
         "import_conflict_policy": "fail",  # fail|overwrite|suffix

src/tui/cli.py

@@ -92,9 +92,7 @@ def create_parser(cls) -> argparse.ArgumentParser:
             "--event-types", nargs="+", help="Filter by event type(s)"
         )
 
-        # Diagnostics
-        g_diag = parser.add_argument_group("Diagnostics")
-        g_diag.add_argument("--debug", action="store_true", help="Enable debug logging")
+        # Diagnostics (debug mode removed; verbose by default)
 
         return parser
 
@@ -137,8 +135,7 @@ async def run(self, args: argparse.Namespace) -> int:
             return 2
         except (OSError, IOError) as e:
             self.console.print(f"[red]Error: {e}[/red]")
-            if args.debug:
-                self.console.print_exception()
+            self.console.print_exception()
             return 1
 
     # Connected modes removed