Skip to content

util: add skipPrototype option to deep equality checks #59762

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
nodejs-github-bot merged 17 commits into from
Sep 14, 2025
Merged

util: add skipPrototype option to deep equality checks #59762

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
3cd8765
lib: update isDeepStrictEqual to support options
miguelmarcondesf Sep 4, 2025
0a88d1b
doc: add options parameter to util.isDeepStrictEqual
miguelmarcondesf Sep 4, 2025
cf0d1eb
lib: add skipPrototypeComparison option to deepStrictEqual for Assert…
miguelmarcondesf Sep 4, 2025
5d5a6f8
test: add skipPrototypeComparison tests for Assert class
miguelmarcondesf Sep 4, 2025
68d6d79
doc: add skipPrototypeComparison option to Assert class
miguelmarcondesf Sep 4, 2025
c9cef45
lib: update notDeepStrictEqual to support skipPrototypeComparison option
miguelmarcondesf Sep 4, 2025
540097a
test: enhance skipPrototypeComparison tests for Assert class
miguelmarcondesf Sep 4, 2025
e8ca02a
test: add skipPrototypeComparison tests for util.isDeepStrictEqual
miguelmarcondesf Sep 4, 2025
642911b
lib: add missing proto
miguelmarcondesf Sep 5, 2025
1532570
lib: add missing proto
miguelmarcondesf Sep 5, 2025
dc0c5bc
lib: refine keyCheck mode condition for strict comparisons
miguelmarcondesf Sep 5, 2025
d14f030
doc: update examples to use class syntax for Foo and Bar
miguelmarcondesf Sep 5, 2025
3c6bf3c
lib: remove unecessary option check
miguelmarcondesf Sep 5, 2025
01daebd
doc: update missing pr number
miguelmarcondesf Sep 8, 2025
b3a4f56
doc: update missing pr number
miguelmarcondesf Sep 8, 2025
87e11ab
lib: simplify isDeepStrictEqual options handling
miguelmarcondesf Sep 8, 2025
0ae1910
lib: update skipPrototypeComparison to skipPrototype
miguelmarcondesf Sep 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
lib: add missing proto 
Co-authored-by: Jordan Harband <ljharb@gmail.com>
  • Loading branch information
@miguelmarcondesf @ljharb
miguelmarcondesf and ljharb authored Sep 5, 2025
commit 153257057822ddaf50a66fe3aa5f4ecfe08759a6
2 changes: 1 addition & 1 deletion lib/assert.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -313,7 +313,7 @@
throw new ERR_MISSING_ARGS('actual', 'expected');
}
if (isDeepEqual === undefined) lazyLoadComparison();
if (!isDeepStrictEqual(actual, expected, { __proto__: null, skipPrototypeComparison: this?.[kOptions]?.skipPrototypeComparison })) {

Check failure on line 316 in lib/assert.js

View workflow job for this annotation

GitHub Actions / lint-js-and-md 

This line has a length of 134. Maximum allowed is 120
innerFail({
actual,
expected,
Expand All @@ -339,7 +339,7 @@
throw new ERR_MISSING_ARGS('actual', 'expected');
}
if (isDeepEqual === undefined) lazyLoadComparison();
if (isDeepStrictEqual(actual, expected, { skipPrototypeComparison: this?.[kOptions]?.skipPrototypeComparison })) {
if (isDeepStrictEqual(actual, expected, { __proto__: null, skipPrototypeComparison: this?.[kOptions]?.skipPrototypeComparison })) {

Check failure on line 342 in lib/assert.js

View workflow job for this annotation

GitHub Actions / lint-js-and-md 

This line has a length of 133. Maximum allowed is 120
innerFail({
actual,
expected,
Expand Down
Loading