foam-ruby 0.1.0.alpha12 → 0.1.0.alpha13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e0cd5d2699e7c0780b212b06e4e272f4e3510253e64df195ebbee3868bc32257
-  data.tar.gz: 8a1625c3cd8c95898dc6f62cadc80af2a140e9b69671f542a2f2e1ed767b2aa6
+  metadata.gz: d499bfb9b269432b8a581f296cc917118d8da08c18ec8bcd0804d099b93a4e46
+  data.tar.gz: 30ed5cf79368e4b044562f9eb8e5385e72af60bf6f91a53826f47aebe01816d4
 SHA512:
-  metadata.gz: c2b2862bed64c2bf2d9b6776c5969ae9f4424564c37d8094f8515eb3696d4be22397a5c56cc56841a28fdefc97529dcc3fa311dfb348d932370b22bf3a36d869
-  data.tar.gz: eefce9a6236a601b986a50ccc6fc4a6b4bd9c79ffb2ea042078c472c8c1f99cf786b1ea21ab10bf970f90d93e05af0c4c8ac65ab104e5011c5f88c6546d5b5fc
+  metadata.gz: 01ad0c424faf4ab05e52ecff9ddb5f5695ed0ed18006f6b53a58af7950f2aedc5f66f92c9af2bfa632a7ea7512a5d4864a8210dc627cabe02277921f6d20d2ee
+  data.tar.gz: 53732245e699e9198dbaf25c43426e0a9811dc3901b45a93dcdadbeb15b12195c95cb2f35d4d81676481f55844c2858ccb0eef8ae9331037801f65bf418f879e

data/lib/foam/ruby/otel_setup.rb

@@ -19,21 +19,47 @@ module Foam
 
       module_function
 
+      # -- public API (called from Foam::Ruby + Middleware) --
+
       def configure!(config)
+        require_instrumentations!
         configure_traces(config)
         configure_logs(config)
       end
 
+      def ensure_processor_attached!
+        return unless @foam_span_processor
+
+        current = OpenTelemetry.tracer_provider
+        return if current.equal?(@attached_provider)
+        return unless current.is_a?(OpenTelemetry::SDK::Trace::TracerProvider)
+
+        current.add_span_processor(@foam_span_processor)
+        @attached_provider = current
+      end
+
+      def ensure_log_processor_attached!
+        return unless @foam_log_processor
+
+        current = OpenTelemetry.logger_provider
+        return if current.equal?(@attached_logger_provider)
+        return unless current.is_a?(OpenTelemetry::SDK::Logs::LoggerProvider)
+
+        current.add_log_record_processor(@foam_log_processor)
+        @attached_logger_provider = current
+      end
+
+      # -- internal --
+
       def require_instrumentations!
-        INSTRUMENTATIONS.each do |gem_name|
-          require gem_name
+        INSTRUMENTATIONS.each do |name|
+          require name
         rescue LoadError
           nil
         end
       end
 
       def configure_traces(config)
-        require_instrumentations!
         @foam_span_processor = OpenTelemetry::SDK::Trace::Export::BatchSpanProcessor.new(
           OpenTelemetry::Exporter::OTLP::Exporter.new(
             endpoint: config.traces_endpoint,
@@ -41,9 +67,9 @@ module Foam
           )
         )
 
-        existing = OpenTelemetry.tracer_provider
-        if existing.is_a?(OpenTelemetry::SDK::Trace::TracerProvider)
-          existing.add_span_processor(@foam_span_processor)
+        provider = OpenTelemetry.tracer_provider
+        if provider.is_a?(OpenTelemetry::SDK::Trace::TracerProvider)
+          provider.add_span_processor(@foam_span_processor)
           install_instrumentations!
         else
           OpenTelemetry::SDK.configure do |c|
@@ -56,26 +82,6 @@ module Foam
         @attached_provider = OpenTelemetry.tracer_provider
       end
 
-      def install_instrumentations!
-        registry = OpenTelemetry::Instrumentation.registry
-        return unless registry
-
-        registry.each do |instrumentation|
-          instrumentation.install unless instrumentation.installed?
-        end
-      end
-
-      def ensure_processor_attached!
-        return unless @foam_span_processor
-
-        current = OpenTelemetry.tracer_provider
-        return if current.equal?(@attached_provider)
-        return unless current.is_a?(OpenTelemetry::SDK::Trace::TracerProvider)
-
-        current.add_span_processor(@foam_span_processor)
-        @attached_provider = current
-      end
-
       def configure_logs(config)
         @foam_log_processor = OpenTelemetry::SDK::Logs::Export::BatchLogRecordProcessor.new(
           OpenTelemetry::Exporter::OTLP::Logs::LogsExporter.new(
@@ -84,9 +90,9 @@ module Foam
           )
         )
 
