wrap script block in IIFE to prevent pollution

compiler/generate/index.js

@@ -111,13 +111,32 @@ export default function generate ( parsed, template, options = {} ) {
 		const defaultExport = parsed.js.content.body.find( node => node.type === 'ExportDefaultDeclaration' );
 
 		if ( defaultExport ) {
-			generator.code.overwrite( defaultExport.start, defaultExport.declaration.start, `const template = ` );
+			const finalNode = parsed.js.content.body[ parsed.js.content.body.length - 1 ];
+			if ( defaultExport === finalNode ) {
+				// export is last property, we can just return it
+				generator.code.overwrite( defaultExport.start, defaultExport.declaration.start, `return ` );
+			} else {
+				// TODO ensure `template` isn't already declared
+				generator.code.overwrite( defaultExport.start, defaultExport.declaration.start, `var template = ` );
+
+				let i = defaultExport.start;
+				while ( /\s/.test( template[ i - 1 ] ) ) i--;
+
+				const indentation = template.slice( i, defaultExport.start );
+				generator.code.insertLeft( finalNode.end, `\n\n${indentation}return template;` );
+			}
 
 			defaultExport.declaration.properties.forEach( prop => {
 				templateProperties[ prop.key.name ] = prop.value;
 			});
+
+			generator.code.insertRight( parsed.js.content.start, 'var template = (function () {' );
+		} else {
+			generator.code.insertRight( parsed.js.content.start, '(function () {' );
 		}
 
+		generator.code.insertLeft( parsed.js.content.end, '}());' );
+
 		[ 'helpers', 'events', 'components' ].forEach( key => {
 			if ( templateProperties[ key ] ) {
 				templateProperties[ key ].properties.forEach( prop => {
@@ -214,13 +233,19 @@ export default function generate ( parsed, template, options = {} ) {
 
 	const constructorName = options.name || 'SvelteComponent';
 
-	const result = deindent`
-		${parsed.js ? `[✂${parsed.js.content.start}-${parsed.js.content.end}✂]` : ``}
+	const topLevelStatements = [];
+
+	if ( parsed.js ) {
+		topLevelStatements.push( `[✂${parsed.js.content.start}-${parsed.js.content.end}✂]` );
+	}
 
-		${parsed.css ? addCss : ``}
+	if ( parsed.css ) {
+		topLevelStatements.push( processCss( parsed ) );
+	}
 
-		${renderers.reverse().join( '\n\n' )}
+	topLevelStatements.push( ...renderers.reverse() );
 
+	topLevelStatements.push( deindent`
 		export default function ${constructorName} ( options ) {
 			var component = this;${generator.usesRefs ? `\nthis.refs = {}` : ``}
 			var state = {};
@@ -308,9 +333,13 @@ export default function generate ( parsed, template, options = {} ) {
 
 			${templateProperties.onrender ? `template.onrender.call( this );` : ``}
 		}
+	` );
+
+	if ( templateProperties.methods ) {
+		topLevelStatements.push( `${constructorName}.prototype = template.methods;` );
+	}
 
-		${templateProperties.methods ? `${constructorName}.prototype = template.methods` : ''}
-	`;
+	const result = topLevelStatements.join( '\n\n' );
 
 	const pattern = /\[✂(\d+)-(\d+)$/;
 

compiler/parse/read/script.js

@@ -9,10 +9,9 @@ export default function readScript ( parser, start, attributes ) {
 		parser.index = scriptStart + token.end;
 		parser.allowWhitespace();
 
-		if ( parser.eat( '</script>' )  ) {
-			scriptEnd = scriptStart + token.end;
-			break;
-		}
+		scriptEnd = parser.index;
+
+		if ( parser.eat( '</script>' ) ) break;
 	}
 
 	const source = spaces( scriptStart ) + parser.template.slice( scriptStart, scriptEnd );

test/parser/script/output.json

@@ -46,7 +46,7 @@
 		"content": {
 			"type": "Program",
 			"start": 34,
-			"end": 95,
+			"end": 96,
 			"body": [
 				{
 					"declaration": {