Description

Fixes #19203.

!default and !global inside a string or function argument were treated as Sass declaration flags. The flag text was sliced off the value, which injected a stray space:

/* input */
$a: "!default";
$b: unquote("!default");
$c: url("x!global");

/* before */
$a: " !default";
$b: unquote(" !default");
$c: url("x !global");

Cause

The directives were detected with /(\s*)(!default).*$/ and /(\s*)(!global).*$/. The .*$ matches the flag text anywhere in the value, including inside strings and url(...) / function arguments, after which everything from the match onward is sliced off.

Fix

In #19245 fisker suggested tightening the pattern to /(\s*)(!default)\s?$/ so it only matches a trailing flag. That is the right direction, but it drops the existing support for a flag followed by a comment, e.g. green !global /* note */, which has a snapshot in tests/format/scss/comments/variable-declaration.scss.

So the patterns now match the flag only when it sits at the end of the value, optionally followed by block comments:

const DEFAULT_SCSS_DIRECTIVE = /(\s*)(!default)(?:\s|\/\*.*?\*\/)*$/s;
const GLOBAL_SCSS_DIRECTIVE = /(\s*)(!global)(?:\s|\/\*.*?\*\/)*$/s;

Flag text inside a string or function no longer matches, while genuine trailing flags keep working, including a flag after a quoted value ($d: "!default" !default;) and a flag followed by a comment.

/* after */
$a: "!default";
$b: unquote("!default");
$c: url("x!global");

$d: "!default" !default;
$e: map-get($map, "!default") !default;

Tests

Added tests/format/scss/flag/19203.scss covering the in-string, url(...), and function-argument cases, plus mixed cases where a string value is followed by a real trailing flag. The existing flag and comments snapshots are unchanged, so genuine flags and flag-with-comment behavior are preserved.

Checklist

• I've added tests to confirm my change works.
• (If changing the API or CLI) I've documented the changes I've made (in the docs/ directory).
• (If the change is user-facing) I've added my changes to changelog_unreleased/*/XXXX.md file following changelog_unreleased/TEMPLATE.md.
• I've read the contributing guidelines.
• I did not use AI to generate this PR.
• (If the above is not checked) I have reviewed the AI-generated content before submitting.