amq-protocol 2.4.0 → 2.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6d6619ef67f8b1e30048f898796d21854eae2a35dfdfe83286e0a5e894cbbf1c
-  data.tar.gz: 0d62824daba6da208b96b7681e4c1db4f5c32a93a7a8a7714b875b8ad8f603e2
+  metadata.gz: e3f43ce885cd75341d0b5b784857aada66eab3990174d7b145b572a04286f41f
+  data.tar.gz: 7ec8798e7d34ec8d4218b209e67d75fc45fe254e80997d761f8c7c2cf0521366
 SHA512:
-  metadata.gz: cac00e1594a26adcb0863b6505ba789e8fce9a1dee3a38584e58cf838479661ba857a37ac3d747c69e7751549e15877a6478dd51559a8003af70c68e856badb1
-  data.tar.gz: 040e923fd9dcf5931a157007835085ec30b7ac44897c5a76ee05545f14f76cf431a3611f73fddb562e6bf9230fb82151fc808bfed24bc2244f12c85164541b6e
+  metadata.gz: 378697156e2adf582c060b1be25c374f68cf7764008cebf926a07a3c1f02f7669680e2d934a4cec94a18f3e779889a45b4998113a076844d4298e230914683c6
+  data.tar.gz: 8ff53d0b35d42dbfa6a515391b26593340446ceb985740bd1ec6d5e0cb043beef4d46e7c262ee918ac753a4a96db746e1dceaba4fc8b479228340045080650ee

data/AGENTS.md

@@ -0,0 +1,23 @@
+# Instructions for AI Agents
+
+## Overview
+
+This is a pure Ruby implementation of a AMQP 0-9-1 protocol parser
+(more specifically% serialization, deserialization, framing) used by
+[Bunny](https://github.com/ruby-amqp/bunny), a Ruby AMQP 0-9-1 client for RabbitMQ.
+
+## Target Ruby Version
+
+This library targets Ruby 3.0 and later versions.
+
+## Comments
+
+ * Only add very important comments, both in tests and in the implementation
+
+## Git Instructions
+
+ * Never add yourself to the list of commit co-authors
+
+## Style Guide
+
+ * Never add full stops to Markdown list items

data/CLAUDE.md

@@ -0,0 +1 @@
+AGENTS.md

data/ChangeLog.md

@@ -1,12 +1,16 @@
-## Changes between 2.4.0 and 2.5.0 (in development)
+## Changes between 2.5.0 and 2.6.0 (in development)
 
 No changes yet.
 
-## Changes between 2.3.4 and 2.4.0 (Dec 30, 2025)
 
-### Bug Fixes
+## Changes between 2.4.0 and 2.5.0 (Dec 31, 2025)
+
+### Additional Consumer Hot Path Optimizations
+
+A few more decode method optimizations for consumer delivery code paths.
 
- * Fixed `BadResponseError` constructor which referenced undefined variable `data` instead of `actual`
+
+## Changes between 2.3.4 and 2.4.0 (Dec 30, 2025)
 
 ### Performance Improvements
 

data/GEMINI.md

@@ -0,0 +1 @@
+AGENTS.md

data/lib/amq/protocol/client.rb

@@ -1569,30 +1569,31 @@ module AMQ
         0x0004,
       ]
 
+      # Optimized decode_properties using getbyte and unpack1
       def self.decode_properties(data)
         offset, data_length, properties = 0, data.bytesize, {}
 
-        compressed_index = data[offset, 2].unpack(PACK_UINT16)[0]
+        compressed_index = data.byteslice(offset, 2).unpack1(PACK_UINT16)
         offset += 2
         while data_length > offset
           DECODE_PROPERTIES_KEYS.each do |key|
             next unless compressed_index >= key
             compressed_index -= key
-            name = DECODE_PROPERTIES[key] || raise(RuntimeError.new("No property found for index #{index.inspect}!"))
+            name = DECODE_PROPERTIES[key] || raise(RuntimeError.new("No property found for index #{key.inspect}!"))
             case DECODE_PROPERTIES_TYPE[key]
             when :shortstr
-              size = data[offset, 1].unpack(PACK_CHAR)[0]
+              size = data.getbyte(offset)
               offset += 1
-              result = data[offset, size]
+              result = data.byteslice(offset, size)
             when :octet
               size = 1
-              result = data[offset, size].unpack(PACK_CHAR).first
+              result = data.getbyte(offset)
             when :timestamp
               size = 8
-              result = Time.at(data[offset, size].unpack(PACK_UINT64_BE).last)
+              result = Time.at(data.byteslice(offset, 8).unpack1(PACK_UINT64_BE))
             when :table
-              size = 4 + data[offset, 4].unpack(PACK_UINT32)[0]
-              result = Table.decode(data[offset, size])
+              size = 4 + data.byteslice(offset, 4).unpack1(PACK_UINT32)
+              result = Table.decode(data.byteslice(offset, size))
             end
             properties[name] = result
             offset += size
@@ -1688,13 +1689,11 @@ module AMQ
         @index = 0x003C0015 # 60, 21, 3932181
         @packed_indexes = [60, 21].pack(PACK_UINT16_X2).freeze
 
+        # Optimized decode using getbyte
         # @return
         def self.decode(data)
