sanitize filenames before SSRing - fixes #1005

src/server-side-rendering/register.js

@@ -26,10 +26,14 @@ function _deregister(extension) {
 
 function _register(extension) {
 	require.extensions[extension] = function(module, filename) {
+		const name = path.basename(filename)
+			.slice(0, -path.extname(filename).length)
+			.replace(/^\d/, '_$&')
+			.replace(/[^a-zA-Z0-9_$]/g, '');
+
 		const options = Object.assign({}, compileOptions, {
 			filename,
-			name: capitalise(path.basename(filename)
-				.replace(new RegExp(`${extension.replace('.', '\\.')}$`), '')),
+			name: capitalise(name),
 			generate: 'ssr'
 		});
 

test/helpers.js

@@ -175,11 +175,17 @@ function capitalise(str) {
 export function showOutput(cwd, options = {}, s = svelte) {
 	glob.sync('**/*.html', { cwd }).forEach(file => {
 		if (file[0] === '_') return;
+
+		const name = path.basename(file)
+			.slice(0, -path.extname(file).length)
+			.replace(/^\d/, '_$&')
+			.replace(/[^a-zA-Z0-9_$]/g, '');
+
 		const { code } = s.compile(
 			fs.readFileSync(`${cwd}/${file}`, 'utf-8'),
 			Object.assign(options, {
 				filename: file,
-				name: capitalise(path.basename(file).replace(/\.html$/, ''))
+				name: capitalise(name)
 			})
 		);
 

test/server-side-rendering/samples/sanitize-name/@foo.html

@@ -0,0 +1 @@
+<p>foo!</p>

test/server-side-rendering/samples/sanitize-name/_expected.html

@@ -0,0 +1 @@
+<p>foo!</p>

test/server-side-rendering/samples/sanitize-name/main.html

@@ -0,0 +1,9 @@
+<Foo/>
+
+<script>
+	import Foo from './@foo.html';
+
+	export default {
+		components: { Foo }
+	};
+</script>