Ability to abort a `directoryOpen`

[Amatewasu]

When calling the function directoryOpen({ recursive: true }) and the user selects a large folder (such as C:\ for Windows users for example), it will try to load too many files and the promise will not resolve.

It would be great to add the ability to abort. I don't know what could be exactly the best parameters to add, but I can think of:

• maxFiles: natural int (after maxFiles found, either the promise resolves prematurely or rejects (second option is best imo))
• timeout: natural int (after timeout ms, the promise is rejected?)
• check (a callback that is called with some information about the folder, such as its name, and then if the callback returns a falsy value, the promise is rejected before opening the recursive folders?)

[tomayac]

I think a timeout would be the most logical. You can do this without any changes actually:

Promise.race([
  directoryOpen({ recursive: true }),
  sleep(2000),
]);

[Amatewasu]

Great idea, thank you!

[Amatewasu]

I identify two problems with Promise.race.

• it does not resolve/reject the directoryOpen promise that still runs in the background, it could be quite resource-intensive if the user selected a large folder
• it includes the elapsed time for the user to select the folder, thereby Promise.race will resolve/reject after the sleep duration if the user takes a while to select the folder

[tomayac]

• it does not resolve/reject the directoryOpen promise that still runs in the background, it could be quite resource-intensive if the user selected a large folder

You could maybe make this work by leveraging the legacySetup parameter.

• it includes the elapsed time for the user to select the folder, thereby Promise.race will resolve/reject after the sleep duration if the user takes a while to select the folder

That's true. I don't think there is a workaround unfortunately.