Merge remote-tracking branch 'origin/main' into sethherr/passwordless-users

# Conflicts:
#	app/components/admin/bikes_table/component.rb
#	app/components/admin/users_table/component.rb
#	app/components/registrations/show/wrapper/component.rb
#	app/components/ui/alerts/base/component.html.erb
#	app/components/ui/alerts/flash_message/component.rb

Author: sethherr

.claude/skills/pr/SKILL.md

@@ -32,6 +32,14 @@ Scope specs the same way — the ones covering what the branch changed, never a
 
 Then review the changed files against the repo's `CLAUDE.md` (root and any nested ones in touched directories) and fix anything that doesn't conform — code-style guidelines (functional style, no argument mutation, omitted hash values like `{x:}`, private methods, unabbreviated names, pithy comments), testing conventions, and frontend rules. Only touch lines this branch already changed; don't reformat unrelated code.
 
+Then check the branch's translations for a hardcoded "bike" where the string means the registration's cycle type — a registration is as often an e-scooter, a stroller or a wheelchair:
+
+```bash
+git diff "origin/$BASE"...HEAD -- '*.en.yml' 'config/locales/en.yml' | grep -in '^+.*bike'
+```
+
+Read each hit. Key names (`about_this_bike:`), the product name ("Bike Index"), and copy that really is bike-only are fine; a value saying "bike" about the registration is not. Fix it by interpolating `%{bike_type}` in the value and passing `bike_type: bike.type` at the call site — `Registrations::Show::CurrentAlerts::ClaimImpound` and `Registrations::Show::WrapperConsumer` are the pattern, and `spec/components/registrations/show/current_alerts/claim_impound/component_spec.rb` shows how to cover it. After hand-editing a `component.en.yml`, run `bundle exec rails prepare_translations` — `bin/lint` doesn't normalize YAML.
+
 Commit these edits before continuing — step 1's `git merge` needs a clean working tree, and committing here is what lets the step 0 edits ride along the merge as ordinary branch commits (which step 3 then pushes).
 
 ### 0.5. Determine the base branch

app/components/admin/bikes_table/component.rb

@@ -8,7 +8,7 @@ module BikesTable
     # checkboxes, and skip_user to drop the owner column.
     class Component < ApplicationComponent
       # Digest of the markup inside the row cache — the cached_markup_digest spec keeps it current
-      MARKUP_DIGEST = "bf42b49bc8da"
+      MARKUP_DIGEST = "1556da37892e"
 
       def initialize(bikes:, no_show_header: false, show_serial: false, render_sortable: false,
         skip_user: false, render_multi_check: false)

app/components/admin/users_table/component.rb

@@ -6,7 +6,7 @@ module UsersTable
     # render_deleted to show the deleted_at column.
     class Component < ApplicationComponent
       # Digest of the markup inside the row cache — the cached_markup_digest spec keeps it current
-      MARKUP_DIGEST = "ad8db631b5f9"
+      MARKUP_DIGEST = "d9337123b3b7"
 
       def initialize(users:, render_sortable: false, render_deleted: false)
         @users = users

app/components/application_component_preview.rb

@@ -24,6 +24,16 @@ def lookbook_organization
 
   private
 
+  def production_notice(rendered)
+    render(UI::Alerts::Base::Component.new(kind: :error,
+      text: "This preview renders a real #{rendered}, so it's disabled in production."))
+  end
+
+  def missing_notice(needed)
+    render(UI::Alerts::Base::Component.new(kind: :warning,
+      text: "Nothing to preview — this environment has no #{needed}."))
+  end
+
   def template
     ActionView::Base.new(
       ActionView::LookupContext.new(ActionController::Base.view_paths),

app/components/org/search_results/bikes_table/component.rb

@@ -9,7 +9,7 @@ module BikesTable
       # registrations on the show page). Pass render_sortable to enable sort links.
       class Component < ApplicationComponent
         # Digest of the markup inside the row cache — the cached_markup_digest spec keeps it current
-        MARKUP_DIGEST = "1d6fe7e5f29d"
+        MARKUP_DIGEST = "bc3a3249ac5f"
 
         delegate :additional_registration_fields, :column_renames, to: :settings_component
 

app/components/registrations/show/claim_impound/component.en.yml

@@ -0,0 +1,29 @@
+---
+en:
+  components:
+    registrations:
+      show:
+        current_alerts:
+          claim_impound:
+            add_a_stolen_bike: add a stolen %{bike_type}
+            choose_stolen_bike: Choose stolen %{bike_type}
+            claim_found_bike: Claim found %{bike_type}
+            claim_impounded_bike: Claim impounded %{bike_type}
+            claim_submitted: This claim was submitted
+            claim_was_status: Your claim was %{status} — you should have received
+              an email with next steps.
+            claim_with_this_bike_type: You have a %{status} claim with this %{bike_type}.
+            does_this_look_like_your_bike: Does this look like your %{bike_type}?
+            need_stolen_bike_html: You need a stolen %{bike_type} registered to file
+              a claim — %{add_link}.
+            no_editing_after_submitting: After submitting, you won't be able to edit
+              the claim.
+            open_claim: Open claim
+            prove_its_yours: Verify your ownership — add details that prove this %{bike_type}
+              is yours
+            save_message: Save message
+            select_your_stolen_bike: Select the stolen %{bike_type} you own that matches
+              this impound
+            submit_claim: Submit claim
+            view_claimed_bike_type: view the claimed %{kind} %{bike_type}
+            your_claim: Your claim

app/components/registrations/show/claim_impound/component.html.erb

@@ -0,0 +1,79 @@
+<%= render(UI::Alerts::Base::Component.new(kind: alert_kind, header: heading, margin_classes: "")) do %>
+  <% if impound_claim.present? %>
+    <% if impound_claim.unsubmitted? %>
+      <%= form_with(model: impound_claim, url: impound_claim_path(impound_claim), builder: BikeIndexFormBuilder, data: {controller: "csrf-refresh"}) do |form| %>
+        <%# Required in the browser only - nothing server-side turns an empty claim away %>
+        <%= render(UI::Forms::Group::Component.new(form_builder: form, attribute: :message, kind: :text_area,
+          label_text: translation(".prove_its_yours", bike_type: impound_claim.bike_type), required: true,
+          html_options: {rows: 4})) %>
+
+        <%= render(UI::Button::Component.new(text: translation(".save_message"), color: :secondary, kind: :submit)) %>
+
+        <%# The same form, so submitting sends the message rather than dropping an unsaved one %>
+        <div class="tw:mt-4">
+          <%= render(UI::Button::Component.new(text: translation(".submit_claim"), color: :purple, kind: :submit,
+            name: form.field_name(:status), value: "submitting")) %>
+        </div>
+
+        <p class="tw:mt-1 tw:text-sm tw:opacity-65">
+          <%= translation(".no_editing_after_submitting") %>
+        </p>
+      <% end %>
+    <% else %>
+      <p>
+        <%= translation(".claim_submitted") %>
+        <%= render(UI::Time::Component.new(time: impound_claim.submitted_at)) %>
+      </p>
+      <% if impound_claim.successful? %>
+        <p class="tw:mt-2">
+          <%= translation(".claim_was_status", status: impound_claim.status_humanized) %>
+        </p>
+      <% end %>
+    <% end %>
+  <% elsif submitting_impound_claim.present? %>
+    <p>
+      <%= translation(".claim_with_this_bike_type", status: submitting_impound_claim.status_humanized, bike_type: @bike.type) %>
+    </p>
+    <% if submitting_impound_claim.bike_claimed.present? %>
+      <p class="tw:mt-2">
+        <%= link_to translation(".view_claimed_bike_type", kind: submitting_impound_claim.kind, bike_type: submitting_impound_claim.bike_claimed.type),
+            registration_path(submitting_impound_claim.bike_claimed), class: "twlink" %>
+      </p>
+    <% end %>
+  <% elsif sign_in_redirect %>
+    <%= render(UI::ButtonLink::Component.new(href: sign_in_redirect, text: claim_button_text, color: :purple)) %>
+  <% elsif stolen_record_options.any? %>
+    <div
+      data-controller="ui--collapse"
+      data-ui--collapse-param-value="contact_owner"
+    >
+      <%= render(UI::Button::Component.new(
+          text: claim_button_text,
+          color: :purple,
+          aria: {expanded: false},
+          data: {"ui--collapse-target": "trigger", action: "ui--collapse#toggle"}
+        )) %>
+
+      <div data-ui--collapse-target="content" class="tw:hidden">
+        <%= form_with(model: new_impound_claim, url: impound_claims_path, builder: BikeIndexFormBuilder, data: {controller: "csrf-refresh"}) do |form| %>
+          <%= form.hidden_field :impound_record_id, value: impound_record.id %>
+
+          <%= render(UI::Forms::Group::Component.new(form_builder: form, attribute: :stolen_record_id,
+            label_text: translation(".select_your_stolen_bike", bike_type: @bike.type), required: true)) do %>
+            <%= render(UI::Forms::Select::Component.new(form_builder: form, attribute: :stolen_record_id,
+              option_tags: stolen_record_options,
+              options: {prompt: translation(".choose_stolen_bike", bike_type: @bike.type)},
+              required: true)) %>
+          <% end %>
+
+          <%= render(UI::Button::Component.new(text: translation(".open_claim"), color: :purple, kind: :submit)) %>
+        <% end %>
+      </div>
+    </div>
+  <% else %>
+    <p class="tw:text-sm tw:opacity-65">
+      <%# underlined because it sits in a text block, where color alone doesn't distinguish it %>
+      <%= translation(".need_stolen_bike_html", bike_type: @bike.type, add_link: link_to(translation(".add_a_stolen_bike", bike_type: @bike.type), new_register_path(status: "status_stolen"), class: "twlink tw:underline", target: "_blank", rel: "noopener")) %>
+    </p>
+  <% end %>
+<% end %>