smart_domain 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8c99a8063c81eea7d1de36a368d27803fd13807ae99271c4d184d1f58c82d67d
-  data.tar.gz: 41c04fc2729bfc7861dbbf396e65c4d04a843c353f493796e72b89af10b1f388
+  metadata.gz: dc4986884824f0ebda3f3ea7051e7303bb416b7497f1b3213a749bdbd2a80918
+  data.tar.gz: 5a65d52f74b31c7ebc0da5d8a63d8b8942b5262746de39826f185d61129f32e7
 SHA512:
-  metadata.gz: 512d73e5e5081d01a2aa204f2d02d27e217d87f143863a5ab890a738ccdd637706e6b21927f08f67c0bfd5bdf55990d3ac8a042a68b5442869d99da844dea9f5
-  data.tar.gz: d3b7c9bae2e7787d7b2bda3604cca244069879584d2d18ce0ef0f60437e4ad7d14d6e6bbbad919059ee3c215bb021d374f31719718ed6b8ae48aa57990f788cc
+  metadata.gz: 06015b7d5dc2598a35fa0ebda62d2c2ece2b1c7a1cfc4719713d70e54fdebde7b713c84f713334853d9244d9f994b88d6a4937cc6b7dcedb25f8800bd7d87551
+  data.tar.gz: 2af6c6ee30a91e21947907885f4662b3565576b9a53631eb216a799167e2ff88f5088bee7b6bf6c5e04c01b9062bec243518aee6f556eb023cda2b784d6f5ea5

data/.rubocop.yml

@@ -0,0 +1,65 @@
+AllCops:
+  NewCops: enable
+  TargetRubyVersion: 3.0
+  SuggestExtensions: false
+  Exclude:
+    - 'vendor/**/*'
+    - 'bin/**/*'
+    - 'node_modules/**/*'
+    - 'examples/**/*'
+
+# Allow longer lines in tests
+Layout/LineLength:
+  Max: 120
+  Exclude:
+    - 'spec/**/*'
+
+# Allow long blocks in specs and rake tasks
+Metrics/BlockLength:
+  Exclude:
+    - 'spec/**/*'
+    - '*.gemspec'
+    - 'lib/**/*.rake'
+
+# Allow longer methods for generators and complex logic
+Metrics/MethodLength:
+  Max: 30
+
+# Allow more complex methods
+Metrics/AbcSize:
+  Max: 30
+  Exclude:
+    - 'lib/generators/**/*'
+
+# Allow more cyclomatic complexity
+Metrics/CyclomaticComplexity:
+  Max: 10
+
+# Allow higher perceived complexity
+Metrics/PerceivedComplexity:
+  Max: 10
+
+# Allow longer classes for handlers
+Metrics/ClassLength:
+  Max: 150
+
+# Allow defining constants in test blocks
+Lint/ConstantDefinitionInBlock:
+  Exclude:
+    - 'spec/**/*'
+
+# Allow duplicate branches (false positives)
+Lint/DuplicateBranch:
+  Enabled: false
+
+# Allow development dependencies in gemspec
+Gemspec/DevelopmentDependencies:
+  Enabled: false
+
+# Allow more than one class per file in specs
+Style/ClassAndModuleChildren:
+  Enabled: false
+
+# Don't require documentation for all classes
+Style/Documentation:
+  Enabled: false

data/README.md

@@ -1,5 +1,10 @@
 # SmartDomain
 
