Skip to content

Convert standalone entry to CJS#9822

Merged
tobias-jarvelov merged 2 commits intomainfrom
fix-standalone-test-executable
Feb 12, 2026
Merged

Convert standalone entry to CJS#9822
tobias-jarvelov merged 2 commits intomainfrom
fix-standalone-test-executable

Conversation

@tobias-jarvelov
Copy link
Contributor

@tobias-jarvelov tobias-jarvelov commented Feb 11, 2026

When using node's single executable application (SEA) feature a known limitation is that only commonjs (CJS) is supported, extract from Node.js docs[1]:

The single executable application feature currently only supports
running a single embedded script using the CommonJS module system.

[1]
https://nodejs.org/docs/latest-v25.x/api/single-executable-applications.html#single-executable-applications


This change is Reviewable

@tobias-jarvelov tobias-jarvelov force-pushed the fix-standalone-test-executable branch 3 times, most recently from 3a62ba8 to c9f8f85 Compare February 12, 2026 09:15
Copy link
Collaborator

@olmoh olmoh left a comment

Choose a reason for hiding this comment

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

LGTM

This file is not used for the development or production builds and
should be included in tsconfig.standalone.json instead.
When using node's single executable application (SEA) feature a known
limitation is that only commonjs (CJS) is supported, extract from
Node.js docs[1]:

> The single executable application feature currently only supports
> running a single embedded script using the CommonJS module system.

[1]
https://nodejs.org/docs/latest-v25.x/api/single-executable-applications.html#single-executable-applications
@tobias-jarvelov tobias-jarvelov force-pushed the fix-standalone-test-executable branch from c9f8f85 to 329b153 Compare February 12, 2026 16:02
@tobias-jarvelov tobias-jarvelov merged commit 9bfe59f into main Feb 12, 2026
11 of 13 checks passed
@tobias-jarvelov tobias-jarvelov deleted the fix-standalone-test-executable branch February 12, 2026 16:04
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