Fix pnpm version check logic.

[iclanton]

I tested upgrading pnpm-sync-lib on microsoft/rushstack#5247. After making the changes in this branch, I'm still getting this error:

>node /Users/ianc/code/rushstack5/libraries/rush-lib/lib/start --debug update

Rush Multi-Project Build Tool 5.153.2 (unmanaged) - https://rushjs.io
Node.js version is 22.12.0 (LTS)


Starting "rush update"
Installing for subspace: build-tests-subspace
Checking Git policy for this repository.


Found files in the "common/git-hooks" folder.
Successfully installed these Git hook scripts: pre-commit

Trying to acquire lock for pnpm-9.15.9
Acquired lock for pnpm-9.15.9
Found pnpm version 9.15.9 in /Users/ianc/.rush/node-v22.12.0/pnpm-9.15.9

Symlinking "/Users/ianc/code/rushstack2/common/temp/pnpm-local"
  --> "/Users/ianc/.rush/node-v22.12.0/pnpm-9.15.9"
Transforming /Users/ianc/code/rushstack2/common/config/subspaces/build-tests-subspace/.npmrc
  --> "/Users/ianc/code/rushstack2/common/temp/build-tests-subspace/.npmrc"

Updating workspace files in /Users/ianc/code/rushstack2/common/temp/build-tests-subspace
Total amount of time spent to hash related package.json files in the injected installation case: 0.42 seconds
Copying "/Users/ianc/code/rushstack2/common/config/subspaces/build-tests-subspace/pnpm-lock.yaml"
  --> "/Users/ianc/code/rushstack2/common/temp/build-tests-subspace/pnpm-lock.yaml"
Copying "/Users/ianc/code/rushstack2/common/config/subspaces/build-tests-subspace/pnpm-lock.yaml"
  --> "/Users/ianc/code/rushstack2/common/temp/build-tests-subspace/pnpm-lock-preinstall.yaml"

Checking installation in "/Users/ianc/code/rushstack2/common/temp/build-tests-subspace"

Skipping package registry setup because packageRegistry.enabled=false
Running "pnpm install" in /Users/ianc/code/rushstack2/common/temp/build-tests-subspace


Invoking package manager: /Users/ianc/.rush/node-v22.12.0/pnpm-9.15.9/node_modules/pnpm/bin/pnpm.cjs install --store /Users/ianc/code/rushstack2/common/temp/pnpm-store --config.cacheDir=/Users/ianc/code/rushstack2/common/temp/pnpm-store --config.stateDir=/Users/ianc/code/rushstack2/common/temp/pnpm-store --prefer-frozen-lockfile --strict-peer-dependencies --config.auto-install-peers=false --config.resolutionMode=highest --config.ignoreCompatibilityDb --recursive --link-workspace-packages false --reporter default

