So it's only available through the error reporter right? I think we still don't have document for it feature yet. Do you think we can add a Reporting Errors page under the Digging Deeper section of Rails guide?

So it's only available through the error reporter right? I

Sorry, I don't think I understand your question.

Do you think we can add a Reporting Errors page under the Digging Deeper section of Rails guide?

Maybe?

I thought it'd be a new API in the Railtie like I proposed here.

Capturing it with the executor will work as well, but it also means the SDKs need to opt-in the error reporter to get it (which I think will encourage more SDKs to opt-in).

However, because it's reported via the execution wrapper, we only get unhandled_error.active_support as the source.

I hope Rails can provide more accurate source whenever possible, which in this case seems to be possible?

Rails.error_reporter.record(source: "rails_runner.railties") do
  # runner execution
end

Wdyt?

Wdyt?

I don't think it's necessary. source makes sense for handled errors you might want to ignore. Once an error isn't handled, I don't see a use case for ignoring them.

Also, for the user reading the report, the backtrace should make the origin of the error obvious.

Sorry I should've explained: I'm also thinking about using the source as a tag of the error events. In Sentry, tagging is one of the most commonly used filtering methods. The same concept also exists on some other platforms like Appsignal.

With the source tag, users can easily browse errors based on their sources (e.g. all Rails runner errors, all library errors), which will be a great addition imo. That's why I want to have accurate source info.

Ok, I think you convinced me again, so I added a distinct source: for each wrap. Let me know what you think.

Thanks for making the change, I think it'll be super helpful for our users!
Do you know when this will likely go out? I assume the end of the year like last year? I'll schedule a time to write the document for this on Rails guide and Sentry's doc.

@casperisfine After this is merged, I'll push a rails-main branch to the SDK based on our prev discussions and start finding people to give it a try. And if there will be another pre-release version that contains these changes it'd be very helpful.
I'll also start drafting relevant document for this feature.