LESS "each" function breaks indentation

[Some14u]

Description

The indentation for some less functions like list functions ("each" specifically) is broken.
https://lesscss.org/functions/#list-functions-each

Input

The code looked like this before beautification:

.my-function(@iterator) {
  each(@iterator, {
    color: red;           // payload: could be anything
  });

  .wrong-indentation-here {
    color: blue;
  }
}

Expected Output

The code should have looked like this after beautification:

.my-function(@iterator) {
  each(@iterator, {
    color: red;           // payload: could be anything
  });

  .wrong-indentation-here {
    color: blue;
  }
}

Actual Output

The code actually looked like this after beautification:

.my-function(@iterator) {
  each(@iterator, {
    color: red;           // payload: could be anything
  });

.wrong-indentation-here {
  color: blue;
}
}

Steps to Reproduce

Looks like the out-dent heuristic that fires when a } is followed by )… ; zeroes the indent even when brace_depth > 0.

Environment

OS:
Windows 10

Settings

[Some14u]

I found related issue: #2124

[Some14u]

What goes wrong

1. After the formatter prints a }, it calls add_new_line(), which creates a new line containing the correct indent for the current depth.

2. If the next significant character is ) (the }); one-liner pattern used in Less/SCSS mixin calls) it immediately does:

   this._output.trim(true);   // delete the newline we just created

   trim(true) pops that brand-new line off the buffer, so the formatter’s “current line” is now the previous line—indented one level less.

3. When ; is printed the formatter calls add_new_line() again. It clones the indent of the shallower line, so everything after the }); block shifts left.

Minimal, backward-compatible fix

--- a/js/src/css/beautifier.js
+++ b/js/src/css/beautifier.js
@@
   this._output.add_new_line();

   // collapse “});” onto one physical line, but keep the indent we need
   if (this._input.peek() === ')') {
     this._output.trim(true);                    // keep `});` on one line
+    this._output.set_indent(this._indentLevel); // restore correct indent
     if (this._options.brace_style === 'expand') {
       this._output.add_new_line(true);
     }
   }

Python mirror:

--- a/python/cssbeautifier/css/beautifier.py
+++ b/python/cssbeautifier/css/beautifier.py
@@
         self._output.add_new_line()

         if self._input.peek() == ')':
            self._output.trim(True)
+           self._output.set_indent(self._indentLevel)
            if self._options.brace_style == 'expand':
               self._output.add_new_line(True)