rigortype 0.3.6 → 0.3.8

data/plugins/rigor-activejob/lib/rigor/plugin/activejob/job_index.rb

@@ -29,19 +29,27 @@ module Rigor
 
         attr_reader :entries
 
+        # `entries` is expected UNIQUE by `class_name`: the by-name Hash is keyed by it, so a second row for
+        # a job would replace the first outright. {JobDiscoverer#merge_redeclarations} is the single home of
+        # that guarantee — a reopened class arrives as one merged row, never as two.
         def initialize(entries)
           @entries = entries.freeze
           @by_name = entries.to_h { |entry| [entry.class_name, entry] }.freeze
           freeze
         end
 
+        # Entries are keyed by the de-rooted constant path (`"WelcomeJob"`, `"Admin::WelcomeJob"` — never
+        # `"::WelcomeJob"`; see {JobDiscoverer}), while a QUERY may legitimately arrive rooted:
+        # `::WelcomeJob.perform_later(1)` renders its receiver as `"::WelcomeJob"`. The root marker is
+        # dropped here, once, so no caller needs a `find(name) || find("::#{name}")` retry (#621).
+        #
         # @return [Entry, nil]
         def find(class_name)
-          @by_name[class_name.to_s]
+          @by_name[strip_leading_namespace(class_name.to_s)]
         end
 
         def known?(class_name)
-          @by_name.key?(class_name.to_s)
+          @by_name.key?(strip_leading_namespace(class_name.to_s))
         end
 
         def empty?
@@ -55,6 +63,11 @@ module Rigor
         def names
           @by_name.keys
         end
+
+        private
+
+        # `::WelcomeJob` → `WelcomeJob`. The query-side half of the key contract — see {#find}.
+        def strip_leading_namespace(name) = name.delete_prefix("::")
       end
     end
   end

data/plugins/rigor-activejob/lib/rigor/plugin/activejob/recurring_scan.rb

@@ -69,7 +69,9 @@ module Rigor
 
         def load_document(path)
           absolute = File.expand_path(path.to_s)
-          return nil unless File.file?(absolute)
+          # ADR-45 WD1b (#613) — probing through the boundary records the missing `config/recurring.yml`
+          # as a dependency, so adding one invalidates the warm run instead of being served past.
+          return nil unless @io_boundary.file?(absolute)
 
           contents = read_safely(absolute)
           contents && parse_safely(contents)

data/plugins/rigor-activejob/lib/rigor/plugin/activejob.rb

@@ -39,7 +39,11 @@ module Rigor
         id: "activejob",
         # Bumped — publishes `:reachability_roots` for `rigor unused` (ADR-102 WD3): the jobs Solid Queue's
         # recurring schedule runs by name, which no `perform_later` call site writes down.
-        version: "0.2.0",
+        #
+        # Bumped 2026-09-02 (#621) — job keys are de-rooted at the producer and a reopened class's
+        # `#perform` envelope is merged rather than clobbered, so a cached 0.2.0 index can carry the
+        # any-arity row the merge replaces (and miss a schedule's reachability root under a rooted name).
+        version: "0.3.0",
         description: "Validates ActiveJob `Job.perform_later` argument arity.",
         config_schema: {
           "job_search_paths" => { kind: :array, default: ["app/jobs"] },

data/plugins/rigor-activerecord/lib/rigor/plugin/activerecord/analyzer.rb

@@ -59,8 +59,8 @@ module Rigor
           model_name = constant_receiver_name(node.receiver)
           return if model_name.nil?
 
-          entry = @model_index.find(model_name) ||
-                  @model_index.find("::#{model_name}")
+          # A rooted receiver (`::User.where(...)`) renders as `"::User"`; `ModelIndex#find` normalises it.
+          entry = @model_index.find(model_name)
           return if entry.nil?
 
           case node.name