recourse 7.11.2 → 7.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 96bb337b5a0517cc5f7715b898c0cffd01c33fc6b8e7a6750f939b7361879fd4
-  data.tar.gz: e77f1f0f2f4a42955669e988302bff0df520aa98a8cb7ce0d171cda7e7bd85ea
+  metadata.gz: 9e4b4229005fd88e1617e55a5f4760da0ebb2d7b897eabd730adeac07c7eb61d
+  data.tar.gz: b84e0db179300948a05e77e9bbdbdcbaad8ef1ee794632f42abeec752b3395ad
 SHA512:
-  metadata.gz: 9d203812049cdcd045323970939f23c4aeb825e5cd904b949e511b8700bd24781ae5cb36ef83c41b7e90547665a5c255c00c4158425a3c7a1c04b86d9182a223
-  data.tar.gz: 17c113285f5c356e86fad57838069d9c42b8c272d19950649be5f992a674de2bf1b1e6a60e75be220ec77ea48e15462459127cb94b7ba6ffffe5a2c9372576da
+  metadata.gz: 2aa90f2ea92f2370ac39c64f0351518785bf3cb3233e7fab52e61cae83d8c90df80362a8c8267d451e8869b5e6da3cf22b9bb38b59c0510d7cc7958355da07e2
+  data.tar.gz: b0c47758910fe5c03c28084afa668adec4d6a07c399894f05ea0a7fbf57983c2988a84f72fed6a2ea84cb8a49c0730f9a894240e3ddfeb85f8a1f256b9081e2e

data/CHANGELOG.md

