Fixed default for FormOptions.

Author: ciscoheat

CHANGELOG.md

@@ -16,6 +16,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Type for `onChange.paths` wasn't strongly typed to `FormPath`.
 - Initial data was dereferenced after calling `superForm`, so it wasn't possible to update it when using `reset`.
+- `FormOptions` type required one type parameter that should've been defaulted to `Record<string, unknown>`.
 
 ## [2.4.0] - 2024-02-20
 

src/lib/client/superForm.ts

@@ -68,7 +68,7 @@ type ValidatorsOption<T extends Record<string, unknown>> =
 // since SuperForm<A|B> is not the same as SuperForm<A> | SuperForm<B>
 // https://www.typescriptlang.org/docs/handbook/2/conditional-types.html#distributive-conditional-types
 export type FormOptions<
-	T extends Record<string, unknown>,
+	T extends Record<string, unknown> = Record<string, unknown>,
 	// eslint-disable-next-line @typescript-eslint/no-explicit-any
 	M = any,
 	In extends Record<string, unknown> = T
@@ -354,7 +354,7 @@ const defaultFormOptions = {
 	multipleSubmits: 'prevent',
 	SPA: undefined,
 	validationMethod: 'auto'
-} satisfies FormOptions<Record<string, unknown>, unknown>;
+} satisfies FormOptions;
 
 function multipleFormIdError(id: string | undefined) {
 	return (

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

@@ -10,6 +10,8 @@
 		onChange({ paths, set, get }) {
 			for (const path of paths) {
 				if (path == 'accept') {
+					// @ts-expect-error Type error
+					set('extra', 'a string', { taint: false });
 					set('extra', 12, { taint: false });
 				} else if (path == 'extra') {
 					if (get(path) == 12) {

src/tests/superForm.test.ts

@@ -1,5 +1,5 @@
 import { zod } from '$lib/adapters/zod.js';
-import { fieldProxy, superForm, type SuperForm } from '$lib/client/index.js';
+import { fieldProxy, superForm, type FormOptions, type SuperForm } from '$lib/client/index.js';
 import { superValidate, type SuperValidated } from '$lib/superValidate.js';
 import { get } from 'svelte/store';
 import { merge } from 'ts-deepmerge';
@@ -302,6 +302,27 @@ describe('Modifying initial data for updating reset', () => {
 	});
 });
 
+describe('FormOptions', () => {
+	it('should work with default type parameters', async () => {
+		const opts: FormOptions = {
+			delayMs: 800,
+			dataType: 'json'
+		};
+
+		const validated = await superValidate(
+			zod(
+				z.object({
+					name: z.string()
+				})
+			)
+		);
+
+		const form = superForm(validated, opts);
+		const delay: number | undefined = form.options.delayMs;
+		expect(delay).toBe(800);
+	});
+});
+
 ///// mockSvelte.ts (must be copy/pasted here) ////////////////////////////////
 
 import { vi } from 'vitest';