jetstream_bridge 3.0.0 → 3.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d47bc85ddad1450a71960670d94969ec5f6416dc98c07c8409e4d7243ec92f3e
-  data.tar.gz: b9130b48f7129d6c61c9f2cddc61a2e292c906c282925a3a550f79c24cc7eff4
+  metadata.gz: 9fa0ea79aaf06fb588e761aaab3afc83f6e41b9427650b1a0ac1f3265d0b4d9c
+  data.tar.gz: fd3c88539aaddc3e36a33ce787ea2337300cf5e215155bfe361291131e459191
 SHA512:
-  metadata.gz: 430fa83af6f8b1d9eb22fdf8edfbd2787c4a0609c6abf6626239a05132f1732bf9ecd8f0e6ad92cab7e53ffd234d0c3df5e626f06c332680a99b84849bdf7bc0
-  data.tar.gz: b17ccb8f7bf525aa5f21917c87932115a11ccb4058d79c81af741ee985203c97d326b3c7cd8960819c4a93f5d088a8d0b39e1f9bbef108e2619e7114e621f74a
+  metadata.gz: d88a7ce64b6a0c52d1819450197d8d67d6c82c812596e79e2630c046e5851d14203fdd4014501de3478ba3f474a1ce5f0787b8168e6198e77a2992bf67564b5b
+  data.tar.gz: 7b67641d6f8af42b1ce43973c0e48cae31aac1d78a41e4d46e07199bb1f4cfd4b30b954583d73c9016c8bb283788dbdf9ceb2b8644c97d821297ddcc00c24910

data/README.md

@@ -12,6 +12,24 @@
   Includes durable consumers, backpressure, retries, <strong>DLQ</strong>, optional <strong>Inbox/Outbox</strong>, and <strong>overlap-safe stream provisioning</strong>
 </p>
 
+<p align="center">
+  <a href="https://github.com/attaradev/jetstream_bridge/actions/workflows/ci.yml">
+    <img src="https://github.com/attaradev/jetstream_bridge/actions/workflows/ci.yml/badge.svg" alt="CI Status"/>
+  </a>
+  <a href="https://codecov.io/gh/attaradev/jetstream_bridge">
+    <img src="https://codecov.io/gh/attaradev/jetstream_bridge/branch/main/graph/badge.svg" alt="Coverage Status"/>
+  </a>
+  <a href="https://rubygems.org/gems/jetstream_bridge">
+    <img src="https://img.shields.io/gem/v/jetstream_bridge.svg" alt="Gem Version"/>
+  </a>
+  <a href="https://rubygems.org/gems/jetstream_bridge">
+    <img src="https://img.shields.io/gem/dt/jetstream_bridge.svg" alt="Downloads"/>
+  </a>
+  <a href="LICENSE">
+    <img src="https://img.shields.io/badge/license-MIT-blue.svg" alt="License"/>
+  </a>
+</p>
+
 <p align="center">
   <a href="#-features">Features</a> •
   <a href="#-install">Install</a> •
@@ -35,7 +53,7 @@
 * ⚡️ **Eager-loaded models** via Railtie (production)
 * 📊 Configurable logging with sensible defaults
 
-### Production-Ready Features (v2.10+)
+### Production-Ready Features
 
 * 🏥 **Health checks** - Monitor NATS connection and stream status
 * 🔄 **Auto-reconnection** - Automatic recovery from connection failures