@@ -7,6 +7,17 @@ For more information about changelogs, check [Keep a Changelog](http://keepachan
 
 ## [Unreleased]
 
+## 7.12.0 - 2026-10-02
+
+* [FEATURE] A route drawn in a module lists the model kept in that module
+
+  `scope module: :admin { recourses :users }` lists an `Admin::User`, and so does a page
+  nested under a post, `admin/posts/users`. Only an Active Record model in the module counts;
+  where there is none, the route lists the top-level class of its last word, as before, so
+  `admin/comments` still lists a `Comment`. A host keeping models in a module will want
+  `def self.use_relative_model_naming? = true` on it, so their routes and params say `user`
+  rather than `admin_user`.
+
 ## 7.11.2 - 2026-09-26
 
 * [FIX] The count in a tab is delimited

data/app/controllers/concerns/recourse/resource_resolution.rb

@@ -78,7 +78,7 @@ module Recourse
     # to match it. Private, the way `recourse_relation` beside it is: naming a model
     # adds no action.
     def recourse_model
-      Recourse.model controller_name
+      Recourse.model controller_path
     end
 
     # What the page calls one of its rows: the model's own word, or the route's for a

data/lib/recourse/helpers/resources.rb

@@ -15,7 +15,7 @@ module Recourse
       def resources_name
         return controller.controller_name.humanize if blob_resource?
 
-        Recourse.known_title controller.controller_name
+        Recourse.known_title controller.controller_path
       end
 
       # Singular, lowercase name of the resource, e.g. 'contact' — and 'photo' on a page
@@ -72,7 +72,7 @@ module Recourse
       # Resolved by the controller, which is the one that knows whether the name is a
       # model of this app's, something a record has attached, or a model a host named.
       def resource_model
-        controller_assign('recourse_model') || Recourse.model(controller.controller_name)
+        controller_assign('recourse_model') || Recourse.model(controller.controller_path)
       end
     end
   end

data/lib/recourse/icons.rb

@@ -8,7 +8,7 @@ module Recourse
   # `Message` may be a host's own class for gathering rows rather than a table of them,
   # and asking it would raise rather than draw nothing.
   def self.known_icon(name)
-    model = name.to_s.split('/').last.classify.safe_constantize
+    model = model? name
 
     known_model_icon model if model.respond_to? :recourse_icon
   end

data/lib/recourse/routes/nested.rb

@@ -25,7 +25,7 @@ module Recourse
       # `/placeholders/:placeholder_id/bookmark`, where nothing linked to it and
       # anything reaching it raised. Asked of the constant rather than the class:
       # loading a model while the routes draw is what Rails 8.2 warns about.
-      def addressable_rows? = Object.const_defined? Recourse.model_name(parent_resource.name)
+      def addressable_rows? = Recourse.model_defined? declared_path(parent_resource.name)
 
       # The place a row of a positioned table holds, at `/teams/5/position`. Recorded
       # nowhere, for the reason the bookmark gives: a tab and a bare-action button both

data/lib/recourse/titles.rb

@@ -33,7 +33,7 @@ module Recourse
     # than a table of them, and asking it would raise rather than read.
     def known_title(name)
       segment = name.to_s.split('/').last
-      model = segment.classify.safe_constantize
+      model = model? name
 
       model.respond_to?(:model_name) ? model_title(model) : segment.humanize
     end

data/lib/recourse/version.rb

@@ -1,4 +1,4 @@
 module Recourse
   # Version of the gem, read by the gemspec and by hosts checking compatibility.
-  VERSION = '7.11.2'
+  VERSION = '7.12.0'
 end

data/lib/recourse.rb

@@ -71,11 +71,58 @@ module Recourse
   # action is a verb — `recourse :sweep, only: :create` — and the gem labels its button
   # from the path alone, so whether a name has a model behind it has to be a question
   # and not an accusation.
-  def self.model?(name) = model_name(name).safe_constantize
+  def self.model?(name) = namespaced_model(name) || model_name(name).safe_constantize
 
   # A namespaced resource is `admin/sources`, and the model it lists is a Source.
   def self.model_name(name) = name.to_s.split('/').last.classify
 
+  # Whether a model answers the name, asked of the constants rather than the classes:
+  # the routes ask while they draw, and loading a model then is what Rails 8.2 warns
+  # about.
+  def self.model_defined?(name)
+    namespaced_names(name).any? { |namespaced| constant namespaced, load: false } ||
+      Object.const_defined?(model_name(name))
+  end
+
+  # A model the app keeps in the route's own module, where there is one: `admin/users`
+  # lists an Admin::User, and so does `admin/posts/users`, nested under a post.
+  # Only an Active Record model counts, so a concern or a host's own class that shares
+  # the name never takes a page over; and where there is none, `admin/comments` lists a
+  # Comment as it always has.
+  private_class_method def self.namespaced_model(name)
+    namespaced_names(name).each do |namespaced|
+      model = constant namespaced
+      return model if model.is_a?(Class) && model < ActiveRecord::Base
+    end
+
+    nil
+  end
+
+  # The names a model in the route's modules goes by, innermost module first. The
+  # segment a nested route adds for its parent is a module no model lives in, so the
+  # module around it is asked next.
+  private_class_method def self.namespaced_names(name)
+    *modules, last = name.to_s.split '/'
+    modules.size.downto(1).map { |depth| [*modules[0, depth], last].join('/').classify }
+  end
+
+  # The constant a name gives, looked up one module at a time and never through Object:
+  # `safe_constantize` answers `Admin::Comment` with the top-level Comment, since a module
+  # looks its constants up there too. `load: false` stops short of the last one,
+  # answering only whether it is there.
+  private_class_method def self.constant(name, load: true)
+    *modules, last = name.split '::'
+    scope = modules.inject Object do |outer, part|
+      break unless outer.const_defined? part, false
+
+      outer.const_get part, false
+    end
+    return unless scope.is_a? Module
+    return unless scope.const_defined? last, false
+
+    load ? scope.const_get(last, false) : true
+  end
+
   # Raised for every failure the gem reports, so hosts can rescue one type.
   class Error < StandardError; end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: recourse
 version: !ruby/object:Gem::Version
-  version: 7.11.2
+  version: 7.12.0
 platform: ruby
 authors:
 - claudiob