Fixed class handling for transport feature.

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
+
+- The [transport feature](https://superforms.rocks/concepts/nested-data#arbitrary-types-in-the-form) released in 2.22.0 didn't fully handle classes. It should work better now, but the classes must be very simple to work. Let me know if you have any problems with a certain class.
+
 ## [2.22.0] - 2024-12-15
 
 ### Added

package.json

@@ -174,7 +174,6 @@
 	},
 	"dependencies": {
 		"devalue": "^5.1.1",
-		"just-clone": "^6.2.0",
 		"memoize-weak": "^1.0.2",
 		"ts-deepmerge": "^7.0.2"
 	},

pnpm-lock.yaml

@@ -1,9 +1,14 @@
 import type { Transport } from '@sveltejs/kit';
 import { Decimal } from 'decimal.js';
+import { RecordId } from './routes/RecordId.js';
 
 export const transport: Transport = {
 	Decimal: {
 		encode: (value) => value instanceof Decimal && value.toString(),
 		decode: (str) => new Decimal(str)
+	},
+	RecordId: {
+		encode: (record) => record instanceof RecordId && [record.id, record.tb],
+		decode: ([id, tb]) => new RecordId(id, tb)
 	}
 };

src/hooks.ts

@@ -0,0 +1,46 @@
+/*
+  Deep clones all properties except functions
+
+  var arr = [1, 2, 3];
+  var subObj = {aa: 1};
+  var obj = {a: 3, b: 5, c: arr, d: subObj};
+  var objClone = clone(obj);
+  arr.push(4);
+  subObj.bb = 2;
+  obj; // {a: 3, b: 5, c: [1, 2, 3, 4], d: {aa: 1}}
+  objClone; // {a: 3, b: 5, c: [1, 2, 3], d: {aa: 1, bb: 2}}
+*/
+
+export function clone<T>(obj: T): T {
+	const type = {}.toString.call(obj).slice(8, -1);
+	if (type == 'Set') {
+		// @ts-expect-error Known type
+		return new Set([...obj].map((value) => clone(value)));
+	}
+	if (type == 'Map') {
+		// @ts-expect-error Known type
+		return new Map([...obj].map((kv) => [clone(kv[0]), clone(kv[1])]));
+	}
+	if (type == 'Date') {
+		// @ts-expect-error Known type
+		return new Date(obj.getTime());
+	}
+	if (type == 'RegExp') {
+		// @ts-expect-error Known type
+		return RegExp(obj.source, (obj as RegExp).flags);
+	}
+	if (type == 'Array') {
+		// eslint-disable-next-line @typescript-eslint/no-explicit-any
+		const result = [] as any;
+		for (const key in obj) {
+			result[key] = clone(obj[key]);
+		}
+		return result;
+	}
+	if (type == 'Object') {
+		return Object.assign(Object.create(Object.getPrototypeOf(obj)), obj);
+	}
+
+	// primitives and non-supported objects (e.g. functions) land here
+	return obj;
+}

src/lib/justClone.ts

@@ -1,6 +1,6 @@
 import type { JSONSchema7Definition } from 'json-schema';
 import type { JSONSchema } from './jsonSchema/index.js';
-import justClone from 'just-clone';
+import { clone as justClone } from './justClone.js';
 import { SchemaError } from './errors.js';
 
 // export type DeepPartial<T> = T extends object

src/lib/utils.ts

@@ -1,6 +1,7 @@
 import { zod } from '$lib/adapters/zod.js';
 import { message, superValidate } from '$lib/server/index.js';
 import { transport } from '../../../../hooks.js';
+import { RecordId } from '../../../RecordId.js';
 import { schema } from './schema.js';
 import { fail } from '@sveltejs/kit';
 
@@ -20,6 +21,7 @@ export const actions = {
 		if (!form.valid) return fail(400, { form });
 
 		form.data.luckyNumber = form.data.luckyNumber.mul(2);
+		form.data.id = new RecordId(form.data.id.id * 2, form.data.id.tb);
 
 		return message(form, 'Your lucky number times 2 is ' + form.data.luckyNumber);
 	}

src/routes/(v2)/v2/transport/+page.server.ts

@@ -29,6 +29,8 @@
 
 {#if $message}<h4>{$message}</h4>{/if}
 
+<p id="record">$form.id: {$form.id}</p>
+
 <form method="POST" use:enhance>
 	<label>
 		Name: <input

src/routes/(v2)/v2/transport/+page.svelte

@@ -1,7 +1,9 @@
 import { Decimal } from 'decimal.js';
 import { z } from 'zod';
+import { RecordId } from '../../../RecordId.js';
 
 export const schema = z.object({
+	id: z.instanceof(RecordId).default(new RecordId(123, 'test')),
 	name: z.string().min(2),
 	luckyNumber: z
 		.instanceof(Decimal)

src/routes/(v2)/v2/transport/schema.ts

@@ -0,0 +1,15 @@
+export class RecordId {
+	public readonly id: number;
+	public readonly tb: string;
+
+	constructor(id: number, tb: string) {
+		if (!id) throw new Error('id is required');
+		if (!tb) throw new Error('tb is required');
+		this.id = id;
+		this.tb = tb;
+	}
+
+	public toString() {
+		return `${this.id} for table ${this.tb}`;
+	}
+}