IgniteUI / igniteui-angular / 36674487087

92%

push

github

fix(migrations): update theme arguments after comments or spaces before colon (#17680)

* fix(migrations): update theme arguments after comments or spaces

Removed or renamed theme arguments were missed when a comment was in
front of them, when there was a space before the colon, or when the
stylesheet was outside sourceRoot. User functions whose name only ended
with a theme function name, such as app-grid-summary-theme, were
changed by mistake.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* fix(migrations): respect Sass scope when updating theme arguments

Distinguish mixin and function calls across comments and limit local
declarations to their enclosing SCSS or indented Sass scopes.

Rename arguments at their code position and check duplicates by argument
name, preserving comments and legacy suffix matching.

Add regression tests for comments, CRLF, and nested declaration scopes.

* fix(migrations): recognize Sass shorthand and equivalent names

Support indented Sass = declarations and + includes while preserving
local declaration scopes and distinguishing function expressions.
Treat underscore and hyphen spellings as equivalent local member names.

Add regression tests and document the primary migration fixes.

* fix(migrations): respect Sass declaration order for immediate calls

Migrate library calls before local mixin or function declarations.
Preserve later local calls and forward references in deferred bodies
and default arguments.

Add regression tests for SCSS and indented Sass and update release notes.

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Stamen Stoychev <sstoychev@infragistics.com>
Co-authored-by: Simeon Simeonoff <sim.simeonoff@gmail.com>

15713 of 17855 branches covered (88.0%)

Branch coverage included in aggregate %.

31580 of 33557 relevant lines covered (94.11%)

38229.3 hits per line