New "strict order" option for preprocessor (recreated)

site/content/docs/04-compile-time.md

@@ -203,6 +203,7 @@ result: {
 	}>,
 	options?: {
 		filename?: string
+		strictOrder?: boolean
 	}
 )
 ```
@@ -305,6 +306,30 @@ const { code } = svelte.preprocess(source, [
 });
 ```
 
+---
+
+User may override default preprocessor order by passing `strictOrder` option.
+
+```js
+const svelte = require('svelte/compiler');
+
+const { code } = svelte.preprocess(source, [
+	{
+		style: () => {
+			console.log('this runs first');
+		}
+	},
+	{
+		markup: () => {
+			console.log('this runs second');
+		}
+	}
+], {
+	filename: 'App.svelte',
+	strictOrder: true
+});
+```
+
 
 ### `svelte.walk`
 

src/compiler/compile/index.ts

@@ -26,7 +26,8 @@ const valid_options = [
 	'css',
 	'loopGuardTimeout',
 	'preserveComments',
-	'preserveWhitespace'
+	'preserveWhitespace',
+	'strictOrder'
 ];
 
 function validate_options(options: CompileOptions, warnings: Warning[]) {

src/compiler/preprocess/index.ts

@@ -67,67 +67,141 @@ async function replace_async(str: string, re: RegExp, func: (...any) => Promise<
 	return out;
 }
 
+async function process_markup(
+	source: string,
+	func: (...any) => Processed | Promise<Processed>,
+	filename: string,
+) {
+
+	const processed: Processed = await func({
+		content: source,
+		filename
+	});
+
+	return {
+		source: processed ? processed.code : source,
+		dependencies: processed.dependencies
+	};
+}
+
+async function process_script(
+	source: string,
+	func: (...any) => Processed | Promise<Processed>,
+	filename: string,
+) {
+	const dependencies = [];
+
+	source = await replace_async(
+		source,
+		/<!--[^]*?-->|<script(\s[^]*?)?>([^]*?)<\/script>/gi,
+		async (match, attributes = '', content) => {
+			if (!attributes && !content) {
+				return match;
+			}
+			attributes = attributes || '';
+			const processed: Processed = await func({
+				content,
+				attributes: parse_attributes(attributes),
+				filename
+			});
+			if (processed && processed.dependencies) {
+				dependencies.push(...processed.dependencies);
+			}
+
+			return processed ? `<script${attributes}>${processed.code}</script>` : match;
+		}
+	);
+
+	return {
+		source,
+		dependencies,
+	};
+}
+
+async function process_style(
+	source: string,
+	func: (...any) => Processed | Promise<Processed>,
+	filename: string,
+) {
+	const dependencies = [];
+
+	source = await replace_async(
+		source,
+		/<!--[^]*?-->|<style(\s[^]*?)?>([^]*?)<\/style>/gi,
+		async (match, attributes = '', content) => {
+			if (!attributes && !content) {
+				return match;
+			}
+			const processed: Processed = await func({
+				content,
+				attributes: parse_attributes(attributes),
+				filename
+			});
+
+			if (processed && processed.dependencies) {
+				dependencies.push(...processed.dependencies);
+			}
+
+			return processed ? `<style${attributes}>${processed.code}</style>` : match;
+		}
+	);
+
+	return {
+		source,
+		dependencies,
+	};
+}
+
+function merge_preprocessors(preprocessors: PreprocessorGroup[]): PreprocessorGroup[] {
+	return [
+		...preprocessors.map(({ markup }) => ({ markup })),
+		...preprocessors.map(({ script }) => ({ script })),
+		...preprocessors.map(({ style }) => ({ style })),
+	];
+}
+
 export default async function preprocess(
 	source: string,
 	preprocessor: PreprocessorGroup | PreprocessorGroup[],
-	options?: { filename?: string }
+	options?: { filename?: string; strictOrder?: boolean }
 ) {
 	// @ts-ignore todo: doublecheck
 	const filename = (options && options.filename) || preprocessor.filename; // legacy
+	const strictOrder = options && options.strictOrder;
 	const dependencies = [];
 
 	const preprocessors = Array.isArray(preprocessor) ? preprocessor : [preprocessor];
 
-	const markup = preprocessors.map(p => p.markup).filter(Boolean);
-	const script = preprocessors.map(p => p.script).filter(Boolean);
-	const style = preprocessors.map(p => p.style).filter(Boolean);
-
-	for (const fn of markup) {
-		const processed = await fn({
-			content: source,
-			filename
-		});
-		if (processed && processed.dependencies) dependencies.push(...processed.dependencies);
-		source = processed ? processed.code : source;
-	}
+	const order = strictOrder
+		? preprocessors
+		: merge_preprocessors(preprocessors);
 
-	for (const fn of script) {
-		source = await replace_async(
-			source,
-			/<!--[^]*?-->|<script(\s[^]*?)?>([^]*?)<\/script>/gi,
-			async (match, attributes = '', content) => {
-				if (!attributes && !content) {
-					return match;
-				}
-				attributes = attributes || '';
-				const processed = await fn({
-					content,
-					attributes: parse_attributes(attributes),
-					filename
-				});
-				if (processed && processed.dependencies) dependencies.push(...processed.dependencies);
-				return processed ? `<script${attributes}>${processed.code}</script>` : match;
+	for (const p of order) {
+		let processed;
+
+		if (p.markup) {
+			processed = await process_markup(source, p.markup, filename);
+			source = processed.source;
+			if (processed.dependencies && processed.dependencies.length) {
+				dependencies.push(...processed.dependencies);
 			}
-		);
-	}
+		}
 
-	for (const fn of style) {
-		source = await replace_async(
-			source,
-			/<!--[^]*?-->|<style(\s[^]*?)?>([^]*?)<\/style>/gi,
-			async (match, attributes = '', content) => {
-				if (!attributes && !content) {
-					return match;
-				}
-				const processed: Processed = await fn({
-					content,
-					attributes: parse_attributes(attributes),
-					filename
-				});
-				if (processed && processed.dependencies) dependencies.push(...processed.dependencies);
-				return processed ? `<style${attributes}>${processed.code}</style>` : match;
+		if (p.script) {
+			processed = await process_script(source, p.script, filename);
+			source = processed.source;
+			if (processed.dependencies && processed.dependencies.length) {
+				dependencies.push(...processed.dependencies);
 			}
-		);
+		}
+
+		if (p.style) {
+			processed = await process_style(source, p.style, filename);
+			source = processed.source;
+			if (processed.dependencies && processed.dependencies.length) {
+				dependencies.push(...processed.dependencies);
+			}
+		}
 	}
 
 	return {

test/preprocess/index.js

@@ -14,10 +14,20 @@ describe('preprocess', () => {
 		}
 
 		(config.skip ? it.skip : solo ? it.only : it)(dir, async () => {
-			const input = fs.readFileSync(`${__dirname}/samples/${dir}/input.svelte`, 'utf-8');
-			const expected = fs.readFileSync(`${__dirname}/samples/${dir}/output.svelte`, 'utf-8');
+			const input = fs.readFileSync(
+				`${__dirname}/samples/${dir}/input.svelte`,
+				'utf-8'
+			);
+			const expected = fs.readFileSync(
+				`${__dirname}/samples/${dir}/output.svelte`,
+				'utf-8'
+			);
 
-			const result = await svelte.preprocess(input, config.preprocess);
+			const result = await svelte.preprocess(
+				input,
+				config.preprocess,
+				config.options
+			);
 			fs.writeFileSync(`${__dirname}/samples/${dir}/_actual.html`, result.code);
 
 			assert.equal(result.code, expected);

test/preprocess/samples/normal-order/_config.js

@@ -0,0 +1,13 @@
+export default {
+	options: {
+		strictOrder: false,
+	},
+	preprocess: [
+		{
+			style: ({ content }) => ({ code: content.replace(/one/g, 'two') }),
+		},
+		{
+			markup: ({ content }) => ({ code: content.replace(/two/g, 'three') }),
+		},
+	],
+};

test/preprocess/samples/normal-order/input.svelte

@@ -0,0 +1,11 @@
+<p>one</p>
+
+<style>
+	.one {
+		color: red;
+	}
+</style>
+
+<script>
+	console.log('one');
+</script>

test/preprocess/samples/normal-order/output.svelte

@@ -0,0 +1,11 @@
+<p>one</p>
+
+<style>
+	.two {
+		color: red;
+	}
+</style>
+
+<script>
+	console.log('one');
+</script>

test/preprocess/samples/strict-order/_config.js

@@ -0,0 +1,13 @@
+export default {
+	options: {
+		strictOrder: true,
+	},
+	preprocess: [
+		{
+			style: ({ content }) => ({ code: content.replace(/one/g, 'two') }),
+		},
+		{
+			markup: ({ content }) => ({ code: content.replace(/two/g, 'three') }),
+		},
+	],
+};

test/preprocess/samples/strict-order/input.svelte

@@ -0,0 +1,11 @@
+<p>one</p>
+
+<style>
+	.one {
+		color: red;
+	}
+</style>
+
+<script>
+	console.log('one');
+</script>

test/preprocess/samples/strict-order/output.svelte

@@ -0,0 +1,11 @@
+<p>one</p>
+
+<style>
+	.three {
+		color: red;
+	}
+</style>
+
+<script>
+	console.log('one');
+</script>