TEDERIs

engineStreamingRequestModel with `addRef = false` can be considered as "dear function, please try to load a model. But if it fails it's OK, at least you tried." `addRef = true`...

I think it's even better to replace **addRef = false** with **persistent = false**. It's more illustrative.

> We can't use `std::unordered_map` to retrieve table arguments in a Lua definition (because a script providing the wrong type ie `{1, 2, 3}` will cause a crash, and I...

LGTM

> > There's the module FileSystem which already does the same and has some more features/functions btw. > > Although could be added nativelly yes if the only purpose is...

The derefencing such as `for (const auto& pWater : *g_pGame->GetWaterManager())` is counterintuitive and makes it difficult to read the code. It's better to add a dedicated method for getting inner...

I think that the length of these constants makes them cumbersome and difficult to use. I doubt that long constructions like `constants.VehicleLightOverride.Disable` can be usefull. I would suggest to make...

In addition, I would suggest to make the table read-only.

As you decided to refactor the existing code I would suggest to take a look at **CRegistryResultCell**. It stores **nVal**, **fVal** and **pVal** all the time but only one can...

Great. But I think you should use the new Lua parser.