Fix TypeScript declaration for sync module

[benmccann]

I added sync.d.ts in #54, but missed that we can't have two declaration files because package.json refers explicitly to index.d.ts:

tiny-glob/package.json

Line 37 in cbc19bd

"types": "./index.d.ts"

This also cuts down on code duplication

[lukeed]

Unfortunately I don;t think this works either. I've had my fair share of this issue because most of my modules have a secondary "mode"

The only thing I've found to work consistently is:

/sync
  |- index.js
  |- index.mjs (optional)
  |- index.d.ts
index.js (main)
index.mjs (optional)
index.d.ts (main definitions)

[benmccann]

What do you do with the types field in package.json in that case? It can only refer to one file, so I'm not sure how you'd have index.d.ts and sync/index.d.ts

[benmccann]

I just tested this three different ways:

• the code in master - works just fine
• the way you suggested - works just fine
• this PR - doesn't work

It's very surprising to me that package.json has this types field that doesn't seem necessary and doesn't seem to have an effect. I'll go ahead and close this since things are working as is. Thanks for the extra set of eyes on this

[lukeed]

To answer your question, you just define "types" as the root/main mode. /sync/index.d.ts gets loaded automatically via the /sync import

It's all very weird and loose