@nate10j Thank you for contributing and starting on this. It's looking good so far.

The rule will need to:

• check container names within declaration values
• account for the full container condition syntax

Container names can appear in the container-name property and the container shorthand property. You can use the value parser to find the container name within the value of these properties.

Container conditions have a broad syntax.

@container <container-condition># {
  <block-contents>
}

And container names can appear zero or multiple times within a @container prelude, e.g. foo and bar in:

@container foo (inline-size > 30em), bar style(--large: true) {}

I think we can use the value parser and keep the logic naive by assuming that any top-level word node that isn't none, or, and, or not is a container name.

You can use custom-property-pattern as a reference for:

• checking both declarations and at-rules
• using the value parser

I've also added some commitable suggestions as part of the first review to get the ball rolling.

Thanks again for contributing! It's a rule that many people will benefit from as we'll turn it on in our standard config.

@nate10j Thanks for working on the support for checking declaration values, too. It's looking good.

I've requested some refactoring changes.

Thank you for making the changes. I've suggested changes for another edge case that we'll need to consider.

It's looking great, thank you.

One last thing, we'll need to account for languageOptions because we're now ignoring global keywords.

I've suggested some changes and additional tests for this.

We'll need to run:

npm i -D jest-preset-stylelint@latest

As part of this PR because we only recently added support for testing languageOptions to our jest preset.

As an aside, thank you for your patience dealing with these changes. It's typical that there are a few rounds of changes when implementing a new rule, as we uncover and address edge cases.

LGTM, thank you.

(We can patch up the unrelated failing tests in another PR.)

CI is now recovered ✅