bug_bunny 4.0.1 → 4.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b15b464fc158fbab2ed65325af31aedda4160fa435762a87bba018a827714ba
-  data.tar.gz: dfeca1173ed11ec9b7f9615b37ff596156988700cd95e7b155335771095d17dd
+  metadata.gz: 599004cdcb498ca746612f60202301ce9783a67ac37d14ecd4c29f51ccaf48bb
+  data.tar.gz: f748ba9a55e91f96a5571f671afa524f7e330cceede0f12c5c7cef6d25d2d2ff
 SHA512:
-  metadata.gz: dc7e4d36f8f5570f134a81cd48d1aeffd647d6a08a6817397b6004ef82111d63f43df1ef01c685da537e1b65102f7abd9f32ea97f3c4ec79254365fad1a417eb
-  data.tar.gz: 5fb2f92c8bec877f3ff81a0db8c50e2859737c5000e6c97a79935408f9a83486f1dc93b8218e5baeaad915e631e7d139a500f7c4d71b6c61d6a1c9f5e45fbf69
+  metadata.gz: 7df1503a55cb62d9f87168a84b8e92fe773b4c9525b143d6f848227e66fe63e5ab75f6191a408b00bb9c8840b44c0f8fdbc36cd0a8d2eb56e02018d09654d269
+  data.tar.gz: d5433d839c45ad95a2d0399535ff629112458e90d0e4e1ec89f97afa450b139bcb7855c64c27042c22bce0bde87b2bfa7bd6f8bae5b879dd7eb0c71ff1d20d3c

data/CHANGELOG.md

@@ -1,4 +1,22 @@
 # Changelog
+
+## [4.1.1] - 2026-03-22
+
+### 🐛 Bug Fixes
+* **Consumer:** Previene memory leak al detener el `TimerTask` de health check previo antes de realizar una reconexión.
+* **Controller:** Corrige la mutación accidental de \`log_tags\` globales al usar una lógica de herencia no destructiva en \`compute_tags\`.
+
+### ✨ Improvements
+* **Controller:** Ahora lanza una excepción \`BugBunny::BadRequest\` (400) si el cuerpo de la petición contiene un JSON inválido, mejorando la depuración en el cliente.
+* **Resource:** Se añadió una protección a \`.with\` (\`ScopeProxy\`) para asegurar que el contexto sea de un solo uso, evitando efectos secundarios en llamadas encadenadas.
+
+## [4.1.0] - 2026-03-22
+
+### 🚀 New Features & Improvements
+* **Faraday-style Client API:** Se introdujo el método \`Client#send\` como punto de entrada genérico, permitiendo una sintaxis más familiar y flexible.
+* **Flexible Delivery Modes:** Introducción del atributo \`delivery_mode\` (:rpc o :publish). Ahora es posible configurar la estrategia de envío a nivel de cliente o por cada petición individual.
+* **Smart Request Defaults:** Los métodos \`request\` y \`publish\` ahora delegan internamente en \`send\`, manteniendo la compatibilidad pero beneficiándose de la nueva arquitectura de peticiones.
+
 ## [4.0.1] - 2026-03-13
 
 ### 🐛 Bug Fixes

data/README.md

@@ -203,6 +203,26 @@ client = BugBunny::Client.new(pool: BUG_BUNNY_POOL) do |stack|
   stack.use BugBunny::Middleware::JsonResponse
 end
 
+# 1. Método genérico 'send' (Estilo Faraday)
+# El comportamiento (RPC o Fire-and-forget) depende de 'delivery_mode'
+client.delivery_mode = :rpc # Default
+client.send('users/1', method: :get)
+
+# 2. Configuración flexible del modo de entrega
+# Por cada petición
+client.send('logs', method: :post, body: { msg: 'system_up' }, delivery_mode: :publish)
+
+# O mediante un bloque para configuración avanzada
+client.send('users/1') do |req|
+  req.method = :get
+  req.delivery_mode = :rpc
+  req.timeout = 5
+end
+
+# 3. Métodos de conveniencia (Atajos)
+client.request('users/1') # Siempre :rpc
+client.publish('events', body: { type: 'click' }) # Siempre :publish
+
 # Ahora el cliente devolverá Hashes y lanzará errores si el worker falla
 response = client.request('users/1', method: :get)
 ```

data/lib/bug_bunny/client.rb

@@ -21,6 +21,9 @@ module BugBunny
     # @return [BugBunny::Middleware::Stack] La pila de middlewares configurada.
     attr_reader :stack
 
