active_storage-async_variants-ui 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 279f3c8402b7c582403983ff679ecf6fead34d5a90e72f8952f27647ce9f1a5b
-  data.tar.gz: d931c539a3661e61c0f9fa20a144d399f8f10910169fb5aed75206cdda09c99a
+  metadata.gz: e005a7ad73b45c7f8d6e7ede189bb0aa1e17a295d7ca68096ba745a3e8dfa264
+  data.tar.gz: 4da955bf3e81bdd833602160fc8508f79c3d05c159ec6229ffb241254df1894e
 SHA512:
-  metadata.gz: b58f2d6f42a0122f79ea9d1c831cd53476353be2f2a98666bd0983919343a92dd20b4045c6803246ddc458aaa9e912412eee3baa5362d7fb446a418abdc206e8
-  data.tar.gz: 5e6a2837652e6e17388f3abf00e22a087ebe1cb6ca5e7d383e6a8a2a1d43f7f800ac78a05541cb91ecf95692bcbfa6a843b730842719588abc7dff6d92c282d7
+  metadata.gz: 98a2bc9515eb63b696f92936673124ecd33234f924c7b8fc00741da28ab5e1d5aad21923565645e1be0c042f42c57e5b7930af2845b46deddd215f2cb392a647
+  data.tar.gz: 8bc97a17fd4641e9c4824c768f2fb72e23c247014e906d6279f15e750a60337a16d20d1d6ece692588494f213bd8520aaf9b76e5f7a2d7d7e5837702b5be8691

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [0.2.0]
+
+- Video previews on the stock preview graph: a video's image variant keeps its record on the extracted-frame blob (`preview_image`), so the state endpoint now rebuilds a `Preview` (via a `repr=preview` param the helper round-trips) and reads state, progress, and the processed URL from there. Format-bearing variations are canonicalized to the declared named variant so a URL-rebuilt digest matches the stored record.
+- Retry goes through the representation's `retry!`, so a video preview's failed record on the frame blob is actually replaced; previously the endpoint searched the source blob with the wrong digest and retry silently did nothing for videos.
+- Fix: the failed state's retry form now carries `repr`, so retrying a video preview replaces its frame record instead of recording a stray variant on the video blob.
+- Requires `active_storage-async_variants` >= 0.10. Hand `image_tag`/`video_tag` a video's `representation(...)` (a `Preview`), not its `variant(...)`, while it is unprocessed.
+
 ## [0.1.0]
 
 - Initial release. Extracted from `active_storage-async_variants` (≤ 0.8.0), which now ships the UI-free plumbing. This gem layers on the opinionated UI:

data/app/controllers/active_storage/async_variants/states_controller.rb

@@ -26,17 +26,14 @@ module ActiveStorage
       end
 
       def retry
-        @variant.blob.variant_records.where(
-          variation_digest: @variant.variation.digest,
-          state: "failed",
-        ).destroy_all
-        @variant.enqueue!
+        @variant.retry!
 
         redirect_to Rails.application.routes.url_helpers.async_variant_state_path(
           signed_blob_id: params[:signed_blob_id],
           variation_key: params[:variation_key],
           kind: params[:kind],
           direct: params[:direct],
+          repr: params[:repr],
           opts: async_variant_html_options.presence,
         ), status: :see_other
       end
@@ -47,12 +44,29 @@ module ActiveStorage
 
       def set_variant
         blob = ActiveStorage::Blob.find_signed!(params[:signed_blob_id])
-        variation = ActiveStorage::Variation.decode(params[:variation_key])
-        @variant = blob.variant(variation)
+        variation = canonical_variation(blob, ActiveStorage::Variation.decode(params[:variation_key]))
+        # A video's image preview keeps its variant record on the extracted-frame
+        # blob, so rebuild it as a Preview to read state from the right place;
+        # everything else is a plain Variant. The helper round-trips which one
+        # via the `repr` param.
+        @variant = params[:repr] == "preview" ? blob.preview(variation) : blob.variant(variation)
       rescue ActiveSupport::MessageVerifier::InvalidSignature
         head :not_found
       end
 
+      # The URL key round-trips a declared variant's transformations to strings
+      # (format: :webp -> "webp"), which Marshal-digests differently from the
+      # symbol form the variant record was created with -- so the record lookup
+      # would miss. Recover the declared transformations (symbol-preserving) when
+      # this variation matches a named variant; NamedVariantScan is itself
+      # format-string/symbol tolerant.
+      def canonical_variation(blob, variation)
+        found = ActiveStorage::AsyncVariants::NamedVariantScan.find(blob, variation)
+        return variation unless found
+        attachment, name, _ = found
+        ActiveStorage::Variation.wrap(attachment.send(:named_variants)[name].transformations)
+      end
+
       def async_variant_kind
         params[:kind].to_s == "video" ? :video : :image
       end

data/app/views/active_storage/async_variants/states/_failed.html.erb

@@ -16,7 +16,7 @@
     </style>
     <%# In the light DOM so its submit reaches Turbo, and so preventDefault on the
         buttons doesn't also cancel the submit. %>
-    <form action="<%= Rails.application.routes.url_helpers.async_variant_state_retry_path(signed_blob_id:, variation_key:, kind:, direct:, opts: html_options) %>"
+    <form action="<%= Rails.application.routes.url_helpers.async_variant_state_retry_path(signed_blob_id:, variation_key:, kind:, direct:, repr:, opts: html_options) %>"
           method="post"
           id="<%= form_id %>"
           data-turbo-frame="<%= async_variant_frame_id(variant) %>"

data/app/views/active_storage/async_variants/states/show.html.erb

@@ -6,5 +6,6 @@
     direct: async_variant_direct?,
     signed_blob_id: params[:signed_blob_id],
     variation_key: params[:variation_key],
+    repr: params[:repr],
   } %>
 <% end %>

data/lib/active_storage/async_variants/helper.rb

@@ -56,6 +56,11 @@ module ActiveStorage
           variation_key: variant.variation.key,
           kind:,
           direct:,
+          # Preserve which representation the caller passed: a video's image
+          # preview keeps its variant record on the extracted-frame blob, so the
+          # state endpoint must rebuild it as a Preview (blob.preview), not a
+          # Variant (blob.variant) -- otherwise it reads state off the wrong blob.
+          repr: variant.is_a?(ActiveStorage::Preview) ? "preview" : "variant",
           opts: html_options.slice(*PASS_THROUGH_HTML_OPTIONS),
         )
       end

data/lib/active_storage/async_variants/ui/version.rb

@@ -3,7 +3,7 @@
 module ActiveStorage
   module AsyncVariants
     module UI
-      VERSION = "0.1.0"
+      VERSION = "0.2.0"
     end
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: active_storage-async_variants-ui
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Micah Geisel
 bindir: exe
 cert_chain: []
-date: 2026-06-10 00:00:00.000000000 Z
+date: 2026-11-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_storage-async_variants
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.10'
 - !ruby/object:Gem::Dependency
   name: turbo-rails
   requirement: !ruby/object:Gem::Requirement