Typescript Optional rvn

[tzarger]

Hello,

I am seeing something that potentially could be error prone or at least when you think you have loaded or query, you may have a null error runtime

I see in your tests that you do something like:
await req.rvn?.store(test)
... and ...
await req.rvn?.people?.store(person)

Is there any way to type that so we do not need to know if nullable or not? Even when using
const routeOptions = { rvn: { autoSession: true } }; it still shows as nullable.

[simoneb]

req.rvn will only be there if you enable it explicitly, meaning that we can't know whether you did or not. I think the current behavior is correct, and if you want to avoid typing issues you should just tell TS that in your route it is there using req.rvn!

[ilteoood]

In addition to what @simoneb said, you can always extend the FastifyRequest and remove the optional constraint