fix(export): confirmation before all exports

Author: sifferhans

frontend/app/components/export/ExportForm.vue

@@ -168,22 +168,25 @@ const selectMU1 = () =>
 const selectMU2 = () =>
     setLangs(props.config.languages.filter((l) => l.mu2).map((l) => l.code));
 
-// Bulk runs can launch many workflows at once, so confirm first.
+// Every export is confirmed first — bulk runs can launch many workflows at
+// once, and single-asset runs are still irreversible.
 const confirmOpen = ref(false);
 
+const confirmTitle = computed(() =>
+    props.bulkMode ? t("export.bulkConfirmTitle") : t("export.confirmTitle"),
+);
+
 const confirmMessage = computed(() =>
-    t("export.bulkConfirmMessage", {
-        n: exportableAssets.value.length,
-        d: selectedDestCount.value,
-    }),
+    props.bulkMode
+        ? t("export.bulkConfirmMessage", {
+              n: exportableAssets.value.length,
+              d: selectedDestCount.value,
+          })
+        : t("export.confirmMessage", { d: selectedDestCount.value }),
 );
 
 function attemptExport() {
-    if (props.bulkMode) {
-        confirmOpen.value = true;
-        return;
-    }
-    startExport();
+    confirmOpen.value = true;
 }
 
 function confirmExport() {
@@ -534,10 +537,10 @@ function startExport() {
             </div>
         </div>
 
-        <!-- Bulk export confirmation -->
+        <!-- Export confirmation (both single-asset and bulk) -->
         <DesignDialog
             v-model:open="confirmOpen"
-            :title="$t('export.bulkConfirmTitle')"
+            :title="confirmTitle"
             :description="confirmMessage"
         >
             <div class="flex w-full justify-end gap-2">
@@ -549,7 +552,11 @@ function startExport() {
                     icon="tabler:file-export"
                     @click="confirmExport"
                 >
-                    {{ $t("export.bulkStart") }}
+                    {{
+                        bulkMode
+                            ? $t("export.bulkStart")
+                            : $t("export.startExport")
+                    }}
                 </DesignButton>
             </div>
         </DesignDialog>

frontend/app/components/vb-export/VbExportForm.vue

@@ -105,22 +105,27 @@ const selectionSummary = computed(() =>
 
 /* ---------------------------------------------------------------- actions --- */
 
-// Bulk runs can launch many workflows at once, so confirm first.
+// Every export is confirmed first — bulk runs can launch many workflows at
+// once, and single-asset runs are still irreversible.
 const confirmOpen = ref(false);
 
+const confirmTitle = computed(() =>
+    props.bulkMode
+        ? t("vbExport.bulkConfirmTitle")
+        : t("vbExport.confirmTitle"),
+);
+
 const confirmMessage = computed(() =>
-    t("vbExport.bulkConfirmMessage", {
-        n: exportableAssets.value.length,
-        d: selectedDestCount.value,
-    }),
+    props.bulkMode
+        ? t("vbExport.bulkConfirmMessage", {
+              n: exportableAssets.value.length,
+              d: selectedDestCount.value,
+          })
+        : t("vbExport.confirmMessage", { d: selectedDestCount.value }),
 );
 
 function attemptExport() {
-    if (props.bulkMode) {
-        confirmOpen.value = true;
-        return;
-    }
-    startExport();
+    confirmOpen.value = true;
 }
 
 function confirmExport() {
@@ -299,10 +304,10 @@ function startExport() {
             </div>
         </div>
 
-        <!-- Bulk export confirmation -->
+        <!-- Export confirmation (both single-asset and bulk) -->
         <DesignDialog
             v-model:open="confirmOpen"
-            :title="$t('vbExport.bulkConfirmTitle')"
+            :title="confirmTitle"
             :description="confirmMessage"
         >
             <div class="flex w-full justify-end gap-2">
@@ -314,7 +319,11 @@ function startExport() {
                     icon="tabler:file-export"
                     @click="confirmExport"
                 >
-                    {{ $t("vbExport.bulkStart") }}
+                    {{
+                        bulkMode
+                            ? $t("vbExport.bulkStart")
+                            : $t("vbExport.startExport")
+                    }}
                 </DesignButton>
             </div>
         </DesignDialog>

frontend/docs/design-system-migration.md

@@ -234,6 +234,20 @@ files. Pending human visual review.
   changing the `DesignButton` variant.
 - Focus ring: every interactive `Design*` element carries `ds-focus-ring` (see prior entry).
 
+### 2026-06-23 — confirm single-asset exports too (behavior change, not migration)
+
+Investigated a report of "export started without a confirmation dialog." Root cause: the
+export-trigger logic (`attemptExport`) was **unchanged by the migration** (git diff of the script
+is empty) — confirmation only ever gated *bulk* mode; single-asset exports have always fired
+immediately. Verified `DesignDialog` opens correctly via `v-model:open` in isolation, so not a
+dialog regression.
+
+Per user decision, single-asset exports now also confirm. Both `ExportForm.vue` and
+`VbExportForm.vue`: `attemptExport()` now always opens the dialog; added `confirmTitle` +
+branched `confirmMessage` computeds (bulk vs single); dialog confirm-button label is bulk/single
+aware. New i18n keys `export.confirmTitle`/`confirmMessage` and `vbExport.confirmTitle`/
+`confirmMessage` in `en.json` + `nb.json`. `pnpm typecheck` ✅.
+
 ### Next steps (pick up here)
 
 1. **Human visual review** of `/export` (with and without `?id=`) in light + dark. Compare against

frontend/locales/en.json

@@ -129,6 +129,8 @@
         "bulkStart": "Start bulk export",
         "bulkConfirmTitle": "Start bulk export?",
         "bulkConfirmMessage": "This will start export for {n} asset(s) to {d} destination(s).",
+        "confirmTitle": "Start export?",
+        "confirmMessage": "This will start the export to {d} destination(s).",
         "cancel": "Cancel",
         "bulkStartedCount": "Started export for {n} asset(s)",
         "bulkFailedCount": "{n} failed"
@@ -159,6 +161,8 @@
         "bulkStart": "Start bulk export",
         "bulkConfirmTitle": "Start bulk export?",
         "bulkConfirmMessage": "This will start export for {n} asset(s) to {d} destination(s).",
+        "confirmTitle": "Start export?",
+        "confirmMessage": "This will start the export to {d} destination(s).",
         "cancel": "Cancel",
         "bulkStartedCount": "Started export for {n} asset(s)",
         "bulkFailedCount": "{n} failed"

frontend/locales/nb.json

@@ -129,6 +129,8 @@
         "bulkStart": "Start masseeksport",
         "bulkConfirmTitle": "Starte masseeksport?",
         "bulkConfirmMessage": "Dette starter eksport for {n} element(er) til {d} destinasjon(er).",
+        "confirmTitle": "Starte eksport?",
+        "confirmMessage": "Dette starter eksporten til {d} destinasjon(er).",
         "cancel": "Avbryt",
         "bulkStartedCount": "Startet eksport for {n} element(er)",
         "bulkFailedCount": "{n} feilet"
@@ -159,6 +161,8 @@
         "bulkStart": "Start masseeksport",
         "bulkConfirmTitle": "Starte masseeksport?",
         "bulkConfirmMessage": "Dette starter eksport for {n} element(er) til {d} destinasjon(er).",
+        "confirmTitle": "Starte eksport?",
+        "confirmMessage": "Dette starter eksporten til {d} destinasjon(er).",
         "cancel": "Avbryt",
         "bulkStartedCount": "Startet eksport for {n} element(er)",
         "bulkFailedCount": "{n} feilet"