woods 2.0.0.beta2 → 2.0.0.beta4

data/lib/woods/extractors/mailer_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require 'digest'
 require_relative 'ast_source_extraction'
 require_relative 'shared_utility_methods'
@@ -27,7 +29,9 @@ module Woods
       include RouteHelperResolver
 
       def initialize
-        @mailer_base = defined?(ApplicationMailer) ? ApplicationMailer : ActionMailer::Base
+        if defined?(ActionMailer::Base)
+          @mailer_base = defined?(ApplicationMailer) ? ApplicationMailer : ActionMailer::Base
+        end
         build_route_helper_map
       end
 
@@ -45,7 +49,9 @@ module Woods
       #
       # @return [Array<Class>]
       def discoverable_classes
-        @mailer_base.descendants
+        return [] unless @mailer_base
+
+        @mailer_base.descendants.select { |mailer| app_defined_mailer?(mailer) }
       end
 
       # Extract a single mailer
@@ -53,8 +59,7 @@ module Woods
       # @param mailer [Class] The mailer class
       # @return [ExtractedUnit] The extracted unit
       def extract_mailer(mailer)
-        return nil if mailer.name.nil?
-        return nil if mailer == ActionMailer::Base
+        return nil unless app_defined_mailer?(mailer)
 
         file_path = source_file_for(mailer)
 
@@ -76,12 +81,22 @@ module Woods
 
         unit
       rescue StandardError => e
-        Rails.logger.error("Failed to extract mailer #{mailer.name}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract mailer #{mailer.name}: #{e.message}")
         nil
       end
 
       private
 
+      # Discovery and direct incremental extraction must agree on ownership;
+      # otherwise a gem class with a fabricated path becomes a phantom unit.
+      def app_defined_mailer?(mailer)
+        return false unless @mailer_base && mailer.name
+        return false if mailer == ActionMailer::Base
+
+        path = source_file_for(mailer)
+        path && File.file?(path) && app_source?(path, Rails.root.to_s)
+      end
+
       # Locate the source file for a mailer class.
       #
       # Convention path first, then introspection via {#resolve_source_location}
@@ -101,12 +116,8 @@ module Woods
       # ──────────────────────────────────────────────────────────────────────
 
       def annotate_source(source, mailer)
-        actions = mailer.action_methods.to_a
-        default_from = begin
-          mailer.default[:from]
-        rescue StandardError
-          nil
-        end
+        actions = mailer.action_methods.sort
+        default_from = extract_defaults(mailer)[:from]
 
         <<~ANNOTATION
           # ╔═══════════════════════════════════════════════════════════════════════╗
@@ -124,7 +135,7 @@ module Woods
       # ──────────────────────────────────────────────────────────────────────
 
       def extract_metadata(mailer, source)