-        existing = OpenTelemetry.logger_provider
-        if existing.is_a?(OpenTelemetry::SDK::Logs::LoggerProvider)
-          existing.add_log_record_processor(@foam_log_processor)
+        provider = OpenTelemetry.logger_provider
+        if provider.is_a?(OpenTelemetry::SDK::Logs::LoggerProvider)
+          provider.add_log_record_processor(@foam_log_processor)
         else
           resource = OpenTelemetry::SDK::Resources::Resource.create(
             "service.name" => config.service_name,
@@ -100,15 +106,10 @@ module Foam
         @attached_logger_provider = OpenTelemetry.logger_provider
       end
 
-      def ensure_log_processor_attached!
-        return unless @foam_log_processor
-
-        current = OpenTelemetry.logger_provider
-        return if current.equal?(@attached_logger_provider)
-        return unless current.is_a?(OpenTelemetry::SDK::Logs::LoggerProvider)
-
-        current.add_log_record_processor(@foam_log_processor)
-        @attached_logger_provider = current
+      def install_instrumentations!
+        OpenTelemetry::Instrumentation.registry&.each do |instrumentation|
+          instrumentation.install unless instrumentation.installed?
+        end
       end
     end
   end

data/lib/foam/ruby/version.rb

@@ -2,6 +2,6 @@
 
 module Foam
   module Ruby
-    VERSION = "0.1.0.alpha12"
+    VERSION = "0.1.0.alpha13"
   end
 end

data/lib/foam/ruby.rb

@@ -19,7 +19,7 @@ module Foam
         config.service_name = service_name if service_name
 
         unless config.valid?
-          warn "[foam-ruby] Missing required config — set OTEL_FOAM_COLLECTOR_BEARER_TOKEN and OTEL_SERVICE_NAME, " \
+          warn "[foam-ruby] Missing required config — set FOAM_API_TOKEN and OTEL_SERVICE_NAME, " \
                "or pass token: and service_name: to Foam::Ruby.init"
           return
         end
@@ -29,18 +29,15 @@ module Foam
         require_relative "ruby/middleware"
 
         OtelSetup.configure!(config)
-        insert_middleware!
-        schedule_logger_attach!
-        bridge_sentry!
+        setup_rails! if rails_app?
         @initialized = true
       rescue Exception => e # rubocop:disable Lint/RescueException
         warn "[foam-ruby] init failed: #{e.message}"
         @initialized = false
       end
 
-      # Ensures Foam's processors and logger broadcast are attached in the
-      # current process. Called by Middleware on the first request per worker
-      # PID — a single integer comparison on every subsequent request.
+      # Re-attaches Foam's processors and logger broadcast in the current
+      # process. Called by Middleware on first request per worker PID.
       # Public so non-Rails apps can call it from custom fork hooks.
       def ensure_worker_setup!
         return unless initialized?
@@ -93,45 +90,37 @@ module Foam
         false
       end
 
-      def schedule_logger_attach!
-        return unless defined?(::Rails) && ::Rails.respond_to?(:application) && ::Rails.application
-
-        ::Rails.application.config.after_initialize { attach_otel_logger! }
-      rescue Exception # rubocop:disable Lint/RescueException
-        nil
+      def rails_app?
+        defined?(::Rails) && ::Rails.respond_to?(:application) && ::Rails.application
       end
 
-      def insert_middleware!
-        return unless defined?(::Rails) && ::Rails.respond_to?(:application) && ::Rails.application
-
+      def setup_rails!
         ::Rails.application.middleware.insert(0, Foam::Ruby::Middleware)
+
+        ::Rails.application.config.after_initialize do
+          attach_otel_logger!
+          bridge_sentry!
+        end
       rescue Exception # rubocop:disable Lint/RescueException
         nil
       end
 
       def bridge_sentry!
-        return unless defined?(::Rails) && ::Rails.respond_to?(:application) && ::Rails.application
-
-        ::Rails.application.config.after_initialize do
-          next unless defined?(::Sentry)
-
-          ::Sentry.singleton_class.prepend(Module.new do
-            def capture_exception(exception, **options, &block)
-              Foam::Ruby.capture_exception(exception) if exception.is_a?(::Exception)
-              super
-            end
-          end)
-        end
+        return unless defined?(::Sentry)
+
+        ::Sentry.singleton_class.prepend(Module.new do
+          def capture_exception(exception, **options, &block)
+            Foam::Ruby.capture_exception(exception) if exception.is_a?(::Exception)
+            super
+          end
+        end)
       rescue Exception # rubocop:disable Lint/RescueException
         nil
       end
 
       def attach_otel_logger!
         return unless defined?(::Rails) && ::Rails.respond_to?(:logger) && ::Rails.logger
-
-        unless OpenTelemetry.logger_provider.is_a?(OpenTelemetry::SDK::Logs::LoggerProvider)
-          return
-        end
+        return unless OpenTelemetry.logger_provider.is_a?(OpenTelemetry::SDK::Logs::LoggerProvider)
 
         logger = ::Rails.logger
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foam-ruby
 version: !ruby/object:Gem::Version
-  version: 0.1.0.alpha12
+  version: 0.1.0.alpha13
 platform: ruby
 authors:
 - Foam