bug_bunny 4.9.1 → 4.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c70dd01d8d666fe56dbb95bc925ee431133cc0c342dc113a0775a4dddd4ce135
-  data.tar.gz: 2917d1105969a8173a8b9f70944d9b5839122a56185a5334db0e1c3a04594d2c
+  metadata.gz: af9746f85e544f059873513a7ace9b478acb6664b9053bb90d1963afb1d72848
+  data.tar.gz: 32854fb9e2a67f8e5cb537550d957704443d95183d22d5305b48c3207edaf05b
 SHA512:
-  metadata.gz: 241b7e0e684aa866d98940a2a3bc587ed13cf1da65c5708ff983c439078c40d823f13cac54ee8bf6d735dda138050c598c8cbf06759605523b8b2207de0e8e4d
-  data.tar.gz: 0b4f509d46dbe3ebd43bf8421bd9938bd2aa5d68ca23e639c3c1d232d5ad9d41b55cec0a4a0096cf8481e46c5d06163ce959d0f0167636d9084bf5d8aa3c9ba2
+  metadata.gz: 90169fb942803e87f47cf1af2fb2b4cb4d66112aaf4ab6632bb6a8d5031383e1fc0105a4cf2449eed847822509a71c1e4695c8cacb9907889c286cc5cfefe70b
+  data.tar.gz: 39262e92b89fb583769bcfaa8a17972584b0aaeda9f197ff3cf7cf4c45941ca0ee0b3aa154e8073d4004f5213bb1852aa16cc6c6f809951b2bcf3c76b85fa66d

data/lib/bug_bunny/consumer.rb

@@ -176,7 +176,7 @@ module BugBunny
 
       safe_log(:info, 'consumer.message_received', method: http_method, path: path,
                                                    routing_key: delivery_info.routing_key, **otel_fields)
-      safe_log(:debug, 'consumer.message_received_body', body: body.truncate(200))
+      safe_log(:info, 'consumer.message_received_body', body: body&.truncate(500), body_size: body&.size || 0)
 
       # ===================================================================
       # 3. Ruteo Declarativo
@@ -256,7 +256,7 @@ module BugBunny
     rescue StandardError => e
       safe_log(:error, 'consumer.execution_error', duration_s: duration_s(start_time), **exception_metadata(e))
       safe_log(:debug, 'consumer.execution_error_backtrace', backtrace: e.backtrace.first(5).join(' | '))
-      handle_fatal_error(properties, 500, 'Internal Server Error', e.message)
+      handle_fatal_error(properties, 500, 'Internal Server Error', e.message, e)
       session.channel.reject(delivery_info.delivery_tag, false)
     end
 
@@ -267,7 +267,12 @@ module BugBunny
     # @param correlation_id [String] ID para correlacionar la respuesta con la petición original.
     # @return [void]
     def reply(payload, reply_to, correlation_id)
-      safe_log(:debug, 'consumer.rpc_reply', reply_to: reply_to, messaging_message_id: correlation_id)
+      safe_log(:info, 'consumer.rpc_reply',
+               reply_to: reply_to,
+               messaging_message_id: correlation_id,
+               response_status: payload[:status],
+               response_body: payload[:body]&.to_json&.truncate(500),
+               response_body_size: payload[:body]&.to_json&.size || 0)
       otel_headers = BugBunny::OTel.messaging_headers(
         operation: 'publish',
         destination: '',
@@ -287,14 +292,20 @@ module BugBunny
     # Maneja errores fatales asegurando que el cliente reciba una respuesta.
     # Evita que el cliente RPC se quede esperando hasta el timeout.
     #
+    # @param properties [Bunny::MessageProperties] Headers y propiedades AMQP.
+    # @param status [Integer] Código de estado HTTP.
+    # @param error_title [String] Título del error.
+    # @param detail [String] Detalle del error.
+    # @param exception [StandardError, nil] Excepción original (para status 500).
     # @api private
-    def handle_fatal_error(properties, status, error_title, detail)
+    def handle_fatal_error(properties, status, error_title, detail, exception = nil)
       return unless properties.reply_to
 
-      error_payload = {
-        status: status,
-        body: { error: error_title, detail: detail }
-      }
+      body = { error: error_title, detail: detail }
+
+      body[:bug_bunny_exception] = BugBunny::RemoteError.serialize(exception) if status == 500 && exception
+
+      error_payload = { status: status, body: body }
       reply(error_payload, properties.reply_to, properties.correlation_id)
     end
 

data/lib/bug_bunny/controller.rb

@@ -224,7 +224,12 @@ module BugBunny
       {
         status: 500,
         headers: response_headers,
-        body: { error: 'Internal Server Error', detail: exception.message, type: exception.class.name }
+        body: {
+          error: 'Internal Server Error',
+          detail: exception.message,
+          type: exception.class.name,
+          bug_bunny_exception: BugBunny::RemoteError.serialize(exception)
+        }
       }
     end
 

data/lib/bug_bunny/middleware/raise_error.rb

@@ -50,6 +50,14 @@ module BugBunny
           # Pasamos el body crudo; UnprocessableEntity lo procesará en exception.rb
           raise BugBunny::UnprocessableEntity, body
         when 500..599
+          if body.is_a?(Hash) && body['bug_bunny_exception']
+            exception_data = body['bug_bunny_exception']
+            raise BugBunny::RemoteError.new(
+              exception_data['class'],
+              exception_data['message'],
+              exception_data['backtrace'] || []
+            )
+          end
           raise BugBunny::InternalServerError, format_error_message(body)
         else
           handle_unknown_error(status, body)

data/lib/bug_bunny/producer.rb

@@ -89,7 +89,9 @@ module BugBunny
         BugBunny.configuration.on_rpc_reply&.call(result[:headers])
 
         safe_log(:debug, 'producer.rpc_response_received',
-                 messaging_system: 'rabbitmq', messaging_operation: 'receive', messaging_message_id: cid)
+                 messaging_system: 'rabbitmq', messaging_operation: 'receive', messaging_message_id: cid,
+                 response_body: result[:body]&.truncate(500),
+                 response_headers: result[:headers]&.to_json&.truncate(300))
 
         parse_response(result[:body])
       ensure
