feat(`util.is()`): introduce by AugustinMauroy · Pull Request #119 · nodejs/userland-migrations

AugustinMauroy · 2025-07-24T15:02:57Z

Description

Close #116

recipes/util-is/README.md

avivkeller · 2025-07-24T15:47:43Z

@AugustinMauroy Can you check the original implementations and make sure these match them?

If they do, can you make that clear?

AugustinMauroy · 2025-07-24T20:15:37Z

@avivkeller & @ljharb I just applied what the docs said. So I will not change it for now.

Here docs ref

recipes/util-is/src/workflow.ts

AugustinMauroy · 2025-07-25T08:14:42Z

I'm sorry @ljharb but If you think there are better way to handle that please first update node api doc and then we will update this codemod

ljharb · 2025-07-25T15:32:36Z

For the opinion ones, fine, but one of them is a bug and is just broken. It shouldn't matter what's in the docs, but it matters what code does.

AugustinMauroy · 2025-07-25T20:27:59Z

For the opinion ones, fine, but one of them is a bug and is just broken. It shouldn't matter what's in the docs, but it matters what code does.

I'm annoyed because I'm caught between two stools. On the one hand, I agree with you. On the other, I want to follow the documentation strictly so that the org node is consistent.

ljharb · 2025-07-26T06:48:25Z

That seems like an arbitrary and counterproductive constraint to me. The docs aren’t the source of truth, they need to reflect the actual source of truth, the code.

AugustinMauroy · 2025-07-29T20:07:59Z

cc @ljharb synced with the pr

ljharb · 2025-07-29T20:10:28Z

LGTM, thanks!

recipes/util-is/src/workflow.ts

utils/src/ast-grep/import-statement.ts

recipes/util-is/workflow.yml

recipes/util-is/codemod.yml

Co-authored-by: Bruno Rodrigues <swe@brunocroh.com>

recipes/util-is/package.json

JakobJingleheimer

Looking good!

Aside from a few small things / nets, I think it's just missing support for dynamic imports?

recipes/util-is/README.md

recipes/util-is/codemod.yml

recipes/util-is/src/workflow.ts

recipes/util-is/tests/expected/file-1.js

recipes/util-is/workflow.yml

AugustinMauroy · 2025-08-20T09:42:29Z

I think it's just missing support for dynamic imports?

Any of our codemod support that so if we want to support that do it in aside pr that create utility + update codemod

JakobJingleheimer · 2025-08-21T07:32:04Z

Any of our codemod support that so if we want to support that do it in aside pr that create utility + update codemod

I think all of our migrations must support that in order to claim the deprecation is handled, which I believe is the intention. If it's best to create a shared util to do it, let's do that right away.

I thought you had previously said all of our current ones do handle this (and your message here also says they do, but I'm guessing from the rest that they actually don't?).

AugustinMauroy · 2025-08-21T09:04:26Z

think all of our migrations must support that in order to claim the deprecation is handled, which I believe is the intention. If it's best to create a shared util to do it, let's do that right away.

I just realized that we have utility for that but I was never used

I thought you had previously said all of our current ones do handle this (and your message here also says they do, but I'm guessing from the rest that they actually don't?).

It's a misunderstanding there are any of our codemod that support that

Co-Authored-By: Jacob Smith <3012099+JakobJingleheimer@users.noreply.github.com>

recipes/util-is/README.md

JakobJingleheimer

Did dynamic import support get added? I think that was my only previous request.

JakobJingleheimer · 2025-09-16T08:22:03Z

recipes/util-is/src/workflow.ts

 * 6. util.isFunction() → typeof value === 'function'
 * 7. util.isNull() → value === null
- * 8. util.isNullOrUndefined() → value == null
+ * 8. util.isNullOrUndefined() → value === null || value === undefined


very nit since it's a comment:

Suggested change

* 8. util.isNullOrUndefined() → value === null || value === undefined

* 8. util.isNullOrUndefined() → value == null

https://github.com/nodejs/node/blob/main/doc/api/deprecations.md#dep0051-utilisnullorundefined

Yes, I understand :)

null == null // true undefined == null // true false == null // false 0 == null // false '' == null // false

The difference is in document.all, I believe, which doesn't apply to node but would if someone cargoculted the info into a browser.

so what should I do ?

I don't understand @ljharb's comment, so I'm not sure.

But this thread was based on a nit—it's not blocking. The blocking issue is supporting dynamic import.

document.all == null but document.all !== null && typeof document.all !== 'undefined'.

Ah. I think we can ignore that extreme edge-case.

JakobJingleheimer · 2025-09-16T08:22:23Z

recipes/util-is/tests/expected/file-1.js

 	console.log('someValue is null');
 }
