pg_reports 0.6.1 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 77275298b8ef6a1dba986e9b47849d1685e2d09cacdd64e3b086b20ab403574c
-  data.tar.gz: 893da425321112adf5d4f9944fb72028898c77119ec95ad92d27c21681a80a0e
+  metadata.gz: a61a108ff2ba498edaa622cfc11f3f746d56a54f1b964ced3e941516c2eaaec4
+  data.tar.gz: 9e0d3da7d606309d51c64e5daf96e66875a6645f23649433d98001209be175d7
 SHA512:
-  metadata.gz: c99a83908899a5896734b7025b4b593dad0c0d4045b9578e1912ccb332aa9f29dcb0c1e79fe69a403f4a248e99a540fc1b0627af44d7ddc1c1c889c738c1ffa7
-  data.tar.gz: 4ff3fe8ae5a907dee4a6eb9186adb1e9819d49e62c84c7447c49999dd2c4f09d01cd962711461437f18f1853ba1c3ea9b226af5a822a373034f61f93521ef1f5
+  metadata.gz: f25452a5db23c9288b02334d4c900b838d063e8d754695d0047f931b24d1ec7c2b6a98fbfea3898277df0c124a3cbc52ed40f22e325adc85af503a6d9db43351
+  data.tar.gz: 046aca0f21ec77c649f3a5a0658aac92ba4c44bedbd15ec979838b183574b16fe67fcaebf8e7b6b3727de9f082712b7f67ed321c0f8c89723cffbc59cd93e845

data/CHANGELOG.md

@@ -7,6 +7,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.6.2] - 2026-04-25
+
+### Added
+
+- **Native Rails QueryLogs source_location support** — `AnnotationParser` now recognizes the `source_location` tag emitted by `ActiveRecord::QueryLogs` and splits it into separate `:file` / `:line` fields for the dashboard's source column. Values are URL-decoded first, since Rails CGI-escapes tag values (so `%2F.../%3A19` becomes `/.../foo.rb:19`).
+
+### Fixed
+
+- **`counter_cache_issues` crashed/misreported on Rails 7.1+ Hash form.** `belongs_to :user, counter_cache: :col` is internally normalized to `{active: true, column: :col}` on Rails 7.1+, but the helper only handled `true` / Symbol / String — so the column came out as `Hash#inspect` (`{active: true, column: "usage_count"}`) and the suggested migration was malformed. `counter_cache_column_name` now handles all four forms (Boolean, Symbol, String, Hash with `:column` key, Hash without `:column`).
+- **`polymorphic_without_index` crashed with `NoMethodError: undefined method '&' for an instance of String`** when a table had an expression index (e.g. `CREATE INDEX ON x (LOWER(email))`). PostgreSQL returns `IndexDefinition#columns` as a String for expression indexes, not an Array. Both the polymorphic check and `coverage_label` now filter out non-array indexes.
+
+### Changed
+
+- **README simplified** — full reports listing moved to [docs/reports.md](docs/reports.md), long sections (EXPLAIN ANALYZE, SQL Query Monitor, Connection pool analytics, IDE integration, Telegram delivery, raw query execution, source tracking) collapsed into `<details>` blocks. From 582 lines to ~340.
+- **Query source tracking** documentation now leads with native `ActiveRecord::QueryLogs` (Rails 7.0+) including a `source_location` lambda example. Marginalia mentioned only as the option for Rails < 7.0.
+
 ## [0.6.1] - 2026-04-24
 
 ### Added

data/README.md

@@ -131,25 +131,40 @@ end
 </details>
 
 <details>
-<summary><strong>Query source tracking (Marginalia / Rails query logs)</strong></summary>
+<summary><strong>Query source tracking (Rails query logs)</strong></summary>
 
-PgReports parses query annotations to show **where queries originated**.
+PgReports parses query annotations to show **where queries originated**. On Rails 7.0+ use the built-in `ActiveRecord::QueryLogs` (no extra gem needed). On older Rails, install [Marginalia](https://github.com/basecamp/marginalia) — PgReports auto-detects both formats.
 
-[Marginalia](https://github.com/basecamp/marginalia):
+Minimal setup — adds controller/action:
 
 ```ruby
-gem "marginalia"
+# config/application.rb
+config.active_record.query_log_tags_enabled = true
+config.active_record.query_log_tags = [:controller, :action]
 ```
 
-Rails 7+ query logs:
+To also surface **file path and line number** (so source links jump to the actual call site, not just the controller), add a custom `source_location` lambda that walks `caller_locations` and skips gem/framework frames:
 
 ```ruby
 # config/application.rb
 config.active_record.query_log_tags_enabled = true
-config.active_record.query_log_tags = [:controller, :action]
+config.active_record.query_log_tags = [
+  :controller,
+  :action,
+  :job,
+  {
+    source_location: -> {
+      ignore = %r{/(gems|active_record|active_support|active_model|railties|
+                    action_controller|action_view|action_pack|action_dispatch|
+                    rack|core_ext|relation|associations|scoping|connection_adapters)/}x
+      loc = caller_locations.find { |l| !l.path.match?(ignore) }
+      "#{loc.path}:#{loc.lineno}" if loc
+    }
+  }
+]
 ```
 
-Either form is auto-detected; controller/action and file:line appear in the **source** column on report rows.
+PgReports recognizes the `source_location` tag and splits it into file and line for the **source** column.
 
 </details>
 
@@ -284,7 +299,10 @@ The Export dropdown includes **Copy Prompt** (visible on actionable reports). It
 
 </details>
 
-## Telegram
+<details>
+<summary><strong>Telegram delivery</strong></summary>
+
+Get a bot token from [@BotFather](https://t.me/BotFather) and your chat ID from [@userinfobot](https://t.me/userinfobot), then:
 
 ```ruby
 PgReports.configure do |config|
@@ -296,7 +314,9 @@ PgReports.slow_queries.send_to_telegram
 PgReports.health_report.send_to_telegram_as_file
 ```
 
-Get a bot token from [@BotFather](https://t.me/BotFather) and your chat ID from [@userinfobot](https://t.me/userinfobot).
+Reports under ~50 rows go as a message; larger ones are sent as a file attachment.
+
+</details>
 
 ## Development
 

data/lib/pg_reports/annotation_parser.rb

@@ -1,10 +1,12 @@
 # frozen_string_literal: true
 
+require "cgi"
+
 module PgReports
   # Parses SQL query comments to extract source location and metadata
   # Supports:
   # - Marginalia format: /*application:myapp,controller:users,action:index*/
-  # - Rails QueryLogs: /*action='index',controller='users'*/
+  # - Rails QueryLogs: /*action='index',controller='users',source_location='app/foo.rb:42'*/
   #
   module AnnotationParser
     class << self
@@ -32,6 +34,16 @@ module PgReports
           end
         end
 
+        # Rails QueryLogs custom tag — Rails URL-encodes tag values (CGI.escape) to keep
+        # SQL comments safe, so "/" → "%2F" and ":" → "%3A". Decode then split into :file/:line.
+        if result[:source_location] && !result[:file]
+          decoded = CGI.unescape(result[:source_location].to_s)
+          if (match = decoded.match(%r{^(.+):(\d+)$}))
+            result[:file] = match[1]
+            result[:line] = match[2]
+          end
+        end
+
         result
       end
 

data/lib/pg_reports/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PgReports
-  VERSION = "0.6.1"
+  VERSION = "0.6.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pg_reports
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.6.2
 platform: ruby
 authors:
 - Eldar Avatov