microlinkhq / browserless / 31309275340

81%

push

github

fix(function): resolve nodePaths under an isolated store (#875)

`nodePaths` tells isolated-function which dependencies a snippet requires
are already on disk, so it can skip installing them. It was derived by
walking two levels up from `@cloudflare/puppeteer`, which lands on the
consumer's `node_modules` only under a flat install. With pnpm it lands
inside `node_modules/.pnpm/@cloudflare+puppeteer@1.2.0/node_modules`,
which holds that package's own dependencies and nothing else:

    entries: @cloudflare @puppeteer debug devtools-protocol ws

So the check never matched and every snippet paid a full install of a
package already present — measured at ~400ms per cold compile, three
rounds, consistent.

`module.paths` is the resolution chain Node itself would use from this
file, so it reaches the consumer under either layout. The original entry
is kept ahead of it, leaving a flat install byte-identical.


Claude-Session: https://claude.ai/code/session_01YKVuh1R8HtB7SX7eKa7UsJ

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>

728 of 888 branches covered (81.98%)

Branch coverage included in aggregate %.

8 of 8 new or added lines in 1 file covered. (100.0%)

3841 of 4777 relevant lines covered (80.41%)

104892.61 hits per line