mobility_active_storage 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f4973582dfa47dc2c6bdd8f1003bf6b3e4347be60ab61ab653e588629cee3588
-  data.tar.gz: fbc607bf032ac1c19dc3f73ce41ceabba5ce8372e2cb71977158a8f17bc45e55
+  metadata.gz: 7ed33baa135ecd69527c111652956b1420c0bce45052b3725cc5d4e55715257e
+  data.tar.gz: ff07c4fd501a56b1b57008d7f294517c5159d27ef4c7844984012251b9b49343
 SHA512:
-  metadata.gz: 7cc3a587ed7f117a685b9d7236597a3b50eef206537e83554e27b32ac9d803560d08f93f99e2df554afee1af1cf558d6025942da2b9bd8c4514d36d63e6cde49
-  data.tar.gz: fbb2f83b8e98ea1f26019a2302fab216dd7fd9b6b677a09e5fb1b785766ed149e6df0c5479e27cac85f92748474f49637791e47953c51fae215ba1bc61e69abe
+  metadata.gz: 776fef9bd8b7dffb06b3e1b4c9955bb3b5acebec0e601d265df4793dfcb98e5e7f5a12d9aeec8992cfed204b832201cbc38a7bdd5f8894edb8bcd0db418508f7
+  data.tar.gz: b62a6b3ae3dad73be4f415f1753fd3af573efb2c0e72b87fd57b09820a8a651f80e041587b95a91c7be3c0756dd3dfe9d605f282a106df4a10702d7cd2813254

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [0.1.2] - 2026-08-11
+
+- `#{attribute}_locales` now answers in one query instead of one per configured locale. It probed
+  each locale's attachment proxy in turn, so a model declared for 73 locales issued 73 statements
+  every time it was asked -- and any caller that reads it per row (a serializer, a validation) paid
+  that per row. Pending attachment changes still take precedence over the stored rows, so a staged
+  attach counts before it is saved and a staged purge stops counting straight away.
+
 ## [0.1.1] - 2026-08-10
 
 - No user-facing changes. Released to verify the trusted-publishing workflow; the packaged files

data/README.md

@@ -94,6 +94,10 @@ For `has_one_translated_attached :document`:
 
 `has_many_translated_attached :photos` mirrors these with `ActiveStorage::Attached::Many`.
 
+`document_locales` costs one query however many locales the attribute declares, and counts pending
+changes the way `attached?` does: a staged attach is included before it is saved, a staged purge
+excluded straight away. It does not memoize, so hold the result if you need it more than once.
+
 ### Fallbacks
 
 Off by default. Enable per attribute:

data/lib/mobility_active_storage/backend_methods.rb

@@ -50,6 +50,13 @@ module MobilityActiveStorage
         normalized_locales.include?(Mobility.normalize_locale(locale))
       end
 
+      # Every configured locale's attachment name. Memoized per attribute: the locale set is
+      # fixed when the class body runs.
+      def attachment_names(attribute)
+        @attachment_names ||= {}
+        @attachment_names[attribute] ||= normalized_locales.map { |locale| "#{attribute}_#{locale}" }
+      end
+
       # Mobility's query plugin asks the backend for an Arel node so it can build a predicate.
       # An attachment is a row in active_storage_attachments, not a comparable column value, so
       # there is nothing meaningful to compare. Fail with an explanation rather than the
@@ -136,8 +143,24 @@ module MobilityActiveStorage
     end
 
     # Yields each configured locale that actually has an attachment.
+    #
+    # One query for the whole locale set, not one per locale: every locale is a row in
+    # active_storage_attachments under a suffixed name, so which locales are present is a single
+    # pluck on that table. Probing each proxy instead costs a query per configured locale, and a
+    # model declared for a large locale set pays that on every read -- 73 statements to answer a
+    # question one statement answers.
+    #
+    # Pending changes win over the stored rows, as Active Storage's own +attached?+ does: an
+    # attach counts before it is saved, a purge stops counting straight away.
     def each_locale
-      options[:locales].each { |locale| yield locale if attached(locale).attached? }
+      persisted = persisted_attachment_names
+
+      options[:locales].each do |locale|
+        name = attachment_name(locale)
+        # attached? reads the staged change without touching the database.
+        present = model.attachment_changes.key?(name) ? attached(locale).attached? : persisted.include?(name)
+        yield locale if present
+      end
     end
 
     # The Active Storage proxy for +locale+, without any fallback handling.
@@ -149,6 +172,20 @@ module MobilityActiveStorage
 
     private
 
+    # Which of this attribute's attachment names have rows. distinct because a collection
+    # attachment repeats its name once per file, and polymorphic_name so an STI subclass looks
+    # itself up under the type Active Storage actually stored.
+    def persisted_attachment_names
+      return Set.new if model.new_record?
+
+      ActiveStorage::Attachment
+        .where(record_type: model.class.polymorphic_name, record_id: model.id,
+               name: self.class.attachment_names(attribute))
+        .distinct
+        .pluck(:name)
+        .to_set
+    end
+
     def attachment_name(locale)
       self.class.attachment_name_for(attribute, locale)
     end

data/lib/mobility_active_storage/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MobilityActiveStorage
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mobility_active_storage
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Glen Barnes