preinstall script in v2.23.2 breaks npm/yarn installs when used as a dependency

[jmills4122]

apify-client@2.23.2 added "preinstall": "npx only-allow pnpm" to package.json (via #883). This appears to be intended to enforce pnpm for local development, but the script ships in the published npm tarball and fires when apify-client is installed as a dependency of other packages.

This breaks npm install -g apify-cli and any project that depends on apify-client via npm or yarn:

  npm error path .../node_modules/apify-client
  npm error command failed
  npm error command sh -c npx only-allow pnpm

Potential fix: Either remove the preinstall script from the published tarball , or rely on the packageManager field + Corepack which only applies at the repo level. Note that only-allow is archived and has a known issue with firing during dependency installation.

Affected: v2.23.2 only (v2.23.1 is fine)

[jmills4122]

I opened a pr with a proposal fix here since it looks like packageManager was already being used to enforce pnpm for dev.