source_monitor 0.2.0 → 0.3.0

data/app/helpers/source_monitor/application_helper.rb

@@ -1,5 +1,9 @@
+# frozen_string_literal: true
+
 module SourceMonitor
   module ApplicationHelper
+    include TableSortHelper
+    include HealthBadgeHelper
     def source_monitor_stylesheet_bundle_tag
       stylesheet_link_tag("source_monitor/application", "data-turbo-track": "reload")
     rescue StandardError => error
@@ -191,118 +195,6 @@ module SourceMonitor
       end
     end
 
-    def source_health_badge(source, override: nil)
-      return override if override.present?
-
-      status = source&.health_status.presence || "healthy"
-
-      mapping = {
-        "healthy" => { label: "Healthy", classes: "bg-green-100 text-green-700", show_spinner: false },
-        "warning" => { label: "Needs Attention", classes: "bg-amber-100 text-amber-700", show_spinner: false },
-        "critical" => { label: "Failing", classes: "bg-rose-100 text-rose-700", show_spinner: false },
-        "declining" => { label: "Declining", classes: "bg-orange-100 text-orange-700", show_spinner: false },
-        "improving" => { label: "Improving", classes: "bg-sky-100 text-sky-700", show_spinner: false },
-        "auto_paused" => { label: "Auto-Paused", classes: "bg-amber-100 text-amber-700", show_spinner: false },
-        "unknown" => { label: "Unknown", classes: "bg-slate-100 text-slate-600", show_spinner: false }
-      }
-
-      mapping.fetch(status) { mapping.fetch("unknown") }.merge(status: status)
-    end
-
-    def source_health_actions(source)
-      status = source&.health_status.presence || "healthy"
-      helpers = SourceMonitor::Engine.routes.url_helpers
-
-      case status
-      when "critical", "declining"
-        [
-          {
-            key: :full_fetch,
-            label: "Queue Full Fetch",
-            description: "Runs the full fetch pipeline immediately and updates items if the feed responds.",
-            path: helpers.source_fetch_path(source),
-            method: :post,
-            data: { testid: "source-health-action-full_fetch" }
-          },
-          {
-            key: :health_check,
-            label: "Run Health Check",
-            description: "Sends a single request to confirm the feed is reachable without modifying stored items.",
-            path: helpers.source_health_check_path(source),
-            method: :post,
-            data: { testid: "source-health-action-health_check" }
-          }
-        ]
-      when "auto_paused"
-        [
-          {
-            key: :reset,
-            label: "Reset to Active Status",
-            description: "Clears the pause window, failure counters, and schedules the next fetch using the configured interval.",
-            path: helpers.source_health_reset_path(source),
-            method: :post,
-            data: { testid: "source-health-action-reset" }
-          }
-        ]
-      else
-        []
-      end
-    end
-
-    def interactive_health_status?(source, override: nil)
-      return false if override.present?
-
-      %w[critical declining auto_paused].include?(source&.health_status.presence)
-    end
-
-    def table_sort_direction(search_object, attribute)
-      return unless search_object.respond_to?(:sorts)
-
-      sort = search_object.sorts.detect { |s| s && s.name == attribute.to_s }
-      sort&.dir
-    end
-
-    def table_sort_arrow(search_object, attribute, default: nil)
-      direction = table_sort_direction(search_object, attribute) || default&.to_s
-
-      case direction
-      when "asc"
-        "▲"
-      when "desc"
-        "▼"
-      else
-        "↕"
-      end
-    end
-
-    def table_sort_aria(search_object, attribute)
-      direction = table_sort_direction(search_object, attribute)
-
-      case direction
-      when "asc"
-        "ascending"
-      when "desc"
-        "descending"
-      else
-        "none"
-      end
-    end
-
-    def table_sort_link(search_object, attribute, label, frame:, default_order:, secondary: [], html_options: {})
-      sort_targets = [ attribute, *Array(secondary) ]
-      options = {
-        default_order: default_order,
-        hide_indicator: true
-      }.merge(html_options)
-
-      options[:data] = (options[:data] || {}).merge(turbo_frame: frame)
-      options[:data][:turbo_action] ||= "advance"
-
-      sort_link(search_object, attribute, sort_targets, options) do
-        tag.span(label, class: "inline-flex items-center gap-1")
-      end
-    end
-
     def formatted_setting_value(value)
       case value
       when TrueClass

data/app/helpers/source_monitor/health_badge_helper.rb