@@ -125,7 +127,12 @@ module BugBunny
       safe_log(:info, 'producer.publish', method: verb, path: target, **otel_fields)
       safe_log(:debug, 'producer.publish_detail', messaging_destination_name: request.exchange,
                                                   exchange_opts: final_x_opts)
-      safe_log(:debug, 'producer.publish_payload', payload: payload.truncate(300)) if payload.is_a?(String)
+      return unless payload.is_a?(String)
+
+      safe_log(:info, 'producer.publish_payload',
+               payload: payload.truncate(500),
+               payload_class: payload.class.name,
+               body_size: request.body.nil? ? 0 : request.body.size)
     end
 
     # Serializa el mensaje para su transporte.

data/lib/bug_bunny/remote_error.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+module BugBunny
+  # Error 500 especial que propagationa información de una excepción remota.
+  #
+  # Cuando un controller levanta una excepción no manejada en el worker, esta clase
+  # permite al llamador RPC acceder a:
+  # - La clase original de la excepción (ej: ActiveRecord::RecordNotFound)
+  # - El mensaje original
+  # - El backtrace completo para debugging
+  #
+  # Mantiene compatibilidad hacia atrás: si la respuesta no contiene
+  # bug_bunny_exception, se comporta como un InternalServerError común.
+  class RemoteError < ServerError
+    # @return [String] La clase de la excepción remota (ej: 'ActiveRecord::RecordNotFound').
+    attr_reader :original_class
+
+    # @return [String] El mensaje original de la excepción.
+    attr_reader :original_message
+
+    # @return [Array<String>] El backtrace original de la excepción.
+    attr_reader :original_backtrace
+
+    # Serializa una excepción para transmitirse como parte de la respuesta.
+    #
+    # @param exception [StandardError] La excepción a serializar.
+    # @param max_lines [Integer] Máximo de líneas del backtrace (default 25).
+    # @return [Hash] Estructura con class, message y backtrace.
+    def self.serialize(exception, max_lines: 25)
+      {
+        class: exception.class.name,
+        message: exception.message,
+        backtrace: exception.backtrace&.first(max_lines) || []
+      }
+    end
+
+    # Inicializa la excepción remota propagada desde el worker.
+    #
+    # @param original_class [String] Nombre completo de la clase de la excepción.
+    # @param message [String] Mensaje de la excepción.
+    # @param backtrace [Array<String>] Stack trace completo.
+    def initialize(original_class, message, backtrace)
+      @original_class = original_class
+      @original_message = message
+      @original_backtrace = backtrace || []
+      super(message)
+      set_backtrace(backtrace || [])
+    end
+
+    # @return [String] Representación legible de la excepción.
+    def to_s
+      "#{self.class.name}(#{original_class}): #{message}"
+    end
+  end
+end

data/lib/bug_bunny/resource.rb

@@ -401,6 +401,17 @@ module BugBunny
       attributes['id'] || @extra_attributes['id'] || @extra_attributes['ID'] || @extra_attributes['Id'] || @extra_attributes['_id']
     end
 