-        actions = mailer.action_methods.to_a
+        actions = mailer.action_methods.sort
 
         {
           # Actions (mail methods)
@@ -156,7 +167,7 @@ module Woods
 
       def extract_defaults(mailer)
         mailer_defaults = mailer.default
-        mailer_defaults.slice(:from, :reply_to, :cc, :bcc).compact
+        mailer_defaults.slice(:from, :reply_to, :cc, :bcc).compact.transform_values { |value| stable_filter(value) }
       rescue StandardError
         {}
       end
@@ -167,7 +178,7 @@ module Woods
 
           result = {
             type: :"#{cb.kind}_action",
-            filter: cb.filter.to_s
+            filter: callback_filter_label(cb)
           }
           result[:only] = only if only.any?
           result[:except] = except if except.any?
@@ -259,7 +270,7 @@ module Woods
       # ──────────────────────────────────────────────────────────────────────
 
       def build_action_chunks(mailer, _source)
-        mailer.action_methods.filter_map do |action|
+        mailer.action_methods.sort.filter_map do |action|
           action_source = extract_action_source(mailer, action)
           next if action_source.nil? || action_source.strip.empty?
 

data/lib/woods/extractors/manager_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require_relative 'shared_utility_methods'
 require_relative 'shared_dependency_scanner'
 
@@ -69,7 +71,7 @@ module Woods
 
         unit
       rescue StandardError => e
-        Rails.logger.error("Failed to extract manager #{file_path}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract manager #{file_path}: #{e.message}")
         nil
       end
 

data/lib/woods/extractors/method_parameters.rb

@@ -0,0 +1,53 @@
+# frozen_string_literal: true
+
+require 'prism'
+
+module Woods
+  module Extractors
+    # Syntactic method signatures only: never evaluate a default expression or
+    # mistake names inside one for additional parameters. Consumers retain their
+    # existing metadata shapes by selecting the fields they expose.
+    module MethodParameters
+      module_function
+
+      def extract(source, method_name)
+        parsed = Prism.parse(source)
+        return [] unless parsed.success?
+
+        method = find_method(parsed.value, method_name.to_sym)
+        return [] unless method&.parameters
+
+        method.parameters.compact_child_nodes.flat_map { |node| describe(node) }
+      end
+
+      def find_method(node, name)
+        return nil if node.is_a?(Prism::SingletonClassNode)
+        return node if node.is_a?(Prism::DefNode) && node.receiver.nil? && node.name == name
+
+        node.compact_child_nodes.each do |child|
+          found = find_method(child, name)
+          return found if found
+        end
+        nil
+      end
+
+      def describe(node)
+        # Destructured positional arguments contain declarations, while a
+        # default's subtree contains values and must never be walked.
+        return node.compact_child_nodes.flat_map { |child| describe(child) } if node.is_a?(Prism::MultiTargetNode)
+        return [] unless node.respond_to?(:name) && node.name
+
+        optional = node.is_a?(Prism::OptionalParameterNode) || node.is_a?(Prism::OptionalKeywordParameterNode)
+        keyword = node.is_a?(Prism::RequiredKeywordParameterNode) ||
+                  node.is_a?(Prism::OptionalKeywordParameterNode) || node.is_a?(Prism::KeywordRestParameterNode)
+        splat = if node.is_a?(Prism::KeywordRestParameterNode)
+                  :double
+                elsif node.is_a?(Prism::RestParameterNode)
+                  :single
+                end
+        [{ name: node.name.to_s, has_default: optional, keyword: keyword, splat: splat }]
+      end
+      private_class_method :find_method, :describe
+    end
+  end
+end

data/lib/woods/extractors/middleware_argument.rb

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+
+module Woods
+  module Extractors
+    # Stable, readable middleware arguments without inspecting opaque runtime state.
+    # Literal strings and custom #to_s implementations remain application data.
+    module MiddlewareArgument
+      module_function
+
+      # @param value [Object] Runtime middleware argument
+      # @return [String] Deterministic description for ordinary Ruby runtime values
+      def render(value, ancestors = [], nested: false)
+        return "<recursive #{value.class}>" if ancestors.any? { |ancestor| ancestor.equal?(value) }
+
+        ancestors += [value]
+        case value
+        when String then nested ? value.inspect : value
+        when Symbol then nested ? value.inspect : value.to_s
+        when NilClass then nested ? 'nil' : ''
+        when Array then "[#{value.map { |item| render(item, ancestors, nested: true) }.join(', ')}]"
+        when Hash
+          pairs = value.map { |key, item| "#{render(key, ancestors, nested: true)}=>#{render(item, ancestors, nested: true)}" }
+          "{#{pairs.join(', ')}}"
+        when Module then module_name(value)
+        when Proc then "#<#{value.lambda? ? 'lambda' : 'Proc'} #{value.source_location&.join(':') || 'native'}>"
+        else
+          # Only Object's identity-based implementation is replaced. Never scrub
+          # arbitrary hex strings or override application-defined textual values.
+          if [Kernel, Object].include?(value.method(:to_s).owner)
+            "#<#{module_name(value.class)}>"
+          else
+            value.to_s
+          end
+        end
+      end
+
+      def module_name(value)
+        return value.name if constant_path?(value.name)
+
+        parent = value.is_a?(Class) ? " < #{module_name(value.superclass)}" : ''
+        methods = (value.instance_methods(false) + value.private_instance_methods(false)).uniq.sort.map do |name|
+          "#{name}@#{value.instance_method(name).source_location&.join(':') || 'native'}"
+        end
+        "#<anonymous #{value.class}#{parent}#{" (#{methods.join(', ')})" unless methods.empty?}>"
+      end
+
+      # Ruby temporary names are explicitly not constant paths and can contain
+      # runtime identities (Rails 8.2 uses these for executor/reloader classes).
+      # Validate each segment with Ruby itself, including Unicode constants,
+      # against an empty namespace: never resolve or autoload application names.
+      def constant_path?(name)
+        return false if name.nil? || name.empty?
+
+        namespace = Module.new
+        name.split('::', -1).all? do |segment|
+          namespace.const_defined?(segment, false)
+          true
+        end
+      rescue NameError
+        false
+      end
+      private_class_method :module_name, :constant_path?
+    end
+  end
+end

data/lib/woods/extractors/middleware_extractor.rb

@@ -1,5 +1,9 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
+require_relative 'middleware_argument'
+
 module Woods
   module Extractors
     # MiddlewareExtractor handles Rack middleware stack extraction via runtime introspection.
@@ -42,7 +46,7 @@ module Woods
 
         [unit]
       rescue StandardError => e
-        Rails.logger.error("Failed to extract middleware stack: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract middleware stack: #{e.message}")
         []
       end
 
@@ -80,7 +84,9 @@ module Woods
       # @param position [Integer] Position in the stack
       # @return [Hash, nil]
       def extract_single_middleware(middleware, position)
-        name = if middleware.respond_to?(:name)
+        name = if middleware.respond_to?(:klass) && middleware.klass.is_a?(Module)
+                 MiddlewareArgument.render(middleware.klass)
+               elsif middleware.respond_to?(:name)
                  middleware.name
                elsif middleware.respond_to?(:klass)
                  middleware.klass.to_s
@@ -89,7 +95,7 @@ module Woods
                end
 
         args = if middleware.respond_to?(:args)
-                 middleware.args.map(&:to_s)
+                 middleware.args.map { |argument| MiddlewareArgument.render(argument) }
                else
                  []
                end

data/lib/woods/extractors/migration_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require_relative 'shared_utility_methods'
 require_relative 'shared_dependency_scanner'
 
@@ -122,7 +124,7 @@ module Woods
 
         unit
       rescue StandardError => e
-        Rails.logger.error("Failed to extract migration #{file_path}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract migration #{file_path}: #{e.message}")
         nil
       end
 

data/lib/woods/extractors/model_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require 'digest'
 require_relative '../ast/parser'
 require_relative 'shared_utility_methods'
@@ -36,6 +38,12 @@ module Woods
         (?:after|before)_(?:add|remove)_for_         # collection callbacks
       )/x
 
+      # Rails exposes one chain per event; each entry supplies its own kind.
+      # before_commit is itself an event, unlike before_save and after_commit.
+      CALLBACK_EVENTS = %i[
+        validation save create update destroy commit rollback before_commit initialize find touch
+      ].freeze
+
       # Warnings collected during extraction (skipped associations, failed models)
       attr_reader :warnings
 
@@ -97,7 +105,7 @@ module Woods
         unit
       rescue StandardError => e
         @warnings << "Failed to extract model #{model.name}: #{e.message}"
-        Rails.logger.error("Failed to extract model #{model.name}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract model #{model.name}: #{e.message}")
         nil
       end
 
@@ -384,6 +392,9 @@ module Woods
 
       # Find the file path for a concern
       def concern_path_for(mod)
+        reflected = resolve_source_location(mod, app_root: Rails.root.to_s, fallback: nil)
+        return reflected if app_source?(reflected, "#{Rails.root}/") && File.file?(reflected)
+
         possible_paths = [
           Rails.root.join("app/models/concerns/#{mod.name.underscore}.rb"),
           Rails.root.join("app/controllers/concerns/#{mod.name.underscore}.rb"),
@@ -407,6 +418,7 @@ module Woods
       #   composite source does not carry.
       # @return [Hash]
       def extract_metadata(model, source = nil, inlined_concerns: nil)
+        callbacks = extract_callbacks(model)
         {
           # Core identifiers
           table_name: model.table_name,
@@ -416,15 +428,11 @@ module Woods
           # Relationships and behaviors
           associations: extract_associations(model),
           validations: extract_validations(model),
-          callbacks: extract_callbacks(model),
+          callbacks: callbacks,
           scopes: extract_scopes(model, source),
           enums: extract_enums(model),
           inlined_concerns: inlined_concerns || build_model_source_with_concerns(model, source).last,
 
-          # API surface
-          class_methods: model.methods(false).sort,
-          instance_methods: filter_instance_methods(model.instance_methods(false)).sort,
-
           # Inheritance
           sti_column: model.inheritance_column,
           is_sti_base: sti_base?(model),
@@ -433,7 +441,7 @@ module Woods
 
           # Metrics for retrieval ranking
           loc: count_loc(model, source),
-          callback_count: callback_count(model),
+          callback_count: callbacks.length,
           association_count: model.reflect_on_all_associations.size,
           validation_count: model._validators.values.flatten.size,
 
@@ -449,6 +457,12 @@ module Woods
                      []
                    end,
 
+          # Schema loading can define methods (including Rails' optimized
+          # constructor). Reflect only after the schema reads above so a cold
+          # full extraction and a warmed incremental run see the same API.
+          class_methods: model.methods(false).sort,
+          instance_methods: filter_instance_methods(model.instance_methods(false)).sort,
+
           # Foreign keys as the connection reports them. The target table's
           # database is a graph-level concern, resolved by the
           # cross_database_edges report, not stored per model here (#280).
@@ -605,7 +619,7 @@ module Woods
             entry = {
               attribute: attribute,
               type: v.class.name.demodulize.underscore.sub(/_validator$/, ''),
-              options: v.options.except(:if, :unless, :on),
+              options: v.options.except(:if, :unless, :on).transform_values { |value| stable_filter(value) },
               conditions: format_validation_conditions(v)
             }
             entry[:implicit_belongs_to] = true if implicit_belongs_to_validator?(model, v)
@@ -616,23 +630,12 @@ module Woods
 
       # Extract all callbacks with their full chain
       def extract_callbacks(model)
-        callback_types = %i[
-          before_validation after_validation
-          before_save after_save around_save
-          before_create after_create around_create
-          before_update after_update around_update
-          before_destroy after_destroy around_destroy
-          after_commit after_rollback
-          after_initialize after_find
-          after_touch
-        ]
-
-        callback_types.flat_map do |type|
-          callbacks = model.send("_#{type}_callbacks")
+        CALLBACK_EVENTS.flat_map do |event|
+          callbacks = model.send("_#{event}_callbacks")
           callbacks.map do |cb|
             {
-              type: type,
-              filter: cb.filter.to_s,
+              type: event == :before_commit ? :before_commit : :"#{cb.kind}_#{event}",
+              filter: callback_filter_label(cb),
               kind: cb.kind, # :before, :after, :around
               conditions: format_callback_conditions(cb)
             }
@@ -1221,17 +1224,6 @@ module Woods
       # Helper methods
       # ──────────────────────────────────────────────────────────────────────
 
-      def callback_count(model)
-        %i[validation save create update destroy commit rollback].sum do |type|
-          # CallbackChain includes Enumerable but defines no #size on any
-          # supported Rails version — #size raises and the rescue would
-          # zero the count. #count is the only correct API here.
-          model.send("_#{type}_callbacks").count
-        rescue StandardError
-          0
-        end
-      end
-
       def count_loc(model, source = nil)
         if source
           source.lines.count { |l| l.strip.present? && !l.strip.start_with?('#') }

data/lib/woods/extractors/package_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require 'yaml'
 require 'pathname'
 
@@ -69,7 +71,7 @@ module Woods
         unit.dependencies = dependencies.map { |dep| { type: :package, target: dep, via: :package_dependency } }
         unit
       rescue StandardError => e
-        Rails.logger.error("Failed to extract package from #{file_path}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract package from #{file_path}: #{e.message}")
         nil
       end
 
@@ -103,13 +105,30 @@ module Woods
         @package_files ||= begin
           patterns = Array(@packwerk['package_paths'] || DEFAULT_PACKAGE_PATHS)
           excludes = Array(@packwerk['exclude'] || DEFAULT_EXCLUDE)
-          found = patterns.flat_map do |pattern|
-            Dir.glob(@rails_root.join(pattern.to_s, PACKAGE_FILE).to_s)
-          end
+          found = if patterns == DEFAULT_PACKAGE_PATHS && excludes == DEFAULT_EXCLUDE
+                    default_package_files
+                  else
+                    patterns.flat_map { |pattern| Dir.glob(@rails_root.join(pattern.to_s, PACKAGE_FILE).to_s) }
+                  end
           found.uniq.reject { |path| excluded?(relativize(path), excludes) }.sort
         end
       end
 
+      # Prune only the known default top-level exclusions before recursion.
+      # Recursive glob does not descend through symlink directories; keep that
+      # behavior when turning top-level entries into separate glob roots.
+      def default_package_files
+        found = Dir.glob(PACKAGE_FILE, base: @rails_root).map { |name| @rails_root.join(name).to_s }
+        Dir.glob('*', base: @rails_root).each do |name|
+          directory = @rails_root.join(name)
+          next unless directory.directory? && !directory.symlink?
+          next if excluded?("#{name}/#{PACKAGE_FILE}", DEFAULT_EXCLUDE)
+
+          found.concat(Dir.glob("**/#{PACKAGE_FILE}", base: directory).map { |path| directory.join(path).to_s })
+        end
+        found
+      end
+
       def excluded?(relative, excludes)
         excludes.any? { |glob| File.fnmatch?(glob.to_s, relative, File::FNM_PATHNAME | File::FNM_EXTGLOB) }
       end
@@ -122,6 +141,7 @@ module Woods
         data = YAML.safe_load_file(path, permitted_classes: [Symbol], aliases: true)
         data.is_a?(Hash) ? data : {}
       rescue StandardError => e
+        SourceInputs::ConsumerErrors.record(self)
         Rails.logger.warn("[Woods] Could not read #{PACKWERK_CONFIG}: #{e.message}")
         {}
       end

data/lib/woods/extractors/phlex_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require_relative 'component_discovery'
 require_relative 'shared_utility_methods'
 require_relative 'shared_dependency_scanner'
@@ -87,7 +89,7 @@ module Woods
 
         unit
       rescue StandardError => e
-        Rails.logger.error("Failed to extract component #{component.name}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract component #{component.name}: #{e.message}")
         nil
       end
 

data/lib/woods/extractors/policy_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require_relative 'shared_utility_methods'
 require_relative 'shared_dependency_scanner'
 
@@ -69,7 +71,7 @@ module Woods
 
         unit
       rescue StandardError => e
-        Rails.logger.error("Failed to extract policy #{file_path}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract policy #{file_path}: #{e.message}")
         nil
       end
 

data/lib/woods/extractors/poro_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require_relative 'shared_utility_methods'
 require_relative 'shared_dependency_scanner'
 require_relative 'source_nesting'
@@ -80,14 +82,16 @@ module Woods
           file_path: file_path
         )
 
+        parent_class = extract_parent_class(source, class_name)
+
         unit.namespace    = extract_namespace(class_name)
-        unit.source_code  = annotate_source(source, class_name)
-        unit.metadata     = extract_metadata(source, class_name)
+        unit.source_code  = annotate_source(source, class_name, parent_class)
+        unit.metadata     = extract_metadata(source, parent_class)
         unit.dependencies = extract_dependencies(source)
 
         unit
       rescue StandardError => e
-        Rails.logger.error("Failed to extract PORO #{file_path}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract PORO #{file_path}: #{e.message}")
         nil
       end
 
@@ -175,10 +179,10 @@ module Woods
       #
       # @param source [String] Ruby source code
       # @param class_name [String] The class name
+      # @param parent_class [String, nil] Selected explicit parent
       # @return [String] Annotated source
-      def annotate_source(source, class_name)
-        parent = extract_parent_class(source)
-        parent_label = parent || 'none'
+      def annotate_source(source, class_name, parent_class)
+        parent_label = parent_class || 'none'
 
         annotation = <<~ANNOTATION
           # ╔═══════════════════════════════════════════════════════════════════════╗
@@ -198,14 +202,14 @@ module Woods
       # Build the metadata hash for a PORO unit.
       #
       # @param source [String] Ruby source code
-      # @param class_name [String] The class name
+      # @param parent_class [String, nil] Selected explicit parent
       # @return [Hash] PORO metadata
-      def extract_metadata(source, _class_name)
+      def extract_metadata(source, parent_class)
         {
           public_methods: extract_public_methods(source),
           class_methods: extract_class_methods(source),
           initialize_params: extract_initialize_params(source),
-          parent_class: extract_parent_class(source),
+          parent_class: parent_class,
           loc: count_loc(source),
           method_count: source.scan(/def\s+(?:self\.)?\w+/).size
         }

data/lib/woods/extractors/pundit_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require_relative 'shared_utility_methods'
 require_relative 'shared_dependency_scanner'
 
@@ -67,7 +69,7 @@ module Woods
 
         unit
       rescue StandardError => e
-        Rails.logger.error("Failed to extract Pundit policy #{file_path}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract Pundit policy #{file_path}: #{e.message}")
         nil
       end
 

data/lib/woods/extractors/rails_source_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require_relative '../extracted_unit'
 
 module Woods
@@ -268,7 +270,7 @@ module Woods
 
         unit
       rescue StandardError => e
-        Rails.logger.error("Failed to extract Rails source #{file_path}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract Rails source #{file_path}: #{e.message}")
         nil
       end
 
@@ -302,7 +304,7 @@ module Woods
 
         unit
       rescue StandardError => e
-        Rails.logger.error("Failed to extract gem source #{file_path}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract gem source #{file_path}: #{e.message}")
         nil
       end
 

data/lib/woods/extractors/rake_task_extractor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../source_inputs/consumer_errors'
+
 require_relative 'line_neutralizer'
 require_relative 'shared_utility_methods'
 require_relative 'shared_dependency_scanner'
@@ -73,7 +75,7 @@ module Woods
           build_unit(task_data, file_path, data[:source], sibling_definitions(task_data[:full_name], file_path))
         end
       rescue StandardError => e
-        Rails.logger.error("Failed to extract rake tasks from #{file_path}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to extract rake tasks from #{file_path}: #{e.message}")
         []
       end
 
@@ -100,7 +102,7 @@ module Woods
           { source: source, tasks: parse_tasks(source) }
         end
       rescue StandardError => e
-        Rails.logger.error("Failed to scan rake tasks in #{file}: #{e.message}")
+        SourceInputs::ConsumerErrors.log(self, "Failed to scan rake tasks in #{file}: #{e.message}")
         nil
       end