Merge pull request #694 from rollup/gh-690

update to latest magic-string

package.json

@@ -48,7 +48,7 @@
     "eslint": "^2.9.0",
     "estree-walker": "^0.2.0",
     "istanbul": "^0.4.0",
-    "magic-string": "^0.10.1",
+    "magic-string": "^0.15.1",
     "mocha": "^2.3.3",
     "remap-istanbul": "^0.5.1",
     "rollup": "^0.26.2",

src/Bundle.js

@@ -1,4 +1,4 @@
-import MagicString from 'magic-string';
+import { Bundle as MagicStringBundle } from 'magic-string';
 import first from './utils/first.js';
 import { blank, forOwn, keys } from './utils/object.js';
 import Module from './Module.js';
@@ -282,7 +282,7 @@ export default class Bundle {
 		// Determine export mode - 'default', 'named', 'none'
 		const exportMode = getExportMode( this, options.exports, options.moduleName );
 
-		let magicString = new MagicString.Bundle({ separator: '\n\n' });
+		let magicString = new MagicStringBundle({ separator: '\n\n' });
 		let usedModules = [];
 
 		this.orderedModules.forEach( module => {

src/Module.js

@@ -488,15 +488,16 @@ export default class Module {
 						const declaration = this.declarations[ name ];
 
 						if ( declaration.exportName && declaration.isReassigned ) {
-							magicString.insert( statement.end, `;\nexports.${name} = ${declaration.render( es6 )}` );
+							magicString.insertLeft( statement.end, `;\nexports.${name} = ${declaration.render( es6 )}` );
 						}
 					});
 				}
 
 				if ( statement.node.isSynthetic ) {
 					// insert `var/let/const` if necessary
-					magicString.insert( statement.start, `${statement.node.kind} ` );
-					magicString.overwrite( statement.end, statement.next, ';\n' ); // TODO account for trailing newlines
+					magicString.insertRight( statement.start, `${statement.node.kind} ` );
+					magicString.insertLeft( statement.end, ';' );
+					magicString.overwrite( statement.end, statement.next, '\n' ); // TODO account for trailing newlines
 				}
 			}
 
@@ -527,7 +528,7 @@ export default class Module {
 					}
 
 					if ( reference.isShorthandProperty ) {
-						magicString.insert( end, `: ${name}` );
+						magicString.insertLeft( end, `: ${name}` );
 					} else {
 						magicString.overwrite( start, end, name, true );
 					}

test/cli/multiple-targets-shared-config/_expected/cjs.js.map

@@ -1 +1 @@
-{"version":3,"file":"cjs.js","sources":["../main.js"],"sourcesContent":["export default 0;\n"],"names":[],"mappings":";;WAAe,CAAC,CAAC,;;"}
+{"version":3,"file":"cjs.js","sources":["../main.js"],"sourcesContent":["export default 0;\n"],"names":[],"mappings":";;AAAA,WAAe,CAAC,CAAC,;;"}

test/cli/multiple-targets-shared-config/_expected/es6.js.map

@@ -1 +1 @@
-{"version":3,"file":"es6.js","sources":["../main.js"],"sourcesContent":["export default 0;\n"],"names":[],"mappings":"WAAe,CAAC,CAAC,;;"}
+{"version":3,"file":"es6.js","sources":["../main.js"],"sourcesContent":["export default 0;\n"],"names":[],"mappings":"AAAA,WAAe,CAAC,CAAC,;;"}

test/test.js

@@ -13,7 +13,6 @@ var FUNCTION = path.resolve( __dirname, 'function' );
 var FORM = path.resolve( __dirname, 'form' );
 var SOURCEMAPS = path.resolve( __dirname, 'sourcemaps' );
 var CLI = path.resolve( __dirname, 'cli' );
-var INCREMENTAL = path.resolve( __dirname, 'incremental' );
 
 var PROFILES = [
 	{ format: 'amd' },