add TypeScript definitions

[Narazaka]

I have created TypeScript typedefs for probably almost all of the public APIs. (cf. partial https://github.com/colinbdclark/osc.js/pull/105 )

I've typed them tightly enough that type completion should work in many places.

[colinbdclark]

Thanks @Narazaka, it is really kind of you to share a PR for these type definitions. I'll try to take a closer look this weekend and get it merged in.

[Polyterative]

Any updates on this? I really need types to work with osc in Angular/React

[colinbdclark]

Hi @Polyterative, I'm glad to hear you're excited about this PR. I work on osc.js in my spare time, and it's been very busy recently with a residency and recording, so while this is definitely the next feature I will merge in, I'm not really sure yet when I'll have enough free time to look closely at it.

But you can help! Since it sounds like this is an important feature to you, and you're actually a TypeScript user (whereas I am not), I wonder if you'd be willing to take some time to test out @Narazaka's branch and confirm that it is working as expected for you? It would be very helpful, thanks.

[JoueBien]

I'm running with these types and my first notes would be:

Can we expose types for instances?

How these definitions are set up makes it basically impossible to set any variable type with out it being inferred. To be able to set up a class I have to do:

function UDPPortFunc () {
  return new UDPPort({})
}

class X32 {
  _UDPPort?: ReturnType<typeof UDPPortFunc>
}

and then cast as any because the shape doesn't fit new UDPPort()

Unless I'm missing some obscure way to get the return type off the type for the UDPPort version of PortConstructor?

[lachlansleight]

Hey just a bump here - it looks like this is ready to be merged, the diff is just a little garbled. It's literally just one line that needs to be added to package.json and everything else is new files. TypeScript support is 30 seconds of work from a contributor away ;)

[code-ape]

Voicing need here as well for type definitions. Thanks!

[cronin4392]

Been using this a lot lately. Would love to find out how we can get this merged. I can help out where possible.

[JoueBien]

Having used these types for a year or so I can't say I'd recommend them. The types don't follow half of the TS suggest practices. Things that I would recommend changing:

• the classes constructor's return a types don't match the class definitions.
  • use declare class instead of interface which should fix the above.
• make the arguments, listener and transport types not depending on a quagmire of <T> passed to <T> (some of these are like 5 levels deep) to get the right shape. Changing this would make it easier to use the types in consuming code + make the intellisense show something actually useful.
  • args shouldn't be so complicated type wise - there's a tone of type inheritance where a much more simple set of types would do & make it much easier to use.
  • .on and .once should have a <T> specked for the returned arg[] in the response. This way you can specify what the listener have arg wise with out having to cast it or unwrap it as "all of the above".

[colinbdclark]

Thanks @JoueBien for the confirmation. At some point I hope to work on an API-breaking version of osc.js that fixes some of the issues with the design that have been bugging me for years. At that point I'll try to learn enough TypeScript to produce viable bindings for the library. But honestly, I'm not certain when that will be. In the meantime, if this is important to people in the community, I'm willing to look at pull requests by contributors if they are comprehensive.