Skip to content

feat: add prefer-process-get-builtin-module rule - #554

Merged
aladdin-add merged 1 commit into
eslint-community:masterfrom
ColumbusLabs:codex/prefer-process-get-builtin-module
Aug 8, 2026
Merged

feat: add prefer-process-get-builtin-module rule#554
aladdin-add merged 1 commit into
eslint-community:masterfrom
ColumbusLabs:codex/prefer-process-get-builtin-module

Conversation

@ColumbusLabs

Copy link
Copy Markdown

Closes #475.

Summary

  • add prefer-process-get-builtin-module for global require(), createRequire bindings, and directly awaited built-in imports
  • gate reports to Node.js ^20.16.0 or >=22.3.0
  • ignore arbitrary/shadowed/reassigned require and process bindings
  • keep the rule report-only because module namespace and require cache semantics can differ
  • add tests, documentation, registry entry, and generated README metadata

Validation

  • npm test (3,094 passing, 3 pending)
  • ESLint, TypeScript, Markdown, and generated-doc checks
  • plugin registry readback
  • git diff --check

Comment thread docs/rules/prefer-process-get-builtin-module.md
@aladdin-add

Copy link
Copy Markdown

Could you please resolve the conflict? Thanks!

@ColumbusLabs
ColumbusLabs force-pushed the codex/prefer-process-get-builtin-module branch from e7bcfe2 to 4531076 Compare August 8, 2026 02:21
@aladdin-add
aladdin-add merged commit 8d9bd05 into eslint-community:master Aug 8, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New Rule: prefer-process-get-builtin-module

2 participants