Remix clientAction breaks on form reset

[edmundhung]

Discussed in #476

^{Originally posted by QzCurious February 21, 2024}

Describe the bug and the expected behavior

Submitting a conform driven <Form /> by "press Enter on input" should not make a real POST submission to server. It should behave the same as clicking a submit button.

Quick comparison:

	Submit by click submit button	Submit by press Enter
<Form />	Acts the same as others bandicam.2024-02-23.17-38-15-684.mp4	Acts the same as others bandicam.2024-02-23.17-39-02-273.mp4
<Form /> with conform	Acts the same as others bandicam.2024-02-23.17-50-41-381.mp4	Bug: It acts the same as others AND make a real POST submission to server bandicam.2024-02-23.17-51-57-065.mp4

I've tried hard to make a very minimum reproduction to reveal root cause. I believe the bug is occurring with all of these conditions are met:

• clientAction reply with { resetForm: true }

  export const clientAction = async ({ request }: ClientActionFunctionArgs) => {
    const formData = await request.formData();
    const submission = parseWithZod(formData, { schema });
    return submission.reply({ resetForm: true });
  };

• useForm with lastResult (of cause) AND shouldValidate being "onBlur"

  const [form, fields] = useForm({
    shouldValidate: "onBlur",
    lastResult,
  });

Apply one of the following changes workaround the bug:

• Submitting the <Form /> by submit button works fine
• submission.reply(), that is, without { resetForm: true }
• Use action instead of clientAction (But I want a SPA app)
• shouldValidate being onInput or onSubmit

Conform version

v1.0.2

Steps to Reproduce the Bug or Issue

1. git clone https://github.com/QzCurious/conform-remix-vite-form-submission-bug
2. It's a Remix Vite App
3. pnpm i
4. pnpm dev and open the web page
5. Focus on "remix <Form /> with conform" and press Enter
6. Bug happens: <Form /> is handled by clientAction, AND it make a real POST submission to server.

What browsers are you seeing the problem on?

No response

Screenshots or Videos

Screen.Recording.2024-02-22.at.12.16.31.AM.mov

Additional context

Remix with Vite were just marked stable today.