+    # Representación legible del recurso.
+    # Muestra solo ID y atributos principales, sin detalles de infraestructura.
+    #
+    # @return [String]
+    def inspect
+      infra_keys = %w[routing_key exchange exchange_type exchange_options queue_options _id]
+      attrs = @extra_attributes.merge(attributes).reject { |k, _| infra_keys.include?(k) || k == 'id' }
+      attr_str = attrs.first(5).map { |k, v| "#{k}=#{v.inspect}" }.join(' ')
+      "#<#{self.class.name} id=#{id.inspect} persisted=#{@persisted}#{" #{attr_str}" unless attr_str.empty?}>"
+    end
+
     def id=(value)
       if self.class.attribute_names.include?('id')
         super
@@ -461,6 +472,7 @@ module BugBunny
     # @return [Boolean]
     def destroy
       return false unless persisted?
+      return false unless id
 
       run_callbacks(:destroy) do
         path = "#{self.class.resource_name}/#{id}"

data/lib/bug_bunny/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module BugBunny
-  VERSION = '4.9.1'
+  VERSION = '4.10.0'
 end

data/lib/bug_bunny.rb

@@ -4,6 +4,7 @@ require 'bunny'
 require 'logger'
 require_relative 'bug_bunny/version'
 require_relative 'bug_bunny/exception'
+require_relative 'bug_bunny/remote_error'
 require_relative 'bug_bunny/configuration'
 require_relative 'bug_bunny/observability'
 require_relative 'bug_bunny/otel'

data/skills.lock

@@ -1,5 +1,5 @@
 ---
-synced_at: '2026-04-06 14:08:54'
+synced_at: '2026-04-08 09:13:30'
 skills:
 - name: agent-review
   scope: local

data/spec/integration/resource_spec.rb

@@ -110,4 +110,48 @@ RSpec.describe BugBunny::Resource, :integration do
       end
     end
   end
+
+  describe '#inspect' do
+    let(:node) { SpecNode.new(id: '123', name: 'test-node', status: 'active', ip: '192.168.1.1', port: 8080) }
+
+    it 'muestra id y persisted' do
+      expect(node.inspect).to include('id="123"')
+      expect(node.inspect).to include('persisted=false')
+    end
+
+    it 'muestra atributos principales sin detalles de infraestructura' do
+      result = node.inspect
+
+      expect(result).to include('name="test-node"')
+      expect(result).to include('status="active"')
+      expect(result).to include('ip="192.168.1.1"')
+      expect(result).to include('port=8080')
+      expect(result).not_to include('routing_key')
+      expect(result).not_to include('exchange')
+    end
+
+    it 'filtra atributos de infraestructura cuando están presentes' do
+      node.routing_key = 'radius_1'
+      node.exchange = 'test_exchange'
+      node.exchange_type = 'topic'
+      node.exchange_options = { durable: true }
+      node.persisted = true
+
+      result = node.inspect
+
+      expect(result).not_to include('routing_key')
+      expect(result).not_to include('exchange')
+      expect(result).not_to include('exchange_type')
+      expect(result).not_to include('exchange_options')
+      expect(result).to include('persisted=true')
+    end
+
+    it 'limita a 5 atributos principales' do
+      node = SpecNode.new(id: '1', a: '1', b: '2', c: '3', d: '4', e: '5', f: '6')
+
+      result = node.inspect
+
+      expect(result.scan('=').length).to be <= 7 # id + persisted + max 5 attrs
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bug_bunny
 version: !ruby/object:Gem::Version
-  version: 4.9.1
+  version: 4.10.0
 platform: ruby
 authors:
 - gabix
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-04-07 00:00:00.000000000 Z
+date: 2026-04-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bunny
@@ -249,6 +249,7 @@ files:
 - lib/bug_bunny/otel.rb
 - lib/bug_bunny/producer.rb
 - lib/bug_bunny/railtie.rb
+- lib/bug_bunny/remote_error.rb
 - lib/bug_bunny/request.rb
 - lib/bug_bunny/resource.rb
 - lib/bug_bunny/routing/route.rb
@@ -300,7 +301,7 @@ metadata:
   homepage_uri: https://github.com/gedera/bug_bunny
   source_code_uri: https://github.com/gedera/bug_bunny
   changelog_uri: https://github.com/gedera/bug_bunny/blob/main/CHANGELOG.md
-  documentation_uri: https://github.com/gedera/bug_bunny/blob/v4.9.1/skill
+  documentation_uri: https://github.com/gedera/bug_bunny/blob/v4.10.0/skill
 post_install_message:
 rdoc_options: []
 require_paths: