dbwatcher 1.1.3 → 1.1.5

data/app/assets/stylesheets/dbwatcher/components/_tabulator.scss

@@ -35,24 +35,39 @@
         position: sticky;
         left: 0;
         z-index: 20;
-        background: #f3f3f3;
+        background: #f3f3f3 !important;
+        background-color: #f3f3f3 !important;
         box-shadow: 2px 0 4px rgba(0, 0, 0, 0.1);
+        will-change: transform;
+        transform: translateZ(0);
+        opacity: 1 !important;
+        transition: opacity 0.1s ease-out;
       }
 
       &.sticky-left-1 {
         position: sticky;
         left: 60px;
         z-index: 19;
-        background: #f3f3f3;
+        background: #f3f3f3 !important;
+        background-color: #f3f3f3 !important;
         box-shadow: 2px 0 4px rgba(0, 0, 0, 0.1);
+        will-change: transform;
+        transform: translateZ(0);
+        opacity: 1 !important;
+        transition: opacity 0.1s ease-out;
       }
 
       &.sticky-left-2 {
         position: sticky;
         left: 108px;
         z-index: 18;
-        background: #f3f3f3;
+        background: #f3f3f3 !important;
+        background-color: #f3f3f3 !important;
         box-shadow: 2px 0 4px rgba(0, 0, 0, 0.1);
+        will-change: transform;
+        transform: translateZ(0);
+        opacity: 1 !important;
+        transition: opacity 0.1s ease-out;
       }
     }
   }
@@ -115,25 +130,40 @@
           &.sticky-left-0 {
             position: sticky;
             left: 0;
-            background: white;
+            background: white !important;
+            background-color: white !important;
             z-index: 5;
             box-shadow: 2px 0 4px rgba(0, 0, 0, 0.05);
+            will-change: transform;
+            transform: translateZ(0);
+            opacity: 1 !important;
+            transition: opacity 0.1s ease-out;
           }
 
           &.sticky-left-1 {
             position: sticky;
             left: 60px;
-            background: white;
+            background: white !important;
+            background-color: white !important;
             z-index: 4;
             box-shadow: 2px 0 4px rgba(0, 0, 0, 0.05);
+            will-change: transform;
+            transform: translateZ(0);
+            opacity: 1 !important;
+            transition: opacity 0.1s ease-out;
           }
 
           &.sticky-left-2 {
             position: sticky;
             left: 108px;
-            background: white;
+            background: white !important;
+            background-color: white !important;
             z-index: 3;
             box-shadow: 2px 0 4px rgba(0, 0, 0, 0.05);
+            will-change: transform;
+            transform: translateZ(0);
+            opacity: 1 !important;
+            transition: opacity 0.1s ease-out;
           }
         }
 
@@ -143,7 +173,9 @@
             &.sticky-left-0,
             &.sticky-left-1,
             &.sticky-left-2 {
-              background: #f9fafb;
+              background: #f9fafb !important;
+              background-color: #f9fafb !important;
+              opacity: 1 !important;
             }
           }
         }
@@ -151,38 +183,50 @@
         // Operation-specific sticky cell backgrounds
         &.operation-insert .tabulator-cell {
           &.sticky-left-0, &.sticky-left-1, &.sticky-left-2 {
-            background-color: rgba(16, 185, 129, 0.05);
+            background: rgba(16, 185, 129, 0.05) !important;
+            background-color: rgba(16, 185, 129, 0.05) !important;
+            opacity: 1 !important;
           }
         }
 
         &.operation-update .tabulator-cell {
           &.sticky-left-0, &.sticky-left-1, &.sticky-left-2 {
-            background-color: rgba(108, 173, 223, 0.05);
+            background: rgba(108, 173, 223, 0.05) !important;
+            background-color: rgba(108, 173, 223, 0.05) !important;
+            opacity: 1 !important;
           }
         }
 
         &.operation-delete .tabulator-cell {
           &.sticky-left-0, &.sticky-left-1, &.sticky-left-2 {
-            background-color: rgba(239, 68, 68, 0.05);
+            background: rgba(239, 68, 68, 0.05) !important;
+            background-color: rgba(239, 68, 68, 0.05) !important;
+            opacity: 1 !important;
           }
         }
 
         // Operation-specific sticky cell hover backgrounds
         &.operation-insert:hover .tabulator-cell {
           &.sticky-left-0, &.sticky-left-1, &.sticky-left-2 {
-            background-color: rgba(16, 185, 129, 0.1);
+            background: rgba(16, 185, 129, 0.1) !important;
+            background-color: rgba(16, 185, 129, 0.1) !important;
+            opacity: 1 !important;
           }
         }
 
         &.operation-update:hover .tabulator-cell {
           &.sticky-left-0, &.sticky-left-1, &.sticky-left-2 {
-            background-color: rgba(108, 173, 223, 0.1);
+            background: rgba(108, 173, 223, 0.1) !important;
+            background-color: rgba(108, 173, 223, 0.1) !important;
+            opacity: 1 !important;
           }
         }
 
         &.operation-delete:hover .tabulator-cell {
           &.sticky-left-0, &.sticky-left-1, &.sticky-left-2 {
-            background-color: rgba(239, 68, 68, 0.1);
+            background: rgba(239, 68, 68, 0.1) !important;
+            background-color: rgba(239, 68, 68, 0.1) !important;
+            opacity: 1 !important;
           }
         }
       }

data/app/controllers/dbwatcher/api/v1/sessions_controller.rb

@@ -8,32 +8,23 @@ module Dbwatcher
 
         def tables_data
           Rails.logger.info "API::V1::SessionsController#tables_data: Getting tables for session #{@session.id}"
-
-          # Paginated, filtered tables data
-          # Convert ActionController::Parameters to a hash before passing to service
-          service = Dbwatcher::Services::Api::TablesDataService.new(@session, filter_params.to_h)
+          service = Dbwatcher::Services::Api::TablesDataService.new(@session, tables_data_params)
           render json: service.call
         end
 
         def summary_data
           Rails.logger.info "API::V1::SessionsController#summary_data: Getting summary for session #{@session.id}"
-
-          # Aggregated summary statistics
           service = Dbwatcher::Services::Api::SummaryDataService.new(@session)
           render json: service.call
         end
 
         def diagram_data
           Rails.logger.info "API::V1::SessionsController#diagram_data: Getting diagram for session #{@session.id}"
-
-          # Generated diagram content with caching
-          # Convert ActionController::Parameters to a hash before passing to service
-          diagram_params = params.to_unsafe_h
           service = Dbwatcher::Services::Api::DiagramDataService.new(@session, params[:type], diagram_params)
           result = service.call
 
           if result[:error]
-            render json: { error: result[:error] }, status: :unprocessable_entity
+            render_error(result[:error])
           else
             render json: result
           end
@@ -41,13 +32,11 @@ module Dbwatcher
 
         def timeline_data
           Rails.logger.info "API::V1::SessionsController#timeline_data: Getting timeline for session #{@session.id}"
-
-          # Timeline data processed from session changes
           service = Dbwatcher::Services::TimelineDataService.new(@session)
           result = service.call
 
           if result[:errors].any?
-            render json: { error: result[:errors].first[:message] }, status: :unprocessable_entity
+            render_error(result[:errors].first[:message])
           else
             render json: result
           end
@@ -55,7 +44,6 @@ module Dbwatcher
 
         def diagram_types
           Rails.logger.info "API::V1::SessionsController#diagram_types: Getting available diagram types"
