Export WorkerPool related types to make pool typing easier

[sebqq]

Hello. First of all, thank you for this great library.

Would it be possible to export ExposedToThreadType, QueuedTask and WorkerPool types to use in our apps? With these types available it would be much easier to type Pool and/or array of tasks inside our apps like:

import { Pool, spawn, ExposedToThreadType, QueuedTask, WorkerPoolType } from 'threads';

...

class App {
    queue: QueuedTask<ExposedToThreadType<OurWorkerModule>, boolean>[];
    pool: WorkerPool<ExposedToThreadType<OurWorkerModule>>;
...

or is there any other way to type these variables? Thank you!

[andywer]

Hi @sebqq, generally speaking there is no real reason against it.

The resulting code doesn't look too nice, though… We should rename some of them if we want to expose them to the user. How about something like this?

import { spawn, ExposedToThreadType, Pool, QueuedTask } from 'threads';

...

type OurWorkerThreadApi = ExposedAs<OurWorkerModule>;

class App {
    queue: QueuedTask<OurWorkerThreadApi, boolean>[];
    pool: Pool<OurWorkerThreadApi>;
...

PS: Was there a particular reason why you didn't use Pool to type App#pool?

[sebqq]

That API looks really nice! It would be great.

I didn't use Pool because somehow I was not able to make it work that way. In fact, I'm currently even using patch-package to export these types by myself 😃

EDIT:

PS: Was there a particular reason why you didn't use Pool to type App#pool?

Actually you're right. I could just use Pool<OurWorkerThreadApi> as you shown in your example. :)