errorgap 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e386d7d085d300b5364be4f27918288b5b62509895afe19ed525afee51c3f92a
-  data.tar.gz: 158ae2443f061e6ff63487dd0f4fe061a54ea4ba7933d65305743d42f84fcb1f
+  metadata.gz: cabe7a082d767651c4e56f067d77fa21486056cde995842eb4ef7f8531adbf00
+  data.tar.gz: 10cc33121d252ce987eb92ccd2e57301ea4399bafa42c2b8cada4ab0dd4d1f63
 SHA512:
-  metadata.gz: f43d61f09d546dc69c6a837e2ae7fee478589f510990e2740119f1945236b179feac4364c70e73459da787e4110757398cb876e44976d4709085cd8303887f96
-  data.tar.gz: a20b31f480acfe98543fb8a79d79b1d14427461a9b08d5847249dfa88b67633bcecac0af213d089b1d383d85032ef2f1cb7bb7de51befc108f5cffd1fa9e9c8a
+  metadata.gz: 283fcdf5b95bcbf34a175b2e9249c5e2a5b4c16f46bedfe9852f088214ada0e6aa022d84f816a4c692b36291c9a3d8a0f099ee0f18beb15cf5c856d014ecdb14
+  data.tar.gz: a66f945ca3d16ee996aeec660dc24351baa3799e7b7404ce209202e98a3c10e6df7eac940c4bdea82e4b7ca86391c98ed3b718c174c0cfe5499485cfb14bd1b2

data/CHANGELOG.md

@@ -5,6 +5,27 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.3.0] - 2026-07-16
+
+### Fixed
+
+- APM span collection now works when `config.apm_enabled` is set in a Rails
+  app initializer (the common case). The `sql.active_record` subscriber was
+  previously installed only if APM was already enabled during railtie
+  initialization, which runs before `config/initializers` — so transactions
+  shipped without spans and the time breakdown showed everything as
+  "App / other".
+
+### Added
+
+- In-app backtrace frames now include a source excerpt (`source: {start_line,
+  lines}`, ±6 lines around the failing line) so the errorgap UI can render
+  backtrace source without a repository integration.
+
+- View-rendering time is now recorded as a `view` span (from Rails'
+  `process_action.action_controller` `view_runtime`), populating the
+  "View rendering" segment of the route time breakdown.
+
 ## [0.2.0] - 2026-07-10
 
 ### Added
@@ -27,5 +48,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   timing and DB query spans, sampled by `config.apm_sample_rate`.
 - Authentication via the `X-Errorgap-Project-Key` header.
 
+[0.3.0]: https://github.com/errorgaphq/errorgap-ruby/compare/v0.2.0...v0.3.0
 [0.2.0]: https://github.com/errorgaphq/errorgap-ruby/compare/v0.1.0...v0.2.0
 [0.1.0]: https://github.com/errorgaphq/errorgap-ruby/releases/tag/v0.1.0

data/lib/errorgap/notice.rb

@@ -4,6 +4,13 @@ require "time"
 
 module Errorgap
   class Notice
+    # Lines of context shipped either side of the failing line for in-app
+    # frames, and a cap on how many frames include an excerpt so deep
+    # backtraces don't inflate the payload.
+    SOURCE_RADIUS = 6
+    MAX_SOURCE_FRAMES = 25
+    MAX_SOURCE_LINE_LENGTH = 400
+
     def self.from_exception(error, configuration:, context: {}, environment: {}, session: {}, params: {})
       new(
         error: error,
@@ -54,18 +61,47 @@ module Errorgap
     end
 
     def backtrace_frames
+      source_frames = 0
       Array(@error.backtrace).map.with_index do |line, index|
         file, line_number, function = parse_backtrace_line(line)
-        {
+        frame = {
           file: relative_file(file),
           line: line_number,
           function: function,
           in_app: in_app?(file),
           index: index
-        }.compact
+        }
+        if in_app?(file) && source_frames < MAX_SOURCE_FRAMES &&
+           (source = source_excerpt(file, line_number))
+          frame[:source] = source
+          source_frames += 1
+        end
+        frame.compact
       end
     end
 
+    # Reads the lines around the failing line so the server can show source
+    # without needing repository access. Returns nil when the file cannot be
+    # read (precompiled deploys, eval'd code, template paths).
+    def source_excerpt(file, line_number)
+      return nil unless line_number && line_number >= 1
+      return nil unless file && File.file?(file) && File.readable?(file)
+
+      lines = File.readlines(file)
+      return nil if lines.empty? || line_number > lines.length
+
+      start_line = [line_number - SOURCE_RADIUS, 1].max
+      end_line = [line_number + SOURCE_RADIUS, lines.length].min
+      {
+        start_line: start_line,
+        lines: lines[(start_line - 1)..(end_line - 1)].map do |text|
+          text.chomp[0, MAX_SOURCE_LINE_LENGTH]
+        end
+      }
+    rescue StandardError
+      nil
+    end
+
     def parse_backtrace_line(line)
       match = line.match(/\A(.+?):(\d+)(?::in `(.*)')?\z/)
       return [line, nil, nil] unless match

data/lib/errorgap/rails/railtie.rb

@@ -18,7 +18,11 @@ module Errorgap
       end
 
       initializer "errorgap.install_span_collector" do
-        SpanCollector.install if Errorgap.configuration.apm_enabled
+        # Unconditional: apps enable APM in config/initializers, which runs
+        # after railtie initializers, so the flag cannot be checked here. The
+        # subscribers no-op unless the middleware starts a per-request
+        # collection, which does check the flag.
+        SpanCollector.install
       end
 
       generators do

data/lib/errorgap/span_collector.rb

@@ -14,9 +14,16 @@ module Errorgap
     SKIP_NAMES = %w[SCHEMA EXPLAIN CACHE].freeze
 
     class << self
-      # Call once at boot (from Railtie) when APM is enabled.
+      # Call once at boot (from Railtie). Subscribers only record while a
+      # transaction is being collected (the middleware starts one per request
+      # when APM is enabled), so installing unconditionally is safe — and
+      # required, since app initializers that enable APM run after railtie
+      # initializers.
       def install
         return unless defined?(ActiveSupport::Notifications)
+        return if @installed
+
+        @installed = true
 
         ActiveSupport::Notifications.subscribe("sql.active_record") do |*, payload|
           next unless Thread.current[THREAD_KEY]
@@ -28,6 +35,17 @@ module Errorgap
 
           store << Span.new(kind: "db", sql: sql, duration_ms: duration_ms.round(3))
         end
+
+        # Rails reports view_runtime with database time already subtracted, so
+        # this does not double count the db spans above.
+        ActiveSupport::Notifications.subscribe("process_action.action_controller") do |*, payload|
+          next unless Thread.current[THREAD_KEY]
+
+          view_ms = payload[:view_runtime]
+          next unless view_ms&.positive?
+
+          store << Span.new(kind: "view", duration_ms: view_ms.round(3))
+        end
       end
 
       def start

data/lib/errorgap/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Errorgap
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: errorgap
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Errorgap
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-07-11 00:00:00.000000000 Z
+date: 2026-07-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -38,6 +38,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '5.0'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.1'
 description: Captures Ruby exceptions and sends them to a Errorgap server.
 email:
 - support@errorgap.com