Removed redundant check for NaN in parseShardPair

[Connormiha]

Summary

The final .filter(shard => !Number.isNaN(shard)) was redundant and safely removed.

Here’s why:

The previous .filter(d => /^\d+$/.test(d)) already ensures that only strings containing digits only (e.g., "123") are passed to .map(...).

parseInt("123", 10) will always return a valid number (123), not NaN, when the input is purely numeric.

Therefore, checking !Number.isNaN(shard) after parsing is unnecessary — it's dead code that doesn't affect the result.

The cleanup removes an unnecessary filter() step that was guarding against a case that cannot occur due to the previous regex check. It simplifies the code while keeping the same functionality.

Test plan

[netlify]

✅ Deploy Preview for jestjs ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	ec83be0
🔍 Latest deploy log	https://app.netlify.com/projects/jestjs/deploys/687f85f7ff4c950008f7830d
😎 Deploy Preview	https://deploy-preview-15742--jestjs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[pkg-pr-new]

Open in StackBlitz

babel-jest

npm i https://pkg.pr.new/babel-jest@15742

babel-plugin-jest-hoist

npm i https://pkg.pr.new/babel-plugin-jest-hoist@15742

babel-preset-jest

npm i https://pkg.pr.new/babel-preset-jest@15742

create-jest

npm i https://pkg.pr.new/create-jest@15742

@jest/diff-sequences

npm i https://pkg.pr.new/@jest/diff-sequences@15742

expect

npm i https://pkg.pr.new/expect@15742

@jest/expect-utils

npm i https://pkg.pr.new/@jest/expect-utils@15742

jest

npm i https://pkg.pr.new/jest@15742

jest-changed-files

npm i https://pkg.pr.new/jest-changed-files@15742

jest-circus

npm i https://pkg.pr.new/jest-circus@15742

jest-cli

npm i https://pkg.pr.new/jest-cli@15742

jest-config

npm i https://pkg.pr.new/jest-config@15742

@jest/console

npm i https://pkg.pr.new/@jest/console@15742

@jest/core

npm i https://pkg.pr.new/@jest/core@15742

@jest/create-cache-key-function

npm i https://pkg.pr.new/@jest/create-cache-key-function@15742

jest-diff

npm i https://pkg.pr.new/jest-diff@15742

jest-docblock

npm i https://pkg.pr.new/jest-docblock@15742

jest-each

npm i https://pkg.pr.new/jest-each@15742

@jest/environment

npm i https://pkg.pr.new/@jest/environment@15742

jest-environment-jsdom

npm i https://pkg.pr.new/jest-environment-jsdom@15742

@jest/environment-jsdom-abstract

npm i https://pkg.pr.new/@jest/environment-jsdom-abstract@15742

jest-environment-node

npm i https://pkg.pr.new/jest-environment-node@15742

@jest/expect

npm i https://pkg.pr.new/@jest/expect@15742

@jest/fake-timers

npm i https://pkg.pr.new/@jest/fake-timers@15742

@jest/get-type

npm i https://pkg.pr.new/@jest/get-type@15742

@jest/globals

npm i https://pkg.pr.new/@jest/globals@15742

jest-haste-map

npm i https://pkg.pr.new/jest-haste-map@15742

jest-jasmine2

npm i https://pkg.pr.new/jest-jasmine2@15742

jest-leak-detector

npm i https://pkg.pr.new/jest-leak-detector@15742

jest-matcher-utils

npm i https://pkg.pr.new/jest-matcher-utils@15742

jest-message-util

npm i https://pkg.pr.new/jest-message-util@15742

jest-mock

npm i https://pkg.pr.new/jest-mock@15742

@jest/pattern

npm i https://pkg.pr.new/@jest/pattern@15742

jest-phabricator

npm i https://pkg.pr.new/jest-phabricator@15742

jest-regex-util

npm i https://pkg.pr.new/jest-regex-util@15742

@jest/reporters

npm i https://pkg.pr.new/@jest/reporters@15742

jest-resolve

npm i https://pkg.pr.new/jest-resolve@15742

jest-resolve-dependencies

npm i https://pkg.pr.new/jest-resolve-dependencies@15742

jest-runner

npm i https://pkg.pr.new/jest-runner@15742

jest-runtime

npm i https://pkg.pr.new/jest-runtime@15742

@jest/schemas

npm i https://pkg.pr.new/@jest/schemas@15742

jest-snapshot

npm i https://pkg.pr.new/jest-snapshot@15742

@jest/snapshot-utils

npm i https://pkg.pr.new/@jest/snapshot-utils@15742

@jest/source-map

npm i https://pkg.pr.new/@jest/source-map@15742

@jest/test-result

npm i https://pkg.pr.new/@jest/test-result@15742

@jest/test-sequencer

npm i https://pkg.pr.new/@jest/test-sequencer@15742

@jest/transform

npm i https://pkg.pr.new/@jest/transform@15742

@jest/types

npm i https://pkg.pr.new/@jest/types@15742

jest-util

npm i https://pkg.pr.new/jest-util@15742

jest-validate

npm i https://pkg.pr.new/jest-validate@15742

jest-watcher

npm i https://pkg.pr.new/jest-watcher@15742

jest-worker

npm i https://pkg.pr.new/jest-worker@15742

pretty-format

npm i https://pkg.pr.new/pretty-format@15742

commit: ec83be0

[cpojer]

Thanks. Maybe it was vibe coded.