+    # @return [Symbol] El modo de entrega por defecto para este cliente (:rpc o :publish).
+    attr_accessor :delivery_mode
+
     # Inicializa un nuevo cliente.
     #
     # @param pool [ConnectionPool] Pool de conexiones a RabbitMQ configurado previamente.
@@ -30,9 +33,22 @@ module BugBunny
       raise ArgumentError, "BugBunny::Client requiere un 'pool:'" if pool.nil?
       @pool = pool
       @stack = BugBunny::Middleware::Stack.new
+      @delivery_mode = :rpc
       yield(@stack) if block_given?
     end
 
+    # Realiza una petición general al estilo Faraday.
+    # El comportamiento (RPC o Fire-and-forget) depende de {#delivery_mode}.
+    #
+    # @param url [String] La ruta del recurso.
+    # @param args [Hash] Opciones de configuración.
+    # @yield [req] Bloque para configurar el objeto Request directamente.
+    def send(url, **args)
+      run_in_pool(url, args) do |req|
+        yield req if block_given?
+      end
+    end
+
     # Realiza una petición Síncrona (RPC / Request-Response).
     #
     # Envía un mensaje y bloquea la ejecución del hilo actual hasta recibir respuesta.
@@ -48,7 +64,8 @@ module BugBunny
     # @yield [req] Bloque para configurar el objeto Request directamente.
     # @return [Hash] La respuesta del servidor.
     def request(url, **args)
-      run_in_pool(:rpc, url, args) do |req|
+      send(url, **args) do |req|
+        req.delivery_mode = :rpc
         yield req if block_given?
       end
     end
@@ -60,7 +77,8 @@ module BugBunny
     # @yield [req] Bloque para configurar el objeto Request.
     # @return [void]
     def publish(url, **args)
-      run_in_pool(:fire, url, args) do |req|
+      send(url, **args) do |req|
+        req.delivery_mode = :publish
         yield req if block_given?
       end
     end
@@ -70,15 +88,16 @@ module BugBunny
     # Ejecuta la lógica de envío dentro del contexto del Pool.
     # Mapea los argumentos al objeto Request y ejecuta la cadena de middlewares.
     #
-    # @param method_name [Symbol] El método del productor a llamar (:rpc o :fire).
     # @param url [String] La ruta destino.
     # @param args [Hash] Argumentos pasados a los métodos públicos.
     # @yield [req] Bloque para configuración adicional del Request.
-    def run_in_pool(method_name, url, args)
+    def run_in_pool(url, args)
       # 1. Builder del Request
       req = BugBunny::Request.new(url)
 
       # 2. Syntactic Sugar: Mapeo de argumentos a atributos del Request
+      req.delivery_mode    = delivery_mode           # Default del cliente
+      req.delivery_mode    = args[:delivery_mode]    if args[:delivery_mode]
       req.method           = args[:method]           if args[:method]
       req.body             = args[:body]             if args[:body]
       req.exchange         = args[:exchange]         if args[:exchange]
@@ -101,6 +120,10 @@ module BugBunny
         producer = BugBunny::Producer.new(session)
 
         begin
+          # Mapeo de delivery_mode al método del productor (:rpc o :fire)
+          # :publish se mapea a :fire por consistencia interna.
+          method_name = req.delivery_mode == :publish ? :fire : :rpc
+
           # Onion Architecture: La acción final es llamar al Producer real.
           final_action = ->(env) { producer.send(method_name, env) }
 

data/lib/bug_bunny/consumer.rb

@@ -44,6 +44,7 @@ module BugBunny
     # @param connection [Bunny::Session] Conexión nativa de Bunny.
     def initialize(connection)
       @session = BugBunny::Session.new(connection)
+      @health_timer = nil
     end
 
     # Inicia la suscripción a la cola y comienza el bucle de procesamiento.
@@ -244,12 +245,16 @@ module BugBunny
     # @param q_name [String] Nombre de la cola a monitorear.
     # @return [void]
     def start_health_check(q_name)
+      # Detener el timer anterior antes de crear uno nuevo (evita leak en cada retry)
+      @health_timer&.shutdown
+      @health_timer = nil
+
       file_path = BugBunny.configuration.health_check_file
 
       # Toque inicial para indicar al orquestador que el worker arrancó correctamente
       touch_health_file(file_path) if file_path
 
-      Concurrent::TimerTask.new(execution_interval: BugBunny.configuration.health_check_interval) do
+      @health_timer = Concurrent::TimerTask.new(execution_interval: BugBunny.configuration.health_check_interval) do
         # 1. Verificamos la salud de RabbitMQ (si falla, levanta un error y corta la ejecución del bloque)
         session.channel.queue_declare(q_name, passive: true)
 
