Skip to content

Node Protocol plugin needs to handle Node Protocol only modules #482

@jadejr

Description

@jadejr

Reproduction link or steps

The builtin modules (via import { builtinModules } from 'node:module) on node versions like 24.x show some modules that REQUIRE the node prefix. This cannot be avoided.

[
 // all others snipped
  'worker_threads',     
  'zlib',
  'node:sea',
  'node:sqlite',
   'node:test', 
   'node:test/reporters'
]

The node protocol plugin should not strip it if it exist for modules like node:sqlite, nor should it add it if already exists.

I had the option enabled, so i ended up with node:node:sqlite.

What is expected?

I would expect it to not add or strip the node protocol for modules that require the node protocol

What is actually happening?

It adds the node protocol to builtin modules that require the node protocol

Any additional comments?

No response

Metadata

Metadata

Assignees

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions