pg_reports 0.3.1 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: be278e9401573fc93ab6d619427d7356609bf26a03cb69ac6a5dbbc2f7b39409
-  data.tar.gz: b5fae395bbae259280751183344407b69eb2e90da268b380e5bfb97a6ddbf382
+  metadata.gz: 7a57deebae9eda0df4aceb3cfeb1912e2559a7cd70c3c7f197fb86f06bb689b1
+  data.tar.gz: 9369b5d6f54d8b0f2939f57d429f477aa0a8c2a862add70e2116232f364fe1e7
 SHA512:
-  metadata.gz: f76f4ff3f31c36aae1b21dc080283d6a49d5a41b6b9405a569548eae21df5a8f5c7f0c205a88d712f84a0b4ad166486a82bec9dbed0a0185fc2301ca375ceb6f
-  data.tar.gz: cf31a2c98627e7d0a8aab41fa041682371fa75b2673500f9d65700a246cef7c6f323e6e1bc9238d4793b0e49a21785b0446cb3216de1c8759877c17e593532fc
+  metadata.gz: 0e6248b26056ffe059105c2399f7a9c8e607e3f34030d4af6f14b324deb76435163b8b2116ff5d7502350410f0a6a20368ac3757b9247d30328da3330a941a86
+  data.tar.gz: 5d67f8d3c5663421b839301c3c4463502b7cbaeb57d3351d808ff00ebb355141b6c7cf676df091f7a2bb52238809cb02b8e9123d16b7a597bbc636e0cf888540

data/CHANGELOG.md

@@ -5,6 +5,42 @@ 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.4.0] - 2026-01-29
+
+### Added
+
+- **YAML-based Report Configuration System** - declarative report definitions:
+  - Each report now defined in a single YAML file (1 file = 1 report approach)
+  - Migrated all 31 reports from Ruby methods to YAML format
+  - New classes: `ReportDefinition`, `Filter`, `ReportLoader`, `ModuleGenerator`
+  - Support for post-SQL filtering with operators (eq, ne, lt, lte, gt, gte)
+  - Title interpolation with variable substitution using `${variable}` syntax
+  - Enrichment hooks pattern for data transformation
+  - Problem explanations configuration in YAML (field → explanation key mapping)
+- **Filter UI** on report pages for manual parameter input:
+  - Collapsible filter section with form inputs for each parameter
+  - Support for threshold overrides (limit, min_duration_seconds, etc.)
+  - Real-time report refresh with custom filter values
+  - Horizontal layout with descriptions and inputs side-by-side
+  - Dark theme styling for filter inputs and labels
+
+### Changed
+
+- Report definitions now use declarative YAML files instead of repetitive Ruby methods (~800 lines of code eliminated)
+- Modules now use metaprogramming to dynamically generate methods from YAML at load time
+- Dashboard category cards width increased from 280px to 350px for better readability
+- Documentation and filter sections positioned side-by-side in responsive grid layout
+- Filter parameters section positioned below documentation and collapsed by default
+- Reduced vertical spacing between sections for more compact layout
+- Problem field to explanation key mapping moved from hardcoded JavaScript to server-driven YAML configuration
+
+### Fixed
+
+- CSS Grid `align-items: stretch` causing synchronized expansion of documentation and filter blocks
+- Visual "chin" appearing on closed filter details block due to default margins
+- TypeError when report has no limit parameter default value
+- Excessive margins between report sections
+
 ## [0.3.0] - 2026-01-28
 
 ### Added

data/app/controllers/pg_reports/dashboard_controller.rb

@@ -51,6 +51,9 @@ module PgReports
       @thresholds = Dashboard::ReportsRegistry.thresholds(@report_key)
       @problem_fields = Dashboard::ReportsRegistry.problem_fields(@report_key)
 
+      # Load filter parameters from YAML
+      @report_filters = load_report_filters(@category, @report_key)
+
       @report = execute_report(@category, @report_key)
     rescue => e
       @error = e.message
@@ -61,9 +64,13 @@ module PgReports
       category = params[:category].to_sym
       report_key = params[:report].to_sym
 
-      report = execute_report(category, report_key)
+      # Extract filter parameters from request
+      filter_params = extract_filter_params
+
+      report = execute_report(category, report_key, **filter_params)
       thresholds = Dashboard::ReportsRegistry.thresholds(report_key)
       problem_fields = Dashboard::ReportsRegistry.problem_fields(report_key)
+      problem_explanations = load_problem_explanations(category, report_key)
 
       render json: {
         success: true,
@@ -73,7 +80,8 @@ module PgReports
         total: report.size,
         generated_at: report.generated_at.strftime("%Y-%m-%d %H:%M:%S"),
         thresholds: thresholds,
-        problem_fields: problem_fields
+        problem_fields: problem_fields,
+        problem_explanations: problem_explanations
       }
     rescue => e
       render json: {success: false, error: e.message}, status: :unprocessable_entity
@@ -240,6 +248,15 @@ module PgReports
     end
 
     def create_migration
+      # Only allow migration creation in development environment
+      unless Rails.env.development?
+        render json: {
+          success: false,
+          error: "Migration creation is only allowed in development environment"
+        }, status: :forbidden
+        return
+      end
+
       file_name = params[:file_name]
       code = params[:code]
 
@@ -280,13 +297,51 @@ module PgReports
       @categories = Dashboard::ReportsRegistry.all
     end
 
-    def execute_report(category, report_key)
+    def load_report_filters(category, report_key)
+      definition = ReportLoader.get(category.to_s, report_key.to_s)
+      return {} unless definition
+
+      definition.filter_parameters
+    end
+
+    def load_problem_explanations(category, report_key)
+      definition = ReportLoader.get(category.to_s, report_key.to_s)
+      return {} unless definition
+
+      definition.problem_explanations
+    end
+
+    def extract_filter_params
+      # Allow common filter parameters
+      allowed = [:limit, :min_duration_seconds, :min_calls]
+      result = {}
+
+      allowed.each do |key|
+        if params[key].present?
+          value = params[key].to_s
+          # Convert to appropriate type
+          result[key] = value.match?(/^\d+$/) ? value.to_i : value
+        end
+      end
+
+      # Also allow threshold overrides (calls_threshold, etc.)
+      params.each do |key, value|
+        if key.to_s.end_with?('_threshold') && value.present?
+          result[key.to_sym] = value.to_i
+        end
+      end
+
+      result
+    end
+
+    def execute_report(category, report_key, **filter_params)
       mod = case category
       when :queries then Modules::Queries
       when :indexes then Modules::Indexes
       when :tables then Modules::Tables
       when :connections then Modules::Connections
       when :system then Modules::System
+      when :schema_analysis then Modules::SchemaAnalysis
       else raise ArgumentError, "Unknown category: #{category}"
       end
 
@@ -294,7 +349,7 @@ module PgReports
         raise ArgumentError, "Unknown report: #{report_key}"
       end
 
-      mod.public_send(report_key)
+      mod.public_send(report_key, **filter_params)
     end
 
     def substitute_params(query, params_hash)

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

@@ -122,7 +122,7 @@
     /* Categories Grid */
     .categories-grid {
       display: grid;
-      grid-template-columns: repeat(auto-fit, minmax(280px, 1fr));
+      grid-template-columns: repeat(auto-fit, minmax(350px, 1fr));
       gap: 1.5rem;
       margin-bottom: 2rem;
     }

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

@@ -94,11 +94,18 @@
       <button class="modal-close" onclick="closeMigrationModal()">&times;</button>
     </div>
     <div class="modal-body" id="migration-modal-body">
+      <div class="migration-warning" style="background: rgba(255, 152, 0, 0.1); border: 1px solid rgba(255, 152, 0, 0.3); padding: 1rem; border-radius: 8px; margin-bottom: 1rem;">
+        <p style="margin: 0 0 0.5rem 0; font-weight: 600; color: #ffb74d;">⚠️ Warning</p>
+        <p style="margin: 0; color: #ffcc80; font-size: 0.875rem; line-height: 1.5;">
+          Creating a migration will generate a migration file in your project. Running this migration will drop the index from the database, which may significantly impact application performance.
+          <strong>This operation should only be performed in a local development environment.</strong>
+        </p>
+      </div>
       <p class="settings-label">Generated migration to remove the index:</p>
       <div id="migration-code" class="migration-code"></div>
       <div class="migration-actions">
         <button class="btn btn-secondary" onclick="copyMigrationCode()">📋 Copy Code</button>
-        <button class="btn btn-primary" onclick="createMigrationFile()">📁 Create File & Open in IDE</button>
+        <button class="btn btn-primary" id="create-migration-btn" onclick="createMigrationFile()">📁 Create File & Open in IDE</button>
       </div>
     </div>
   </div>

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

@@ -1,7 +1,10 @@
 <script>
   let currentReportData = null;
+  let currentProblemExplanations = {};
   const category = '<%= @category %>';
   const reportKey = '<%= @report_key %>';
+  const railsEnv = '<%= Rails.env %>';
+  const isDevelopment = railsEnv === 'development';
   let syncingScroll = false;
   let currentSort = { column: null, direction: 'asc' };
 
@@ -331,21 +334,16 @@
     document.getElementById('problem-modal').style.display = 'none';
   }
 
-  // Map problem fields to explanation keys
+  // Map problem fields to explanation keys using server-provided mapping
   function getExplanationKey(problems) {
     const fields = problems.map(p => p.field);
 
-    if (fields.includes('mean_time_ms')) return 'high_mean_time';
-    if (fields.includes('calls')) return 'high_calls';
-    if (fields.includes('total_time_ms')) return 'high_total_time';
-    if (fields.includes('cache_hit_ratio')) return 'low_cache_hit';
-    if (fields.includes('seq_scan') || fields.includes('seq_tup_read')) return 'high_seq_scan';
-    if (fields.includes('idx_scan')) return 'unused_index';
-    if (fields.includes('bloat_ratio') || fields.includes('bloat_size')) return 'high_bloat';
-    if (fields.includes('dead_tuple_ratio') || fields.includes('n_dead_tup')) return 'many_dead_tuples';
-    if (fields.includes('duration_seconds') || fields.includes('duration')) return 'long_running';
-    if (fields.includes('blocked_count')) return 'blocking';
-    if (fields.includes('idle_in_transaction')) return 'idle_in_transaction';
+    // Use server-provided problem explanations mapping
+    for (const field of fields) {
+      if (currentProblemExplanations[field]) {
+        return currentProblemExplanations[field];
+      }
+    }
 
     return '';
   }
@@ -541,11 +539,15 @@
 
   function buildDetailRow(row, columns, rowIndex, thresholds, problemFields) {
     let html = '<div class="row-detail">';
-    const hasQuery = columns.includes('query') && row.query;
-    const hasIndexName = columns.includes('index_name') && row.index_name;
+    const hasQuery = columns.includes('query') || row.query;
+    const hasIndexName = columns.includes('index_name') || row.index_name;
     const rowId = generateRowId(row);
 
-    columns.forEach(col => {
+    // Show all fields from row, not just columns
+    // First show fields from columns array in order, then remaining fields
+    const allKeys = [...new Set([...columns, ...Object.keys(row)])];
+
+    allKeys.forEach(col => {
       const value = row[col] ?? '';
       const strValue = String(value);
       const isQuery = col === 'query';
@@ -599,12 +601,16 @@
       }
     }
 
-    // Show migration button for index reports
+    // Show migration button for index reports (only in development)
     if (hasIndexName && (category === 'indexes')) {
       const indexName = row.index_name;
       const tableName = row.table_name || row.tablename || '';
       const schemaName = row.schema_name || row.schemaname || 'public';
-      html += `<button class="btn-migration" onclick="event.stopPropagation(); showMigrationModal('${escapeHtml(indexName)}', '${escapeHtml(tableName)}', '${escapeHtml(schemaName)}')">🗑️ Generate Migration</button>`;
+      if (isDevelopment) {
+        html += `<button class="btn-migration" onclick="event.stopPropagation(); showMigrationModal('${escapeHtml(indexName)}', '${escapeHtml(tableName)}', '${escapeHtml(schemaName)}')">🗑️ Generate Migration</button>`;
+      } else {
+        html += `<button class="btn-migration" disabled title="Migration generation is only available in development environment" style="opacity: 0.5; cursor: not-allowed;">🗑️ Generate Migration (Development Only)</button>`;
+      }
     }
 
     html += `</div>`;
@@ -634,12 +640,27 @@
     if (telegramBtn) telegramBtn.style.display = 'none';
 
     try {
+      // Collect filter parameters from form
+      const filterParams = {};
+      document.querySelectorAll('[data-param]').forEach(input => {
+        const paramName = input.dataset.param;
+        const value = input.value;
+        if (value && value.trim() !== '') {
+          filterParams[paramName] = value;
+        }
+      });
+
       const response = await fetch(`${pgReportsRoot}/${cat}/${report}/run`, {
         method: 'POST',
         headers: {
           'Content-Type': 'application/json',
           'X-CSRF-Token': document.querySelector('meta[name="csrf-token"]')?.content || ''
-        }
+        },
+        body: JSON.stringify({
+          category: cat,
+          report: report,
+          ...filterParams
+        })
       });
 
       const data = await response.json();
@@ -653,6 +674,7 @@
         }
 
         currentReportData = data;
+        currentProblemExplanations = data.problem_explanations || {};
         const thresholds = data.thresholds || {};
         const problemFields = data.problem_fields || [];
 
@@ -1239,6 +1261,22 @@ end
 
     document.getElementById('migration-code').textContent = migrationCode;
     document.getElementById('migration-modal').style.display = 'flex';
+
+    // Disable "Create File" button in non-development environments
+    const createBtn = document.getElementById('create-migration-btn');
+    if (createBtn) {
+      if (!isDevelopment) {
+        createBtn.disabled = true;
+        createBtn.style.opacity = '0.5';
+        createBtn.style.cursor = 'not-allowed';
+        createBtn.title = 'Migration file creation is only available in development environment';
+      } else {
+        createBtn.disabled = false;
+        createBtn.style.opacity = '1';
+        createBtn.style.cursor = 'pointer';
+        createBtn.title = '';
+      }
+    }
   }
 
   function closeMigrationModal() {

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

@@ -4,7 +4,6 @@
     background: var(--bg-card);
     border: 1px solid var(--border-color);
     border-radius: 12px;
-    margin-bottom: 0.75rem;
     overflow: hidden;
   }
 
@@ -1143,4 +1142,126 @@
     gap: 0.75rem;
     justify-content: flex-end;
   }
+
+  /* Report details grid (documentation + filters side by side) */
+  .report-details-grid {
+    display: grid;
+    grid-template-columns: 1fr 1fr;
+    gap: 1rem;
+    align-items: start;
+  }
+
+  @media (max-width: 1200px) {
+    .report-details-grid {
+      grid-template-columns: 1fr;
+    }
+  }
+
+  /* Filter Section */
+  .filter-section {
+    background: var(--bg-card);
+    border: 1px solid var(--border-color);
+    border-radius: 12px;
+    overflow: hidden;
+  }
+
+  .filter-details {
+    margin: 0;
+    display: block;
+  }
+
+  .filter-toggle {
+    display: flex;
+    align-items: center;
+    gap: 0.75rem;
+    padding: 1rem 1.25rem;
+    margin: 0;
+    cursor: pointer;
+    user-select: none;
+    color: var(--text-secondary);
+    font-weight: 500;
+    transition: all 0.15s;
+  }
+
+  .filter-toggle:hover {
+    background: var(--bg-tertiary);
+    color: var(--text-primary);
+  }
+
+  .filter-details[open] .filter-toggle {
+    border-bottom: 1px solid var(--border-color);
+    background: var(--bg-tertiary);
+  }
+
+  .filter-details[open] .toggle-icon {
+    transform: rotate(90deg);
+  }
+
+  .filter-details:not([open]) .filter-toggle {
+    background: transparent;
+  }
+
+  .filter-details:not([open]) .filter-toggle:hover {
+    background: var(--bg-tertiary);
+  }
+
+  .filter-content {
+    padding: 1rem 1.25rem;
+    background: var(--bg-card);
+  }
+
+  .filter-grid {
+    display: flex;
+    flex-direction: column;
+    gap: 0.75rem;
+  }
+
+  .filter-item {
+    display: grid;
+    grid-template-columns: 1fr auto;
+    gap: 1.5rem;
+    align-items: start;
+  }
+
+  .filter-label {
+    display: flex;
+    flex-direction: column;
+    gap: 0.25rem;
+    font-weight: 500;
+    color: var(--text-primary);
+    font-size: 0.9rem;
+    line-height: 1.4;
+  }
+
+  .filter-description {
+    font-size: 0.8rem;
+    font-weight: 400;
+    color: var(--text-secondary);
+    opacity: 0.8;
+  }
+
+  .filter-current-value {
+    font-size: 0.8rem;
+    font-weight: 400;
+    color: var(--accent-blue);
+    opacity: 0.9;
+  }
+
+  .filter-input {
+    padding: 0.625rem 0.875rem;
+    border: 1px solid var(--border-color);
+    border-radius: 8px;
+    font-size: 1rem;
+    background: var(--bg-primary);
+    color: var(--text-primary);
+    transition: all 0.15s;
+    min-width: 120px;
+    max-width: 200px;
+  }
+
+  .filter-input:focus {
+    outline: none;
+    border-color: var(--accent-blue);
+    box-shadow: 0 0 0 3px rgba(59, 130, 246, 0.1);
+  }
 </style>

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

@@ -50,59 +50,101 @@
     </div>
   <% end %>
 
-  <!-- Collapsible Documentation Section -->
-  <% if @documentation && @documentation[:what].present? %>
-    <details class="documentation-section">
-      <summary class="documentation-toggle">
-        <span class="toggle-icon">▶</span>
-        <span>📖 What does this report show?</span>
-      </summary>
-      <div class="documentation-content">
-        <% if @documentation[:what].present? %>
-          <div class="doc-block">
-            <h4>📋 What</h4>
-            <p><%= @documentation[:what] %></p>
-          </div>
-        <% end %>
+  <div class="report-details-grid">
+    <!-- Collapsible Documentation Section -->
+    <% if @documentation && @documentation[:what].present? %>
+      <details class="documentation-section">
+        <summary class="documentation-toggle">
+          <span class="toggle-icon">▶</span>
+          <span>📖 What does this report show?</span>
+        </summary>
+        <div class="documentation-content">
+          <% if @documentation[:what].present? %>
+            <div class="doc-block">
+              <h4>📋 What</h4>
+              <p><%= @documentation[:what] %></p>
+            </div>
+          <% end %>
 
-        <% if @documentation[:why].present? %>
-          <div class="doc-block">
-            <h4>❓ Why It Matters</h4>
-            <p><%= @documentation[:why] %></p>
-          </div>
-        <% end %>
-
-        <% if @documentation[:nuances].present? %>
-          <div class="doc-block">
-            <h4>⚠️ Nuances</h4>
-            <ul class="nuances-list">
-              <% @documentation[:nuances].each do |nuance| %>
-                <li><%= nuance %></li>
-              <% end %>
-            </ul>
-          </div>
-        <% end %>
-
-        <% if @thresholds.present? %>
-          <div class="thresholds-block">
-            <h4>📊 Thresholds</h4>
-            <div class="thresholds-grid">
-              <% @thresholds.each do |field, values| %>
-                <div class="threshold-item">
-                  <span class="threshold-field"><%= field %></span>
-                  <span class="threshold-warning">⚠️ Warning: <%= values[:warning] %></span>
-                  <span class="threshold-critical">🔴 Critical: <%= values[:critical] %></span>
-                  <% if values[:inverted] %>
-                    <span class="threshold-note">(lower is worse)</span>
-                  <% end %>
+          <% if @documentation[:why].present? %>
+            <div class="doc-block">
+              <h4>❓ Why It Matters</h4>
+              <p><%= @documentation[:why] %></p>
+            </div>
+          <% end %>
+
+          <% if @documentation[:nuances].present? %>
+            <div class="doc-block">
+              <h4>⚠️ Nuances</h4>
+              <ul class="nuances-list">
+                <% @documentation[:nuances].each do |nuance| %>
+                  <li><%= nuance %></li>
+                <% end %>
+              </ul>
+            </div>
+          <% end %>
+
+          <% if @thresholds.present? %>
+            <div class="thresholds-block">
+              <h4>📊 Thresholds</h4>
+              <div class="thresholds-grid">
+                <% @thresholds.each do |field, values| %>
+                  <div class="threshold-item">
+                    <span class="threshold-field"><%= field %></span>
+                    <span class="threshold-warning">⚠️ Warning: <%= values[:warning] %></span>
+                    <span class="threshold-critical">🔴 Critical: <%= values[:critical] %></span>
+                    <% if values[:inverted] %>
+                      <span class="threshold-note">(lower is worse)</span>
+                    <% end %>
+                  </div>
+                <% end %>
+              </div>
+            </div>
+          <% end %>
+        </div>
+      </details>
+    <% end %>
+
+    <!-- Filter Parameters Section -->
+    <% if @report_filters.present? %>
+      <div class="filter-section">
+        <details class="filter-details">
+          <summary class="filter-toggle">
+            <span class="toggle-icon">▶</span>
+            <span>🔍 Параметры фильтрации</span>
+          </summary>
+          <div class="filter-content">
+            <div class="filter-grid">
+              <% @report_filters.each do |name, config| %>
+                <div class="filter-item">
+                  <label class="filter-label">
+                    <%= config[:label] %>
+                    <% if config[:description] %>
+                      <span class="filter-description"><%= config[:description] %></span>
+                    <% end %>
+                    <% if config[:is_threshold] && config[:current_config] %>
+                      <span class="filter-current-value">(current: <%= config[:current_config] %>)</span>
+                    <% end %>
+                  </label>
+                  <input
+                    type="<%= config[:type] == 'integer' ? 'number' : 'text' %>"
+                    class="filter-input"
+                    data-param="<%= name %>"
+                    value="<%= config[:default] %>"
+                    <% if config[:type] == 'integer' %>
+                      min="0"
+                      step="1"
+                    <% end %>
+                    placeholder="<%= config[:default] %>"
+                  >
                 </div>
               <% end %>
             </div>
           </div>
-        <% end %>
+        </details>
       </div>
-    </details>
-  <% end %>
+    <% end %>
+  </div>
 
   <!-- Saved Records Section -->
   <div class="saved-records-section" id="saved-records-section" style="display: none;">

data/config/locales/en.yml

@@ -295,6 +295,19 @@ en:
           - "Target value: >99% for OLTP, >95% for mixed workload."
           - "Low cache hit: increase shared_buffers (up to 25% RAM), or the problem is in queries."
 
+      # === SCHEMA ANALYSIS ===
+      missing_validations:
+        title: "Missing Validations"
+        what: "Unique indexes in the database without corresponding uniqueness validations in Rails models."
+        how: "Analyzes all unique indexes (including composite ones) and checks if the corresponding Rails model has validates :column, uniqueness: true validation."
+        nuances:
+          - "Database constraints (indexes) and model validations serve different purposes — both should be present."
+          - "Unique index prevents duplicates at database level, validation provides user-friendly error messages."
+          - "For composite indexes (a, b), need validation with scope: validates :a, uniqueness: { scope: :b }"
+          - "Some tables may not have models — this is normal for join tables or legacy tables."
+          - "Validations in concerns and parent models are also detected."
+          - "Primary keys don't need validations — they are automatically unique."
+
     # Problem explanations for highlighting
     problems:
       high_mean_time: "High mean execution time. Consider adding indexes or optimizing the query."

data/config/locales/ru.yml

@@ -295,6 +295,19 @@ ru:
           - "Целевое значение: >99% для OLTP, >95% для mixed workload."
           - "Низкий cache hit: увеличьте shared_buffers (до 25% RAM), или проблема в запросах."
 
+      # === SCHEMA ANALYSIS ===
+      missing_validations:
+        title: "Отсутствующие валидации"
+        what: "Уникальные индексы в базе данных без соответствующих валидаций uniqueness в Rails-моделях."
+        how: "Анализирует все уникальные индексы (включая композитные) и проверяет наличие validates :column, uniqueness: true в соответствующей Rails-модели."
+        nuances:
+          - "Ограничения БД (индексы) и валидации модели служат разным целям — оба должны присутствовать."
+          - "Уникальный индекс предотвращает дубликаты на уровне БД, валидация даёт понятные сообщения об ошибках."
+          - "Для композитных индексов (a, b) нужна валидация со scope: validates :a, uniqueness: { scope: :b }"
+          - "Некоторые таблицы могут не иметь моделей — это нормально для join-таблиц или legacy-таблиц."
+          - "Валидации в concerns и родительских моделях также детектируются."
+          - "Первичные ключи не требуют валидаций — они автоматически уникальны."
+
     # Problem explanations for highlighting
     problems:
       high_mean_time: "Высокое среднее время выполнения. Рассмотрите добавление индексов или оптимизацию запроса."