refactor(policy): streamline policy validation logic and enhance exception interface

Author: dgarciasquare1

lib/src/core/policy_manager.dart

@@ -86,54 +86,43 @@ class PolicyManager extends ChangeNotifier {
         final key = entry.key;
         final value = entry.value;
 
-        try {
-          if (value == null) {
-            LogHandler.warning(
-              'Skipping null policy value',
-              context: {'role': key},
-              operation: 'policy_validation_skip',
-            );
-            continue;
-          }
-
-          if (value is! List) {
-            LogHandler.warning(
-              'Skipping invalid policy value type',
-              context: {
-                'role': key,
-                'expected_type': 'List',
-                'actual_type': value.runtimeType.toString(),
-              },
-              operation: 'policy_validation_skip',
-            );
-            continue;
-          }
-
-          if (value.any((item) => item is! String)) {
-            LogHandler.warning(
-              'Skipping policy with non-string content items',
-              context: {'role': key},
-              operation: 'policy_validation_skip',
-            );
-            continue;
-          }
-
-          // Create the policy and add to valid policies
-          final policy = Policy(
-            roleName: key,
-            allowedContent: value.cast<String>(),
+        if (value == null) {
+          LogHandler.warning(
+            'Skipping null policy value',
+            context: {'role': key},
+            operation: 'policy_validation_skip',
+          );
+          continue;
+        }
+
+        if (value is! List) {
+          LogHandler.warning(
+            'Skipping invalid policy value type',
+            context: {
+              'role': key,
+              'expected_type': 'List',
+              'actual_type': value.runtimeType.toString(),
+            },
+            operation: 'policy_validation_skip',
           );
-          validPolicies[key] = policy.toJson();
-        } catch (e, stackTrace) {
-          LogHandler.error(
-            'Failed to process policy',
-            error: e,
-            stackTrace: stackTrace,
+          continue;
+        }
+
+        if (value.any((item) => item is! String)) {
+          LogHandler.warning(
+            'Skipping policy with non-string content items',
             context: {'role': key},
-            operation: 'policy_processing_error',
+            operation: 'policy_validation_skip',
           );
-          // Continue with other policies
+          continue;
         }
+
+        // Create the policy and add to valid policies
+        final policy = Policy(
+          roleName: key,
+          allowedContent: value.cast<String>(),
+        );
+        validPolicies[key] = policy.toJson();
       }
 
       _policies = JsonHandler.parseMap(

lib/src/exceptions/i_policy_sdk_exceptions.dart

@@ -1,51 +1,33 @@
-/// Base exception class for all policy SDK related errors.
+/// Base exception interface for all policy SDK related errors.
 ///
-/// This abstract class provides a common interface for all exceptions
+/// This abstract interface provides a common contract for all exceptions
 /// thrown by the policy engine, ensuring consistent error handling
 /// and messaging across the SDK.
 abstract class IPolicySDKException implements Exception {
-  /// Creates a new PolicySDKException with the given error message.
-  ///
-  /// [message] should provide a clear description of what went wrong.
-  const IPolicySDKException(this.message);
-
   /// The error message describing the exception.
-  final String message;
+  String get message;
 
   @override
-  String toString() => 'PolicySDKException: $message';
+  String toString();
 }
 
-/// Abstract exception for detailed policy SDK errors with contextual information.
+/// Abstract interface for detailed policy SDK errors with contextual information.
 ///
-/// This class extends [IPolicySDKException] to provide additional context for
+/// This interface extends [IPolicySDKException] to provide additional context for
 /// errors that occur within the policy engine, such as the specific key involved,
 /// the original error thrown, and a map of field-specific validation errors.
-/// Subclasses should use this to represent exceptions where more granular
+/// Implementations should use this to represent exceptions where more granular
 /// diagnostic information is valuable for debugging or reporting.
 abstract class IDetailPolicySDKException implements IPolicySDKException {
-  /// Creates a new [IDetailPolicySDKException] with an error [message] and optional details.
-  ///
-  /// [message] provides a human-readable description of the error.
-  /// [key] optionally identifies the specific key or field related to the error.
-  /// [originalError] optionally contains the original error object that triggered this exception.
-  /// [errors] optionally provides a map of field-specific validation errors.
-  const IDetailPolicySDKException(
-    this.message, {
-    this.key,
-    this.originalError,
-    this.errors,
-  });
-
   @override
-  final String message;
+  String get message;
 
   /// The specific key or field that caused the error, if applicable.
-  final String? key;
+  String? get key;
 
   /// The original error object that led to this exception, if available.
-  final Object? originalError;
+  Object? get originalError;
 
   /// A map of field-specific validation errors encountered during processing, if any.
-  final Map<String, String>? errors;
+  Map<String, String>? get errors;
 }

test/core/policy_manager_test.dart

@@ -67,8 +67,7 @@ class MockPolicyEvaluator implements IPolicyEvaluator {
 
 class ThrowingPolicy extends Policy {
   const ThrowingPolicy(
-      {required String roleName, required List<String> allowedContent})
-      : super(roleName: roleName, allowedContent: allowedContent);
+      {required super.roleName, required super.allowedContent});
 
   static Policy fromJson(Map<String, dynamic> json) {
     throw StateError('Forced error in fromJson');