Remove @remix-run/node as a direct dependency

[BenJeau]

Hi 👋,

Thanks for such a great library! I was looking at the dependency tree of my React vitejs-based application and I saw a lot of dependencies (50) coming from @tanstack/react-form related to remix. More than half of my dependencies comes from @remix-run/node from @tanstack/react-form.

Would it be possible to make it optional somehow?

I'm unfamiliar with how npm/node.js works in that regard, but I feel like this dependency might have been missed and shouldn't be included especially if the user of the library isn't using remix.

It seems to only be used here https://github.com/TanStack/form/blob/a4400289b14ad2a9e27a6f9b664891b1604bc163/packages/react-form/src/start/utils.ts

Let me know what you think and if there's a way to achieve this or if its completely unfeasible.

Thanks!

[mremondi]

Would also love to see this change. [email protected] has a vulnerability in it (albeit low risk) that I would love to either upgrade or get rid of this dependency altogether

[Balastrong]

That's a valid point, let's check if we still need that package or if we can remove it.

Since the Start adapter is currently broken (breaking changes on Start) it might be the chance to also have a look at this dependency!

See #1023

[stephen776]

Hello- I’ll add that the dependency on remix v2.15 packages will break things for anyone trying to use react-router v7 (the upgrade path from remix v2)

[stephen776]

Also - can i ask just to clarify… what is the reasoning behind putting the form error state into a cookie in the first place? Just not sure I was following looking at the code for the start integration