bug_bunny 4.11.1 → 4.13.0

data/spec/unit/session_spec.rb

@@ -69,6 +69,88 @@ RSpec.describe BugBunny::Session do
     end
   end
 
+  describe '#on_return handler' do
+    let(:return_info) do
+      Struct.new(:reply_code, :reply_text, :exchange, :routing_key)
+            .new(312, 'NO_ROUTE', 'evt_x', 'unbound.key')
+    end
+    let(:properties) { double('properties') }
+    let(:body) { '{"a":1}' }
+
+    before do
+      # Logger en memoria para inspeccionar el default
+      @log_io = StringIO.new
+      BugBunny.configuration.logger = Logger.new(@log_io)
+      BugBunny.configuration.on_return = nil
+    end
+
+    after do
+      BugBunny.configuration.logger = Logger.new($stdout).tap { |l| l.level = Logger::INFO }
+      BugBunny.configuration.on_return = nil
+    end
+
+    it 'se registra sobre el exchange cuando publisher_confirms está activo' do
+      exchange = session.exchange(name: 'evt_x', type: 'topic')
+
+      expect { exchange.fire_return(return_info, properties, body) }.not_to raise_error
+    end
+
+    it 'registra una sola vez por nombre de exchange' do
+      first  = session.exchange(name: 'evt_x', type: 'topic')
+      second = session.exchange(name: 'evt_x', type: 'topic')
+
+      expect(second).to be(first)
+      expect(session.instance_variable_get(:@configured_returns)).to include('evt_x' => true)
+    end
+
+    it 'NO se registra cuando publisher_confirms está desactivado' do
+      fresh_channel = BunnyMocks::FakeChannel.new(true)
+      fresh_conn = BunnyMocks::FakeConnection.new(true, fresh_channel)
+      no_confirms = described_class.new(fresh_conn, publisher_confirms: false)
+      exchange = no_confirms.exchange(name: 'evt_x', type: 'topic')
+
+      expect(exchange.instance_variable_get(:@on_return_handler)).to be_nil
+    end
+
+    it 'NO se registra sobre el default exchange (name vacío)' do
+      default = session.exchange # name nil → default_exchange
+
+      expect(default.instance_variable_get(:@on_return_handler)).to be_nil
+    end
+
+    it 'invoca el callback de Configuration#on_return cuando está definido' do
+      received = nil
+      BugBunny.configuration.on_return = lambda { |ri, props, b|
+        received = { rk: ri.routing_key, props: props, body: b }
+      }
+
+      exchange = session.exchange(name: 'evt_x', type: 'topic')
+      exchange.fire_return(return_info, properties, body)
+
+      expect(received).to include(rk: 'unbound.key', body: '{"a":1}')
+    end
+
+    it 'logea session.broker_return como :warn cuando no hay callback' do
+      exchange = session.exchange(name: 'evt_x', type: 'topic')
+      exchange.fire_return(return_info, properties, body)
+
+      log = @log_io.string
+      expect(log).to include('event=session.broker_return')
+      expect(log).to include('reply_code=312')
+      expect(log).to include('routing_key=unbound.key')
+      expect(log).to include('body_size=7')
+    end
+
+    it 'no propaga excepciones del callback de usuario' do
+      BugBunny.configuration.on_return = ->(_, _, _) { raise 'boom' }
+
+      exchange = session.exchange(name: 'evt_x', type: 'topic')
+
+      expect { exchange.fire_return(return_info, properties, body) }.not_to raise_error
+      expect(@log_io.string).to include('event=session.on_return_failed')
+    end
+  end
+
   describe '#close' do
     it 'cierra el canal y lo nilifica' do
       session.channel

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bug_bunny
 version: !ruby/object:Gem::Version
-  version: 4.11.1
+  version: 4.13.0
 platform: ruby
 authors:
 - gabix
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-04-09 00:00:00.000000000 Z
+date: 2026-05-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bunny
@@ -269,7 +269,6 @@ files:
 - skill/references/routing.md
 - skill/references/testing.md
 - skills-lock.json
-- skills.lock
 - skills.yml
 - spec/integration/client_spec.rb
 - spec/integration/consumer_middleware_spec.rb
@@ -304,7 +303,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.11.1/skill
+  documentation_uri: https://github.com/gedera/bug_bunny/blob/v4.13.0/skill
 post_install_message:
 rdoc_options: []
 require_paths:

data/skills.lock

@@ -1,33 +0,0 @@
----
-synced_at: '2026-04-08 09:13:30'
-skills:
-- name: agent-review
-  scope: local
-  path: "/Users/gabriel/src/gems/bug_bunny/.agents/skills/agent-review"
-- name: ai-reports
-  scope: local
-  path: "/Users/gabriel/src/gems/bug_bunny/.agents/skills/ai-reports"
-- name: documentation-writer
-  scope: local
-  path: "/Users/gabriel/src/gems/bug_bunny/.agents/skills/documentation-writer"
-- name: find-skills
-  scope: local
-  path: "/Users/gabriel/src/gems/bug_bunny/.agents/skills/find-skills"
-- name: gem-release
-  scope: local
-  path: "/Users/gabriel/src/gems/bug_bunny/.agents/skills/gem-release"
-- name: quality-code
-  scope: local
-  path: "/Users/gabriel/src/gems/bug_bunny/.agents/skills/quality-code"
-- name: rabbitmq-expert
-  scope: local
-  path: "/Users/gabriel/src/gems/bug_bunny/.agents/skills/rabbitmq-expert"
-- name: skill-builder
-  scope: local
-  path: "/Users/gabriel/src/gems/bug_bunny/.agents/skills/skill-builder"
-- name: skill-manager
-  scope: local
-  path: "/Users/gabriel/src/gems/bug_bunny/.agents/skills/skill-manager"
-- name: yard
-  scope: local
-  path: "/Users/gabriel/src/gems/bug_bunny/.agents/skills/yard"