completion-kit 0.26.1 → 0.26.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0e0435edbf9ccffb90715ab2d16fca2d158ad8e81767d550c3290c61a18de5ef
-  data.tar.gz: 365c545d14c7e9795f822cbda48d88327d70ea2b825276a4cb3b5e57b3ca2f6d
+  metadata.gz: 1fd4ab63494debc29d699a8cc14a9f5fc2a30c52f3d85e7c1afeedb8566eb55b
+  data.tar.gz: 46ad64c09d27c95c98f284efb3ab8fc1b9790c8c84d85a38461ed5fb67cc42fe
 SHA512:
-  metadata.gz: 046de8a9141dcd65497cde60b4fe5fae8cae31c7813f1c973469e4294a531292414d035979e1594e56829a0c3e81fd13eacd6d59221da9b8009a0f29574f38ba
-  data.tar.gz: 913f8ccbf9a907c958a29bd8e6af9c272d3fb55e4b1dd526c51dc4fc310fa20a06901818665adc09fa2ce440f141feb0d751f84b3a2bcaf2938c6183ff78cd9c
+  metadata.gz: 56277125c6b63a340796b7b409ca7f6307d2e8ad35ff751ef9e1f01a2a61706b72e8e43960dc43b2d167f370a3cbb95b140ca6f7cc3607e2c74cb499f6e940bf
+  data.tar.gz: b57d0ba59815c246c48f704cc75d5a6e91b301c7783b518e3a98f0acd6b413b1026318f762a1e6595c3726f803007b55c5e920699382c461ffa8176b85a7bec5

data/app/assets/javascripts/completion_kit/application.js

@@ -290,6 +290,30 @@ document.addEventListener("change", function(e) {
   }
 });
 
