pg_reports 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 76df6e762c2183af15f4409d137e5705f97249922d6c5c5521ba6a7903d3e3f9
-  data.tar.gz: 7e52e17a4fa3b3961a1b3d9c5c64174376e38e0d97732017d3a248df8c19528f
+  metadata.gz: ebdc855958933d677530a6bee64fe5534e3b13244e4b851ee3cd06b37e1df10b
+  data.tar.gz: 1fae6baba2619b57b10ad86ef37678a34b279c49a7cf0f014810f4b167c2ae80
 SHA512:
-  metadata.gz: eed06fbb5e836b8ecd1452319e4a0af13ffe750525953fec4ee4aed49e9d04e47df6f8404afe29162950885b41cccdaf4cd1e0ec22f2b87917ee64d931245fc4
-  data.tar.gz: 3c6d3b1f7c04781428eaf8d8460177ef4e243d114ad88c83b4419ab15817214804184247a0074dd5f69775d7484205969830dd77169b6de52dc35cda8bda7e24
+  metadata.gz: 7b2960a2f1df123331f8cd8d41d258fcd61f45b118451bac7859e890a5fde822266e7ab77e60a5820bf1856548ebf3ec2fff77a3f7938dc8913eca53cb087582
+  data.tar.gz: 8bfeeff753903a9a29329221108628ae2119759f859dd01e2c1395a9ae317a956556fb115bfa28b046b859e784ea42fc0b51df7fee56b7a6a6aef1674335a96a

data/CHANGELOG.md

@@ -5,6 +5,23 @@ 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.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.2.2] - 2026-01-28
+
+### Added
+
+- `fake_source_data` configuration option - enable via `PG_REPORTS_FAKE_SOURCE_DATA=true` env variable or in initializer
+- Support for short controller#action format in source links (e.g., `posts#index` → `app/controllers/posts_controller.rb`)
+
+### Changed
+
+- Fake source data moved to separate partial file for cleaner code organization
+- IDE link click handling improved with event delegation in capture phase
+
+### Fixed
+
+- Source badge clicks now work correctly without triggering row expansion
+- Fallback fonts now use proper sans-serif system font stack when `load_external_fonts` is disabled
+
 ## [0.2.1] - 2026-01-28
 
 ### Added

data/README.md

@@ -116,6 +116,12 @@ PgReports.configure do |config|
     end
   }
 
+  # External fonts (Google Fonts)
+  # Default: false (no external requests)
+  config.load_external_fonts = ENV["PG_REPORTS_LOAD_EXTERNAL_FONTS"] == "true"
+  # or simply:
+  # config.load_external_fonts = true
+
 end
 ```
 
@@ -333,6 +339,17 @@ PgReports.configure do |config|
 end
 ```
 
