Having ´"node"in there actually makes the types for node available in the project. this makes callingrequire` etc. compile. Obviously, this will not work at runtime in the browser, so it should be removed.
Unrelated to that, I suggest to move from foundry-pc-types to foundry-vtt-types.