fix: queue not processing as expected with multiple requests made

[vidarc]

re: #104

not at all used to programming in this language, but this fixed my issue and the other tests still passed. happy to make any other changes if this is not the best way to do this

[vidarc]

@jasnell thoughts? If you have time

[vidarc]

@metcoder95

[vidarc]

maybe I'm missing something, but when running npm run build and npm run test locally, it will generate a file with the name @piscina+locks.node instead of what is currently in the prebuilds folder (node.napi.node). looks like it defaults to using the node.napi.node (or prefers it) instead of the one that is built locally for me. so the pipeline wasn't actually using my code changes from what i'm guessing.

[metcoder95]

You are right, it will prefer always the ones build within prebuilts; that part hasn't been automated yet, but definitely would be nice to not need to pre-built it and rather just let the CI do the job.

[metcoder95]

Can you bring back the prebuilds or couldn't produce them locally?

It seems you were having different output on these files

[vidarc]

I can only produce the MacOS one locally. I don't have access to the other machines to produce those.

[vidarc]

@metcoder95 ok, noticed that the pipeline uploads the built artifacts. downloaded those and added them to this PR. only prebuilt that I would be missing now is the darwin-x64 one, but i only have a M1 Mac, so can't build that myself.

[metcoder95]

Perfect! Thanks, let me check them out and see if I can find a way to generate the x64 one

[vidarc]

@metcoder95 https://github.com/actions/runner-images

added the macos-15-intel option to hopefully build out the missing prebuilt

[metcoder95]

ok, I see what's happening; the change on the module naming cause builds to output another name. I'm opening a PR to address it

[metcoder95]

Can you rebase with latest current?
New changes are there

[metcoder95]

The binaries will be added upon every release