+### External Fonts
+
+By default, PgReports does **not** load external fonts.
+
+```ruby
+PgReports.configure do |config|
+  # Enable loading Google Fonts (optional)
+  config.load_external_fonts = true
+end
+```
+
 ## Telegram Integration
 
 1. Create a bot via [@BotFather](https://t.me/BotFather)

data/app/views/layouts/pg_reports/application.html.erb

@@ -6,9 +6,11 @@
   <meta name="pg-reports-root" content="<%= request.script_name.presence || PgReports::Engine.routes.url_helpers.root_path %>">
   <title>PgReports Dashboard</title>
   <link rel="icon" type="image/svg+xml" href="data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 32 32'%3E%3Crect width='32' height='32' rx='6' fill='%234f46e5'/%3E%3Crect x='5' y='18' width='5' height='9' rx='1' fill='%23fff'/%3E%3Crect x='13.5' y='12' width='5' height='15' rx='1' fill='%23fff'/%3E%3Crect x='22' y='6' width='5' height='21' rx='1' fill='%23fff'/%3E%3C/svg%3E">
-  <link rel="preconnect" href="https://fonts.googleapis.com">
-  <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
-  <link href="https://fonts.googleapis.com/css2?family=JetBrains+Mono:wght@400;500;600&family=Plus+Jakarta+Sans:wght@400;500;600;700&display=swap" rel="stylesheet">
+  <% if PgReports.config.load_external_fonts %>
+    <link rel="preconnect" href="https://fonts.googleapis.com">
+    <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
+    <link href="https://fonts.googleapis.com/css2?family=JetBrains+Mono:wght@400;500;600&family=Plus+Jakarta+Sans:wght@400;500;600;700&display=swap" rel="stylesheet">
+  <% end %>
   <style>
     :root {
       --bg-primary: #0f1114;
@@ -36,7 +38,7 @@
     }
 
     body {
-      font-family: 'Plus Jakarta Sans', -apple-system, BlinkMacSystemFont, sans-serif;
+      font-family: <%= PgReports.config.load_external_fonts ? "'Plus Jakarta Sans', " : "" %>-apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, "Helvetica Neue", Arial, "Noto Sans", sans-serif;
       background: var(--bg-primary);
       color: var(--text-primary);
       min-height: 100vh;
@@ -344,7 +346,7 @@
     .results-table {
       width: 100%;
       border-collapse: collapse;
-      font-family: 'JetBrains Mono', monospace;
+      font-family: <%= PgReports.config.load_external_fonts ? "'JetBrains Mono', " : "" %>SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace;
       font-size: 0.8rem;
     }
 

data/app/views/pg_reports/dashboard/_fake_source_data.html.erb

@@ -0,0 +1,43 @@
+<%# Fake source data for IDE link testing - enable via PG_REPORTS_FAKE_SOURCE_DATA=true %>
+<script>
+  // Known controller methods with line numbers (for IDE link testing)
+  const knownControllerMethods = {
+    'PostsController#show': { file: 'app/controllers/posts_controller.rb', line: 33 },
+    'PostsController#index': { file: 'app/controllers/posts_controller.rb', line: 19 },
+    'PostsController#create': { file: 'app/controllers/posts_controller.rb', line: 43 },
+    'PostsController#update': { file: 'app/controllers/posts_controller.rb', line: 62 },
+    'PostsController#destroy': { file: 'app/controllers/posts_controller.rb', line: 76 }
+  };
+
+  // Fake source values for testing IDE links
+  const fakeSourceValues = [
+    'PostsController#show',
+    'PostsController#index',
+    'PostsController#create',
+    'PostsController#update',
+    'PostsController#destroy',
+    'app/controllers/posts_controller.rb:33',
+    'app/controllers/posts_controller.rb:19',
+    'app/models/post.rb:45:in `find_by_slug`'
+  ];
+
+  // Inject fake source values into data for testing
+  function injectFakeSourceData(data) {
+    if (!data.data || data.data.length === 0) return;
+
+    // Add 'source' column if not present
+    if (!data.columns.includes('source')) {
+      data.columns.push('source');
+    }
+
+    // Inject fake source values
+    data.data.forEach((row, idx) => {
+      if (!row.source || row.source === '' || row.source === null) {
+        row.source = fakeSourceValues[idx % fakeSourceValues.length];
+      }
+    });
+  }
+
+  // Flag to indicate fake source data is enabled
+  window.PG_REPORTS_FAKE_SOURCE_DATA = true;
+</script>

data/app/views/pg_reports/dashboard/show.html.erb

@@ -1216,6 +1216,10 @@
   }
 </style>
 
+<% if PgReports.config.fake_source_data %>
+  <%= render 'pg_reports/dashboard/fake_source_data' %>
+<% end %>
+
 <script>
   let currentReportData = null;
   const category = '<%= @category %>';
@@ -1589,13 +1593,30 @@
       lineNumber = parseInt(fileLineMatch[3], 10);
     }
 
-    // Try to match Controller#action pattern
+    // Try to match Controller#action pattern (e.g., PostsController#index)
     const controllerMatch = source.match(/^(\w+Controller)#(\w+)/);
     if (controllerMatch) {
       methodName = `${controllerMatch[1]}#${controllerMatch[2]}`;
-      // Derive file path from controller name
-      const controllerName = controllerMatch[1].replace(/Controller$/, '').toLowerCase();
-      filePath = `app/controllers/${controllerName}_controller.rb`;
+
+      // Check known methods first for testing (if fake data is enabled)
+      if (typeof knownControllerMethods !== 'undefined' && knownControllerMethods[methodName]) {
+        filePath = knownControllerMethods[methodName].file;
+        lineNumber = knownControllerMethods[methodName].line;
+      } else {
+        // Derive file path from controller name
+        const controllerName = controllerMatch[1].replace(/Controller$/, '').toLowerCase();
+        filePath = `app/controllers/${controllerName}_controller.rb`;
+      }
+    }
+
+    // Try to match short controller#action pattern (e.g., posts#index, dashboard#show)
+    if (!filePath) {
+      const shortMatch = source.match(/^(\w+)#(\w+)$/);
+      if (shortMatch) {
+        const controllerName = shortMatch[1].toLowerCase();
+        methodName = `${shortMatch[1]}#${shortMatch[2]}`;
+        filePath = `app/controllers/${controllerName}_controller.rb`;
+      }
     }
 
     return { filePath, lineNumber, methodName, original: source };
@@ -1694,12 +1715,12 @@
 
     let dropdownHtml = `
       <div class="ide-dropdown">
-        <span class="source-badge clickable" onclick="event.stopPropagation(); toggleIdeDropdown('${dropdownId}', this)" title="${escapeHtml(parsed.original)}">${escapeHtml(parsed.original)}</span>
+        <span class="source-badge clickable" data-dropdown-id="${dropdownId}" title="${escapeHtml(parsed.original)}">${escapeHtml(parsed.original)}</span>
         <div class="ide-dropdown-menu" id="${dropdownId}">
     `;
 
     for (const ide of ideUrls) {
-      dropdownHtml += `<a href="${ide.url}" onclick="event.stopPropagation();">${ide.name}</a>`;
+      dropdownHtml += `<a href="${ide.url}">${ide.name}</a>`;
     }
 
     dropdownHtml += '</div></div>';
@@ -1726,6 +1747,26 @@
     }
   }
 
+  // Event delegation for source badge clicks
+  document.addEventListener('click', function(e) {
+    const badge = e.target.closest('.source-badge.clickable[data-dropdown-id]');
+    if (badge) {
+      e.stopPropagation();
+      e.preventDefault();
+      const dropdownId = badge.dataset.dropdownId;
+      toggleIdeDropdown(dropdownId, badge);
+      return;
+    }
+
+    // Allow clicks on IDE dropdown menu links
+    const ideLink = e.target.closest('.ide-dropdown-menu a');
+    if (ideLink) {
+      e.stopPropagation();
+      // Let the link navigate normally
+      return;
+    }
+  }, true); // Use capture phase to intercept before row click
+
   function buildDetailRow(row, columns, rowIndex, thresholds, problemFields) {
     let html = '<div class="row-detail">';
     const hasQuery = columns.includes('query') && row.query;
@@ -1829,6 +1870,11 @@
       if (loadingEl) loadingEl.style.display = 'none';
 
       if (data.success) {
+        // Inject fake source data for IDE link testing (if enabled)
+        if (typeof injectFakeSourceData === 'function') {
+          injectFakeSourceData(data);
+        }
+
         currentReportData = data;
         const thresholds = data.thresholds || {};
         const problemFields = data.problem_fields || [];

data/lib/pg_reports/configuration.rb

@@ -27,6 +27,12 @@ module PgReports
     # Dashboard settings
     attr_accessor :dashboard_auth               # Proc for dashboard authentication
 
+    # Assets / privacy settings
+    attr_accessor :load_external_fonts          # When true, loads Google Fonts in the dashboard layout
+
+    # Development/testing settings
+    attr_accessor :fake_source_data             # Inject fake source data for IDE link testing
+
     def initialize
       # Telegram
       @telegram_bot_token = ENV.fetch("PG_REPORTS_TELEGRAM_TOKEN", nil)
@@ -52,6 +58,12 @@ module PgReports
 
       # Dashboard
       @dashboard_auth = nil
+
+      # Assets / privacy
+      @load_external_fonts = ActiveModel::Type::Boolean.new.cast(ENV.fetch("PG_REPORTS_LOAD_EXTERNAL_FONTS", false))
+
+      # Development/testing
+      @fake_source_data = ENV.fetch("PG_REPORTS_FAKE_SOURCE_DATA", "false") == "true"
     end
 
     def connection

data/lib/pg_reports/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pg_reports
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Eldar Avatov
@@ -107,6 +107,7 @@ files:
 - README.md
 - app/controllers/pg_reports/dashboard_controller.rb
 - app/views/layouts/pg_reports/application.html.erb
+- app/views/pg_reports/dashboard/_fake_source_data.html.erb
 - app/views/pg_reports/dashboard/index.html.erb
 - app/views/pg_reports/dashboard/show.html.erb
 - config/locales/en.yml