Using superForm with $derived

[unlocomqx]

• Before posting an issue, read the FAQ and search the previous issues.

Description
I tried retuning multiple superforms in an array, but I could not use it in the frontend.
The only convenient way was to destructure each superform in the each tag but svelte doesn't support "declaring" stores out of the top-level of the component

If applicable, a MRE
https://www.sveltelab.dev/9tdm9ul193m3tz5

[unlocomqx]

It didn't occur to me to extract the form into a separate component 🤦
So this would work

{#each superforms as superform}
	<Comp {superform} />
{/each}

Closing.

[unlocomqx]

I'm reopening this issue because I encountered a problem

When I write this code

<script>
const superforms = data.my_forms.map((form) =>
	superForm(form, {
		dataType: 'json'
	})
);
</script>
{#each superforms as superform (superform.formId)}
	<ConditionItem {superform} />
{/each}

It works well but when a new form is added, then that's where I run into a problem

If I try to use $derived

const superforms = $derived(
	data.condition_forms.map((form) =>
		superForm(form, {
			dataType: 'json'
		})
	)
);

I get this error

Uncaught Error: onDestroy can only be used during component initialisation.

Repro https://www.sveltelab.dev/9tdm9ul193m3tz5

• Click the Add button to add a new item (thus a new form)

[unlocomqx]

I found a way to fix it, I pushed my code here

If you'd like, I can make a PR for this

Cheers

[ciscoheat]

A PR for this would be very nice!

[unlocomqx]

Okay, I will send a PR for this

[ciscoheat]

The problem is bigger than I anticipated, it'll require an overhaul of the onDestroy/cleanup handling. It'll have to wait until later. For now, calling superForm within $derived is not possible.

[unlocomqx]

Okay, it will be good to have dynamic forms in the future. I will try to find a workaround for now.

[snide]

Out of curiosity, what is the proscribed workaround for this? Just not to use superforms for an array of forms? Seems common to have a list of things, then have to a button to delete them individually...etc. Are folks just making client side individual API posts here for now?

[unlocomqx]

There's an example here

How to use Superforms with actions on a list of data.

Although, I prefer not using the formId and instead get the submitter and alter the submitted data based on its props.

I also only submit tainted rows as I only need to save the text fields that the user modified.

Sample code below

const {
	....
} = superForm(data.fields_form, {
	dataType: 'json',
	resetForm: false,
	onSubmit({ submitter, jsonData }) {
		const action = submitter?.getAttribute('name') || 'update'
		const action_value = submitter?.getAttribute('value') || ''
		jsonData({
			...$form,
			action,
			action_value,
			fields: Object.fromEntries(
				Object.values($form.fields)
					.filter((f) =>
						Object.keys($tainted?.fields ?? {}).includes(f.id.toString()),
					)
					.map((f) => [ f.id, f ]),
			),
		})
	},
})