-if (someValue == null) {
+if (someValue === null || someValue === undefined) {


nit:

Suggested change

if (someValue === null || someValue === undefined) {

if (someValue == null) {

fix

…d-migrations into feat(`util.is()`)

AugustinMauroy · 2025-09-27T22:15:50Z

Dynamic import work well !
I'm not fan of differing from doc so let's not modify it

x87 · 2025-11-21T16:14:12Z

recipes/util-is/README.md

+| `util.isError(value)`             | `Error.isError(value)`                     |
+| `util.isFunction(value)`          | `typeof value === 'function'`              |
+| `util.isNull(value)`              | `value === null`                           |
+| `util.isNullOrUndefined(value)`   | `value === null || value === undefined`    |


need to escape || as it breaks the markdown

opened a pr for that !

AugustinMauroy requested a review from a team July 24, 2025 15:03

AugustinMauroy changed the title ~~Feat(util.is())~~ feat(util.is()): introduce Jul 24, 2025

ljharb reviewed Jul 24, 2025

View reviewed changes

recipes/util-is/README.md Outdated Show resolved Hide resolved

recipes/util-is/README.md Outdated Show resolved Hide resolved

recipes/util-is/README.md Outdated Show resolved Hide resolved

recipes/util-is/README.md Outdated Show resolved Hide resolved

avivkeller approved these changes Jul 24, 2025

View reviewed changes

ljharb reviewed Jul 24, 2025

View reviewed changes

recipes/util-is/src/workflow.ts Outdated Show resolved Hide resolved

recipes/util-is/src/workflow.ts Outdated Show resolved Hide resolved

recipes/util-is/src/workflow.ts Outdated Show resolved Hide resolved

AugustinMauroy mentioned this pull request Jul 29, 2025

doc: update deprecation guide for util.is**() nodejs/node#59269

Merged

AugustinMauroy force-pushed the feat(`util.is()`) branch 2 times, most recently from 76b6fb5 to 1f71b3e Compare July 29, 2025 20:07

AugustinMauroy requested a review from a team July 29, 2025 21:15

brunocroh reviewed Jul 29, 2025

View reviewed changes

recipes/util-is/src/workflow.ts Outdated Show resolved Hide resolved

utils/src/ast-grep/import-statement.ts Outdated Show resolved Hide resolved

brunocroh reviewed Jul 30, 2025

View reviewed changes

recipes/util-is/workflow.yml Outdated Show resolved Hide resolved

recipes/util-is/codemod.yml Outdated Show resolved Hide resolved

AugustinMauroy mentioned this pull request Jul 30, 2025

feat: handle crypto.createCredentials() depreciation #123

Closed

brunocroh approved these changes Aug 8, 2025

View reviewed changes

AugustinMauroy and others added 6 commits August 12, 2025 10:49

scaffold codemod

21c272e

feat(util.is**()): introduce

22c84fe

WIP

4a5d5f0

improve

4ca8c17

apply code suggestion

5f7369a

Apply suggestions from code review

b52adfd

Co-authored-by: Bruno Rodrigues <swe@brunocroh.com>

AugustinMauroy force-pushed the feat(`util.is()`) branch from 7d65dc8 to b52adfd Compare August 12, 2025 08:49

feat(util.is()): improve

afbc8fd

avivkeller approved these changes Aug 12, 2025

View reviewed changes

brunocroh approved these changes Aug 19, 2025

View reviewed changes

recipes/util-is/package.json Outdated Show resolved Hide resolved

JakobJingleheimer reviewed Aug 20, 2025

View reviewed changes

AugustinMauroy and others added 3 commits August 21, 2025 11:12

Merge branch 'main' into feat(util.is())

7169e83

update

e9df057

update from feedback

f679a55

Co-Authored-By: Jacob Smith <3012099+JakobJingleheimer@users.noreply.github.com>

JakobJingleheimer reviewed Aug 31, 2025

View reviewed changes

recipes/util-is/README.md Outdated Show resolved Hide resolved

recipes/util-is/README.md Outdated Show resolved Hide resolved

recipes/util-is/README.md Outdated Show resolved Hide resolved

update

09a7d8d

brunocroh approved these changes Aug 31, 2025

View reviewed changes

JakobJingleheimer reviewed Sep 16, 2025

View reviewed changes

JakobJingleheimer added blocked:upstream Depends on another PR or external change/fix missing functionality Required use-case missing and removed blocked:upstream Depends on another PR or external change/fix labels Sep 16, 2025

AugustinMauroy added 5 commits September 27, 2025 23:42

Merge branch 'main' into feat(util.is())

82cc1f4

fix

cb95073

fix

3e07153

fix

Merge branch 'feat(util.is())' of https://github.com/nodejs/userlan…

1663c77

…d-migrations into feat(`util.is()`)

support dynamic import

90ab2ca

Update package-lock.json

f1a091f

JakobJingleheimer approved these changes Oct 15, 2025

View reviewed changes

brunocroh approved these changes Oct 15, 2025

View reviewed changes

brunocroh merged commit 0372fc6 into main Oct 15, 2025
30 checks passed

JakobJingleheimer added dep:v23 Migration handles deprecation introduced in node v23 and removed missing functionality Required use-case missing labels Oct 16, 2025

x87 reviewed Nov 21, 2025

View reviewed changes

JakobJingleheimer deleted the feat(`util.is()`) branch December 13, 2025 11:51

	* 8. util.isNullOrUndefined() → value === null \|\| value === undefined
	* 8. util.isNullOrUndefined() → value == null

	if (someValue === null \|\| someValue === undefined) {
	if (someValue == null) {

Uh oh!

Conversation

AugustinMauroy commented Jul 24, 2025

Description

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

avivkeller commented Jul 24, 2025

Uh oh!

AugustinMauroy commented Jul 24, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

AugustinMauroy commented Jul 25, 2025

Uh oh!

ljharb commented Jul 25, 2025

Uh oh!

AugustinMauroy commented Jul 25, 2025

Uh oh!

ljharb commented Jul 26, 2025

Uh oh!

AugustinMauroy commented Jul 29, 2025

Uh oh!

ljharb commented Jul 29, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

JakobJingleheimer left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

AugustinMauroy commented Aug 20, 2025

Uh oh!

JakobJingleheimer commented Aug 21, 2025

Uh oh!

AugustinMauroy commented Aug 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

JakobJingleheimer left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

AugustinMauroy commented Sep 27, 2025

Uh oh!

Uh oh!

AugustinMauroy commented Aug 21, 2025 •

edited

Loading

x87 Nov 21, 2025 •

edited

Loading