[feature] useHelper about drei

[drcmda]

let's add it!

[drcmda]

users should be able to do:

useHelper<SpotLightHelper>(ref, SpotLightHelper, color)

function useHelper<T>(
  ref: React.MutableRefObject<Object3D>,
  proto: T,
  ...args: ConstructorParameters<T>
) {

[drcmda]

that's fine, thats how i do it as well most of the time, eventually it will annoy someone and they fix it ;-)

[gsimone]

Alright, I've made some progress, I'll share here before a pr as I'm not sure about some details.

https://codesandbox.io/s/r3f-use-helper-bm2qu?file=/src/use-helper.tsx

• I'm not sure how to type the arguments to the hook, the second one should be an helper constructor but I can't seem to find any type in threejs that could work for all helpers
• the rest of the arguments would be passed to the constructor, don't know if is the correct way to do it

[gsimone]

So, I updated the sandbox with your suggestions, still:

useHelper<BoxHelper>(mesh, BoxHelper, "cyan");

gives me a type error "Argument of type 'typeof BoxHelper' is not assignable to parameter of type 'BoxHelper'."

This is fixed using useHelper<typeof BoxHelper>(...) ( and I'm not sure if this is desirable)

in the hook I get these type errors:
Type 'T' does not satisfy the constraint 'new (...args: any) => any'
and
This expression is not constructable. when I try to create the new helper instance.

I'm not sure if there is a simpler approach or I'm missing some piece of ts knowledge to make this work

[drcmda]

i dont know TS myself, but in reactthreefiber they have solved args in /src/three-types.ts maybe you find something over there

[gsimone]

I fixed the warning but I think that made everything too "any"ish, maybe I'll just PR now and open an issue later to have somebody more expert with TS check the typings