feat(DEP0150): introduce process-config-mutations-to-separate-config

[Green00101]

Summary

Adds a DEP0150 migration recipe to handle process.config mutations after it became immutable in Node.js v19.
The recipe handles:

• Direct assignments to process.config properties
• Nested property assignments
• Bracket notation assignments
• delete process.config... expressions
• Bare Object.assign(process.config, ...) calls
• Captured Object.assign(process.config, ...) calls by rewriting them to copy into a new object
• Read-only process.config usage, which is preserved unchanged

Closes #408.

Tests

• Ran npm test in recipes/process-config-mutations-to-separate-config
• Added tests:
  • bracket-notation-assignment
  • bracket-notation-delete
  • conditional-modification
  • direct-assignment
  • nested-property
  • nested-write-unchanged
  • new-property
  • object-assign-bare
  • object-assign-captured
  • read-only

This is my first contribution to this project, so feedback is very welcome. I’m happy to make any changes if needed.

[AugustinMauroy]

1. Missing readme at codemod level (take example on other or read CONTRIBUTING.md)
2. run npm I at root level to resolve the new npm workspace

[Green00101]

Hi, thanks for reviewing my PR.

I have updated the code based on your comments.
Please let me know if anything else needs to be changed.

[AugustinMauroy]

that nice. could you also add jsdoc on helper function ?

[brunocroh]

lgtm

[Green00101]

Thanks for taking the time to review my PR.

I have updated the code according to your feedback.

I also noticed that I had several formatting issues. When I tried to run node --run pre-commit, I got the following error: Missing script: "lint:fix;", even though I can see "lint:fix": "biome lint --fix ./" in package.json.

Because of that, I have been running the checks separately instead:

node --run lint:fix
node --run type-check
node --run test

Could this be why some formatting issues were missed? If so, what would be the recommended way to fix it?

[Green00101]

Thanks!

I’ve updated the formatting and pushed the changes.

[AugustinMauroy]

LGMT !

[AugustinMauroy]

Thanks for taking the time to review my PR.

I have updated the code according to your feedback.

I also noticed that I had several formatting issues. When I tried to run node --run pre-commit, I got the following error: Missing script: "lint:fix;", even though I can see "lint:fix": "biome lint --fix ./" in package.json.

Because of that, I have been running the checks separately instead:

node --run lint:fix node --run type-check node --run test

Could this be why some formatting issues were missed? If so, what would be the recommended way to fix it?

Missing script: "lint:fix;" it's not supposed to do that.

also on this repo we only lint mean analyse the code but not format it because automatic formater sometime make strange thing

[Green00101]

Got it, thanks for clarifying.

When I run node --run pre-commit, it shows:

PS C:\userland-migrations> node --run pre-commit
Missing script: "lint:fix;" for C:\userland-migrations\package.json

Available scripts are:
  lint:fix: biome lint --fix ./
  lint: biome lint ./
  pre-commit: node --run lint:fix; node --run type-check; node --run test
  test: npm run test --workspaces
  type-check: tsgo --noEmit

Maybe this is because I’m using Windows.

I’ll fix the formatting issues manually where needed.

[AugustinMauroy]

Got it, thanks for clarifying.

When I run node --run pre-commit, it shows:

PS C:\userland-migrations> node --run pre-commit
Missing script: "lint:fix;" for C:\userland-migrations\package.json

Available scripts are:
  lint:fix: biome lint --fix ./
  lint: biome lint ./
  pre-commit: node --run lint:fix; node --run type-check; node --run test
  test: npm run test --workspaces
  type-check: tsgo --noEmit

Maybe this is because I’m using Windows.

I’ll fix the formatting issues manually where needed.

RIght I got it. command; otherCommand syntax doesn't work for windows you can open an second pr for that