[ajaynomics]

Description

Under Rails 8.1 with eager_load off (rake tasks, console, most dev boots), sentry-rails makes Rails print:

:action_dispatch_response was loaded before application initialization

The railtie referenced ActionController::Live inside the :action_controller load hook to prepend StreamingReporter. That reference autoloads ActionController::Live, which loads ActionDispatch::Response too early, and Rails warns.

Rails 8.1.4 added an :action_controller_live load hook. This PR prepends the reporter from that hook, so nothing touches ActionController::Live until the app does. On older Rails the direct reference stays.

Tested with a new isolated spec, sentry-rails/spec/isolated/streaming_reporter_spec.rb. It asserts the reporter is still prepended, and boots a bare app in a child process to assert initialization leaves ActionController::Live unloaded. The second example fails on master and passes here. spec/isolated and spec/sentry pass on the ruby-3.4_rails-8.1.3 cell; RuboCop is clean on both files.

• resolves: sentry-rails triggers "Prematurely executing load hooks" warning for :action_dispatch_response on Rails 8.2 #2912

[sl0thentr0py]

thx @ajaynomics !

[codecov]

Codecov Report

❌ Patch coverage is 16.66667% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.19%. Comparing base (7d10c02) to head (1e7e1b1).
⚠️ Report is 208 commits behind head on master.

Files with missing lines	Patch %	Lines
sentry-rails/lib/sentry/rails/railtie.rb	16.66%	5 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3109      +/-   ##
==========================================
+ Coverage   90.13%   90.19%   +0.05%     
==========================================
  Files         132      155      +23     
  Lines        5301     6455    +1154     
==========================================
+ Hits         4778     5822    +1044     
- Misses        523      633     +110     

Components	Coverage Δ
sentry-ruby	97.13% <ø> (-0.16%)	⬇️
sentry-rails	57.40% <ø> (+1.89%)	⬆️
sentry-sidekiq	95.08% <ø> (+0.03%)	⬆️
sentry-resque	94.44% <ø> (ø)
sentry-delayed_job	94.68% <ø> (ø)
sentry-opentelemetry	99.55% <100.00%> (+0.24%)	⬆️

Files with missing lines	Coverage Δ
sentry-rails/lib/sentry/rails/railtie.rb	24.17% <16.66%> (-1.44%)	⬇️

... and 72 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.