search

workos / authkit-remix

Authentication and session helpers for using WorkOS & AuthKit with Remix
MIT License
22 stars 7 forks source link

authkitLoader JSON serialization negates Single Fetch capabilities #26

Open ZahraTee opened 3 weeks ago

ZahraTee commented 3 weeks ago

Hey team, we're currently using Authkit with Next and looking into switching over to Remix, so have started using this package while we try it out.

We're taking advantage of Remix's Single Fetch data loading strategy, which is currently feature flagged but is turned on by default for new Remix projects and will be the default in React Router 7. The upshot is that you can return a naked object from loaders with non-JSON serializable data types (like Date, Map or Set), which previously would've been automatically serialized to JSON, even if you didn't explicitly wrap it in json(). We leverage this quite a bit as we were already doing these conversions in Next's BFF to make the data ready to consume in the frontend.

However, since authkitLoader actually does serialize the data explicitly using json(), and since we use it pretty heavily, this means we can't get the benefits of Single Fetch. Our team can work around this temporarily, but I wanted to see if this was on your radar and if this is something you plan to support in the near future.

PaulAsjes commented 2 weeks ago

Thanks for the heads up! We're hoping on doing a bit of a sprint on features for this library soon and I think this is a great candidate. Will keep you updated as we scope out further.