Bryce Kahle
Bryce Kahle
The `size` is actually in the results, just not graphed. See https://github.com/pnpm/node-package-manager-benchmark/blob/d87db894c1ebc3b7edbccf6f73591f5730574483/results/pnpm/1.9.0/react-app.yaml#L6 for example.
> you need privilege escalation to install the NPM driver I believe this is accurate. @derekwbrown can you confirm?
I would recommend avoiding `request`. We specifically switched away from it because it is too heavy for a library that needs to run in the browser.
Can you bump to 60s? We have some requests that can take quite awhile (compiling, flashing, etc.).
@nplanel I think this can wait for 7.40 since it doesn't affect any production code.
> We can significantly bring down this size by stripping dwarf debug sections. Let's at least do this in CI for the packaged build. I think it is less important...
What the reason behind the `protodep` package, given the `agent-payload` repo supports go modules now?
> Also, this PR does not introduce protodep, we have been using it for a while. This PR does add the `agent-payload` to protodep. Since it wasn't necessary before, I'm...
> I'm not sure if I'm missing something from your question I think the `dep` part of `protodep` was confusing me, especially since it uses the same TOML file format.
It seems like some of this could be solved by handling a closed connection later in the lifecycle when it isn't possible for additional data to be sent/recv. Have you...