Skip to content

Refactor/Modernize code - #531

Merged
RyanZim merged 4 commits into
masterfrom
ryan/refactor
Aug 27, 2026
Merged

Refactor/Modernize code#531
RyanZim merged 4 commits into
masterfrom
ryan/refactor

Conversation

@RyanZim

@RyanZim RyanZim commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator
  • Use top-level await
  • Use async functions where there's distinct advantages
  • Use node: prefix consistently
  • Cleanup some unneeded code

@RyanZim
RyanZim requested a balanced review from Copilot August 27, 2026 15:55
@RyanZim RyanZim added this to the v12 milestone Aug 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Modernizes the CLI’s asynchronous flow and standardizes Node.js built-in imports.

Changes:

  • Replaces Promise chains with top-level await and async functions.
  • Standardizes built-in imports using node: prefixes.
  • Removes redundant input normalization and error propagation code.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
index.js Modernizes CLI execution and processing flow.
lib/DependencyGraph.js Updates path import.
lib/DependencyGraph.test.js Updates path import.
lib/getMapfile.js Updates path import.
test/base.js Updates path import.
test/config.js Updates path import.
test/dir.js Updates path import.
test/ext.js Updates path import.
test/glob.js Updates path import.
test/helpers/cli.js Updates Node built-in imports.
test/helpers/env.js Updates path import.
test/helpers/tmp.js Updates path import.
test/replace.js Updates path import.
test/stdin.js Updates Node built-in imports.
test/stdout.js Updates Node built-in imports.
test/watch.js Updates Node built-in imports.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread index.js
@RyanZim
RyanZim merged commit 9656979 into master Aug 27, 2026
7 checks passed
@RyanZim
RyanZim deleted the ryan/refactor branch August 27, 2026 15:59
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.

2 participants