mangle chunk boundaries in compact mode

src/Chunk.ts

@@ -375,7 +375,7 @@ export default class Chunk {
 
 	generateInternalExports(options: OutputOptions) {
 		if (this.isEntryModuleFacade) return;
-		const mangle = options.format === 'system' || options.format === 'es';
+		const mangle = options.format === 'system' || options.format === 'es' || options.compact;
 		let i = 0,
 			safeExportName: string;
 		this.exportNames = Object.create(null);

src/finalisers/cjs.ts

@@ -87,7 +87,7 @@ export default function cjs(
 		exportMode,
 		options.interop,
 		options.compact,
-		`module.exports${_}=`
+		`module.exports${_}=${_}`
 	);
 
 	magicString.prepend(intro);

src/finalisers/shared/getExportBlock.ts

@@ -4,10 +4,12 @@ export default function getExportBlock(
 	exports: ChunkExports,
 	dependencies: ChunkDependencies,
 	exportMode: string,
-	compact: boolean,
 	interop: boolean,
-	mechanism = 'return'
+	compact: boolean,
+	mechanism = 'return '
 ) {
+	const _ = compact ? '' : ' ';
+
 	if (exportMode === 'default') {
 		let local;
 		exports.some(expt => {
@@ -30,11 +32,9 @@ export default function getExportBlock(
 				});
 			});
 		}
-		return `${mechanism} ${local};`;
+		return `${mechanism}${local};`;
 	}
 
-	const _ = compact ? '' : ' ';
-
 	let exportBlock = '';
 
 	// star exports must always output first for precedence

test/chunking-form/samples/chunking-compact/_config.js

@@ -0,0 +1,9 @@
+module.exports = {
+	description: 'chunking compact and mangled output',
+	options: {
+		input: ['main1.js', 'main2.js'],
+		output: {
+			compact: true
+		}
+	}
+};

test/chunking-form/samples/chunking-compact/_expected/amd/chunk-1e9855f7.js

@@ -0,0 +1,6 @@
+define(['exports'],function(exports){'use strict';function fn () {
+  console.log('lib2 fn');
+}function fn$1 () {
+  fn();
+  console.log('dep2 fn');
+}exports.a=fn$1;});

test/chunking-form/samples/chunking-compact/_expected/amd/main1.js

@@ -0,0 +1,8 @@
+define(['./chunk-1e9855f7.js'],function(__chunk_1){'use strict';function fn () {
+  console.log('dep1 fn');
+}class Main1 {
+  constructor () {
+    fn();
+    __chunk_1.a();
+  }
+}return Main1;});

test/chunking-form/samples/chunking-compact/_expected/amd/main2.js

@@ -0,0 +1,12 @@
+define(['external','./chunk-1e9855f7.js'],function(external,__chunk_1){'use strict';function fn () {
+  console.log('lib1 fn');
+  external.fn();
+}function fn$1 () {
+  fn();
+  console.log('dep3 fn');
+}class Main2 {
+  constructor () {
+    fn$1();
+    __chunk_1.a();
+  }
+}return Main2;});

test/chunking-form/samples/chunking-compact/_expected/cjs/chunk-992f09ca.js

@@ -0,0 +1,6 @@
+'use strict';function fn () {
+  console.log('lib2 fn');
+}function fn$1 () {
+  fn();
+  console.log('dep2 fn');
+}exports.a=fn$1;

test/chunking-form/samples/chunking-compact/_expected/cjs/main1.js

@@ -0,0 +1,8 @@
+'use strict';var __chunk_1=require('./chunk-992f09ca.js');function fn () {
+  console.log('dep1 fn');
+}class Main1 {
+  constructor () {
+    fn();
+    __chunk_1.a();
+  }
+}module.exports=Main1;

test/chunking-form/samples/chunking-compact/_expected/cjs/main2.js

@@ -0,0 +1,12 @@
+'use strict';var external=require('external'),__chunk_1=require('./chunk-992f09ca.js');function fn () {
+  console.log('lib1 fn');
+  external.fn();
+}function fn$1 () {
+  fn();
+  console.log('dep3 fn');
+}class Main2 {
+  constructor () {
+    fn$1();
+    __chunk_1.a();
+  }
+}module.exports=Main2;

test/chunking-form/samples/chunking-compact/_expected/es/chunk-5b60a9d9.js

@@ -0,0 +1,6 @@
+function fn () {
+  console.log('lib2 fn');
+}function fn$1 () {
+  fn();
+  console.log('dep2 fn');
+}export{fn$1 as a};

test/chunking-form/samples/chunking-compact/_expected/es/main1.js

@@ -0,0 +1,8 @@
+import {a as fn}from'./chunk-5b60a9d9.js';function fn$1 () {
+  console.log('dep1 fn');
+}class Main1 {
+  constructor () {
+    fn$1();
+    fn();
+  }
+}export default Main1;

test/chunking-form/samples/chunking-compact/_expected/es/main2.js

@@ -0,0 +1,12 @@
+import {fn}from'external';import {a as fn$1}from'./chunk-5b60a9d9.js';function fn$2 () {
+  console.log('lib1 fn');
+  fn();
+}function fn$3 () {
+  fn$2();
+  console.log('dep3 fn');
+}class Main2 {
+  constructor () {
+    fn$3();
+    fn$1();
+  }
+}export default Main2;

test/chunking-form/samples/chunking-compact/_expected/system/chunk-ad0e6b97.js

@@ -0,0 +1,6 @@
+System.register([],function(exports,module){'use strict';return{execute:function(){exports('a',fn$1);function fn () {
+  console.log('lib2 fn');
+}function fn$1 () {
+  fn();
+  console.log('dep2 fn');
+}}};});

test/chunking-form/samples/chunking-compact/_expected/system/main1.js

@@ -0,0 +1,8 @@
+System.register(['./chunk-ad0e6b97.js'],function(exports,module){'use strict';var fn;return{setters:[function(module){fn=module.a;}],execute:function(){function fn$1 () {
+  console.log('dep1 fn');
+}class Main1 {
+  constructor () {
+    fn$1();
+    fn();
+  }
+} exports('default', Main1);}};});

test/chunking-form/samples/chunking-compact/_expected/system/main2.js

@@ -0,0 +1,12 @@
+System.register(['external','./chunk-ad0e6b97.js'],function(exports,module){'use strict';var fn,fn$1;return{setters:[function(module){fn=module.fn;},function(module){fn$1=module.a;}],execute:function(){function fn$2 () {
+  console.log('lib1 fn');
+  fn();
+}function fn$3 () {
+  fn$2();
+  console.log('dep3 fn');
+}class Main2 {
+  constructor () {
+    fn$3();
+    fn$1();
+  }
+} exports('default', Main2);}};});

test/chunking-form/samples/chunking-compact/dep1.js

@@ -0,0 +1,3 @@
+export function fn () {
+  console.log('dep1 fn');
+}

test/chunking-form/samples/chunking-compact/dep2.js

@@ -0,0 +1,6 @@
+import { fn as libfn } from './lib2.js';
+
+export function fn () {
+  libfn();
+  console.log('dep2 fn');
+}

test/chunking-form/samples/chunking-compact/dep3.js

@@ -0,0 +1,6 @@
+import { fn as libfn } from './lib1.js';
+
+export function fn () {
+  libfn();
+  console.log('dep3 fn');
+}

test/chunking-form/samples/chunking-compact/lib1.js

@@ -0,0 +1,6 @@
+import { fn as fn$1 } from 'external';
+
+export function fn () {
+  console.log('lib1 fn');
+  fn$1();
+}

test/chunking-form/samples/chunking-compact/lib2.js

@@ -0,0 +1,3 @@
+export function fn () {
+  console.log('lib2 fn');
+}

test/chunking-form/samples/chunking-compact/main1.js

@@ -0,0 +1,9 @@
+import { fn } from './dep1.js';
+import { fn as fn2 } from './dep2.js';
+
+export default class Main1 {
+  constructor () {
+    fn();
+    fn2();
+  }
+}

test/chunking-form/samples/chunking-compact/main2.js

@@ -0,0 +1,9 @@
+import { fn } from './dep2.js';
+import { fn as fn2 } from './dep3.js';
+
+export default class Main2 {
+  constructor () {
+    fn2();
+    fn();
+  }
+}

test/form/samples/compact/_expected/cjs.js

@@ -1,3 +1,3 @@
 'use strict';function _interopDefault(e){return(e&&(typeof e==='object')&&'default'in e)?e['default']:e}var x=_interopDefault(require('external'));function foo () {
 	console.log( x );
-}module.exports= foo;
+}module.exports=foo;