fix(css/parser): correctly parse container query scroll-state

[ematipico]

Summary

Closes #8842

With the help of an AI agent, it turns out the parser was at fault.

Test Plan

Added new tests

Docs

[changeset-bot]

🦋 Changeset detected

Latest commit: d82d74b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 13 packages

Name	Type
@biomejs/biome	Patch
@biomejs/cli-win32-x64	Patch
@biomejs/cli-win32-arm64	Patch
@biomejs/cli-darwin-x64	Patch
@biomejs/cli-darwin-arm64	Patch
@biomejs/cli-linux-x64	Patch
@biomejs/cli-linux-arm64	Patch
@biomejs/cli-linux-x64-musl	Patch
@biomejs/cli-linux-arm64-musl	Patch
@biomejs/wasm-web	Patch
@biomejs/wasm-bundler	Patch
@biomejs/wasm-nodejs	Patch
@biomejs/backend-jsonrpc	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2d0b8e6 and d82d74b.

⛔ Files ignored due to path filters (1)

• crates/biome_css_formatter/tests/specs/css/atrule/container.css.snap is excluded by !**/*.snap and included by **

📒 Files selected for processing (3)

• .changeset/fix-scroll-state-container-query.md
• crates/biome_css_formatter/tests/specs/css/atrule/container.css
• crates/biome_css_parser/src/syntax/at_rule/container/mod.rs

---

Walkthrough

This PR fixes a CSS formatter bug where @container scroll-state() queries were incorrectly formatted with unwanted spaces between the function name and opening parenthesis. The fix modifies the CSS parser to recognise scroll-state queries as a special case, preventing optional declarator names from being parsed when a scroll-state query is present. Supporting test cases verify the fix handles various scroll-state syntax variations.

Possibly related PRs

• PR #8855: Also modifies container at-rule parsing and formatting to recognise and special-case scroll-state(...) syntax with shared test coverage.

Suggested labels

A-Parser, A-Formatter, L-CSS

Suggested reviewers

• dyc3

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the core fix: correcting CSS parser handling of container query scroll-state syntax.
Description check	✅ Passed	The description is related to the changeset, referencing issue #8842 and explaining that the parser was at fault with tests added.
Linked Issues check	✅ Passed	The PR successfully addresses issue #8842 by fixing the parser to correctly handle scroll-state container queries and adding test coverage.
Out of Scope Changes check	✅ Passed	All changes are scoped to fixing the CSS parser's scroll-state handling and adding relevant tests; no extraneous modifications detected.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/css-scroll-state

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codspeed-hq]

Merging this PR will not alter performance

✅ 29 untouched benchmarks
⏩ 187 skipped benchmarks¹

---

_{Comparing fix/css-scroll-state (d82d74b) with main (2d0b8e6)}

Footnotes

1. 187 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[denbezrukov]

Hmm, that’s weird. Thanks for the fix!
I'll try to figure out what exactly caused it, since I haven't touched that code.