isTainted now works with undefined values in the $tainted store.

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
+
+- `isTainted` now works with undefined values in the `$tainted` store.
+
 ## [2.13.1] - 2024-05-07
 
 ### Fixed

src/lib/client/superForm.ts

@@ -1209,8 +1209,7 @@ export function superForm<
 	): boolean {
 		if (typeof path === 'boolean') return path;
 		if (typeof path === 'object') return Tainted__isObjectTainted(path);
-		if (!Data.tainted) return false;
-		if (!path) return Tainted__isObjectTainted(Data.tainted);
+		if (!Data.tainted || path === undefined) return false;
 
 		const field = pathExists(Data.tainted, splitPath(path));
 		return Tainted__isObjectTainted(field?.value);

src/routes/(v2)/v2/Navigation.svelte

@@ -60,7 +60,8 @@
 		'nested-traverse',
 		'reset-errors',
 		'schemasafe-types',
-		'discriminated-union'
+		'discriminated-union',
+		'simple-tainted'
 	].sort();
 </script>
 

src/routes/(v2)/v2/simple-tainted/+page.server.ts

@@ -0,0 +1,27 @@
+import { message, superValidate } from '$lib/index.js';
+import { zod } from '$lib/adapters/zod.js';
+import { fail } from '@sveltejs/kit';
+import { schema } from './schema.js';
+
+export const load = async () => {
+	const form = await superValidate(zod(schema));
+
+	// Always return { form } in load functions
+	return { form };
+};
+
+export const actions = {
+	default: async ({ request }) => {
+		const form = await superValidate(request, zod(schema));
+		console.log(form);
+
+		if (!form.valid) {
+			// Again, return { form } and things will just work.
+			return fail(400, { form });
+		}
+
+		// TODO: Do something with the validated form.data
+
+		return message(form, 'Form posted successfully!');
+	}
+};

src/routes/(v2)/v2/simple-tainted/+page.svelte

@@ -0,0 +1,48 @@
+<script lang="ts">
+	import { superForm } from '$lib/index.js';
+	import { zodClient } from '$lib/adapters/zod.js';
+	import type { PageData } from './$types.js';
+	import { schema } from './schema.js';
+
+	export let data: PageData;
+
+	const { form, errors, constraints, message, enhance, tainted, isTainted } = superForm(data.form, {
+		dataType: 'json',
+		validators: zodClient(schema)
+	});
+</script>
+
+{#if $message}<h3>{$message}</h3>{/if}
+
+<form method="POST" use:enhance>
+	<label for="name">Name</label>
+	<input
+		type="text"
+		name="name"
+		aria-invalid={$errors.name?.value ? 'true' : undefined}
+		bind:value={$form.name.value}
+		{...$constraints.name?.value}
+	/>
+	{#if $errors.name?.value}<span class="invalid">{$errors.name.value}</span>{/if}
+
+	<label for="email">E-mail</label>
+	<input
+		type="email"
+		name="email"
+		aria-invalid={$errors.email?.value ? 'true' : undefined}
+		bind:value={$form.email.value}
+		{...$constraints.email?.value}
+	/>
+	{#if $errors.email?.value}<span class="invalid">{$errors.email.value}</span>{/if}
+
+	<div><button>Submit</button></div>
+</form>
+
+<p id="tainted">Tainted: {isTainted($tainted)}</p>
+<p id="email-tainted">Tainted email: {isTainted($tainted?.email)}</p>
+
+<style>
+	.invalid {
+		color: red;
+	}
+</style>

src/routes/(v2)/v2/simple-tainted/schema.ts

@@ -0,0 +1,11 @@
+import { z } from 'zod';
+
+// Define at the top-level so it stays in memory and the adapter can be cached
+export const schema = z.object({
+	name: z.object({
+		value: z.string().min(5)
+	}),
+	email: z.object({
+		value: z.string().email()
+	})
+});