Include extensions in preserveModules output filenames for scriptifie…

…d assets

src/Chunk.ts

@@ -30,7 +30,7 @@ import { error } from './utils/error';
 import { sortByExecutionOrder } from './utils/executionOrder';
 import getIndentString from './utils/getIndentString';
 import { makeLegal } from './utils/identifierHelpers';
-import { basename, dirname, isAbsolute, normalize, resolve } from './utils/path';
+import { basename, dirname, extname, isAbsolute, normalize, resolve } from './utils/path';
 import relativeId, { getAliasName } from './utils/relativeId';
 import renderChunk from './utils/renderChunk';
 import { RenderOptions } from './utils/renderHelpers';
@@ -83,6 +83,8 @@ interface FacadeName {
 	name?: string;
 }
 
+const COMMON_JS_EXTENSIONS = ['.js', '.jsx', '.ts', '.tsx'];
+
 function getGlobalName(
 	module: ExternalModule,
 	globals: GlobalsOption,
@@ -280,14 +282,20 @@ export default class Chunk {
 		options: OutputOptions,
 		existingNames: Record<string, any>
 	): string {
-		const sanitizedId = sanitizeFileName(this.orderedModules[0].id);
+		const id = this.orderedModules[0].id;
+		const sanitizedId = sanitizeFileName(id);
 
 		let path: string;
-		if (isAbsolute(this.orderedModules[0].id)) {
+		if (isAbsolute(id)) {
+			const extension = extname(id);
+			console.log({ COMMON_JS_EXTENSIONS, extension });
 			const name = renderNamePattern(
-				options.entryFileNames || '[name].js',
+				options.entryFileNames ||
+					(COMMON_JS_EXTENSIONS.includes(extension) ? '[name].js' : '[name].[ext].js'),
 				'output.entryFileNames',
 				{
+					ext: () => extension.substr(1),
+					extname: () => extension,
 					format: () => (options.format === 'es' ? 'esm' : (options.format as string)),
 					name: () => this.getChunkName()
 				}

test/chunking-form/samples/preserve-modules-scriptified-assets/_config.js

@@ -0,0 +1,21 @@
+module.exports = {
+	description: 'scriptified assets have extension in preserveModules output filename',
+	options: {
+		input: 'src/main.js',
+		preserveModules: true,
+		plugins: [
+			{
+				name: 'str-num-plugin',
+				transform(code, id) {
+					if (id.endsWith('.num')) {
+						return { code: `export default ${code.trim()}` };
+					}
+					if (id.endsWith('.str')) {
+						return { code: `export default "${code.trim()}"` };
+					}
+					return null;
+				}
+			}
+		]
+	}
+};

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/amd/answer.num.js

@@ -0,0 +1,7 @@
+define(['exports'], function (exports) { 'use strict';
+
+	var answer = 42;
+
+	exports.default = answer;
+
+});

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/amd/lorem.str.js

@@ -0,0 +1,7 @@
+define(['exports'], function (exports) { 'use strict';
+
+	var lorem = "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.";
+
+	exports.default = lorem;
+
+});

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/amd/main.js

@@ -0,0 +1,10 @@
+define(['exports', './answer.num', './lorem.str'], function (exports, answer, lorem) { 'use strict';
+
+
+
+	exports.answer = answer.default;
+	exports.lorem = lorem.default;
+
+	Object.defineProperty(exports, '__esModule', { value: true });
+
+});

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/cjs/answer.num.js

@@ -0,0 +1,5 @@
+'use strict';
+
+var answer = 42;
+
+exports.default = answer;

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/cjs/lorem.str.js

@@ -0,0 +1,5 @@
+'use strict';
+
+var lorem = "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.";
+
+exports.default = lorem;

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/cjs/main.js

@@ -0,0 +1,11 @@
+'use strict';
+
+Object.defineProperty(exports, '__esModule', { value: true });
+
+var answer = require('./answer.num.js');
+var lorem = require('./lorem.str.js');
+
+
+
+exports.answer = answer.default;
+exports.lorem = lorem.default;

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/es/answer.num.js

@@ -0,0 +1,3 @@
+var answer = 42;
+
+export default answer;

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/es/main.js

@@ -0,0 +1,2 @@
+export { default as answer } from './answer.num.js';
+export { default as lorem } from './lorem.str.js';

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/system/answer.num.js

@@ -0,0 +1,10 @@
+System.register([], function (exports) {
+	'use strict';
+	return {
+		execute: function () {
+
+			var answer = exports('default', 42);
+
+		}
+	};
+});

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/system/lorem.str.js

@@ -0,0 +1,10 @@
+System.register([], function (exports) {
+	'use strict';
+	return {
+		execute: function () {
+
+			var lorem = exports('default', "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.");
+
+		}
+	};
+});

test/chunking-form/samples/preserve-modules-scriptified-assets/_expected/system/main.js

@@ -0,0 +1,15 @@
+System.register(['./answer.num.js', './lorem.str.js'], function (exports) {
+	'use strict';
+	return {
+		setters: [function (module) {
+			exports('answer', module.default);
+		}, function (module) {
+			exports('lorem', module.default);
+		}],
+		execute: function () {
+
+
+
+		}
+	};
+});

test/chunking-form/samples/preserve-modules-scriptified-assets/src/answer.num

@@ -0,0 +1 @@
+42

test/chunking-form/samples/preserve-modules-scriptified-assets/src/lorem.str

@@ -0,0 +1 @@
+Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

test/chunking-form/samples/preserve-modules-scriptified-assets/src/main.js

@@ -0,0 +1,2 @@
+export { default as answer } from './answer.num';
+export { default as lorem } from './lorem.str';