source_monitor 0.13.0 → 0.13.1

data/lib/source_monitor/import_sessions/wizard.rb

@@ -0,0 +1,612 @@
+# frozen_string_literal: true
+
+require "nokogiri"
+require "uri"
+require "active_support/core_ext/object/blank"
+require "source_monitor/import_sessions/entry_normalizer"
+
+module SourceMonitor
+  module ImportSessions
+    class Wizard
+      ALLOWED_CONTENT_TYPES = %w[text/xml application/xml text/x-opml application/opml].freeze
+      GENERIC_CONTENT_TYPES = %w[application/octet-stream binary/octet-stream].freeze
+
+      class UploadError < StandardError; end
+      TRUE_PARAM_VALUES = [ true, "true", "1", 1, "on" ].freeze
+
+      UploadResult = Struct.new(:status, :errors, :current_step, :preview_context, keyword_init: true)
+
+      PreviewResult = Struct.new(
+        :status,
+        :selected_source_ids,
+        :valid_ids,
+        :current_step,
+        :selection_error,
+        :preview_context,
+        keyword_init: true
+      ) do
+        def blocked?
+          status == :blocked
+        end
+      end
+
+      PreviewContext = Struct.new(
+        :filter,
+        :page,
+        :selected_source_ids,
+        :preview_entries,
+        :filtered_entries,
+        :paginated_entries,
+        :has_next_page,
+        :has_previous_page,
+        keyword_init: true
+      )
+
+      HealthCheckResult = Struct.new(
+        :status,
+        :selected_source_ids,
+        :current_step,
+        :selection_error,
+        :health_check_context,
+        keyword_init: true
+      ) do
+        def blocked?
+          status == :blocked
+        end
+      end
+
+      HealthCheckContext = Struct.new(
+        :selected_source_ids,
+        :health_check_entries,
+        :health_check_target_ids,
+        :health_progress,
+        keyword_init: true
+      )
+
+      ConfirmResult = Struct.new(
+        :status,
+        :selected_source_ids,
+        :selected_entries,
+        :bulk_settings,
+        :selection_error,
+        :message,
+        :history,
+        keyword_init: true
+      ) do
+        def blocked?
+          status == :blocked
+        end
+      end
+
+      ConfirmContext = Struct.new(:selected_source_ids, :selected_entries, :bulk_settings, keyword_init: true)
+
+      def initialize(import_session:, params:, current_step:, now: Time.current)
+        @import_session = import_session
+        @params = params
+        @current_step = current_step
+        @now = now
+      end
+
+      def handle_upload
+        errors = validate_upload
+        return UploadResult.new(status: :invalid, errors: errors, current_step: current_step) if errors.any?
+
+        parsed_entries = parse_opml_file(opml_file)
+        valid_entries = parsed_entries.select { |entry| entry[:status] == "valid" }
+
+        if valid_entries.empty?
+          import_session.update!(
+            opml_file_metadata: build_file_metadata,
+            parsed_sources: parsed_entries,
+            current_step: "upload"
+          )
+
+          return UploadResult.new(
+            status: :invalid,
+            errors: [ "We couldn't find any valid feeds in that OPML file. Check the file and try again." ],
+            current_step: "upload"
+          )
+        end
+
+        next_step = target_step
+        import_session.update!(
+          opml_file_metadata: build_file_metadata.merge("uploaded_at" => now),
+          parsed_sources: parsed_entries,
+          current_step: next_step
+        )
+
+        UploadResult.new(
+          status: :success,
+          errors: [],
+          current_step: next_step,
+          preview_context: preview_context
+        )
+      rescue UploadError => error
+        UploadResult.new(status: :invalid, errors: [ error.message ], current_step: current_step)
+      end
+
+      def handle_preview
+        selected_source_ids = Array(import_session.selected_source_ids).map(&:to_s)
+        preview_entries = annotated_entries(selected_source_ids)
+        selectable_entries = preview_entries.select { |entry| entry[:selectable] }
+
+        valid_ids = if select_all_requested?
+          ids = selectable_entries.map { |entry| entry[:id] }
+          import_session.update_column(:selected_source_ids, ids)
+          ids
+        elsif select_none_requested?
+          import_session.update_column(:selected_source_ids, [])
+          []
+        else
+          requested_ids = build_selection_from_params(selectable_entries)
+          ids = selectable_entries.index_by { |entry| entry[:id] }.slice(*requested_ids).keys
+          import_session.update!(selected_source_ids: ids)
+          ids
+        end
+
+        if advancing_from_preview? && valid_ids.empty?
+          return PreviewResult.new(
+            status: :blocked,
+            selected_source_ids: valid_ids,
+            valid_ids: valid_ids,
+            current_step: current_step,
+            selection_error: "Select at least one new source to continue.",
+            preview_context: preview_context(selected_source_ids: valid_ids)
+          )
+        end
+
+        next_step = target_step
+        import_session.update_column(:current_step, next_step) if import_session.current_step != next_step
+
+        PreviewResult.new(
+          status: :success,
+          selected_source_ids: valid_ids,
+          valid_ids: valid_ids,
+          current_step: next_step,
+          preview_context: preview_context(selected_source_ids: valid_ids)
+        )
+      end
+
+      def handle_health_check
+        selected_source_ids = health_check_selection_from_params
+        import_session.update!(selected_source_ids: selected_source_ids)
+
+        if advancing_from_health_check? && selected_source_ids.blank?
+          deactivate_health_checks
+
+          return HealthCheckResult.new(
+            status: :blocked,
+            selected_source_ids: selected_source_ids,
+            current_step: current_step,
+            selection_error: "Select at least one source to continue.",
+            health_check_context: health_check_context
+          )
+        end
+
+        next_step = target_step
+        deactivate_health_checks if next_step != "health_check"
+        import_session.update_column(:current_step, next_step) if import_session.current_step != next_step
+
+        HealthCheckResult.new(
+          status: :success,
+          selected_source_ids: selected_source_ids,
+          current_step: next_step,
+          health_check_context: (health_check_context if next_step == "health_check")
+        )
+      end
+
+      def handle_confirm
+        context = confirm_context
+
+        if context.selected_entries.empty?
+          return ConfirmResult.new(
+            status: :blocked,
+            selected_source_ids: context.selected_source_ids,
+            selected_entries: context.selected_entries,
+            bulk_settings: context.bulk_settings,
+            selection_error: "Select at least one source to import."
+          )
+        end
+
+        history = SourceMonitor::ImportHistory.create!(
+          user_id: import_session.user_id,
+          bulk_settings: import_session.bulk_settings
+        )
+        SourceMonitor::ImportOpmlJob.perform_later(import_session.id, history.id)
+        import_session.update_column(:current_step, "confirm") if import_session.current_step != "confirm"
+
+        ConfirmResult.new(
+          status: :success,
+          selected_source_ids: context.selected_source_ids,
+          selected_entries: context.selected_entries,
+          bulk_settings: context.bulk_settings,
+          history: history,
+          message: "Import started for #{context.selected_entries.size} sources."
+        )
+      end
+
+      def preview_context(selected_source_ids: nil)
+        filter = permitted_filter(params[:filter]) || "all"
+        page = normalize_page_param(params[:page])
+        selected_source_ids = Array(selected_source_ids || import_session.selected_source_ids).map(&:to_s)
+        preview_entries = annotated_entries(selected_source_ids)
+
+        filtered_entries = filter_entries(preview_entries, filter)
+        paginator = SourceMonitor::Pagination::Paginator.new(
+          scope: filtered_entries,
+          page: page,
+          per_page: preview_per_page
+        ).paginate
+
+        PreviewContext.new(
+          filter: filter,
+          page: paginator.page,
+          selected_source_ids: selected_source_ids,
+          preview_entries: preview_entries,
+          filtered_entries: filtered_entries,
+          paginated_entries: paginator.records,
+          has_next_page: paginator.has_next_page,
+          has_previous_page: paginator.has_previous_page
+        )
+      end
+
+      def preview_context_with_default_selection
+        selected_source_ids = Array(import_session.selected_source_ids).map(&:to_s)
+        preview_entries = annotated_entries(selected_source_ids)
+
+        if selected_source_ids.blank? && preview_entries.present?
+          selected_source_ids = preview_entries.select { |entry| entry[:selectable] }.map { |entry| entry[:id] }
+          import_session.update_column(:selected_source_ids, selected_source_ids)
+        end
+
+        preview_context(selected_source_ids: selected_source_ids)
+      end
+
+      def health_check_context
+        start_health_checks_if_needed
+
+        selected_source_ids = Array(import_session.selected_source_ids).map(&:to_s)
+        entries = health_check_entries(selected_source_ids)
+        target_ids = health_check_targets
+
+        HealthCheckContext.new(
+          selected_source_ids: selected_source_ids,
+          health_check_entries: entries,
+          health_check_target_ids: target_ids,
+          health_progress: health_check_progress(entries)
+        )
+      end
+
+      def deactivate_health_checks
+        return unless import_session.health_checks_active?
+
+        import_session.update_columns(
+          health_checks_active: false,
+          health_check_completed_at: now
+        )
+      end
+
+      def confirm_context
+        selected_source_ids = Array(import_session.selected_source_ids).map(&:to_s)
+        selected_entries = annotated_entries(selected_source_ids)
+          .select { |entry| selected_source_ids.include?(entry[:id]) }
+
+        ConfirmContext.new(
+          selected_source_ids: selected_source_ids,
+          selected_entries: selected_entries,
+          bulk_settings: import_session.bulk_settings || {}
+        )
+      end
+
+      private
+
+      attr_reader :import_session, :params, :current_step, :now
+
+      def validate_upload
+        return [ "Upload an OPML file to continue." ] unless opml_file.present?
+
+        errors = []
+        errors << "The uploaded file is empty. Choose another OPML file." if opml_file.size.to_i <= 0
+
+        if opml_file.content_type.present? && !content_type_allowed?(opml_file.content_type) && !generic_content_type?(opml_file.content_type)
+          errors << "Upload must be an OPML or XML file."
+        end
+
+        errors
+      end
+
+      def opml_file
+        params[:opml_file]
+      end
+
+      def build_file_metadata
+        return {} unless opml_file.respond_to?(:original_filename)
+
+        {
+          "filename" => opml_file.original_filename,
+          "byte_size" => opml_file.size,
+          "content_type" => opml_file.content_type
+        }
+      end
+
+      def content_type_allowed?(content_type)
+        ALLOWED_CONTENT_TYPES.include?(content_type)
+      end
+
+      def generic_content_type?(content_type)
+        GENERIC_CONTENT_TYPES.include?(content_type)
+      end
+
+      def parse_opml_file(file)
+        content = file.read
+        file.rewind if file.respond_to?(:rewind)
+
+        raise UploadError, "The uploaded file appears to be empty." if content.blank?
+
+        document = Nokogiri::XML(content) { |config| config.strict.nonet }
+        raise UploadError, "The uploaded file is not valid XML or OPML." if document.root.nil?
+
+        document.xpath("//outline").each_with_index.filter_map do |outline, index|
+          next unless outline.attribute_nodes.any? { |attr| attr.name.casecmp("xmlurl").zero? }
+
+          build_entry(outline, index)
+        end
+      rescue Nokogiri::XML::SyntaxError => error
+        raise UploadError, "We couldn't parse that OPML file: #{error.message}"
+      end
+
+      def build_entry(outline, index)
+        feed_url = outline_attribute(outline, "xmlUrl")
+        website_url = outline_attribute(outline, "htmlUrl")
+        title = outline_attribute(outline, "title") || outline_attribute(outline, "text")
+
+        if feed_url.blank?
+          return malformed_entry(index, feed_url, title, website_url, "Missing feed URL")
+        end
+
+        unless valid_feed_url?(feed_url)
+          return malformed_entry(index, feed_url, title, website_url, "Feed URL must be HTTP or HTTPS")
+        end
+
+        {
+          id: "outline-#{index}",
+          raw_outline_index: index,
+          feed_url: feed_url,
+          title: title,
+          website_url: website_url,
+          status: "valid",
+          error: nil,
+          health_status: nil,
+          health_error: nil
+        }
+      end
+
+      def malformed_entry(index, feed_url, title, website_url, error)
+        {
+          id: "outline-#{index}",
+          raw_outline_index: index,
+          feed_url: feed_url.presence,
+          title: title,
+          website_url: website_url,
+          status: "malformed",
+          error: error,
+          health_status: nil,
+          health_error: nil
+        }
+      end
+
+      def outline_attribute(outline, name)
+        attribute = outline.attribute_nodes.find { |attr| attr.name.casecmp(name).zero? }
+        attribute&.value.to_s.presence
+      end
+
+      def valid_feed_url?(url)
+        parsed = URI.parse(url)
+        parsed.is_a?(URI::HTTP) && parsed.host.present?
+      rescue URI::InvalidURIError
+        false
+      end
+
+      def annotated_entries(selected_ids)
+        selected_ids ||= []
+        entries = Array(import_session.parsed_sources)
+        return [] if entries.blank?
+
+        normalized = entries.map { |entry| normalize_entry(entry) }
+        feed_urls = normalized.filter_map { |entry| entry[:feed_url]&.downcase }
+        duplicate_lookup = if feed_urls.present?
+          SourceMonitor::Source.where("LOWER(feed_url) IN (?)", feed_urls).pluck(:feed_url).map(&:downcase)
+        else
+          []
+        end
+
+        normalized.map do |entry|
+          duplicate = entry[:feed_url].present? && duplicate_lookup.include?(entry[:feed_url].downcase)
+          entry.merge(
+            duplicate: duplicate,
+            selectable: entry[:status] == "valid" && !duplicate,
+            selected: selected_ids.include?(entry[:id])
+          )
+        end
+      end
+
+      def normalize_entry(entry)
+        SourceMonitor::ImportSessions::EntryNormalizer.normalize(entry)
+      end
+
+      def filter_entries(entries, filter)
+        case filter
+        when "new"
+          entries.select { |entry| entry[:selectable] }
+        when "existing"
+          entries.select { |entry| entry[:duplicate] }
+        else
+          entries
+        end
+      end
+
+      def build_selection_from_params(selectable_entries)
+        ids = import_session_params[:selected_source_ids]
+        return [] unless ids
+
+        Array(ids).map(&:to_s).uniq & selectable_entries.map { |entry| entry[:id] }
+      end
+
+      def advancing_from_preview?
+        target_step != "preview"
+      end
+
+      def health_check_selection_from_params
+        if select_all_requested?
+          return health_check_targets.dup
+        end
+
+        return [] if select_none_requested?
+
+        ids = import_session_params[:selected_source_ids]
+        return Array(import_session.selected_source_ids).map(&:to_s) unless ids
+
+        Array(ids).map(&:to_s).uniq & health_check_targets
+      end
+
+      def advancing_from_health_check?
+        target_step != "health_check"
+      end
+
+      def start_health_checks_if_needed
+        return unless current_step == "health_check"
+
+        jobs_to_enqueue = []
+
+        import_session.with_lock do
+          import_session.reload
+          selected = Array(import_session.selected_source_ids).map(&:to_s)
+
+          if selected.blank?
+            import_session.update_columns(health_checks_active: false, health_check_target_ids: [])
+            next
+          end
+
+          if import_session.health_checks_active? && import_session.health_check_targets.sort == selected.sort
+            next
+          end
+
+          import_session.update!(
+            parsed_sources: reset_health_results(import_session.parsed_sources, selected),
+            health_checks_active: true,
+            health_check_target_ids: selected,
+            health_check_started_at: now,
+            health_check_completed_at: nil
+          )
+
+          jobs_to_enqueue = selected
+        end
+
+        enqueue_health_check_jobs(import_session, jobs_to_enqueue) if jobs_to_enqueue.any?
+      end
+
+      def reset_health_results(entries, target_ids)
+        Array(entries).map do |entry|
+          entry_hash = entry.to_h
+          entry_id = entry_hash["id"] || entry_hash[:id]
+          next entry_hash unless target_ids.include?(entry_id.to_s)
+
+          entry_hash.merge("health_status" => "pending", "health_error" => nil)
+        end
+      end
+
+      def enqueue_health_check_jobs(import_session, target_ids)
+        target_ids.each do |target_id|
+          SourceMonitor::ImportSessionHealthCheckJob.set(wait: 1.second).perform_later(import_session.id, target_id)
+        end
+      end
+
+      def health_check_entries(selected_ids)
+        targets = health_check_targets
+        entries = Array(import_session.parsed_sources).map { |entry| normalize_entry(entry) }
+
+        entries.select { |entry| targets.include?(entry[:id]) }.map do |entry|
+          entry.merge(selected: selected_ids.include?(entry[:id]))
+        end
+      end
+
+      def health_check_progress(entries)
+        total = health_check_targets.size
+        completed = entries.count { |entry| health_check_complete?(entry) }
+
+        {
+          completed: completed,
+          total: total,
+          pending: [ total - completed, 0 ].max,
+          active: import_session.health_checks_active?,
+          done: total.positive? && completed >= total
+        }
+      end
+
+      def health_check_complete?(entry)
+        %w[working failing].include?(entry[:health_status].to_s)
+      end
+
+      def health_check_targets
+        targets = import_session.health_check_targets
+        targets = Array(import_session.selected_source_ids).map(&:to_s) if targets.blank?
+        targets
+      end
+
+      def normalize_page_param(value)
+        number = value.to_i
+        number = 1 if number <= 0
+        number
+      rescue StandardError
+        1
+      end
+
+      def permitted_filter(raw)
+        value = raw.to_s.presence
+        return unless value
+
+        %w[all new existing].find { |candidate| candidate == value }
+      end
+
+      def preview_per_page
+        25
+      end
+
+      def target_step
+        permitted_step(import_session_params[:next_step]) || current_step || ImportSession.default_step
+      end
+
+      def permitted_step(value)
+        step = value.to_s.presence
+        return unless step
+
+        ImportSession::STEP_ORDER.find { |candidate| candidate == step }
+      end
+
+      def import_session_params
+        @import_session_params ||= begin
+          raw = params[:import_session] || params["import_session"] || {}
+          permitted = if raw.respond_to?(:permit)
+            raw.permit(:next_step, :select_all, :select_none, selected_source_ids: [])
+          else
+            raw.to_h
+          end
+
+          SourceMonitor::Security::ParameterSanitizer.sanitize(permitted.to_h).with_indifferent_access
+        end
+      end
+
+      def select_all_requested?
+        truthy_import_session_param?(:select_all)
+      end
+
+      def select_none_requested?
+        truthy_import_session_param?(:select_none)
+      end
+
+      def truthy_import_session_param?(key)
+        TRUE_PARAM_VALUES.include?(import_session_params[key])
+      end
+    end
+  end
+end

