cleanup package.json

[benmccann]

• added types to exports
• removed module which will be ignored since exports is present
• removed require since it duplicates default
• add type field because node may change the default in the future

[marijnh]

The module field will be useful to some older tools. "type": "commonjs" is the default, so I'm not sure why you'd add that.

[benmccann]

I've added module back and removed type

[marijnh]

Is there any situation in which TypeScript will not find the types as they currently are, and needs the types field?

[benmccann]

It's usually necessary if you're dual publishing but I'm not sure if there's case where it's strictly necessary for a cjs-only package

[marijnh]

This all feels very much like a patch-for-the-sake-of-a-patch. Unless there is an actual concrete problem you're solving with these changes, I'm leaning towards closing this PR.

[benmccann]

Fair enough