broken ast

[vankop]

regarding to stylelint/stylelint#4049

.foo1, // this comments is marked as an error
.foo2 {
    color: #fff;
}

Parsers // this comments is marked as an error as selectors

[alexander-akait]

Known issue, we should rewrite parser whole, now we parse comments as part of ast, but it is invalid, we should use leadingComments, trailingComments and etc as do babel (try this in https://astexplorer.net/), it is allow to keep ast clean and works with comments in right way (without pain)

[vankop]

Is it possible to contribute? or this issue still is in discussion?

[alexander-akait]

@vankop it is breaking change, anyway let's do it, in future we should use safe option and throw error on this syntax when for pure css

[vankop]

@evilebottnawi I will take this.

[alexander-akait]

👍

[vankop]

@evilebottnawi should we skip comments in .toString() method?

Currently https://github.com/postcss/postcss-selector-parser/blob/master/src/__tests__/util/helpers.js#L24 we expect that comments will be in output

[alexander-akait]

No, we should print comments

[vankop]

Sorry I was a bit stuck with stringifying comments correctly and gave up =) I hope I will find time on the next week or will create WIP PR.

[daddyschmack]

This issue is occurring in upgrade for angular 12, and i also get this criptic error message that the angular people says comes from this progam after I replace all the // comments:
Error: Unexpected '/'. Escaping Social characters with \ may help

Shouldn't the error message give a location where this error occurs?

[guilhermetod]

@daddyschmack just helped a teammate with this. Apparently this is a problem with CRLF or something related to Stylelint's ignoreFiles.