@@ -0,0 +1,69 @@
+# frozen_string_literal: true
+
+module SourceMonitor
+  module HealthBadgeHelper
+    def source_health_badge(source, override: nil)
+      return override if override.present?
+
+      status = source&.health_status.presence || "healthy"
+
+      mapping = {
+        "healthy" => { label: "Healthy", classes: "bg-green-100 text-green-700", show_spinner: false },
+        "warning" => { label: "Needs Attention", classes: "bg-amber-100 text-amber-700", show_spinner: false },
+        "critical" => { label: "Failing", classes: "bg-rose-100 text-rose-700", show_spinner: false },
+        "declining" => { label: "Declining", classes: "bg-orange-100 text-orange-700", show_spinner: false },
+        "improving" => { label: "Improving", classes: "bg-sky-100 text-sky-700", show_spinner: false },
+        "auto_paused" => { label: "Auto-Paused", classes: "bg-amber-100 text-amber-700", show_spinner: false },
+        "unknown" => { label: "Unknown", classes: "bg-slate-100 text-slate-600", show_spinner: false }
+      }
+
+      mapping.fetch(status) { mapping.fetch("unknown") }.merge(status: status)
+    end
+
+    def source_health_actions(source)
+      status = source&.health_status.presence || "healthy"
+      helpers = SourceMonitor::Engine.routes.url_helpers
+
+      case status
+      when "critical", "declining"
+        [
+          {
+            key: :full_fetch,
+            label: "Queue Full Fetch",
+            description: "Runs the full fetch pipeline immediately and updates items if the feed responds.",
+            path: helpers.source_fetch_path(source),
+            method: :post,
+            data: { testid: "source-health-action-full_fetch" }
+          },
+          {
+            key: :health_check,
+            label: "Run Health Check",
+            description: "Sends a single request to confirm the feed is reachable without modifying stored items.",
+            path: helpers.source_health_check_path(source),
+            method: :post,
+            data: { testid: "source-health-action-health_check" }
+          }
+        ]
+      when "auto_paused"
+        [
+          {
+            key: :reset,
+            label: "Reset to Active Status",
+            description: "Clears the pause window, failure counters, and schedules the next fetch using the configured interval.",
+            path: helpers.source_health_reset_path(source),
+            method: :post,
+            data: { testid: "source-health-action-reset" }
+          }
+        ]
+      else
+        []
+      end
+    end
+
+    def interactive_health_status?(source, override: nil)
+      return false if override.present?
+
+      %w[critical declining auto_paused].include?(source&.health_status.presence)
+    end
+  end
+end

data/app/helpers/source_monitor/table_sort_helper.rb

@@ -0,0 +1,53 @@
+# frozen_string_literal: true
+
+module SourceMonitor
+  module TableSortHelper
+    def table_sort_direction(search_object, attribute)
+      return unless search_object.respond_to?(:sorts)
+
+      sort = search_object.sorts.detect { |s| s && s.name == attribute.to_s }
+      sort&.dir
+    end
+
+    def table_sort_arrow(search_object, attribute, default: nil)
+      direction = table_sort_direction(search_object, attribute) || default&.to_s
+
+      case direction
+      when "asc"
+        "▲"
+      when "desc"
+        "▼"
+      else
+        "↕"
+      end
+    end
+
+    def table_sort_aria(search_object, attribute)
+      direction = table_sort_direction(search_object, attribute)
+
+      case direction
+      when "asc"
+        "ascending"
+      when "desc"
+        "descending"
+      else
+        "none"
+      end
+    end
+
+    def table_sort_link(search_object, attribute, label, frame:, default_order:, secondary: [], html_options: {})
+      sort_targets = [ attribute, *Array(secondary) ]
+      options = {
+        default_order: default_order,
+        hide_indicator: true
+      }.merge(html_options)
+
+      options[:data] = (options[:data] || {}).merge(turbo_frame: frame)
+      options[:data][:turbo_action] ||= "advance"
+
+      sort_link(search_object, attribute, sort_targets, options) do
+        tag.span(label, class: "inline-flex items-center gap-1")
+      end
+    end
+  end
+end

data/app/jobs/source_monitor/application_job.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SourceMonitor
   parent_job = defined?(::ApplicationJob) ? ::ApplicationJob : ActiveJob::Base
 

data/app/models/source_monitor/application_record.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SourceMonitor
   class ApplicationRecord < ActiveRecord::Base
     self.abstract_class = true

data/app/models/source_monitor/log_entry.rb

@@ -2,8 +2,6 @@
 
 module SourceMonitor
   class LogEntry < ApplicationRecord
-    self.table_name = "sourcemon_log_entries"
-
     delegated_type :loggable, types: %w[SourceMonitor::FetchLog SourceMonitor::ScrapeLog SourceMonitor::HealthCheckLog]
 
     belongs_to :source, class_name: "SourceMonitor::Source", inverse_of: :log_entries