fix: honour explicit dependsOnOwnProps=false in getDependsOnOwnProps

[JSap0914]

Problem

getDependsOnOwnProps in src/connect/wrapMapToProps.ts uses a truthy check to decide whether the caller has explicitly set the dependsOnOwnProps flag:

// before
function getDependsOnOwnProps(mapToProps: MapToProps) {
  return mapToProps.dependsOnOwnProps
    ? Boolean(mapToProps.dependsOnOwnProps)
    : mapToProps.length !== 1
}

A truthy check treats false the same as undefined, so an explicitly-set dependsOnOwnProps = false falls through to the length heuristic and is silently overridden.

A concrete case: a mapStateToProps function with 2 parameters but .dependsOnOwnProps = false set (to opt out of ownProps-triggered re-renders) ends up with dependsOnOwnProps = true after the proxy initialises, causing the component to re-run mapStateToProps on every parent prop change even though the author explicitly disabled that.

Fix

Replace the truthy check with a != null check so that both false and true are treated as intentional signals, while undefined/null still falls back to the arity heuristic:

// after
function getDependsOnOwnProps(mapToProps: MapToProps) {
  return mapToProps.dependsOnOwnProps != null
    ? Boolean(mapToProps.dependsOnOwnProps)
    : mapToProps.length !== 1
}

Tests

Four unit tests added in test/connect/wrapMapToProps.spec.ts:

• arity-2 with no explicit flag → inferred true
• arity-1 with no explicit flag → inferred false
• arity-2 with explicit false → respects false (was broken before this fix)
• arity-1 with explicit true → respects true

All 189 existing tests continue to pass unchanged.

AI-assisted contribution.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/react-redux@2331 -D

yarn add https://pkg.pr.new/react-redux@2331.tgz -D

pnpm add https://pkg.pr.new/react-redux@2331.tgz -D

bun add https://pkg.pr.new/react-redux@2331.tgz -D

commit: d659a4b