Title: assert.deepEqual([null], [undefined]) behavior changed in v22.16.0 (undocumented breaking change) · Issue #61583 · nodejs/node · GitHub
Open Graph Title: assert.deepEqual([null], [undefined]) behavior changed in v22.16.0 (undocumented breaking change) · Issue #61583 · nodejs/node
X Title: assert.deepEqual([null], [undefined]) behavior changed in v22.16.0 (undocumented breaking change) · Issue #61583 · nodejs/node
Description: Version v22.16.0+ (and v24.x) Platform All platforms Subsystem assert What steps will reproduce the bug? const assert = require('assert'); assert.deepEqual([null], [undefined]); How often does it reproduce? Is there a required condition?...
Open Graph Description: Version v22.16.0+ (and v24.x) Platform All platforms Subsystem assert What steps will reproduce the bug? const assert = require('assert'); assert.deepEqual([null], [undefined]); How often does it r...
X Description: Version v22.16.0+ (and v24.x) Platform All platforms Subsystem assert What steps will reproduce the bug? const assert = require('assert'); assert.deepEqual([null], [undefined]); How often d...
Opengraph URL: https://github.com/nodejs/node/issues/61583
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"assert.deepEqual([null], [undefined]) behavior changed in v22.16.0 (undocumented breaking change)","articleBody":"### Version\n\nv22.16.0+ (and v24.x)\n\n### Platform\n\nAll platforms\n\n### Subsystem\n\nassert\n\n### What steps will reproduce the bug?\n\n```javascript\nconst assert = require('assert');\nassert.deepEqual([null], [undefined]);\n```\n\n### How often does it reproduce? Is there a required condition?\n\nAlways reproduces on Node \u003e= 22.16.0. Passes on Node \u003c= 22.15.0.\n\n### What is the expected behavior? Why is that the expected behavior?\n\nThe assertion should **pass** because:\n\n1. The [documentation](https://nodejs.org/api/assert.html#assertdeepequalactual-expected-message) states: \"Primitive values are compared with the `==` operator\"\n2. In JavaScript, `null == undefined` is `true`\n3. This worked in all Node versions prior to 22.16.0\n\n### What do you see instead?\n\n```\nAssertionError [ERR_ASSERTION]: Expected values to be loosely deep-equal:\n\n[ null ]\n\nshould loosely deep-equal\n\n[ undefined ]\n```\n\n### Root Cause\n\nPR #57619 (labeled as \"performance improvement\") introduced this change. The modification in [`objEquiv`](https://github.com/nodejs/node/pull/57619/files#diff-2da3538f1eacd02ed1b3c8f7fe3f8c0cee640295830a6f0dd4a6205326578958R785-R793) now checks `a[i] !== undefined` with **strict equality** (`===`) before calling `innerDeepEqual`, which would have used loose equality (`==`):\n\n```javascript\n// Before: called innerDeepEqual() which respects loose equality mode\nif (!innerDeepEqual(a[i], b[i], mode, memos)) { return false; }\n\n// After: checks with strict equality BEFORE calling innerDeepEqual\nif (b[i] === undefined) {\n if (a[i] !== undefined || !hasOwn(a, i)) // \u003c-- uses === not ==\n return false;\n}\n```\n\nWhen comparing `[null]` vs `[undefined]`:\n- `b[0]` is `undefined` → enters first branch \n- `a[0] !== undefined` → `null !== undefined` is `true` (strict equality)\n- Returns `false` immediately, never calls `innerDeepEqual` which would use `==`\n\n### Additional information\n\n- This is an **undocumented breaking change** in an LTS release\n- The PR was not labeled as semver-major\n- Affects anyone comparing arrays that may contain `null` values against expected `undefined` values (common with MongoDB/BSON which stores `undefined` as `null`)\n\nI understand `deepEqual` is marked as \"Legacy\" and docs warn it \"can have surprising results\", but this specific change contradicts the documented behavior and broke existing code without notice.","author":{"url":"https://github.com/iholovin","@type":"Person","name":"iholovin"},"datePublished":"2026-01-30T08:04:02.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":4},"url":"https://github.com/61583/node/issues/61583"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:11f1c62e-2e5f-c994-ee6e-2dfa8efead1e |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 8C0C:C61B2:1BE7AC8:28480BD:6A4CFAE7 |
| html-safe-nonce | 26cfb1a599b0533655eb414856bd9b8edfcb1528f487e3f7833e2726f43db595 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4QzBDOkM2MUIyOjFCRTdBQzg6Mjg0ODBCRDo2QTRDRkFFNyIsInZpc2l0b3JfaWQiOiI4ODU3MjI2OTc4NTk3MzM3ODMxIiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 05a7c3670de349f3d58ee823a8d6c9e6d2e551c9096910d3cdc93edafc1d363f |
| hovercard-subject-tag | issue:3874531346 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/nodejs/node/61583/issue_layout |
| twitter:image | https://opengraph.githubassets.com/68363f323a544d8ab611f8c110fd517ca4d41a1611a9e3249c2192b9715e7efc/nodejs/node/issues/61583 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/68363f323a544d8ab611f8c110fd517ca4d41a1611a9e3249c2192b9715e7efc/nodejs/node/issues/61583 |
| og:image:alt | Version v22.16.0+ (and v24.x) Platform All platforms Subsystem assert What steps will reproduce the bug? const assert = require('assert'); assert.deepEqual([null], [undefined]); How often does it r... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | iholovin |
| hostname | github.com |
| expected-hostname | github.com |
| None | 31e4db13a9e20081f2fac78f441659a6576e582bbb15e153f9c7fb6473aa29f5 |
| turbo-cache-control | no-preview |
| go-import | github.com/nodejs/node git https://github.com/nodejs/node.git |
| octolytics-dimension-user_id | 9950313 |
| octolytics-dimension-user_login | nodejs |
| octolytics-dimension-repository_id | 27193779 |
| octolytics-dimension-repository_nwo | nodejs/node |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 27193779 |
| octolytics-dimension-repository_network_root_nwo | nodejs/node |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 8307e9ba117a26e92a64fb1daf085d795a6552ed |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width