fix: insert the tailwindcss vite plugin at the start of the plugin array

.changeset/big-chefs-return.md

@@ -0,0 +1,5 @@
+---
+'sv': patch
+---
+
+fix: insert the `tailwindcss` vite plugin at the start of the plugin array

packages/addons/tailwindcss/index.ts

@@ -67,7 +67,7 @@ export default defineAddon({
 
 			const pluginsArray = object.property(param1, 'plugins', array.createEmpty());
 			const pluginFunctionCall = functions.call(vitePluginName, []);
-			array.push(pluginsArray, pluginFunctionCall);
+			array.unshift(pluginsArray, pluginFunctionCall);
 
 			return generateCode();
 		});

packages/core/tests/js/arrays/unshift-object-array/output.ts

@@ -0,0 +1,8 @@
+const array = [
+	{
+		test2: 'string'
+	},
+	{
+		test: true
+	}
+];

packages/core/tests/js/arrays/unshift-object-array/run.ts

@@ -0,0 +1,16 @@
+import { array, object, common, variables } from '@sveltejs/cli-core/js';
+import type { ScriptFileEditor } from '@sveltejs/cli-core';
+
+export function run({ ast }: ScriptFileEditor<any>): void {
+	const array1 = array.createEmpty();
+
+	const object1 = object.create({ test: common.expressionFromString('true') });
+	const object2 = object.create({ test2: common.createLiteral('string') });
+	array.unshift(array1, object1);
+	array.unshift(array1, object2);
+	array.unshift(array1, object2); // avoid duplication
+
+	// create declaration so that we serialize everything
+	const declaration = variables.declaration(ast, 'const', 'array', array1);
+	ast.body.push(declaration);
+}

packages/core/tests/js/arrays/unshift-string-array/output.ts

@@ -0,0 +1 @@
+const array = ['test2', 'test'];

packages/core/tests/js/arrays/unshift-string-array/run.ts

@@ -0,0 +1,13 @@
+import { array, variables } from '@sveltejs/cli-core/js';
+import type { ScriptFileEditor } from '@sveltejs/cli-core';
+
+export function run({ ast }: ScriptFileEditor<any>): void {
+	const array1 = array.createEmpty();
+	array.unshift(array1, 'test');
+	array.unshift(array1, 'test2');
+	array.unshift(array1, 'test'); // make sure items are not duplicated
+
+	// create declaration so that we serialize everything
+	const declaration = variables.declaration(ast, 'const', 'array', array1);
+	ast.body.push(declaration);
+}

packages/core/tooling/js/array.ts

@@ -13,30 +13,36 @@ export function push(
 	ast: AstTypes.ArrayExpression,
 	data: string | AstKinds.ExpressionKind | AstKinds.SpreadElementKind
 ): void {
-	if (typeof data === 'string') {
-		const existingLiterals = ast.elements.filter(
-			(x): x is AstTypes.StringLiteral => x?.type == 'StringLiteral'
-		);
-		let literal = existingLiterals.find((x) => x.value == data);
+	insertElement(ast, data, true);
+}
+export function unshift(
+	ast: AstTypes.ArrayExpression,
+	data: string | AstKinds.ExpressionKind | AstKinds.SpreadElementKind
+): void {
+	insertElement(ast, data, false);
+}
 
+function insertElement(
+	ast: AstTypes.ArrayExpression,
+	data: string | AstKinds.ExpressionKind | AstKinds.SpreadElementKind,
+	insertEnd: boolean
+): void {
+	if (typeof data === 'string') {
+		const existingLiterals = ast.elements.filter((x) => x?.type === 'StringLiteral');
+		let literal = existingLiterals.find((x) => x.value === data);
 		if (!literal) {
-			literal = {
-				type: 'StringLiteral',
-				value: data
-			};
-			ast.elements.push(literal);
+			literal = { type: 'StringLiteral', value: data };
+
+			if (insertEnd) ast.elements.push(literal);
+			else ast.elements.unshift(literal);
 		}
 	} else {
-		let anyNodeEquals = false;
 		const elements = ast.elements as AstTypes.ASTNode[];
-		for (const node of elements) {
-			if (areNodesEqual(data, node)) {
-				anyNodeEquals = true;
-			}
-		}
+		const anyNodeEquals = elements.some((node) => areNodesEqual(data, node));
 
 		if (!anyNodeEquals) {
-			ast.elements.push(data);
+			if (insertEnd) ast.elements.push(data);
+			else ast.elements.unshift(data);
 		}
 	}
 }