-          offset = offset = 0 # self-assigning offset to eliminate "assigned but unused variable" warning even if offset is not used in this method
-          length = data[offset, 1].unpack(PACK_CHAR).first
-          offset += 1
-          consumer_tag = data[offset, length]
-          offset += length
+          length = data.getbyte(0)
+          consumer_tag = data.byteslice(1, length)
           self.new(consumer_tag)
         end
 
@@ -1866,26 +1865,25 @@ module AMQ
         @index = 0x003C003C # 60, 60, 3932220
         @packed_indexes = [60, 60].pack(PACK_UINT16_X2).freeze
 
+        # Optimized decode using getbyte and unpack1 for better performance
         # @return
         def self.decode(data)
-          offset = offset = 0 # self-assigning offset to eliminate "assigned but unused variable" warning even if offset is not used in this method
-          length = data[offset, 1].unpack(PACK_CHAR).first
+          offset = 0
+          length = data.getbyte(offset)
           offset += 1
-          consumer_tag = data[offset, length]
+          consumer_tag = data.byteslice(offset, length)
           offset += length
-          delivery_tag = AMQ::Pack.unpack_uint64_big_endian(data[offset, 8]).first
+          delivery_tag = data.byteslice(offset, 8).unpack1(PACK_UINT64_BE)
           offset += 8
-          bit_buffer = data[offset, 1].unpack(PACK_CHAR).first
+          redelivered = (data.getbyte(offset) & 1) != 0
           offset += 1
-          redelivered = (bit_buffer & (1 << 0)) != 0
-          length = data[offset, 1].unpack(PACK_CHAR).first
+          length = data.getbyte(offset)
           offset += 1
-          exchange = data[offset, length]
+          exchange = data.byteslice(offset, length)
           offset += length
-          length = data[offset, 1].unpack(PACK_CHAR).first
+          length = data.getbyte(offset)
           offset += 1
-          routing_key = data[offset, length]
-          offset += length
+          routing_key = data.byteslice(offset, length)
           self.new(consumer_tag, delivery_tag, redelivered, exchange, routing_key)
         end
 
@@ -2009,14 +2007,11 @@ module AMQ
         @index = 0x003C0050 # 60, 80, 3932240
         @packed_indexes = [60, 80].pack(PACK_UINT16_X2).freeze
 
+        # Optimized decode using unpack1 and getbyte
         # @return
         def self.decode(data)
-          offset = offset = 0 # self-assigning offset to eliminate "assigned but unused variable" warning even if offset is not used in this method
-          delivery_tag = AMQ::Pack.unpack_uint64_big_endian(data[offset, 8]).first
-          offset += 8
-          bit_buffer = data[offset, 1].unpack(PACK_CHAR).first
-          offset += 1
-          multiple = (bit_buffer & (1 << 0)) != 0
+          delivery_tag = data.byteslice(0, 8).unpack1(PACK_UINT64_BE)
+          multiple = (data.getbyte(8) & 1) != 0
           self.new(delivery_tag, multiple)
         end
 
@@ -2141,15 +2136,13 @@ module AMQ
         @index = 0x003C0078 # 60, 120, 3932280
         @packed_indexes = [60, 120].pack(PACK_UINT16_X2).freeze
 
+        # Optimized decode using unpack1 and getbyte
         # @return
         def self.decode(data)
-          offset = offset = 0 # self-assigning offset to eliminate "assigned but unused variable" warning even if offset is not used in this method
-          delivery_tag = AMQ::Pack.unpack_uint64_big_endian(data[offset, 8]).first
-          offset += 8
-          bit_buffer = data[offset, 1].unpack(PACK_CHAR).first
-          offset += 1
-          multiple = (bit_buffer & (1 << 0)) != 0
-          requeue = (bit_buffer & (1 << 1)) != 0
+          delivery_tag = data.byteslice(0, 8).unpack1(PACK_UINT64_BE)
+          bit_buffer = data.getbyte(8)
+          multiple = (bit_buffer & 1) != 0
+          requeue = (bit_buffer & 2) != 0
           self.new(delivery_tag, multiple, requeue)
         end
 

data/lib/amq/protocol/frame.rb

@@ -60,9 +60,11 @@ This functionality is part of the https://github.com/ruby-amqp/amq-client librar
         EOF
       end
 
+      # Optimized header decode using unpack1 for single values where appropriate
       def self.decode_header(header)
         raise EmptyResponseError if header == nil || header.empty?
 
+        # Use unpack for multiple values - this is the optimal approach
         type_id, channel, size = header.unpack(PACK_CHAR_UINT16_UINT32)
         type = TYPES_REVERSE[type_id]
         raise FrameTypeError.new(TYPES_OPTIONS) unless type

data/lib/amq/protocol/version.rb

@@ -1,5 +1,5 @@
 module AMQ
   module Protocol
-    VERSION = "2.4.0"
+    VERSION = "2.5.0"
   end # Protocol
 end # AMQ

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: amq-protocol
 version: !ruby/object:Gem::Version
-  version: 2.4.0
+  version: 2.5.0
 platform: ruby
 authors:
 - Jakub Stastny
@@ -28,7 +28,10 @@ files:
 - ".gitmodules"
 - ".rspec"
 - ".travis.yml"
+- AGENTS.md
+- CLAUDE.md
 - ChangeLog.md
+- GEMINI.md
 - Gemfile
 - LICENSE
 - README.md