-
           render json: {
             types: Dbwatcher::Services::Api::DiagramDataService.available_types_with_metadata,
             default_type: "database_tables"
@@ -66,11 +54,19 @@ module Dbwatcher
 
         def find_session
           @session = Storage.sessions.find(params[:id])
-          render json: { error: "Session not found" }, status: :not_found unless @session
+          render_error("Session not found", :not_found) unless @session
+        end
+
+        def tables_data_params
+          params.permit(:id, :table, :operation, :page, :per_page, session: {}).to_h
+        end
+
+        def diagram_params
+          params.permit(:type, :format, :include_columns, :show_relationships, session: {}).to_h
         end
 
-        def filter_params
-          params.permit(:table, :operation, :page, :per_page)
+        def render_error(message, status = :unprocessable_entity)
+          render json: { error: message }, status: status
         end
       end
     end

data/app/controllers/dbwatcher/api/v1/system_info_controller.rb

@@ -166,7 +166,7 @@ module Dbwatcher
         #
         # @return [void]
         def ensure_system_info_enabled
-          return if Dbwatcher.configuration.collect_system_info
+          return if Dbwatcher.configuration.system_info
 
           render json: {
             error: "System information collection is disabled",

data/app/controllers/dbwatcher/dashboard_controller.rb

@@ -10,7 +10,7 @@ module Dbwatcher
       @active_tab = params[:tab] || "overview"
 
       # Add system information if enabled
-      return unless Dbwatcher.configuration.collect_system_info
+      return unless Dbwatcher.configuration.system_info
 
       @system_info_summary = system_info_storage.summary
       @system_info = system_info_storage.cached_info

data/app/views/dbwatcher/dashboard/_overview.html.erb

@@ -118,10 +118,11 @@
           </div>
         <% else %>
           <div class="p-8 text-center text-gray-500">
-            <svg class="w-8 h-8 mx-auto text-gray-400 mb-2" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M12 8v4l3 3m6-3a9 9 0 11-18 0 9 9 0 0118 0z"/>
-            </svg>
+            <%= image_tag "dbwatcher/dbwatcher-tranparent_512x512.png", 
+                  alt: "DBWatcher Logo", 
+                  class: "w-12 h-12 mx-auto opacity-30 mb-2" %>
             <p class="text-sm">No recent sessions</p>
+            <p class="text-xs text-gray-400 mt-1">Start tracking with <code>?dbwatch=true</code></p>
           </div>
         <% end %>
       </div>
@@ -175,11 +176,11 @@
           </div>
         <% else %>
           <div class="p-8 text-center text-gray-500">
-            <svg class="w-8 h-8 mx-auto text-gray-400 mb-2" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M3 7v10a2 2 0 002 2h14a2 2 0 002-2V9a2 2 0 00-2-2H5a2 2 0 00-2-2z"/>
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M8 5a2 2 0 012-2h2a2 2 0 012 2v2H8V5z"/>
-            </svg>
+            <%= image_tag "dbwatcher/dbwatcher-tranparent_512x512.png", 
+                  alt: "DBWatcher Logo", 
+                  class: "w-12 h-12 mx-auto opacity-30 mb-2" %>
             <p class="text-sm">No active tables</p>
+            <p class="text-xs text-gray-400 mt-1">Tables will appear when data changes</p>
           </div>
         <% end %>
       </div>

data/app/views/dbwatcher/sessions/index.html.erb

@@ -41,46 +41,39 @@
   </div>
 
   <!-- Content Area -->
-  <div class="flex-1 overflow-auto p-4">
-
-  <% if @sessions.empty? %>
-    <div class="flex items-center justify-center h-full text-gray-500">
-      <div class="text-center">
-        <svg class="mx-auto h-8 w-8 text-gray-400 mb-2" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="1"
-                d="M9 5H7a2 2 0 00-2 2v12a2 2 0 002 2h10a2 2 0 002-2V7a2 2 0 00-2-2h-2M9 5a2 2 0 002 2h2a2 2 0 002-2M9 5a2 2 0 012-2h2a2 2 0 012 2"/>
-        </svg>
-        <p class="text-xs">No tracking sessions yet</p>
-        <p class="text-xs text-gray-400">Start tracking with <code class="bg-gray-200 px-1 rounded">Dbwatcher.track { ... }</code></p>
+  <div class="flex-1 overflow-auto">
+    <% if @sessions.empty? %>
+      <div class="p-8 text-center text-gray-500">
+        <%= image_tag "dbwatcher/dbwatcher-tranparent_512x512.png", 
+              alt: "DBWatcher Logo", 
+              class: "mx-auto w-12 h-12 opacity-30 mb-3" %>
+        <p class="text-sm font-medium text-gray-600 mb-2">No tracking sessions yet</p>
+        <p class="text-xs text-gray-400">Start tracking with <code class="bg-gray-100 px-1 rounded text-xs">Dbwatcher.track { ... }</code></p>
+        <p class="text-xs text-gray-400 mt-1">or add <code class="bg-gray-100 px-1 rounded text-xs">?dbwatch=true</code> to any URL</p>
       </div>
-    </div>
-  <% else %>
-    <div class="bg-white border border-gray-300 rounded shadow-sm">
-      <table class="compact-table sessions-table w-full">
+    <% else %>
+      <table class="compact-table w-full">
         <thead>
           <tr>
-            <th class="text-left" style="min-width:180px; max-width:260px; width:18%">Session ID</th>
-            <th class="text-left" style="min-width:160px; max-width:260px; width:22%">Name</th>
-            <th class="text-center" style="width:100px">Status</th>
-            <th class="text-center" style="width:100px">Changes</th>
-            <th class="text-right" style="width:120px">Started</th>
-            <th class="text-right" style="width:120px">Duration</th>
-            <th class="text-center" style="width:80px">Actions</th>
+            <th class="text-left" style="width: 18%; min-width: 180px">Session ID</th>
+            <th class="text-left" style="width: 22%; min-width: 160px">Name</th>
+            <th class="text-center" style="width: 100px">Status</th>
+            <th class="text-center" style="width: 100px">Changes</th>
+            <th class="text-right" style="width: 120px">Started</th>
+            <th class="text-right" style="width: 120px">Duration</th>
+            <th class="text-center" style="width: 80px">Actions</th>
           </tr>
         </thead>
         <tbody>
           <% @sessions.each do |session| %>
             <tr class="session-row">
-              <td style="min-width:180px; max-width:260px; width:18%">
-                <span class="font-mono text-xs">
-                  <%= safe_value(session, :id) %>
-                </span>
+              <td class="font-mono text-xs">
+                <%= safe_value(session, :id) %>
               </td>
-              <td style="min-width:160px; max-width:260px; width:22%" title="<%= safe_value(session, :name) %>">
+              <td title="<%= safe_value(session, :name) %>">
                 <%= link_to display_session_name(safe_value(session, :name)),
                     session_path(safe_value(session, :id)),
-                    class: "text-navy-dark hover:text-blue-medium whitespace-normal break-words inline-block",
-                    style: "max-width:260px; overflow-x:auto; display:inline-block;" %>
+                    class: "text-navy-dark hover:text-blue-medium" %>
               </td>
               <td class="text-center">
                 <%= render 'dbwatcher/shared/badge',
@@ -106,44 +99,34 @@
                     ) rescue 'N/A' %>
                 <% end %>
               </td>
-              <td class="text-center actions-cell">
-                <div class="flex gap-1 justify-end">
-                  <%= link_to session_path(safe_value(session, :id)),
-                      class: "compact-button bg-navy-dark text-white hover:bg-blue-medium",
-                      title: "View session details" do %>
-                    <svg class="w-3 h-3" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                      <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                            d="M15 12a3 3 0 11-6 0 3 3 0 016 0z"/>
-                      <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                            d="M2.458 12C3.732 7.943 7.523 5 12 5c4.478 0 8.268 2.943 9.542 7-1.274 4.057-5.064 7-9.542 7-4.477 0-8.268-2.943-9.542-7z"/>
-                    </svg>
-                  <% end %>
-                </div>
+              <td class="text-center">
+                <%= link_to "View", session_path(safe_value(session, :id)),
+                    class: "compact-button bg-navy-dark text-white hover:bg-blue-medium",
+                    title: "View session details" %>
               </td>
             </tr>
           <% end %>
         </tbody>
       </table>
-    </div>
 
-    <!-- No Results Message -->
-    <div
-      x-show="filterText && document.querySelectorAll('.session-row:not(.hidden)').length === 0"
-      x-cloak
-      class="mt-4 text-center py-8 bg-gray-50 border border-gray-200 rounded">
-      <p class="text-gray-500">No sessions match your filter criteria</p>
-      <button
-        @click="filterText = ''"
-        class="mt-2 text-blue-medium hover:text-blue-700 text-sm">
-        Clear filter
-      </button>
-    </div>
-  <% end %>
+      <!-- No Results Message -->
+      <div
+        x-show="filterText && document.querySelectorAll('.session-row:not(.hidden)').length === 0"
+        x-cloak
+        class="p-8 text-center text-gray-500 bg-gray-50">
+        <p class="mb-2">No sessions match your filter criteria</p>
+        <button
+          @click="filterText = ''"
+          class="compact-button bg-gray-500 text-white hover:bg-gray-600">
+          Clear filter
+        </button>
+      </div>
+    <% end %>
+  </div>
 
   <!-- Status Bar -->
-  <div class="h-6 bg-gray-100 border-t border-gray-300 flex items-center px-4 text-xs text-gray-600 mt-4">
-    <span x-show="!filterText"><%= @sessions.count %> sessions total •
-    <%= @sessions.count { |s| session_active?(s) } %> active</span>
+  <div class="h-6 bg-gray-100 border-t border-gray-300 flex items-center px-4 text-xs text-gray-600">
+    <span x-show="!filterText"><%= @sessions.count %> sessions total • <%= @sessions.count { |s| session_active?(s) } %> active</span>
     <span x-show="filterText" x-text="`${document.querySelectorAll('.session-row:not(.hidden)').length} of ${<%= @sessions.count %>} sessions shown`"></span>
     <span class="ml-auto">Last updated: <%= Time.current.strftime("%H:%M:%S") %></span>
   </div>

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

@@ -5,6 +5,14 @@
     <meta name="viewport" content="width=device-width,initial-scale=1">
     <%= csrf_meta_tags %>
 
+    <!-- Favicon -->
+    <%= favicon_link_tag asset_path('dbwatcher/favicon.ico') %>
+    <%= favicon_link_tag asset_path('dbwatcher/favicon-96x96.png'), sizes: '96x96', type: 'image/png' %>
+    <%= favicon_link_tag asset_path('dbwatcher/apple-touch-icon.png'), rel: 'apple-touch-icon', sizes: '180x180' %>
+    <%= favicon_link_tag asset_path('dbwatcher/web-app-manifest-192x192.png'), sizes: '192x192', type: 'image/png' %>
+    <%= favicon_link_tag asset_path('dbwatcher/web-app-manifest-512x512.png'), sizes: '512x512', type: 'image/png' %>
+    <link rel="manifest" href="<%= asset_path('dbwatcher/site.webmanifest') %>"
+
     <!-- Alpine.js and Plugins - ensure plugins load BEFORE Alpine.js core -->
     <script defer src="https://unpkg.com/@alpinejs/collapse@3.x.x/dist/cdn.min.js"></script>
     <script defer src="https://unpkg.com/alpinejs@3.x.x/dist/cdn.min.js"></script>
@@ -119,12 +127,20 @@
              :style="{ width: sidebarCollapsed ? '48px' : sidebarWidth + 'px' }">
         <div class="flex flex-col h-full">
           <!-- Logo -->
-          <div class="h-10 flex items-center justify-between px-3 border-b border-gray-700">
-            <div class="flex items-center gap-2" x-show="!sidebarCollapsed">
-              <svg class="w-5 h-5 text-gold-light" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M21 21l-6-6m2-5a7 7 0 11-14 0 7 7 0 0114 0z"/>
-              </svg>
-              <span class="text-sm font-medium text-white">DB Watcher</span>
+          <div class="h-12 flex items-center justify-between px-3 border-b border-gray-700 bg-gray-800">
+            <div class="flex items-center gap-3" x-show="!sidebarCollapsed">
+              <%= image_tag "dbwatcher/dbwatcher-tranparent_512x512.png",
+                    alt: "DBWatcher Logo",
+                    class: "w-10 h-10" %>
+              <div class="flex flex-col">
+                <span class="text-base font-bold text-white">dbwatcher</span>
+              </div>
+            </div>
+            <!-- Collapsed state logo -->
+            <div class="flex items-center justify-center w-full" x-show="sidebarCollapsed">
+              <%= image_tag "dbwatcher/dbwatcher-tranparent_512x512.png",
+                    alt: "dbwatcher Logo",
+                    class: "w-8 h-8" %>
             </div>
             <button @click="sidebarCollapsed = !sidebarCollapsed"
                     class="p-1 hover:bg-gray-800 rounded text-gray-400">

data/lib/dbwatcher/configuration.rb

@@ -3,93 +3,84 @@
 module Dbwatcher
   # Configuration class for DBWatcher
   #
-  # This class manages all configuration options for DBWatcher, including
-  # storage, tracking, and diagram visualization settings.
+  # Simplified configuration with logical groupings and sensible defaults
   class Configuration
-    # Storage configuration
-    attr_accessor :storage_path, :enabled, :max_sessions, :auto_clean_after_days
+    # Core settings - what users need most
+    attr_accessor :enabled, :storage_path
 
-    # Query tracking configuration
-    attr_accessor :track_queries, :slow_query_threshold, :max_query_logs_per_day
+    # Session management - how data is stored and cleaned
+    attr_accessor :max_sessions, :auto_clean_days
 
-    # Routing configuration
-    attr_accessor :mount_path
+    # Query tracking - performance monitoring
+    attr_accessor :track_queries
 
-    # Diagram configuration
-    attr_accessor :diagram_show_attributes, :diagram_show_methods, :diagram_show_cardinality,
-                  :diagram_max_attributes, :diagram_attribute_types, :diagram_relationship_labels,
-                  :diagram_preserve_table_case, :diagram_direction, :diagram_cardinality_format,
-                  :diagram_show_attribute_count, :diagram_show_method_count
+    # System info - debugging and monitoring
+    attr_accessor :system_info, :debug_mode
 
-    # System information configuration
-    attr_accessor :collect_system_info, :system_info_refresh_interval,
-                  :collect_sensitive_env_vars, :system_info_cache_duration,
-                  :system_info_include_performance_metrics
+    # Advanced diagram options - available but not commonly needed
+    attr_accessor :diagram_show_methods, :diagram_max_attributes,
+                  :diagram_attribute_types, :diagram_relationship_labels,
+                  :diagram_show_attributes, :diagram_show_cardinality
 
     # Initialize with default values
     def initialize
-      # Storage configuration defaults
-      @storage_path = default_storage_path
+      # Core settings
       @enabled = true
+      @storage_path = default_storage_path
+
+      # Session management
       @max_sessions = 50
-      @auto_clean_after_days = 7
+      @auto_clean_days = 7
 
-      # Query tracking configuration defaults
+      # Query tracking
       @track_queries = false
-      @slow_query_threshold = 200 # milliseconds
-      @max_query_logs_per_day = 1000
 
-      # Routing configuration defaults
-      @mount_path = "/dbwatcher"
+      # System info
+      @system_info = true
+      @debug_mode = false
 
-      # Initialize diagram configuration with defaults
-      initialize_diagram_config
+      # Advanced diagram options - sensible defaults
+      @diagram_show_methods = false
+      @diagram_max_attributes = 10
+      @diagram_attribute_types = true
+      @diagram_relationship_labels = true
+      @diagram_show_attributes = true
+      @diagram_show_cardinality = true
+    end
 
-      # Initialize system information configuration with defaults
-      initialize_system_info_config
+    # Fixed defaults for options that are still used in codebase but not configurable
+    def slow_query_threshold
+      200 # Fixed default value
     end
 
-    # Initialize diagram configuration with default values
-    def initialize_diagram_config
-      @diagram_show_attributes = true
-      @diagram_show_methods = false # Hide methods by default
-      @diagram_show_cardinality = true
-      @diagram_max_attributes = 10
-      @diagram_attribute_types = true # Changed from array to boolean
-      @diagram_relationship_labels = true # Changed from symbol to boolean
-      @diagram_preserve_table_case = false # Changed from true to false
-      @diagram_direction = "LR" # Left to right by default
-      @diagram_cardinality_format = :simple # Use simpler 1:N format
-      @diagram_show_attribute_count = true
-      @diagram_show_method_count = true
+    def diagram_direction
+      "LR" # Fixed default value
     end
 
-    # Initialize system information configuration with default values
-    def initialize_system_info_config
-      @collect_system_info = true
-      @system_info_refresh_interval = 5 * 60 # 5 minutes in seconds
-      @collect_sensitive_env_vars = false
-      @system_info_cache_duration = 60 * 60 # 1 hour in seconds
-      @system_info_include_performance_metrics = true
+    # Fixed defaults for complex options that are still used in codebase but not configurable
+    def max_query_logs_per_day = 1000
+    def system_info_refresh_interval = 5 * 60
+    def system_info_cache_duration = 60 * 60
+
+    def collect_sensitive_env_vars?
+      false
+    end
+
+    def system_info_include_performance_metrics?
+      true
     end
 
     # Validate configuration
-    #
-    # @return [Boolean] true if configuration is valid
     def valid?
       validate_storage_path
       validate_max_sessions
-      validate_auto_clean_after_days
-      validate_slow_query_threshold
-      validate_max_query_logs_per_day
+      validate_auto_clean_days
       true
     end
 
     private
 
     # Default storage path based on Rails or current directory
-    #
-    # @return [String] default storage path
     def default_storage_path
       if defined?(Rails) && Rails.respond_to?(:root) && Rails.root
         Rails.root.join("tmp", "dbwatcher").to_s
@@ -116,25 +107,11 @@ module Dbwatcher
       raise ConfigurationError, "max_sessions must be a positive integer"
     end
 
-    # Validate auto clean after days
-    def validate_auto_clean_after_days
-      return if auto_clean_after_days.is_a?(Integer) && auto_clean_after_days.positive?
-
-      raise ConfigurationError, "auto_clean_after_days must be a positive integer"
-    end
-
-    # Validate slow query threshold
-    def validate_slow_query_threshold
-      return if slow_query_threshold.is_a?(Integer) && slow_query_threshold.positive?
-
-      raise ConfigurationError, "slow_query_threshold must be a positive integer"
-    end
-
-    # Validate max query logs per day
-    def validate_max_query_logs_per_day
-      return if max_query_logs_per_day.is_a?(Integer) && max_query_logs_per_day.positive?
+    # Validate auto clean days
+    def validate_auto_clean_days
+      return if auto_clean_days.is_a?(Integer) && auto_clean_days.positive?
 
-      raise ConfigurationError, "max_query_logs_per_day must be a positive integer"
+      raise ConfigurationError, "auto_clean_days must be a positive integer"
     end
   end