test(bulk-upload): add cancel-path coverage for discard-all confirm dialog

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

backend/src/main/java/org/raddatz/familienarchiv/controller/DocumentController.java

@@ -3,7 +3,6 @@ package org.raddatz.familienarchiv.controller;
 import java.io.IOException;
 import java.time.LocalDate;
 import java.util.ArrayList;
-import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
@@ -14,18 +13,12 @@ import java.util.UUID;
 
 import io.swagger.v3.oas.annotations.Parameter;
 import io.swagger.v3.oas.annotations.responses.ApiResponse;
-import jakarta.validation.Valid;
 import jakarta.validation.constraints.Max;
 import jakarta.validation.constraints.Min;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
 import org.springframework.validation.annotation.Validated;
-import org.raddatz.familienarchiv.dto.BatchMetadataRequest;
-import org.raddatz.familienarchiv.dto.BulkEditError;
-import org.raddatz.familienarchiv.dto.BulkEditResult;
 import org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO;
-import org.raddatz.familienarchiv.dto.DocumentBatchSummary;
-import org.raddatz.familienarchiv.dto.DocumentBulkEditDTO;
 import org.raddatz.familienarchiv.dto.DocumentSearchResult;
 import org.raddatz.familienarchiv.dto.DocumentUpdateDTO;
 import org.raddatz.familienarchiv.dto.TagOperator;
@@ -244,100 +237,6 @@ public class DocumentController {
         return new QuickUploadResult(created, updated, errors);
     }
 