data/lib/source_monitor/items/batch_item_creator.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "source_monitor/items/item_creator"
+require "source_monitor/items/normalized_entry"
 
 module SourceMonitor
   module Items
@@ -32,17 +33,17 @@ module SourceMonitor
 
         # Step 1: Pre-parse entries to extract GUIDs and fingerprints for bulk lookup.
         entry_identifiers = @entries.map do |entry|
-          parser = ItemCreator::EntryParser.new(
+          normalized_entry = NormalizedEntry.new(
             source: @source,
             entry: entry,
             content_extractor: content_extractor
           )
-          attrs = parser.parse
-          raw_guid = attrs[:guid]
-          normalized_guid = raw_guid.present? ? raw_guid.downcase : nil
-          guid = normalized_guid.presence || attrs[:content_fingerprint]
 
-          { guid: guid, fingerprint: attrs[:content_fingerprint], raw_guid_present: normalized_guid.present? }
+          {
+            guid: normalized_entry.item_guid,
+            fingerprint: normalized_entry.content_fingerprint,
+            raw_guid_present: normalized_entry.raw_guid_present?
+          }
         end
 
         # Step 2: Batch-fetch existing items by GUID (single query)

data/lib/source_monitor/items/item_creator.rb

@@ -9,6 +9,7 @@ require "source_monitor/instrumentation"
 require "source_monitor/scrapers/readability"
 require "source_monitor/items/item_creator/entry_parser"
 require "source_monitor/items/item_creator/content_extractor"
+require "source_monitor/items/normalized_entry"
 
 module SourceMonitor
   module Items
@@ -49,14 +50,10 @@ module SourceMonitor
       end
 
       def call
-        attributes = build_attributes
-        raw_guid = attributes[:guid]
-        # Normalize GUID to lowercase so the plain btree index on guid is used
-        # for lookups instead of LOWER(guid) which forces sequential scans.
-        normalized_guid = raw_guid.present? ? raw_guid.downcase : nil
-        attributes[:guid] = normalized_guid.presence || attributes[:content_fingerprint]
+        normalized_entry = build_normalized_entry
+        attributes = normalized_entry.item_attributes
 
-        existing_item, matched_by = existing_item_for(attributes, raw_guid_present: normalized_guid.present?)
+        existing_item, matched_by = existing_item_for(attributes, raw_guid_present: normalized_entry.raw_guid_present?)
 
         if existing_item
           apply_attributes(existing_item, attributes)
@@ -70,7 +67,7 @@ module SourceMonitor
           end
         end
 
-        create_new_item(attributes, raw_guid_present: normalized_guid.present?)
+        create_new_item(attributes, raw_guid_present: normalized_entry.raw_guid_present?)
       end
 
       private
@@ -193,12 +190,8 @@ module SourceMonitor
         (record.changed - IGNORED_CHANGE_ATTRIBUTES).any?
       end
 
-      def build_attributes
-        entry_parser.parse
-      end
-
-      def entry_parser
-        @entry_parser ||= EntryParser.new(source: source, entry: entry, content_extractor: content_extractor)
+      def build_normalized_entry
+        @normalized_entry ||= NormalizedEntry.new(source: source, entry: entry, content_extractor: content_extractor)
       end
 
       def content_extractor