From a081d8d0285126a209f367078325939521515a9b Mon Sep 17 00:00:00 2001
From: alistair3149 <alistair31494322@gmail.com>
Date: Wed, 22 Jul 2026 12:43:09 -0400
Subject: [PATCH 1/6] Rename EditSummary component to SummaryAction
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The component backs the save/create dialogs today and will also back the
delete confirmation dialogs, so the edit-specific name no longer fits.

Pure mechanical rename: the component symbol, its file, its LESS class
prefix, and the public-api export. The neowiki-edit-summary-* message
keys are left unchanged — they remain the component's default label.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../LayoutEditor/LayoutEditorDialog.vue       |  4 +--
 .../LayoutsPage/LayoutCreatorDialog.vue       |  4 +--
 .../components/LayoutsPage/LayoutsPage.vue    |  4 +--
 .../MappingsPage/MappingCreatorDialog.vue     | 12 ++++----
 .../components/MappingsPage/MappingsPage.vue  |  4 +--
 .../SchemaEditor/SchemaEditorDialog.vue       |  4 +--
 .../SchemasPage/SchemaCreatorDialog.vue       |  4 +--
 .../components/SchemasPage/SchemasPage.vue    |  4 +--
 .../SubjectCreator/SubjectCreatorDialog.vue   |  4 +--
 .../SubjectEditor/SubjectEditorDialog.vue     |  4 +--
 .../SubjectsManager/SubjectsManagerPage.vue   |  4 +--
 .../{EditSummary.vue => SummaryAction.vue}    | 12 ++++----
 resources/ext.neowiki/src/public-api.ts       |  2 +-
 .../MappingsPage/MappingCreatorDialog.spec.ts |  6 ++--
 .../MappingsPage/MappingsPage.spec.ts         |  2 +-
 .../SchemaEditor/SchemaEditorDialog.spec.ts   | 14 ++++-----
 .../SchemasPage/SchemaCreatorDialog.spec.ts   | 14 ++++-----
 .../SchemasPage/SchemasPage.spec.ts           |  2 +-
 .../SubjectCreatorDialog.spec.ts              | 28 ++++++++---------
 .../SubjectEditor/SubjectEditorDialog.spec.ts | 30 +++++++++----------
 ...tSummary.spec.ts => SummaryAction.spec.ts} |  8 ++---
 21 files changed, 85 insertions(+), 85 deletions(-)
 rename resources/ext.neowiki/src/components/common/{EditSummary.vue => SummaryAction.vue} (91%)
 rename resources/ext.neowiki/tests/components/common/{EditSummary.spec.ts => SummaryAction.spec.ts} (90%)

diff --git a/resources/ext.neowiki/src/components/LayoutEditor/LayoutEditorDialog.vue b/resources/ext.neowiki/src/components/LayoutEditor/LayoutEditorDialog.vue
index 4114e0ac..f7ba966a 100644
--- a/resources/ext.neowiki/src/components/LayoutEditor/LayoutEditorDialog.vue
+++ b/resources/ext.neowiki/src/components/LayoutEditor/LayoutEditorDialog.vue
@@ -14,7 +14,7 @@
 			/>
 
 			<template #footer>
-				<EditSummary
+				<SummaryAction
 					:help-text="$i18n( 'neowiki-edit-summary-help-text-layout' ).text()"
 					:save-button-label="$i18n( 'neowiki-save-layout' ).text()"
 					:save-disabled="!hasChanged"
@@ -34,7 +34,7 @@
 <script setup lang="ts">
 import LayoutEditor from '@/components/LayoutEditor/LayoutEditor.vue';
 import type { LayoutEditorExposes } from '@/components/LayoutEditor/LayoutEditor.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import CloseConfirmationDialog from '@/components/common/CloseConfirmationDialog.vue';
 import { CdxDialog } from '@wikimedia/codex';
 import { Layout } from '@/domain/Layout.ts';
diff --git a/resources/ext.neowiki/src/components/LayoutsPage/LayoutCreatorDialog.vue b/resources/ext.neowiki/src/components/LayoutsPage/LayoutCreatorDialog.vue
index d7adb70e..c0bd76cf 100644
--- a/resources/ext.neowiki/src/components/LayoutsPage/LayoutCreatorDialog.vue
+++ b/resources/ext.neowiki/src/components/LayoutsPage/LayoutCreatorDialog.vue
@@ -13,7 +13,7 @@
 			/>
 
 			<template #footer>
-				<EditSummary
+				<SummaryAction
 					help-text=""
 					:save-button-label="$i18n( 'neowiki-layout-creator-save' ).text()"
 					:save-disabled="!hasChanged"
@@ -35,7 +35,7 @@ import { ref, watch } from 'vue';
 import { CdxDialog } from '@wikimedia/codex';
 import LayoutCreator from './LayoutCreator.vue';
 import type { LayoutCreatorExposes } from './LayoutCreator.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import CloseConfirmationDialog from '@/components/common/CloseConfirmationDialog.vue';
 import { Layout } from '@/domain/Layout.ts';
 import type { LayoutSaveHandler } from '@/components/LayoutEditor/LayoutEditorDialog.vue';
diff --git a/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue b/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue
index 76040bc5..c8fb680f 100644
--- a/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue
+++ b/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue
@@ -96,7 +96,7 @@
 			</I18nSlot>
 
 			<template #footer>
-				<EditSummary
+				<SummaryAction
 					help-text=""
 					:save-button-label="$i18n( 'neowiki-layout-delete-confirm-delete' ).text()"
 					:save-disabled="false"