+[![Gem Version](https://badge.fury.io/rb/smart_domain.svg)](https://badge.fury.io/rb/smart_domain)
+[![Downloads](https://img.shields.io/gem/dt/smart_domain.svg)](https://rubygems.org/gems/smart_domain)
+[![License](https://img.shields.io/badge/license-MIT-blue.svg)](https://opensource.org/licenses/MIT)
+[![Ruby](https://img.shields.io/badge/ruby-%3E%3D%203.0-ruby.svg)](https://www.ruby-lang.org)
+
 **Domain-Driven Design and Event-Driven Architecture for Rails**
 
 SmartDomain brings battle-tested DDD/EDA patterns from platform to Ruby on Rails applications. It provides domain events, an event bus, generic handlers, and Rails generators for rapid domain scaffolding.

data/Rakefile

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 
-require "rubocop/rake_task"
+require 'rubocop/rake_task'
 
 RuboCop::RakeTask.new
 

data/lib/generators/smart_domain/domain/domain_generator.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require "rails/generators"
-require "rails/generators/named_base"
+require 'rails/generators'
+require 'rails/generators/named_base'
 
 module SmartDomain
   module Generators
@@ -22,18 +22,18 @@ module SmartDomain
     #     user_updated_event.rb
     #     user_deleted_event.rb
     class DomainGenerator < Rails::Generators::NamedBase
-      source_root File.expand_path("templates", __dir__)
+      source_root File.expand_path('templates', __dir__)
 
-      desc "Generate a complete domain structure with service, events, and policy"
+      desc 'Generate a complete domain structure with service, events, and policy'
 
       class_option :skip_service, type: :boolean, default: false,
-                                   desc: "Skip generating domain service"
+                                  desc: 'Skip generating domain service'
       class_option :skip_policy, type: :boolean, default: false,
-                                  desc: "Skip generating domain policy"
+                                 desc: 'Skip generating domain policy'
       class_option :skip_events, type: :boolean, default: false,
-                                  desc: "Skip generating domain events"
+                                 desc: 'Skip generating domain events'
       class_option :skip_setup, type: :boolean, default: false,
-                                 desc: "Skip generating setup file"
+                                desc: 'Skip generating setup file'
 
       # Generate domain directory structure
       def create_domain_directory
@@ -44,30 +44,30 @@ module SmartDomain
       def create_service
         return if options[:skip_service]
 
-        template "service.rb.tt", "#{domain_path}/#{file_name}_service.rb"
+        template 'service.rb.tt', "#{domain_path}/#{file_name}_service.rb"
       end
 
       # Generate domain events in app/events/
       def create_events
         return if options[:skip_events]
 
-        template "events/created_event.rb.tt", "app/events/#{file_name}_created_event.rb"
-        template "events/updated_event.rb.tt", "app/events/#{file_name}_updated_event.rb"
-        template "events/deleted_event.rb.tt", "app/events/#{file_name}_deleted_event.rb"
+        template 'events/created_event.rb.tt', "app/events/#{file_name}_created_event.rb"
+        template 'events/updated_event.rb.tt', "app/events/#{file_name}_updated_event.rb"
+        template 'events/deleted_event.rb.tt', "app/events/#{file_name}_deleted_event.rb"
       end
 
       # Generate domain policy
       def create_policy
         return if options[:skip_policy]
 
-        template "policy.rb.tt", "app/policies/#{file_name}_policy.rb"
+        template 'policy.rb.tt', "app/policies/#{file_name}_policy.rb"
       end
 
       # Generate setup file for event registration
       def create_setup
         return if options[:skip_setup]
 
-        template "setup.rb.tt", "#{domain_path}/setup.rb"
+        template 'setup.rb.tt', "#{domain_path}/setup.rb"
       end
 
       # Show instructions
@@ -84,10 +84,10 @@ module SmartDomain
         say "  #{domain_path}/setup.rb" unless options[:skip_setup]
 
         say "\nNext steps:"
-        say "  1. Review and customize the generated files"
+        say '  1. Review and customize the generated files'
         say "  2. Add business logic to #{class_name}Service"
         say "  3. Customize authorization rules in #{class_name}Policy"
-        say "  4. Restart your Rails server to load the domain setup"
+        say '  4. Restart your Rails server to load the domain setup'
       end
 
       private

data/lib/generators/smart_domain/install/install_generator.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "rails/generators"
+require 'rails/generators'
 
 module SmartDomain
   module Generators
@@ -16,42 +16,42 @@ module SmartDomain
     #   - app/handlers/ directory
     #   - app/policies/ directory
     class InstallGenerator < Rails::Generators::Base
-      source_root File.expand_path("templates", __dir__)
+      source_root File.expand_path('templates', __dir__)
 
-      desc "Install SmartDomain into your Rails application"
+      desc 'Install SmartDomain into your Rails application'
 
       # Create initializer
       def create_initializer
-        template "initializer.rb", "config/initializers/smart_domain.rb"
+        template 'initializer.rb', 'config/initializers/smart_domain.rb'
       end
 
       # Create directory structure
       def create_directory_structure
-        create_file "app/domains/.keep"
-        create_file "app/events/.keep"
-        create_file "app/handlers/.keep"
-        create_file "app/policies/.keep"
-        create_file "app/services/.keep"
+        create_file 'app/domains/.keep'
+        create_file 'app/events/.keep'
+        create_file 'app/handlers/.keep'
+        create_file 'app/policies/.keep'
+        create_file 'app/services/.keep'
       end
 
       # Create base event class
       def create_application_event
-        template "application_event.rb", "app/events/application_event.rb"
+        template 'application_event.rb', 'app/events/application_event.rb'
       end
 
       # Create base policy class
       def create_application_policy
-        template "application_policy.rb", "app/policies/application_policy.rb"
+        template 'application_policy.rb', 'app/policies/application_policy.rb'
       end
 
       # Create base service class
       def create_application_service
-        template "application_service.rb", "app/services/application_service.rb"
+        template 'application_service.rb', 'app/services/application_service.rb'
       end
 
       # Show post-install message
       def show_readme
-        readme "README" if behavior == :invoke
+        readme 'README' if behavior == :invoke
       end
     end
   end

data/lib/smart_domain/configuration.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "logger"
+require 'logger'
 
 module SmartDomain
   # Configuration for SmartDomain gem.

data/lib/smart_domain/domain/exceptions.rb

@@ -84,7 +84,7 @@ module SmartDomain
     #   )
     class BusinessRuleError < Error
       def initialize(message, code: :business_rule_violation, details: {})
-        super(message, code: code, details: details)
+        super
       end
     end
 
@@ -134,7 +134,7 @@ module SmartDomain
     #     'User does not have permission to delete this resource'
     #   )
     class UnauthorizedError < Error
-      def initialize(message = "Unauthorized", action: nil, resource: nil)
+      def initialize(message = 'Unauthorized', action: nil, resource: nil)
         super(
           message,
           code: :unauthorized,

data/lib/smart_domain/domain/service.rb

@@ -150,7 +150,7 @@ module SmartDomain
         attributes[:organization_id] ||= current_organization_id
 
         # Auto-fill actor fields if event includes ActorMixin
-        if event_class.instance_methods.include?(:actor_id)
+        if event_class.method_defined?(:actor_id)
           attributes[:actor_id] ||= current_user_id&.to_s
           attributes[:actor_email] ||= current_user&.email
         end

data/lib/smart_domain/event/adapters/memory.rb

@@ -18,7 +18,10 @@ module SmartDomain
       class Memory
         def initialize
           @handlers = Hash.new { |h, k| h[k] = [] }
-          @logger = ActiveSupport::TaggedLogging.new(Logger.new($stdout))
+        end
+
+        def logger
+          @logger ||= SmartDomain.configuration.logger
         end
 
         # Subscribe a handler to an event type
@@ -30,16 +33,30 @@ module SmartDomain
 
         # Publish an event to all subscribed handlers
         # @param event [SmartDomain::Event::Base] Event to publish
+        # @raise [ValidationError] If event is invalid
         def publish(event)
+          # Validate event before publishing
+          unless event.is_a?(SmartDomain::Event::Base)
+            raise SmartDomain::Event::ValidationError, 'Event must be a SmartDomain::Event::Base'
+          end
+
+          begin
+            unless event.valid?
+              raise SmartDomain::Event::ValidationError, "Invalid event: #{event.errors.full_messages.join(', ')}"
+            end
+          rescue NoMethodError => e
+            raise SmartDomain::Event::ValidationError, "Malformed event: #{e.message}"
+          end
+
           # Find all matching handlers (exact match + wildcard patterns)
           matching_handlers = find_matching_handlers(event.event_type)
 
           if matching_handlers.empty?
-            @logger.debug "[SmartDomain::Memory] No handlers for event type: #{event.event_type}"
+            logger.debug "[SmartDomain::Memory] No handlers for event type: #{event.event_type}"
             return
           end
 
-          @logger.debug "[SmartDomain::Memory] Notifying #{matching_handlers.size} handler(s) for #{event.event_type}"
+          logger.debug "[SmartDomain::Memory] Notifying #{matching_handlers.size} handler(s) for #{event.event_type}"
 
           matching_handlers.each do |handler|
             handle_event(handler, event)
@@ -69,9 +86,7 @@ module SmartDomain
           handlers = []
 
           @handlers.each do |pattern, pattern_handlers|
-            if event_type_matches?(event_type, pattern)
-              handlers.concat(pattern_handlers)
-            end
+            handlers.concat(pattern_handlers) if event_type_matches?(event_type, pattern)
           end
 
           handlers.uniq
@@ -86,7 +101,7 @@ module SmartDomain
           return true if event_type == pattern
 
           # Wildcard pattern match (e.g., "user.*" matches "user.created")
-          if pattern.end_with?(".*")
+          if pattern.end_with?('.*')
             prefix = pattern[0..-3] # Remove ".*"
             return event_type.start_with?("#{prefix}.")
           end
@@ -100,8 +115,8 @@ module SmartDomain
         def handle_event(handler, event)
           handler.handle(event)
         rescue StandardError => e
-          @logger.error "[SmartDomain::Memory] Handler #{handler.class.name} failed: #{e.message}"
-          @logger.error e.backtrace.join("\n")
+          logger.error "[SmartDomain::Memory] Error in event handler #{handler.class.name}: #{e.message}"
+          logger.error e.backtrace.join("\n")
           # Swallow exception to not affect other handlers
         end
       end

data/lib/smart_domain/event/base.rb

@@ -1,12 +1,15 @@
 # frozen_string_literal: true
 
-require "active_model"
-require "active_support/core_ext/object/blank"
-require "securerandom"
-require "logger"
+require 'active_model'
+require 'active_support/core_ext/object/blank'
+require 'securerandom'
+require 'logger'
 
 module SmartDomain
   module Event
+    # Validation error for domain events
+    class ValidationError < StandardError; end
+
     # Base class for all domain events in the system.
     #
     # Domain events represent significant business occurrences that other
@@ -58,7 +61,9 @@ module SmartDomain
       def initialize(attributes = {})
         super
         freeze_event
-        validate!
+        return if valid?
+
+        raise ValidationError, "Event validation failed: #{errors.full_messages.join(', ')}"
       end
 
       # Convert event to hash
@@ -101,13 +106,16 @@ module SmartDomain
     #   event = UserCreatedEvent.new(...)
     #   bus.publish(event)
     class Bus
-      attr_reader :adapter, :logger
+      attr_reader :adapter
 
       # Initialize event bus with optional adapter
       # @param adapter [Object, Symbol] Event bus adapter or adapter name (default: Memory adapter)
       def initialize(adapter: nil)
         @adapter = resolve_adapter(adapter)
-        @logger = ActiveSupport::TaggedLogging.new(Logger.new($stdout))
+      end
+
+      def logger
+        @logger ||= SmartDomain.configuration.logger
       end
 
       # Subscribe a handler to a specific event type
@@ -115,7 +123,7 @@ module SmartDomain
       # @param handler [Object] Handler object that responds to #handle(event)
       def subscribe(event_type, handler)
         @adapter.subscribe(event_type, handler)
-        @logger.info "[SmartDomain] Event handler subscribed: #{handler.class.name} -> #{event_type}"
+        logger.info "[SmartDomain] Event handler subscribed: #{handler.class.name} -> #{event_type}"
       end
 
       # Publish an event to all registered handlers
@@ -124,8 +132,8 @@ module SmartDomain
       def publish(event)
         validate_event!(event)
 
-        @logger.info "[SmartDomain] Publishing event: #{event.event_type} (#{event.event_id})"
-        @logger.debug "[SmartDomain] Event details: #{event.to_h}"
+        logger.info "[SmartDomain] Publishing event: #{event.event_type} (#{event.event_id})"
+        logger.debug "[SmartDomain] Event details: #{event.to_h}"
 
         @adapter.publish(event)
       end
@@ -149,15 +157,20 @@ module SmartDomain
 
       # Validate event before publishing
       # @param event [Object] Event to validate
-      # @raise [ArgumentError] If event is not valid
+      # @raise [ArgumentError] If event is not a Base instance
+      # @raise [ValidationError] If event validation fails
       def validate_event!(event)
         unless event.is_a?(Base)
           raise ArgumentError, "Event must be a SmartDomain::Event::Base, got #{event.class.name}"
         end
 
-        return if event.valid?
+        begin
+          return if event.valid?
 
-        raise ArgumentError, "Event validation failed: #{event.errors.full_messages.join(', ')}"
+          raise ValidationError, "Event validation failed: #{event.errors.full_messages.join(', ')}"
+        rescue NoMethodError => e
+          raise ValidationError, "Malformed event: #{e.message}"
+        end
       end
     end
 

data/lib/smart_domain/event/handler.rb

@@ -37,7 +37,7 @@ module SmartDomain
       # @param event [SmartDomain::Event::Base] Event to handle
       # @raise [NotImplementedError] If not implemented by subclass
       def handle(event)
-        raise NotImplementedError, "#{self.class.name} must implement #handle(event)"
+        raise NotImplementedError, 'Subclasses must implement #handle(event)'
       end
 
       # Check if this handler can handle a specific event type
@@ -48,7 +48,7 @@ module SmartDomain
       # @param event_type [String] Event type to check
       # @return [Boolean] True if handler can handle this event type
       def can_handle?(event_type)
-        raise NotImplementedError, "#{self.class.name} must implement #can_handle?(event_type)"
+        raise NotImplementedError, 'Subclasses must implement #can_handle?(event_type)'
       end
 
       # Handle event asynchronously using ActiveJob
@@ -58,12 +58,14 @@ module SmartDomain
       #
       # @param event [SmartDomain::Event::Base] Event to handle
       # @raise [RuntimeError] If ActiveJob is not loaded
+      # @raise [ValidationError] If event is invalid
       def handle_async(event)
         unless defined?(ActiveJob)
           raise "ActiveJob is required for async event handling. Please require 'active_job' in your application."
         end
 
-        event.validate!
+        raise ValidationError, "Event validation failed: #{event.errors.full_messages.join(', ')}" unless event.valid?
+
         HandlerJob.perform_later(self.class.name, event.to_h)
       end
     end

data/lib/smart_domain/event/mixins.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "active_support/concern"
+require 'active_support/concern'
 
 module SmartDomain
   module Event
@@ -89,6 +89,14 @@ module SmartDomain
         attribute :changed_fields, default: -> { [] }
         attribute :old_values, default: -> { {} }
         attribute :new_values, default: -> { {} }
+
+        validate :changed_fields_must_not_be_empty
+
+        private
+
+        def changed_fields_must_not_be_empty
+          errors.add(:changed_fields, "can't be blank") if changed_fields.blank? || changed_fields.empty?
+        end
       end
 
       # Helper to extract changes from an ActiveRecord model
@@ -150,6 +158,8 @@ module SmartDomain
 
       included do
         attribute :reason, :string
+
+        validates :reason, presence: true
       end
     end
   end

data/lib/smart_domain/event/registration.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative "../handlers/audit_handler"
-require_relative "../handlers/metrics_handler"
+require_relative '../handlers/audit_handler'
+require_relative '../handlers/metrics_handler'
 
 module SmartDomain
   module Event
@@ -86,8 +86,8 @@ module SmartDomain
         # Log what was registered
         if include_audit || include_metrics
           handlers_registered = []
-          handlers_registered << "audit" if include_audit
-          handlers_registered << "metrics" if include_metrics
+          handlers_registered << 'audit' if include_audit
+          handlers_registered << 'metrics' if include_metrics
 
           logger.info "[SmartDomain::Registration] Standard handlers registered for #{domain} domain: " \
                       "#{handlers_registered.join(', ')} (#{events.size} events)"

data/lib/smart_domain/generators/domain_generator.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 
 # Load the domain generator
-require_relative "../../generators/smart_domain/domain/domain_generator"
+require_relative '../../generators/smart_domain/domain/domain_generator'

data/lib/smart_domain/generators/install_generator.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 
 # Load the install generator
-require_relative "../../generators/smart_domain/install/install_generator"
+require_relative '../../generators/smart_domain/install/install_generator'

data/lib/smart_domain/handlers/audit_handler.rb

@@ -41,7 +41,7 @@ module SmartDomain
       # @param event_type [String] Event type to check
       # @return [Boolean] True if event belongs to this domain
       def can_handle?(event_type)
-        return true if @domain == "*"
+        return true if @domain == '*'
 
         event_type.start_with?("#{@domain}.")
       end
@@ -49,7 +49,7 @@ module SmartDomain
       # Handle audit logging for an event
       # @param event [SmartDomain::Event::Base] Event to audit
       def handle(event)
-        action = event.event_type.split(".").last
+        action = event.event_type.split('.').last
 
         # 1. Log to Rails logger (structured)
         log_audit_event(event, action)
@@ -117,17 +117,15 @@ module SmartDomain
         return unless defined?(AuditEvent)
 
         AuditEvent.create!(
-          event_type: event.event_type.upcase.tr(".", "_"),
-          event_category: map_event_category(event.event_type),
-          user_id: extract_actor_id(event),
+          event_id: event.event_id,
+          event_type: event.event_type,
+          aggregate_id: event.aggregate_id,
+          aggregate_type: event.aggregate_type,
           organization_id: event.organization_id,
-          ip_address: extract_ip_address(event),
-          user_agent: extract_user_agent(event),
-          old_values: extract_old_values(event),
-          new_values: extract_new_values(event),
-          occurred_at: event.occurred_at,
+          category: map_event_category(event.event_type),
           risk_level: assess_risk_level(event.event_type),
-          compliance_flags: build_compliance_flags(event)
+          event_data: event.to_h,
+          occurred_at: event.occurred_at
         )
       rescue StandardError => e
         @logger.warn "[SmartDomain::AuditHandler] Failed to write to audit table: #{e.message}"
@@ -138,14 +136,14 @@ module SmartDomain
       # @return [String] Audit category
       def map_event_category(event_type)
         case event_type
-        when /^auth\./
-          "authentication"
+        when /^auth\.|logged_in|logged_out|login|logout|authenticated|password/
+          'authentication'
         when /accessed|viewed|patient\./
-          "data_access"
+          'data_access'
         when /created|updated|deleted|assigned|removed/
-          "admin_action"
+          'admin_action'
         else
-          "system_event"
+          'system_event'
         end
       end
 
@@ -155,11 +153,11 @@ module SmartDomain
       def assess_risk_level(event_type)
         case event_type
         when /suspended|deleted|revoked|failed|rejected/
-          "HIGH"
+          'HIGH'
         when /updated|changed|assigned/
-          "MEDIUM"
+          'MEDIUM'
         else
-          "LOW"
+          'LOW'
         end
       end
 
@@ -167,35 +165,35 @@ module SmartDomain
       # @param event [SmartDomain::Event::Base] Event
       # @return [String, nil] Actor ID
       def extract_actor_id(event)
-        event.try(:actor_id) || event.attributes["actor_id"]
+        event.try(:actor_id) || event.attributes['actor_id']
       end
 
       # Extract ip_address from event (SecurityContextMixin)
       # @param event [SmartDomain::Event::Base] Event
       # @return [String, nil] IP address
       def extract_ip_address(event)
-        event.try(:ip_address) || event.attributes["ip_address"]
+        event.try(:ip_address) || event.attributes['ip_address']
       end
 
       # Extract user_agent from event (SecurityContextMixin)
       # @param event [SmartDomain::Event::Base] Event
       # @return [String, nil] User agent
       def extract_user_agent(event)
-        event.try(:user_agent) || event.attributes["user_agent"]
+        event.try(:user_agent) || event.attributes['user_agent']
       end
 
       # Extract old_values from event (ChangeTrackingMixin)
       # @param event [SmartDomain::Event::Base] Event
       # @return [Hash, nil] Old values
       def extract_old_values(event)
-        event.try(:old_values) || event.attributes["old_values"]
+        event.try(:old_values) || event.attributes['old_values']
       end
 
       # Extract new_values from event (ChangeTrackingMixin)
       # @param event [SmartDomain::Event::Base] Event
       # @return [Hash, nil] New values
       def extract_new_values(event)
-        event.try(:new_values) || event.attributes["new_values"]
+        event.try(:new_values) || event.attributes['new_values']
       end
 
       # Build compliance flags for audit record

data/lib/smart_domain/handlers/metrics_handler.rb

@@ -36,7 +36,7 @@ module SmartDomain
       # @param event_type [String] Event type to check
       # @return [Boolean] True if event belongs to this domain
       def can_handle?(event_type)
-        return true if @domain == "*"
+        return true if @domain == '*'
 
         event_type.start_with?("#{@domain}.")
       end
@@ -47,7 +47,14 @@ module SmartDomain
         metric_name = build_metric_name(event)
         tags = build_metric_tags(event)
 
+        # Emit counter metric
         emit_metric(metric_name, tags)
+
+        # Emit timing metric if duration is present
+        if event.respond_to?(:duration) && event.duration
+          timing_name = "#{metric_name}.duration"
+          emit_metric(timing_name, tags.merge(duration_ms: event.duration))
+        end
       rescue StandardError => e
         # Never fail on metrics handler errors
         @logger.warn "[SmartDomain::MetricsHandler] Metrics collection failed: #{e.message}"

data/lib/smart_domain/integration/active_record.rb

@@ -145,14 +145,12 @@ module SmartDomain
         return if @pending_domain_events.blank?
 
         @pending_domain_events.each do |event|
-          begin
-            SmartDomain::Event.bus.publish(event)
-          rescue StandardError => e
-            # Log error but don't raise (events should be fire-and-forget)
-            logger = defined?(Rails) ? Rails.logger : Logger.new($stdout)
-            logger.error "[SmartDomain] Failed to publish event: #{e.message}"
-            logger.error e.backtrace.join("\n")
-          end
+          SmartDomain::Event.bus.publish(event)
+        rescue StandardError => e
+          # Log error but don't raise (events should be fire-and-forget)
+          logger = defined?(Rails) ? Rails.logger : Logger.new($stdout)
+          logger.error "[SmartDomain] Failed to publish event: #{e.message}"
+          logger.error e.backtrace.join("\n")
         end
 
         clear_domain_events

data/lib/smart_domain/railtie.rb

@@ -12,8 +12,8 @@ module SmartDomain
 
     # Add generators to load path
     generators do
-      require_relative "generators/install_generator"
-      require_relative "generators/domain_generator"
+      require_relative 'generators/install_generator'
+      require_relative 'generators/domain_generator'
     end
 
     # Configuration hook
@@ -27,35 +27,35 @@ module SmartDomain
 
     # Rake tasks
     rake_tasks do
-      load "smart_domain/tasks/domains.rake"
+      load 'smart_domain/tasks/domains.rake'
     end
 
-    private
-
     # Load all domain setup files
     def self.load_domain_setups
-      setup_files = Dir[Rails.root.join("app/domains/**/setup.rb")]
+      setup_files = Dir[Rails.root.join('app/domains/**/setup.rb')]
 
       setup_files.each do |setup_file|
-        begin
-          require setup_file
-
-          # Extract domain module name from path
-          # e.g., app/domains/user_management/setup.rb -> UserManagement
-          domain_path = setup_file.gsub(Rails.root.join("app/domains/").to_s, "")
-          domain_name = domain_path.split("/").first.camelize
-
-          # Call setup! method if defined
-          domain_module = domain_name.constantize rescue nil
-          if domain_module && domain_module.respond_to?(:setup!)
-            domain_module.setup!
-            Rails.logger.info "[SmartDomain] Loaded domain: #{domain_name}"
-          end
-        rescue StandardError => e
-          Rails.logger.error "[SmartDomain] Failed to load domain setup: #{setup_file}"
-          Rails.logger.error e.message
-          Rails.logger.error e.backtrace.join("\n")
+        require setup_file
+
+        # Extract domain module name from path
+        # e.g., app/domains/user_management/setup.rb -> UserManagement
+        domain_path = setup_file.gsub(Rails.root.join('app/domains/').to_s, '')
+        domain_name = domain_path.split('/').first.camelize
+
+        # Call setup! method if defined
+        domain_module = begin
+          domain_name.constantize
+        rescue StandardError
+          nil
+        end
+        if domain_module.respond_to?(:setup!)
+          domain_module.setup!
+          Rails.logger.info "[SmartDomain] Loaded domain: #{domain_name}"
         end
+      rescue StandardError => e
+        Rails.logger.error "[SmartDomain] Failed to load domain setup: #{setup_file}"
+        Rails.logger.error e.message
+        Rails.logger.error e.backtrace.join("\n")
       end
     end
   end

data/lib/smart_domain/tasks/domains.rake

@@ -1,43 +1,47 @@
 # frozen_string_literal: true
 
 namespace :active_domain do
-  desc "List all registered domains"
+  desc 'List all registered domains'
   task domains: :environment do
-    setup_files = Dir[Rails.root.join("app/domains/**/setup.rb")]
+    setup_files = Dir[Rails.root.join('app/domains/**/setup.rb')]
 
     if setup_files.empty?
-      puts "No domains found in app/domains/"
-      puts "Generate your first domain with: rails generate active_domain:domain User"
+      puts 'No domains found in app/domains/'
+      puts 'Generate your first domain with: rails generate active_domain:domain User'
     else
-      puts "Registered domains:"
+      puts 'Registered domains:'
       setup_files.each do |setup_file|
-        domain_path = setup_file.gsub(Rails.root.join("app/domains/").to_s, "")
-        domain_name = domain_path.split("/").first
+        domain_path = setup_file.gsub(Rails.root.join('app/domains/').to_s, '')
+        domain_name = domain_path.split('/').first
         puts "  - #{domain_name}"
       end
       puts "\nTotal: #{setup_files.size} domain(s)"
     end
   end
 
-  desc "Reload all domain setups"
+  desc 'Reload all domain setups'
   task reload: :environment do
-    setup_files = Dir[Rails.root.join("app/domains/**/setup.rb")]
+    setup_files = Dir[Rails.root.join('app/domains/**/setup.rb')]
 
     puts "Reloading #{setup_files.size} domain(s)..."
 
     setup_files.each do |setup_file|
       load setup_file
 
-      domain_path = setup_file.gsub(Rails.root.join("app/domains/").to_s, "")
-      domain_name = domain_path.split("/").first.camelize
+      domain_path = setup_file.gsub(Rails.root.join('app/domains/').to_s, '')
+      domain_name = domain_path.split('/').first.camelize
 
-      domain_module = domain_name.constantize rescue nil
-      if domain_module && domain_module.respond_to?(:setup!)
+      domain_module = begin
+        domain_name.constantize
+      rescue StandardError
+        nil
+      end
+      if domain_module.respond_to?(:setup!)
         domain_module.setup!
         puts "  ✓ #{domain_name}"
       end
     end
 
-    puts "Done!"
+    puts 'Done!'
   end
 end

data/lib/smart_domain/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SmartDomain
-  VERSION = "0.1.1"
+  VERSION = '0.1.2'
 end

data/lib/smart_domain.rb

@@ -1,38 +1,38 @@
 # frozen_string_literal: true
 
-require "active_model"
-require "active_support"
-require "active_support/core_ext"
-require "active_record"
-require "logger"
+require 'active_model'
+require 'active_support'
+require 'active_support/core_ext'
+require 'active_record'
+require 'logger'
 
-require_relative "smart_domain/version"
-require_relative "smart_domain/configuration"
+require_relative 'smart_domain/version'
+require_relative 'smart_domain/configuration'
 
 # Event system
-require_relative "smart_domain/event/base"
-require_relative "smart_domain/event/mixins"
-require_relative "smart_domain/event/handler"
-require_relative "smart_domain/event/adapters/memory"
+require_relative 'smart_domain/event/base'
+require_relative 'smart_domain/event/mixins'
+require_relative 'smart_domain/event/handler'
+require_relative 'smart_domain/event/adapters/memory'
 
 # Handlers
-require_relative "smart_domain/handlers/audit_handler"
-require_relative "smart_domain/handlers/metrics_handler"
+require_relative 'smart_domain/handlers/audit_handler'
+require_relative 'smart_domain/handlers/metrics_handler'
 
 # Event registration
-require_relative "smart_domain/event/registration"
+require_relative 'smart_domain/event/registration'
 
 # Domain patterns
-require_relative "smart_domain/domain/exceptions"
-require_relative "smart_domain/domain/policy"
-require_relative "smart_domain/domain/service"
+require_relative 'smart_domain/domain/exceptions'
+require_relative 'smart_domain/domain/policy'
+require_relative 'smart_domain/domain/service'
 
 # Rails integration
-require_relative "smart_domain/integration/active_record"
-require_relative "smart_domain/integration/multi_tenancy"
+require_relative 'smart_domain/integration/active_record'
+require_relative 'smart_domain/integration/multi_tenancy'
 
 # Railtie (loads automatically if Rails is present)
-require_relative "smart_domain/railtie" if defined?(Rails::Railtie)
+require_relative 'smart_domain/railtie' if defined?(Rails::Railtie)
 
 # SmartDomain - Domain-Driven Design and Event-Driven Architecture for Rails
 #

data/smart_domain.gemspec

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
-require_relative "lib/smart_domain/version"
+require_relative 'lib/smart_domain/version'
 
 Gem::Specification.new do |spec|
-  spec.name = "smart_domain"
+  spec.name = 'smart_domain'
   spec.version = SmartDomain::VERSION
-  spec.authors = ["Rachid Al Maach"]
-  spec.email = ["rachid@qraft.nl"]
+  spec.authors = ['Rachid Al Maach']
+  spec.email = ['rachid@qraft.nl']
 
-  spec.summary = "Smart Domain-Driven Design and Event-Driven Architecture for Rails"
+  spec.summary = 'Smart Domain-Driven Design and Event-Driven Architecture for Rails'
   spec.description = <<~DESC
     SmartDomain brings battle-tested DDD/EDA patterns to Rails applications.
     Inspired by the Aeyes healthcare platform, it provides domain events,
@@ -16,13 +16,14 @@ Gem::Specification.new do |spec|
     Features 70% boilerplate reduction through intelligent event handling and
     AI-augmented development patterns.
   DESC
-  spec.homepage = "https://github.com/rachid/smart_domain"
-  spec.license = "MIT"
-  spec.required_ruby_version = ">= 3.0.0"
+  spec.homepage = 'https://github.com/rachid/smart_domain'
+  spec.license = 'MIT'
+  spec.required_ruby_version = '>= 3.0.0'
 
-  spec.metadata["homepage_uri"] = spec.homepage
-  spec.metadata["source_code_uri"] = "https://github.com/rachid/smart_domain"
-  spec.metadata["changelog_uri"] = "https://github.com/rachid/smart_domain/blob/main/CHANGELOG.md"
+  spec.metadata['homepage_uri'] = spec.homepage
+  spec.metadata['source_code_uri'] = 'https://github.com/rachid/smart_domain'
+  spec.metadata['changelog_uri'] = 'https://github.com/rachid/smart_domain/blob/main/CHANGELOG.md'
+  spec.metadata['rubygems_mfa_required'] = 'true'
 
   spec.files = Dir.chdir(__dir__) do
     `git ls-files -z`.split("\x0").reject do |f|
@@ -31,23 +32,23 @@ Gem::Specification.new do |spec|
     end
   end
 
-  spec.bindir = "exe"
+  spec.bindir = 'exe'
   spec.executables = spec.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
   # Core dependencies
-  spec.add_dependency "rails", ">= 7.0"
-  spec.add_dependency "activemodel", ">= 7.0"
-  spec.add_dependency "activesupport", ">= 7.0"
-  spec.add_dependency "activerecord", ">= 7.0"
+  spec.add_dependency 'activemodel', '>= 7.0'
+  spec.add_dependency 'activerecord', '>= 7.0'
+  spec.add_dependency 'activesupport', '>= 7.0'
+  spec.add_dependency 'rails', '>= 7.0'
 
   # Development dependencies
-  spec.add_development_dependency "rspec", "~> 3.12"
-  spec.add_development_dependency "rspec-rails", "~> 6.0"
-  spec.add_development_dependency "factory_bot_rails", "~> 6.2"
-  spec.add_development_dependency "faker", "~> 3.2"
-  spec.add_development_dependency "rubocop", "~> 1.50"
-  spec.add_development_dependency "rubocop-rails", "~> 2.19"
-  spec.add_development_dependency "rubocop-rspec", "~> 2.20"
-  spec.add_development_dependency "sqlite3", ">= 2.1"
+  spec.add_development_dependency 'factory_bot_rails', '~> 6.2'
+  spec.add_development_dependency 'faker', '~> 3.2'
+  spec.add_development_dependency 'rspec', '~> 3.12'
+  spec.add_development_dependency 'rspec-rails', '~> 6.0'
+  spec.add_development_dependency 'rubocop', '~> 1.50'
+  spec.add_development_dependency 'rubocop-rails', '~> 2.19'
+  spec.add_development_dependency 'rubocop-rspec', '~> 2.20'
+  spec.add_development_dependency 'sqlite3', '>= 2.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: smart_domain
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Rachid Al Maach
@@ -11,7 +11,7 @@ cert_chain: []
 date: 2025-12-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rails
+  name: activemodel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -25,7 +25,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '7.0'
 - !ruby/object:Gem::Dependency
-  name: activemodel
+  name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +53,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '7.0'
 - !ruby/object:Gem::Dependency
-  name: activerecord
+  name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,61 +67,61 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '7.0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: factory_bot_rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.12'
+        version: '6.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.12'
+        version: '6.2'
 - !ruby/object:Gem::Dependency
-  name: rspec-rails
+  name: faker
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '6.0'
+        version: '3.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '6.0'
+        version: '3.2'
 - !ruby/object:Gem::Dependency
-  name: factory_bot_rails
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '6.2'
+        version: '3.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '6.2'
+        version: '3.12'
 - !ruby/object:Gem::Dependency
-  name: faker
+  name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: '6.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
@@ -191,6 +191,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".rspec"
+- ".rubocop.yml"
 - CHANGELOG.md
 - LICENSE
 - README.md
@@ -286,7 +287,6 @@ files:
 - examples/blog_app/config/initializers/inflections.rb
 - examples/blog_app/config/initializers/smart_domain.rb
 - examples/blog_app/config/locales/en.yml
-- examples/blog_app/config/master.key
 - examples/blog_app/config/puma.rb
 - examples/blog_app/config/queue.yml
 - examples/blog_app/config/recurring.yml
@@ -349,7 +349,6 @@ files:
 - lib/smart_domain/railtie.rb
 - lib/smart_domain/tasks/domains.rake
 - lib/smart_domain/version.rb
-- smart_domain-0.1.0.gem
 - smart_domain.gemspec
 homepage: https://github.com/rachid/smart_domain
 licenses:
@@ -358,6 +357,7 @@ metadata:
   homepage_uri: https://github.com/rachid/smart_domain
   source_code_uri: https://github.com/rachid/smart_domain
   changelog_uri: https://github.com/rachid/smart_domain/blob/main/CHANGELOG.md
+  rubygems_mfa_required: 'true'
 post_install_message:
 rdoc_options: []
 require_paths:

data/examples/blog_app/config/master.key

@@ -1 +0,0 @@
-c993036f99084b62923212c97ff582b8