Scope: all 5 workspace projects
.../build-tests-subspace/rush-lib-test   |  WARN  deprecated [email protected]
 WARN  10 deprecated subdependencies found: @humanwhocodes/[email protected], @humanwhocodes/[email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
.                                        |  +32 +++
Progress: resolved 836, reused 836, downloaded 0, added 32, done
Done in 2.5s using pnpm v9.15.9

pnpm-sync: Starting operation...
pnpm-sync: pnpm-lock.yaml file path: /Users/ianc/code/rushstack2/common/temp/build-tests-subspace/pnpm-lock.yaml
pnpm-sync: .pnpm folder path: /Users/ianc/code/rushstack2/common/temp/build-tests-subspace/node_modules/.pnpm
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/rush-lib/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/terminal/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/apps/heft/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/rigs/local-node-rig/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/rush-sdk/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/eslint/eslint-config/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/heft-plugins/heft-lint-plugin/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/heft-plugins/heft-typescript-plugin/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/heft-config-file/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/lookup-by-path/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/node-core-library/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/package-deps-hash/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/package-extractor/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/rig-package/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/stream-collator/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/ts-command-line/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/operation-graph/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/apps/api-extractor/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/rigs/heft-node-rig/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/eslint/local-eslint-config/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/eslint/eslint-patch/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/eslint/eslint-plugin/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/eslint/eslint-plugin-packlets/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/eslint/eslint-plugin-security/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/api-extractor-model/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/heft-plugins/heft-api-extractor-plugin/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/heft-plugins/heft-jest-plugin/node_modules/.pnpm-sync.json
pnpm-sync: Writing /Users/ianc/code/rushstack2/libraries/tree-pattern/node_modules/.pnpm-sync.json
pnpm-sync: Regenerated .pnpm-sync.json in 22 ms

Installing for subspace: default
Checking Git policy for this repository.


Found files in the "common/git-hooks" folder.
Successfully installed these Git hook scripts: pre-commit

Trying to acquire lock for pnpm-9.15.9
Acquired lock for pnpm-9.15.9
Found pnpm version 9.15.9 in /Users/ianc/.rush/node-v22.12.0/pnpm-9.15.9

Symlinking "/Users/ianc/code/rushstack2/common/temp/pnpm-local"
  --> "/Users/ianc/.rush/node-v22.12.0/pnpm-9.15.9"
Transforming /Users/ianc/code/rushstack2/common/config/subspaces/default/.npmrc
  --> "/Users/ianc/code/rushstack2/common/temp/default/.npmrc"

Updating workspace files in /Users/ianc/code/rushstack2/common/temp/default
Total amount of time spent to hash related package.json files in the injected installation case: 0.00 seconds
Copying "/Users/ianc/code/rushstack2/common/config/subspaces/default/pnpm-lock.yaml"
  --> "/Users/ianc/code/rushstack2/common/temp/default/pnpm-lock.yaml"
Copying "/Users/ianc/code/rushstack2/common/config/subspaces/default/pnpm-lock.yaml"
  --> "/Users/ianc/code/rushstack2/common/temp/default/pnpm-lock-preinstall.yaml"

Checking installation in "/Users/ianc/code/rushstack2/common/temp/default"

Skipping package registry setup because packageRegistry.enabled=falseDeleting files from /Users/ianc/code/rushstack2/common/temp/default/node_modules

Running "pnpm install" in /Users/ianc/code/rushstack2/common/temp/default


Invoking package manager: /Users/ianc/.rush/node-v22.12.0/pnpm-9.15.9/node_modules/pnpm/bin/pnpm.cjs install --store /Users/ianc/code/rushstack2/common/temp/pnpm-store --config.cacheDir=/Users/ianc/code/rushstack2/common/temp/pnpm-store --config.stateDir=/Users/ianc/code/rushstack2/common/temp/pnpm-store --prefer-frozen-lockfile --strict-peer-dependencies --config.auto-install-peers=false --config.resolutionMode=highest --config.ignoreCompatibilityDb --recursive --link-workspace-packages false --reporter default

Scope: all 154 workspace projects
.../heft-node-basic-tutorial             |  WARN  deprecated [email protected]
../../../libraries/rush-lib              |  WARN  deprecated [email protected]
.../rush-vscode-extension                |  WARN  deprecated [email protected]
 WARN  40 deprecated subdependencies found: @aws-cdk/[email protected], @aws-cdk/[email protected], @aws-cdk/[email protected], @babel/[email protected], @babel/[email protected], @babel/[email protected], @babel/[email protected], @babel/[email protected], @babel/[email protected], @fluentui/[email protected], @humanwhocodes/[email protected], @humanwhocodes/[email protected], @humanwhocodes/[email protected], @humanwhocodes/[email protected], @humanwhocodes/[email protected], @npmcli/[email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
.                                        |    +2729 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Progress: resolved 2734, reused 2664, downloaded 0, added 2729, done

The command failed:
 /Users/ianc/code/rushstack2/common/temp/pnpm-local/node_modules/.bin/pnpm install --store /Users/ianc/code/rushstack2/common/temp/pnpm-store --config.cacheDir=/Users/ianc/code/rushstack2/common/temp/pnpm-store --config.stateDir=/Users/ianc/code/rushstack2/common/temp/pnpm-store --prefer-frozen-lockfile --strict-peer-dependencies --config.auto-install-peers=false --config.resolutionMode=highest --config.ignoreCompatibilityDb --recursive --link-workspace-packages false --reporter default
ERROR: Error: Process exited with code 1
Giving up after 1 attempts


ERROR: Process exited with code 1

Error: Process exited with code 1
    at ChildProcess.<anonymous> (/Users/ianc/code/rushstack5/libraries/node-core-library/lib/Executable.js:331:28)
    at ChildProcess.emit (node:events:524:28)
    at maybeClose (node:internal/child_process:1101:16)
    at ChildProcess._handle.onexit (node:internal/child_process:304:5)