@@ -258,7 +263,8 @@ module BugBunny
       rescue StandardError => e
         BugBunny.configuration.logger.warn("[BugBunny::Consumer] ⚠️  Queue check failed: #{e.message}. Reconnecting session...")
         session.close
-      end.execute
+      end
+      @health_timer.execute
     end
 
     # Actualiza la fecha de modificación del archivo de health check (touchfile).

data/lib/bug_bunny/controller.rb

@@ -146,11 +146,6 @@ module BugBunny
     def process(body)
       prepare_params(body)
 
-      # Inyección de configuración global de logs si el controlador no define propios
-      if self.class.log_tags.empty? && BugBunny.configuration.log_tags.any?
-        self.class.log_tags = BugBunny.configuration.log_tags
-      end
-
       action_name = headers[:action].to_sym
       current_arounds = resolve_callbacks(self.class.around_actions, action_name)
 
@@ -245,12 +240,11 @@ module BugBunny
       if body.is_a?(Hash)
         params.merge!(body)
       elsif body.is_a?(String) && headers[:content_type].to_s.include?('json')
-        parsed = begin
-                   JSON.parse(body)
-                 rescue JSON::ParserError
-                   nil
-                 end
-        params.merge!(parsed) if parsed
+        begin
+          params.merge!(JSON.parse(body))
+        rescue JSON::ParserError => e
+          raise BugBunny::BadRequest, "Invalid JSON in request body: #{e.message}"
+        end
       else
         self.raw_string = body
       end
@@ -284,7 +278,8 @@ module BugBunny
     end
 
     def compute_tags
-      self.class.log_tags.map do |tag|
+      tags = self.class.log_tags.presence || BugBunny.configuration.log_tags
+      tags.map do |tag|
         case tag
         when Proc
           tag.call(self)

data/lib/bug_bunny/request.rb

@@ -16,6 +16,7 @@ module BugBunny
   # @attr routing_key [String] La routing key específica. Si es nil, se usará {#path}.
   # @attr timeout [Integer] Tiempo máximo en segundos para timeout RPC.
   #
+  # @attr delivery_mode [Symbol] El modo de entrega (:rpc o :publish).
   # @attr exchange_options [Hash] Opciones específicas para la declaración del Exchange en esta petición.
   # @attr queue_options [Hash] Opciones específicas para la declaración de la Cola en esta petición.
   class Request
@@ -27,6 +28,7 @@ module BugBunny
     attr_accessor :exchange_type
     attr_accessor :routing_key
     attr_accessor :timeout
+    attr_accessor :delivery_mode
 
     # Configuración de Infraestructura Específica
     attr_accessor :exchange_options
@@ -48,12 +50,12 @@ module BugBunny
       @timestamp = Time.now.to_i
       @persistent = false
       @exchange_type = 'direct'
+      @delivery_mode = :rpc
 
       # Inicialización de opciones de infraestructura para evitar errores de nil durante el merge.
       @exchange_options = {}
       @queue_options = {}
     end
-
     # Calcula la Routing Key final que se usará en RabbitMQ.
     #
     # Principio: "Convention over Configuration".

data/lib/bug_bunny/resource.rb

@@ -154,9 +154,24 @@ module BugBunny
       end
 
       # Proxy para el encadenamiento del método `.with`.
+      # Solo puede usarse para UNA llamada de método: el contexto se restaura al finalizar.
       class ScopeProxy < BasicObject
-        def initialize(target, keys, old_values); @target = target; @keys = keys; @old_values = old_values; end
-        def method_missing(method, *args, &block); @target.public_send(method, *args, &block); ensure; @keys.each { |k, v| ::Thread.current[v] = @old_values[k] }; end
+        def initialize(target, keys, old_values)
+          @target = target
+          @keys = keys
+          @old_values = old_values
+          @used = false
+        end
+
+        def method_missing(method, *args, &block)
+          if @used
+            ::Kernel.raise ::BugBunny::Error, "ScopeProxy is single-use. Call .with again for a new context."
+          end
+          @used = true
+          @target.public_send(method, *args, &block)
+        ensure
+          @keys.each { |k, v| ::Thread.current[v] = @old_values[k] }
+        end
       end
 
       # Calcula la routing key final.

data/lib/bug_bunny/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module BugBunny
-  VERSION = "4.0.1"
+  VERSION = "4.1.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bug_bunny
 version: !ruby/object:Gem::Version
-  version: 4.0.1
+  version: 4.1.1
 platform: ruby
 authors:
 - gabix
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-03-13 00:00:00.000000000 Z
+date: 2026-03-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bunny