refactor(persons): centralise PersonType, PERSON_TYPES and normalizePersonType in person-validation

Removes four independent PersonType type declarations and the duplicated
TYPES/PERSON_TYPES arrays. normalizePersonType moves from the edit route
module into the shared lib so page.server.test.ts no longer imports from a
route. Both server actions now use normalizePersonType for personType
extraction instead of an inline type cast.

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

frontend/src/lib/components/PersonTypeSelector.svelte

@@ -2,9 +2,7 @@
 import { untrack } from 'svelte';
 import { radioGroupNav } from '$lib/actions/radioGroupNav';
 import { m } from '$lib/paraglide/messages.js';
-
+import { PERSON_TYPES as TYPES, type PersonType } from '$lib/person-validation';
-const TYPES = ['PERSON', 'INSTITUTION', 'GROUP', 'UNKNOWN'] as const;
-type PersonType = (typeof TYPES)[number];
 
 let {
 	value = 'PERSON',

frontend/src/lib/person-validation.ts

@@ -1,3 +1,10 @@
+export const PERSON_TYPES = ['PERSON', 'INSTITUTION', 'GROUP', 'UNKNOWN'] as const;
+export type PersonType = (typeof PERSON_TYPES)[number];
+
+export function normalizePersonType(raw: string | undefined | null): PersonType {
+	return raw === 'SKIP' ? 'UNKNOWN' : ((raw ?? 'PERSON') as PersonType);
+}
+
 export function validatePersonFields(
 	personType: string,
 	firstName: string | undefined | null,

frontend/src/routes/persons/[id]/edit/+page.server.ts

@@ -1,13 +1,7 @@
 import { error, fail, redirect } from '@sveltejs/kit';
 import { createApiClient } from '$lib/api.server';
 import { getErrorMessage } from '$lib/errors';
-import { validatePersonFields } from '$lib/person-validation';
+import { normalizePersonType, validatePersonFields } from '$lib/person-validation';
-
-type PersonType = 'PERSON' | 'INSTITUTION' | 'GROUP' | 'UNKNOWN' | 'SKIP';
-
-export function normalizePersonType(raw: string | undefined | null): Exclude<PersonType, 'SKIP'> {
-	return raw === 'SKIP' ? 'UNKNOWN' : ((raw ?? 'PERSON') as Exclude<PersonType, 'SKIP'>);
-}
 
 export async function load({ params, fetch, locals }) {
 	const canWrite =
@@ -37,11 +31,7 @@ export async function load({ params, fetch, locals }) {
 export const actions = {
 	update: async ({ request, params, fetch }) => {
 		const formData = await request.formData();
-		const personType = (formData.get('personType')?.toString() ?? 'PERSON') as
+		const personType = normalizePersonType(formData.get('personType')?.toString());
-			| 'PERSON'
-			| 'INSTITUTION'
-			| 'GROUP'
-			| 'UNKNOWN';
 		const title = formData.get('title')?.toString().trim() || undefined;
 		const firstName = formData.get('firstName')?.toString().trim();
 		const lastName = formData.get('lastName')?.toString().trim();

frontend/src/routes/persons/[id]/edit/PersonEditForm.svelte

@@ -2,8 +2,7 @@
 import { untrack } from 'svelte';
 import { m } from '$lib/paraglide/messages.js';
 import PersonTypeSelector from '$lib/components/PersonTypeSelector.svelte';
-
+import { PERSON_TYPES as TYPES, type PersonType } from '$lib/person-validation';
-type PersonType = 'PERSON' | 'INSTITUTION' | 'GROUP' | 'UNKNOWN';
 
 let {
 	person
@@ -20,7 +19,6 @@ let {
 	};
 } = $props();
 
-const TYPES = ['PERSON', 'INSTITUTION', 'GROUP', 'UNKNOWN'] as const;
 let selectedType = $state<PersonType>(
 	untrack(() =>
 		TYPES.includes(person.personType as PersonType) ? (person.personType as PersonType) : 'PERSON'

frontend/src/routes/persons/[id]/edit/page.server.test.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'vitest';
-import { normalizePersonType } from './+page.server';
+import { normalizePersonType } from '$lib/person-validation';
 
 describe('edit load — SKIP → UNKNOWN normalization', () => {
 	it('maps SKIP to UNKNOWN', () => {

frontend/src/routes/persons/new/+page.server.ts

@@ -1,7 +1,7 @@
 import { error, fail, redirect } from '@sveltejs/kit';
 import { createApiClient } from '$lib/api.server';
 import { getErrorMessage } from '$lib/errors';
-import { validatePersonFields } from '$lib/person-validation';
+import { normalizePersonType, validatePersonFields } from '$lib/person-validation';
 
 export async function load({ locals }: { locals: App.Locals }) {
 	const canWrite =
@@ -14,11 +14,7 @@ export async function load({ locals }: { locals: App.Locals }) {
 export const actions = {
 	default: async ({ request, fetch }) => {
 		const formData = await request.formData();
-		const personType = (formData.get('personType')?.toString() ?? 'PERSON') as
+		const personType = normalizePersonType(formData.get('personType')?.toString());
-			| 'PERSON'
-			| 'INSTITUTION'
-			| 'GROUP'
-			| 'UNKNOWN';
 		const title = formData.get('title')?.toString().trim() || undefined;
 		const firstName = formData.get('firstName')?.toString().trim();
 		const lastName = formData.get('lastName')?.toString().trim();

frontend/src/routes/persons/new/+page.svelte

@@ -3,9 +3,7 @@ import { untrack } from 'svelte';
 import { m } from '$lib/paraglide/messages.js';
 import BackButton from '$lib/components/BackButton.svelte';
 import PersonTypeSelector from '$lib/components/PersonTypeSelector.svelte';
-
+import { PERSON_TYPES as TYPES, type PersonType } from '$lib/person-validation';
-type PersonType = 'PERSON' | 'INSTITUTION' | 'GROUP' | 'UNKNOWN';
-const TYPES = ['PERSON', 'INSTITUTION', 'GROUP', 'UNKNOWN'] as const;
 
 let { form } = $props();