Skip to content

#12058 - Refactor: Node.js built-in modules should be imported using the node: protocol - #12059

Open
AlexeyGirin with Copilot wants to merge 2 commits into
masterfrom
copilot/refactor-node-imports
Open

AlexeyGirin with Copilot wants to merge 2 commits into
masterfrom
copilot/refactor-node-imports

Conversation

Copilot AI commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

How the feature works? / How did you fix the issue?

Node.js built-ins used bare module names, which could be confused with npm packages. Updated the issue-reported imports and requires to use explicit node: specifiers.

  • ES modules: Prefixed built-in imports in the Vite/Rollup configs and file transformers.
  • CommonJS: Prefixed fs and path requires in the schema compilation script.
import path from 'node:path';
const fs = require('node:fs');

Check list

  • unit-tests written
  • e2e-tests written
  • documentation updated
  • PR name follows the pattern #1234 – issue name
  • branch name doesn't contain '#'
  • PR is linked with the issue
  • base branch (master or release/xx) is correct
  • task status changed to "Code review"
  • reviewers are notified about the pull request
Copilot AI changed the title [WIP] Refactor Node.js built-in modules to use node: protocol Oct 1, 2026
Copilot AI requested a review from AlexeyGirin October 1, 2026 14:40
@AlexeyGirin
AlexeyGirin marked this pull request as ready for review October 1, 2026 14:42
@AlexeyGirin AlexeyGirin changed the title Refactor Node.js built-in imports to use the node: protocol Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment