fix: don't mark selector lists inside :global with multiple items as unused

Regression from #15762

Fixes #15816

Author: dummdidumm

.changeset/wild-actors-retire.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: don't mark selector lists inside `:global` with multiple items as unused

packages/svelte/src/compiler/phases/3-transform/css/index.js

@@ -198,7 +198,7 @@ const visitors = {
 	SelectorList(node, { state, next, path }) {
 		// Only add comments if we're not inside a complex selector that itself is unused or a global block
 		if (
-			(!is_in_global_block(path) || node.children.length > 1) &&
+			(!is_in_global_block(path) || (node.children.length > 1 && is_in_global_block(path, true))) &&
 			!path.find((n) => n.type === 'ComplexSelector' && !n.metadata.used)
 		) {
 			const children = node.children;
@@ -376,11 +376,23 @@ const visitors = {
 };
 
 /**
- *
  * @param {Array<AST.CSS.Node>} path
+ * @param {boolean} check_parent_only
  */
-function is_in_global_block(path) {
-	return path.some((node) => node.type === 'Rule' && node.metadata.is_global_block);
+function is_in_global_block(path, check_parent_only = false) {
+	for (let i = path.length - 1; i >= 0; i--) {
+		const node = path[i];
+
+		if (node.type === 'Rule') {
+			if (check_parent_only) {
+				return node.metadata.is_global_block;
+			} else if (node.metadata.is_global_block) {
+				return true;
+			}
+		}
+	}
+
+	return false;
 }
 
 /**

packages/svelte/tests/css/samples/global-block/_config.js

@@ -7,28 +7,28 @@ export default test({
 			code: 'css_unused_selector',
 			message: 'Unused CSS selector ".unused :global"',
 			start: {
-				line: 69,
+				line: 73,
 				column: 1,
-				character: 917
+				character: 964
 			},
 			end: {
-				line: 69,
+				line: 73,
 				column: 16,
-				character: 932
+				character: 979
 			}
 		},
 		{
 			code: 'css_unused_selector',
 			message: 'Unused CSS selector "unused :global"',
 			start: {
-				line: 100,
+				line: 104,
 				column: 29,
-				character: 1223
+				character: 1270
 			},
 			end: {
-				line: 100,
+				line: 104,
 				column: 43,
-				character: 1237
+				character: 1284
 			}
 		}
 	]

packages/svelte/tests/css/samples/global-block/expected.css

@@ -3,6 +3,10 @@
 		.x {
 			color: green;
 		}
+
+		.a, .selector, .list {
+			color: green;
+		}
 	/*}*/
 
 	div.svelte-xyz {

packages/svelte/tests/css/samples/global-block/input.svelte

@@ -5,6 +5,10 @@
 		.x {
 			color: green;
 		}
+
+		.a, .selector, .list {
+			color: green;
+		}
 	}
 
 	div :global {