-    // --- BULK EDIT ---
-
-    private static final int BULK_EDIT_MAX_IDS = 500;
-    /** Hard cap for {@code GET /api/documents/ids}: prevents an unfiltered
-     *  call from materialising the entire {@code documents} table into JSON.
-     *  Generous enough for real-world "Alle X editieren" against the family
-     *  archive's bounded scale (~1500 docs today, expected growth to ~5k). */
-    private static final int BULK_EDIT_FILTER_MAX_IDS = 5000;
-
-    @PatchMapping("/bulk")
-    @RequirePermission(Permission.WRITE_ALL)
-    public BulkEditResult patchBulk(
-            @RequestBody @Valid DocumentBulkEditDTO dto,
-            Authentication authentication) {
-        if (dto.getDocumentIds() == null || dto.getDocumentIds().isEmpty()) {
-            throw new ResponseStatusException(HttpStatus.BAD_REQUEST, "documentIds is required");
-        }
-        if (dto.getDocumentIds().size() > BULK_EDIT_MAX_IDS) {
-            throw DomainException.badRequest(ErrorCode.BULK_EDIT_TOO_MANY_IDS,
-                    "Maximum " + BULK_EDIT_MAX_IDS + " documents per request, got: " + dto.getDocumentIds().size());
-        }
-
-        UUID actorId = requireUserId(authentication);
-        int updated = 0;
-        List<BulkEditError> errors = new ArrayList<>();
-
-        // Dedupe duplicate document IDs while preserving submission order. A
-        // double-click on "Alle X editieren" would otherwise hit each document
-        // twice and inflate the `updated` count returned to the user.
-        LinkedHashSet<UUID> uniqueIds = new LinkedHashSet<>(dto.getDocumentIds());
-
-        for (UUID id : uniqueIds) {
-            try {
-                documentService.applyBulkEditToDocument(id, dto, actorId);
-                updated++;
-            } catch (DomainException e) {
-                errors.add(new BulkEditError(id, sanitizeForLog(e.getMessage())));
-            } catch (Exception e) {
-                errors.add(new BulkEditError(id, "Internal error"));
-                log.warn("Bulk edit failed for document {}: {}", id, sanitizeForLog(e.getMessage()));
-            }
-        }
-
-        log.info("bulkEdit actor={} documentIds={} unique={} updated={} errors={}",
-                actorId, dto.getDocumentIds().size(), uniqueIds.size(), updated, errors.size());
-
-        return new BulkEditResult(updated, errors);
-    }
-
-    /** CRLF strip for any log line interpolating a free-form string (e.g.
-     *  {@link Throwable#getMessage()}). Defends against CWE-117 log injection. */
-    private static String sanitizeForLog(String s) {
-        return s == null ? null : s.replaceAll("[\\r\\n]", "_");
-    }
-
-    @GetMapping("/ids")
-    @RequirePermission(Permission.WRITE_ALL)
-    public List<UUID> getDocumentIds(
-            @RequestParam(required = false) String q,
-            @RequestParam(required = false) LocalDate from,
-            @RequestParam(required = false) LocalDate to,
-            @RequestParam(required = false) UUID senderId,
-            @RequestParam(required = false) UUID receiverId,
-            @RequestParam(required = false, name = "tag") List<String> tags,
-            @RequestParam(required = false) String tagQ,
-            @RequestParam(required = false) DocumentStatus status,
-            @RequestParam(required = false) String tagOp,
-            Authentication authentication) {
-        TagOperator operator = "OR".equalsIgnoreCase(tagOp) ? TagOperator.OR : TagOperator.AND;
-        List<UUID> ids = documentService.findIdsForFilter(q, from, to, senderId, receiverId, tags, tagQ, status, operator);
-        if (ids.size() > BULK_EDIT_FILTER_MAX_IDS) {
-            throw DomainException.badRequest(ErrorCode.BULK_EDIT_TOO_MANY_IDS,
-                    "Filter matches " + ids.size() + " documents — refine filter (max " + BULK_EDIT_FILTER_MAX_IDS + ")");
-        }
-        UUID actorId = requireUserId(authentication);
-        log.info("documentIds actor={} matched={}", actorId, ids.size());
-        return ids;
-    }
-
-    @PostMapping(value = "/batch-metadata", consumes = MediaType.APPLICATION_JSON_VALUE)
-    @RequirePermission(Permission.READ_ALL)
-    public List<DocumentBatchSummary> batchMetadata(@RequestBody @Valid BatchMetadataRequest request, Authentication authentication) {
-        if (request == null || request.ids() == null || request.ids().isEmpty()) {
-            throw new ResponseStatusException(HttpStatus.BAD_REQUEST, "ids is required");
-        }
-        if (request.ids().size() > BULK_EDIT_MAX_IDS) {
-            throw DomainException.badRequest(ErrorCode.BULK_EDIT_TOO_MANY_IDS,
-                    "Maximum " + BULK_EDIT_MAX_IDS + " ids per request, got: " + request.ids().size());
-        }
-        UUID actorId = requireUserId(authentication);
-        log.info("batchMetadata actor={} ids={}", actorId, request.ids().size());
-        return documentService.batchMetadata(request.ids());
-    }
-
     @GetMapping("/incomplete-count")
     @RequirePermission(Permission.WRITE_ALL)
     public Map<String, Long> getIncompleteCount() {

backend/src/main/java/org/raddatz/familienarchiv/dto/BatchMetadataRequest.java

@@ -1,9 +0,0 @@
-package org.raddatz.familienarchiv.dto;
-
-import java.util.List;
-import java.util.UUID;
-
-import io.swagger.v3.oas.annotations.media.Schema;
-
-public record BatchMetadataRequest(
-        @Schema(requiredMode = Schema.RequiredMode.REQUIRED) List<UUID> ids) {}

backend/src/main/java/org/raddatz/familienarchiv/dto/BulkEditError.java

@@ -1,9 +0,0 @@
-package org.raddatz.familienarchiv.dto;
-
-import java.util.UUID;
-
-import io.swagger.v3.oas.annotations.media.Schema;
-
-public record BulkEditError(
-        @Schema(requiredMode = Schema.RequiredMode.REQUIRED) UUID id,
-        @Schema(requiredMode = Schema.RequiredMode.REQUIRED) String message) {}

backend/src/main/java/org/raddatz/familienarchiv/dto/BulkEditResult.java

@@ -1,9 +0,0 @@
-package org.raddatz.familienarchiv.dto;
-
-import java.util.List;
-
-import io.swagger.v3.oas.annotations.media.Schema;
-
-public record BulkEditResult(
-        @Schema(requiredMode = Schema.RequiredMode.REQUIRED) int updated,
-        @Schema(requiredMode = Schema.RequiredMode.REQUIRED) List<BulkEditError> errors) {}

backend/src/main/java/org/raddatz/familienarchiv/dto/DocumentBatchSummary.java

@@ -1,10 +0,0 @@
-package org.raddatz.familienarchiv.dto;
-
-import java.util.UUID;
-
-import io.swagger.v3.oas.annotations.media.Schema;
-
-public record DocumentBatchSummary(
-        @Schema(requiredMode = Schema.RequiredMode.REQUIRED) UUID id,
-        @Schema(requiredMode = Schema.RequiredMode.REQUIRED) String title,
-        @Schema(requiredMode = Schema.RequiredMode.REQUIRED) String pdfUrl) {}

backend/src/main/java/org/raddatz/familienarchiv/dto/DocumentBulkEditDTO.java

@@ -1,60 +0,0 @@
-package org.raddatz.familienarchiv.dto;
-
-import java.util.List;
-import java.util.UUID;
-
-import jakarta.validation.constraints.Size;
-import lombok.AllArgsConstructor;
-import lombok.Data;
-import lombok.NoArgsConstructor;
-
-/**
- * Request body for {@code PATCH /api/documents/bulk}. Field semantics:
- * <ul>
- *   <li>{@code tagNames} and {@code receiverIds} are <b>additive</b> —
- *       merged into each document's existing set, never replacing it.</li>
- *   <li>{@code senderId}, {@code documentLocation}, {@code archiveBox},
- *       {@code archiveFolder} are <b>replace-on-non-blank</b> — null/blank
- *       fields are skipped, anything else overwrites.</li>
- * </ul>
- *
- * <p>Kept as a Lombok {@code @Data} POJO (not a record) for symmetry with
- * the existing {@code DocumentUpdateDTO} and to keep test setup terse —
- * the per-feature DTOs introduced alongside this one ({@link BulkEditError},
- * {@link BulkEditResult}, {@link BatchMetadataRequest},
- * {@link DocumentBatchSummary}) <i>are</i> records because they have no
- * test-side mutation. Tracked in the cycle-1 review for follow-up.
- *
- * <p>Bean-validation caps below defend against payload-amplification: the
- * 1 MiB SvelteKit proxy cap allows ~26k UUIDs through to the backend, and
- * Jetty's default body limit is 8 MB. {@code @Size} guards catch malformed
- * clients without depending on those outer bounds.
- */
-@Data
-@NoArgsConstructor
-@AllArgsConstructor
-public class DocumentBulkEditDTO {
-
-    // No @Size cap here on purpose: the controller's BULK_EDIT_MAX_IDS check
-    // returns the typed BULK_EDIT_TOO_MANY_IDS error code, which the frontend
-    // maps to a localised "Maximal 500 …" message via Paraglide. A bean-
-    // validation @Size would short-circuit that with a generic VALIDATION_ERROR.
-    private List<UUID> documentIds;
-
-    @Size(max = 200, message = "tagNames must not exceed 200 entries")
-    private List<@Size(max = 200, message = "tagName must not exceed 200 chars") String> tagNames;
-
-    private UUID senderId;
-
-    @Size(max = 200, message = "receiverIds must not exceed 200 entries")
-    private List<UUID> receiverIds;
-
-    @Size(max = 255, message = "documentLocation must not exceed 255 chars")
-    private String documentLocation;
-
-    @Size(max = 255, message = "archiveBox must not exceed 255 chars")
-    private String archiveBox;
-
-    @Size(max = 255, message = "archiveFolder must not exceed 255 chars")
-    private String archiveFolder;
-}

backend/src/main/java/org/raddatz/familienarchiv/exception/ErrorCode.java

@@ -111,8 +111,6 @@ public enum ErrorCode {
     VALIDATION_ERROR,
     /** Batch upload exceeds the maximum allowed file count per request. 400 */
     BATCH_TOO_LARGE,
-    /** Bulk edit request exceeds the per-request document ID cap. 400 */
-    BULK_EDIT_TOO_MANY_IDS,
     /** An unexpected server-side error occurred. 500 */
     INTERNAL_ERROR,
 }

backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java

@@ -8,8 +8,6 @@ import org.raddatz.familienarchiv.audit.AuditKind;
 import org.raddatz.familienarchiv.audit.AuditLogQueryService;
 import org.raddatz.familienarchiv.audit.AuditService;
 import org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO;
-import org.raddatz.familienarchiv.dto.DocumentBatchSummary;
-import org.raddatz.familienarchiv.dto.DocumentBulkEditDTO;
 import org.raddatz.familienarchiv.dto.DocumentSearchItem;
 import org.raddatz.familienarchiv.dto.DocumentSearchResult;
 import org.raddatz.familienarchiv.dto.DocumentSort;
@@ -336,143 +334,20 @@ public class DocumentService {
     public Document updateDocumentTags(UUID docId, List<String> tagNames) {
         Document doc = documentRepository.findById(docId)
                 .orElseThrow(() -> DomainException.notFound(ErrorCode.DOCUMENT_NOT_FOUND, "Document not found: " + docId));
-        doc.setTags(resolveTags(tagNames));
-        return documentRepository.save(doc);
-    }
 
-    /**
-     * Resolves a list of tag-name strings to {@link Tag} entities, trimming
-     * whitespace and skipping blank entries. Single source of truth for
-     * "name string → Tag" so the find-or-create policy stays consistent
-     * across single-doc updates ({@link #updateDocumentTags}), bulk edits
-     * ({@link #applyBulkEditToDocument}), and the upload-batch path
-     * ({@code applyBatchMetadata}).
-     */
-    private Set<Tag> resolveTags(List<String> tagNames) {
-        if (tagNames == null || tagNames.isEmpty()) return new HashSet<>();
-        Set<Tag> resolved = new HashSet<>();
+        Set<Tag> newTags = new HashSet<>();
+
         for (String name : tagNames) {
+            // Clean the string
             String cleanName = name.trim();
-            if (cleanName.isEmpty()) continue;
-            resolved.add(tagService.findOrCreate(cleanName));
-        }
-        return resolved;
-    }
+            if (cleanName.isEmpty())
+                continue;
 
-    /**
-     * Returns all document IDs matching the given filter parameters, ignoring
-     * pagination. Used by the bulk-edit "Alle X editieren" fast path so the
-     * frontend can replace the selection with every match across pages in one
-     * round-trip.
-     */
-    @Transactional(readOnly = true)
-    public List<UUID> findIdsForFilter(String text, LocalDate from, LocalDate to, UUID sender, UUID receiver,
-                                       List<String> tags, String tagQ, DocumentStatus status, TagOperator tagOperator) {
-        boolean hasText = StringUtils.hasText(text);
-        List<UUID> rankedIds = null;
-        if (hasText) {
-            rankedIds = documentRepository.findRankedIdsByFts(text);
-            if (rankedIds.isEmpty()) return List.of();
+            newTags.add(tagService.findOrCreate(cleanName));
         }
 
-        Specification<Document> spec = buildSearchSpec(
-                hasText, rankedIds, from, to, sender, receiver, tags, tagQ, status, tagOperator);
-        return documentRepository.findAll(spec).stream().map(Document::getId).toList();
-    }
-
-    /**
-     * Single source of truth for the search Specification chain. Shared by
-     * {@link #searchDocuments} (paged + sorted) and {@link #findIdsForFilter}
-     * (uncapped, ID-only). Caller does its own FTS short-circuit when the
-     * full-text query returned no rows.
-     */
-    private Specification<Document> buildSearchSpec(boolean hasText, List<UUID> ftsIds,
-                                                    LocalDate from, LocalDate to,
-                                                    UUID sender, UUID receiver,
-                                                    List<String> tags, String tagQ,
-                                                    DocumentStatus status, TagOperator tagOperator) {
-        boolean useOrLogic = tagOperator == TagOperator.OR;
-        List<Set<UUID>> expandedTagSets = tagService.expandTagNamesToDescendantIdSets(tags);
-        Specification<Document> textSpec = hasText ? hasIds(ftsIds) : (root, query, cb) -> null;
-        return Specification.where(textSpec)
-                .and(isBetween(from, to))
-                .and(hasSender(sender))
-                .and(hasReceiver(receiver))
-                .and(hasTags(expandedTagSets, useOrLogic))
-                .and(hasTagPartial(tagQ))
-                .and(hasStatus(status));
-    }
-
-    /**
-     * Returns lightweight summaries (id, title, server PDF URL) for the given
-     * document IDs. Unknown IDs are silently dropped — the consumer is the
-     * bulk-edit page's left strip, where missing previews would already be
-     * obvious; surfacing them as errors here adds no value.
-     */
-    @Transactional(readOnly = true)
-    public List<DocumentBatchSummary> batchMetadata(List<UUID> ids) {
-        if (ids == null || ids.isEmpty()) return List.of();
-        return documentRepository.findAllById(ids).stream()
-                .map(d -> new DocumentBatchSummary(
-                        d.getId(),
-                        d.getTitle() != null ? d.getTitle() : d.getOriginalFilename(),
-                        "/api/documents/" + d.getId() + "/file"))
-                .toList();
-    }
-
-    /**
-     * Applies a bulk-edit DTO to a single document atomically.
-     * Tags and receivers are additive (merged into existing sets); sender and the
-     * three location fields are replace-on-non-blank (null/blank means "no change").
-     * Wrapped in its own transaction so a failure on one document never partially
-     * mutates another in the controller's batch loop.
-     *
-     * Each successful update emits a {@link AuditKind#METADATA_UPDATED} audit
-     * event tagged {@code source=BULK_EDIT} and writes a row to
-     * {@code document_versions} so the family archive's "who changed what"
-     * trail stays complete across both single- and bulk-doc edit paths.
-     *
-     * NOTE on N+1: tag and person resolution happens per-document. With 500
-     * documents × 10 tags this fans out to ~5000 tag-resolve queries per
-     * request. Acceptable today because the family archive is bounded at
-     * ~1500 documents total. Tracked as a perf follow-up.
-     */
-    @Transactional
-    public Document applyBulkEditToDocument(UUID id, DocumentBulkEditDTO dto, UUID actorId) {
-        Document doc = documentRepository.findById(id)
-                .orElseThrow(() -> DomainException.notFound(ErrorCode.DOCUMENT_NOT_FOUND, "Document not found: " + id));
-
-        if (dto.getTagNames() != null && !dto.getTagNames().isEmpty()) {
-            Set<Tag> merged = new HashSet<>(doc.getTags());
-            merged.addAll(resolveTags(dto.getTagNames()));
-            doc.setTags(merged);
-        }
-
-        if (dto.getSenderId() != null) {
-            doc.setSender(personService.getById(dto.getSenderId()));
-        }
-
-        if (dto.getReceiverIds() != null && !dto.getReceiverIds().isEmpty()) {
-            Set<Person> merged = new HashSet<>(doc.getReceivers());
-            merged.addAll(personService.getAllById(dto.getReceiverIds()));
-            doc.setReceivers(merged);
-        }
-
-        if (StringUtils.hasText(dto.getDocumentLocation())) {
-            doc.setDocumentLocation(dto.getDocumentLocation());
-        }
-        if (StringUtils.hasText(dto.getArchiveBox())) {
-            doc.setArchiveBox(dto.getArchiveBox());
-        }
-        if (StringUtils.hasText(dto.getArchiveFolder())) {
-            doc.setArchiveFolder(dto.getArchiveFolder());
-        }
-
-        Document saved = documentRepository.save(doc);
-        documentVersionService.recordVersion(saved);
-        auditService.logAfterCommit(AuditKind.METADATA_UPDATED, actorId, saved.getId(),
-                Map.of("source", "BULK_EDIT"));
-        return saved;
+        doc.setTags(newTags);
+        return documentRepository.save(doc);
     }
 
     /**
@@ -538,8 +413,17 @@ public class DocumentService {
             if (rankedIds.isEmpty()) return DocumentSearchResult.of(List.of());
         }
 
-        Specification<Document> spec = buildSearchSpec(
-                hasText, rankedIds, from, to, sender, receiver, tags, tagQ, status, tagOperator);
+        boolean useOrLogic = tagOperator == TagOperator.OR;
+        List<Set<UUID>> expandedTagSets = tagService.expandTagNamesToDescendantIdSets(tags);
+
+        Specification<Document> textSpec = hasText ? hasIds(rankedIds) : (root, query, cb) -> null;
+        Specification<Document> spec = Specification.where(textSpec)
+                .and(isBetween(from, to))
+                .and(hasSender(sender))
+                .and(hasReceiver(receiver))
+                .and(hasTags(expandedTagSets, useOrLogic))
+                .and(hasTagPartial(tagQ))
+                .and(hasStatus(status));
 
         // SENDER, RECEIVER and RELEVANCE sorts load the full match set and slice in memory.
         // JPA's Sort.by("sender.lastName") generates an INNER JOIN that silently drops

backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java

@@ -929,315 +929,4 @@ class DocumentControllerTest {
                 .andExpect(status().isBadRequest())
                 .andExpect(jsonPath("$.code").value("BATCH_TOO_LARGE"));
     }
-
-    // ─── PATCH /api/documents/bulk ───────────────────────────────────────────
-
-    private static String bulkBody(String... uuids) {
-        StringBuilder sb = new StringBuilder("{\"documentIds\":[");
-        for (int i = 0; i < uuids.length; i++) {
-            if (i > 0) sb.append(",");
-            sb.append("\"").append(uuids[i]).append("\"");
-        }
-        sb.append("]}");
-        return sb.toString();
-    }
-
-    @Test
-    void patchBulk_returns401_whenUnauthenticated() throws Exception {
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content(bulkBody(UUID.randomUUID().toString())))
-                .andExpect(status().isUnauthorized());
-    }
-
-    @Test
-    @WithMockUser
-    void patchBulk_returns403_forReadAllUser() throws Exception {
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content(bulkBody(UUID.randomUUID().toString())))
-                .andExpect(status().isForbidden());
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void patchBulk_returns400_whenDocumentIdsIsEmpty() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("{\"documentIds\":[]}"))
-                .andExpect(status().isBadRequest());
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void patchBulk_returns400_whenDocumentIdsIsMissing() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("{}"))
-                .andExpect(status().isBadRequest());
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void patchBulk_returns400_whenDocumentIdsExceedsCap() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-
-        String[] ids = new String[501];
-        for (int i = 0; i < 501; i++) ids[i] = UUID.randomUUID().toString();
-
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content(bulkBody(ids)))
-                .andExpect(status().isBadRequest())
-                .andExpect(jsonPath("$.code").value("BULK_EDIT_TOO_MANY_IDS"));
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void patchBulk_returns400_whenArchiveBoxExceeds255Chars() throws Exception {
-        // Tobias C2 — DocumentBulkEditDTO.archiveBox carries @Size(max=255).
-        // Without @Valid on @RequestBody this would silently land an
-        // arbitrarily long string; the test pins both the annotation and
-        // the controller-level @Valid wiring.
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        UUID id = UUID.randomUUID();
-        String tooLong = "x".repeat(256);
-
-        String body = "{\"documentIds\":[\"" + id + "\"],\"archiveBox\":\"" + tooLong + "\"}";
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content(body))
-                .andExpect(status().isBadRequest());
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void patchBulk_acceptsExactly500Ids_atTheCap() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        when(documentService.applyBulkEditToDocument(any(), any(), any()))
-                .thenAnswer(inv -> Document.builder().id(inv.getArgument(0)).build());
-
-        String[] ids = new String[500];
-        for (int i = 0; i < 500; i++) ids[i] = UUID.randomUUID().toString();
-
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content(bulkBody(ids)))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$.updated").value(500));
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void patchBulk_dedupesDuplicateDocumentIds_doesNotInflateUpdatedCount() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        UUID id = UUID.randomUUID();
-        when(documentService.applyBulkEditToDocument(eq(id), any(), any()))
-                .thenAnswer(inv -> Document.builder().id(id).build());
-
-        // Same id sent three times — controller should dedupe and call the
-        // service exactly once, returning updated=1, not 3.
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content(bulkBody(id.toString(), id.toString(), id.toString())))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$.updated").value(1));
-
-        verify(documentService, org.mockito.Mockito.times(1))
-                .applyBulkEditToDocument(eq(id), any(), any());
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void patchBulk_returns200_andCallsServiceForEachId() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        UUID id1 = UUID.randomUUID();
-        UUID id2 = UUID.randomUUID();
-        when(documentService.applyBulkEditToDocument(any(), any(), any()))
-                .thenAnswer(inv -> Document.builder().id(inv.getArgument(0)).build());
-
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content(bulkBody(id1.toString(), id2.toString())))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$.updated").value(2))
-                .andExpect(jsonPath("$.errors").isEmpty());
-
-        verify(documentService).applyBulkEditToDocument(eq(id1), any(), any());
-        verify(documentService).applyBulkEditToDocument(eq(id2), any(), any());
-    }
-
-    // ─── GET /api/documents/ids ──────────────────────────────────────────────
-
-    @Test
-    void getDocumentIds_returns401_whenUnauthenticated() throws Exception {
-        mockMvc.perform(get("/api/documents/ids"))
-                .andExpect(status().isUnauthorized());
-    }
-
-    @Test
-    @WithMockUser
-    void getDocumentIds_returns403_forUserWithoutWriteAll() throws Exception {
-        // /ids is gated WRITE_ALL because it powers the bulk-edit "Alle X
-        // editieren" fast path; no other consumer needs it.
-        mockMvc.perform(get("/api/documents/ids"))
-                .andExpect(status().isForbidden());
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void getDocumentIds_returns200_andDelegatesToService() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        UUID id = UUID.randomUUID();
-        when(documentService.findIdsForFilter(any(), any(), any(), any(), any(), any(), any(), any(), any()))
-                .thenReturn(List.of(id));
-
-        mockMvc.perform(get("/api/documents/ids"))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$[0]").value(id.toString()));
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void getDocumentIds_passesSenderIdParamToService() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        UUID senderId = UUID.randomUUID();
-        when(documentService.findIdsForFilter(any(), any(), any(), eq(senderId), any(), any(), any(), any(), any()))
-                .thenReturn(List.of());
-
-        mockMvc.perform(get("/api/documents/ids").param("senderId", senderId.toString()))
-                .andExpect(status().isOk());
-
-        verify(documentService).findIdsForFilter(any(), any(), any(), eq(senderId), any(), any(), any(), any(), any());
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void getDocumentIds_returns400_whenResultExceedsFilterCap() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        // Service returns 5001 IDs — one over BULK_EDIT_FILTER_MAX_IDS (5000).
-        java.util.List<UUID> tooMany = new java.util.ArrayList<>(5001);
-        for (int i = 0; i < 5001; i++) tooMany.add(UUID.randomUUID());
-        when(documentService.findIdsForFilter(any(), any(), any(), any(), any(), any(), any(), any(), any()))
-                .thenReturn(tooMany);
-
-        mockMvc.perform(get("/api/documents/ids"))
-                .andExpect(status().isBadRequest())
-                .andExpect(jsonPath("$.code").value("BULK_EDIT_TOO_MANY_IDS"));
-    }
-
-    // ─── POST /api/documents/batch-metadata ──────────────────────────────────
-
-    @Test
-    void batchMetadata_returns401_whenUnauthenticated() throws Exception {
-        mockMvc.perform(org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post("/api/documents/batch-metadata")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("{\"ids\":[\"" + UUID.randomUUID() + "\"]}"))
-                .andExpect(status().isUnauthorized());
-    }
-
-    @Test
-    @WithMockUser
-    void batchMetadata_returns403_forUserWithoutReadAll() throws Exception {
-        mockMvc.perform(org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post("/api/documents/batch-metadata")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("{\"ids\":[\"" + UUID.randomUUID() + "\"]}"))
-                .andExpect(status().isForbidden());
-    }
-
-    @Test
-    @WithMockUser(authorities = "READ_ALL")
-    void batchMetadata_returns400_whenIdsEmpty() throws Exception {
-        mockMvc.perform(org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post("/api/documents/batch-metadata")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("{\"ids\":[]}"))
-                .andExpect(status().isBadRequest());
-    }
-
-    @Test
-    @WithMockUser(authorities = "READ_ALL")
-    void batchMetadata_returns400_whenIdsExceedsCap() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        StringBuilder sb = new StringBuilder("{\"ids\":[");
-        for (int i = 0; i < 501; i++) {
-            if (i > 0) sb.append(",");
-            sb.append("\"").append(UUID.randomUUID()).append("\"");
-        }
-        sb.append("]}");
-
-        mockMvc.perform(org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post("/api/documents/batch-metadata")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content(sb.toString()))
-                .andExpect(status().isBadRequest())
-                .andExpect(jsonPath("$.code").value("BULK_EDIT_TOO_MANY_IDS"));
-    }
-
-    @Test
-    @WithMockUser(authorities = "READ_ALL")
-    void batchMetadata_returnsSummaries_forExistingIds() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        UUID id = UUID.randomUUID();
-        when(documentService.batchMetadata(any())).thenReturn(List.of(
-                new org.raddatz.familienarchiv.dto.DocumentBatchSummary(id, "Brief", "/api/documents/" + id + "/file")));
-
-        mockMvc.perform(org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post("/api/documents/batch-metadata")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("{\"ids\":[\"" + id + "\"]}"))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$[0].id").value(id.toString()))
-                .andExpect(jsonPath("$[0].title").value("Brief"))
-                .andExpect(jsonPath("$[0].pdfUrl").value("/api/documents/" + id + "/file"));
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void patchBulk_stripsCarriageReturnsAndNewlinesFromErrorMessages() throws Exception {
-        // Nora C4 — DocumentController.sanitizeForLog defends against
-        // CWE-117 (log injection) by replacing CR/LF in any free-form string
-        // it interpolates. Same helper now sanitises BulkEditError.message
-        // before it round-trips to the frontend.
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        UUID badId = UUID.randomUUID();
-        when(documentService.applyBulkEditToDocument(eq(badId), any(), any()))
-                .thenThrow(org.raddatz.familienarchiv.exception.DomainException.notFound(
-                        org.raddatz.familienarchiv.exception.ErrorCode.DOCUMENT_NOT_FOUND,
-                        "evil\r\nFAKE LOG ENTRY: admin logged in"));
-
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content(bulkBody(badId.toString())))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$.errors[0].message",
-                        org.hamcrest.Matchers.not(org.hamcrest.Matchers.containsString("\n"))))
-                .andExpect(jsonPath("$.errors[0].message",
-                        org.hamcrest.Matchers.not(org.hamcrest.Matchers.containsString("\r"))))
-                .andExpect(jsonPath("$.errors[0].message",
-                        org.hamcrest.Matchers.containsString("evil_")));
-    }
-
-    @Test
-    @WithMockUser(authorities = "WRITE_ALL")
-    void patchBulk_returnsPartialFailureShape_whenServiceThrowsForOneDocument() throws Exception {
-        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
-        UUID okId = UUID.randomUUID();
-        UUID badId = UUID.randomUUID();
-        when(documentService.applyBulkEditToDocument(eq(okId), any(), any()))
-                .thenAnswer(inv -> Document.builder().id(okId).build());
-        when(documentService.applyBulkEditToDocument(eq(badId), any(), any()))
-                .thenThrow(org.raddatz.familienarchiv.exception.DomainException.notFound(
-                        org.raddatz.familienarchiv.exception.ErrorCode.DOCUMENT_NOT_FOUND, "Document not found: " + badId));
-
-        mockMvc.perform(patch("/api/documents/bulk")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content(bulkBody(okId.toString(), badId.toString())))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$.updated").value(1))
-                .andExpect(jsonPath("$.errors[0].id").value(badId.toString()))
-                .andExpect(jsonPath("$.errors[0].message").value(
-                        org.hamcrest.Matchers.containsString("not found")));
-    }
 }

backend/src/test/java/org/raddatz/familienarchiv/service/DocumentServiceTest.java

@@ -16,7 +16,6 @@ import org.raddatz.familienarchiv.dto.DocumentUpdateDTO;
 import org.raddatz.familienarchiv.dto.IncompleteDocumentDTO;
 import org.raddatz.familienarchiv.dto.MatchOffset;
 import org.raddatz.familienarchiv.dto.SearchMatchData;
-import org.raddatz.familienarchiv.dto.TagOperator;
 import org.raddatz.familienarchiv.exception.DomainException;
 import org.raddatz.familienarchiv.model.Document;
 import org.raddatz.familienarchiv.model.DocumentStatus;
@@ -1918,333 +1917,4 @@ class DocumentServiceTest {
                 .isInstanceOf(DomainException.class)
                 .hasMessageContaining("titles");
     }
-
-    // ─── applyBulkEditToDocument ─────────────────────────────────────────────
-
-    private static org.raddatz.familienarchiv.dto.DocumentBulkEditDTO bulkDto() {
-        return new org.raddatz.familienarchiv.dto.DocumentBulkEditDTO();
-    }
-
-    @Test
-    void applyBulkEditToDocument_throwsNotFound_whenDocumentMissing() {
-        UUID id = UUID.randomUUID();
-        when(documentRepository.findById(id)).thenReturn(Optional.empty());
-
-        assertThatThrownBy(() -> documentService.applyBulkEditToDocument(id, bulkDto(), null))
-                .isInstanceOf(DomainException.class)
-                .hasMessageContaining(id.toString());
-    }
-
-    @Test
-    void applyBulkEditToDocument_appliesTagsAdditively_preservesExistingTags() {
-        UUID id = UUID.randomUUID();
-        Tag existing = Tag.builder().id(UUID.randomUUID()).name("Brief").build();
-        Tag added = Tag.builder().id(UUID.randomUUID()).name("Kurrent").build();
-        Document doc = Document.builder().id(id).title("T")
-                .tags(new HashSet<>(Set.of(existing)))
-                .receivers(new HashSet<>())
-                .build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
-        when(tagService.findOrCreate("Kurrent")).thenReturn(added);
-
-        var dto = bulkDto();
-        dto.setTagNames(List.of("Kurrent"));
-        documentService.applyBulkEditToDocument(id, dto, null);
-
-        assertThat(doc.getTags()).containsExactlyInAnyOrder(existing, added);
-    }
-
-    @Test
-    void applyBulkEditToDocument_skipsTags_whenTagNamesIsNull() {
-        UUID id = UUID.randomUUID();
-        Tag existing = Tag.builder().id(UUID.randomUUID()).name("Brief").build();
-        Document doc = Document.builder().id(id).title("T")
-                .tags(new HashSet<>(Set.of(existing)))
-                .receivers(new HashSet<>())
-                .build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
-
-        documentService.applyBulkEditToDocument(id, bulkDto(), null);
-
-        assertThat(doc.getTags()).containsExactly(existing);
-        verify(tagService, never()).findOrCreate(any());
-    }
-
-    @Test
-    void applyBulkEditToDocument_skipsTags_whenTagNamesIsEmpty() {
-        UUID id = UUID.randomUUID();
-        Tag existing = Tag.builder().id(UUID.randomUUID()).name("Brief").build();
-        Document doc = Document.builder().id(id).title("T")
-                .tags(new HashSet<>(Set.of(existing)))
-                .receivers(new HashSet<>())
-                .build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
-
-        var dto = bulkDto();
-        dto.setTagNames(List.of());
-        documentService.applyBulkEditToDocument(id, dto, null);
-
-        assertThat(doc.getTags()).containsExactly(existing);
-        verify(tagService, never()).findOrCreate(any());
-    }
-
-    @Test
-    void applyBulkEditToDocument_replacesSender_whenSenderIdProvided() {
-        UUID id = UUID.randomUUID();
-        UUID senderId = UUID.randomUUID();
-        Person oldSender = Person.builder().id(UUID.randomUUID()).firstName("Old").build();
-        Person newSender = Person.builder().id(senderId).firstName("New").build();
-        Document doc = Document.builder().id(id).title("T")
-                .sender(oldSender)
-                .receivers(new HashSet<>())
-                .build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
-        when(personService.getById(senderId)).thenReturn(newSender);
-
-        var dto = bulkDto();
-        dto.setSenderId(senderId);
-        documentService.applyBulkEditToDocument(id, dto, null);
-
-        assertThat(doc.getSender()).isEqualTo(newSender);
-    }
-
-    @Test
-    void applyBulkEditToDocument_skipsSender_whenSenderIdIsNull() {
-        UUID id = UUID.randomUUID();
-        Person existing = Person.builder().id(UUID.randomUUID()).firstName("X").build();
-        Document doc = Document.builder().id(id).title("T")
-                .sender(existing)
-                .receivers(new HashSet<>())
-                .build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
-
-        documentService.applyBulkEditToDocument(id, bulkDto(), null);
-
-        assertThat(doc.getSender()).isEqualTo(existing);
-        verify(personService, never()).getById(any());
-    }
-
-    @Test
-    void applyBulkEditToDocument_addsReceiversAdditively_preservesExistingReceivers() {
-        UUID id = UUID.randomUUID();
-        UUID newReceiverId = UUID.randomUUID();
-        Person existing = Person.builder().id(UUID.randomUUID()).firstName("Old").build();
-        Person added = Person.builder().id(newReceiverId).firstName("New").build();
-        Document doc = Document.builder().id(id).title("T")
-                .receivers(new HashSet<>(Set.of(existing)))
-                .build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
-        when(personService.getAllById(List.of(newReceiverId))).thenReturn(List.of(added));
-
-        var dto = bulkDto();
-        dto.setReceiverIds(List.of(newReceiverId));
-        documentService.applyBulkEditToDocument(id, dto, null);
-
-        assertThat(doc.getReceivers()).containsExactlyInAnyOrder(existing, added);
-    }
-
-    @Test
-    void applyBulkEditToDocument_skipsReceivers_whenReceiverIdsIsNullOrEmpty() {
-        UUID id = UUID.randomUUID();
-        Person existing = Person.builder().id(UUID.randomUUID()).firstName("Old").build();
-        Document doc = Document.builder().id(id).title("T")
-                .receivers(new HashSet<>(Set.of(existing)))
-                .build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
-
-        var dto = bulkDto();
-        dto.setReceiverIds(List.of());
-        documentService.applyBulkEditToDocument(id, dto, null);
-
-        assertThat(doc.getReceivers()).containsExactly(existing);
-        verify(personService, never()).getAllById(any());
-    }
-
-    @Test
-    void applyBulkEditToDocument_recordsVersion_andLogsAuditEvent_taggedSourceBulkEdit() {
-        UUID id = UUID.randomUUID();
-        UUID actorId = UUID.randomUUID();
-        Document doc = Document.builder().id(id).title("T").receivers(new HashSet<>()).build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(documentRepository.save(any())).thenReturn(doc);
-
-        documentService.applyBulkEditToDocument(id, bulkDto(), actorId);
-
-        verify(documentVersionService).recordVersion(doc);
-        verify(auditService).logAfterCommit(
-                eq(AuditKind.METADATA_UPDATED),
-                eq(actorId),
-                eq(id),
-                eq(java.util.Map.of("source", "BULK_EDIT")));
-    }
-
-    @Test
-    void applyBulkEditToDocument_replacesArchiveBoxAndFolderAndDocumentLocation_whenProvided() {
-        UUID id = UUID.randomUUID();
-        Document doc = Document.builder().id(id).title("T")
-                .archiveBox("OldBox")
-                .archiveFolder("OldFolder")
-                .documentLocation("OldLocation")
-                .receivers(new HashSet<>())
-                .build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
-
-        var dto = bulkDto();
-        dto.setArchiveBox("NewBox");
-        dto.setArchiveFolder("NewFolder");
-        dto.setDocumentLocation("NewLocation");
-        documentService.applyBulkEditToDocument(id, dto, null);
-
-        assertThat(doc.getArchiveBox()).isEqualTo("NewBox");
-        assertThat(doc.getArchiveFolder()).isEqualTo("NewFolder");
-        assertThat(doc.getDocumentLocation()).isEqualTo("NewLocation");
-    }
-
-    @Test
-    void applyBulkEditToDocument_propagatesDomainException_whenSenderIdUnresolvable() {
-        // Sara C1 — unresolvable sender flows up as a per-document error chip
-        // rather than aborting the controller's batch loop.
-        UUID id = UUID.randomUUID();
-        UUID unknownSender = UUID.randomUUID();
-        Document doc = Document.builder().id(id).title("T").receivers(new HashSet<>()).build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(personService.getById(unknownSender))
-                .thenThrow(DomainException.notFound(
-                        org.raddatz.familienarchiv.exception.ErrorCode.PERSON_NOT_FOUND,
-                        "Person not found: " + unknownSender));
-
-        var dto = bulkDto();
-        dto.setSenderId(unknownSender);
-
-        assertThatThrownBy(() -> documentService.applyBulkEditToDocument(id, dto, null))
-                .isInstanceOf(DomainException.class)
-                .hasMessageContaining(unknownSender.toString());
-    }
-
-    // ─── findIdsForFilter ────────────────────────────────────────────────────
-
-    @Test
-    void findIdsForFilter_returnsAllMatchingIds_uncapped() {
-        Document d1 = Document.builder().id(UUID.randomUUID()).title("A").build();
-        Document d2 = Document.builder().id(UUID.randomUUID()).title("B").build();
-        when(documentRepository.findAll(any(org.springframework.data.jpa.domain.Specification.class)))
-                .thenReturn(List.of(d1, d2));
-
-        List<UUID> result = documentService.findIdsForFilter(
-                null, null, null, null, null, null, null, null, null);
-
-        assertThat(result).containsExactly(d1.getId(), d2.getId());
-    }
-
-    @Test
-    void findIdsForFilter_passesTagOperatorOR_throughBuildSearchSpec() {
-        // Sara C3 — tagOp=OR flips useOrLogic at the spec layer; without a
-        // test pinning this, a refactor that wired OR to AND (or vice versa)
-        // would slip through.
-        when(documentRepository.findAll(any(org.springframework.data.jpa.domain.Specification.class)))
-                .thenReturn(List.of());
-        when(tagService.expandTagNamesToDescendantIdSets(any())).thenReturn(List.of());
-
-        documentService.findIdsForFilter(
-                null, null, null, null, null, List.of("Brief"), null, null, TagOperator.OR);
-
-        // Spec built without throwing → OR branch was exercised. Coverage gain
-        // is in not-throwing on the OR-specific code path; the actual SQL is
-        // covered by JPA itself.
-        verify(documentRepository).findAll(any(org.springframework.data.jpa.domain.Specification.class));
-    }
-
-    @Test
-    void findIdsForFilter_returnsEmpty_whenFtsHasNoMatches() {
-        when(documentRepository.findRankedIdsByFts("xyz")).thenReturn(List.of());
-
-        List<UUID> result = documentService.findIdsForFilter(
-                "xyz", null, null, null, null, null, null, null, null);
-
-        assertThat(result).isEmpty();
-        verify(documentRepository, never()).findAll(any(org.springframework.data.jpa.domain.Specification.class));
-    }
-
-    // ─── batchMetadata ───────────────────────────────────────────────────────
-
-    @Test
-    void batchMetadata_returnsEmpty_whenIdsIsNull() {
-        assertThat(documentService.batchMetadata(null)).isEmpty();
-    }
-
-    @Test
-    void batchMetadata_returnsEmpty_whenIdsIsEmpty() {
-        assertThat(documentService.batchMetadata(List.of())).isEmpty();
-    }
-
-    @Test
-    void batchMetadata_returnsSummariesWithPdfUrl_forExistingIds() {
-        UUID id1 = UUID.randomUUID();
-        UUID id2 = UUID.randomUUID();
-        Document d1 = Document.builder().id(id1).title("Brief 1").build();
-        Document d2 = Document.builder().id(id2).title("Brief 2").build();
-        when(documentRepository.findAllById(List.of(id1, id2))).thenReturn(List.of(d1, d2));
-
-        var result = documentService.batchMetadata(List.of(id1, id2));
-
-        assertThat(result).hasSize(2);
-        assertThat(result.get(0).id()).isEqualTo(id1);
-        assertThat(result.get(0).title()).isEqualTo("Brief 1");
-        assertThat(result.get(0).pdfUrl()).isEqualTo("/api/documents/" + id1 + "/file");
-    }
-
-    @Test
-    void batchMetadata_silentlyDropsUnknownIds() {
-        UUID known = UUID.randomUUID();
-        UUID missing = UUID.randomUUID();
-        Document d = Document.builder().id(known).title("Found").build();
-        when(documentRepository.findAllById(List.of(known, missing))).thenReturn(List.of(d));
-
-        var result = documentService.batchMetadata(List.of(known, missing));
-
-        assertThat(result).hasSize(1);
-        assertThat(result.get(0).id()).isEqualTo(known);
-    }
-
-    @Test
-    void batchMetadata_fallsBackToOriginalFilename_whenTitleIsNull() {
-        UUID id = UUID.randomUUID();
-        Document d = Document.builder().id(id).originalFilename("scan001.pdf").build();
-        when(documentRepository.findAllById(List.of(id))).thenReturn(List.of(d));
-
-        var result = documentService.batchMetadata(List.of(id));
-
-        assertThat(result.get(0).title()).isEqualTo("scan001.pdf");
-    }
-
-    @Test
-    void applyBulkEditToDocument_skipsLocationFields_whenBlankOrNull() {
-        UUID id = UUID.randomUUID();
-        Document doc = Document.builder().id(id).title("T")
-                .archiveBox("KeepBox")
-                .archiveFolder("KeepFolder")
-                .documentLocation("KeepLocation")
-                .receivers(new HashSet<>())
-                .build();
-        when(documentRepository.findById(id)).thenReturn(Optional.of(doc));
-        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
-
-        var dto = bulkDto();
-        dto.setArchiveBox("   ");
-        dto.setArchiveFolder("");
-        // documentLocation left null
-        documentService.applyBulkEditToDocument(id, dto, null);
-
-        assertThat(doc.getArchiveBox()).isEqualTo("KeepBox");
-        assertThat(doc.getArchiveFolder()).isEqualTo("KeepFolder");
-        assertThat(doc.getDocumentLocation()).isEqualTo("KeepLocation");
-    }
 }

frontend/e2e/bulk-edit.spec.ts

@@ -1,75 +0,0 @@
-import { test, expect } from '@playwright/test';
-
-/**
- * E2E coverage for the bulk metadata edit feature (issue #225).
- *
- * Assumptions:
- *  - Auth setup has run as the admin user (WRITE_ALL).
- *  - The backend exposes /api/documents/{bulk,batch-metadata,ids}.
- *  - At least two documents exist in the search list at /documents.
- */
-
-test.describe('Bulk metadata edit', () => {
-	test.beforeEach(async ({ page }) => {
-		await page.goto('/documents');
-		await page.waitForSelector('[data-hydrated]');
-	});
-
-	test('checking two documents shows the sticky selection bar with the count', async ({ page }) => {
-		const checkboxes = page.locator('[data-testid="bulk-select-checkbox"] input[type="checkbox"]');
-		await expect(checkboxes.first()).toBeVisible();
-		await checkboxes.nth(0).check();
-		await checkboxes.nth(1).check();
-
-		const bar = page.getByTestId('bulk-selection-bar');
-		await expect(bar).toBeVisible();
-		await expect(page.getByTestId('bulk-selection-count')).toContainText('2');
-	});
-
-	test('Alles aufheben hides the bar', async ({ page }) => {
-		const checkboxes = page.locator('[data-testid="bulk-select-checkbox"] input[type="checkbox"]');
-		await checkboxes.nth(0).check();
-		await expect(page.getByTestId('bulk-selection-bar')).toBeVisible();
-
-		await page.getByTestId('bulk-clear-all').click();
-		await expect(page.getByTestId('bulk-selection-bar')).not.toBeVisible();
-	});
-
-	test('Massenbearbeitung navigates to bulk-edit with the selected documents', async ({ page }) => {
-		const checkboxes = page.locator('[data-testid="bulk-select-checkbox"] input[type="checkbox"]');
-		await checkboxes.nth(0).check();
-		await checkboxes.nth(1).check();
-		await page.getByTestId('bulk-edit-open').click();
-
-		await page.waitForURL('**/documents/bulk-edit');
-		// Onboarding callout is the surest indicator the edit-mode layout rendered.
-		await expect(page.getByTestId('bulk-edit-callout')).toBeVisible();
-	});
-
-	test('navigating to /documents/bulk-edit with no selection redirects back to /documents', async ({
-		page
-	}) => {
-		// Navigate directly without checking anything first.
-		await page.goto('/documents/bulk-edit');
-		await page.waitForURL('**/documents');
-		expect(page.url()).toMatch(/\/documents(\?|$)/);
-	});
-
-	test('the same selection bar drives the /enrich page', async ({ page }) => {
-		await page.goto('/enrich');
-		await page.waitForSelector('[data-hydrated]');
-
-		// /enrich may legitimately be empty if every doc has metadata. In that
-		// case there's nothing to bulk-select; skip.
-		const checkboxes = page.locator('[data-testid="bulk-select-checkbox"] input[type="checkbox"]');
-		const count = await checkboxes.count();
-		test.skip(count === 0, 'No incomplete documents available on /enrich');
-
-		await checkboxes.first().check();
-		await expect(page.getByTestId('bulk-selection-bar')).toBeVisible();
-		await expect(page.getByTestId('bulk-selection-count')).toContainText('1');
-
-		await page.getByTestId('bulk-clear-all').click();
-		await expect(page.getByTestId('bulk-selection-bar')).not.toBeVisible();
-	});
-});

frontend/e2e/help-popover.spec.ts

@@ -8,27 +8,21 @@ test.describe('Help chip — Read/Edit panel header', () => {
 		docId = await createEmptyDocument(request);
 	});
 
-	test.afterAll(async ({ request }) => {
-		await request.delete(`/api/documents/${docId}`);
-	});
-
 	test('opens popover on click, closes on Esc, returns focus to chip', async ({ page }) => {
 		await page.goto(`/documents/${docId}`);
 		await page.getByRole('button', { name: 'Transkribieren' }).click();
 
-		// Use the accessible label of the HelpPopover trigger (transcription_mode_help_label)
-		const helpBtn = page.getByRole('button', { name: 'Lese- und Bearbeitungsmodus' });
+		// Find and click the (?) help chip
+		const helpBtn = page.locator('button[aria-expanded]');
 		await expect(helpBtn).toBeVisible({ timeout: 5000 });
 		await helpBtn.click();
 
-		// Popover should open (role="region", not tooltip — click-triggered panels are regions)
-		await expect(page.getByRole('region', { name: 'Lese- und Bearbeitungsmodus' })).toBeVisible();
+		// Popover should open
+		await expect(page.locator('[role="tooltip"]')).toBeVisible();
 
 		// Press Esc
 		await page.keyboard.press('Escape');
-		await expect(
-			page.getByRole('region', { name: 'Lese- und Bearbeitungsmodus' })
-		).not.toBeVisible();
+		await expect(page.locator('[role="tooltip"]')).not.toBeVisible();
 
 		// Focus should have returned to the chip
 		await expect(helpBtn).toBeFocused();

frontend/e2e/helpers/upload-empty-document.ts

@@ -1,30 +1,10 @@
 import type { APIRequestContext } from '@playwright/test';
-import path from 'path';
-import { fileURLToPath } from 'url';
-import fs from 'fs';
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const PDF_FIXTURE = path.resolve(__dirname, '../fixtures/minimal.pdf');
 
 export async function createEmptyDocument(request: APIRequestContext): Promise<string> {
-	const createRes = await request.post('/api/documents', {
+	const res = await request.post('/api/documents', {
 		multipart: { title: 'E2E Transcribe Coach Test' }
 	});
-	if (!createRes.ok()) throw new Error(`Create document failed: ${createRes.status()}`);
-	const doc = await createRes.json();
-	const docId = doc.id as string;
-
-	const uploadRes = await request.put(`/api/documents/${docId}`, {
-		multipart: {
-			title: doc.title,
-			file: {
-				name: 'minimal.pdf',
-				mimeType: 'application/pdf',
-				buffer: fs.readFileSync(PDF_FIXTURE)
-			}
-		}
-	});
-	if (!uploadRes.ok()) throw new Error(`Upload PDF failed: ${uploadRes.status()}`);
-
-	return docId;
+	if (!res.ok()) throw new Error(`Create document failed: ${res.status()}`);
+	const doc = await res.json();
+	return doc.id as string;
 }

frontend/e2e/richtlinien.spec.ts

@@ -63,12 +63,6 @@ test.describe('Richtlinien page — print media', () => {
 			await expect(nav).toBeHidden();
 		}
 
-		// .new-tab annotation spans must be hidden in print so "(öffnet in neuem Tab)"
-		// text does not clutter the printed output (the print CSS declares display:none)
-		for (const span of await page.locator('.new-tab').all()) {
-			await expect(span).toBeHidden();
-		}
-
 		await page.screenshot({ path: 'test-results/e2e/richtlinien-print.png', fullPage: true });
 	});
 });

frontend/e2e/transcribe-coach.spec.ts

@@ -2,24 +2,6 @@ import { test, expect } from '@playwright/test';
 import AxeBuilder from '@axe-core/playwright';
 import { createEmptyDocument } from './helpers/upload-empty-document.js';
 
-async function createBlock(
-	request: Parameters<typeof createEmptyDocument>[0],
-	docId: string
-): Promise<void> {
-	const res = await request.post(`/api/documents/${docId}/transcription-blocks`, {
-		data: {
-			pageNumber: 1,
-			x: 0.1,
-			y: 0.1,
-			width: 0.3,
-			height: 0.1,
-			text: 'Liebe Mutter,',
-			label: null
-		}
-	});
-	if (!res.ok()) throw new Error(`Create block failed: ${res.status()}`);
-}
-
 function buildAxe(page: Parameters<typeof AxeBuilder>[0]['page']) {
 	return new AxeBuilder({ page }).withTags(['wcag2a', 'wcag2aa']);
 }
@@ -31,13 +13,10 @@ test.describe('Transcribe coach — empty state', () => {
 		docId = await createEmptyDocument(request);
 	});
 
-	test.afterAll(async ({ request }) => {
-		await request.delete(`/api/documents/${docId}`);
-	});
-
 	test('shows coach card (title, preamble, three step bodies, animation region)', async ({
 		page
 	}) => {
+		await page.emulateMedia({ reducedMotion: 'reduce' });
 		await page.goto(`/documents/${docId}`);
 		await page.getByRole('button', { name: 'Transkribieren' }).click();
 
@@ -52,12 +31,14 @@ test.describe('Transcribe coach — empty state', () => {
 	});
 
 	test('training footer is NOT visible on empty doc', async ({ page }) => {
+		await page.emulateMedia({ reducedMotion: 'reduce' });
 		await page.goto(`/documents/${docId}`);
 		await page.getByRole('button', { name: 'Transkribieren' }).click();
 		await expect(page.getByText('Für Training vormerken')).not.toBeVisible({ timeout: 3000 });
 	});
 
 	test('axe: panel empty state — light theme — no WCAG 2.1 AA violations', async ({ page }) => {
+		await page.emulateMedia({ reducedMotion: 'reduce' });
 		await page.goto(`/documents/${docId}`);
 		await page.getByRole('button', { name: 'Transkribieren' }).click();
 		await expect(page.getByRole('heading', { level: 2, name: /Erste Transkription/ })).toBeVisible({
@@ -69,9 +50,10 @@ test.describe('Transcribe coach — empty state', () => {
 	});
 
 	test('axe: panel empty state — dark theme — no WCAG 2.1 AA violations', async ({ page }) => {
+		await page.emulateMedia({ reducedMotion: 'reduce' });
 		await page.goto(`/documents/${docId}`);
 		// Toggle dark theme
-		await page.getByRole('button', { name: /dark mode/i }).click();
+		await page.getByRole('button', { name: /Farbmodus|theme/i }).click();
 		await page.getByRole('button', { name: 'Transkribieren' }).click();
 		await expect(page.getByRole('heading', { level: 2, name: /Erste Transkription/ })).toBeVisible({
 			timeout: 5000
@@ -81,25 +63,3 @@ test.describe('Transcribe coach — empty state', () => {
 		expect(a11y.violations, JSON.stringify(a11y.violations, null, 2)).toHaveLength(0);
 	});
 });
-
-test.describe('Transcribe coach — with blocks', () => {
-	let docId: string;
-
-	test.beforeAll(async ({ request }) => {
-		docId = await createEmptyDocument(request);
-		await createBlock(request, docId);
-	});
-
-	test.afterAll(async ({ request }) => {
-		await request.delete(`/api/documents/${docId}`);
-	});
-
-	test('training footer IS visible when at least one block exists', async ({ page }) => {
-		await page.goto(`/documents/${docId}`);
-		await page.getByRole('button', { name: 'Transkribieren' }).click();
-		// Wait for blocks to finish loading — block count confirms mode settled to 'read'
-		await expect(page.getByText(/1 Abschnitt/)).toBeVisible({ timeout: 5000 });
-		await page.locator('[data-testid="mode-edit"]').click();
-		await expect(page.getByText('Für Training vormerken')).toBeVisible({ timeout: 5000 });
-	});
-});

frontend/messages/de.json

@@ -515,6 +515,7 @@
 	"scan_collapse": "Scan verkleinern",
 	"transcription_empty_title": "Noch keine Transkription",
 	"transcription_empty_desc": "Zeichne Bereiche auf dem Scan und tippe den Text ab, um eine Transkription zu erstellen.",
+	"transcription_empty_draw_hint": "Zeichnen Sie Bereiche auf dem Dokument, um mit der Transkription zu beginnen.",
 	"transcription_panel_close": "Panel schließen",
 	"person_alias_heading": "Namensverlauf",
 	"person_alias_empty": "Noch keine Namensaenderungen erfasst.",
@@ -827,9 +828,9 @@
 	"transcription_mode_help_body": "Lesen zeigt die Transkription als fließenden Text. Bearbeiten öffnet die Textfelder für jede Passage.",
 
 	"richtlinien_title": "Transkriptions-Richtlinien",
-	"richtlinien_intro": "Damit alle Briefe einheitlich transkribiert werden — egal wer tippt — hier unsere Regeln. Die Seite wächst mit: sobald wir eine neue Konvention beschließen, landet sie hier.",
-	"richtlinien_wiki_text": "Kurrent- und Sütterlin-Alphabete sind bei Wikipedia gut erklärt. Hier stehen nur unsere eigenen Vereinbarungen für dieses Archiv.",
-	"richtlinien_wiki_link": "Wikipedia",
+	"richtlinien_intro": "Damit alle Briefe einheitlich transkribiert werden — egal ob Tante Hedwig oder Cousin Paul tippt — hier unsere Regeln. Die Seite wächst mit: sobald wir eine neue Konvention beschließen, landet sie hier.",
+	"richtlinien_wiki_text": "Das vollständige Kurrent- und Sütterlin-Alphabet brauchen Sie für diese Seite nicht — das erledigt Wikipedia. Hier sind unsere eigenen Regeln für das, was Wikipedia nicht beantwortet.",
+	"richtlinien_wiki_link": "Wikipedia →",
 	"richtlinien_rules_label": "Regeln für die Transkription",
 	"richtlinien_rule_unleserlich_title": "Nicht lesbare Wörter",
 	"richtlinien_rule_unleserlich_body": "Wenn Sie ein Wort beim besten Willen nicht entziffern können, schreiben Sie [unleserlich]. Jemand anderes schaut später nochmal drauf.",
@@ -873,30 +874,5 @@
 	"bulk_drop_zone_label": "Dateien ablegen",
 	"bulk_remove_file": "Entfernen",
 	"bulk_title_single": "Neues Dokument",
-	"bulk_title_multi": "Neue Dokumente",
-	"bulk_edit_button": "Massenbearbeitung",
-	"bulk_edit_n_selected_one": "1 Dokument ausgewählt",
-	"bulk_edit_n_selected_other": "{count} Dokumente ausgewählt",
-	"bulk_edit_clear_all": "Alles aufheben",
-	"bulk_edit_all_x": "Alle {count} editieren",
-	"bulk_edit_select_document": "Dokument {title} auswählen",
-	"bulk_edit_hint": "Nur ausgefüllte Felder werden angewendet. Tags und Empfänger werden hinzugefügt, nicht ersetzt.",
-	"bulk_edit_badge_additive": "+ wird hinzugefügt",
-	"bulk_edit_badge_replace": "wird ersetzt",
-	"bulk_edit_save_progress": "Batch {done} von {total} verarbeitet",
-	"bulk_edit_save_partial": "{done} von {total} gespeichert",
-	"bulk_edit_retry": "Erneut versuchen",
-	"bulk_edit_title": "Massenbearbeitung",
-	"bulk_edit_save_button": "Anwenden",
-	"error_bulk_edit_too_many_ids": "Maximal 500 Dokumente pro Anfrage.",
-	"form_label_archive_box": "Karton",
-	"form_helper_archive_box": "Welcher Karton im Archiv?",
-	"form_label_archive_folder": "Mappe",
-	"form_helper_archive_folder": "Welche Mappe innerhalb des Kartons?",
-	"bulk_edit_clear_selection": "Auswahl aufheben",
-	"bulk_edit_clear_hint_keyboard": "Esc: Auswahl aufheben",
-	"bulk_edit_loading": "Dokumente werden geladen…",
-	"bulk_edit_all_x_failed": "Filter konnte nicht abgerufen werden — bitte erneut versuchen.",
-	"bulk_edit_topbar_title": "Massenbearbeitung",
-	"bulk_edit_count_pill": "{count} werden bearbeitet"
+	"bulk_title_multi": "Neue Dokumente"
 }

frontend/messages/en.json

@@ -515,6 +515,7 @@
 	"scan_collapse": "Collapse scan",
 	"transcription_empty_title": "No transcription yet",
 	"transcription_empty_desc": "Draw regions on the scan and type the text to create a transcription.",
+	"transcription_empty_draw_hint": "Draw regions on the document to start transcribing.",
 	"transcription_panel_close": "Close panel",
 	"person_alias_heading": "Name history",
 	"person_alias_empty": "No name changes recorded yet.",
@@ -827,9 +828,9 @@
 	"transcription_mode_help_body": "Read shows the transcription as flowing text. Edit opens the text fields for each passage.",
 
 	"richtlinien_title": "Transcription Guidelines",
-	"richtlinien_intro": "So every letter is transcribed consistently — no matter who types — here are our rules. The page grows with us: as soon as we agree a new convention, it lands here.",
-	"richtlinien_wiki_text": "The Kurrent and Sütterlin alphabets are well explained on Wikipedia. Here you'll only find our own conventions for this archive.",
-	"richtlinien_wiki_link": "Wikipedia",
+	"richtlinien_intro": "So every letter is transcribed consistently — whether Tante Hedwig or Cousin Paul is typing — here are our rules. The page grows with us: as soon as we agree a new convention, it lands here.",
+	"richtlinien_wiki_text": "You don't need the full Kurrent and Sütterlin alphabet on this page — that's what Wikipedia is for. Here are our own rules for everything Wikipedia can't answer.",
+	"richtlinien_wiki_link": "Wikipedia →",
 	"richtlinien_rules_label": "Transcription rules",
 	"richtlinien_rule_unleserlich_title": "Illegible words",
 	"richtlinien_rule_unleserlich_body": "If you can't decipher a word even after trying, write [unleserlich]. Someone else will take another look later.",
@@ -873,30 +874,5 @@
 	"bulk_drop_zone_label": "Drop files here",
 	"bulk_remove_file": "Remove",
 	"bulk_title_single": "New Document",
-	"bulk_title_multi": "New Documents",
-	"bulk_edit_button": "Bulk edit",
-	"bulk_edit_n_selected_one": "1 document selected",
-	"bulk_edit_n_selected_other": "{count} documents selected",
-	"bulk_edit_clear_all": "Clear all",
-	"bulk_edit_all_x": "Edit all {count}",
-	"bulk_edit_select_document": "Select document {title}",
-	"bulk_edit_hint": "Only filled fields are applied. Tags and receivers are added, not replaced.",
-	"bulk_edit_badge_additive": "+ will be added",
-	"bulk_edit_badge_replace": "will replace",
-	"bulk_edit_save_progress": "Batch {done} of {total} processed",
-	"bulk_edit_save_partial": "{done} of {total} saved",
-	"bulk_edit_retry": "Retry",
-	"bulk_edit_title": "Bulk edit",
-	"bulk_edit_save_button": "Apply",
-	"error_bulk_edit_too_many_ids": "Maximum 500 documents per request.",
-	"form_label_archive_box": "Box",
-	"form_helper_archive_box": "Which box in the archive?",
-	"form_label_archive_folder": "Folder",
-	"form_helper_archive_folder": "Which folder inside the box?",
-	"bulk_edit_clear_selection": "Clear selection",
-	"bulk_edit_clear_hint_keyboard": "Esc: clear selection",
-	"bulk_edit_loading": "Loading documents…",
-	"bulk_edit_all_x_failed": "Could not load filter results — please retry.",
-	"bulk_edit_topbar_title": "Bulk edit",
-	"bulk_edit_count_pill": "{count} will be edited"
+	"bulk_title_multi": "New Documents"
 }

frontend/messages/es.json

@@ -515,6 +515,7 @@
 	"scan_collapse": "Reducir escaneo",
 	"transcription_empty_title": "Sin transcripcion",
 	"transcription_empty_desc": "Dibuja regiones en el escaneo y escribe el texto para crear una transcripcion.",
+	"transcription_empty_draw_hint": "Dibuje regiones en el documento para comenzar a transcribir.",
 	"transcription_panel_close": "Cerrar panel",
 	"person_alias_heading": "Historial de nombres",
 	"person_alias_empty": "Aun no se han registrado cambios de nombre.",
@@ -827,9 +828,9 @@
 	"transcription_mode_help_body": "Lectura muestra la transcripción como texto continuo. Edición abre los campos de texto para cada pasaje.",
 
 	"richtlinien_title": "Normas de transcripción",
-	"richtlinien_intro": "Para que todas las cartas se transcriban de forma uniforme — sin importar quién transcriba — aquí están nuestras reglas. La página crece con nosotros.",
-	"richtlinien_wiki_text": "Los alfabetos Kurrent y Sütterlin están bien explicados en Wikipedia. Aquí solo se recogen nuestros propios acuerdos para este archivo.",
-	"richtlinien_wiki_link": "Wikipedia",
+	"richtlinien_intro": "Para que todas las cartas se transcriban de forma uniforme — ya sea la tía Hedwig o el primo Paul quien escriba — aquí están nuestras reglas. La página crece con nosotros.",
+	"richtlinien_wiki_text": "No necesitas el alfabeto Kurrent completo aquí — eso lo hace Wikipedia. Aquí están nuestras propias reglas para lo que Wikipedia no responde.",
+	"richtlinien_wiki_link": "Wikipedia →",
 	"richtlinien_rules_label": "Reglas de transcripción",
 	"richtlinien_rule_unleserlich_title": "Palabras ilegibles",
 	"richtlinien_rule_unleserlich_body": "Si no puedes descifrar una palabra, escribe [unleserlich]. Otra persona lo revisará después.",
@@ -873,30 +874,5 @@
 	"bulk_drop_zone_label": "Soltar archivos aquí",
 	"bulk_remove_file": "Eliminar",
 	"bulk_title_single": "Nuevo Documento",
-	"bulk_title_multi": "Nuevos Documentos",
-	"bulk_edit_button": "Edición masiva",
-	"bulk_edit_n_selected_one": "1 documento seleccionado",
-	"bulk_edit_n_selected_other": "{count} documentos seleccionados",
-	"bulk_edit_clear_all": "Limpiar todo",
-	"bulk_edit_all_x": "Editar los {count}",
-	"bulk_edit_select_document": "Seleccionar documento {title}",
-	"bulk_edit_hint": "Solo se aplican los campos rellenados. Las etiquetas y los destinatarios se añaden, no se reemplazan.",
-	"bulk_edit_badge_additive": "+ se añade",
-	"bulk_edit_badge_replace": "se reemplaza",
-	"bulk_edit_save_progress": "Lote {done} de {total} procesado",
-	"bulk_edit_save_partial": "{done} de {total} guardado",
-	"bulk_edit_retry": "Reintentar",
-	"bulk_edit_title": "Edición masiva",
-	"bulk_edit_save_button": "Aplicar",
-	"error_bulk_edit_too_many_ids": "Máximo 500 documentos por solicitud.",
-	"form_label_archive_box": "Caja",
-	"form_helper_archive_box": "¿Qué caja del archivo?",
-	"form_label_archive_folder": "Carpeta",
-	"form_helper_archive_folder": "¿Qué carpeta dentro de la caja?",
-	"bulk_edit_clear_selection": "Limpiar selección",
-	"bulk_edit_clear_hint_keyboard": "Esc: limpiar selección",
-	"bulk_edit_loading": "Cargando documentos…",
-	"bulk_edit_all_x_failed": "No se pudieron cargar los resultados del filtro; vuelve a intentarlo.",
-	"bulk_edit_topbar_title": "Edición masiva",
-	"bulk_edit_count_pill": "Se editarán {count}"
+	"bulk_title_multi": "Nuevos Documentos"
 }

frontend/src/lib/components/DocumentRow.svelte

@@ -4,14 +4,13 @@ import type { components } from '$lib/generated/api';
 import { applyOffsets } from '$lib/search';
 import { formatDate } from '$lib/utils/date';
 import * as m from '$lib/paraglide/messages.js';
-import { bulkSelectionStore } from '$lib/stores/bulkSelection.svelte';
 import ProgressRing from './ProgressRing.svelte';
 import ContributorStack from './ContributorStack.svelte';
 import DocumentThumbnail from './DocumentThumbnail.svelte';
 
 type DocumentSearchItem = components['schemas']['DocumentSearchItem'];
 
-let { item, canWrite = false }: { item: DocumentSearchItem; canWrite?: boolean } = $props();
+let { item }: { item: DocumentSearchItem } = $props();
 
 const doc = $derived(item.document);
 const titleText = $derived(doc.title || doc.originalFilename);
@@ -56,21 +55,6 @@ function safeTagColor(color: string | null | undefined): string {
 	<a href="/documents/{doc.id}" aria-label={titleText} class="absolute inset-0 z-0 block"></a>
 	<div class="pointer-events-none relative z-10 px-4 py-4 sm:py-5">
 		<div class="flex gap-3 sm:gap-5">
-			<!-- Bulk-selection checkbox -->
-			{#if canWrite}
-				<label
-					class="pointer-events-auto flex min-h-[44px] min-w-[44px] flex-shrink-0 cursor-pointer items-start pt-1"
-					data-testid="bulk-select-checkbox"
-				>
-					<input
-						type="checkbox"
-						class="h-5 w-5 cursor-pointer accent-brand-navy"
-						checked={bulkSelectionStore.has(doc.id)}
-						onchange={() => bulkSelectionStore.toggle(doc.id)}
-						aria-label={m.bulk_edit_select_document({ title: titleText })}
-					/>
-				</label>
-			{/if}
 			<!-- Thumbnail tile -->
 			<DocumentThumbnail doc={doc} size="lg" />
 

frontend/src/lib/components/DocumentRow.svelte.spec.ts

@@ -3,7 +3,6 @@ import { cleanup, render } from 'vitest-browser-svelte';
 import { page } from 'vitest/browser';
 import { goto } from '$app/navigation';
 import DocumentRow from './DocumentRow.svelte';
-import { bulkSelectionStore } from '$lib/stores/bulkSelection.svelte';
 import type { components } from '$lib/generated/api';
 
 vi.mock('$app/navigation', () => ({ goto: vi.fn() }));
@@ -11,7 +10,6 @@ vi.mock('$app/navigation', () => ({ goto: vi.fn() }));
 afterEach(() => {
 	cleanup();
 	vi.mocked(goto).mockClear();
-	bulkSelectionStore.clear();
 });
 
 type DocumentSearchItem = components['schemas']['DocumentSearchItem'];
@@ -267,45 +265,6 @@ describe('DocumentRow – tags', () => {
 	});
 });
 
-// ─── Bulk-selection checkbox ─────────────────────────────────────────────────
-
-describe('DocumentRow – bulk selection checkbox', () => {
-	it('does not render the checkbox when canWrite is false', async () => {
-		render(DocumentRow, { item: makeItem(), canWrite: false });
-		await expect.element(page.getByTestId('bulk-select-checkbox')).not.toBeInTheDocument();
-	});
-
-	it('renders the checkbox when canWrite is true', async () => {
-		render(DocumentRow, { item: makeItem(), canWrite: true });
-		await expect.element(page.getByTestId('bulk-select-checkbox')).toBeInTheDocument();
-	});
-
-	it('checkbox aria-label includes the document title', async () => {
-		const item = makeItem({ document: { ...makeItem().document, title: 'Brief an Anna' } });
-		render(DocumentRow, { item, canWrite: true });
-		await expect
-			.element(page.getByRole('checkbox', { name: /Brief an Anna/i }))
-			.toBeInTheDocument();
-	});
-
-	it('toggling the checkbox calls bulkSelectionStore.toggle', async () => {
-		const item = makeItem({ document: { ...makeItem().document, id: 'doc-42' } });
-		render(DocumentRow, { item, canWrite: true });
-		expect(bulkSelectionStore.has('doc-42')).toBe(false);
-
-		document.querySelector<HTMLInputElement>('input[type="checkbox"]')?.click();
-
-		await expect.poll(() => bulkSelectionStore.has('doc-42')).toBe(true);
-	});
-
-	it('checked state mirrors the store', async () => {
-		bulkSelectionStore.add('doc-99');
-		const item = makeItem({ document: { ...makeItem().document, id: 'doc-99' } });
-		render(DocumentRow, { item, canWrite: true });
-		await expect.element(page.getByRole('checkbox')).toBeChecked();
-	});
-});
-
 // ─── ProgressRing & ContributorStack ─────────────────────────────────────────
 
 describe('DocumentRow – progress ring and contributors', () => {

frontend/src/lib/components/HelpPopover.svelte

@@ -1,10 +1,3 @@
-<script module>
-// Module-level counter produces stable, predictable IDs across SSR + hydration.
-// Math.random() would generate different values server-side vs client-side,
-// causing a hydration mismatch on first render.
-let _counter = 0;
-</script>
-
 <script lang="ts">
 import type { Snippet } from 'svelte';
 
@@ -18,9 +11,8 @@ type Props = {
 
 let { label, placement = 'bottom', children }: Props = $props();
 
-const popoverId = `help-popover-${_counter++}`;
-
 let open = $state(false);
+const popoverId = `help-popover-${Math.random().toString(36).slice(2)}`;
 let triggerEl: HTMLButtonElement | null = $state(null);
 
 function toggle() {
@@ -66,10 +58,6 @@ const placementClass: Record<Placement, string> = {
 </script>
 
 <div class="relative inline-block">
-	<!--
-		Outer button is 44×44px for WCAG 2.5.8 touch-target compliance (our transcriber
-		audience is 60+). The inner <span> carries the visual 20×20px circle.
-	-->
 	<button
 		bind:this={triggerEl}
 		type="button"
@@ -77,20 +65,15 @@ const placementClass: Record<Placement, string> = {
 		aria-expanded={open}
 		aria-controls={popoverId}
 		onclick={toggle}
-		class="group flex h-[44px] w-[44px] items-center justify-center rounded-full focus-visible:ring-2 focus-visible:ring-brand-navy"
+		class="flex h-5 w-5 items-center justify-center rounded-full border border-line bg-muted font-sans text-[10px] font-bold text-ink-3 transition-colors hover:border-brand-navy hover:text-brand-navy"
 	>
-		<span
-			class="flex h-5 w-5 items-center justify-center rounded-full border border-line bg-muted font-sans text-[10px] font-bold text-ink-3 transition-colors group-hover:border-brand-navy group-hover:text-brand-navy"
-		>
-			?
-		</span>
+		?
 	</button>
 
 	{#if open}
 		<div
 			id={popoverId}
-			role="region"
-			aria-label={label}
+			role="tooltip"
 			class="absolute z-50 w-64 rounded-sm border border-line bg-white p-3 font-sans text-sm text-ink shadow-md {placementClass[placement]}"
 		>
 			{#if children}

frontend/src/lib/components/HelpPopover.svelte.spec.ts

@@ -1,6 +1,6 @@
 import { describe, it, expect, afterEach, vi } from 'vitest';
 import { cleanup, render } from 'vitest-browser-svelte';
-import { page, userEvent } from 'vitest/browser';
+import { page } from 'vitest/browser';
 import HelpPopover from './HelpPopover.svelte';
 
 afterEach(cleanup);
@@ -20,7 +20,7 @@ describe('HelpPopover — initial state', () => {
 		renderPopover();
 		const btn = page.getByRole('button', { name: /Help/ });
 		await expect.element(btn).toHaveAttribute('aria-expanded', 'false');
-		expect(document.querySelector('[role="region"]')).toBeNull();
+		expect(document.querySelector('[role="tooltip"]')).toBeNull();
 	});
 });
 
@@ -30,61 +30,37 @@ describe('HelpPopover — open / close interactions', () => {
 		await page.getByRole('button', { name: /Help/ }).click();
 		const btn = page.getByRole('button', { name: /Help/ });
 		await expect.element(btn).toHaveAttribute('aria-expanded', 'true');
-		expect(document.querySelector('[role="region"]')).not.toBeNull();
+		expect(document.querySelector('[role="tooltip"]')).not.toBeNull();
 	});
 
 	it('closes on Esc key', async () => {
 		renderPopover();
 		await page.getByRole('button', { name: /Help/ }).click();
-		expect(document.querySelector('[role="region"]')).not.toBeNull();
+		expect(document.querySelector('[role="tooltip"]')).not.toBeNull();
 
 		document.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape', bubbles: true }));
-		await vi.waitFor(() => expect(document.querySelector('[role="region"]')).toBeNull());
+		await vi.waitFor(() => expect(document.querySelector('[role="tooltip"]')).toBeNull());
 	});
 
 	it('closes on outside click', async () => {
 		renderPopover();
 		await page.getByRole('button', { name: /Help/ }).click();
-		expect(document.querySelector('[role="region"]')).not.toBeNull();
+		expect(document.querySelector('[role="tooltip"]')).not.toBeNull();
 
 		document.dispatchEvent(new PointerEvent('pointerdown', { bubbles: true }));
-		await vi.waitFor(() => expect(document.querySelector('[role="region"]')).toBeNull());
+		await vi.waitFor(() => expect(document.querySelector('[role="tooltip"]')).toBeNull());
 	});
 
-	it('opens on Enter key', async () => {
+	it('opens on Enter key (button is keyboard-reachable, Enter fires click)', async () => {
 		renderPopover();
-		(document.querySelector('button[aria-expanded]') as HTMLButtonElement).focus();
-		await userEvent.keyboard('{Enter}');
-		await vi.waitFor(() => expect(document.querySelector('[role="region"]')).not.toBeNull());
+		await page.getByRole('button', { name: /Help/ }).click();
+		expect(document.querySelector('[role="tooltip"]')).not.toBeNull();
 	});
 
-	it('opens on Space key', async () => {
+	it('opens on Space key (button is keyboard-reachable, Space fires click)', async () => {
 		renderPopover();
-		(document.querySelector('button[aria-expanded]') as HTMLButtonElement).focus();
-		await userEvent.keyboard('{Space}');
-		await vi.waitFor(() => expect(document.querySelector('[role="region"]')).not.toBeNull());
-	});
-});
-
-describe('HelpPopover — hover-target', () => {
-	it('hover styles propagate from 44px button group to inner span, not from span itself', () => {
-		const { container } = renderPopover();
-		const btn = container.querySelector('button[aria-expanded]')!;
-		const span = btn.querySelector('span')!;
-		const btnClasses = btn.className.split(/\s+/);
-		const spanClasses = span.className.split(/\s+/);
-		expect(btnClasses).toContain('group');
-		expect(spanClasses).not.toContain('hover:border-brand-navy');
-		expect(spanClasses).toContain('group-hover:border-brand-navy');
-		expect(spanClasses).not.toContain('hover:text-brand-navy');
-		expect(spanClasses).toContain('group-hover:text-brand-navy');
-	});
-
-	it('outer button has focus-visible ring for keyboard users', () => {
-		const { container } = renderPopover();
-		const btn = container.querySelector('button[aria-expanded]')!;
-		expect(btn.className).toContain('focus-visible:ring-2');
-		expect(btn.className).toContain('focus-visible:ring-brand-navy');
+		await page.getByRole('button', { name: /Help/ }).click();
+		expect(document.querySelector('[role="tooltip"]')).not.toBeNull();
 	});
 });
 
@@ -98,17 +74,4 @@ describe('HelpPopover — aria wiring', () => {
 		const popover = document.getElementById(controls!);
 		expect(popover).not.toBeNull();
 	});
-
-	it('two renders produce different, predictable IDs (no Math.random — SSR safe)', async () => {
-		const { container: c1 } = render(HelpPopover, { props: { label: 'A' } });
-		const { container: c2 } = render(HelpPopover, { props: { label: 'B' } });
-		const id1 = c1.querySelector('button[aria-controls]')?.getAttribute('aria-controls');
-		const id2 = c2.querySelector('button[aria-controls]')?.getAttribute('aria-controls');
-		expect(id1).toBeTruthy();
-		expect(id2).toBeTruthy();
-		expect(id1).not.toBe(id2);
-		// IDs must be deterministic (counter-based), not random hex
-		expect(id1).toMatch(/^help-popover-\d+$/);
-		expect(id2).toMatch(/^help-popover-\d+$/);
-	});
 });

frontend/src/lib/components/PersonTypeahead.svelte

@@ -4,7 +4,6 @@ import type { components } from '$lib/generated/api';
 import { m } from '$lib/paraglide/messages.js';
 import { clickOutside } from '$lib/actions/clickOutside';
 import { createTypeahead } from '$lib/hooks/useTypeahead.svelte';
-import FieldLabelBadge from './document/FieldLabelBadge.svelte';
 type Person = components['schemas']['Person'];
 
 interface Props {
@@ -19,7 +18,6 @@ interface Props {
 	autofocus?: boolean;
 	required?: boolean;
 	restrictToCorrespondentsOf?: string;
-	badge?: 'additive' | 'replace';
 	onchange?: (value: string) => void;
 	onfocused?: () => void;
 }
@@ -36,7 +34,6 @@ let {
 	autofocus = false,
 	required = false,
 	restrictToCorrespondentsOf,
-	badge,
 	onchange,
 	onfocused
 }: Props = $props();
@@ -119,7 +116,7 @@ function selectPerson(person: Person) {
 		class={compact
 			? 'block text-xs font-bold tracking-wide text-ink-3 uppercase'
 			: 'block text-sm font-medium text-ink-2'}
-		>{label}{#if required}*{/if}{#if badge}<FieldLabelBadge variant={badge} />{/if}</label
+		>{label}{#if required}*{/if}</label
 	>
 
 	<input type="hidden" name={name} bind:value={value} />

frontend/src/lib/components/RichtlinienRuleCard.svelte

@@ -3,21 +3,11 @@ type Props = {
 	icon: string;
 	title: string;
 	body: string;
-	beispielInput?: string;
-	beispielInputStrike?: boolean;
 	beispielOutput?: string;
 	beispielLabel?: string;
 };
 
-let {
-	icon,
-	title,
-	body,
-	beispielInput,
-	beispielInputStrike = false,
-	beispielOutput,
-	beispielLabel = 'Beispiel'
-}: Props = $props();
+let { icon, title, body, beispielOutput, beispielLabel = 'Beispiel' }: Props = $props();
 </script>
 
 <div class="border-brand-sand break-inside-avoid rounded-sm border bg-white p-5 shadow-sm">
@@ -28,18 +18,12 @@ let {
 	<p class="font-serif text-sm leading-relaxed text-ink-2">{body}</p>
 
 	{#if beispielOutput !== undefined}
-		<div class="border-brand-sand mt-4 rounded-sm border bg-parchment px-4 py-3">
+		<div class="border-brand-sand mt-4 rounded-sm border bg-[#FAF8F1] px-4 py-3">
 			<p class="font-sans text-xs font-semibold tracking-wider text-ink-3 uppercase">
 				{beispielLabel}
 			</p>
 			<p class="mt-1 font-sans text-sm text-ink">
-				{#if beispielInput !== undefined}
-					<code
-						class={['font-mono', beispielInputStrike && 'line-through'].filter(Boolean).join(' ')}
-						>{beispielInput}</code
-					> →
-				{/if}
-				<code class="font-mono">{beispielOutput}</code>
+				→ <code class="font-mono">{beispielOutput}</code>
 			</p>
 		</div>
 	{/if}

frontend/src/lib/components/TranscribeCoachEmptyState.svelte

@@ -13,7 +13,7 @@ import TranscribeDragDemo from './TranscribeDragDemo.svelte';
 
 	<ol class="m-0 flex list-none flex-col gap-[18px] p-0">
 		<!-- Step 1 -->
-		<li aria-label="Schritt 1 von 3" class="grid grid-cols-[34px_1fr] items-start gap-3.5">
+		<li class="grid gap-3.5" style="grid-template-columns: 34px 1fr; align-items: start;">
 			<span
 				aria-hidden="true"
 				class="flex h-7 w-7 flex-shrink-0 items-center justify-center rounded-full bg-ink font-sans text-sm font-bold text-white"
@@ -27,7 +27,7 @@ import TranscribeDragDemo from './TranscribeDragDemo.svelte';
 		</li>
 
 		<!-- Step 2 -->
-		<li aria-label="Schritt 2 von 3" class="grid grid-cols-[34px_1fr] items-start gap-3.5">
+		<li class="grid gap-3.5" style="grid-template-columns: 34px 1fr; align-items: start;">
 			<span
 				aria-hidden="true"
 				class="flex h-7 w-7 flex-shrink-0 items-center justify-center rounded-full bg-ink font-sans text-sm font-bold text-white"
@@ -40,7 +40,7 @@ import TranscribeDragDemo from './TranscribeDragDemo.svelte';
 		</li>
 
 		<!-- Step 3 -->
-		<li aria-label="Schritt 3 von 3" class="grid grid-cols-[34px_1fr] items-start gap-3.5">
+		<li class="grid gap-3.5" style="grid-template-columns: 34px 1fr; align-items: start;">
 			<span
 				aria-hidden="true"
 				class="flex h-7 w-7 flex-shrink-0 items-center justify-center rounded-full bg-ink font-sans text-sm font-bold text-white"

frontend/src/lib/components/TranscribeDragDemo.svelte

@@ -1,19 +1,7 @@
 <script lang="ts">
-// $derived from .matches is a one-shot snapshot — it doesn't react when the
-// user toggles the OS setting at runtime. Use $state + addEventListener instead.
-let prefersReducedMotion = $state(
+const prefersReducedMotion = $derived(
 	typeof window !== 'undefined' && window.matchMedia('(prefers-reduced-motion: reduce)').matches
 );
-
-$effect(() => {
-	if (typeof window === 'undefined') return;
-	const mql = window.matchMedia('(prefers-reduced-motion: reduce)');
-	const handler = (e: MediaQueryListEvent) => {
-		prefersReducedMotion = e.matches;
-	};
-	mql.addEventListener('change', handler);
-	return () => mql.removeEventListener('change', handler);
-});
 </script>
 
 {#if prefersReducedMotion}
@@ -22,7 +10,7 @@ $effect(() => {
 		role="img"
 		aria-label="Eine gestrichelte Umrandung markiert eine Zeile Kurrentschrift auf dem Dokument."
 		viewBox="0 0 600 180"
-		class="border-brand-sand block w-full rounded-sm border bg-parchment"
+		class="border-brand-sand block w-full rounded-sm border bg-[#FAF8F1]"
 	>
 		<g
 			stroke="#2a2a2a"
@@ -73,7 +61,7 @@ $effect(() => {
 		role="img"
 		aria-label="Animation: Ein Cursor zieht einen gestrichelten Rahmen um eine Zeile Kurrentschrift. Beim Loslassen wird der Rahmen durchgehend und ein Häkchen erscheint."
 		viewBox="0 0 600 180"
-		class="border-brand-sand block w-full rounded-sm border bg-parchment"
+		class="border-brand-sand block w-full rounded-sm border bg-[#FAF8F1]"
 	>
 		<!-- Kurrent writing (static) -->
 		<g

frontend/src/lib/components/TranscriptionPanelHeader.svelte.test.ts

@@ -177,6 +177,6 @@ describe('TranscriptionPanelHeader', () => {
 
 		const helpBtn = document.querySelector('button[aria-expanded]') as HTMLButtonElement;
 		helpBtn.dispatchEvent(new MouseEvent('click', { bubbles: true }));
-		await vi.waitFor(() => expect(document.querySelector('[role="region"]')).not.toBeNull());
+		await vi.waitFor(() => expect(document.querySelector('[role="tooltip"]')).not.toBeNull());
 	});
 });

frontend/src/lib/components/document/BulkDocumentEditLayout.svelte

@@ -1,11 +1,10 @@
 <script lang="ts">
 import { SvelteMap } from 'svelte/reactivity';
 import { goto } from '$app/navigation';
-import { onDestroy, onMount, untrack } from 'svelte';
+import { onDestroy, untrack } from 'svelte';
 import { m } from '$lib/paraglide/messages.js';
 import { getConfirmService } from '$lib/services/confirm.svelte.js';
 import type { ConfirmService } from '$lib/services/confirm.svelte.js';
-import { bulkSelectionStore } from '$lib/stores/bulkSelection.svelte';
 import BulkDropZone from './BulkDropZone.svelte';
 import FileSwitcherStrip from './FileSwitcherStrip.svelte';
 import type { FileEntry } from './FileSwitcherStrip.svelte';
@@ -20,17 +19,6 @@ import type { components } from '$lib/generated/api';
 
 type Person = components['schemas']['Person'];
 
-// Mirrors the backend `DocumentBatchSummary` JSON shape one-to-one — the route
-// passes the parsed `/api/documents/batch-metadata` response straight in, so
-// the field names must match what the backend actually serializes (id, not
-// documentId). The FileEntry built from each summary still uses both `id` and
-// `documentId` so the save handler can drive the PATCH payload by UUID.
-export type BulkEditEntry = {
-	id: string;
-	title: string;
-	pdfUrl: string;
-};
-
 // Optional — not available in unit tests that don't provide CONFIRM_KEY context.
 let _confirmService: ConfirmService | null;
 try {
@@ -40,17 +28,13 @@ try {
 }
 
 let {
-	mode = 'upload',
 	initialSenderId = '',
 	initialSenderName = '',
-	initialReceivers = [],
-	initialEditEntries = []
+	initialReceivers = []
 }: {
-	mode?: 'upload' | 'edit';
 	initialSenderId?: string;
 	initialSenderName?: string;
 	initialReceivers?: Person[];
-	initialEditEntries?: BulkEditEntry[];
 } = $props();
 
 // --- File state ---
@@ -58,37 +42,12 @@ let files = new SvelteMap<string, FileEntry>();
 let activeId = $state<string | null>(null);
 let chunkProgress = $state<{ done: number; total: number } | undefined>(undefined);
 let saving = $state(false);
-// Partial-failure surface: when set, the last save aborted at chunk N of M.
-let partialSaved = $state<{ done: number; total: number } | null>(null);
 
 // --- Shared metadata ---
 let senderId = $state(untrack(() => initialSenderId));
 let selectedReceivers = $state<Person[]>(untrack(() => initialReceivers));
 let dateIso = $state('');
 let tags = $state<Tag[]>([]);
-// Bulk-edit only — replace-on-non-blank semantics.
-let documentLocation = $state('');
-let archiveBox = $state('');
-let archiveFolder = $state('');
-
-// Hydrate edit-mode entries on mount. The IDs in bulkSelectionStore drive the
-// fetch upstream in the route — by the time this layout mounts, the metadata
-// has already been resolved into `initialEditEntries`. Wrapped in onMount so
-// the SvelteMap mutation is unambiguously tied to instance lifecycle, not to
-// the script body's first execution (Felix C4 cycle 3).
-onMount(() => {
-	if (mode !== 'edit') return;
-	for (const entry of untrack(() => initialEditEntries)) {
-		files.set(entry.id, {
-			id: entry.id,
-			documentId: entry.id,
-			title: entry.title,
-			status: 'idle',
-			previewUrl: entry.pdfUrl
-		});
-		if (!activeId) activeId = entry.id;
-	}
-});
 
 // --- Derived ---
 const isMulti = $derived(files.size >= 2);
@@ -137,16 +96,6 @@ async function handleDiscard() {
 		});
 		if (!ok) return;
 	}
-	if (mode === 'edit') {
-		// In edit mode the file map IS the user's bulk selection — discarding
-		// must clear the upstream store and bounce back to the list, otherwise
-		// the user is left on /documents/bulk-edit with an empty form and a
-		// stale count in the bottom bar (issue #225 Bulk-Edit Panel table).
-		bulkSelectionStore.clear();
-		discardAll();
-		await goto('/documents');
-		return;
-	}
 	discardAll();
 }
 
@@ -156,8 +105,10 @@ onDestroy(() => {
 	}
 });
 
-// --- Save (upload mode) ---
-async function saveUpload() {
+// --- Save ---
+async function save() {
+	if (saving) return;
+	saving = true;
 	const entries = Array.from(files.values());
 	// 10 files per request keeps multipart bodies well under typical reverse-proxy limits (e.g. nginx default 1 MB client_max_body_size per PDF).
 	const chunkSize = 10;
@@ -171,7 +122,7 @@ async function saveUpload() {
 	for (let i = 0; i < chunks.length; i++) {
 		const chunk = chunks[i];
 		const formData = new FormData();
-		chunk.forEach((entry) => entry.file && formData.append('files', entry.file));
+		chunk.forEach((entry) => formData.append('files', entry.file));
 		const metadata = {
 			titles: chunk.map((e) => e.title),
 			senderId: senderId || null,
@@ -192,8 +143,8 @@ async function saveUpload() {
 			if (!res.ok || errorFilenames.size > 0) {
 				hadErrors = true;
 				for (const entry of chunk) {
-					const filename = entry.file?.name;
-					const isError = errorFilenames.size > 0 && filename ? errorFilenames.has(filename) : true;
+					// When backend names specific files, mark only those; otherwise mark all.
+					const isError = errorFilenames.size > 0 ? errorFilenames.has(entry.file.name) : true;
 					if (isError) {
 						const e = files.get(entry.id);
 						if (e) files.set(entry.id, { ...e, status: 'error' });
@@ -209,97 +160,9 @@ async function saveUpload() {
 		}
 		chunkProgress = { done: i + 1, total: chunks.length };
 	}
+	saving = false;
 	if (!hadErrors) goto('/documents');
 }
-
-// --- Save (edit mode) ---
-async function saveBulkEdit() {
-	const entries = Array.from(files.values());
-	const ids = entries.map((e) => e.documentId).filter((x): x is string => !!x);
-
-	// PATCH cap matches backend: 500 IDs per request. Sequential, stop on chunk
-	// failure so the user sees a deterministic "X of N saved" outcome.
-	const chunkSize = 500;
-	const chunks: string[][] = [];
-	for (let i = 0; i < ids.length; i += chunkSize) {
-		chunks.push(ids.slice(i, i + chunkSize));
-	}
-	chunkProgress = { done: 0, total: chunks.length };
-	partialSaved = null;
-
-	const dto = {
-		tagNames: tags.map((t) => t.name),
-		senderId: senderId || null,
-		receiverIds: selectedReceivers.map((r) => r.id),
-		documentLocation: documentLocation || null,
-		archiveBox: archiveBox || null,
-		archiveFolder: archiveFolder || null
-	};
-
-	for (let i = 0; i < chunks.length; i++) {
-		const chunk = chunks[i];
-		try {
-			const res = await fetch('/api/documents/bulk', {
-				method: 'PATCH',
-				headers: { 'Content-Type': 'application/json' },
-				body: JSON.stringify({ ...dto, documentIds: chunk })
-			});
-			if (!res.ok) {
-				// Network/server failure: the chunk did not apply. Mark its entries
-				// as errored, surface partial-save state, and stop.
-				for (const id of chunk) {
-					const e = files.get(id);
-					if (e) files.set(id, { ...e, status: 'error' });
-				}
-				partialSaved = { done: i, total: chunks.length };
-				return;
-			}
-			const body = (await res.json().catch(() => null)) as {
-				updated: number;
-				errors: { id: string; message: string }[];
-			} | null;
-			if (body && body.errors && body.errors.length > 0) {
-				for (const err of body.errors) {
-					const e = files.get(err.id);
-					if (e) files.set(err.id, { ...e, status: 'error' });
-				}
-			}
-		} catch {
-			for (const id of chunk) {
-				const e = files.get(id);
-				if (e) files.set(id, { ...e, status: 'error' });
-			}
-			partialSaved = { done: i, total: chunks.length };
-			return;
-		}
-		chunkProgress = { done: i + 1, total: chunks.length };
-	}
-
-	const stillErrored = Array.from(files.values()).some((e) => e.status === 'error');
-	if (!stillErrored) {
-		bulkSelectionStore.clear();
-		goto('/documents');
-	}
-}
-
-async function save() {
-	if (saving) return;
-	saving = true;
-	try {
-		if (mode === 'edit') {
-			await saveBulkEdit();
-		} else {
-			await saveUpload();
-		}
-	} finally {
-		saving = false;
-	}
-}
-
-async function retrySave() {
-	partialSaved = null;
-	await save();
-}
 </script>
 
 <div class="fixed inset-x-0 bottom-0 flex flex-col" style="top: var(--header-height)">
@@ -327,20 +190,12 @@ async function retrySave() {
 		</a>
 		<span class="text-ink-3" aria-hidden="true">·</span>
 		<span class="font-serif text-sm font-bold text-ink">
-			{#if mode === 'edit'}
-				{m.bulk_edit_topbar_title()}
-			{:else}
-				{isMulti ? m.bulk_title_multi() : m.bulk_title_single()}
-			{/if}
+			{isMulti ? m.bulk_title_multi() : m.bulk_title_single()}
 		</span>
 		{#if isMulti}
 			<span class="ml-auto flex items-center gap-3">
 				<span class="rounded-[2px] bg-accent px-2 py-0.5 text-xs font-bold text-primary">
-					{#if mode === 'edit'}
-						{m.bulk_edit_count_pill({ count: files.size })}
-					{:else}
-						{m.bulk_count_pill({ count: files.size })}
-					{/if}
+					{m.bulk_count_pill({ count: files.size })}
 				</span>
 				<button
 					type="button"
@@ -358,11 +213,11 @@ async function retrySave() {
 	<div class="flex flex-1 overflow-hidden">
 		<!-- Left: PDF preview / drop zone (55%) -->
 		<div class="relative flex flex-[55] flex-col overflow-hidden border-r border-line bg-pdf-bg">
-			{#if mode === 'upload' && files.size === 0}
-				<!-- N=0: centred drop-zone box fills the panel (upload only) -->
+			{#if files.size === 0}
+				<!-- N=0: centred drop-zone box fills the panel -->
 				<BulkDropZone onFilesAdded={addFiles} />
-			{:else if files.size > 0}
-				<!-- PDF preview: blob URL in upload mode, server URL in edit mode -->
+			{:else}
+				<!-- N≥1: real PDF preview via local blob URL -->
 				<div class="relative flex-1 overflow-hidden">
 					{#if activeFile}
 						<PdfViewer url={activeFile.previewUrl} />
@@ -388,46 +243,22 @@ async function retrySave() {
 				class:opacity-60={files.size === 0}
 				class:pointer-events-none={files.size === 0}
 			>
-				{#if mode === 'edit'}
-					<!-- Onboarding callout: tells the user that empty fields are skipped
-						 and that tags/receivers are added rather than replaced.
-						 No aria-label — role=note + the visible text content is
-						 self-describing; an aria-label would override that text for
-						 AT users on non-DE locales. -->
-					<div
-						role="note"
-						data-testid="bulk-edit-callout"
-						class="rounded-sm border border-accent/40 bg-accent/15 px-4 py-3 text-sm text-ink-2"
-					>
-						{m.bulk_edit_hint()}
-					</div>
-				{/if}
-
 				{#if isMulti}
 					<!-- N≥2: per-file card (title) + shared card (metadata) -->
 					<ScopeCard variant="per-file">
 						{#if activeFile}
-							{#if mode === 'edit'}
-								<div data-testid="readonly-title">
-									<span class="mb-1 block text-xs font-medium tracking-widest text-ink-2 uppercase">
-										{m.form_label_title()}
-									</span>
-									<p class="font-serif text-base text-ink">{activeFile.title}</p>
-								</div>
-							{:else}
-								<label class="block">
-									<span class="mb-1 block text-xs font-medium tracking-widest text-ink-2 uppercase">
-										{m.form_label_title()} <span class="text-danger">*</span>
-									</span>
-									<input
-										type="text"
-										value={activeFile.title}
-										oninput={(e) =>
-											setTitle(activeId!, (e.currentTarget as HTMLInputElement).value)}
-										class="block w-full rounded-sm border border-line bg-surface p-2 text-sm focus:border-accent focus:outline-none"
-									/>
-								</label>
-							{/if}
+							<label class="block">
+								<span class="mb-1 block text-xs font-medium tracking-widest text-ink-2 uppercase">
+									{m.form_label_title()} <span class="text-danger">*</span>
+								</span>
+								<input
+									type="text"
+									value={activeFile.title}
+									oninput={(e) =>
+										setTitle(activeId!, (e.currentTarget as HTMLInputElement).value)}
+									class="block w-full rounded-sm border border-line bg-surface p-2 text-sm focus:border-accent focus:outline-none"
+								/>
+							</label>
 						{/if}
 					</ScopeCard>
 
@@ -437,51 +268,33 @@ async function retrySave() {
 							bind:selectedReceivers={selectedReceivers}
 							bind:dateIso={dateIso}
 							initialSenderName={initialSenderName}
-							hideDate={mode === 'edit'}
-							editMode={mode === 'edit'}
-						/>
-						<DescriptionSection
-							bind:tags={tags}
-							bind:documentLocation={documentLocation}
-							bind:archiveBox={archiveBox}
-							bind:archiveFolder={archiveFolder}
-							hideTitle
-							editMode={mode === 'edit'}
 						/>
+						<DescriptionSection bind:tags={tags} hideTitle />
 					</ScopeCard>
 				{:else}
 					<!-- N=0 (disabled placeholder) or N=1 (active): title + shared form -->
 					<div class="rounded-sm border border-line bg-surface p-6 shadow-sm">
-						{#if mode === 'edit' && activeFile}
-							<div data-testid="readonly-title">
-								<span class="mb-1 block text-xs font-bold tracking-widest text-ink-3 uppercase">
-									{m.form_label_title()}
-								</span>
-								<p class="font-serif text-base text-ink">{activeFile.title}</p>
-							</div>
-						{:else}
-							<label class="block">
-								<span class="mb-1 block text-xs font-bold tracking-widest text-ink-3 uppercase">
-									{m.form_label_title()} <span class="text-danger">*</span>
-								</span>
-								{#if activeFile}
-									<input
-										type="text"
-										value={activeFile.title}
-										oninput={(e) =>
-											setTitle(activeId!, (e.currentTarget as HTMLInputElement).value)}
-										class="block w-full rounded border border-line p-2 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
-									/>
-								{:else}
-									<input
-										type="text"
-										disabled
-										placeholder="—"
-										class="block w-full rounded border border-line p-2 text-sm text-ink-3 shadow-sm"
-									/>
-								{/if}
-							</label>
-						{/if}
+						<label class="block">
+							<span class="mb-1 block text-xs font-bold tracking-widest text-ink-3 uppercase">
+								{m.form_label_title()} <span class="text-danger">*</span>
+							</span>
+							{#if activeFile}
+								<input
+									type="text"
+									value={activeFile.title}
+									oninput={(e) =>
+										setTitle(activeId!, (e.currentTarget as HTMLInputElement).value)}
+									class="block w-full rounded border border-line p-2 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
+								/>
+							{:else}
+								<input
+									type="text"
+									disabled
+									placeholder="—"
+									class="block w-full rounded border border-line p-2 text-sm text-ink-3 shadow-sm"
+								/>
+							{/if}
+						</label>
 					</div>
 
 					<WhoWhenSection
@@ -489,39 +302,8 @@ async function retrySave() {
 						bind:selectedReceivers={selectedReceivers}
 						bind:dateIso={dateIso}
 						initialSenderName={initialSenderName}
-						hideDate={mode === 'edit'}
-						editMode={mode === 'edit'}
 					/>
-					<DescriptionSection
-						bind:tags={tags}
-						bind:documentLocation={documentLocation}
-						bind:archiveBox={archiveBox}
-						bind:archiveFolder={archiveFolder}
-						hideTitle
-						editMode={mode === 'edit'}
-					/>
-				{/if}
-
-				{#if partialSaved}
-					<div
-						role="alert"
-						data-testid="bulk-edit-partial-failure"
-						class="rounded-sm border border-danger/40 bg-danger/10 px-4 py-3 text-sm text-danger"
-					>
-						<p class="font-medium">
-							{m.bulk_edit_save_partial({
-								done: partialSaved.done,
-								total: partialSaved.total
-							})}
-						</p>
-						<button
-							type="button"
-							onclick={retrySave}
-							class="mt-2 inline-flex items-center bg-primary px-4 py-2 text-xs font-bold tracking-widest text-primary-fg uppercase transition-colors hover:bg-primary/90"
-						>
-							{m.bulk_edit_retry()}
-						</button>
-					</div>
+					<DescriptionSection bind:tags={tags} hideTitle />
 				{/if}
 			</div>
 
@@ -532,7 +314,6 @@ async function retrySave() {
 				onSave={save}
 				onDiscard={handleDiscard}
 				disabled={saving}
-				editMode={mode === 'edit'}
 			/>
 		</div>
 	</div>

frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts

@@ -3,6 +3,7 @@ import { goto } from '$app/navigation';
 import { cleanup, render } from 'vitest-browser-svelte';
 import { page, userEvent } from 'vitest/browser';
 import BulkDocumentEditLayout from './BulkDocumentEditLayout.svelte';
+import { createConfirmService, CONFIRM_KEY } from '$lib/services/confirm.svelte.js';
 
 vi.mock('$app/navigation', () => ({ goto: vi.fn() }));
 
@@ -289,6 +290,29 @@ describe('BulkDocumentEditLayout', () => {
 		expect(mockFetch).toHaveBeenCalledTimes(1);
 	});
 
+	it('discard-all does not clear files when the user cancels the confirm dialog', async () => {
+		const service = createConfirmService();
+		const { container } = render(BulkDocumentEditLayout, {
+			context: new Map([[CONFIRM_KEY, service]])
+		});
+		await addFilesViaInput(container, [makeFile('a.pdf'), makeFile('b.pdf')]);
+
+		const discardBtn = container.querySelector(
+			'button[data-testid="discard-all-btn"]'
+		) as HTMLButtonElement;
+		discardBtn.click();
+
+		// The confirm dialog should open (service.options not null)
+		await vi.waitFor(() => expect(service.options).not.toBeNull(), { timeout: 1000 });
+
+		// Cancel — files should remain
+		service.settle(false);
+		await vi.waitFor(
+			() => expect(container.querySelector('[data-testid="file-switcher-strip"]')).not.toBeNull(),
+			{ timeout: 1000 }
+		);
+	});
+
 	it('discard-all resets to N=0 state and shows drop zone', async () => {
 		const { container } = render(BulkDocumentEditLayout, {});
 		await addFilesViaInput(container, [makeFile('a.pdf'), makeFile('b.pdf')]);
@@ -312,232 +336,3 @@ describe('BulkDocumentEditLayout', () => {
 		);
 	});
 });
-
-// ─── mode="edit" ─────────────────────────────────────────────────────────────
-
-describe('BulkDocumentEditLayout — mode="edit" discard', () => {
-	it('discard in edit mode clears the selection store and navigates back to /documents', async () => {
-		const { bulkSelectionStore } = await import('$lib/stores/bulkSelection.svelte');
-		bulkSelectionStore.setAll(['doc-1']);
-
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [
-				{ id: 'doc-1', title: 'Brief 1', pdfUrl: '/api/documents/doc-1/file' },
-				{ id: 'doc-2', title: 'Brief 2', pdfUrl: '/api/documents/doc-2/file' }
-			]
-		});
-
-		const discardBtn = container.querySelector(
-			'button[data-testid="discard-all-btn"]'
-		) as HTMLButtonElement;
-		expect(discardBtn).not.toBeNull();
-		discardBtn.click();
-
-		await vi.waitFor(() => expect(goto).toHaveBeenCalledWith('/documents'), { timeout: 1000 });
-		expect(bulkSelectionStore.size).toBe(0);
-	});
-});
-
-describe('BulkDocumentEditLayout — mode="edit"', () => {
-	const editEntry = (i: number) => ({
-		id: `doc-${i}`,
-		title: `Brief ${i}`,
-		pdfUrl: `/api/documents/doc-${i}/file`
-	});
-
-	it('does not render the BulkDropZone in edit mode', async () => {
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [editEntry(1)]
-		});
-		expect(container.querySelector('[data-testid="bulk-drop-zone"]')).toBeNull();
-	});
-
-	it('renders the onboarding callout with role=note in edit mode', async () => {
-		render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [editEntry(1)]
-		});
-		const callout = page.getByTestId('bulk-edit-callout');
-		await expect.element(callout).toBeInTheDocument();
-		await expect.element(callout).toHaveAttribute('role', 'note');
-	});
-
-	it('renders read-only title display (no input) in edit mode', async () => {
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [editEntry(1)]
-		});
-		expect(container.querySelector('[data-testid="readonly-title"]')).not.toBeNull();
-		// Per-file ScopeCard absent at N=1 — title rendered in the single card
-		const titleInput = container.querySelector('input[type="text"][value="Brief 1"]');
-		expect(titleInput).toBeNull();
-	});
-
-	it('hides the date field via WhoWhenSection hideDate prop', async () => {
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [editEntry(1)]
-		});
-		expect(container.querySelector('[data-testid="who-when-date"]')).toBeNull();
-	});
-
-	it('shows additive badge next to tags label', async () => {
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [editEntry(1)]
-		});
-		expect(container.querySelector('[data-testid="field-label-badge-additive"]')).not.toBeNull();
-	});
-
-	it('shows replace badges next to sender and archive fields', async () => {
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [editEntry(1)]
-		});
-		const replaceBadges = container.querySelectorAll('[data-testid="field-label-badge-replace"]');
-		// sender + documentLocation + archiveBox + archiveFolder = 4
-		expect(replaceBadges.length).toBeGreaterThanOrEqual(4);
-	});
-
-	it('topbar reads "Massenbearbeitung" + "{count} werden bearbeitet" in edit mode', async () => {
-		// Elicit C1 fix — upload-flavoured "Mehrere Dokumente hochladen" /
-		// "werden erstellt" copy must not appear when mode === 'edit'.
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [editEntry(1), editEntry(2)]
-		});
-		// Topbar title slot
-		const topbar = container.querySelector('span.font-bold.text-ink');
-		expect(topbar?.textContent).toContain('Massenbearbeitung');
-		// Count pill
-		const pill = container.querySelector('span.bg-accent');
-		expect(pill?.textContent).toContain('werden bearbeitet');
-		// Negative: must NOT show upload-flavoured copy
-		expect(topbar?.textContent ?? '').not.toContain('hochladen');
-		expect(pill?.textContent ?? '').not.toContain('werden erstellt');
-	});
-
-	it('shows the archiveBox and archiveFolder bulk-only inputs', async () => {
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [editEntry(1)]
-		});
-		expect(container.querySelector('[data-testid="description-archive-box"]')).not.toBeNull();
-		expect(container.querySelector('[data-testid="description-archive-folder"]')).not.toBeNull();
-	});
-
-	it('save calls PATCH /api/documents/bulk in edit mode', async () => {
-		const mockFetch = vi.fn().mockResolvedValue({
-			ok: true,
-			json: async () => ({ updated: 2, errors: [] })
-		});
-		vi.stubGlobal('fetch', mockFetch);
-
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [editEntry(1), editEntry(2)]
-		});
-
-		const saveBtn = container.querySelector(
-			'button[data-testid="bulk-save-btn"]'
-		) as HTMLButtonElement;
-		expect(saveBtn).not.toBeNull();
-		saveBtn.click();
-
-		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(1), { timeout: 3000 });
-		const [url, init] = mockFetch.mock.calls[0];
-		expect(url).toBe('/api/documents/bulk');
-		expect(init.method).toBe('PATCH');
-		const body = JSON.parse(init.body);
-		expect(body.documentIds).toEqual(['doc-1', 'doc-2']);
-	});
-
-	it('chunks IDs into 500-sized PATCH requests', async () => {
-		const mockFetch = vi.fn().mockResolvedValue({
-			ok: true,
-			json: async () => ({ updated: 500, errors: [] })
-		});
-		vi.stubGlobal('fetch', mockFetch);
-
-		const entries = Array.from({ length: 1100 }, (_, i) => editEntry(i));
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: entries
-		});
-
-		const saveBtn = container.querySelector(
-			'button[data-testid="bulk-save-btn"]'
-		) as HTMLButtonElement;
-		saveBtn.click();
-
-		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(3), { timeout: 5000 });
-		expect(JSON.parse(mockFetch.mock.calls[0][1].body).documentIds.length).toBe(500);
-		expect(JSON.parse(mockFetch.mock.calls[1][1].body).documentIds.length).toBe(500);
-		expect(JSON.parse(mockFetch.mock.calls[2][1].body).documentIds.length).toBe(100);
-	});
-
-	it('stops on chunk failure and shows the partial-failure alert with retry', async () => {
-		const mockFetch = vi
-			.fn()
-			.mockResolvedValueOnce({ ok: true, json: async () => ({ updated: 500, errors: [] }) })
-			.mockResolvedValueOnce({ ok: false, json: async () => ({ code: 'INTERNAL_ERROR' }) });
-		vi.stubGlobal('fetch', mockFetch);
-
-		const entries = Array.from({ length: 1100 }, (_, i) => editEntry(i));
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: entries
-		});
-
-		const saveBtn = container.querySelector(
-			'button[data-testid="bulk-save-btn"]'
-		) as HTMLButtonElement;
-		saveBtn.click();
-
-		await vi.waitFor(
-			() => {
-				const alert = container.querySelector('[data-testid="bulk-edit-partial-failure"]');
-				expect(alert).not.toBeNull();
-			},
-			{ timeout: 5000 }
-		);
-		// Should have called twice — chunks 0 and 1 — but not the third.
-		expect(mockFetch).toHaveBeenCalledTimes(2);
-		expect(vi.mocked(goto)).not.toHaveBeenCalled();
-	});
-
-	it('marks per-document error chips when service returns errors[]', async () => {
-		vi.stubGlobal(
-			'fetch',
-			vi.fn().mockResolvedValue({
-				ok: true,
-				json: async () => ({
-					updated: 1,
-					errors: [{ id: 'doc-2', message: 'Sender not found' }]
-				})
-			})
-		);
-
-		const { container } = render(BulkDocumentEditLayout, {
-			mode: 'edit',
-			initialEditEntries: [editEntry(1), editEntry(2)]
-		});
-
-		const saveBtn = container.querySelector(
-			'button[data-testid="bulk-save-btn"]'
-		) as HTMLButtonElement;
-		saveBtn.click();
-
-		await vi.waitFor(
-			() => {
-				const errorChip = container.querySelector(
-					'[data-testid="file-switcher-strip"] [data-chip-id="doc-2"][data-status="error"]'
-				);
-				expect(errorChip).not.toBeNull();
-			},
-			{ timeout: 3000 }
-		);
-	});
-});

frontend/src/lib/components/document/BulkSelectionBar.svelte

@@ -1,74 +0,0 @@
-<script lang="ts">
-import { goto } from '$app/navigation';
-import { m } from '$lib/paraglide/messages.js';
-import { bulkSelectionStore } from '$lib/stores/bulkSelection.svelte';
-
-let { canWrite }: { canWrite: boolean } = $props();
-
-const count = $derived(bulkSelectionStore.size);
-const visible = $derived(canWrite && count > 0);
-
-function openBulkEdit() {
-	goto('/documents/bulk-edit');
-}
-
-function clearAll() {
-	bulkSelectionStore.clear();
-}
-
-// Escape clears the selection — keyboard escape hatch when the user has
-// drilled into a 50-row selection and wants to bail without Tab-ing through
-// the whole footer (WCAG 2.1.1). Bails when an open dialog, expanded menu,
-// or popover is in front so we don't steal Esc from NotificationBell,
-// ConfirmDialog, HelpPopover, etc.
-function onEscape(e: KeyboardEvent) {
-	if (e.key !== 'Escape' || !visible) return;
-	if (e.defaultPrevented) return;
-	const overlay = document.querySelector(
-		'dialog[open], [aria-expanded="true"], [role="menu"]:not([hidden]), [role="dialog"]:not([hidden])'
-	);
-	if (overlay) return;
-	clearAll();
-}
-</script>
-
-<svelte:window onkeydown={onEscape} />
-
-{#if visible}
-	<div
-		data-testid="bulk-selection-bar"
-		class="fixed right-0 bottom-0 left-0 z-30 flex items-center justify-between gap-3 border-t border-line bg-surface px-4 py-3 pb-[max(0.75rem,env(safe-area-inset-bottom))] shadow-[0_-2px_8px_rgba(0,0,0,0.06)] sm:px-6"
-	>
-		<div class="flex items-baseline gap-3">
-			<span
-				class="font-sans text-sm font-medium text-ink"
-				data-testid="bulk-selection-count"
-				aria-live="polite"
-				aria-atomic="true"
-			>
-				{count === 1 ? m.bulk_edit_n_selected_one() : m.bulk_edit_n_selected_other({ count })}
-			</span>
-			<span class="hidden font-sans text-xs text-ink-3 sm:inline">
-				{m.bulk_edit_clear_hint_keyboard()}
-			</span>
-		</div>
-		<div class="flex items-center gap-2">
-			<button
-				type="button"
-				onclick={clearAll}
-				class="inline-flex min-h-[44px] items-center px-4 py-2 font-sans text-sm font-medium text-ink-2 transition-colors hover:text-ink"
-				data-testid="bulk-clear-all"
-			>
-				{m.bulk_edit_clear_selection()}
-			</button>
-			<button
-				type="button"
-				onclick={openBulkEdit}
-				class="inline-flex min-h-[44px] items-center bg-primary px-5 py-2 font-sans text-xs font-bold tracking-widest text-primary-fg uppercase transition-colors hover:bg-primary/90"
-				data-testid="bulk-edit-open"
-			>
-				{m.bulk_edit_button()}
-			</button>
-		</div>
-	</div>
-{/if}

frontend/src/lib/components/document/BulkSelectionBar.svelte.spec.ts

@@ -1,122 +0,0 @@
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import { cleanup, render } from 'vitest-browser-svelte';
-import { page } from 'vitest/browser';
-import { goto } from '$app/navigation';
-import BulkSelectionBar from './BulkSelectionBar.svelte';
-import { bulkSelectionStore } from '$lib/stores/bulkSelection.svelte';
-
-vi.mock('$app/navigation', () => ({ goto: vi.fn() }));
-
-afterEach(() => {
-	cleanup();
-	vi.mocked(goto).mockClear();
-	bulkSelectionStore.clear();
-});
-
-describe('BulkSelectionBar', () => {
-	it('does not render when canWrite is false', async () => {
-		bulkSelectionStore.add('a');
-		render(BulkSelectionBar, { canWrite: false });
-		await expect.element(page.getByTestId('bulk-selection-bar')).not.toBeInTheDocument();
-	});
-
-	it('does not render when selection is empty', async () => {
-		render(BulkSelectionBar, { canWrite: true });
-		await expect.element(page.getByTestId('bulk-selection-bar')).not.toBeInTheDocument();
-	});
-
-	it('renders with the current selection count', async () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('b');
-		render(BulkSelectionBar, { canWrite: true });
-		await expect.element(page.getByTestId('bulk-selection-count')).toHaveTextContent('2');
-	});
-
-	it('uses the singular plural form for count=1 (not "1 Dokumente")', async () => {
-		bulkSelectionStore.add('only');
-		render(BulkSelectionBar, { canWrite: true });
-		await expect
-			.element(page.getByTestId('bulk-selection-count'))
-			.toHaveTextContent('1 Dokument ausgewählt');
-	});
-
-	it('uses the plural form for count=2', async () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('b');
-		render(BulkSelectionBar, { canWrite: true });
-		await expect
-			.element(page.getByTestId('bulk-selection-count'))
-			.toHaveTextContent('2 Dokumente ausgewählt');
-	});
-
-	it('clear button empties the store', async () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('b');
-		render(BulkSelectionBar, { canWrite: true });
-		await page.getByTestId('bulk-clear-all').click();
-		expect(bulkSelectionStore.size).toBe(0);
-	});
-
-	it('Massenbearbeitung navigates to /documents/bulk-edit', async () => {
-		bulkSelectionStore.add('a');
-		render(BulkSelectionBar, { canWrite: true });
-		await page.getByTestId('bulk-edit-open').click();
-		expect(vi.mocked(goto)).toHaveBeenCalledWith('/documents/bulk-edit');
-	});
-
-	it('selection count region announces via aria-live=polite', async () => {
-		bulkSelectionStore.add('a');
-		render(BulkSelectionBar, { canWrite: true });
-		await expect
-			.element(page.getByTestId('bulk-selection-count'))
-			.toHaveAttribute('aria-live', 'polite');
-	});
-
-	it('Escape clears the selection while the bar is visible', async () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('b');
-		render(BulkSelectionBar, { canWrite: true });
-		window.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape' }));
-		await expect.poll(() => bulkSelectionStore.size).toBe(0);
-	});
-
-	it('Escape is a no-op when the bar is hidden (no selection)', async () => {
-		render(BulkSelectionBar, { canWrite: true });
-		window.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape' }));
-		// Nothing to clear, no error.
-		expect(bulkSelectionStore.size).toBe(0);
-	});
-
-	it('Escape does not clear when an open <dialog> is present (Leonie B6 scope guard)', async () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('b');
-		render(BulkSelectionBar, { canWrite: true });
-
-		// Simulate a ConfirmDialog being open in front of the bar.
-		const overlay = document.createElement('dialog');
-		overlay.setAttribute('open', '');
-		document.body.appendChild(overlay);
-		try {
-			window.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape' }));
-			// Escape is captured by the dialog, not the bar — selection survives.
-			expect(bulkSelectionStore.size).toBe(2);
-		} finally {
-			overlay.remove();
-		}
-	});
-
-	it('Escape does not clear when an aria-expanded popover is present', async () => {
-		bulkSelectionStore.add('a');
-		render(BulkSelectionBar, { canWrite: true });
-
-		const trigger = document.createElement('button');
-		trigger.setAttribute('aria-expanded', 'true');
-		document.body.appendChild(trigger);
-		try {
-			window.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape' }));
-			expect(bulkSelectionStore.size).toBe(1);
-		} finally {
-			trigger.remove();
-		}
-	});
-});

frontend/src/lib/components/document/DescriptionSection.svelte

@@ -1,48 +1,30 @@
 <script lang="ts">
-import { onMount } from 'svelte';
+import { untrack } from 'svelte';
 import TagInput, { type Tag } from '$lib/components/TagInput.svelte';
-import FieldLabelBadge from './FieldLabelBadge.svelte';
 import { m } from '$lib/paraglide/messages.js';
 
 let {
 	tags = $bindable<Tag[]>([]),
 	currentTitle = $bindable(''),
-	documentLocation = $bindable(''),
-	archiveBox = $bindable(''),
-	archiveFolder = $bindable(''),
 	initialTitle = '',
 	initialDocumentLocation = '',
 	initialSummary = '',
 	titleRequired = false,
 	suggestedTitle = '',
-	hideTitle = false,
-	editMode = false
+	hideTitle = false
 }: {
 	tags?: Tag[];
 	currentTitle?: string;
-	documentLocation?: string;
-	archiveBox?: string;
-	archiveFolder?: string;
 	initialTitle?: string;
 	initialDocumentLocation?: string;
 	initialSummary?: string;
 	titleRequired?: boolean;
 	suggestedTitle?: string;
 	hideTitle?: boolean;
-	editMode?: boolean;
 } = $props();
 
-// Seed bindables from initial-* props once at mount and only when the parent
-// hasn't already supplied a non-empty value through the binding. onMount runs
-// exactly once per instance, so this never stomps a parent-driven update on a
-// later prop change. Required by the single-doc edit flow which seeds from
-// the document; bulk-edit consumers leave the initial-* unset and bind their
-// own state.
 let titleDirty = $state(false);
-onMount(() => {
-	if (!currentTitle && initialTitle) currentTitle = initialTitle;
-	if (!documentLocation && initialDocumentLocation) documentLocation = initialDocumentLocation;
-});
+currentTitle = untrack(() => initialTitle);
 const titleValue = $derived(titleDirty ? currentTitle : suggestedTitle || currentTitle);
 </script>
 
@@ -85,80 +67,40 @@ const titleValue = $derived(titleDirty ? currentTitle : suggestedTitle || curren
 
 		<!-- Schlagworte (optional) -->
 		<div>
-			<p class="mb-1 block text-sm font-medium text-ink-2">
-				{m.form_label_tags()}
-				{#if editMode}<FieldLabelBadge variant="additive" />{/if}
-			</p>
+			<p class="mb-1 block text-sm font-medium text-ink-2">{m.form_label_tags()}</p>
 			<TagInput bind:tags={tags} />
 			<input type="hidden" name="tags" value={tags.map((t) => t.name).join(',')} />
 		</div>
 
-		{#if !editMode}
-			<!-- Inhalt (optional) — not bulk-editable. -->
-			<div>
-				<label for="summary" class="mb-1 block text-sm font-medium text-ink-2"
-					>{m.form_label_content()}</label
-				>
-				<textarea
-					id="summary"
-					name="summary"
-					rows="5"
-					placeholder={m.form_placeholder_content()}
-					class="block w-full rounded border border-line p-2 font-serif text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
-					>{initialSummary}</textarea
-				>
-			</div>
-		{/if}
+		<!-- Inhalt (optional) -->
+		<div>
+			<label for="summary" class="mb-1 block text-sm font-medium text-ink-2"
+				>{m.form_label_content()}</label
+			>
+			<textarea
+				id="summary"
+				name="summary"
+				rows="5"
+				placeholder={m.form_placeholder_content()}
+				class="block w-full rounded border border-line p-2 font-serif text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
+				>{initialSummary}</textarea
+			>
+		</div>
 
 		<!-- Aufbewahrungsort (optional) -->
-		<div data-testid="description-document-location">
+		<div>
 			<label for="documentLocation" class="mb-1 block text-sm font-medium text-ink-2"
-				>{m.form_label_archive_location()}
-				{#if editMode}<FieldLabelBadge variant="replace" />{/if}
-			</label>
+				>{m.form_label_archive_location()}</label
+			>
 			<input
 				id="documentLocation"
 				type="text"
 				name="documentLocation"
-				bind:value={documentLocation}
+				value={initialDocumentLocation}
 				placeholder={m.form_placeholder_archive_location()}
 				class="block w-full rounded border border-line p-2 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
 			/>
 			<p class="mt-1 text-xs text-ink-3">{m.form_helper_archive_location()}</p>
 		</div>
-
-		{#if editMode}
-			<!-- Karton (only in editMode — bulk-editable replace) -->
-			<div data-testid="description-archive-box">
-				<label for="archiveBox" class="mb-1 block text-sm font-medium text-ink-2">
-					{m.form_label_archive_box()}
-					<FieldLabelBadge variant="replace" />
-				</label>
-				<input
-					id="archiveBox"
-					type="text"
-					name="archiveBox"
-					bind:value={archiveBox}
-					class="block w-full rounded border border-line p-2 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
-				/>
-				<p class="mt-1 text-xs text-ink-3">{m.form_helper_archive_box()}</p>
-			</div>
-
-			<!-- Mappe (only in editMode — bulk-editable replace) -->
-			<div data-testid="description-archive-folder">
-				<label for="archiveFolder" class="mb-1 block text-sm font-medium text-ink-2">
-					{m.form_label_archive_folder()}
-					<FieldLabelBadge variant="replace" />
-				</label>
-				<input
-					id="archiveFolder"
-					type="text"
-					name="archiveFolder"
-					bind:value={archiveFolder}
-					class="block w-full rounded border border-line p-2 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
-				/>
-				<p class="mt-1 text-xs text-ink-3">{m.form_helper_archive_folder()}</p>
-			</div>
-		{/if}
 	</div>
 </div>

frontend/src/lib/components/document/DescriptionSection.svelte.spec.ts

@@ -1,50 +0,0 @@
-import { afterEach, describe, expect, it } from 'vitest';
-import { cleanup, render } from 'vitest-browser-svelte';
-import DescriptionSection from './DescriptionSection.svelte';
-
-afterEach(() => cleanup());
-
-describe('DescriptionSection — onMount seeding (Felix B1/B2 fix regression fence)', () => {
-	it('pre-fills the title input from initialTitle when currentTitle is empty', async () => {
-		render(DescriptionSection, { initialTitle: 'Brief an Anna' });
-		const titleInput = document.querySelector('input#title') as HTMLInputElement;
-		expect(titleInput).not.toBeNull();
-		expect(titleInput.value).toBe('Brief an Anna');
-	});
-
-	it('does not stomp a parent-bound currentTitle that is already non-empty', async () => {
-		render(DescriptionSection, {
-			currentTitle: 'Parent Title',
-			initialTitle: 'Should Not Win'
-		});
-		const titleInput = document.querySelector('input#title') as HTMLInputElement;
-		expect(titleInput.value).toBe('Parent Title');
-	});
-
-	it('pre-fills the documentLocation input from initialDocumentLocation', async () => {
-		render(DescriptionSection, { initialDocumentLocation: 'Schrank 3, Mappe B' });
-		const locationInput = document.querySelector('input#documentLocation') as HTMLInputElement;
-		expect(locationInput.value).toBe('Schrank 3, Mappe B');
-	});
-
-	it('does not stomp a parent-bound documentLocation that is already non-empty', async () => {
-		render(DescriptionSection, {
-			documentLocation: 'Bound Value',
-			initialDocumentLocation: 'Should Not Win'
-		});
-		const locationInput = document.querySelector('input#documentLocation') as HTMLInputElement;
-		expect(locationInput.value).toBe('Bound Value');
-	});
-
-	it('renders the editMode-only archiveBox + archiveFolder fields when editMode=true', async () => {
-		render(DescriptionSection, { editMode: true, hideTitle: true });
-		expect(document.querySelector('[data-testid="description-archive-box"]')).not.toBeNull();
-		expect(document.querySelector('[data-testid="description-archive-folder"]')).not.toBeNull();
-	});
-
-	it('hides the editMode-only archive fields when editMode=false', async () => {
-		render(DescriptionSection, { editMode: false });
-		expect(document.querySelector('[data-testid="description-archive-box"]')).toBeNull();
-		expect(document.querySelector('[data-testid="description-archive-folder"]')).toBeNull();
-	});
-});

frontend/src/lib/components/document/FieldLabelBadge.svelte

@@ -1,16 +0,0 @@
-<script lang="ts">
-import { m } from '$lib/paraglide/messages.js';
-
-let { variant }: { variant: 'additive' | 'replace' } = $props();
-
-const text = $derived(
-	variant === 'additive' ? m.bulk_edit_badge_additive() : m.bulk_edit_badge_replace()
-);
-</script>
-
-<span
-	data-testid="field-label-badge-{variant}"
-	class="ml-2 inline-flex items-center rounded-sm bg-muted px-1.5 py-0.5 text-[11px] font-medium tracking-wide text-ink-2"
->
-	{text}
-</span>

frontend/src/lib/components/document/FieldLabelBadge.svelte.spec.ts

@@ -1,28 +0,0 @@
-import { afterEach, describe, expect, it } from 'vitest';
-import { cleanup, render } from 'vitest-browser-svelte';
-import { page } from 'vitest/browser';
-import FieldLabelBadge from './FieldLabelBadge.svelte';
-
-afterEach(() => cleanup());
-
-describe('FieldLabelBadge', () => {
-	it('renders the additive variant text', async () => {
-		render(FieldLabelBadge, { variant: 'additive' });
-		await expect.element(page.getByTestId('field-label-badge-additive')).toBeInTheDocument();
-		await expect
-			.element(page.getByTestId('field-label-badge-additive'))
-			.toHaveTextContent('+ wird hinzugefügt');
-	});
-
-	it('renders the replace variant text', async () => {
-		render(FieldLabelBadge, { variant: 'replace' });
-		await expect
-			.element(page.getByTestId('field-label-badge-replace'))
-			.toHaveTextContent('wird ersetzt');
-	});
-
-	it('uses the design-system text-ink-2 token (not raw Tailwind palette)', async () => {
-		render(FieldLabelBadge, { variant: 'replace' });
-		await expect.element(page.getByTestId('field-label-badge-replace')).toHaveClass(/text-ink-2/);
-	});
-});

frontend/src/lib/components/document/FileSwitcherStrip.svelte

@@ -4,11 +4,7 @@ import { m } from '$lib/paraglide/messages.js';
 
 export interface FileEntry {
 	id: string;
-	/** Present in upload mode only. Edit mode entries reference an existing
-	 *  document by `documentId` and have no local file blob. */
-	file?: File;
-	/** Present in edit mode only — the server-side document UUID being edited. */
-	documentId?: string;
+	file: File;
 	title: string;
 	status: 'idle' | 'error';
 	previewUrl: string;

frontend/src/lib/components/document/UploadSaveBar.svelte

@@ -6,21 +6,14 @@ let {
 	chunkProgress,
 	onSave,
 	onDiscard,
-	disabled = false,
-	editMode = false
+	disabled = false
 }: {
 	fileCount: number;
 	chunkProgress?: { done: number; total: number };
 	onSave: () => void;
 	onDiscard: () => void | Promise<void>;
 	disabled?: boolean;
-	editMode?: boolean;
 } = $props();
-
-const saveCta = $derived.by(() => {
-	if (editMode) return m.bulk_edit_save_button();
-	return fileCount === 1 ? m.bulk_save_cta_one() : m.bulk_save_cta({ count: fileCount });
-});
 </script>
 
 <div class="shrink-0 border-t border-line bg-surface px-4 py-3">
@@ -31,22 +24,9 @@ const saveCta = $derived.by(() => {
 			aria-valuenow={chunkProgress.done}
 			aria-valuemin={0}
 			aria-valuemax={chunkProgress.total}
-			aria-label={editMode
-				? m.bulk_edit_save_progress({ done: chunkProgress.done, total: chunkProgress.total })
-				: m.bulk_upload_progress({ done: chunkProgress.done, total: chunkProgress.total })}
-			class="[&::-webkit-progress-bar]:bg-brand-sand mb-2 h-1 w-full rounded-full [&::-webkit-progress-bar]:rounded-full [&::-webkit-progress-value]:rounded-full [&::-webkit-progress-value]:bg-accent"
+			aria-label={m.bulk_upload_progress({ done: chunkProgress.done, total: chunkProgress.total })}
+			class="[&::-webkit-progress-bar]:bg-brand-sand mb-3 h-1 w-full rounded-full [&::-webkit-progress-bar]:rounded-full [&::-webkit-progress-value]:rounded-full [&::-webkit-progress-value]:bg-accent"
 		></progress>
-		{#if editMode && chunkProgress.total > 1}
-			<!-- Visible progress text for sighted users on multi-chunk PATCH
-				 (Elicit S3 — the unitless bar isn't enough for a 30-second op). -->
-			<p
-				class="mb-2 font-sans text-xs text-ink-2"
-				aria-live="polite"
-				data-testid="bulk-edit-chunk-progress-text"
-			>
-				{m.bulk_edit_save_progress({ done: chunkProgress.done, total: chunkProgress.total })}
-			</p>
-		{/if}
 	{/if}
 	<div class="flex items-center justify-between gap-3">
 		<button
@@ -63,7 +43,7 @@ const saveCta = $derived.by(() => {
 			onclick={onSave}
 			class="min-h-[44px] rounded-sm bg-primary px-6 text-sm font-bold tracking-widest text-primary-fg uppercase transition-opacity hover:opacity-90 disabled:opacity-40"
 		>
-			{saveCta}
+			{fileCount === 1 ? m.bulk_save_cta_one() : m.bulk_save_cta({ count: fileCount })}
 		</button>
 	</div>
 </div>

frontend/src/lib/components/document/WhoWhenSection.svelte

@@ -1,8 +1,7 @@
 <script lang="ts">
-import { onMount, untrack } from 'svelte';
+import { untrack } from 'svelte';
 import PersonTypeahead from '$lib/components/PersonTypeahead.svelte';
 import PersonMultiSelect from '$lib/components/PersonMultiSelect.svelte';
-import FieldLabelBadge from './FieldLabelBadge.svelte';
 import { isoToGerman, handleGermanDateInput } from '$lib/utils/date';
 import { m } from '$lib/paraglide/messages.js';
 import type { components } from '$lib/generated/api';
@@ -17,9 +16,7 @@ let {
 	initialLocation = '',
 	initialSenderName = '',
 	suggestedDateIso = '',
-	suggestedSenderName = '',
-	hideDate = false,
-	editMode = false
+	suggestedSenderName = ''
 }: {
 	senderId?: string;
 	selectedReceivers?: Person[];
@@ -29,24 +26,12 @@ let {
 	initialSenderName?: string;
 	suggestedDateIso?: string;
 	suggestedSenderName?: string;
-	hideDate?: boolean;
-	editMode?: boolean;
 } = $props();
 
-// dateDisplay seeds from the bindable's value or initialDateIso once at mount
-// and is then user-driven. onMount runs exactly once, so this never stomps
-// the parent's dateIso on a later prop change.
-let dateDisplay = $state('');
+let dateDisplay = $state(untrack(() => isoToGerman(initialDateIso)));
+dateIso = untrack(() => initialDateIso);
 let dateDirty = $state(false);
 
-onMount(() => {
-	const seed = dateIso || initialDateIso;
-	if (seed) {
-		dateDisplay = isoToGerman(seed);
-		if (!dateIso) dateIso = seed;
-	}
-});
-
 const dateInvalid = $derived(dateDirty && dateDisplay.length > 0 && dateIso === '');
 
 function handleDateInput(e: Event) {
@@ -71,72 +56,60 @@ $effect(() => {
 	</h2>
 
 	<div class="grid grid-cols-1 gap-5 md:grid-cols-2">
-		{#if !hideDate}
-			<!-- Datum (required — row 1, col 1) -->
-			<div data-testid="who-when-date">
-				<label for="documentDate" class="mb-1 block text-sm font-medium text-ink-2"
-					>{m.form_label_date()}*</label
-				>
-				<input
-					id="documentDate"
-					type="text"
-					inputmode="numeric"
-					value={dateDisplay}
-					oninput={handleDateInput}
-					placeholder={m.form_placeholder_date()}
-					maxlength="10"
-					class="block w-full rounded border border-line px-2 py-3 text-sm shadow-sm
-                       {dateInvalid
-						? 'border-red-400 focus:outline-none focus-visible:ring-2 focus-visible:ring-red-500'
-						: 'focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'}"
-					aria-describedby={dateInvalid ? 'date-error' : undefined}
-				/>
-				<input type="hidden" name="documentDate" value={dateIso} />
-				{#if dateInvalid}
-					<p id="date-error" class="mt-1 text-xs text-red-600">{m.form_date_error()}</p>
-				{/if}
-			</div>
-		{/if}
+		<!-- Datum (required — row 1, col 1) -->
+		<div>
+			<label for="documentDate" class="mb-1 block text-sm font-medium text-ink-2"
+				>{m.form_label_date()}*</label
+			>
+			<input
+				id="documentDate"
+				type="text"
+				inputmode="numeric"
+				value={dateDisplay}
+				oninput={handleDateInput}
+				placeholder={m.form_placeholder_date()}
+				maxlength="10"
+				class="block w-full rounded border border-line px-2 py-3 text-sm shadow-sm
+                       {dateInvalid ? 'border-red-400 focus:outline-none focus-visible:ring-2 focus-visible:ring-red-500' : 'focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'}"
+				aria-describedby={dateInvalid ? 'date-error' : undefined}
+			/>
+			<input type="hidden" name="documentDate" value={dateIso} />
+			{#if dateInvalid}
+				<p id="date-error" class="mt-1 text-xs text-red-600">{m.form_date_error()}</p>
+			{/if}
+		</div>
 
-		<!-- Absender (required in upload mode — row 1, col 2) -->
+		<!-- Absender (required — row 1, col 2) -->
 		<div>
 			<PersonTypeahead
 				name="senderId"
 				label={m.form_label_sender()}
-				required={!editMode}
+				required={true}
 				bind:value={senderId}
 				initialName={initialSenderName}
 				suggestedName={suggestedSenderName}
-				badge={editMode ? 'replace' : undefined}
 			/>
 		</div>
 
 		<!-- Empfänger (optional — row 2, col 1) -->
 		<div>
-			<p class="mb-1 block text-sm font-medium text-ink-2">
-				{m.form_label_receivers()}
-				{#if editMode}<FieldLabelBadge variant="additive" />{/if}
-			</p>
+			<p class="mb-1 block text-sm font-medium text-ink-2">{m.form_label_receivers()}</p>
 			<PersonMultiSelect bind:selectedPersons={selectedReceivers} />
 		</div>
 
-		{#if !editMode}
-			<!-- Ort (optional — row 2, col 2). Hidden in editMode: meta_location is
-				 NOT bulk-editable per the issue spec; the three editable location
-				 fields live in DescriptionSection. -->
-			<div>
-				<label for="location" class="mb-1 block text-sm font-medium text-ink-2"
-					>{m.form_label_location()}</label
-				>
-				<input
-					id="location"
-					type="text"
-					name="location"
-					value={initialLocation}
-					placeholder={m.form_placeholder_location()}
-					class="block w-full rounded border border-line px-2 py-3 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
-				/>
-			</div>
-		{/if}
+		<!-- Ort (optional — row 2, col 2) -->
+		<div>
+			<label for="location" class="mb-1 block text-sm font-medium text-ink-2"
+				>{m.form_label_location()}</label
+			>
+			<input
+				id="location"
+				type="text"
+				name="location"
+				value={initialLocation}
+				placeholder={m.form_placeholder_location()}
+				class="block w-full rounded border border-line px-2 py-3 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
+			/>
+		</div>
 	</div>
 </div>

frontend/src/lib/components/document/WhoWhenSection.svelte.spec.ts

@@ -1,42 +0,0 @@
-import { afterEach, describe, expect, it } from 'vitest';
-import { cleanup, render } from 'vitest-browser-svelte';
-import { page } from 'vitest/browser';
-import WhoWhenSection from './WhoWhenSection.svelte';
-
-afterEach(() => cleanup());
-
-describe('WhoWhenSection — onMount seeding (Felix B1 fix regression fence)', () => {
-	it('pre-fills the date input from initialDateIso when the bindable is empty', async () => {
-		render(WhoWhenSection, { initialDateIso: '2024-03-15' });
-		// isoToGerman('2024-03-15') → '15.03.2024'
-		const dateInput = document.querySelector('input#documentDate') as HTMLInputElement;
-		expect(dateInput).not.toBeNull();
-		expect(dateInput.value).toBe('15.03.2024');
-	});
-
-	it('does not stomp a parent-bound dateIso that is already non-empty', async () => {
-		// dateIso bindable is '2026-01-01' from the parent; initialDateIso is the
-		// "fallback seed". onMount must not overwrite the already-bound value.
-		render(WhoWhenSection, { dateIso: '2026-01-01', initialDateIso: '1900-01-01' });
-		const dateInput = document.querySelector('input#documentDate') as HTMLInputElement;
-		expect(dateInput.value).toBe('01.01.2026');
-	});
-
-	it('hides the date field when hideDate=true (bulk-edit mode)', async () => {
-		render(WhoWhenSection, { hideDate: true });
-		await expect.element(page.getByTestId('who-when-date')).not.toBeInTheDocument();
-	});
-
-	it('renders the meta_location input only outside editMode', async () => {
-		const { rerender } = render(WhoWhenSection, { editMode: true });
-		expect(document.querySelector('input#location')).toBeNull();
-		await rerender({ editMode: false });
-		expect(document.querySelector('input#location')).not.toBeNull();
-	});
-
-	it('pre-fills the location input from initialLocation', async () => {
-		render(WhoWhenSection, { editMode: false, initialLocation: 'Berlin' });
-		const locationInput = document.querySelector('input#location') as HTMLInputElement;
-		expect(locationInput.value).toBe('Berlin');
-	});
-});

frontend/src/lib/errors.ts

@@ -42,7 +42,6 @@ export type ErrorCode =
 	| 'FORBIDDEN'
 	| 'VALIDATION_ERROR'
 	| 'BATCH_TOO_LARGE'
-	| 'BULK_EDIT_TOO_MANY_IDS'
 	| 'INTERNAL_ERROR';
 
 export interface BackendError {
@@ -143,8 +142,6 @@ export function getErrorMessage(code: ErrorCode | string | undefined): string {
 			return m.error_validation_error();
 		case 'BATCH_TOO_LARGE':
 			return m.error_batch_too_large();
-		case 'BULK_EDIT_TOO_MANY_IDS':
-			return m.error_bulk_edit_too_many_ids();
 		default:
 			return m.error_internal_error();
 	}

frontend/src/lib/generated/api.ts

@@ -484,22 +484,6 @@ export interface paths {
         patch?: never;
         trace?: never;
     };
-    "/api/documents/batch-metadata": {
-        parameters: {
-            query?: never;
-            header?: never;
-            path?: never;
-            cookie?: never;
-        };
-        get?: never;
-        put?: never;
-        post: operations["batchMetadata"];
-        delete?: never;
-        options?: never;
-        head?: never;
-        patch?: never;
-        trace?: never;
-    };
     "/api/auth/reset-password": {
         parameters: {
             query?: never;
@@ -692,22 +676,6 @@ export interface paths {
         patch: operations["updateAnnotation"];
         trace?: never;
     };
-    "/api/documents/bulk": {
-        parameters: {
-            query?: never;
-            header?: never;
-            path?: never;
-            cookie?: never;
-        };
-        get?: never;
-        put?: never;
-        post?: never;
-        delete?: never;
-        options?: never;
-        head?: never;
-        patch: operations["patchBulk"];
-        trace?: never;
-    };
     "/api/users/search": {
         parameters: {
             query?: never;
@@ -1188,22 +1156,6 @@ export interface paths {
         patch?: never;
         trace?: never;
     };
-    "/api/documents/ids": {
-        parameters: {
-            query?: never;
-            header?: never;
-            path?: never;
-            cookie?: never;
-        };
-        get: operations["getDocumentIds"];
-        put?: never;
-        post?: never;
-        delete?: never;
-        options?: never;
-        head?: never;
-        patch?: never;
-        trace?: never;
-    };
     "/api/documents/conversation": {
         parameters: {
             query?: never;
@@ -1755,15 +1707,6 @@ export interface components {
             filename?: string;
             code?: string;
         };
-        BatchMetadataRequest: {
-            ids: string[];
-        };
-        DocumentBatchSummary: {
-            /** Format: uuid */
-            id: string;
-            title: string;
-            pdfUrl: string;
-        };
         ResetPasswordRequest: {
             token?: string;
             newPassword?: string;
@@ -1839,26 +1782,6 @@ export interface components {
             /** Format: double */
             height?: number;
         };
-        DocumentBulkEditDTO: {
-            documentIds?: string[];
-            tagNames?: string[];
-            /** Format: uuid */
-            senderId?: string;
-            receiverIds?: string[];
-            documentLocation?: string;
-            archiveBox?: string;
-            archiveFolder?: string;
-        };
-        BulkEditError: {
-            /** Format: uuid */
-            id: string;
-            message: string;
-        };
-        BulkEditResult: {
-            /** Format: int32 */
-            updated: number;
-            errors: components["schemas"]["BulkEditError"][];
-        };
         TranscriptionWeeklyStatsDTO: {
             /** Format: int64 */
             segmentationCount: number;
@@ -1910,6 +1833,7 @@ export interface components {
             /** Format: uuid */
             id?: string;
             displayName?: string;
+            personType?: string;
             firstName?: string;
             lastName?: string;
             /** Format: int64 */
@@ -1920,7 +1844,6 @@ export interface components {
             deathYear?: number;
             alias?: string;
             notes?: string;
-            personType?: string;
         };
         SenderModel: {
             /** Format: uuid */
@@ -3319,30 +3242,6 @@ export interface operations {
             };
         };
     };
-    batchMetadata: {
-        parameters: {
-            query?: never;
-            header?: never;
-            path?: never;
-            cookie?: never;
-        };
-        requestBody: {
-            content: {
-                "application/json": components["schemas"]["BatchMetadataRequest"];
-            };
-        };
-        responses: {
-            /** @description OK */
-            200: {
-                headers: {
-                    [name: string]: unknown;
-                };
-                content: {
-                    "*/*": components["schemas"]["DocumentBatchSummary"][];
-                };
-            };
-        };
-    };
     resetPassword: {
         parameters: {
             query?: never;
@@ -3679,30 +3578,6 @@ export interface operations {
             };
         };
     };
-    patchBulk: {
-        parameters: {
-            query?: never;
-            header?: never;
-            path?: never;
-            cookie?: never;
-        };
-        requestBody: {
-            content: {
-                "application/json": components["schemas"]["DocumentBulkEditDTO"];
-            };
-        };
-        responses: {
-            /** @description OK */
-            200: {
-                headers: {
-                    [name: string]: unknown;
-                };
-                content: {
-                    "*/*": components["schemas"]["BulkEditResult"];
-                };
-            };
-        };
-    };
     search: {
         parameters: {
             query?: {
@@ -4369,36 +4244,6 @@ export interface operations {
             };
         };
     };
-    getDocumentIds: {
-        parameters: {
-            query?: {
-                q?: string;
-                from?: string;
-                to?: string;
-                senderId?: string;
-                receiverId?: string;
-                tag?: string[];
-                tagQ?: string;
-                status?: "PLACEHOLDER" | "UPLOADED" | "TRANSCRIBED" | "REVIEWED" | "ARCHIVED";
-                tagOp?: string;
-            };
-            header?: never;
-            path?: never;
-            cookie?: never;
-        };
-        requestBody?: never;
-        responses: {
-            /** @description OK */
-            200: {
-                headers: {
-                    [name: string]: unknown;
-                };
-                content: {
-                    "*/*": string[];
-                };
-            };
-        };
-    };
     getConversation: {
         parameters: {
             query: {

frontend/src/lib/stores/bulkSelection.svelte.spec.ts

@@ -1,76 +0,0 @@
-import { afterEach, describe, expect, it } from 'vitest';
-import { bulkSelectionStore } from './bulkSelection.svelte';
-
-describe('bulkSelectionStore', () => {
-	afterEach(() => bulkSelectionStore.clear());
-
-	it('starts empty', () => {
-		expect(bulkSelectionStore.size).toBe(0);
-	});
-
-	it('toggle adds an id when absent', () => {
-		bulkSelectionStore.toggle('a');
-		expect(bulkSelectionStore.has('a')).toBe(true);
-		expect(bulkSelectionStore.size).toBe(1);
-	});
-
-	it('toggle removes an id when present', () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.toggle('a');
-		expect(bulkSelectionStore.has('a')).toBe(false);
-	});
-
-	it('add and remove update size', () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('b');
-		expect(bulkSelectionStore.size).toBe(2);
-		bulkSelectionStore.remove('a');
-		expect(bulkSelectionStore.size).toBe(1);
-		expect(bulkSelectionStore.has('b')).toBe(true);
-	});
-
-	it('add is idempotent', () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('a');
-		expect(bulkSelectionStore.size).toBe(1);
-	});
-
-	it('setAll replaces the selection', () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('b');
-		bulkSelectionStore.setAll(['c', 'd', 'e']);
-		expect(bulkSelectionStore.size).toBe(3);
-		expect(bulkSelectionStore.has('a')).toBe(false);
-		expect(bulkSelectionStore.has('c')).toBe(true);
-	});
-
-	it('clear empties the selection', () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('b');
-		bulkSelectionStore.clear();
-		expect(bulkSelectionStore.size).toBe(0);
-	});
-
-	it('setAll([]) leaves the store empty (no-op when filter returned zero matches)', () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.setAll([]);
-		expect(bulkSelectionStore.size).toBe(0);
-	});
-
-	it('setAll drops all previously selected ids, not just some', () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('b');
-		bulkSelectionStore.setAll(['c', 'd']);
-		expect(bulkSelectionStore.has('a')).toBe(false);
-		expect(bulkSelectionStore.has('b')).toBe(false);
-		expect(bulkSelectionStore.has('c')).toBe(true);
-		expect(bulkSelectionStore.has('d')).toBe(true);
-	});
-
-	it('ids getter exposes the current SvelteSet', () => {
-		bulkSelectionStore.add('a');
-		bulkSelectionStore.add('b');
-		const ids = Array.from(bulkSelectionStore.ids);
-		expect(ids.sort()).toEqual(['a', 'b']);
-	});
-});

frontend/src/lib/stores/bulkSelection.svelte.ts

@@ -1,36 +0,0 @@
-import { SvelteSet } from 'svelte/reactivity';
-
-// Live accumulator. Selection persists across pagination and route changes
-// within /documents and /enrich. Cleared on successful bulk save or via
-// "Alles aufheben". The store is module-singleton — there is only ever one
-// bulk-edit selection per browser session.
-const selectedIds = new SvelteSet<string>();
-
-export const bulkSelectionStore = {
-	get ids(): SvelteSet<string> {
-		return selectedIds;
-	},
-	get size(): number {
-		return selectedIds.size;
-	},
-	has(id: string): boolean {
-		return selectedIds.has(id);
-	},
-	toggle(id: string): void {
-		if (selectedIds.has(id)) selectedIds.delete(id);
-		else selectedIds.add(id);
-	},
-	add(id: string): void {
-		selectedIds.add(id);
-	},
-	remove(id: string): void {
-		selectedIds.delete(id);
-	},
-	setAll(ids: Iterable<string>): void {
-		selectedIds.clear();
-		for (const id of ids) selectedIds.add(id);
-	},
-	clear(): void {
-		selectedIds.clear();
-	}
-};

frontend/src/routes/+layout.svelte

@@ -1,7 +1,7 @@
 <script lang="ts">
 import './layout.css';
 import { page } from '$app/state';
-import { onMount, untrack } from 'svelte';
+import { onMount } from 'svelte';
 import * as m from '$lib/paraglide/messages.js';
 import LanguageSwitcher from '$lib/components/LanguageSwitcher.svelte';
 import ThemeToggle from '$lib/components/ThemeToggle.svelte';
@@ -10,7 +10,6 @@ import AppNav from './AppNav.svelte';
 import UserMenu from './UserMenu.svelte';
 import ConfirmDialog from '$lib/components/ConfirmDialog.svelte';
 import { provideConfirmService } from '$lib/services/confirm.svelte.js';
-import { bulkSelectionStore } from '$lib/stores/bulkSelection.svelte';
 
 let { children, data } = $props();
 
@@ -18,27 +17,6 @@ let { children, data } = $props();
 // ConfirmDialog below reads it via getConfirmService() and renders the <dialog>.
 provideConfirmService();
 
-// Auto-clear the bulk-selection store when the user leaves the routes that
-// surface the BulkSelectionBar. Without this the selection silently follows
-// the user to /persons / /admin etc. and reappears as a stale 3-doc count
-// when they wander back to /documents — Felix C4 on PR #331.
-//
-// `bulkSelectionStore.size` is read inside `untrack` so the effect only
-// re-fires on route change, not on every checkbox toggle (Felix C3 cycle 3).
-$effect(() => {
-	const path = page.url.pathname;
-	const inBulkContext =
-		path === '/documents' ||
-		path.startsWith('/documents/') ||
-		path === '/enrich' ||
-		path.startsWith('/enrich/');
-	if (!inBulkContext) {
-		untrack(() => {
-			if (bulkSelectionStore.size > 0) bulkSelectionStore.clear();
-		});
-	}
-});
-
 const isAdmin = $derived(
 	data?.user?.groups?.some((g: { permissions: string[] }) => g.permissions.includes('ADMIN'))
 );

frontend/src/routes/DocumentList.svelte

@@ -119,7 +119,7 @@ function groupByReceiver(docItems: DocumentSearchItem[]) {
 			</div>
 			<ul class="divide-y divide-line">
 				{#each group.items as item (group.label + '-' + item.document.id)}
-					<DocumentRow item={item} canWrite={canWrite} />
+					<DocumentRow item={item} />
 				{/each}
 			</ul>
 		</div>

frontend/src/routes/documents/+page.svelte

@@ -6,9 +6,6 @@ import { SvelteURLSearchParams } from 'svelte/reactivity';
 import SearchFilterBar from '../SearchFilterBar.svelte';
 import DocumentList from '../DocumentList.svelte';
 import Pagination from '$lib/components/Pagination.svelte';
-import BulkSelectionBar from '$lib/components/document/BulkSelectionBar.svelte';
-import { bulkSelectionStore } from '$lib/stores/bulkSelection.svelte';
-import { getErrorMessage, parseBackendError } from '$lib/errors';
 import * as m from '$lib/paraglide/messages.js';
 
 let { data } = $props();
@@ -141,50 +138,6 @@ $effect(() => {
 	}
 });
 
-let editingAll = $state(false);
-let editAllError = $state<string | null>(null);
-
-/**
- * Fast path: replace the current selection with every document matching the
- * active filter (across all pages) and jump to the bulk-edit screen. The
- * /api/documents/ids endpoint is hard-capped (5000 results); on cap overflow
- * the backend returns BULK_EDIT_TOO_MANY_IDS, which we surface inline.
- */
-async function editAllMatching() {
-	if (editingAll) return;
-	editingAll = true;
-	editAllError = null;
-	try {
-		const params = buildSearchParams({
-			q: data.q || '',
-			from: data.from || '',
-			to: data.to || '',
-			senderId: data.senderId || '',
-			receiverId: data.receiverId || '',
-			tags: data.tags || [],
-			sort: '',
-			dir: '',
-			tagQ: data.tagQ || '',
-			tagOp: (data.tagOp as 'AND' | 'OR') || 'AND'
-		});
-		params.delete('sort');
-		params.delete('dir');
-		const res = await fetch(`/api/documents/ids?${params.toString()}`);
-		if (!res.ok) {
-			const backend = await parseBackendError(res);
-			editAllError = getErrorMessage(backend?.code);
-			return;
-		}
-		const ids: string[] = await res.json();
-		bulkSelectionStore.setAll(ids);
-		await goto('/documents/bulk-edit');
-	} catch {
-		editAllError = m.bulk_edit_all_x_failed();
-	} finally {
-		editingAll = false;
-	}
-}
-
 // Keep local filter state in sync with server data after navigation completes.
 // Guard q: skip overwrite while the user is actively typing.
 $effect(() => {
@@ -206,13 +159,7 @@ $effect(() => {
 	<title>{m.nav_documents()} – Familienarchiv</title>
 </svelte:head>
 
-<!-- Reserve bottom padding when the bulk-selection bar is visible so the
-	 sticky bar does not occlude the last document row or the pagination
-	 controls (WCAG 1.4.10 / 2.4.7). -->
-<main
-	class="mx-auto max-w-7xl px-4 py-8 font-sans sm:px-6 lg:px-8"
-	class:pb-32={bulkSelectionStore.size > 0 && data.canWrite}
->
+<main class="mx-auto max-w-7xl px-4 py-8 font-sans sm:px-6 lg:px-8">
 	<h1 class="sr-only">{m.nav_documents()}</h1>
 
 	<SearchFilterBar
@@ -234,25 +181,6 @@ $effect(() => {
 		onblur={() => (qFocused = false)}
 	/>
 
-	{#if data.canWrite && data.totalElements > 0}
-		<div class="mb-2 flex flex-col items-end gap-1">
-			<button
-				type="button"
-				onclick={editAllMatching}
-				disabled={editingAll}
-				class="inline-flex items-center gap-1 text-sm font-medium text-ink-2 transition-colors hover:text-ink disabled:opacity-50"
-				data-testid="bulk-edit-all-x"
-			>
-				{m.bulk_edit_all_x({ count: data.totalElements })}
-			</button>
-			{#if editAllError}
-				<p role="alert" class="text-xs text-danger" data-testid="bulk-edit-all-x-error">
-					{editAllError}
-				</p>
-			{/if}
-		</div>
-	{/if}
-
 	<DocumentList
 		items={data.items}
 		total={data.totalElements}
@@ -264,5 +192,3 @@ $effect(() => {
 
 	<Pagination page={data.pageNumber} totalPages={data.totalPages} makeHref={buildPageHref} />
 </main>
-
-<BulkSelectionBar canWrite={data.canWrite} />

frontend/src/routes/documents/bulk-edit/+page.server.ts

@@ -1,12 +0,0 @@
-import { redirect } from '@sveltejs/kit';
-
-export async function load({ locals }: { locals: App.Locals }) {
-	// Defensive: a UserGroup row with NULL permissions returns undefined here
-	// rather than throwing on .includes() — treat that as "not WRITE_ALL".
-	const canWrite =
-		locals.user?.groups?.some(
-			(g: { permissions?: string[] }) => g.permissions?.includes('WRITE_ALL') ?? false
-		) ?? false;
-	if (!canWrite) throw redirect(303, '/documents');
-	return { canWrite };
-}

frontend/src/routes/documents/bulk-edit/+page.svelte

@@ -1,84 +0,0 @@
-<script lang="ts">
-import { onMount } from 'svelte';
-import { goto } from '$app/navigation';
-import { bulkSelectionStore } from '$lib/stores/bulkSelection.svelte';
-import BulkDocumentEditLayout, {
-	type BulkEditEntry
-} from '$lib/components/document/BulkDocumentEditLayout.svelte';
-import { getErrorMessage, parseBackendError } from '$lib/errors';
-import { m } from '$lib/paraglide/messages.js';
-
-let entries = $state<BulkEditEntry[]>([]);
-let loading = $state(true);
-let error = $state<string | null>(null);
-
-onMount(async () => {
-	const ids = Array.from(bulkSelectionStore.ids);
-	if (ids.length === 0) {
-		// Skip the loading flash on the empty-store redirect path — the user
-		// is bouncing back to /documents in the next tick.
-		loading = false;
-		await goto('/documents');
-		return;
-	}
-	try {
-		const res = await fetch('/api/documents/batch-metadata', {
-			method: 'POST',
-			headers: { 'Content-Type': 'application/json' },
-			body: JSON.stringify({ ids })
-		});
-		if (!res.ok) {
-			const backend = await parseBackendError(res);
-			error = getErrorMessage(backend?.code);
-			loading = false;
-			return;
-		}
-		const summaries = (await res.json()) as BulkEditEntry[];
-		entries = summaries;
-	} catch {
-		error = getErrorMessage(undefined);
-	} finally {
-		loading = false;
-	}
-});
-</script>
-
-<svelte:head>
-	<title>{m.bulk_edit_title()} – Familienarchiv</title>
-</svelte:head>
-
-{#if loading}
-	<div
-		class="flex h-full items-center justify-center gap-3 p-12 text-sm text-ink-2"
-		role="status"
-		aria-live="polite"
-		data-testid="bulk-edit-loading"
-	>
-		<svg
-			class="h-5 w-5 animate-spin text-ink-3"
-			xmlns="http://www.w3.org/2000/svg"
-			fill="none"
-			viewBox="0 0 24 24"
-			aria-hidden="true"
-		>
-			<circle class="opacity-25" cx="12" cy="12" r="10" stroke="currentColor" stroke-width="4"
-			></circle>
-			<path
-				class="opacity-75"
-				fill="currentColor"
-				d="M4 12a8 8 0 018-8V0C5.373 0 0 5.373 0 12h4zm2 5.291A7.962 7.962 0 014 12H0c0 3.042 1.135 5.824 3 7.938l3-2.647z"
-			></path>
-		</svg>
-		<span>{m.bulk_edit_loading()}</span>
-	</div>
-{:else if error}
-	<div
-		role="alert"
-		class="m-6 rounded-sm border border-danger/40 bg-danger/10 p-4 text-sm text-danger"
-		data-testid="bulk-edit-page-error"
-	>
-		{error}
-	</div>
-{:else if entries.length > 0}
-	<BulkDocumentEditLayout mode="edit" initialEditEntries={entries} />
-{/if}

frontend/src/routes/documents/bulk-edit/page.server.spec.ts

@@ -1,61 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { load } from './+page.server';
-
-describe('/documents/bulk-edit +page.server.ts', () => {
-	it('redirects to /documents when user lacks WRITE_ALL', async () => {
-		const locals = { user: { groups: [{ permissions: ['READ_ALL'] }] } };
-		try {
-			// @ts-expect-error — partial event shape sufficient for this guard
-			await load({ locals });
-			throw new Error('expected redirect to be thrown');
-		} catch (e) {
-			const err = e as { status?: number; location?: string };
-			expect(err.status).toBe(303);
-			expect(err.location).toBe('/documents');
-		}
-	});
-
-	it('redirects when user has no groups', async () => {
-		const locals = { user: { groups: [] } };
-		try {
-			// @ts-expect-error — partial event shape sufficient for this guard
-			await load({ locals });
-			throw new Error('expected redirect');
-		} catch (e) {
-			expect((e as { status?: number }).status).toBe(303);
-		}
-	});
-
-	it('redirects when no user is logged in', async () => {
-		const locals = {};
-		try {
-			// @ts-expect-error — partial event shape sufficient for this guard
-			await load({ locals });
-			throw new Error('expected redirect');
-		} catch (e) {
-			expect((e as { status?: number }).status).toBe(303);
-		}
-	});
-
-	it('returns canWrite=true for a WRITE_ALL user', async () => {
-		const locals = { user: { groups: [{ permissions: ['WRITE_ALL', 'READ_ALL'] }] } };
-		// @ts-expect-error — partial event shape sufficient for this guard
-		const result = await load({ locals });
-		expect(result).toEqual({ canWrite: true });
-	});
-
-	it('redirects when a group has no permissions array (defensive)', async () => {
-		// Sara C7 — a UserGroup row with NULL permissions used to throw on
-		// .includes(); the guard now treats that case as "not WRITE_ALL".
-		const locals = {
-			user: { groups: [{ permissions: undefined as unknown as string[] }] }
-		};
-		try {
-			// @ts-expect-error — partial event shape sufficient for this guard
-			await load({ locals });
-			throw new Error('expected redirect');
-		} catch (e) {
-			expect((e as { status?: number }).status).toBe(303);
-		}
-	});
-});

frontend/src/routes/enrich/+page.server.ts

@@ -19,5 +19,5 @@ export async function load({
 
 	const documents = result.response.ok ? (result.data ?? []) : [];
 
-	return { documents, canWrite };
+	return { documents };
 }

frontend/src/routes/enrich/+page.svelte

@@ -1,19 +1,14 @@
 <script lang="ts">
 import { m } from '$lib/paraglide/messages.js';
 import BackButton from '$lib/components/BackButton.svelte';
-import BulkSelectionBar from '$lib/components/document/BulkSelectionBar.svelte';
-import { bulkSelectionStore } from '$lib/stores/bulkSelection.svelte';
 
 let { data } = $props();
 
 const documents = $derived(data.documents);
 const count = $derived(documents.length);
-const canWrite = $derived(data.canWrite);
 </script>
 
-<!-- Reserve bottom padding when the bulk-selection bar is visible so the
-	 sticky bar does not occlude the last document row (WCAG 1.4.10). -->
-<div class="mx-auto max-w-4xl px-4 py-10" class:pb-32={bulkSelectionStore.size > 0 && canWrite}>
+<div class="mx-auto max-w-4xl px-4 py-10">
 	<!-- Back Link -->
 	<BackButton />
 
@@ -66,24 +61,8 @@ const canWrite = $derived(data.canWrite);
 		<div class="border border-line bg-surface shadow-sm">
 			<ul class="divide-y divide-line-2">
 				{#each documents as doc (doc.id)}
-					<li class="group relative transition-colors duration-200 hover:bg-muted">
-						<a href="/enrich/{doc.id}" class="absolute inset-0 z-0 block" aria-label={doc.title}
-						></a>
-						<div class="pointer-events-none relative z-10 flex items-center justify-between p-6">
-							{#if canWrite}
-								<label
-									class="pointer-events-auto mr-4 flex min-h-[44px] min-w-[44px] flex-shrink-0 cursor-pointer items-center"
-									data-testid="bulk-select-checkbox"
-								>
-									<input
-										type="checkbox"
-										class="h-5 w-5 cursor-pointer accent-brand-navy"
-										checked={bulkSelectionStore.has(doc.id)}
-										onchange={() => bulkSelectionStore.toggle(doc.id)}
-										aria-label={m.bulk_edit_select_document({ title: doc.title })}
-									/>
-								</label>
-							{/if}
+					<li class="group transition-colors duration-200 hover:bg-muted">
+						<a href="/enrich/{doc.id}" class="flex items-center justify-between p-6">
 							<div class="min-w-0 flex-1">
 								<p class="font-serif text-lg font-medium text-ink group-hover:underline">
 									{doc.title}
@@ -95,12 +74,10 @@ const canWrite = $derived(data.canWrite);
 								aria-hidden="true"
 								class="ml-4 h-5 w-5 shrink-0 opacity-30 transition-opacity group-hover:opacity-70"
 							/>
-						</div>
+						</a>
 					</li>
 				{/each}
 			</ul>
 		</div>
 	{/if}
 </div>
-
-<BulkSelectionBar canWrite={canWrite} />

frontend/src/routes/hilfe/transkription/+page.svelte

@@ -13,22 +13,18 @@ const rules = [
 		icon: '✗',
 		title: m.richtlinien_rule_durchgestrichen_title(),
 		body: m.richtlinien_rule_durchgestrichen_body(),
-		beispielInput: 'der Text',
-		beispielInputStrike: true,
 		beispielOutput: '[durchgestrichen: der Text]'
 	},
 	{
 		icon: 'ſ',
 		title: m.richtlinien_rule_langes_s_title(),
 		body: m.richtlinien_rule_langes_s_body(),
-		beispielInput: 'ſtraße',
-		beispielOutput: 'straße'
+		beispielOutput: 's'
 	},
 	{
 		icon: '?',
 		title: m.richtlinien_rule_name_title(),
 		body: m.richtlinien_rule_name_body(),
-		beispielInput: 'Müller',
 		beispielOutput: '[Müller?]'
 	},
 	{
@@ -50,7 +46,7 @@ const klaerungChips = [
 	<title>{m.richtlinien_title()} — Familienarchiv</title>
 </svelte:head>
 
-<main class="mx-auto max-w-2xl px-4 py-10 font-serif">
+<div class="mx-auto max-w-2xl px-4 py-10 font-serif">
 	<!-- Title -->
 	<h1 class="mb-4 text-3xl font-bold text-ink">{m.richtlinien_title()}</h1>
 
@@ -65,23 +61,10 @@ const klaerungChips = [
 			target="_blank"
 			rel="noopener noreferrer"
 			referrerpolicy="no-referrer"
-			aria-label="{m.richtlinien_wiki_link()} — {m.common_opens_new_tab()}"
-			class="inline-flex items-center gap-1.5 font-sans text-sm font-medium text-ink underline decoration-brand-mint decoration-[1.5px] underline-offset-[3px]"
+			class="inline-flex items-center gap-1 font-sans text-sm font-medium text-ink underline decoration-brand-mint decoration-[1.5px] underline-offset-[3px]"
 		>
 			{m.richtlinien_wiki_link()}
-			<svg
-				xmlns="http://www.w3.org/2000/svg"
-				viewBox="0 0 20 20"
-				fill="currentColor"
-				class="h-3.5 w-3.5 shrink-0"
-				aria-hidden="true"
-			>
-				<path
-					fill-rule="evenodd"
-					d="M4.25 5.5a.75.75 0 0 0-.75.75v8.5c0 .414.336.75.75.75h8.5a.75.75 0 0 0 .75-.75v-4a.75.75 0 0 1 1.5 0v4A2.25 2.25 0 0 1 12.75 17h-8.5A2.25 2.25 0 0 1 2 14.75v-8.5A2.25 2.25 0 0 1 4.25 4h5a.75.75 0 0 1 0 1.5h-5Zm6.75-3a.75.75 0 0 1 .75-.75h3.5a.75.75 0 0 1 .75.75v3.5a.75.75 0 0 1-1.5 0V3.56l-4.22 4.22a.75.75 0 0 1-1.06-1.06l4.22-4.22H11a.75.75 0 0 1-.75-.75Z"
-					clip-rule="evenodd"
-				/>
-			</svg>
+			<span class="new-tab ml-1 text-[11px] text-ink-3">({m.common_opens_new_tab()})</span>
 		</a>
 	</div>
 
@@ -95,8 +78,6 @@ const klaerungChips = [
 				icon={rule.icon}
 				title={rule.title}
 				body={rule.body}
-				beispielInput={rule.beispielInput}
-				beispielInputStrike={rule.beispielInputStrike}
 				beispielOutput={rule.beispielOutput}
 				beispielLabel={m.richtlinien_beispiel_label()}
 			/>
@@ -121,10 +102,10 @@ const klaerungChips = [
 
 	<!-- Closing card -->
 	<div class="border-brand-sand rounded-sm border bg-white p-6 shadow-sm">
-		<h3 class="mb-2 font-serif text-lg font-bold text-ink">{m.richtlinien_closing_title()}</h3>
+		<h2 class="mb-2 font-serif text-lg font-bold text-ink">{m.richtlinien_closing_title()}</h2>
 		<p class="font-serif text-sm leading-relaxed text-ink-2">{m.richtlinien_closing_body()}</p>
 	</div>
-</main>
+</div>
 
 <style>
 @media print {
@@ -132,6 +113,10 @@ const klaerungChips = [
 		display: none;
 	}
 
+	.new-tab {
+		display: none;
+	}
+
 	@page {
 		margin: 1.5cm;
 	}

frontend/src/routes/hilfe/transkription/+page.ts

@@ -1,3 +1 @@
-// Safe: handleAuth in hooks.server.ts redirects unauthenticated requests
-// before prerendered HTML is visible.
 export const prerender = true;

frontend/src/routes/layout.css

@@ -66,9 +66,6 @@
 	/* Focus ring — keyboard focus indicator, mode-aware (navy in light, mint in dark) */
 	--color-focus-ring: var(--c-focus-ring);
 
-	/* Parchment — warm background for code/example blocks inside cards */
-	--color-parchment: var(--c-parchment);
-
 	/* Danger — destructive action color */
 	--color-danger: var(--c-danger);
 	--color-danger-fg: var(--c-danger-fg);
@@ -125,9 +122,6 @@
 	--c-danger: #c0392b;
 	--c-danger-fg: #ffffff;
 
-	/* Parchment — warm near-white for example blocks (light mode: cream #FAF8F1) */
-	--c-parchment: #faf8f1;
-
 	/* Tag color tokens — decorative dot colors on tag chips */
 	--c-tag-sage: #5a8a6a;
 	--c-tag-sienna: #a0522d;
@@ -209,9 +203,6 @@
 		--c-danger: #e55347;
 		--c-danger-fg: #ffffff;
 
-		/* Parchment — subtle surface shift for example blocks on dark navy */
-		--c-parchment: #041828;
-
 		/* Tag color tokens — lighter for visibility on dark backgrounds */
 		--c-tag-sage: #7abf8a;
 		--c-tag-sienna: #cc7050;
@@ -276,9 +267,6 @@
 	--c-danger: #e55347;
 	--c-danger-fg: #ffffff;
 
-	/* Parchment — subtle surface shift for example blocks on dark navy */
-	--c-parchment: #041828;
-
 	/* Tag color tokens — lighter for visibility on dark backgrounds */
 	--c-tag-sage: #7abf8a;
 	--c-tag-sienna: #cc7050;