Simplified diagnostics class

Author: b3ncr0w

Runtime/Scripts/BetaHubDiagnostics.cs

@@ -11,184 +11,68 @@ public interface IContextualDiagnostics
         /// <summary>
         /// Logs an informational message for an operation.
         /// </summary>
-        /// <param name="operation">The operation name</param>
-        /// <param name="message">The message to log</param>
         void LogInfo(string operation, string message);
 
         /// <summary>
         /// Logs a successful operation completion.
         /// </summary>
-        /// <param name="operation">The operation name</param>
-        /// <param name="message">The success message</param>
         void LogSuccess(string operation, string message);
 
         /// <summary>
         /// Logs progress information for a long-running operation.
         /// </summary>
-        /// <param name="operation">The operation name</param>
-        /// <param name="message">The progress message</param>
         void LogProgress(string operation, string message);
 
         /// <summary>
-        /// Logs an error with automatic exception categorization and retry analysis.
+        /// Logs an error with a custom message and exception details.
         /// </summary>
-        /// <param name="operation">The operation name</param>
-        /// <param name="ex">The exception that occurred</param>
-        void LogError(string operation, Exception ex);
-
-        /// <summary>
-        /// Logs an error with explicit categorization and retry information.
-        /// </summary>
-        /// <param name="operation">The operation name</param>
-        /// <param name="message">The error message</param>
-        /// <param name="category">The error category</param>
-        /// <param name="isRetryable">Whether the error is retryable</param>
-        void LogError(string operation, string message, BetaHubErrorCategory category = BetaHubErrorCategory.Unknown, bool isRetryable = false);
+        void LogError(string operation, string message, Exception ex);
     }
 
     /// <summary>
-    /// Main entry point for BetaHub diagnostics. Provides contextual diagnostics services
-    /// for consistent error handling and logging across the plugin.
+    /// Main entry point for BetaHub diagnostics.
     /// </summary>
     public static class BetaHubDiagnostics
     {
         /// <summary>
         /// Creates a contextual diagnostics service for a specific component or operation context.
         /// </summary>
-        /// <param name="contextName">The name of the context (e.g., "Issue(abc123)", "GameRecorder", etc.)</param>
-        /// <returns>A contextual diagnostics service instance</returns>
         public static IContextualDiagnostics ForContext(string contextName)
         {
             return new ContextualDiagnostics(contextName);
         }
     }
 
     /// <summary>
-    /// Implementation of contextual diagnostics that provides consistent logging and error handling
-    /// for a specific context (component, operation, or scope).
+    /// Implementation of contextual diagnostics that provides consistent logging.
     /// </summary>
     internal class ContextualDiagnostics : IContextualDiagnostics
     {
         private readonly string _contextName;
 
-        /// <summary>
-        /// Initializes a new instance of contextual diagnostics for the specified context.
-        /// </summary>
-        /// <param name="contextName">The context name to include in all log messages</param>
         public ContextualDiagnostics(string contextName)
         {
             _contextName = contextName ?? throw new ArgumentNullException(nameof(contextName));
         }
 
-        /// <summary>
-        /// Logs an informational message for an operation.
-        /// </summary>
-        /// <param name="operation">The operation name</param>
-        /// <param name="message">The message to log</param>
         public void LogInfo(string operation, string message)
         {
             Debug.Log($"[INFO] {_contextName}.{operation}: {message}");
         }
 
-        /// <summary>
-        /// Logs a successful operation completion.
-        /// </summary>
-        /// <param name="operation">The operation name</param>
-        /// <param name="message">The success message</param>
         public void LogSuccess(string operation, string message)
         {
             Debug.Log($"[SUCCESS] {_contextName}.{operation}: {message}");
         }
 
-        /// <summary>
-        /// Logs progress information for a long-running operation.
-        /// </summary>
-        /// <param name="operation">The operation name</param>
-        /// <param name="message">The progress message</param>
         public void LogProgress(string operation, string message)
         {
             Debug.Log($"[PROGRESS] {_contextName}.{operation}: {message}");
         }
 
-        /// <summary>
-        /// Logs an error with automatic exception categorization and retry analysis.
-        /// </summary>
-        /// <param name="operation">The operation name</param>
-        /// <param name="ex">The exception that occurred</param>
-        public void LogError(string operation, Exception ex)
-        {
-            var category = BetaHubErrorHelper.CategorizeException(ex);
-            var isRetryable = BetaHubErrorHelper.IsRetryable(ex);
-            LogError(operation, ex.Message, category, isRetryable);
-        }
-
-        /// <summary>
-        /// Logs an error with explicit categorization and retry information.
-        /// </summary>
-        /// <param name="operation">The operation name</param>
-        /// <param name="message">The error message</param>
-        /// <param name="category">The error category</param>
-        /// <param name="isRetryable">Whether the error is retryable</param>
-        public void LogError(string operation, string message, BetaHubErrorCategory category = BetaHubErrorCategory.Unknown, bool isRetryable = false)
+        public void LogError(string operation, string message, Exception ex)
         {
-            string retryInfo = isRetryable ? " (Retryable)" : " (Not retryable)";
-            Debug.LogError($"[ERROR] {_contextName}.{operation} [{category}]: {message}{retryInfo}");
+            Debug.LogError($"[ERROR] {_contextName}.{operation}: {message} - {ex.Message}");
         }
     }
-
-    /// <summary>
-    /// Configuration class for BetaHub diagnostics behavior.
-    /// Allows customization of logging levels and formatting.
-    /// </summary>
-    public static class BetaHubDiagnosticsConfig
-    {
-        /// <summary>
-        /// Gets or sets whether to include timestamps in log messages.
-        /// Default: false (Unity Console already includes timestamps)
-        /// </summary>
-        public static bool IncludeTimestamps { get; set; } = false;
-
-        /// <summary>
-        /// Gets or sets whether to include stack traces in error logs.
-        /// Default: false (Unity Console shows stack traces separately)
-        /// </summary>
-        public static bool IncludeStackTraces { get; set; } = false;
-
-        /// <summary>
-        /// Gets or sets the minimum log level to output.
-        /// Can be used to reduce log verbosity in production builds.
-        /// </summary>
-        public static BetaHubLogLevel MinimumLogLevel { get; set; } = BetaHubLogLevel.Info;
-    }
-
-    /// <summary>
-    /// Log levels for BetaHub diagnostics.
-    /// </summary>
-    public enum BetaHubLogLevel
-    {
-        /// <summary>
-        /// Progress and detailed information
-        /// </summary>
-        Progress = 0,
-
-        /// <summary>
-        /// General information
-        /// </summary>
-        Info = 1,
-
-        /// <summary>
-        /// Success notifications
-        /// </summary>
-        Success = 2,
-
-        /// <summary>
-        /// Error messages only
-        /// </summary>
-        Error = 3,
-
-        /// <summary>
-        /// No logging
-        /// </summary>
-        None = 4
-    }
 }