@@ -118,7 +118,7 @@ import { useLayoutStore } from '@/stores/LayoutStore.ts';
 import { Layout } from '@/domain/Layout.ts';
 import LayoutCreatorDialog from './LayoutCreatorDialog.vue';
 import LayoutEditorDialog from '@/components/LayoutEditor/LayoutEditorDialog.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import I18nSlot from '@/components/common/I18nSlot.vue';
 
 const paginationSizeOptions: { value: number }[] = [
diff --git a/resources/ext.neowiki/src/components/MappingsPage/MappingCreatorDialog.vue b/resources/ext.neowiki/src/components/MappingsPage/MappingCreatorDialog.vue
index 76120c0e..eaa1f59f 100644
--- a/resources/ext.neowiki/src/components/MappingsPage/MappingCreatorDialog.vue
+++ b/resources/ext.neowiki/src/components/MappingsPage/MappingCreatorDialog.vue
@@ -22,8 +22,8 @@
 		</CdxField>
 
 		<template #footer>
-			<EditSummary
-				ref="editSummaryRef"
+			<SummaryAction
+				ref="summaryActionRef"
 				help-text=""
 				:save-button-label="$i18n( 'neowiki-mapping-creator-save' ).text()"
 				:save-disabled="saving"
@@ -37,7 +37,7 @@
 import { ref, watch } from 'vue';
 import { CdxDialog, CdxField, CdxTextInput } from '@wikimedia/codex';
 import type { ValidationStatusType } from '@wikimedia/codex';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 
 // The reserved projection name (RdfPageProjector::PROJECTION). The backend rejects a Mapping:Native
 // save (MappingName via MappingContentHandler::validateSave, case-insensitively); this client check
@@ -61,16 +61,16 @@ const mappingName = ref( '' );
 const nameError = ref( '' );
 const nameStatus = ref<ValidationStatusType>( 'default' );
 const saving = ref( false );
-const editSummaryRef = ref<InstanceType<typeof EditSummary> | null>( null );
+const summaryActionRef = ref<InstanceType<typeof SummaryAction> | null>( null );
 
 function close(): void {
 	emit( 'update:open', false );
 }
 
-// Enter in the name field saves through the EditSummary so the currently entered summary is used,
+// Enter in the name field saves through the SummaryAction so the currently entered summary is used,
 // exactly as clicking its save button does.
 function onNameEnter(): void {
-	editSummaryRef.value?.submit();
+	summaryActionRef.value?.submit();
 }
 
 function onDialogUpdateOpen( value: boolean ): void {
diff --git a/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue b/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue
index 418d87db..6f4f3d07 100644
--- a/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue
+++ b/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue
@@ -89,7 +89,7 @@
 			</I18nSlot>
 
 			<template #footer>
-				<EditSummary
+				<SummaryAction
 					help-text=""
 					:save-button-label="$i18n( 'neowiki-mapping-delete-confirm-delete' ).text()"
 					:save-disabled="false"
@@ -108,7 +108,7 @@ import { cdxIconAdd, cdxIconEdit, cdxIconTrash } from '@wikimedia/codex-icons';
 import { NeoWikiExtension } from '@/NeoWikiExtension.ts';
 import { useMappingPermissions } from '@/composables/useMappingPermissions.ts';
 import MappingCreatorDialog from './MappingCreatorDialog.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import I18nSlot from '@/components/common/I18nSlot.vue';
 
 const paginationSizeOptions: { value: number }[] = [
diff --git a/resources/ext.neowiki/src/components/SchemaEditor/SchemaEditorDialog.vue b/resources/ext.neowiki/src/components/SchemaEditor/SchemaEditorDialog.vue
index 778dfbfe..746b0c7f 100644
--- a/resources/ext.neowiki/src/components/SchemaEditor/SchemaEditorDialog.vue
+++ b/resources/ext.neowiki/src/components/SchemaEditor/SchemaEditorDialog.vue
@@ -16,7 +16,7 @@
 			/>
 
 			<template #footer>
-				<EditSummary
+				<SummaryAction
 					:help-text="$i18n( 'neowiki-edit-summary-help-text-schema' ).text()"
 					:save-button-label="$i18n( 'neowiki-save-schema' ).text()"
 					:save-disabled="!hasChanged"
@@ -35,7 +35,7 @@
 
 <script setup lang="ts">
 import SchemaEditor, { SchemaEditorExposes } from '@/components/SchemaEditor/SchemaEditor.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import CloseConfirmationDialog from '@/components/common/CloseConfirmationDialog.vue';
 import { CdxDialog } from '@wikimedia/codex';
 import { Schema } from '@/domain/Schema.ts';
diff --git a/resources/ext.neowiki/src/components/SchemasPage/SchemaCreatorDialog.vue b/resources/ext.neowiki/src/components/SchemasPage/SchemaCreatorDialog.vue
index 14738791..c53cd1a6 100644
--- a/resources/ext.neowiki/src/components/SchemasPage/SchemaCreatorDialog.vue
+++ b/resources/ext.neowiki/src/components/SchemasPage/SchemaCreatorDialog.vue
@@ -15,7 +15,7 @@
 			/>
 
 			<template #footer>
-				<EditSummary
+				<SummaryAction
 					help-text=""
 					:save-button-label="$i18n( 'neowiki-schema-creator-save' ).text()"
 					:save-disabled="!hasChanged"
@@ -37,7 +37,7 @@ import { ref, watch } from 'vue';
 import { CdxDialog } from '@wikimedia/codex';
 import SchemaCreator from '@/components/SchemaCreator/SchemaCreator.vue';
 import type { SchemaCreatorExposes } from '@/components/SchemaCreator/SchemaCreator.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import CloseConfirmationDialog from '@/components/common/CloseConfirmationDialog.vue';
 import { Schema } from '@/domain/Schema.ts';
 import { useSchemaStore } from '@/stores/SchemaStore.ts';
diff --git a/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue b/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue
index 96b76658..e724c723 100644
--- a/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue
+++ b/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue
@@ -91,7 +91,7 @@
 			</I18nSlot>
 
 			<template #footer>
-				<EditSummary
+				<SummaryAction
 					help-text=""
 					:save-button-label="$i18n( 'neowiki-schema-delete-confirm-delete' ).text()"
 					:save-disabled="false"
@@ -114,7 +114,7 @@ import { Schema } from '@/domain/Schema.ts';
 import type { SchemaSummary } from '@/application/SchemaLookup.ts';
 import SchemaCreatorDialog from './SchemaCreatorDialog.vue';
 import SchemaEditorDialog from '@/components/SchemaEditor/SchemaEditorDialog.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import I18nSlot from '@/components/common/I18nSlot.vue';
 
 const paginationSizeOptions: { value: number }[] = [
diff --git a/resources/ext.neowiki/src/components/SubjectCreator/SubjectCreatorDialog.vue b/resources/ext.neowiki/src/components/SubjectCreator/SubjectCreatorDialog.vue
index 969074ff..4f6f4d23 100644
--- a/resources/ext.neowiki/src/components/SubjectCreator/SubjectCreatorDialog.vue
+++ b/resources/ext.neowiki/src/components/SubjectCreator/SubjectCreatorDialog.vue
@@ -137,7 +137,7 @@
 			v-else-if="selectedSchemaName"
 			#footer
 		>
-			<EditSummary
+			<SummaryAction
 				help-text=""
 				:save-button-label="$i18n( 'neowiki-subject-creator-save' ).text()"
 				:save-disabled="!hasChanged"
@@ -175,7 +175,7 @@ import { CdxMessage } from '@wikimedia/codex';
 import SubjectEditor from '@/components/SubjectEditor/SubjectEditor.vue';
 import SchemaCreator from '@/components/SchemaCreator/SchemaCreator.vue';
 import type { SchemaCreatorExposes } from '@/components/SchemaCreator/SchemaCreator.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import SchemaPicker from '@/components/common/SchemaPicker.vue';
 import CloseConfirmationDialog from '@/components/common/CloseConfirmationDialog.vue';
 import SchemaAbandonmentDialog from '@/components/SubjectCreator/SchemaAbandonmentDialog.vue';
diff --git a/resources/ext.neowiki/src/components/SubjectEditor/SubjectEditorDialog.vue b/resources/ext.neowiki/src/components/SubjectEditor/SubjectEditorDialog.vue
index b5de7554..84c472dc 100644
--- a/resources/ext.neowiki/src/components/SubjectEditor/SubjectEditorDialog.vue
+++ b/resources/ext.neowiki/src/components/SubjectEditor/SubjectEditorDialog.vue
@@ -77,7 +77,7 @@
 
 			<!-- TODO: We should make this into a component-->
 			<template #footer>
-				<EditSummary
+				<SummaryAction
 					help-text=""
 					:save-button-label="$i18n( 'neowiki-subject-editor-save' ).text()"
 					:save-disabled="!hasChanged"
@@ -106,7 +106,7 @@
 <script setup lang="ts">
 import { ref, nextTick, computed, watch } from 'vue';
 import SubjectEditor from '@/components/SubjectEditor/SubjectEditor.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import I18nSlot from '@/components/common/I18nSlot.vue';
 import { CdxButton, CdxDialog, CdxIcon, CdxMessage } from '@wikimedia/codex';
 import { cdxIconClose } from '@wikimedia/codex-icons';
diff --git a/resources/ext.neowiki/src/components/SubjectsManager/SubjectsManagerPage.vue b/resources/ext.neowiki/src/components/SubjectsManager/SubjectsManagerPage.vue
index 3cde0132..01a0d11b 100644
--- a/resources/ext.neowiki/src/components/SubjectsManager/SubjectsManagerPage.vue
+++ b/resources/ext.neowiki/src/components/SubjectsManager/SubjectsManagerPage.vue
@@ -430,7 +430,7 @@
 				<strong>{{ deletingLabel }}</strong>
 			</I18nSlot>
 			<template #footer>
-				<EditSummary
+				<SummaryAction
 					help-text=""
 					:save-button-label="$i18n( 'neowiki-managesubjects-delete-confirm-button' ).text()"
 					:save-disabled="false"
@@ -472,7 +472,7 @@ import { Schema } from '@/domain/Schema';
 import { SubjectId } from '@/domain/SubjectId';
 import SubjectCreatorDialog from '@/components/SubjectCreator/SubjectCreatorDialog.vue';
 import SubjectEditorDialog from '@/components/SubjectEditor/SubjectEditorDialog.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import I18nSlot from '@/components/common/I18nSlot.vue';
 import SubjectStatementsView from '@/components/SubjectsManager/SubjectStatementsView.vue';
 import { setPendingNotification } from '@/presentation/PendingNotification.ts';
diff --git a/resources/ext.neowiki/src/components/common/EditSummary.vue b/resources/ext.neowiki/src/components/common/SummaryAction.vue
similarity index 91%
rename from resources/ext.neowiki/src/components/common/EditSummary.vue
rename to resources/ext.neowiki/src/components/common/SummaryAction.vue
index 9823e387..52b44801 100644
--- a/resources/ext.neowiki/src/components/common/EditSummary.vue
+++ b/resources/ext.neowiki/src/components/common/SummaryAction.vue
@@ -1,10 +1,10 @@
 <template>
-	<div class="ext-neowiki-edit-summary">
+	<div class="ext-neowiki-summary-action">
 		<CdxAccordion @toggle="onAccordionToggle">
 			<template #title>
-				<span class="ext-neowiki-edit-summary__label">
+				<span class="ext-neowiki-summary-action__label">
 					{{ $i18n( 'neowiki-edit-summary-label' ).text() }}
-					<span class="ext-neowiki-edit-summary__optional-flag">
+					<span class="ext-neowiki-summary-action__optional-flag">
 						{{ $i18n( 'cdx-label-optional-flag' ).text() }}
 					</span>
 				</span>
@@ -28,12 +28,12 @@
 
 		<div
 			v-if="props.helpText"
-			class="ext-neowiki-edit-summary__help-text"
+			class="ext-neowiki-summary-action__help-text"
 		>
 			{{ props.helpText }}
 		</div>
 
-		<div class="ext-neowiki-edit-summary__actions">
+		<div class="ext-neowiki-summary-action__actions">
 			<CdxButton
 				:action="props.saveButtonAction"
 				weight="primary"
@@ -97,7 +97,7 @@ defineExpose( { submit: onSaveClick } );
 <style lang="less">
 @import ( reference ) '@wikimedia/codex-design-tokens/theme-wikimedia-ui.less';
 
-.ext-neowiki-edit-summary {
+.ext-neowiki-summary-action {
 	.cdx-accordion {
 		border-bottom: 0;
 
diff --git a/resources/ext.neowiki/src/public-api.ts b/resources/ext.neowiki/src/public-api.ts
index 8d906d45..d3fa4c31 100644
--- a/resources/ext.neowiki/src/public-api.ts
+++ b/resources/ext.neowiki/src/public-api.ts
@@ -151,7 +151,7 @@ export { default as UrlDisplay } from './components/Value/UrlDisplay.vue';
 export { default as UrlInput } from './components/Value/UrlInput.vue';
 export { default as Infobox } from './components/Views/Infobox.vue';
 export { default as CloseConfirmationDialog } from './components/common/CloseConfirmationDialog.vue';
-export { default as EditSummary } from './components/common/EditSummary.vue';
+export { default as SummaryAction } from './components/common/SummaryAction.vue';
 export { default as I18nSlot } from './components/common/I18nSlot.vue';
 export { default as NeoMultiLookupInput } from './components/common/NeoMultiLookupInput.vue';
 export { default as NeoMultiTextInput } from './components/common/NeoMultiTextInput.vue';
diff --git a/resources/ext.neowiki/tests/components/MappingsPage/MappingCreatorDialog.spec.ts b/resources/ext.neowiki/tests/components/MappingsPage/MappingCreatorDialog.spec.ts
index 1fa33d42..ab95e731 100644
--- a/resources/ext.neowiki/tests/components/MappingsPage/MappingCreatorDialog.spec.ts
+++ b/resources/ext.neowiki/tests/components/MappingsPage/MappingCreatorDialog.spec.ts
@@ -12,7 +12,7 @@ const CdxDialogStub = {
 	emits: [ 'update:open' ],
 };
 
-const EditSummaryStub = {
+const SummaryActionStub = {
 	template: '<button class="edit-summary-stub" @click="$emit( \'save\', \'\' )">save</button>',
 	emits: [ 'save' ],
 };
@@ -24,7 +24,7 @@ function mountDialog(): VueWrapper {
 			mocks: { $i18n: createI18nMock() },
 			stubs: {
 				CdxDialog: CdxDialogStub,
-				EditSummary: EditSummaryStub,
+				SummaryAction: SummaryActionStub,
 			},
 		},
 	} );
@@ -35,7 +35,7 @@ async function enterNameAndSave( wrapper: VueWrapper, name: string ): Promise<vo
 	await wrapper.find( '.edit-summary-stub' ).trigger( 'click' );
 }
 
-// Uses the real EditSummary so pressing Enter exercises its exposed submit
+// Uses the real SummaryAction so pressing Enter exercises its exposed submit
 // (which saves with whatever summary is currently entered).
 function mountDialogWithRealEditSummary(): VueWrapper {
 	return mount( MappingCreatorDialog, {
diff --git a/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts b/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts
index bb17ddde..b8096711 100644
--- a/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts
+++ b/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts
@@ -83,7 +83,7 @@ function mountComponent( summaries: MappingSummary[] = [] ): VueWrapper {
 			mocks: { $i18n: createI18nMock() },
 			stubs: {
 				MappingCreatorDialog: MappingCreatorDialogStub,
-				EditSummary: true,
+				SummaryAction: true,
 				I18nSlot: true,
 				CdxIcon: true,
 			},
diff --git a/resources/ext.neowiki/tests/components/SchemaEditor/SchemaEditorDialog.spec.ts b/resources/ext.neowiki/tests/components/SchemaEditor/SchemaEditorDialog.spec.ts
index 71b7f078..d25e76b8 100644
--- a/resources/ext.neowiki/tests/components/SchemaEditor/SchemaEditorDialog.spec.ts
+++ b/resources/ext.neowiki/tests/components/SchemaEditor/SchemaEditorDialog.spec.ts
@@ -2,7 +2,7 @@ import { mount, VueWrapper, flushPromises } from '@vue/test-utils';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 import SchemaEditorDialog from '@/components/SchemaEditor/SchemaEditorDialog.vue';
 import SchemaEditor from '@/components/SchemaEditor/SchemaEditor.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import CloseConfirmationDialog from '@/components/common/CloseConfirmationDialog.vue';
 import { CdxDialog } from '@wikimedia/codex';
 import { Schema } from '@/domain/Schema.ts';
@@ -21,7 +21,7 @@ const SchemaEditorStub = {
 	},
 };
 
-const EditSummaryStub = {
+const SummaryActionStub = {
 	template: '<div class="edit-summary-stub"></div>',
 	props: [ 'helpText', 'saveButtonLabel', 'saveDisabled' ],
 	emits: [ 'save' ],
@@ -42,7 +42,7 @@ describe( 'SchemaEditorDialog', () => {
 
 	const stubs = {
 		SchemaEditor: SchemaEditorStub,
-		EditSummary: EditSummaryStub,
+		SummaryAction: SummaryActionStub,
 		CloseConfirmationDialog: CloseConfirmationDialogStub,
 	};
 
@@ -65,7 +65,7 @@ describe( 'SchemaEditorDialog', () => {
 			const wrapper = mountComponent();
 			await flushPromises();
 
-			expect( wrapper.findComponent( EditSummary ).props( 'saveDisabled' ) ).toBe( true );
+			expect( wrapper.findComponent( SummaryAction ).props( 'saveDisabled' ) ).toBe( true );
 		} );
 
 		it( 'enables save after a change is made', async () => {
@@ -74,7 +74,7 @@ describe( 'SchemaEditorDialog', () => {
 
 			await wrapper.findComponent( SchemaEditor ).vm.$emit( 'change' );
 
-			expect( wrapper.findComponent( EditSummary ).props( 'saveDisabled' ) ).toBe( false );
+			expect( wrapper.findComponent( SummaryAction ).props( 'saveDisabled' ) ).toBe( false );
 		} );
 
 		it( 'disables save again when dialog reopens', async () => {
@@ -82,12 +82,12 @@ describe( 'SchemaEditorDialog', () => {
 			await flushPromises();
 
 			await wrapper.findComponent( SchemaEditor ).vm.$emit( 'change' );
-			expect( wrapper.findComponent( EditSummary ).props( 'saveDisabled' ) ).toBe( false );
+			expect( wrapper.findComponent( SummaryAction ).props( 'saveDisabled' ) ).toBe( false );
 
 			await wrapper.setProps( { open: false } );
 			await wrapper.setProps( { open: true } );
 
-			expect( wrapper.findComponent( EditSummary ).props( 'saveDisabled' ) ).toBe( true );
+			expect( wrapper.findComponent( SummaryAction ).props( 'saveDisabled' ) ).toBe( true );
 		} );
 	} );
 
diff --git a/resources/ext.neowiki/tests/components/SchemasPage/SchemaCreatorDialog.spec.ts b/resources/ext.neowiki/tests/components/SchemasPage/SchemaCreatorDialog.spec.ts
index bf85a8a8..8fee7063 100644
--- a/resources/ext.neowiki/tests/components/SchemasPage/SchemaCreatorDialog.spec.ts
+++ b/resources/ext.neowiki/tests/components/SchemasPage/SchemaCreatorDialog.spec.ts
@@ -2,7 +2,7 @@ import { mount, VueWrapper, flushPromises } from '@vue/test-utils';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { createPinia, setActivePinia } from 'pinia';
 import SchemaCreatorDialog from '@/components/SchemasPage/SchemaCreatorDialog.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import { useSchemaStore } from '@/stores/SchemaStore.ts';
 import { createI18nMock, setupMwMock } from '../../VueTestHelpers.ts';
 import { CdxDialog } from '@wikimedia/codex';
@@ -37,7 +37,7 @@ const SchemaCreatorStub = {
 	},
 };
 
-const EditSummaryStub = {
+const SummaryActionStub = {
 	template: '<div class="edit-summary-stub"><button class="save-button" @click="$emit( \'save\', \'\' )">Save</button></div>',
 	props: [ 'helpText', 'saveButtonLabel', 'saveDisabled' ],
 	emits: [ 'save' ],
@@ -66,7 +66,7 @@ describe( 'SchemaCreatorDialog', () => {
 				plugins: [ pinia ],
 				stubs: {
 					SchemaCreator: SchemaCreatorStub,
-					EditSummary: EditSummaryStub,
+					SummaryAction: SummaryActionStub,
 					CloseConfirmationDialog: CloseConfirmationDialogStub,
 					CdxDialog: CdxDialogStub,
 					CdxIcon: true,
@@ -100,7 +100,7 @@ describe( 'SchemaCreatorDialog', () => {
 
 		( wrapper.findComponent( SchemaCreatorStub ).vm as any ).setStubValid( false );
 
-		await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+		await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 		await flushPromises();
 
 		expect( schemaStore.saveSchema ).not.toHaveBeenCalled();
@@ -109,7 +109,7 @@ describe( 'SchemaCreatorDialog', () => {
 	it( 'saves schema and emits created on success', async () => {
 		const wrapper = mountComponent();
 
-		await wrapper.findComponent( EditSummary ).vm.$emit( 'save', 'My summary' );
+		await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', 'My summary' );
 		await flushPromises();
 
 		expect( schemaStore.saveSchema ).toHaveBeenCalledWith(
@@ -130,7 +130,7 @@ describe( 'SchemaCreatorDialog', () => {
 	it( 'uses default summary when none provided', async () => {
 		const wrapper = mountComponent();
 
-		await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+		await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 		await flushPromises();
 
 		expect( schemaStore.saveSchema ).toHaveBeenCalledWith(
@@ -147,7 +147,7 @@ describe( 'SchemaCreatorDialog', () => {
 
 		const wrapper = mountComponent();
 
-		await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+		await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 		await flushPromises();
 
 		expect( mw.notify ).toHaveBeenCalledWith(
diff --git a/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts b/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts
index fea18029..e1ef6c2d 100644
--- a/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts
+++ b/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts
@@ -92,7 +92,7 @@ function mountComponent( summaries: unknown[] = [] ): VueWrapper {
 			stubs: {
 				SchemaCreatorDialog: SchemaCreatorDialogStub,
 				SchemaEditorDialog: SchemaEditorDialogStub,
-				EditSummary: true,
+				SummaryAction: true,
 				I18nSlot: true,
 				CdxIcon: true,
 			},
diff --git a/resources/ext.neowiki/tests/components/SubjectCreator/SubjectCreatorDialog.spec.ts b/resources/ext.neowiki/tests/components/SubjectCreator/SubjectCreatorDialog.spec.ts
index 6f06d1d7..4aae9398 100644
--- a/resources/ext.neowiki/tests/components/SubjectCreator/SubjectCreatorDialog.spec.ts
+++ b/resources/ext.neowiki/tests/components/SubjectCreator/SubjectCreatorDialog.spec.ts
@@ -4,7 +4,7 @@ import { ref } from 'vue';
 import SubjectCreatorDialog from '@/components/SubjectCreator/SubjectCreatorDialog.vue';
 import SchemaPicker from '@/components/common/SchemaPicker.vue';
 import SchemaCreator from '@/components/SchemaCreator/SchemaCreator.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import { createPinia, setActivePinia } from 'pinia';
 import { useSubjectStore } from '@/stores/SubjectStore.ts';
 import { useSchemaStore } from '@/stores/SchemaStore.ts';
@@ -83,7 +83,7 @@ const SchemaCreatorStub = {
 	},
 };
 
-const EditSummaryStub = {
+const SummaryActionStub = {
 	template: '<div class="edit-summary-stub"><button class="save-button" @click="$emit( \'save\', \'\' )">Save</button></div>',
 	props: [ 'helpText', 'saveButtonLabel', 'saveDisabled' ],
 	emits: [ 'save' ],
@@ -132,7 +132,7 @@ describe( 'SubjectCreatorDialog', () => {
 					SchemaPicker: SchemaPickerStub,
 					SubjectEditor: SubjectEditorStub,
 					SchemaCreator: SchemaCreatorStub,
-					EditSummary: EditSummaryStub,
+					SummaryAction: SummaryActionStub,
 					CloseConfirmationDialog: CloseConfirmationDialogStub,
 					SchemaAbandonmentDialog: SchemaAbandonmentDialogStub,
 					CdxButton: true,
@@ -297,7 +297,7 @@ describe( 'SubjectCreatorDialog', () => {
 		await wrapper.findComponent( SchemaPicker ).vm.$emit( 'select', SCHEMA_NAME );
 		await flushPromises();
 
-		await wrapper.findComponent( EditSummary ).vm.$emit( 'save', 'test summary' );
+		await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', 'test summary' );
 		await flushPromises();
 
 		expect( subjectStore.createMainSubject ).toHaveBeenCalledWith(
@@ -315,7 +315,7 @@ describe( 'SubjectCreatorDialog', () => {
 		await wrapper.findComponent( SchemaPicker ).vm.$emit( 'select', SCHEMA_NAME );
 		await flushPromises();
 
-		await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+		await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 		await flushPromises();
 
 		expect( subjectStore.createMainSubject ).toHaveBeenCalledWith(
@@ -333,7 +333,7 @@ describe( 'SubjectCreatorDialog', () => {
 		await wrapper.findComponent( SchemaPicker ).vm.$emit( 'select', SCHEMA_NAME );
 		await flushPromises();
 
-		await wrapper.findComponent( EditSummary ).vm.$emit( 'save', 'test summary' );
+		await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', 'test summary' );
 		await flushPromises();
 
 		expect( subjectStore.createChildSubject ).toHaveBeenCalledWith(
@@ -354,7 +354,7 @@ describe( 'SubjectCreatorDialog', () => {
 		await wrapper.findComponent( SchemaPicker ).vm.$emit( 'select', SCHEMA_NAME );
 		await flushPromises();
 
-		await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+		await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 		await flushPromises();
 
 		expect( mw.storage.session.set ).toHaveBeenCalledWith( 'neowiki-subject-creator-success', '1' );
@@ -371,7 +371,7 @@ describe( 'SubjectCreatorDialog', () => {
 		await wrapper.findComponent( SchemaPicker ).vm.$emit( 'select', SCHEMA_NAME );
 		await flushPromises();
 
-		await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+		await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 		await flushPromises();
 
 		expect( mw.notify ).toHaveBeenCalledWith(
@@ -393,7 +393,7 @@ describe( 'SubjectCreatorDialog', () => {
 		await labelInput.setValue( '' );
 		await flushPromises();
 
-		await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+		await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 		await flushPromises();
 
 		expect( subjectStore.createMainSubject ).not.toHaveBeenCalled();
@@ -485,7 +485,7 @@ describe( 'SubjectCreatorDialog', () => {
 
 			await clickContinue( wrapper );
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', 'Created subject' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', 'Created subject' );
 			await flushPromises();
 
 			expect( schemaStore.saveSchema ).toHaveBeenCalledWith(
@@ -511,7 +511,7 @@ describe( 'SubjectCreatorDialog', () => {
 			await switchToNewSchema( wrapper );
 			await clickContinue( wrapper );
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', 'My edit summary' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', 'My edit summary' );
 			await flushPromises();
 
 			expect( schemaStore.saveSchema ).toHaveBeenCalledWith(
@@ -526,7 +526,7 @@ describe( 'SubjectCreatorDialog', () => {
 			await switchToNewSchema( wrapper );
 			await clickContinue( wrapper );
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 			await flushPromises();
 
 			expect( schemaStore.saveSchema ).toHaveBeenCalledWith(
@@ -545,7 +545,7 @@ describe( 'SubjectCreatorDialog', () => {
 
 			await clickContinue( wrapper );
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 			await flushPromises();
 
 			expect( mw.notify ).toHaveBeenCalledWith(
@@ -916,7 +916,7 @@ describe( 'SubjectCreatorDialog', () => {
 			await flushPromises();
 			await wrapper.findComponent( SchemaPicker ).vm.$emit( 'select', SCHEMA_NAME );
 			await flushPromises();
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 			await flushPromises();
 		}
 
diff --git a/resources/ext.neowiki/tests/components/SubjectEditor/SubjectEditorDialog.spec.ts b/resources/ext.neowiki/tests/components/SubjectEditor/SubjectEditorDialog.spec.ts
index b8602f02..329ce28b 100644
--- a/resources/ext.neowiki/tests/components/SubjectEditor/SubjectEditorDialog.spec.ts
+++ b/resources/ext.neowiki/tests/components/SubjectEditor/SubjectEditorDialog.spec.ts
@@ -13,7 +13,7 @@ import { useSubjectStore } from '@/stores/SubjectStore.ts';
 import { Service } from '@/NeoWikiServices.ts';
 import SchemaEditorDialog from '@/components/SchemaEditor/SchemaEditorDialog.vue';
 import SubjectEditor from '@/components/SubjectEditor/SubjectEditor.vue';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import CloseConfirmationDialog from '@/components/common/CloseConfirmationDialog.vue';
 import { CdxDialog } from '@wikimedia/codex';
 import { createI18nMock, setupMwMock } from '../../VueTestHelpers.ts';
@@ -32,7 +32,7 @@ const SubjectEditorStub = {
 	},
 };
 
-const EditSummaryStub = {
+const SummaryActionStub = {
 	template: '<div class="edit-summary-stub"></div>',
 	props: [ 'helpText', 'saveButtonLabel', 'saveDisabled' ],
 	emits: [ 'save' ],
@@ -153,7 +153,7 @@ describe( 'SubjectEditorDialog', () => {
 	const saveButtonTestStubs = {
 		SubjectEditor: SubjectEditorStub,
 		SchemaEditorDialog: true,
-		EditSummary: EditSummaryStub,
+		SummaryAction: SummaryActionStub,
 	};
 
 	describe( 'Save button', () => {
@@ -161,7 +161,7 @@ describe( 'SubjectEditorDialog', () => {
 			const wrapper = mountComponent( true, saveButtonTestStubs );
 			await flushPromises();
 
-			expect( wrapper.findComponent( EditSummary ).props( 'saveDisabled' ) ).toBe( true );
+			expect( wrapper.findComponent( SummaryAction ).props( 'saveDisabled' ) ).toBe( true );
 		} );
 
 		it( 'enables save after a change is made', async () => {
@@ -170,7 +170,7 @@ describe( 'SubjectEditorDialog', () => {
 
 			await wrapper.findComponent( SubjectEditor ).vm.$emit( 'change' );
 
-			expect( wrapper.findComponent( EditSummary ).props( 'saveDisabled' ) ).toBe( false );
+			expect( wrapper.findComponent( SummaryAction ).props( 'saveDisabled' ) ).toBe( false );
 		} );
 
 		it( 'disables save again when dialog reopens', async () => {
@@ -178,12 +178,12 @@ describe( 'SubjectEditorDialog', () => {
 			await flushPromises();
 
 			await wrapper.findComponent( SubjectEditor ).vm.$emit( 'change' );
-			expect( wrapper.findComponent( EditSummary ).props( 'saveDisabled' ) ).toBe( false );
+			expect( wrapper.findComponent( SummaryAction ).props( 'saveDisabled' ) ).toBe( false );
 
 			await wrapper.setProps( { open: false } );
 			await wrapper.setProps( { open: true } );
 
-			expect( wrapper.findComponent( EditSummary ).props( 'saveDisabled' ) ).toBe( true );
+			expect( wrapper.findComponent( SummaryAction ).props( 'saveDisabled' ) ).toBe( true );
 		} );
 	} );
 
@@ -278,7 +278,7 @@ describe( 'SubjectEditorDialog', () => {
 	const validationTestStubs = {
 		SubjectEditor: SubjectEditorStub,
 		SchemaEditorDialog: true,
-		EditSummary: EditSummaryStub,
+		SummaryAction: SummaryActionStub,
 	};
 
 	describe( 'ValidationFailedError handling', () => {
@@ -293,7 +293,7 @@ describe( 'SubjectEditorDialog', () => {
 			const wrapper = mountComponent( true, validationTestStubs, onSave );
 			await flushPromises();
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 			await flushPromises();
 
 			const passedViolations = wrapper.findComponent( SubjectEditor ).props( 'serverViolations' ) as SubjectViolation[];
@@ -313,7 +313,7 @@ describe( 'SubjectEditorDialog', () => {
 			const wrapper = mountComponent( true, validationTestStubs, onSave );
 			await flushPromises();
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 			await flushPromises();
 
 			expect( wrapper.emitted( 'update:open' ) ).toBeUndefined();
@@ -330,7 +330,7 @@ describe( 'SubjectEditorDialog', () => {
 			const wrapper = mountComponent( true, validationTestStubs, onSave );
 			await flushPromises();
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 			await flushPromises();
 
 			expect( ( mw.notify as Mock ).mock.calls ).toContainEqual( [
@@ -350,7 +350,7 @@ describe( 'SubjectEditorDialog', () => {
 			const wrapper = mountComponent( true, validationTestStubs, onSave );
 			await flushPromises();
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 			await flushPromises();
 
 			expect( wrapper.find( '.ext-neowiki-subject-editor__form-errors' ).exists() ).toBe( true );
@@ -371,7 +371,7 @@ describe( 'SubjectEditorDialog', () => {
 			const wrapper = mountComponent( true, validationTestStubs, onSave );
 			await flushPromises();
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 			await flushPromises();
 
 			expect( ( wrapper.findComponent( SubjectEditor ).props( 'serverViolations' ) as SubjectViolation[] ) ).toHaveLength( 1 );
@@ -391,7 +391,7 @@ describe( 'SubjectEditorDialog', () => {
 			const wrapper = mountComponent( true, validationTestStubs, onSave );
 			await flushPromises();
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 			await flushPromises();
 
 			expect( ( mw.notify as Mock ).mock.calls ).toContainEqual( [
@@ -442,7 +442,7 @@ describe( 'SubjectEditorDialog', () => {
 			const wrapper = mountComponent( true, validationTestStubs, onSave );
 			await flushPromises();
 
-			await wrapper.findComponent( EditSummary ).vm.$emit( 'save', '' );
+			await wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
 			await flushPromises();
 
 			const passed = wrapper.findComponent( SubjectEditor ).props( 'serverViolations' ) as SubjectViolation[];
diff --git a/resources/ext.neowiki/tests/components/common/EditSummary.spec.ts b/resources/ext.neowiki/tests/components/common/SummaryAction.spec.ts
similarity index 90%
rename from resources/ext.neowiki/tests/components/common/EditSummary.spec.ts
rename to resources/ext.neowiki/tests/components/common/SummaryAction.spec.ts
index 362f4b64..dd6c2f0d 100644
--- a/resources/ext.neowiki/tests/components/common/EditSummary.spec.ts
+++ b/resources/ext.neowiki/tests/components/common/SummaryAction.spec.ts
@@ -1,19 +1,19 @@
 import { mount, VueWrapper } from '@vue/test-utils';
 import { beforeEach, describe, expect, it } from 'vitest';
-import EditSummary from '@/components/common/EditSummary.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
 import { CdxButton, CdxIcon } from '@wikimedia/codex';
 import { cdxIconCheck, cdxIconTrash } from '@wikimedia/codex-icons';
 import { createI18nMock, setupMwMock } from '../../VueTestHelpers.ts';
 
 const $i18n = createI18nMock();
 
-describe( 'EditSummary', () => {
+describe( 'SummaryAction', () => {
 	beforeEach( () => {
 		setupMwMock( { functions: [ 'message', 'msg' ] } );
 	} );
 
-	function mountComponent( props: Partial<InstanceType<typeof EditSummary>[ '$props' ]> = {} ): VueWrapper {
-		return mount( EditSummary, {
+	function mountComponent( props: Partial<InstanceType<typeof SummaryAction>[ '$props' ]> = {} ): VueWrapper {
+		return mount( SummaryAction, {
 			props: {
 				helpText: '',
 				saveButtonLabel: 'Save',

From b2e0e1e4ea86b5b79a3b212704238b996c4847bc Mon Sep 17 00:00:00 2001
From: alistair3149 <alistair31494322@gmail.com>
Date: Wed, 22 Jul 2026 12:46:13 -0400
Subject: [PATCH 2/6] Add optional label/placeholder props to SummaryAction

The field is hardwired to the "Edit summary" wording. To let the delete
confirmation dialog reuse this component and label the field a "reason",
expose optional `label` and `placeholder` props that fall back to the
existing edit-summary messages, so every current caller is unchanged.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../src/components/common/SummaryAction.vue   | 14 +++++++---
 .../components/common/SummaryAction.spec.ts   | 27 +++++++++++++++++++
 2 files changed, 37 insertions(+), 4 deletions(-)

diff --git a/resources/ext.neowiki/src/components/common/SummaryAction.vue b/resources/ext.neowiki/src/components/common/SummaryAction.vue
index 52b44801..79d73353 100644
--- a/resources/ext.neowiki/src/components/common/SummaryAction.vue
+++ b/resources/ext.neowiki/src/components/common/SummaryAction.vue
@@ -3,7 +3,7 @@
 		<CdxAccordion @toggle="onAccordionToggle">
 			<template #title>
 				<span class="ext-neowiki-summary-action__label">
-					{{ $i18n( 'neowiki-edit-summary-label' ).text() }}
+					{{ props.label || $i18n( 'neowiki-edit-summary-label' ).text() }}
 					<span class="ext-neowiki-summary-action__optional-flag">
 						{{ $i18n( 'cdx-label-optional-flag' ).text() }}
 					</span>
@@ -15,13 +15,13 @@
 				:hide-label="true"
 			>
 				<template #label>
-					{{ $i18n( 'neowiki-edit-summary-label' ).text() }}
+					{{ props.label || $i18n( 'neowiki-edit-summary-label' ).text() }}
 				</template>
 
 				<CdxTextArea
 					ref="textAreaRef"
 					v-model="editSummary"
-					:placeholder="$i18n( 'neowiki-edit-summary-placeholder' ).text()"
+					:placeholder="props.placeholder || $i18n( 'neowiki-edit-summary-placeholder' ).text()"
 				/>
 			</CdxField>
 		</CdxAccordion>
@@ -60,10 +60,16 @@ const props = withDefaults(
 		saveDisabled: boolean;
 		saveButtonAction?: 'progressive' | 'destructive';
 		saveButtonIcon?: Icon;
+		label?: string;
+		placeholder?: string;
 	}>(),
 	{
 		saveButtonAction: 'progressive',
-		saveButtonIcon: () => cdxIconCheck
+		saveButtonIcon: () => cdxIconCheck,
+		// Default to the edit-summary wording; callers such as the delete dialog override these
+		// to label the field a "reason".
+		label: '',
+		placeholder: ''
 	}
 );
 
diff --git a/resources/ext.neowiki/tests/components/common/SummaryAction.spec.ts b/resources/ext.neowiki/tests/components/common/SummaryAction.spec.ts
index dd6c2f0d..7605b391 100644
--- a/resources/ext.neowiki/tests/components/common/SummaryAction.spec.ts
+++ b/resources/ext.neowiki/tests/components/common/SummaryAction.spec.ts
@@ -75,4 +75,31 @@ describe( 'SummaryAction', () => {
 		const icon = wrapper.findComponent( CdxIcon );
 		expect( icon.props( 'icon' ) ).toBe( cdxIconTrash );
 	} );
+
+	it( 'defaults the field label to the edit-summary message', () => {
+		const wrapper = mountComponent();
+
+		expect( wrapper.text() ).toContain( 'neowiki-edit-summary-label' );
+	} );
+
+	it( 'uses the provided label prop over the default', () => {
+		const wrapper = mountComponent( { label: 'Reason' } );
+
+		expect( wrapper.text() ).toContain( 'Reason' );
+		expect( wrapper.text() ).not.toContain( 'neowiki-edit-summary-label' );
+	} );
+
+	it( 'defaults the field placeholder to the edit-summary message', () => {
+		const wrapper = mountComponent();
+
+		expect( wrapper.find( 'textarea' ).attributes( 'placeholder' ) )
+			.toBe( 'neowiki-edit-summary-placeholder' );
+	} );
+
+	it( 'uses the provided placeholder prop over the default', () => {
+		const wrapper = mountComponent( { placeholder: 'Why are you deleting this?' } );
+
+		expect( wrapper.find( 'textarea' ).attributes( 'placeholder' ) )
+			.toBe( 'Why are you deleting this?' );
+	} );
 } );

From 777fd8e0b4263c573aff0a56732494c8e35a7682 Mon Sep 17 00:00:00 2001
From: alistair3149 <alistair31494322@gmail.com>
Date: Wed, 22 Jul 2026 12:53:20 -0400
Subject: [PATCH 3/6] Add DeletePageDialog with generic delete messages

Extract the page-deletion confirmation shared by Special:Schemas,
Special:Layouts, and Special:Mappings into a self-contained component: a
confirmation dialog wrapping SummaryAction (destructive action + trash
icon, "reason" label) that performs the delete via the API and notifies.

Introduce a generic neowiki-delete-* message set keyed on the page name,
replacing the need for per-entity delete wording. The three pages are
migrated onto this component in the following commit.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 extension.json                                |   8 ++
 i18n/en.json                                  |   9 ++
 i18n/qqq.json                                 |   9 ++
 .../components/common/DeletePageDialog.vue    |  78 +++++++++++++
 .../common/DeletePageDialog.spec.ts           | 103 ++++++++++++++++++
 5 files changed, 207 insertions(+)
 create mode 100644 resources/ext.neowiki/src/components/common/DeletePageDialog.vue
 create mode 100644 resources/ext.neowiki/tests/components/common/DeletePageDialog.spec.ts

diff --git a/extension.json b/extension.json
index 4a4757ed..96122017 100644
--- a/extension.json
+++ b/extension.json
@@ -457,6 +457,14 @@
 				"neowiki-edit-summary-label",
 				"neowiki-edit-summary-placeholder",
 				"neowiki-edit-summary-help-text-schema",
+				"neowiki-delete-confirm-title",
+				"neowiki-delete-confirm-message",
+				"neowiki-delete-confirm-button",
+				"neowiki-delete-reason-label",
+				"neowiki-delete-reason-placeholder",
+				"neowiki-delete-summary-default",
+				"neowiki-delete-success",
+				"neowiki-delete-error",
 				"neowiki-subject-editor-title",
 				"neowiki-subject-editor-save",
 				"neowiki-schema-editor-new-property",
diff --git a/i18n/en.json b/i18n/en.json
index 69d563df..56a93072 100644
--- a/i18n/en.json
+++ b/i18n/en.json
@@ -102,6 +102,15 @@
 	"neowiki-edit-summary-placeholder": "Describe what you changed",
 	"neowiki-edit-summary-help-text-schema": "Changes affect all Subjects using this Schema.",
 
+	"neowiki-delete-confirm-title": "Delete $1?",
+	"neowiki-delete-confirm-message": "The page $1 will be permanently deleted.",
+	"neowiki-delete-confirm-button": "Delete",
+	"neowiki-delete-reason-label": "Reason",
+	"neowiki-delete-reason-placeholder": "Reason for deletion",
+	"neowiki-delete-summary-default": "Deleted via the NeoWiki UI",
+	"neowiki-delete-success": "Deleted $1",
+	"neowiki-delete-error": "Failed to delete $1",
+
 	"neowiki-subject-editor-title": "Editing $1",
 	"neowiki-subject-editor-save": "Save changes",
 
diff --git a/i18n/qqq.json b/i18n/qqq.json
index 258ad370..1e423205 100644
--- a/i18n/qqq.json
+++ b/i18n/qqq.json
@@ -127,6 +127,15 @@
 	"neowiki-editing-layout": "Title of the dialog for editing a layout. $1 is the layout name.",
 	"neowiki-save-layout": "Label for the save button in the layout editor dialog.",
 	"neowiki-edit-summary-help-text-layout": "Help text shown below the edit summary in the layout editor dialog.",
+
+	"neowiki-delete-confirm-title": "Title of the confirmation dialog shown before deleting a NeoWiki page (Schema, Layout, or Mapping). $1 is the page name.",
+	"neowiki-delete-confirm-message": "Body of the delete confirmation dialog. $1 is the page name, shown in bold.",
+	"neowiki-delete-confirm-button": "Label for the button that confirms deletion in the delete confirmation dialog.",
+	"neowiki-delete-reason-label": "Label for the optional deletion-reason field in the delete confirmation dialog.",
+	"neowiki-delete-reason-placeholder": "Placeholder for the optional deletion-reason field in the delete confirmation dialog.",
+	"neowiki-delete-summary-default": "Default deletion reason recorded in the deletion log when the user enters none.",
+	"neowiki-delete-success": "Success notification shown after a page is deleted. $1 is the page name.",
+	"neowiki-delete-error": "Error notification title shown when deleting a page fails. $1 is the page name.",
 	"neowiki-layout-editor-description": "Label for the description field in the layout editor.",
 	"neowiki-layout-editor-description-placeholder": "Placeholder text for the description field in the layout editor.",
 	"neowiki-layout-editor-schema": "Label for the read-only schema field in the layout editor.",
diff --git a/resources/ext.neowiki/src/components/common/DeletePageDialog.vue b/resources/ext.neowiki/src/components/common/DeletePageDialog.vue
new file mode 100644
index 00000000..536d0ab6
--- /dev/null
+++ b/resources/ext.neowiki/src/components/common/DeletePageDialog.vue
@@ -0,0 +1,78 @@
+<template>
+	<CdxDialog
+		:open="props.open"
+		:title="$i18n( 'neowiki-delete-confirm-title', props.displayName ).text()"
+		:use-close-button="true"
+		@update:open="onUpdateOpen"
+	>
+		<I18nSlot message-key="neowiki-delete-confirm-message">
+			<strong>{{ props.displayName }}</strong>
+		</I18nSlot>
+
+		<template #footer>
+			<SummaryAction
+				help-text=""
+				:label="$i18n( 'neowiki-delete-reason-label' ).text()"
+				:placeholder="$i18n( 'neowiki-delete-reason-placeholder' ).text()"
+				:save-button-label="$i18n( 'neowiki-delete-confirm-button' ).text()"
+				save-button-action="destructive"
+				:save-button-icon="cdxIconTrash"
+				:save-disabled="false"
+				@save="executeDelete"
+			/>
+		</template>
+	</CdxDialog>
+</template>
+
+<script setup lang="ts">
+import { CdxDialog } from '@wikimedia/codex';
+import { cdxIconTrash } from '@wikimedia/codex-icons';
+import SummaryAction from '@/components/common/SummaryAction.vue';
+import I18nSlot from '@/components/common/I18nSlot.vue';
+
+const props = defineProps<{
+	open: boolean;
+	// The full title of the page to delete, e.g. `Schema:Person`.
+	pageTitle: string;
+	// The bare name shown in the confirmation message and notifications.
+	displayName: string;
+}>();
+
+const emit = defineEmits<{
+	'update:open': [ value: boolean ];
+	'deleted': [];
+}>();
+
+function onUpdateOpen( value: boolean ): void {
+	emit( 'update:open', value );
+}
+
+// The dialog closes as soon as the deletion is confirmed; success and failure are then surfaced
+// through notifications, matching the behaviour of the per-page dialogs this replaces.
+async function executeDelete( summary: string ): Promise<void> {
+	emit( 'update:open', false );
+
+	const reason = summary || mw.msg( 'neowiki-delete-summary-default' );
+
+	try {
+		const api = new mw.Api();
+		const token = await api.getEditToken();
+		await api.post( {
+			action: 'delete',
+			title: props.pageTitle,
+			reason: reason,
+			token: token
+		} );
+		mw.notify( mw.msg( 'neowiki-delete-success', props.displayName ), { type: 'success' } );
+		emit( 'deleted' );
+	} catch ( error ) {
+		mw.notify(
+			error instanceof Error ? error.message : String( error ),
+			{
+				title: mw.msg( 'neowiki-delete-error', props.displayName ),
+				type: 'error'
+			}
+		);
+	}
+}
+</script>
diff --git a/resources/ext.neowiki/tests/components/common/DeletePageDialog.spec.ts b/resources/ext.neowiki/tests/components/common/DeletePageDialog.spec.ts
new file mode 100644
index 00000000..19434d9d
--- /dev/null
+++ b/resources/ext.neowiki/tests/components/common/DeletePageDialog.spec.ts
@@ -0,0 +1,103 @@
+import { mount, VueWrapper, flushPromises } from '@vue/test-utils';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import DeletePageDialog from '@/components/common/DeletePageDialog.vue';
+import SummaryAction from '@/components/common/SummaryAction.vue';
+import { cdxIconTrash } from '@wikimedia/codex-icons';
+import { createI18nMock } from '../../VueTestHelpers.ts';
+
+const getEditTokenMock = vi.fn();
+const postMock = vi.fn();
+const notifyMock = vi.fn();
+
+const CdxDialogStub = {
+	template: '<div v-if="open" class="cdx-dialog-stub"><slot /><slot name="footer" /></div>',
+	props: [ 'open', 'title', 'useCloseButton' ],
+	emits: [ 'update:open' ],
+};
+
+const SummaryActionStub = {
+	template: '<button class="summary-action-stub" @click="$emit( \'save\', \'\' )">save</button>',
+	props: [ 'saveButtonAction', 'saveButtonIcon', 'saveButtonLabel', 'saveDisabled', 'label', 'placeholder', 'helpText' ],
+	emits: [ 'save' ],
+};
+
+function mountDialog( props: Record<string, unknown> = {} ): VueWrapper {
+	return mount( DeletePageDialog, {
+		props: {
+			open: true,
+			pageTitle: 'Schema:Person',
+			displayName: 'Person',
+			...props,
+		},
+		global: {
+			mocks: { $i18n: createI18nMock() },
+			stubs: {
+				CdxDialog: CdxDialogStub,
+				SummaryAction: SummaryActionStub,
+				I18nSlot: true,
+			},
+		},
+	} );
+}
+
+describe( 'DeletePageDialog', () => {
+	beforeEach( () => {
+		getEditTokenMock.mockReset().mockResolvedValue( 'token' );
+		postMock.mockReset().mockResolvedValue( {} );
+		notifyMock.mockReset();
+		vi.stubGlobal( 'mw', {
+			msg: vi.fn( ( key: string, ...params: string[] ) => ( params.length ? key + params.join( '' ) : key ) ),
+			notify: notifyMock,
+			Api: vi.fn( function ( this: { getEditToken: typeof getEditTokenMock; post: typeof postMock } ) {
+				this.getEditToken = getEditTokenMock;
+				this.post = postMock;
+			} ),
+		} );
+	} );
+
+	it( 'configures the confirm button as destructive with the trash icon', () => {
+		const action = mountDialog().findComponent( SummaryAction );
+
+		expect( action.props( 'saveButtonAction' ) ).toBe( 'destructive' );
+		expect( action.props( 'saveButtonIcon' ) ).toBe( cdxIconTrash );
+	} );
+
+	it( 'deletes the page with the entered reason and emits deleted on success', async () => {
+		const wrapper = mountDialog();
+
+		wrapper.findComponent( SummaryAction ).vm.$emit( 'save', 'spam page' );
+		await flushPromises();
+
+		expect( postMock ).toHaveBeenCalledWith( expect.objectContaining( {
+			action: 'delete',
+			title: 'Schema:Person',
+			reason: 'spam page',
+			token: 'token',
+		} ) );
+		expect( notifyMock ).toHaveBeenCalled();
+		expect( wrapper.emitted( 'deleted' ) ).toHaveLength( 1 );
+		expect( wrapper.emitted( 'update:open' ) ).toContainEqual( [ false ] );
+	} );
+
+	it( 'falls back to the default summary when no reason is entered', async () => {
+		const wrapper = mountDialog();
+
+		wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
+		await flushPromises();
+
+		expect( postMock ).toHaveBeenCalledWith( expect.objectContaining( {
+			reason: 'neowiki-delete-summary-default',
+		} ) );
+	} );
+
+	it( 'notifies an error and does not emit deleted when the delete fails', async () => {
+		postMock.mockRejectedValue( new Error( 'boom' ) );
+		const wrapper = mountDialog();
+
+		wrapper.findComponent( SummaryAction ).vm.$emit( 'save', '' );
+		await flushPromises();
+
+		expect( notifyMock ).toHaveBeenCalledWith( 'boom', expect.objectContaining( { type: 'error' } ) );
+		expect( wrapper.emitted( 'deleted' ) ).toBeUndefined();
+	} );
+} );

From 5b0ea459144b712e5383e37c70bb5cad2a379545 Mon Sep 17 00:00:00 2001
From: alistair3149 <alistair31494322@gmail.com>
Date: Wed, 22 Jul 2026 13:04:30 -0400
Subject: [PATCH 4/6] Migrate Schemas/Layouts/Mappings pages to
 DeletePageDialog

Replace the duplicated delete-confirmation dialog and executeDelete
logic in all three special pages with the shared DeletePageDialog, and
retire the 18 now-unused per-entity delete messages in favour of the
generic neowiki-delete-* set. The per-entity row-button aria-labels
(neowiki-<entity>-delete) stay, since the trigger button lives on each
page.

Also register neowiki-schema-delete, which was defined but never added
to the ResourceLoader module: the Schemas delete button aria-label (and,
via the old per-entity keys, its whole confirmation dialog) rendered raw
message keys. This is the reported "Special:Schemas is missing the i18n
messages" bug.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 extension.json                                | 13 +----
 i18n/en.json                                  | 18 -------
 i18n/qqq.json                                 | 18 -------
 .../components/LayoutsPage/LayoutsPage.vue    | 54 +++----------------
 .../components/MappingsPage/MappingsPage.vue  | 54 +++----------------
 .../components/SchemasPage/SchemasPage.vue    | 54 +++----------------
 .../MappingsPage/MappingsPage.spec.ts         | 13 +++--
 .../SchemasPage/SchemasPage.spec.ts           | 12 +++--
 8 files changed, 37 insertions(+), 199 deletions(-)

diff --git a/extension.json b/extension.json
index 96122017..9810b996 100644
--- a/extension.json
+++ b/extension.json
@@ -501,6 +501,7 @@
 				"neowiki-subject-creator-creating-schema",
 				"neowiki-subject-creator-back",
 				"neowiki-edit-schema",
+				"neowiki-schema-delete",
 				"neowiki-schema-editor-summary-default",
 				"neowiki-schema-editor-success",
 				"neowiki-schema-editor-error",
@@ -581,12 +582,6 @@
 				"neowiki-mapping-creator-summary-default",
 				"neowiki-edit-mapping",
 				"neowiki-mapping-delete",
-				"neowiki-mapping-delete-confirm-title",
-				"neowiki-mapping-delete-confirm-message",
-				"neowiki-mapping-delete-confirm-delete",
-				"neowiki-mapping-delete-summary-default",
-				"neowiki-mapping-delete-success",
-				"neowiki-mapping-delete-error",
 				"neowiki-layout-creator-title",
 				"neowiki-layout-creator-button",
 				"neowiki-layout-creator-name-field",
@@ -602,12 +597,6 @@
 				"neowiki-layout-creator-error",
 				"neowiki-layout-creator-summary-default",
 				"neowiki-layout-delete",
-				"neowiki-layout-delete-confirm-title",
-				"neowiki-layout-delete-confirm-message",
-				"neowiki-layout-delete-confirm-delete",
-				"neowiki-layout-delete-summary-default",
-				"neowiki-layout-delete-success",
-				"neowiki-layout-delete-error",
 				"neowiki-managesubjects-title",
 				"neowiki-managesubjects-not-applicable",
 				"neowiki-managesubjects-add-button",
diff --git a/i18n/en.json b/i18n/en.json
index 56a93072..bb9a54cc 100644
--- a/i18n/en.json
+++ b/i18n/en.json
@@ -197,12 +197,6 @@
 	"neowiki-schema-sidebar-all-schemas": "All schemas",
 
 	"neowiki-schema-delete": "Delete schema",
-	"neowiki-schema-delete-confirm-title": "Delete schema?",
-	"neowiki-schema-delete-confirm-message": "The schema $1 will be permanently deleted.",
-	"neowiki-schema-delete-confirm-delete": "Delete",
-	"neowiki-schema-delete-summary-default": "Delete schema via NeoWiki UI",
-	"neowiki-schema-delete-success": "Deleted $1 schema",
-	"neowiki-schema-delete-error": "Failed to delete $1 schema.",
 
 	"neowiki-close-confirmation-title": "Discard unsaved changes?",
 	"neowiki-close-confirmation-message": "Your changes have not been saved and will be lost.",
@@ -278,12 +272,6 @@
 	"neowiki-mapping-creator-summary-default": "Create mapping via NeoWiki UI",
 	"neowiki-edit-mapping": "Edit mapping",
 	"neowiki-mapping-delete": "Delete mapping",
-	"neowiki-mapping-delete-confirm-title": "Delete mapping?",
-	"neowiki-mapping-delete-confirm-message": "The mapping $1 will be permanently deleted.",
-	"neowiki-mapping-delete-confirm-delete": "Delete",
-	"neowiki-mapping-delete-summary-default": "Delete mapping via NeoWiki UI",
-	"neowiki-mapping-delete-success": "Deleted $1 mapping",
-	"neowiki-mapping-delete-error": "Failed to delete $1 mapping.",
 
 	"neowiki-layout-creator-title": "Create layout",
 	"neowiki-layout-creator-button": "Create",
@@ -301,12 +289,6 @@
 	"neowiki-layout-creator-summary-default": "Create layout via NeoWiki UI",
 
 	"neowiki-layout-delete": "Delete layout",
-	"neowiki-layout-delete-confirm-title": "Delete layout?",
-	"neowiki-layout-delete-confirm-message": "The layout $1 will be permanently deleted.",
-	"neowiki-layout-delete-confirm-delete": "Delete",
-	"neowiki-layout-delete-summary-default": "Delete layout via NeoWiki UI",
-	"neowiki-layout-delete-success": "Deleted $1 layout",
-	"neowiki-layout-delete-error": "Failed to delete $1 layout.",
 
 	"neowiki-managesubjects-tab": "Data",
 	"neowiki-managesubjects-title": "Subjects on $1",
diff --git a/i18n/qqq.json b/i18n/qqq.json
index 1e423205..e64a69f7 100644
--- a/i18n/qqq.json
+++ b/i18n/qqq.json
@@ -86,12 +86,6 @@
 	"neowiki-schema-sidebar-all-schemas": "Label for the link in the NeoWiki section of the sidebar on Schema pages that navigates to [[Special:Schemas]].",
 
 	"neowiki-schema-delete": "Aria label for the delete button on a Schema row in the schemas list.",
-	"neowiki-schema-delete-confirm-title": "Title of the confirmation dialog shown when deleting a schema.",
-	"neowiki-schema-delete-confirm-message": "Message in the delete confirmation dialog. $1 is the schema name.",
-	"neowiki-schema-delete-confirm-delete": "Label for the delete button in the schema deletion dialog.",
-	"neowiki-schema-delete-summary-default": "Default deletion reason used when deleting a schema via the UI.",
-	"neowiki-schema-delete-success": "Success notification shown after deleting a schema. $1 is the schema name.",
-	"neowiki-schema-delete-error": "Error notification title shown when deleting a schema fails. $1 is the schema name.",
 
 	"neowiki-close-confirmation-title": "Title of the confirmation dialog shown when closing a dialog with unsaved changes.",
 	"neowiki-close-confirmation-message": "Message in the confirmation dialog asking the user to confirm discarding unsaved changes.",
@@ -176,12 +170,6 @@
 	"neowiki-mapping-creator-summary-default": "Default edit summary used when creating a mapping via the UI.",
 	"neowiki-edit-mapping": "Aria label for the edit button on a Mapping row in the mappings list.",
 	"neowiki-mapping-delete": "Aria label for the delete button on a Mapping row in the mappings list.",
-	"neowiki-mapping-delete-confirm-title": "Title of the confirmation dialog shown when deleting a mapping.",
-	"neowiki-mapping-delete-confirm-message": "Message in the delete confirmation dialog. $1 is the mapping name.",
-	"neowiki-mapping-delete-confirm-delete": "Label for the delete button in the mapping deletion dialog.",
-	"neowiki-mapping-delete-summary-default": "Default deletion reason used when deleting a mapping via the UI.",
-	"neowiki-mapping-delete-success": "Success notification shown after deleting a mapping. $1 is the mapping name.",
-	"neowiki-mapping-delete-error": "Error notification title shown when deleting a mapping fails. $1 is the mapping name.",
 
 	"neowiki-layout-creator-title": "Title of the dialog for creating a new layout.",
 	"neowiki-layout-creator-button": "Label for the button that opens the layout creation dialog on [[Special:Layouts]].",
@@ -199,12 +187,6 @@
 	"neowiki-layout-creator-summary-default": "Default edit summary used when creating a layout via the UI.",
 
 	"neowiki-layout-delete": "Aria label for the delete button on a Layout row in the layouts list.",
-	"neowiki-layout-delete-confirm-title": "Title of the confirmation dialog shown when deleting a layout.",
-	"neowiki-layout-delete-confirm-message": "Message in the delete confirmation dialog. $1 is the layout name.",
-	"neowiki-layout-delete-confirm-delete": "Label for the delete button in the layout deletion dialog.",
-	"neowiki-layout-delete-summary-default": "Default deletion reason used when deleting a layout via the UI.",
-	"neowiki-layout-delete-success": "Success notification shown after deleting a layout. $1 is the layout name.",
-	"neowiki-layout-delete-error": "Error notification title shown when deleting a layout fails. $1 is the layout name.",
 	"neowiki-property-type-select": "Label for the select/dropdown property type shown in the property type selector.",
 	"neowiki-property-editor-min-exceeds-max": "Error shown in the schema editor when the minimum value exceeds the maximum value for a number property.",
 	"neowiki-property-editor-options": "Label for the options field in the select property type editor, where schema creators define the allowed values.",
diff --git a/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue b/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue
index c8fb680f..b218f8f9 100644
--- a/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue
+++ b/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue
@@ -85,31 +85,19 @@
 			@update:open="onEditorOpenChange"
 		/>
 
-		<CdxDialog
+		<DeletePageDialog
 			:open="isDeleteConfirmOpen"
-			:title="$i18n( 'neowiki-layout-delete-confirm-title' ).text()"
-			:use-close-button="true"
+			:page-title="`Layout:${ deletingLayoutName }`"
+			:display-name="deletingLayoutName"
 			@update:open="isDeleteConfirmOpen = $event"
-		>
-			<I18nSlot message-key="neowiki-layout-delete-confirm-message">
-				<strong>{{ deletingLayoutName }}</strong>
-			</I18nSlot>
-
-			<template #footer>
-				<SummaryAction
-					help-text=""
-					:save-button-label="$i18n( 'neowiki-layout-delete-confirm-delete' ).text()"
-					:save-disabled="false"
-					@save="executeDelete"
-				/>
-			</template>
-		</CdxDialog>
+			@deleted="fetchLayouts( lastOffset, pageSize )"
+		/>
 	</div>
 </template>
 
 <script setup lang="ts">
 import { ref, shallowRef, onMounted, nextTick } from 'vue';
-import { CdxButton, CdxDialog, CdxIcon, CdxTable } from '@wikimedia/codex';
+import { CdxButton, CdxIcon, CdxTable } from '@wikimedia/codex';
 import type { TableColumn } from '@wikimedia/codex';
 import { cdxIconAdd, cdxIconEdit, cdxIconTrash } from '@wikimedia/codex-icons';
 import { NeoWikiExtension } from '@/NeoWikiExtension.ts';
@@ -118,8 +106,7 @@ import { useLayoutStore } from '@/stores/LayoutStore.ts';
 import { Layout } from '@/domain/Layout.ts';
 import LayoutCreatorDialog from './LayoutCreatorDialog.vue';
 import LayoutEditorDialog from '@/components/LayoutEditor/LayoutEditorDialog.vue';
-import SummaryAction from '@/components/common/SummaryAction.vue';
-import I18nSlot from '@/components/common/I18nSlot.vue';
+import DeletePageDialog from '@/components/common/DeletePageDialog.vue';
 
 const paginationSizeOptions: { value: number }[] = [
 	{ value: 10 },
@@ -266,33 +253,6 @@ function confirmDelete( layoutName: string ): void {
 	isDeleteConfirmOpen.value = true;
 }
 
-async function executeDelete( summary: string ): Promise<void> {
-	isDeleteConfirmOpen.value = false;
-	const name = deletingLayoutName.value;
-	const reason = summary || mw.msg( 'neowiki-layout-delete-summary-default' );
-
-	try {
-		const api = new mw.Api();
-		const token = await api.getEditToken();
-		await api.post( {
-			action: 'delete',
-			title: `Layout:${ name }`,
-			reason: reason,
-			token: token
-		} );
-		mw.notify( mw.msg( 'neowiki-layout-delete-success', name ), { type: 'success' } );
-		await fetchLayouts( lastOffset.value, pageSize.value );
-	} catch ( error ) {
-		mw.notify(
-			error instanceof Error ? error.message : String( error ),
-			{
-				title: mw.msg( 'neowiki-layout-delete-error', name ),
-				type: 'error'
-			}
-		);
-	}
-}
-
 onMounted( async () => {
 	await checkCreatePermission();
 	await checkEditPermission( '' );
diff --git a/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue b/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue
index 6f4f3d07..92a33bdb 100644
--- a/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue
+++ b/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue
@@ -78,38 +78,25 @@
 			@created="onMappingCreated"
 		/>
 
-		<CdxDialog
+		<DeletePageDialog
 			:open="isDeleteConfirmOpen"
-			:title="$i18n( 'neowiki-mapping-delete-confirm-title' ).text()"
-			:use-close-button="true"
+			:page-title="`Mapping:${ deletingMappingName }`"
+			:display-name="deletingMappingName"
 			@update:open="isDeleteConfirmOpen = $event"
-		>
-			<I18nSlot message-key="neowiki-mapping-delete-confirm-message">
-				<strong>{{ deletingMappingName }}</strong>
-			</I18nSlot>
-
-			<template #footer>
-				<SummaryAction
-					help-text=""
-					:save-button-label="$i18n( 'neowiki-mapping-delete-confirm-delete' ).text()"
-					:save-disabled="false"
-					@save="executeDelete"
-				/>
-			</template>
-		</CdxDialog>
+			@deleted="fetchMappings( lastOffset, pageSize )"
+		/>
 	</div>
 </template>
 
 <script setup lang="ts">
 import { ref, onMounted } from 'vue';
-import { CdxButton, CdxDialog, CdxIcon, CdxTable } from '@wikimedia/codex';
+import { CdxButton, CdxIcon, CdxTable } from '@wikimedia/codex';
 import type { TableColumn } from '@wikimedia/codex';
 import { cdxIconAdd, cdxIconEdit, cdxIconTrash } from '@wikimedia/codex-icons';
 import { NeoWikiExtension } from '@/NeoWikiExtension.ts';
 import { useMappingPermissions } from '@/composables/useMappingPermissions.ts';
 import MappingCreatorDialog from './MappingCreatorDialog.vue';
-import SummaryAction from '@/components/common/SummaryAction.vue';
-import I18nSlot from '@/components/common/I18nSlot.vue';
+import DeletePageDialog from '@/components/common/DeletePageDialog.vue';
 
 const paginationSizeOptions: { value: number }[] = [
 	{ value: 10 },
@@ -216,33 +203,6 @@ function confirmDelete( mappingName: string ): void {
 	isDeleteConfirmOpen.value = true;
 }
 
-async function executeDelete( summary: string ): Promise<void> {
-	isDeleteConfirmOpen.value = false;
-	const name = deletingMappingName.value;
-	const reason = summary || mw.msg( 'neowiki-mapping-delete-summary-default' );
-
-	try {
-		const api = new mw.Api();
-		const token = await api.getEditToken();
-		await api.post( {
-			action: 'delete',
-			title: `Mapping:${ name }`,
-			reason: reason,
-			token: token
-		} );
-		mw.notify( mw.msg( 'neowiki-mapping-delete-success', name ), { type: 'success' } );
-		await fetchMappings( lastOffset.value, pageSize.value );
-	} catch ( error ) {
-		mw.notify(
-			error instanceof Error ? error.message : String( error ),
-			{
-				title: mw.msg( 'neowiki-mapping-delete-error', name ),
-				type: 'error'
-			}
-		);
-	}
-}
-
 onMounted( async () => {
 	await checkCreatePermission();
 	await checkEditPermission( '' );
diff --git a/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue b/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue
index e724c723..642b3e84 100644
--- a/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue
+++ b/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue
@@ -80,31 +80,19 @@
 			@update:open="onEditorOpenChange"
 		/>
 
-		<CdxDialog
+		<DeletePageDialog
 			:open="isDeleteConfirmOpen"
-			:title="$i18n( 'neowiki-schema-delete-confirm-title' ).text()"
-			:use-close-button="true"
+			:page-title="`Schema:${ deletingSchemaName }`"
+			:display-name="deletingSchemaName"
 			@update:open="isDeleteConfirmOpen = $event"
-		>
-			<I18nSlot message-key="neowiki-schema-delete-confirm-message">
-				<strong>{{ deletingSchemaName }}</strong>
-			</I18nSlot>
-
-			<template #footer>
-				<SummaryAction
-					help-text=""
-					:save-button-label="$i18n( 'neowiki-schema-delete-confirm-delete' ).text()"
-					:save-disabled="false"
-					@save="executeDelete"
-				/>
-			</template>
-		</CdxDialog>
+			@deleted="fetchSchemas( lastOffset, pageSize )"
+		/>
 	</div>
 </template>
 
 <script setup lang="ts">
 import { ref, shallowRef, onMounted, nextTick } from 'vue';
-import { CdxButton, CdxDialog, CdxIcon, CdxTable } from '@wikimedia/codex';
+import { CdxButton, CdxIcon, CdxTable } from '@wikimedia/codex';
 import type { TableColumn } from '@wikimedia/codex';
 import { cdxIconAdd, cdxIconEdit, cdxIconTrash } from '@wikimedia/codex-icons';
 import { NeoWikiExtension } from '@/NeoWikiExtension.ts';
@@ -114,8 +102,7 @@ import { Schema } from '@/domain/Schema.ts';
 import type { SchemaSummary } from '@/application/SchemaLookup.ts';
 import SchemaCreatorDialog from './SchemaCreatorDialog.vue';
 import SchemaEditorDialog from '@/components/SchemaEditor/SchemaEditorDialog.vue';
-import SummaryAction from '@/components/common/SummaryAction.vue';
-import I18nSlot from '@/components/common/I18nSlot.vue';
+import DeletePageDialog from '@/components/common/DeletePageDialog.vue';
 
 const paginationSizeOptions: { value: number }[] = [
 	{ value: 10 },
@@ -240,33 +227,6 @@ function confirmDelete( schemaName: string ): void {
 	isDeleteConfirmOpen.value = true;
 }
 
-async function executeDelete( summary: string ): Promise<void> {
-	isDeleteConfirmOpen.value = false;
-	const name = deletingSchemaName.value;
-	const reason = summary || mw.msg( 'neowiki-schema-delete-summary-default' );
-
-	try {
-		const api = new mw.Api();
-		const token = await api.getEditToken();
-		await api.post( {
-			action: 'delete',
-			title: `Schema:${ name }`,
-			reason: reason,
-			token: token
-		} );
-		mw.notify( mw.msg( 'neowiki-schema-delete-success', name ), { type: 'success' } );
-		await fetchSchemas( lastOffset.value, pageSize.value );
-	} catch ( error ) {
-		mw.notify(
-			error instanceof Error ? error.message : String( error ),
-			{
-				title: mw.msg( 'neowiki-schema-delete-error', name ),
-				type: 'error'
-			}
-		);
-	}
-}
-
 onMounted( async () => {
 	await checkCreatePermission();
 	await checkEditPermission( '' );
diff --git a/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts b/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts
index b8096711..e1271cd3 100644
--- a/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts
+++ b/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts
@@ -3,8 +3,9 @@ import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { ref } from 'vue';
 import MappingsPage from '@/components/MappingsPage/MappingsPage.vue';
 import MappingCreatorDialog from '@/components/MappingsPage/MappingCreatorDialog.vue';
+import DeletePageDialog from '@/components/common/DeletePageDialog.vue';
 import { createI18nMock, setupMwMock } from '../../VueTestHelpers.ts';
-import { CdxButton, CdxDialog } from '@wikimedia/codex';
+import { CdxButton } from '@wikimedia/codex';
 
 interface MappingSummary {
 	name: string;
@@ -83,8 +84,7 @@ function mountComponent( summaries: MappingSummary[] = [] ): VueWrapper {
 			mocks: { $i18n: createI18nMock() },
 			stubs: {
 				MappingCreatorDialog: MappingCreatorDialogStub,
-				SummaryAction: true,
-				I18nSlot: true,
+				DeletePageDialog: true,
 				CdxIcon: true,
 			},
 		},
@@ -241,7 +241,7 @@ describe( 'MappingsPage', () => {
 		expect( findDeleteButtons( wrapper ) ).toHaveLength( 0 );
 	} );
 
-	it( 'opens the delete confirmation when the delete button is clicked', async () => {
+	it( 'opens the delete confirmation for the clicked mapping', async () => {
 		canDeleteMappingRef.value = true;
 		const wrapper = mountComponent( [
 			{ name: 'EDM', schemas: [ 'Person' ] },
@@ -250,7 +250,10 @@ describe( 'MappingsPage', () => {
 
 		await findDeleteButtons( wrapper )[ 0 ].trigger( 'click' );
 
-		expect( wrapper.findComponent( CdxDialog ).props( 'open' ) ).toBe( true );
+		const dialog = wrapper.findComponent( DeletePageDialog );
+		expect( dialog.props( 'open' ) ).toBe( true );
+		expect( dialog.props( 'pageTitle' ) ).toBe( 'Mapping:EDM' );
+		expect( dialog.props( 'displayName' ) ).toBe( 'EDM' );
 	} );
 
 	it( 'navigates to the raw-JSON edit view when the edit button is clicked', async () => {
diff --git a/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts b/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts
index e1ef6c2d..b2a462e9 100644
--- a/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts
+++ b/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts
@@ -4,8 +4,9 @@ import { ref } from 'vue';
 import SchemasPage from '@/components/SchemasPage/SchemasPage.vue';
 import SchemaCreatorDialog from '@/components/SchemasPage/SchemaCreatorDialog.vue';
 import SchemaEditorDialog from '@/components/SchemaEditor/SchemaEditorDialog.vue';
+import DeletePageDialog from '@/components/common/DeletePageDialog.vue';
 import { createI18nMock, setupMwMock } from '../../VueTestHelpers.ts';
-import { CdxButton, CdxDialog } from '@wikimedia/codex';
+import { CdxButton } from '@wikimedia/codex';
 import { Schema } from '@/domain/Schema.ts';
 import { PropertyDefinitionList } from '@/domain/PropertyDefinitionList.ts';
 
@@ -92,8 +93,7 @@ function mountComponent( summaries: unknown[] = [] ): VueWrapper {
 			stubs: {
 				SchemaCreatorDialog: SchemaCreatorDialogStub,
 				SchemaEditorDialog: SchemaEditorDialogStub,
-				SummaryAction: true,
-				I18nSlot: true,
+				DeletePageDialog: true,
 				CdxIcon: true,
 			},
 		},
@@ -230,7 +230,7 @@ describe( 'SchemasPage', () => {
 		expect( wrapper.findComponent( SchemaEditorDialog ).exists() ).toBe( false );
 	} );
 
-	it( 'opens delete confirmation when delete button is clicked', async () => {
+	it( 'opens the delete confirmation for the clicked schema', async () => {
 		canEditSchemaRef.value = true;
 		const wrapper = mountComponent( [
 			{ name: 'Person', description: '', propertyCount: 3 },
@@ -239,7 +239,9 @@ describe( 'SchemasPage', () => {
 
 		await findDeleteButtons( wrapper )[ 0 ].trigger( 'click' );
 
-		const dialog = wrapper.findComponent( CdxDialog );
+		const dialog = wrapper.findComponent( DeletePageDialog );
 		expect( dialog.props( 'open' ) ).toBe( true );
+		expect( dialog.props( 'pageTitle' ) ).toBe( 'Schema:Person' );
+		expect( dialog.props( 'displayName' ) ).toBe( 'Person' );
 	} );
 } );

From 8df6d9031f67cca4ef7220b1e6e4ef62cdf05736 Mon Sep 17 00:00:00 2001
From: alistair3149 <alistair31494322@gmail.com>
Date: Wed, 22 Jul 2026 13:29:47 -0400
Subject: [PATCH 5/6] Address code review: LayoutsPage delete test + spec
 helper rename

- Add LayoutsPage.spec.ts. LayoutsPage had no spec, so its delete wiring
  (`Layout:${name}` -> DeletePageDialog) was the one untested page after
  the migration; a wrong namespace prefix could ship silently. Mirrors
  the Schemas/Mappings "opens the delete confirmation" coverage.
- Rename the leftover mountDialogWithRealEditSummary helper in
  MappingCreatorDialog.spec.ts to mountDialogWithRealSummaryAction.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../LayoutsPage/LayoutsPage.spec.ts           | 138 ++++++++++++++++++
 .../MappingsPage/MappingCreatorDialog.spec.ts |   6 +-
 2 files changed, 141 insertions(+), 3 deletions(-)
 create mode 100644 resources/ext.neowiki/tests/components/LayoutsPage/LayoutsPage.spec.ts

diff --git a/resources/ext.neowiki/tests/components/LayoutsPage/LayoutsPage.spec.ts b/resources/ext.neowiki/tests/components/LayoutsPage/LayoutsPage.spec.ts
new file mode 100644
index 00000000..32a21bd4
--- /dev/null
+++ b/resources/ext.neowiki/tests/components/LayoutsPage/LayoutsPage.spec.ts
@@ -0,0 +1,138 @@
+import { mount, VueWrapper, flushPromises } from '@vue/test-utils';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { ref } from 'vue';
+import LayoutsPage from '@/components/LayoutsPage/LayoutsPage.vue';
+import DeletePageDialog from '@/components/common/DeletePageDialog.vue';
+import { createI18nMock, setupMwMock } from '../../VueTestHelpers.ts';
+import { CdxButton } from '@wikimedia/codex';
+
+interface LayoutSummary {
+	name: string;
+	schema: string;
+	type: string;
+	description: string;
+}
+
+const canCreateLayoutsRef = ref( false );
+const canEditLayoutRef = ref( false );
+const checkCreatePermissionMock = vi.fn();
+const checkEditPermissionMock = vi.fn();
+
+let layoutsResponse: { layouts: LayoutSummary[]; totalRows: number } = { layouts: [], totalRows: 0 };
+
+vi.mock( '@/composables/useLayoutPermissions.ts', () => ( {
+	useLayoutPermissions: () => ( {
+		canCreateLayouts: canCreateLayoutsRef,
+		canEditLayout: canEditLayoutRef,
+		checkCreatePermission: checkCreatePermissionMock,
+		checkEditPermission: checkEditPermissionMock,
+	} ),
+} ) );
+
+// The store is only exercised by the editor path, not by the deletion flow under test.
+vi.mock( '@/stores/LayoutStore.ts', () => ( {
+	useLayoutStore: () => ( {
+		fetchLayout: vi.fn(),
+		getLayout: vi.fn(),
+	} ),
+} ) );
+
+vi.mock( '@/NeoWikiExtension.ts', () => ( {
+	NeoWikiExtension: {
+		getInstance: () => ( {
+			getMediaWiki: () => ( {
+				util: { wikiScript: () => '/rest.php' },
+			} ),
+			newHttpClient: () => ( {
+				get: vi.fn().mockResolvedValue( {
+					ok: true,
+					json: () => Promise.resolve( layoutsResponse ),
+				} ),
+			} ),
+		} ),
+	},
+} ) );
+
+function findEditButtons( wrapper: VueWrapper ): VueWrapper[] {
+	return wrapper.findAllComponents( CdxButton )
+		.filter( ( btn ) => btn.attributes( 'aria-label' ) === 'neowiki-edit-layout' );
+}
+
+function findDeleteButtons( wrapper: VueWrapper ): VueWrapper[] {
+	return wrapper.findAllComponents( CdxButton )
+		.filter( ( btn ) => btn.attributes( 'aria-label' ) === 'neowiki-layout-delete' );
+}
+
+function mountComponent( summaries: LayoutSummary[] = [] ): VueWrapper {
+	layoutsResponse = { layouts: summaries, totalRows: summaries.length };
+	setupMwMock( { functions: [ 'msg', 'util', 'message', 'notify' ] } );
+
+	return mount( LayoutsPage, {
+		global: {
+			mocks: { $i18n: createI18nMock() },
+			stubs: {
+				LayoutCreatorDialog: true,
+				LayoutEditorDialog: true,
+				DeletePageDialog: true,
+				CdxIcon: true,
+			},
+		},
+	} );
+}
+
+const sampleLayout: LayoutSummary = {
+	name: 'CompanyOverview',
+	schema: 'Company',
+	type: 'infobox',
+	description: 'Overview',
+};
+
+describe( 'LayoutsPage', () => {
+	beforeEach( () => {
+		canCreateLayoutsRef.value = false;
+		canEditLayoutRef.value = false;
+		checkCreatePermissionMock.mockClear();
+		checkEditPermissionMock.mockClear();
+		layoutsResponse = { layouts: [], totalRows: 0 };
+	} );
+
+	it( 'links each layout name to its Layout page', async () => {
+		const wrapper = mountComponent( [ sampleLayout ] );
+		await flushPromises();
+
+		const link = wrapper.find( 'a[href="/wiki/Layout:CompanyOverview"]' );
+		expect( link.exists() ).toBe( true );
+		expect( link.text() ).toBe( 'CompanyOverview' );
+	} );
+
+	it( 'shows edit and delete buttons when the user may edit layouts', async () => {
+		canEditLayoutRef.value = true;
+		const wrapper = mountComponent( [ sampleLayout ] );
+		await flushPromises();
+
+		expect( findEditButtons( wrapper ) ).toHaveLength( 1 );
+		expect( findDeleteButtons( wrapper ) ).toHaveLength( 1 );
+	} );
+
+	it( 'hides edit and delete buttons without edit permission', async () => {
+		canEditLayoutRef.value = false;
+		const wrapper = mountComponent( [ sampleLayout ] );
+		await flushPromises();
+
+		expect( findEditButtons( wrapper ) ).toHaveLength( 0 );
+		expect( findDeleteButtons( wrapper ) ).toHaveLength( 0 );
+	} );
+
+	it( 'opens the delete confirmation for the clicked layout', async () => {
+		canEditLayoutRef.value = true;
+		const wrapper = mountComponent( [ sampleLayout ] );
+		await flushPromises();
+
+		await findDeleteButtons( wrapper )[ 0 ].trigger( 'click' );
+
+		const dialog = wrapper.findComponent( DeletePageDialog );
+		expect( dialog.props( 'open' ) ).toBe( true );
+		expect( dialog.props( 'pageTitle' ) ).toBe( 'Layout:CompanyOverview' );
+		expect( dialog.props( 'displayName' ) ).toBe( 'CompanyOverview' );
+	} );
+} );
diff --git a/resources/ext.neowiki/tests/components/MappingsPage/MappingCreatorDialog.spec.ts b/resources/ext.neowiki/tests/components/MappingsPage/MappingCreatorDialog.spec.ts
index ab95e731..25a4e94f 100644
--- a/resources/ext.neowiki/tests/components/MappingsPage/MappingCreatorDialog.spec.ts
+++ b/resources/ext.neowiki/tests/components/MappingsPage/MappingCreatorDialog.spec.ts
@@ -37,7 +37,7 @@ async function enterNameAndSave( wrapper: VueWrapper, name: string ): Promise<vo
 
 // Uses the real SummaryAction so pressing Enter exercises its exposed submit
 // (which saves with whatever summary is currently entered).
-function mountDialogWithRealEditSummary(): VueWrapper {
+function mountDialogWithRealSummaryAction(): VueWrapper {
 	return mount( MappingCreatorDialog, {
 		props: { open: true },
 		global: {
@@ -95,7 +95,7 @@ describe( 'MappingCreatorDialog', () => {
 
 	it( 'submits when Enter is pressed in the name field', async () => {
 		createMock.mockResolvedValue( {} );
-		const wrapper = mountDialogWithRealEditSummary();
+		const wrapper = mountDialogWithRealSummaryAction();
 
 		await wrapper.find( 'input' ).setValue( 'EDM' );
 		await wrapper.find( 'input' ).trigger( 'keyup.enter' );
@@ -110,7 +110,7 @@ describe( 'MappingCreatorDialog', () => {
 		createMock.mockReturnValue( new Promise( ( resolve ) => {
 			resolveCreate = resolve;
 		} ) );
-		const wrapper = mountDialogWithRealEditSummary();
+		const wrapper = mountDialogWithRealSummaryAction();
 
 		await wrapper.find( 'input' ).setValue( 'EDM' );
 		await wrapper.find( 'input' ).trigger( 'keyup.enter' );

From b48960b875e150b5a99aa4bf5f1999294d35f247 Mon Sep 17 00:00:00 2001
From: alistair3149 <alistair31494322@gmail.com>
Date: Wed, 22 Jul 2026 13:45:05 -0400
Subject: [PATCH 6/6] Name the entity type in the delete confirmation message

"The page EDM will be permanently deleted" was doubly wrong: users do
not necessarily know a Schema/Layout/Mapping is stored as a page, and
"permanently" is false in MediaWiki (a deleted revision is retained and
can be undeleted).

The message now reads "The <type> <name> will be deleted" (e.g. "The
mapping EDM will be deleted"), with the name kept bold. DeletePageDialog
gains a typeLabel prop; each page passes its entity noun via the new
neowiki-{schema,layout,mapping}-noun messages.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 extension.json                                             | 3 +++
 i18n/en.json                                               | 5 ++++-
 i18n/qqq.json                                              | 7 +++++--
 .../ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue | 1 +
 .../src/components/MappingsPage/MappingsPage.vue           | 1 +
 .../ext.neowiki/src/components/SchemasPage/SchemasPage.vue | 1 +
 .../ext.neowiki/src/components/common/DeletePageDialog.vue | 5 ++++-
 .../tests/components/LayoutsPage/LayoutsPage.spec.ts       | 1 +
 .../tests/components/MappingsPage/MappingsPage.spec.ts     | 1 +
 .../tests/components/SchemasPage/SchemasPage.spec.ts       | 1 +
 .../tests/components/common/DeletePageDialog.spec.ts       | 1 +
 11 files changed, 23 insertions(+), 4 deletions(-)

diff --git a/extension.json b/extension.json
index 9810b996..f3da85af 100644
--- a/extension.json
+++ b/extension.json
@@ -460,6 +460,9 @@
 				"neowiki-delete-confirm-title",
 				"neowiki-delete-confirm-message",
 				"neowiki-delete-confirm-button",
+				"neowiki-schema-noun",
+				"neowiki-layout-noun",
+				"neowiki-mapping-noun",
 				"neowiki-delete-reason-label",
 				"neowiki-delete-reason-placeholder",
 				"neowiki-delete-summary-default",
diff --git a/i18n/en.json b/i18n/en.json
index bb9a54cc..25496a46 100644
--- a/i18n/en.json
+++ b/i18n/en.json
@@ -103,8 +103,11 @@
 	"neowiki-edit-summary-help-text-schema": "Changes affect all Subjects using this Schema.",
 
 	"neowiki-delete-confirm-title": "Delete $1?",
-	"neowiki-delete-confirm-message": "The page $1 will be permanently deleted.",
+	"neowiki-delete-confirm-message": "The $1 will be deleted.",
 	"neowiki-delete-confirm-button": "Delete",
+	"neowiki-schema-noun": "schema",
+	"neowiki-layout-noun": "layout",
+	"neowiki-mapping-noun": "mapping",
 	"neowiki-delete-reason-label": "Reason",
 	"neowiki-delete-reason-placeholder": "Reason for deletion",
 	"neowiki-delete-summary-default": "Deleted via the NeoWiki UI",
diff --git a/i18n/qqq.json b/i18n/qqq.json
index e64a69f7..78302e4a 100644
--- a/i18n/qqq.json
+++ b/i18n/qqq.json
@@ -122,9 +122,12 @@
 	"neowiki-save-layout": "Label for the save button in the layout editor dialog.",
 	"neowiki-edit-summary-help-text-layout": "Help text shown below the edit summary in the layout editor dialog.",
 
-	"neowiki-delete-confirm-title": "Title of the confirmation dialog shown before deleting a NeoWiki page (Schema, Layout, or Mapping). $1 is the page name.",
-	"neowiki-delete-confirm-message": "Body of the delete confirmation dialog. $1 is the page name, shown in bold.",
+	"neowiki-delete-confirm-title": "Title of the confirmation dialog shown before deleting a NeoWiki page (Schema, Layout, or Mapping). $1 is the entity name.",
+	"neowiki-delete-confirm-message": "Body of the delete confirmation dialog. $1 is the entity type noun (see neowiki-schema-noun etc.) followed by the entity name in bold, e.g. \"schema '''Foo'''\".",
 	"neowiki-delete-confirm-button": "Label for the button that confirms deletion in the delete confirmation dialog.",
+	"neowiki-schema-noun": "Lowercase common noun for a Schema, used inside sentences such as the delete confirmation message (\"The schema Foo will be deleted\").",
+	"neowiki-layout-noun": "Lowercase common noun for a Layout, used inside sentences such as the delete confirmation message (\"The layout Foo will be deleted\").",
+	"neowiki-mapping-noun": "Lowercase common noun for an ontology Mapping, used inside sentences such as the delete confirmation message (\"The mapping Foo will be deleted\").",
 	"neowiki-delete-reason-label": "Label for the optional deletion-reason field in the delete confirmation dialog.",
 	"neowiki-delete-reason-placeholder": "Placeholder for the optional deletion-reason field in the delete confirmation dialog.",
 	"neowiki-delete-summary-default": "Default deletion reason recorded in the deletion log when the user enters none.",
diff --git a/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue b/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue
index b218f8f9..219e024f 100644
--- a/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue
+++ b/resources/ext.neowiki/src/components/LayoutsPage/LayoutsPage.vue
@@ -89,6 +89,7 @@
 			:open="isDeleteConfirmOpen"
 			:page-title="`Layout:${ deletingLayoutName }`"
 			:display-name="deletingLayoutName"
+			:type-label="$i18n( 'neowiki-layout-noun' ).text()"
 			@update:open="isDeleteConfirmOpen = $event"
 			@deleted="fetchLayouts( lastOffset, pageSize )"
 		/>
diff --git a/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue b/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue
index 92a33bdb..e3964824 100644
--- a/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue
+++ b/resources/ext.neowiki/src/components/MappingsPage/MappingsPage.vue
@@ -82,6 +82,7 @@
 			:open="isDeleteConfirmOpen"
 			:page-title="`Mapping:${ deletingMappingName }`"
 			:display-name="deletingMappingName"
+			:type-label="$i18n( 'neowiki-mapping-noun' ).text()"
 			@update:open="isDeleteConfirmOpen = $event"
 			@deleted="fetchMappings( lastOffset, pageSize )"
 		/>
diff --git a/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue b/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue
index 642b3e84..7ff7f9a8 100644
--- a/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue
+++ b/resources/ext.neowiki/src/components/SchemasPage/SchemasPage.vue
@@ -84,6 +84,7 @@
 			:open="isDeleteConfirmOpen"
 			:page-title="`Schema:${ deletingSchemaName }`"
 			:display-name="deletingSchemaName"
+			:type-label="$i18n( 'neowiki-schema-noun' ).text()"
 			@update:open="isDeleteConfirmOpen = $event"
 			@deleted="fetchSchemas( lastOffset, pageSize )"
 		/>
diff --git a/resources/ext.neowiki/src/components/common/DeletePageDialog.vue b/resources/ext.neowiki/src/components/common/DeletePageDialog.vue
index 536d0ab6..86fad3f8 100644
--- a/resources/ext.neowiki/src/components/common/DeletePageDialog.vue
+++ b/resources/ext.neowiki/src/components/common/DeletePageDialog.vue
@@ -6,7 +6,7 @@
 		@update:open="onUpdateOpen"
 	>
 		<I18nSlot message-key="neowiki-delete-confirm-message">
-			<strong>{{ props.displayName }}</strong>
+			{{ props.typeLabel }} <strong>{{ props.displayName }}</strong>
 		</I18nSlot>
 
 		<template #footer>
@@ -36,6 +36,9 @@ const props = defineProps<{
 	pageTitle: string;
 	// The bare name shown in the confirmation message and notifications.
 	displayName: string;
+	// The lowercase entity noun shown before the name, e.g. `schema`, so the message reads
+	// "The schema Person will be deleted" rather than exposing that it is a wiki page.
+	typeLabel: string;
 }>();
 
 const emit = defineEmits<{
diff --git a/resources/ext.neowiki/tests/components/LayoutsPage/LayoutsPage.spec.ts b/resources/ext.neowiki/tests/components/LayoutsPage/LayoutsPage.spec.ts
index 32a21bd4..236f754e 100644
--- a/resources/ext.neowiki/tests/components/LayoutsPage/LayoutsPage.spec.ts
+++ b/resources/ext.neowiki/tests/components/LayoutsPage/LayoutsPage.spec.ts
@@ -134,5 +134,6 @@ describe( 'LayoutsPage', () => {
 		expect( dialog.props( 'open' ) ).toBe( true );
 		expect( dialog.props( 'pageTitle' ) ).toBe( 'Layout:CompanyOverview' );
 		expect( dialog.props( 'displayName' ) ).toBe( 'CompanyOverview' );
+		expect( dialog.props( 'typeLabel' ) ).toBe( 'neowiki-layout-noun' );
 	} );
 } );
diff --git a/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts b/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts
index e1271cd3..479b38e8 100644
--- a/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts
+++ b/resources/ext.neowiki/tests/components/MappingsPage/MappingsPage.spec.ts
@@ -254,6 +254,7 @@ describe( 'MappingsPage', () => {
 		expect( dialog.props( 'open' ) ).toBe( true );
 		expect( dialog.props( 'pageTitle' ) ).toBe( 'Mapping:EDM' );
 		expect( dialog.props( 'displayName' ) ).toBe( 'EDM' );
+		expect( dialog.props( 'typeLabel' ) ).toBe( 'neowiki-mapping-noun' );
 	} );
 
 	it( 'navigates to the raw-JSON edit view when the edit button is clicked', async () => {
diff --git a/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts b/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts
index b2a462e9..e22ba1ce 100644
--- a/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts
+++ b/resources/ext.neowiki/tests/components/SchemasPage/SchemasPage.spec.ts
@@ -243,5 +243,6 @@ describe( 'SchemasPage', () => {
 		expect( dialog.props( 'open' ) ).toBe( true );
 		expect( dialog.props( 'pageTitle' ) ).toBe( 'Schema:Person' );
 		expect( dialog.props( 'displayName' ) ).toBe( 'Person' );
+		expect( dialog.props( 'typeLabel' ) ).toBe( 'neowiki-schema-noun' );
 	} );
 } );
diff --git a/resources/ext.neowiki/tests/components/common/DeletePageDialog.spec.ts b/resources/ext.neowiki/tests/components/common/DeletePageDialog.spec.ts
index 19434d9d..c4774f10 100644
--- a/resources/ext.neowiki/tests/components/common/DeletePageDialog.spec.ts
+++ b/resources/ext.neowiki/tests/components/common/DeletePageDialog.spec.ts
@@ -27,6 +27,7 @@ function mountDialog( props: Record<string, unknown> = {} ): VueWrapper {
 			open: true,
 			pageTitle: 'Schema:Person',
 			displayName: 'Person',
+			typeLabel: 'schema',
 			...props,
 		},
 		global: {