postcss-bundler : fix parser (#1137)

Author: romainmenke

plugin-packs/postcss-bundler/.tape.mjs

@@ -19,6 +19,10 @@ const testCases = {
 		message: "throws on files that don't exist",
 		exception: /Failed to find 'imports\/does-not-exist.css'/,
 	},
+	'ignore': {
+		message: "ignores incorrect syntax",
+		warnings: 2,
+	},
 	'examples/example': {
 		message: 'minimal example',
 	},

plugin-packs/postcss-bundler/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changes to PostCSS Bundler
 
+### Unreleased (patch)
+
+- Ignore `@import` statements where `supports` or `layer` conditions precede the `url` value.
+
 ### 1.0.4
 
 _September 24, 2023_

plugin-packs/postcss-bundler/dist/index.cjs

@@ -19,7 +19,7 @@ export function applyConditions(bundle: Array<Statement>, atRule: (defaults?: At
 					stmt.conditions[0].media,
 					stmt.conditions[0].supports,
 				)}`;
-			} else if (stmt.conditions.length > 1) {
+			} else {
 				const reverseConditions = stmt.conditions.slice().reverse();
 				const first = reverseConditions.pop()!;
 				let params = `${stmt.fullUri} ${formatImportPrelude(

plugin-packs/postcss-bundler/dist/index.mjs

@@ -63,6 +63,10 @@ export function parseAtImport(params: string) {
 			}
 		}
 
+		if (!uri) {
+			return false;
+		}
+
 		if (
 			isTokenNode(componentValue) &&
 			componentValue.value[0] === TokenType.Ident &&

plugin-packs/postcss-bundler/src/postcss-import/lib/apply-conditions.ts

@@ -0,0 +1,2 @@
+@import layer(foo) "imports/basic.css";
+@import supports(foo) "imports/basic.css";

plugin-packs/postcss-bundler/src/postcss-import/lib/parse-at-import.ts

@@ -0,0 +1 @@
+