@@ -53,7 +71,7 @@
 
 ```ruby
 # Gemfile
-gem "jetstream_bridge", "~> 2.10"
+gem "jetstream_bridge", "~> 3.0"
 ```
 
 ```bash
@@ -335,7 +353,7 @@ health = JetstreamBridge.health_check
 #   connected_at: "2025-11-22T20:00:00Z",
 #   stream: { exists: true, name: "...", ... },
 #   config: { env: "production", ... },
-#   version: "2.10.0"
+#   version: "3.0.0"
 # }
 
 # Force-connect & ensure topology at boot or in a check

data/lib/generators/jetstream_bridge/health_check/health_check_generator.rb

@@ -13,8 +13,8 @@ module JetstreamBridge
       end
 
       def add_route
-        route_content = "  # JetStream Bridge health check endpoint\n" \
-                       "  get '/health/jetstream', to: 'jetstream_health#show'"
+        route_content = "  # JetStream Bridge health check endpoint\n  " \
+                        "get '/health/jetstream', to: 'jetstream_health#show'"
 
         if File.exist?('config/routes.rb')
           inject_into_file 'config/routes.rb', after: /Rails\.application\.routes\.draw do\n/ do
@@ -28,11 +28,11 @@ module JetstreamBridge
       end
 
       def show_usage
-        say "\n" + '=' * 70, :green
+        say "\n#{'=' * 70}", :green
         say 'Health Check Endpoint Created!', :green
         say '=' * 70, :green
         say "\nThe health check endpoint is now available at:"
-        say "  GET /health/jetstream", :cyan
+        say '  GET /health/jetstream', :cyan
         say "\nExample response:"
         say <<~EXAMPLE, :white
           {
@@ -54,10 +54,10 @@ module JetstreamBridge
           }
         EXAMPLE
         say "\nUse this endpoint for:"
-        say "  • Kubernetes liveness/readiness probes", :white
-        say "  • Docker health checks", :white
-        say "  • Monitoring and alerting", :white
-        say "  • Load balancer health checks", :white
+        say '  • Kubernetes liveness/readiness probes', :white
+        say '  • Docker health checks', :white
+        say '  • Monitoring and alerting', :white
+        say '  • Load balancer health checks', :white
         say '=' * 70, :green
       end
     end

data/lib/generators/jetstream_bridge/migrations/templates/create_jetstream_inbox_events.rb.erb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-class CreateJetstreamInboxEvents < ActiveRecord::Migration[7.0]
+class CreateJetstreamInboxEvents < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
   def change
     create_table :jetstream_inbox_events do |t|
       t.string   :event_id                              # preferred dedupe key

data/lib/generators/jetstream_bridge/migrations/templates/create_jetstream_outbox_events.rb.erb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-class CreateJetstreamOutboxEvents < ActiveRecord::Migration[7.0]
+class CreateJetstreamOutboxEvents < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
   def change
     create_table :jetstream_outbox_events do |t|
       t.string  :event_id, null: false

data/lib/jetstream_bridge/consumer/consumer.rb

@@ -28,7 +28,7 @@ module JetstreamBridge
       @idle_backoff  = IDLE_SLEEP_SECS
       @running       = true
       @shutdown_requested = false
-      @jts           = Connection.connect!
+      @jts = Connection.connect!
 
       ensure_destination!
 
@@ -162,22 +162,20 @@ module JetstreamBridge
     def drain_inflight_messages
       return unless @psub
 
-      Logging.info("Draining in-flight messages...", tag: 'JetstreamBridge::Consumer')
+      Logging.info('Draining in-flight messages...', tag: 'JetstreamBridge::Consumer')
       # Process any pending messages with a short timeout
       5.times do
-        begin
-          msgs = @psub.fetch(@batch_size, timeout: 1)
-          break if msgs.nil? || msgs.empty?
-
-          msgs.each { |m| process_one(m) }
-        rescue NATS::Timeout, NATS::IO::Timeout
-          break
-        rescue StandardError => e
-          Logging.warn("Error draining messages: #{e.class} #{e.message}", tag: 'JetstreamBridge::Consumer')
-          break
-        end
+        msgs = @psub.fetch(@batch_size, timeout: 1)
+        break if msgs.nil? || msgs.empty?
+
+        msgs.each { |m| process_one(m) }
+      rescue NATS::Timeout, NATS::IO::Timeout
+        break
+      rescue StandardError => e
+        Logging.warn("Error draining messages: #{e.class} #{e.message}", tag: 'JetstreamBridge::Consumer')
+        break
       end
-      Logging.info("Drain complete", tag: 'JetstreamBridge::Consumer')
+      Logging.info('Drain complete', tag: 'JetstreamBridge::Consumer')
     rescue StandardError => e
       Logging.error("Drain failed: #{e.class} #{e.message}", tag: 'JetstreamBridge::Consumer')
     end

data/lib/jetstream_bridge/consumer/message_processor.rb

@@ -79,7 +79,7 @@ module JetstreamBridge
       Oj.load(data, mode: :strict)
     rescue Oj::ParseError => e
       dlq_success = @dlq.publish(msg, ctx,
-                                  reason: 'malformed_json', error_class: e.class.name, error_message: e.message)
+                                 reason: 'malformed_json', error_class: e.class.name, error_message: e.message)
       if dlq_success
         msg.ack
         Logging.warn(
@@ -106,7 +106,7 @@ module JetstreamBridge
       )
     rescue *UNRECOVERABLE_ERRORS => e
       dlq_success = @dlq.publish(msg, ctx,
-                                  reason: 'unrecoverable', error_class: e.class.name, error_message: e.message)
+                                 reason: 'unrecoverable', error_class: e.class.name, error_message: e.message)
       if dlq_success
         msg.ack
         Logging.warn(
@@ -130,9 +130,9 @@ module JetstreamBridge
       if ctx.deliveries >= max_deliver
         # Only ACK if DLQ publish succeeds
         dlq_success = @dlq.publish(msg, ctx,
-                                    reason: 'max_deliver_exceeded',
-                                    error_class: error.class.name,
-                                    error_message: error.message)
+                                   reason: 'max_deliver_exceeded',
+                                   error_class: error.class.name,
+                                   error_message: error.message)
 
         if dlq_success
           msg.ack

data/lib/jetstream_bridge/core/connection.rb

@@ -86,9 +86,7 @@ module JetstreamBridge
 
     # Public API for getting connection timestamp
     # @return [Time, nil] timestamp when connection was established
-    def connected_at
-      @connected_at
-    end
+    attr_reader :connected_at
 
     private
 

data/lib/jetstream_bridge/core/connection_factory.rb

@@ -90,9 +90,7 @@ module JetstreamBridge
           jts = client.jetstream
 
           # Ensure JetStream responds to #nc for compatibility
-          unless jts.respond_to?(:nc)
-            jts.define_singleton_method(:nc) { client }
-          end
+          jts.define_singleton_method(:nc) { client } unless jts.respond_to?(:nc)
 
           jts
         end

data/lib/jetstream_bridge/core/debug_helper.rb

@@ -8,22 +8,22 @@ module JetstreamBridge
     class << self
       # Print comprehensive debug information about the current setup
       def debug_info
-      info = {
-        config: config_debug,
-        connection: connection_debug,
-        stream: stream_debug,
-        health: JetstreamBridge.health_check
-      }
+        info = {
+          config: config_debug,
+          connection: connection_debug,
+          stream: stream_debug,
+          health: JetstreamBridge.health_check
+        }
 
-      Logging.info("=== JetStream Bridge Debug Info ===", tag: 'JetstreamBridge::Debug')
-      info.each do |section, data|
-        Logging.info("#{section.to_s.upcase}:", tag: 'JetstreamBridge::Debug')
-        log_hash(data, indent: 2)
-      end
-      Logging.info("=== End Debug Info ===", tag: 'JetstreamBridge::Debug')
+        Logging.info('=== JetStream Bridge Debug Info ===', tag: 'JetstreamBridge::Debug')
+        info.each do |section, data|
+          Logging.info("#{section.to_s.upcase}:", tag: 'JetstreamBridge::Debug')
+          log_hash(data, indent: 2)
+        end
+        Logging.info('=== End Debug Info ===', tag: 'JetstreamBridge::Debug')
 
-      info
-    end
+        info
+      end
 
       private
 
@@ -34,9 +34,21 @@ module JetstreamBridge
           app_name: cfg.app_name,
           destination_app: cfg.destination_app,
           stream_name: cfg.stream_name,
-          source_subject: (cfg.source_subject rescue 'ERROR'),
-          destination_subject: (cfg.destination_subject rescue 'ERROR'),
-          dlq_subject: (cfg.dlq_subject rescue 'ERROR'),
+          source_subject: begin
+            cfg.source_subject
+          rescue StandardError
+            'ERROR'
+          end,
+          destination_subject: begin
+            cfg.destination_subject
+          rescue StandardError
+            'ERROR'
+          end,
+          dlq_subject: begin
+            cfg.dlq_subject
+          rescue StandardError
+            'ERROR'
+          end,
           durable_name: cfg.durable_name,
           nats_urls: cfg.nats_urls,
           max_deliver: cfg.max_deliver,

data/lib/jetstream_bridge/core/duration.rb

@@ -72,7 +72,7 @@ module JetstreamBridge
       case default_unit
       when :auto
         # Preserve existing heuristic for compatibility but log deprecation warning
-        if defined?(Logging) && num > 0 && num < 1_000
+        if defined?(Logging) && num.positive? && num < 1_000
           Logging.debug(
             "Duration :auto heuristic treating #{num} as seconds. " \
             "Consider specifying default_unit: :s or :ms for clarity.",

data/lib/jetstream_bridge/core/retry_strategy.rb

@@ -40,8 +40,9 @@ module JetstreamBridge
           sleep delay
         end
       end
-    rescue => e
+    rescue StandardError => e
       raise unless retryable?(e)
+
       raise RetryExhausted, "Failed after #{attempts} attempts: #{e.message}"
     end
 

data/lib/jetstream_bridge/models/event_envelope.rb

@@ -109,7 +109,10 @@ module JetstreamBridge
       def deep_freeze(obj)
         case obj
         when Hash
-          obj.each { |k, v| deep_freeze(k); deep_freeze(v) }
+          obj.each do |k, v|
+            deep_freeze(k)
+            deep_freeze(v)
+          end
           obj.freeze
         when Array
           obj.each { |item| deep_freeze(item) }

data/lib/jetstream_bridge/models/subject.rb

@@ -53,7 +53,7 @@ module JetstreamBridge
       end
 
       def ==(other)
-        other.is_a?(Subject) ? @value == other.value : @value == other.to_s
+        @value == (other.is_a?(Subject) ? other.value : other.to_s)
       end
 
       alias eql? ==

data/lib/jetstream_bridge/publisher/publisher.rb

@@ -96,8 +96,8 @@ module JetstreamBridge
     # ---- /Outbox path ----
 
     # Retry using strategy pattern
-    def with_retries
-      @retry_strategy.execute(context: 'Publisher') { yield }
+    def with_retries(&block)
+      @retry_strategy.execute(context: 'Publisher', &block)
     rescue RetryStrategy::RetryExhausted => e
       log_error(false, e)
     end

data/lib/jetstream_bridge/railtie.rb

@@ -23,11 +23,9 @@ module JetstreamBridge
     initializer 'jetstream_bridge.validate_config', after: :load_config_initializers do |app|
       if Rails.env.development? || Rails.env.test?
         app.config.after_initialize do
-          begin
-            JetstreamBridge.config.validate! if JetstreamBridge.config.destination_app
-          rescue JetstreamBridge::ConfigurationError => e
-            Rails.logger.warn "[JetStream Bridge] Configuration warning: #{e.message}"
-          end
+          JetstreamBridge.config.validate! if JetstreamBridge.config.destination_app
+        rescue JetstreamBridge::ConfigurationError => e
+          Rails.logger.warn "[JetStream Bridge] Configuration warning: #{e.message}"
         end
       end
     end
@@ -35,7 +33,7 @@ module JetstreamBridge
     # Add console helper methods
     console do
       Rails.logger.info "[JetStream Bridge] Loaded v#{JetstreamBridge::VERSION}"
-      Rails.logger.info "[JetStream Bridge] Use JetstreamBridge.health_check to check status"
+      Rails.logger.info '[JetStream Bridge] Use JetstreamBridge.health_check to check status'
     end
 
     # Load rake tasks

data/lib/jetstream_bridge/tasks/install.rake

@@ -87,8 +87,8 @@ namespace :jetstream_bridge do
     begin
       jts = JetstreamBridge.ensure_topology!
       puts '✓ Successfully connected to NATS'
-      puts "✓ JetStream is available"
-      puts "✓ Stream topology ensured"
+      puts '✓ JetStream is available'
+      puts '✓ Stream topology ensured'
 
       # Check if we can get account info
       info = jts.account_info

data/lib/jetstream_bridge/version.rb

@@ -4,5 +4,5 @@
 #
 # Version constant for the gem.
 module JetstreamBridge
-  VERSION = '3.0.0'
+  VERSION = '3.0.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jetstream_bridge
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.0.1
 platform: ruby
 authors:
 - Mike Attara
@@ -16,7 +16,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '6.0'
+        version: 7.1.5.2
     - - "<"
       - !ruby/object:Gem::Version
         version: '8.0'
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '6.0'
+        version: 7.1.5.2
     - - "<"
       - !ruby/object:Gem::Version
         version: '8.0'
@@ -36,7 +36,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '6.0'
+        version: 7.1.5.2
     - - "<"
       - !ruby/object:Gem::Version
         version: '8.0'
@@ -46,10 +46,24 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '6.0'
+        version: 7.1.5.2
     - - "<"
       - !ruby/object:Gem::Version
         version: '8.0'
+- !ruby/object:Gem::Dependency
+  name: mutex_m
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: nats-pure
   requirement: !ruby/object:Gem::Requirement
@@ -84,20 +98,6 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '4.0'
-- !ruby/object:Gem::Dependency
-  name: mutex_m
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 description: |-
   Publisher/Consumer utilities for NATS JetStream with environment-scoped subjects,
   overlap guards, DLQ routing, retries/backoff, and optional Inbox/Outbox patterns.