ciscoheat / sveltekit-superforms

Making SvelteKit forms a pleasure to use!
https://superforms.rocks
MIT License
2.22k stars 66 forks source link

Handling `Record<string, string>` Input with Superforms #447

Open mpost opened 4 months ago

mpost commented 4 months ago

Description:

We are using a Zod schema with Record<string, string> to support a multi-language message. Our schema is defined as follows:

z.object({
  message: z.record(z.string()).optional(),
});

This allows inputs such as:

{
  message: {
    de: "guten tag",
    en: "good day",
    fr: "bonjour",
    // ...
  }
}

Our goal is to build a custom component that accepts the message key as input and displays the various messages within a single component. While this works fine for single values (e.g., message: "hello"), we are unable to correctly pass the Record based message to a component.

Current Implementation:

Here is our current code, which does not have the correct type for the field value. Typically, you would use FormPathLeaves<T> directly, but that is not possible with Record. Additionally, the code has incorrect types for the $errors and $value stores, as it treats them as singular values.

<script lang="ts" context="module">
  type T = Record<string, unknown>;
</script>

<script lang="ts" generics="T extends Record<string, unknown>">
  import MultiLanguageTextarea from '$lib/components/form/MultiLanguageTextarea.svelte';
  import { formFieldProxy, type SuperForm, type FormPathLeaves } from 'sveltekit-superforms';
  import type { Writable } from 'svelte/store';

  export let form: SuperForm<T>;
  export let field: string;

  $: leaves = field as FormPathLeaves<T>
  $: ({ value, errors } = formFieldProxy(form, leaves))
</script>

<MultiLanguageTextarea bind:value={$value} errors={$errors} {...$$restProps} />

Question:

What is the correct way to handle a Record<string, string> based input when using Superforms?

Expected Behavior:

We need a solution that correctly types the field value and ensures the $errors and $value stores are correctly typed as Record<string, string>.

Any guidance or examples on how to achieve this would be greatly appreciated. Thank you!

ciscoheat commented 3 months ago

What is the error you're getting? This works very well for me:

<script lang="ts" context="module">
    type T = Record<string, unknown>;
</script>

<script lang="ts" generics="T extends Record<string, unknown>">
    import { formFieldProxy, type SuperForm, type FormPathLeaves } from '$lib/index.js';

    export let form: SuperForm<T>;
    export let field: FormPathLeaves<T>;

    $: ({ value, errors } = formFieldProxy(form, field));
</script>

<label>
    {field}: <input name={field} bind:value={$value} aria-invalid={$errors ? 'true' : undefined} />
    {#if $errors}<span class="invalid">{$errors}</span>{/if}
</label>
<script lang="ts">
  const superform = superForm(data.form, {
    taintedMessage: false,
    dataType: 'json'
  });
</script>

<RecordField field="message.name" form={superform} />
mpost commented 3 months ago

Thanks for getting back in the issue. In your example you bind to the RecordField

<RecordField field="message.name" form={superform} />

which likely implies that message.name represents a singular value. In my example we want to work with nested data structure and pass that down to the custom component. Eg

{
  message: {
    de: "guten tag",
    en: "good day",
    fr: "bonjour",
    // ...
  }
}

We would want to bind to the data.message field, which should pass down the Record of languages to the underlying <MultiLanguageTextArea>.

tijptjik commented 6 days ago

Just tacking on another example here for what I'd hope would one day be supported:

I'm interested in accessing the constraints for a nested objects where the keys are unknown at compile time.

export const CustomProperty = z.object({
  type: z.enum(customPropertyTypes),
  key: z.string().min(3, { message: 'Key should have at least 3 characters' }),
  ...
})

export const CustomPropertySchema = z.object({
  classifiers: z.record(z.string(), CustomProperty),
  specifiers: z.record(z.string(), CustomProperty),
});

export let ProjectInsert = createInsertSchema(project, {
  metadata: CustomPropertySchema
});

The constraints object returned from running superValidate() with zod(ProjectInsert) - i.e. form = await superValidate(formData, zod(ProjectInsert)); with the following constraints:

metadata:
​​    classifiers: 
​​        required: true
    specifiers :
​​        required: true

The more deeply nested constraints are not available -- I've confirmed with @ciscoheat that this currently cannot be achieved.

Note that validation DOES take the nested rules into consideration, and the errors show up in the right path. However, it would also be great to have access to the constraints of nested fields.