Skip to content

Feat: add getModuleDependencies utillity #326

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

Jump to bottom
Open
brunocroh wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Feat: add getModuleDependencies utillity #326

brunocroh wants to merge 16 commits into from

Conversation

@brunocroh
Copy link
Member

@brunocroh brunocroh commented Jan 3, 2026 •
edited
Loading

Add a getModuleDependencies utility to replace cases like the one below:

  return [
		...getNodeRequireCalls(node, name),
		...getNodeImportStatements(node, name),
		...getNodeImportCalls(node, name),
	];

Also update the recipes to use it.

brunocroh and others added 14 commits January 3, 2026 14:12
@brunocroh @JakobJingleheimer
add getModuleDependencies utility function
f99c856 
Co-authored-by: Jacob Smith <3012099+JakobJingleheimer@users.noreply.github.com>
@brunocroh
improve error log from resolve-binding-path
21403d3
@brunocroh
refactor(chalk-to-util-styletext): add getModuleDependencies
23c3e99
@brunocroh
refactor(crypto-fips-to-getFips): add getModuleDependencies
186b1a2
@brunocroh
refactor(crypto-rsa-pss-update): add getModuleDependencies
cc420d2
@brunocroh
refactor(dirent-path-to-parent-path): add getModuleDependencies
5cf382c
@brunocroh
refactor(fs-truncate-fd-deprecation): add getModuleDependencies
398fcf4
@brunocroh
refactor(process-assert-to-node-assert): add getModuleDependencies
054cdc1
@brunocroh
refactor(repl-classes-with-new): add getModuleDependencies
220b581
@brunocroh
refactor(slow-buffer-to-buffer-lloc-unsafe-slow): add getModuleDepend…
fe7935a 
…encies
@brunocroh
refactor(util-extend-to-object-assign): add getModuleDependencies
729064c
@brunocroh
refactor(util-is): add getModuleDependencies
74bad05
@brunocroh
refactor(util-print-to-console.log): add getModuleDependencies
14b707a
@brunocroh
refactor(zlib-bytesread-to-byteswritten): add getModuleDependencies
278e108
@brunocroh brunocroh mentioned this pull request Jan 3, 2026
Open
@brunocroh brunocroh changed the title Feat/add get dep nodes Feat: add getModuleDependencies utillity Jan 3, 2026
@AugustinMauroy
Copy link
Member

AugustinMauroy commented Jan 3, 2026

Noice !

could you document it in the utils readme ?

@brunocroh @AugustinMauroy
docs(module-dependencies): add documentation about getModuleDependenc…
12dbe9e 
…ies fn

Co-authored-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>
@brunocroh
Copy link
Member Author

brunocroh commented Jan 3, 2026

Noice !

could you document it in the utils readme ?

good catch, done. 🚀

@brunocroh brunocroh requested a review from a team January 9, 2026 09:40
@brunocroh brunocroh added the awaiting reviewer Author has responded and needs action from the reviewer label Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AugustinMauroy AugustinMauroy AugustinMauroy approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

awaiting reviewer Author has responded and needs action from the reviewer

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@brunocroh @AugustinMauroy