cli: revert default cjs build extension to .cjs.js
Signed-off-by: Patrik Oldsberg <poldsberg@gmail.com>
This commit is contained in:
@@ -49,6 +49,7 @@ const MODULE_EXTS = ['.mjs', '.mts'];
|
||||
const COMMONJS_EXTS = ['.cjs', '.cts'];
|
||||
const MOD_EXT = '.mjs';
|
||||
const CJS_EXT = '.cjs';
|
||||
const CJS_JS_EXT = '.cjs.js';
|
||||
|
||||
function isFileImport(source: string) {
|
||||
if (source.startsWith('.')) {
|
||||
@@ -169,7 +170,7 @@ export async function makeRollupConfigs(
|
||||
// file extensions. That way we are left with a combination of .cjs and .mjs
|
||||
// files where the module format in the file matches the file extension.
|
||||
if (options.outputs.has(Output.cjs)) {
|
||||
const defaultExt = targetPkg.type === 'module' ? MOD_EXT : CJS_EXT;
|
||||
const defaultExt = targetPkg.type === 'module' ? MOD_EXT : CJS_JS_EXT;
|
||||
const outputOpts: OutputOptions = {
|
||||
dir: distDir,
|
||||
entryFileNames(chunkInfo) {
|
||||
|
||||
@@ -167,7 +167,7 @@ describe('package build transforms', () => {
|
||||
outputs: new Set([Output.cjs]),
|
||||
workspacePackages: [],
|
||||
});
|
||||
const values = await import(resolvePath(pkgPath, 'dist/index.cjs')).then(
|
||||
const values = await import(resolvePath(pkgPath, 'dist/index.cjs.js')).then(
|
||||
m => m.values,
|
||||
);
|
||||
expect(values).toEqual({
|
||||
@@ -180,7 +180,7 @@ describe('package build transforms', () => {
|
||||
dyn: exportValues.all,
|
||||
});
|
||||
|
||||
expect(loadFixture('pkg-commonjs/dist/print.cjs')).toEqual({
|
||||
expect(loadFixture('pkg-commonjs/dist/print.cjs.js')).toEqual({
|
||||
depCommonJs: expectedExports.commonJs,
|
||||
depDefault: expectedExports.commonJs,
|
||||
dynCommonJs: expectedExports.commonJs,
|
||||
@@ -199,7 +199,7 @@ describe('package build transforms', () => {
|
||||
outputs: new Set([Output.cjs]),
|
||||
workspacePackages: [],
|
||||
});
|
||||
const values = await import(resolvePath(pkgPath, 'dist/index.cjs')).then(
|
||||
const values = await import(resolvePath(pkgPath, 'dist/index.cjs.js')).then(
|
||||
m => m.values,
|
||||
);
|
||||
expect(values).toEqual({
|
||||
@@ -212,7 +212,7 @@ describe('package build transforms', () => {
|
||||
dyn: exportValues.all,
|
||||
});
|
||||
|
||||
expect(loadFixture('pkg-default/dist/print.cjs')).toEqual({
|
||||
expect(loadFixture('pkg-default/dist/print.cjs.js')).toEqual({
|
||||
depCommonJs: expectedExports.commonJs,
|
||||
depDefault: expectedExports.commonJs,
|
||||
dynCommonJs: expectedExports.commonJs,
|
||||
|
||||
Reference in New Issue
Block a user