+function ckApplyProviderFields(scope) {
+  if (!scope) return;
+  var select = scope.querySelector("[data-ck-provider-select]");
+  if (!select) return;
+  var provider = select.value;
+  scope.querySelectorAll("[data-ck-provider-field]").forEach(function(field) {
+    var wanted = (field.getAttribute("data-ck-provider-field") || "").split(",");
+    field.hidden = wanted.indexOf(provider) === -1;
+  });
+}
+
+document.addEventListener("turbo:load", function() {
+  document.querySelectorAll("[data-ck-provider-select]").forEach(function(select) {
+    ckApplyProviderFields(select.closest("form") || document);
+  });
+});
+
+document.addEventListener("change", function(e) {
+  var target = e.target;
+  if (target && target.matches && target.matches("[data-ck-provider-select]")) {
+    ckApplyProviderFields(target.closest("form") || document);
+  }
+});
+
 document.addEventListener("click", function(e) {
   var btn = e.target.closest("[data-ck-apply]");
   if (!btn) return;
@@ -378,3 +402,51 @@ document.addEventListener("keydown", function(e) {
   else if (e.key === "Enter" || e.key === " ") { e.preventDefault(); if (options[idx]) ckSelectChoose(sel, options[idx]); }
   else if (e.key === "Escape") { e.preventDefault(); ckSelectClose(sel); trigger.focus(); }
 });
+
+function ckConfirmModal(message, formEl, submitter) {
+  var dialog = document.getElementById("ck-confirm-modal");
+  if (!dialog || typeof dialog.showModal !== "function") {
+    return Promise.resolve(window.confirm(message));
+  }
+  var messageEl = dialog.querySelector("#ck-confirm-message");
+  var acceptBtn = dialog.querySelector("[data-ck-confirm-accept]");
+  var cancelBtn = dialog.querySelector("[data-ck-confirm-cancel]");
+  if (messageEl) messageEl.textContent = message || "Are you sure?";
+  var label = (submitter && submitter.getAttribute("data-ck-confirm-label")) || "Confirm";
+  var tone = (submitter && submitter.getAttribute("data-ck-confirm-tone")) || "";
+  if (acceptBtn) {
+    acceptBtn.textContent = label;
+    acceptBtn.className = "ck-button " + (tone === "danger" ? "ck-button--danger" : "ck-button--primary");
+  }
+  return new Promise(function(resolve) {
+    function onClose() {
+      dialog.removeEventListener("close", onClose);
+      acceptBtn.removeEventListener("click", onAccept);
+      cancelBtn.removeEventListener("click", onCancel);
+      dialog.removeEventListener("click", onBackdrop);
+      resolve(dialog.returnValue === "accept");
+    }
+    function onAccept() { dialog.close("accept"); }
+    function onCancel() { dialog.close("cancel"); }
+    function onBackdrop(e) { if (e.target === dialog) dialog.close("cancel"); }
+    dialog.addEventListener("close", onClose);
+    acceptBtn.addEventListener("click", onAccept);
+    cancelBtn.addEventListener("click", onCancel);
+    dialog.addEventListener("click", onBackdrop);
+    dialog.returnValue = "";
+    dialog.showModal();
+    var focusTarget = (tone === "danger" ? cancelBtn : acceptBtn);
+    if (focusTarget) focusTarget.focus();
+  });
+}
+
+function ckInstallConfirm() {
+  if (!window.Turbo) return;
+  if (Turbo.config && Turbo.config.forms) {
+    Turbo.config.forms.confirm = ckConfirmModal;
+  } else if (typeof Turbo.setConfirmMethod === "function") {
+    Turbo.setConfirmMethod(ckConfirmModal);
+  }
+}
+document.addEventListener("turbo:load", ckInstallConfirm);
+ckInstallConfirm();

data/app/assets/stylesheets/completion_kit/application.css

@@ -2259,6 +2259,21 @@ label.ck-checkbox[hidden] {
   outline-offset: 2px;
 }
 
+.ck-icon-btn:disabled {
+  opacity: 0.4;
+  cursor: not-allowed;
+}
+
+.ck-icon-btn:disabled:hover {
+  background: none;
+  border-color: transparent;
+  color: var(--ck-dim);
+}
+
+.ck-delete-form {
+  display: contents;
+}
+
 .ck-icon-btn--spinning svg {
   animation: ck-spin 0.8s linear infinite;
 }
@@ -2711,6 +2726,17 @@ select.ck-input {
   backdrop-filter: blur(4px);
 }
 
+.ck-modal--sm {
+  max-width: min(92vw, 440px);
+}
+
+.ck-confirm__message {
+  margin: 0;
+  color: var(--ck-muted);
+  font-size: 0.95rem;
+  line-height: 1.55;
+}
+
 .ck-modal[open] {
   animation: ck-modal-in 0.18s ease-out;
 }

data/app/helpers/completion_kit/application_helper.rb

@@ -29,6 +29,21 @@ module CompletionKit
       "#{base} #{styles}"
     end
 
+    def ck_delete_trigger(form_id:, label:, confirm: nil, disabled: false, title: nil)
+      content_tag(
+        :button,
+        type: "submit",
+        form: (disabled ? nil : form_id),
+        class: "ck-icon-btn",
+        title: title || label,
+        "aria-label": label,
+        disabled: disabled,
+        data: (disabled ? {} : {turbo_confirm: confirm, ck_confirm_label: "Delete", ck_confirm_tone: "danger"})
+      ) do
+        heroicon_tag "trash", variant: :outline, size: 16, "aria-hidden": "true"
+      end
+    end
+
     def ck_badge_classes(kind)
       case kind.to_s
       when "high"

data/app/views/completion_kit/datasets/_form.html.erb

@@ -1,3 +1,11 @@
+<% if dataset.persisted? %>
+  <% dataset_delete_id = "ck_delete_dataset_#{dataset.id}" %>
+  <% runs_n = dataset.runs.count %>
+  <% responses_n = CompletionKit::Response.where(run_id: dataset.runs.select(:id)).count %>
+  <% dataset_delete_confirm = runs_n.zero? ?
+       "Delete \"#{dataset.name}\"? It has no runs." :
+       "Delete \"#{dataset.name}\"? Cascades through #{pluralize(runs_n, 'run')} and #{pluralize(responses_n, 'response')} (and their reviews) that used this dataset." %>
+<% end %>
 <%= form_with(model: dataset, local: true) do |form| %>
   <% if dataset.errors.any? %>
     <div class="ck-flash ck-flash--alert" role="alert">
@@ -32,21 +40,7 @@
 
     <div class="ck-actions">
       <% if dataset.persisted? %>
-        <% runs_n = dataset.runs.count %>
-        <% responses_n = CompletionKit::Response.where(run_id: dataset.runs.select(:id)).count %>
-        <% confirm = if runs_n.zero?
-             "Delete \"#{dataset.name}\"? It has no runs."
-           else
-             "Delete \"#{dataset.name}\"? Cascades through #{pluralize(runs_n, 'run')} and #{pluralize(responses_n, 'response')} (and their reviews) that used this dataset."
-           end %>
-        <%= button_to dataset_path(dataset), method: :delete,
-              form_class: "inline-block",
-              class: "ck-icon-btn",
-              title: "Delete dataset",
-              "aria-label": "Delete dataset",
-              data: { turbo_confirm: confirm } do %>
-          <%= heroicon_tag "trash", variant: :outline, size: 16, "aria-hidden": "true" %>
-        <% end %>
+        <%= ck_delete_trigger(form_id: dataset_delete_id, label: "Delete dataset", confirm: dataset_delete_confirm) %>
       <% end %>
       <%= link_to "Cancel", datasets_path, class: ck_button_classes(:light, variant: :outline), tabindex: "0" %>
       <%= form.submit(dataset.persisted? ? "Save dataset" : "Create dataset", class: ck_button_classes(:dark)) %>
@@ -81,3 +75,7 @@ document.addEventListener('turbo:load', ckDatasetHeaderPreview);
 ckDatasetHeaderPreview();
   </script>
 <% end %>
+
+<% if dataset.persisted? %>
+  <%= render "completion_kit/shared/delete_form", form_id: dataset_delete_id, path: dataset_path(dataset) %>
+<% end %>

data/app/views/completion_kit/metric_groups/_form.html.erb

@@ -1,3 +1,10 @@
+<% if metric_group.persisted? %>
+  <% metric_group_delete_id = "ck_delete_metric_group_#{metric_group.id}" %>
+  <% members_n = metric_group.metrics.count %>
+  <% metric_group_delete_confirm = members_n.zero? ?
+       "Delete \"#{metric_group.name}\"? It has no members." :
+       "Delete \"#{metric_group.name}\"? Removes this grouping. #{pluralize(members_n, 'member metric')} #{members_n == 1 ? 'is' : 'are'} kept." %>
+<% end %>
 <%= form_with(model: metric_group, url: metric_group.persisted? ? metric_group_path(metric_group) : metric_groups_path, local: true) do |form| %>
   <% name_error = metric_group.errors[:name].first %>
 
@@ -81,21 +88,14 @@
 
     <div class="ck-actions">
       <% if metric_group.persisted? %>
-        <% members_n = metric_group.metrics.count %>
-        <% confirm = members_n.zero? ?
-             "Delete \"#{metric_group.name}\"? It has no members." :
-             "Delete \"#{metric_group.name}\"? Removes this grouping. #{pluralize(members_n, 'member metric')} #{members_n == 1 ? 'is' : 'are'} kept." %>
-        <%= button_to metric_group_path(metric_group), method: :delete,
-              form_class: "inline-block",
-              class: "ck-icon-btn",
-              title: "Delete metric group",
-              "aria-label": "Delete metric group",
-              data: { turbo_confirm: confirm } do %>
-          <%= heroicon_tag "trash", variant: :outline, size: 16, "aria-hidden": "true" %>
-        <% end %>
+        <%= ck_delete_trigger(form_id: metric_group_delete_id, label: "Delete metric group", confirm: metric_group_delete_confirm) %>
       <% end %>
       <%= link_to "Cancel", metrics_path, class: ck_button_classes(:light, variant: :outline), tabindex: "0" %>
       <%= form.submit(metric_group.persisted? ? "Save metric group" : "Create metric group", class: ck_button_classes(:dark)) %>
     </div>
   </div>
 <% end %>
+
+<% if metric_group.persisted? %>
+  <%= render "completion_kit/shared/delete_form", form_id: metric_group_delete_id, path: metric_group_path(metric_group) %>
+<% end %>

data/app/views/completion_kit/metrics/_form.html.erb

@@ -3,6 +3,58 @@
 <% suggestion_bands = suggestion ? Array(suggestion.rubric_bands).each_with_object({}) { |b, h| h[b["stars"].to_i] = b["description"].to_s } : {} %>
 <% suggested_instruction = suggestion&.instruction.to_s %>
 <% instruction_changed = suggestion && suggested_instruction.present? && suggested_instruction != metric.instruction.to_s %>
+<% if metric.persisted? %>
+  <% metric_delete_id = "ck_delete_metric_#{metric.id}" %>
+  <% groups_n = metric.metric_groups.count %>
+  <% reviews_n = metric.reviews.count %>
+  <% parts = [] %>
+  <% parts << "in #{pluralize(groups_n, 'metric group')} (removed from each)" if groups_n > 0 %>
+  <% parts << "scored in #{pluralize(reviews_n, 'review')} (scores kept, link cleared)" if reviews_n > 0 %>
+  <% metric_delete_confirm = parts.empty? ? "Delete \"#{metric.name}\"? It's not in use." : "Delete \"#{metric.name}\"? It's #{parts.to_sentence}." %>
+<% end %>
+
+<% if edit_draft %>
+  <% pub = local_assigns[:published_metric_version] %>
+  <% draft_instr_changed = pub && pub.instruction.to_s != edit_draft.instruction.to_s %>
+  <% draft_rubric_changed = pub && pub.rubric_bands != edit_draft.rubric_bands %>
+  <div class="ck-suggestion-banner" role="status">
+    <div class="ck-suggestion-banner__body">
+      <p class="ck-kicker">Draft pending</p>
+      <p class="ck-meta-copy">The form below shows your unpublished draft. Publish to replace the live<%= " instruction" if draft_instr_changed %><%= " and" if draft_instr_changed && draft_rubric_changed %><%= " rubric" if draft_rubric_changed %> for future runs, or keep editing.</p>
+    </div>
+    <div class="ck-suggestion-banner__actions">
+      <%= button_to "Discard draft", dismiss_suggestion_metric_path(metric, draft_id: edit_draft.id, back_to: "edit"),
+            method: :delete, form_class: "inline-block",
+            class: ck_button_classes(:light, variant: :outline),
+            data: { turbo_confirm: "Drop this draft?", ck_confirm_label: "Discard", ck_confirm_tone: "danger" } %>
+      <%= button_to "Publish this version", publish_draft_metric_path(metric, draft_id: edit_draft.id),
+            method: :post, form_class: "inline-block",
+            class: ck_button_classes(:dark) %>
+    </div>
+  </div>
+<% end %>
+
+<% if suggestion %>
+  <div class="ck-suggestion-banner" role="status">
+    <div class="ck-suggestion-banner__body">
+      <p class="ck-kicker ck-kicker--icon"><%= heroicon_tag "sparkles", variant: :outline, class: "ck-magic-icon", "aria-hidden": "true" %>Proposed changes</p>
+      <p class="ck-meta-copy">Based on human reviews, here are some proposed changes to the metric.</p>
+    </div>
+    <div class="ck-suggestion-banner__actions">
+      <%= button_to suggest_variants_metric_path(metric, back_to: "edit"),
+            method: :post, form_class: "inline-block", class: "ck-icon-btn",
+            title: "Try again", "aria-label": "Try again",
+            data: { turbo_confirm: "Replace these changes with fresh ones from the model?" } do %><%= heroicon_tag "arrow-path", variant: :outline, size: 16, "aria-hidden": "true" %><% end %>
+      <%= button_to dismiss_suggestion_metric_path(metric, draft_id: suggestion.id, back_to: "edit"),
+            method: :delete, form_class: "inline-block", class: "ck-icon-btn",
+            title: "Discard these changes", "aria-label": "Discard",
+            data: { turbo_confirm: "Drop these changes?", ck_confirm_label: "Discard", ck_confirm_tone: "danger" } do %><%= heroicon_tag "trash", variant: :outline, size: 16, "aria-hidden": "true" %><% end %>
+      <%= button_to "Apply all", publish_draft_metric_path(metric, draft_id: suggestion.id),
+            method: :post, form_class: "inline-block",
+            class: ck_button_classes(:dark) %>
+    </div>
+  </div>
+<% end %>
 
 <%= form_with(model: metric, local: true) do |form| %>
   <% if metric.errors.any? %>
@@ -16,49 +68,6 @@
     </div>
   <% end %>
 
-  <% if edit_draft %>
-    <% pub = local_assigns[:published_metric_version] %>
-    <% draft_instr_changed = pub && pub.instruction.to_s != edit_draft.instruction.to_s %>
-    <% draft_rubric_changed = pub && pub.rubric_bands != edit_draft.rubric_bands %>
-    <div class="ck-suggestion-banner" role="status">
-      <div class="ck-suggestion-banner__body">
-        <p class="ck-kicker">Draft pending</p>
-        <p class="ck-meta-copy">The form below shows your unpublished draft. Publish to replace the live<%= " instruction" if draft_instr_changed %><%= " and" if draft_instr_changed && draft_rubric_changed %><%= " rubric" if draft_rubric_changed %> for future runs, or keep editing.</p>
-      </div>
-      <div class="ck-suggestion-banner__actions">
-        <%= button_to "Discard draft", dismiss_suggestion_metric_path(metric, draft_id: edit_draft.id, back_to: "edit"),
-              method: :delete, form_class: "inline-block",
-              class: ck_button_classes(:light, variant: :outline),
-              data: { turbo_confirm: "Drop this draft?" } %>
-        <%= button_to "Publish this version", publish_draft_metric_path(metric, draft_id: edit_draft.id),
-              method: :post, form_class: "inline-block",
-              class: ck_button_classes(:dark) %>
-      </div>
-    </div>
-  <% end %>
-
-  <% if suggestion %>
-    <div class="ck-suggestion-banner" role="status">
-      <div class="ck-suggestion-banner__body">
-        <p class="ck-kicker ck-kicker--icon"><%= heroicon_tag "sparkles", variant: :outline, class: "ck-magic-icon", "aria-hidden": "true" %>Proposed changes</p>
-        <p class="ck-meta-copy">Based on human reviews, here are some proposed changes to the metric.</p>
-      </div>
-      <div class="ck-suggestion-banner__actions">
-        <%= button_to suggest_variants_metric_path(metric, back_to: "edit"),
-              method: :post, form_class: "inline-block", class: "ck-icon-btn",
-              title: "Try again", "aria-label": "Try again",
-              data: { turbo_confirm: "Replace these changes with fresh ones from the model?" } do %><%= heroicon_tag "arrow-path", variant: :outline, size: 16, "aria-hidden": "true" %><% end %>
-        <%= button_to dismiss_suggestion_metric_path(metric, draft_id: suggestion.id, back_to: "edit"),
-              method: :delete, form_class: "inline-block", class: "ck-icon-btn",
-              title: "Discard these changes", "aria-label": "Discard",
-              data: { turbo_confirm: "Drop these changes?" } do %><%= heroicon_tag "trash", variant: :outline, size: 16, "aria-hidden": "true" %><% end %>
-        <%= button_to "Apply all", publish_draft_metric_path(metric, draft_id: suggestion.id),
-              method: :post, form_class: "inline-block",
-              class: ck_button_classes(:dark) %>
-      </div>
-    </div>
-  <% end %>
-
   <div class="ck-card ck-form-card">
     <div class="ck-field">
       <%= form.label :name, "Metric name", class: "ck-label" %>
@@ -255,23 +264,14 @@
 
     <div class="ck-actions">
       <% if metric.persisted? %>
-        <% groups_n = metric.metric_groups.count %>
-        <% reviews_n = metric.reviews.count %>
-        <% parts = [] %>
-        <% parts << "in #{pluralize(groups_n, 'metric group')} (removed from each)" if groups_n > 0 %>
-        <% parts << "scored in #{pluralize(reviews_n, 'review')} (scores kept, link cleared)" if reviews_n > 0 %>
-        <% confirm = parts.empty? ? "Delete \"#{metric.name}\"? It's not in use." : "Delete \"#{metric.name}\"? It's #{parts.to_sentence}." %>
-        <%= button_to metric_path(metric), method: :delete,
-              form_class: "inline-block",
-              class: "ck-icon-btn ck-icon-btn--form",
-              title: "Delete metric",
-              "aria-label": "Delete metric",
-              data: { turbo_confirm: confirm } do %>
-          <%= heroicon_tag "trash", variant: :outline, size: 24, "aria-hidden": "true" %>
-        <% end %>
+        <%= ck_delete_trigger(form_id: metric_delete_id, label: "Delete metric", confirm: metric_delete_confirm) %>
       <% end %>
       <%= link_to "Cancel", metrics_path, class: ck_button_classes(:light, variant: :outline), tabindex: "0" %>
       <%= form.submit(metric.persisted? ? "Save metric" : "Create metric", class: ck_button_classes(:dark)) %>
     </div>
   </div>
 <% end %>
+
+<% if metric.persisted? %>
+  <%= render "completion_kit/shared/delete_form", form_id: metric_delete_id, path: metric_path(metric) %>
+<% end %>

data/app/views/completion_kit/prompts/_form.html.erb

@@ -1,3 +1,11 @@
+<% if prompt.persisted? %>
+  <% prompt_delete_id = "ck_delete_prompt_#{prompt.id}" %>
+  <% runs_n = prompt.runs.count %>
+  <% responses_n = prompt.responses.count %>
+  <% prompt_delete_confirm = runs_n.zero? ?
+       "Delete \"#{prompt.display_name}\"? This version has no runs." :
+       "Delete \"#{prompt.display_name}\"? Cascades through #{pluralize(runs_n, 'run')} and #{pluralize(responses_n, 'response')} (and their reviews). Other versions of this prompt are untouched." %>
+<% end %>
 <%= form_with(model: prompt, local: true) do |form| %>
   <% if prompt.errors.any? %>
     <div class="ck-flash ck-flash--alert" role="alert">
@@ -11,7 +19,7 @@
   <% end %>
 
   <% if prompt.persisted? && prompt.runs.exists? %>
-    <p class="ck-version-note">Editing <%= prompt.version_label %> — it has runs, so saving creates a new version of this prompt.</p>
+    <p class="ck-version-note">Editing <%= prompt.version_label %>. It has runs, so saving creates a new version of this prompt.</p>
   <% end %>
 
   <div class="ck-card ck-form-card">
@@ -39,7 +47,7 @@
       <% available = CompletionKit::ApiConfig.available_models(scope: :generation) %>
       <% if available.any? %>
         <div class="ck-select-with-action">
-          <%= form.select :llm_model, ck_grouped_models(available, prompt.llm_model), { include_blank: "— Select a model —" }, { class: "ck-input", id: "prompt_llm_model" } %>
+          <%= form.select :llm_model, ck_grouped_models(available, prompt.llm_model), { include_blank: "Select a model" }, { class: "ck-input", id: "prompt_llm_model" } %>
           <% ck_refreshing = CompletionKit::ProviderCredential.discovery_in_progress? %>
           <button type="button" class="ck-icon-btn<%= ' ck-icon-btn--spinning' if ck_refreshing %>" title="Refresh models" <%= 'disabled' if ck_refreshing %> onclick="ckRefreshModels()"><svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16" fill="currentColor" width="16" height="16"><path fill-rule="evenodd" d="M13.836 2.477a.75.75 0 0 1 .75.75v3.182a.75.75 0 0 1-.75.75h-3.182a.75.75 0 0 1 0-1.5h1.37l-.84-.841a4.5 4.5 0 0 0-7.08.681.75.75 0 0 1-1.264-.808 6 6 0 0 1 9.44-.908l.84.84V3.227a.75.75 0 0 1 .75-.75Zm-.911 7.5A.75.75 0 0 1 13.199 11a6 6 0 0 1-9.44.908l-.84-.84v1.68a.75.75 0 0 1-1.5 0V9.567a.75.75 0 0 1 .75-.75h3.182a.75.75 0 0 1 0 1.5h-1.37l.84.841a4.5 4.5 0 0 0 7.08-.681.75.75 0 0 1 1.024-.274Z" clip-rule="evenodd"/></svg></button>
         </div>
@@ -59,24 +67,14 @@
 
     <div class="ck-actions">
       <% if prompt.persisted? %>
-        <% runs_n = prompt.runs.count %>
-        <% responses_n = prompt.responses.count %>
-        <% confirm = if runs_n.zero?
-             "Delete \"#{prompt.display_name}\"? This version has no runs."
-           else
-             "Delete \"#{prompt.display_name}\"? Cascades through #{pluralize(runs_n, 'run')} and #{pluralize(responses_n, 'response')} (and their reviews). Other versions of this prompt are untouched."
-           end %>
-        <%= button_to prompt_path(prompt), method: :delete,
-              form_class: "inline-block",
-              class: "ck-icon-btn",
-              title: "Delete prompt",
-              "aria-label": "Delete prompt",
-              data: { turbo_confirm: confirm } do %>
-          <%= heroicon_tag "trash", variant: :outline, size: 16, "aria-hidden": "true" %>
-        <% end %>
+        <%= ck_delete_trigger(form_id: prompt_delete_id, label: "Delete prompt", confirm: prompt_delete_confirm) %>
       <% end %>
       <%= link_to "Cancel", prompts_path, class: ck_button_classes(:light, variant: :outline), tabindex: "0" %>
       <%= form.submit(prompt.persisted? ? "Save prompt" : "Create prompt", class: ck_button_classes(:dark), disabled: available.empty?) %>
     </div>
   </div>
 <% end %>
+
+<% if prompt.persisted? %>
+  <%= render "completion_kit/shared/delete_form", form_id: prompt_delete_id, path: prompt_path(prompt) %>
+<% end %>

data/app/views/completion_kit/provider_credentials/_form.html.erb

@@ -1,3 +1,8 @@
+<% if provider_credential.persisted? %>
+  <% pc_delete_id = "ck_delete_provider_#{provider_credential.id}" %>
+  <% pc_delete_blocked = provider_credential.in_use? %>
+  <% pc_delete_confirm = "Delete the #{provider_credential.display_provider} provider and its discovered models? This can't be undone." %>
+<% end %>
 <%= form_with(model: provider_credential, local: true) do |form| %>
   <% if provider_credential.errors.any? %>
     <div class="ck-flash ck-flash--alert" role="alert">
@@ -13,7 +18,7 @@
   <div class="ck-card ck-form-card">
     <div class="ck-field">
       <%= form.label :provider, class: "ck-label" %>
-      <%= form.select :provider, CompletionKit::ProviderCredential::PROVIDERS.map { |p| [CompletionKit::ProviderCredential::PROVIDER_LABELS[p] || p.titleize, p] }, {}, { class: "ck-input" } %>
+      <%= form.select :provider, CompletionKit::ProviderCredential::PROVIDERS.map { |p| [CompletionKit::ProviderCredential::PROVIDER_LABELS[p] || p.titleize, p] }, {}, { class: "ck-input", data: { ck_provider_select: true } } %>
     </div>
 
     <div class="ck-field">
@@ -28,35 +33,28 @@
       <%= ck_field_error(form, :api_endpoint) %>
     </div>
 
-    <div class="ck-field">
+    <div class="ck-field" data-ck-provider-field="azure_foundry"<%= provider_credential.azure_foundry? ? "" : " hidden" %>>
       <%= form.label :api_version, "API version", class: "ck-label" %>
-      <%= form.text_field :api_version, class: "ck-input", placeholder: "Only needed for Azure AI Foundry (e.g. 2024-10-21)", **ck_field_aria(form, :api_version) %>
+      <%= form.text_field :api_version, class: "ck-input", placeholder: "e.g. 2024-10-21", **ck_field_aria(form, :api_version) %>
       <%= ck_field_error(form, :api_version) %>
     </div>
 
     <div class="ck-actions">
+      <% if provider_credential.persisted? %>
+        <%= ck_delete_trigger(form_id: pc_delete_id, label: "Delete provider", confirm: pc_delete_confirm, disabled: pc_delete_blocked, title: (pc_delete_blocked ? provider_credential.in_use_message : "Delete provider")) %>
+      <% end %>
       <%= link_to "Cancel", provider_credentials_path, class: ck_button_classes(:light, variant: :outline), tabindex: "0" %>
       <%= form.submit(provider_credential.persisted? ? "Save provider" : "Create provider", class: ck_button_classes(:dark)) %>
     </div>
+    <% if provider_credential.persisted? && pc_delete_blocked %>
+      <p class="ck-field-hint"><%= provider_credential.in_use_message %></p>
+    <% end %>
 
   </div>
 <% end %>
 
-<% if provider_credential.persisted? %>
-  <% if provider_credential.in_use? %>
-    <p class="ck-field-hint"><%= provider_credential.in_use_message %></p>
-  <% else %>
-    <div class="ck-actions">
-      <%= button_to provider_credential_path(provider_credential), method: :delete,
-            form_class: "inline-block",
-            class: "ck-icon-btn",
-            title: "Delete provider",
-            "aria-label": "Delete provider",
-            data: { turbo_confirm: "Delete the #{provider_credential.display_provider} provider and its discovered models? This can't be undone." } do %>
-        <%= heroicon_tag "trash", variant: :outline, size: 16, "aria-hidden": "true" %>
-      <% end %>
-    </div>
-  <% end %>
+<% if provider_credential.persisted? && !pc_delete_blocked %>
+  <%= render "completion_kit/shared/delete_form", form_id: pc_delete_id, path: provider_credential_path(provider_credential) %>
 <% end %>
 
 <% if provider_credential.persisted? %>

data/app/views/completion_kit/runs/_actions.html.erb

@@ -1,6 +1,6 @@
 <div class="ck-actions" id="run_actions">
   <% running = run.status == "running" %>
-  <%= button_to run_path(run), method: :delete, form_class: "inline-block", class: "ck-icon-btn", title: "Delete run", "aria-label": "Delete run", disabled: running, data: { turbo_confirm: "Delete this run and all its responses?" } do %><%= heroicon_tag "trash", variant: :outline, size: 16, "aria-hidden": "true" %><% end %>
+  <%= button_to run_path(run), method: :delete, form_class: "inline-block", class: "ck-icon-btn", title: "Delete run", "aria-label": "Delete run", disabled: running, data: { turbo_confirm: "Delete this run and all its responses?", ck_confirm_label: "Delete", ck_confirm_tone: "danger" } do %><%= heroicon_tag "trash", variant: :outline, size: 16, "aria-hidden": "true" %><% end %>
   <% if running %>
     <%= link_to "Edit", edit_run_path(run), class: ck_button_classes(:light, variant: :outline) + " disabled", "aria-disabled": "true", tabindex: "-1" %>
   <% else %>

data/app/views/completion_kit/shared/_delete_form.html.erb

@@ -0,0 +1,2 @@
+<%# locals: (form_id:, path:) %>
+<%= form_with url: path, method: :delete, id: form_id, class: "ck-delete-form" do %><% end %>

data/app/views/completion_kit/tags/_form.html.erb

@@ -1,3 +1,10 @@
+<% if tag.persisted? %>
+  <% tag_delete_id = "ck_delete_tag_#{tag.id}" %>
+  <% applied_n = tag.taggings.count %>
+  <% tag_delete_confirm = applied_n.zero? ?
+       "Delete \"#{tag.name}\"? It's not currently applied to anything." :
+       "Delete \"#{tag.name}\"? It's currently applied to #{pluralize(applied_n, 'item')}. They'll lose this tag." %>
+<% end %>
 <%= form_with(model: tag, local: true) do |form| %>
   <div class="ck-card ck-form-card">
     <% name_error = tag.errors[:name].first %>
@@ -18,23 +25,14 @@
 
 <div class="ck-actions">
       <% if tag.persisted? %>
-        <% applied_n = tag.taggings.count %>
-        <% confirm = if applied_n.zero?
-             "Delete \"#{tag.name}\"? It's not currently applied to anything."
-           else
-             "Delete \"#{tag.name}\"? It's currently applied to #{pluralize(applied_n, 'item')} — they'll lose this tag."
-           end %>
-        <%= button_to tag_path(tag), method: :delete,
-              form_class: "inline-block",
-              class: "ck-icon-btn",
-              title: "Delete tag",
-              "aria-label": "Delete tag",
-              data: { turbo_confirm: confirm } do %>
-          <%= heroicon_tag "trash", variant: :outline, size: 16, "aria-hidden": "true" %>
-        <% end %>
+        <%= ck_delete_trigger(form_id: tag_delete_id, label: "Delete tag", confirm: tag_delete_confirm) %>
       <% end %>
       <%= link_to "Cancel", tags_path, class: ck_button_classes(:light, variant: :outline), tabindex: "0" %>
       <%= form.submit(tag.persisted? ? "Save tag" : "Create tag", class: ck_button_classes(:dark)) %>
     </div>
   </div>
 <% end %>
+
+<% if tag.persisted? %>
+  <%= render "completion_kit/shared/delete_form", form_id: tag_delete_id, path: tag_path(tag) %>
+<% end %>

data/app/views/layouts/completion_kit/application.html.erb

@@ -60,5 +60,22 @@
     <%= yield %>
     </div>
   </main>
+
+  <dialog id="ck-confirm-modal" class="ck-modal ck-modal--sm" role="alertdialog" aria-labelledby="ck-confirm-title" aria-describedby="ck-confirm-message">
+    <article class="ck-modal__panel" tabindex="-1">
+      <header class="ck-modal__header">
+        <div class="ck-modal__heading">
+          <h2 class="ck-modal__title" id="ck-confirm-title">Please confirm</h2>
+        </div>
+      </header>
+      <div class="ck-modal__body">
+        <p class="ck-confirm__message" id="ck-confirm-message"></p>
+      </div>
+      <div class="ck-modal__footer">
+        <button type="button" class="<%= ck_button_classes(:light, variant: :outline) %>" data-ck-confirm-cancel>Cancel</button>
+        <button type="button" class="<%= ck_button_classes(:dark) %>" data-ck-confirm-accept>Confirm</button>
+      </div>
+    </article>
+  </dialog>
 </body>
 </html>

data/lib/completion_kit/version.rb

@@ -1,3 +1,3 @@
 module CompletionKit
-  VERSION = "0.26.1"
+  VERSION = "0.26.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: completion-kit
 version: !ruby/object:Gem::Version
-  version: 0.26.1
+  version: 0.26.3
 platform: ruby
 authors:
 - Damien Bastin
@@ -421,6 +421,7 @@ files:
 - app/views/completion_kit/runs/new.html.erb
 - app/views/completion_kit/runs/show.html.erb
 - app/views/completion_kit/shared/_branded_select.html.erb
+- app/views/completion_kit/shared/_delete_form.html.erb
 - app/views/completion_kit/shared/_settings_nav.html.erb
 - app/views/completion_kit/suggestions/_scoreboard.html.erb
 - app/views/completion_kit/suggestions/_state.html.erb