Fixed error mapping for deeply nested errors where no default value existed.

Author: ciscoheat

CHANGELOG.md

@@ -5,6 +5,12 @@ Headlines: Added, Changed, Deprecated, Removed, Fixed, Security
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased]
+
+### Fixed
+
+- Fixed error mapping for deeply nested errors where no default value existed.
+
 ## [2.24.0] - 2025-03-09
 
 ### Added

src/lib/errors.ts

@@ -234,22 +234,28 @@ export function replaceInvalidDefaults<T extends Record<string, unknown>>(
 		for (const error of Errors) {
 			if (!error.path) continue;
 
-			Defaults_traverseAndReplace({
-				path: error.path,
-				value: pathExists(Defaults, error.path)?.value
-			});
+			Defaults_traverseAndReplace(
+				{
+					path: error.path,
+					value: pathExists(Defaults, error.path)?.value
+				},
+				true
+			);
 		}
 	}
 
 	//#endregion
 
 	//#region Defaults
 
-	function Defaults_traverseAndReplace(defaultPath: {
-		path: (string | number | symbol)[];
-		// eslint-disable-next-line @typescript-eslint/no-explicit-any
-		value: any;
-	}): void {
+	function Defaults_traverseAndReplace(
+		defaultPath: {
+			path: (string | number | symbol)[];
+			// eslint-disable-next-line @typescript-eslint/no-explicit-any
+			value: any;
+		},
+		traversingErrors = false
+	): void {
 		const currentPath = defaultPath.path;
 		if (!currentPath || !currentPath[0]) return;
 		if (typeof currentPath[0] === 'string' && preprocessed?.includes(currentPath[0])) return;
@@ -279,9 +285,11 @@ export function replaceInvalidDefaults<T extends Record<string, unknown>>(
 			const typePath = currentPath.filter((p) => /\D/.test(String(p)));
 			const pathTypes = traversePath(Types, typePath, (path) => {
 				//console.log(path.path, path.value); //debug
-				return '__items' in path.value ? path.value.__items : path.value;
+				return path.value && '__items' in path.value ? path.value.__items : path.value;
 			});
 			if (!pathTypes) {
+				// Return if checking for errors, as there may be deep errors that doesn't exist in the defaults.
+				if (traversingErrors) return;
 				throw new SchemaError('No types found for defaults', currentPath);
 			}
 

src/routes/(v2)/v2/issue-583/+page.server.ts

@@ -0,0 +1,35 @@
+import type { Actions, PageServerLoad } from './$types.js';
+
+import { superValidate, message } from '$lib/index.js';
+import { zod } from '$lib/adapters/zod.js';
+import { fail } from '@sveltejs/kit';
+import { schema } from './schema.js';
+
+const now = new Date();
+const pastDate = new Date();
+pastDate.setDate(now.getDate() - 7); // Subtract 7 days
+
+export const load: PageServerLoad = async () => {
+	const data = {
+		updates: [
+			{
+				updates: {
+					dueDate: pastDate
+				}
+			}
+		]
+	};
+	const form = await superValidate(data, zod(schema), { errors: false });
+	return { form };
+};
+
+export const actions: Actions = {
+	default: async ({ request }) => {
+		const form = await superValidate(request, zod(schema));
+		console.dir(form, { depth: 5 });
+
+		if (!form.valid) return fail(400, { form });
+
+		return message(form, 'Form posted successfully!');
+	}
+};

src/routes/(v2)/v2/issue-583/+page.svelte

@@ -0,0 +1,70 @@
+<script lang="ts">
+	import { page } from '$app/state';
+	import { superForm } from '$lib/index.js';
+	import SuperDebug from '$lib/index.js';
+
+	let { data } = $props();
+
+	const { form, errors, message, enhance } = superForm(data.form, { dataType: 'json' });
+</script>
+
+<h3>Superforms testing ground - Zod</h3>
+
+{#if $message}
+	<!-- eslint-disable-next-line svelte/valid-compile -->
+	<div class="status" class:error={page.status >= 400} class:success={page.status == 200}>
+		{$message}
+	</div>
+{/if}
+
+<form method="POST" use:enhance>
+	<p>Submit and check the error</p>
+	{#if $errors.updates?.[0].updates?.dueDate}
+		<p class="invalid">{$errors.updates[0].updates.dueDate}</p>
+	{/if}
+	<button>Submit</button>
+</form>
+
+<hr />
+<p>
+	💥 <a target="_blank" href="https://superforms.rocks">Created with Superforms for SvelteKit</a> 💥
+</p>
+
+<style>
+	.invalid {
+		color: red;
+	}
+
+	.status {
+		color: white;
+		padding: 4px;
+		padding-left: 8px;
+		border-radius: 2px;
+		font-weight: 500;
+	}
+
+	.status.success {
+		background-color: seagreen;
+	}
+
+	.status.error {
+		background-color: #ff2a02;
+	}
+
+	input {
+		background-color: #ddd;
+	}
+
+	a {
+		text-decoration: underline;
+	}
+
+	hr {
+		margin-top: 4rem;
+	}
+
+	form {
+		padding-top: 1rem;
+		padding-bottom: 1rem;
+	}
+</style>

src/routes/(v2)/v2/issue-583/schema.ts

@@ -0,0 +1,34 @@
+import { z } from 'zod';
+
+const dueDateValueSchema = z
+	.date()
+	.refine(
+		(v) => {
+			if (v) {
+				return v.getTime() > Date.now();
+			}
+			return true;
+		},
+		{ message: 'Due date must be in the future' }
+	)
+	.nullable()
+	.optional();
+
+export const itemUpdatePropertiesSchema = z.object({
+	dueDate: dueDateValueSchema
+});
+
+const itemUpdateSchema = z.object({
+	//itemId: z.string().superRefine(zodSuperRefiners.textLaxed()),
+	//childId: z.string().nullable().superRefine(zodSuperRefiners.textLaxed()),
+	updates: itemUpdatePropertiesSchema
+});
+const riskImprovementItemUpdateSchema = z.object({
+	itemid: z.string()
+});
+
+export const schema = z.object({
+	updates: z.array(itemUpdateSchema.or(riskImprovementItemUpdateSchema))
+});
+
+export type Schema = z.infer<typeof schema>;