feat(process-assert-to-assert): introduce

[matheusmorett2]

Closes #197

Add codemod recipe process-assert-to-assert to handle deprecation DEP0100.

Before

const { assert: nodeAssert, env } = require("process");
nodeAssert(condition, "Assertion valid");

After

const assert = require("node:assert");
const { env } = require("process");
assert(condition, "Assertion valid");

---

Fix: remove-binding.ts

handle scenario where have alias in require

[AugustinMauroy]

I didn't see any test for that kind cases

import process from "node:process"

But good first pr

[brunocroh]

Awesome first contribution 🎉, Thank you @matheusmorett2 !

Just small fixes needed

[brunocroh]

Please fix the conflict in the remove-binding file

[AugustinMauroy]

For require call not in cjs context the transformed code may be wrong.

For example this before

import { createRequire} from 'node:module';
const require = createRequire(import.meta.dirname)

process.assert(true);

Broken after with the current implementation

const assert = require('node:assert');
import { createRequire} from 'node:module';
const require = createRequire(import.meta.dirname)

assert(true);

In this case I expect that require is undefined.

Solutions:

1. Try to found a logic that handle that.
2. Add it at inline comment so we know for the future

[AugustinMauroy]

hey @matheusmorett2 any news on this pr ?

[avivkeller]

I think this could use some refactoring. There's a lot of nesting, among other things.

Can you break it up, and look at the other mods for inspiration?

[AugustinMauroy]

bump @matheusmorett2