From 50a358b294807ee8d03d1ce089e18a0bbe61dcaa Mon Sep 17 00:00:00 2001
From: Jennie Robinson Faber <jennie@jenniefaber.com>
Date: Thu, 9 Apr 2026 23:52:00 +0100
Subject: [PATCH] feat(wiki): add batch tag remove mode to admin wiki page

Add add/remove toggle to batch tag picker. Clean up unused requireAdmin
import from wiki sync route.
---
 app/pages/admin/wiki.vue           | 37 ++++++++++++++++++++----------
 server/api/admin/wiki/sync.post.js |  1 -
 tests/server/api/wiki-sync.test.js |  4 ----
 3 files changed, 25 insertions(+), 17 deletions(-)

diff --git a/app/pages/admin/wiki.vue b/app/pages/admin/wiki.vue
index 3edeff9..a8c3358 100644
--- a/app/pages/admin/wiki.vue
+++ b/app/pages/admin/wiki.vue
@@ -58,13 +58,17 @@
         Select all in "{{ collectionFilter }}"
       </button>
       <div class="batch-tag-picker">
-        <select v-model="batchTagToAdd" aria-label="Tag to add" class="batch-select">
-          <option value="">Add tag...</option>
+        <select v-model="batchTagMode" aria-label="Batch tag mode" class="batch-select batch-mode-select">
+          <option value="add">Add</option>
+          <option value="remove">Remove</option>
+        </select>
+        <select v-model="batchTagSelected" :aria-label="batchTagMode === 'add' ? 'Tag to add' : 'Tag to remove'" class="batch-select">
+          <option value="">Select tag...</option>
           <option v-for="tag in availableTags" :key="tag.slug" :value="tag.slug">{{ tag.label }}</option>
         </select>
         <button
           class="btn btn-primary"
-          :disabled="!batchTagToAdd || batchApplying"
+          :disabled="!batchTagSelected || batchApplying"
           @click="applyBatchTag"
         >
           {{ batchApplying ? 'Applying...' : 'Apply' }}
@@ -402,27 +406,31 @@ const saveArticleTags = async () => {
 }
 
 // ---- Batch Tagging ----
-const batchTagToAdd = ref('')
+const batchTagMode = ref('add')
+const batchTagSelected = ref('')
 const batchApplying = ref(false)
 
 const applyBatchTag = async () => {
-  if (!batchTagToAdd.value || !selectedIds.value.length) return
+  if (!batchTagSelected.value || !selectedIds.value.length) return
   batchApplying.value = true
   try {
+    const body = { articleIds: selectedIds.value }
+    if (batchTagMode.value === 'add') {
+      body.addTags = [batchTagSelected.value]
+    } else {
+      body.removeTags = [batchTagSelected.value]
+    }
     const result = await $fetch('/api/admin/wiki/batch-tag', {
       method: 'POST',
-      body: {
-        articleIds: selectedIds.value,
-        addTags: [batchTagToAdd.value],
-      },
+      body,
     })
     await refresh()
+    const action = batchTagMode.value === 'add' ? 'added to' : 'removed from'
     toast.add({
-      title: 'Batch tag applied',
-      description: `${result.modified} articles updated`,
+      title: `Tag ${action} ${result.modified} articles`,
       color: 'green',
     })
-    batchTagToAdd.value = ''
+    batchTagSelected.value = ''
     selectedIds.value = []
   } catch (err) {
     toast.add({
@@ -553,6 +561,11 @@ const applyBatchTag = async () => {
   outline: none;
 }
 
+.batch-mode-select {
+  width: auto;
+  min-width: 80px;
+}
+
 /* ---- TABLE ---- */
 .table-wrap {
   padding: 0 28px 24px;
diff --git a/server/api/admin/wiki/sync.post.js b/server/api/admin/wiki/sync.post.js
index c0b9dd2..5295ea0 100644
--- a/server/api/admin/wiki/sync.post.js
+++ b/server/api/admin/wiki/sync.post.js
@@ -1,6 +1,5 @@
 import WikiArticle from '../../../models/wikiArticle.js'
 import { connectDB } from '../../../utils/mongoose.js'
-import { requireAdmin } from '../../../utils/auth.js'
 import { fetchAllDocuments, extractSummary } from '../../../utils/outline.js'
 
 export default defineEventHandler(async (event) => {
diff --git a/tests/server/api/wiki-sync.test.js b/tests/server/api/wiki-sync.test.js
index ce19882..fdf8a2d 100644
--- a/tests/server/api/wiki-sync.test.js
+++ b/tests/server/api/wiki-sync.test.js
@@ -8,16 +8,12 @@ vi.mock('../../../server/models/wikiArticle.js', () => ({
   }
 }))
 vi.mock('../../../server/utils/mongoose.js', () => ({ connectDB: vi.fn() }))
-vi.mock('../../../server/utils/auth.js', () => ({
-  requireAdmin: vi.fn()
-}))
 vi.mock('../../../server/utils/outline.js', () => ({
   fetchAllDocuments: vi.fn(),
   extractSummary: vi.fn((text) => text || '')
 }))
 
 import WikiArticle from '../../../server/models/wikiArticle.js'
-import { requireAdmin } from '../../../server/utils/auth.js'
 import { fetchAllDocuments, extractSummary } from '../../../server/utils/outline.js'
 import syncHandler from '../../../server/api/admin/wiki/sync.post.js'
 import { createMockEvent } from '../helpers/createMockEvent.js'