less 4.7.0 → 4.8.0

package/dist/less.cjs

@@ -1,5 +1,5 @@
 /**
- * Less - Leaner CSS v4.7.0
+ * Less - Leaner CSS v4.8.0
  * http://lesscss.org
  *
  * Copyright (c) 2009-2026, Alexis Sellier <self@cloudhead.net>
@@ -4470,6 +4470,64 @@
             warn('A bare @variable in an at-rule prelude is deprecated. Use @{variable} interpolation instead.', index, 'DEPRECATED', 'variable-in-at-rule-prelude');
         }
 
+        /**
+         * Numeric-leading variable names are a Less extension rather than valid CSS
+         * identifier syntax. Keep accepting them through Less 4, but make the Less 5
+         * migration visible at every actual variable reference or definition.
+         *
+         * @param {string} name - either an @-prefixed name or the name inside @{...}
+         * @param {number} index - source position of the variable token
+         */
+        function warnNumericVariableName(name, index) {
+            if (/^(?:@@?)?[0-9]/.test(name)) {
+                warn('Variable names beginning with a number are deprecated and will be removed in Less 5.x. Rename the variable to start with a valid identifier character.', index, 'DEPRECATED', 'numeric-variable-name');
+            }
+        }
+
+        /**
+         * A lone '-' is not a CSS identifier. Less historically accepts it as a
+         * variable name; retain that in Less 4 only and warn on definitions,
+         * ordinary references, and interpolation references.
+         *
+         * @param {string} name - either @-/@@-, or the name inside @{...}
+         * @param {number} index - source position of the variable token
+         */
+        function warnDashOnlyVariableName(name, index) {
+            const bareName = name.replace(/^@@?/, '');
+            if (bareName === '-') {
+                warn('The dash-only variable names @- and @{-} are deprecated and will be removed in Less 5.x. Rename the variable to use a valid identifier.', index, 'DEPRECATED', 'dash-only-variable-name');
+            }
+        }
+
+        /**
+         * A lone '-' is not a CSS identifier. Less historically accepts it as a mixin
+         * name after '.' or '#'; retain that in Less 4 only.
+         *
+         * @param {string} name
+         * @param {number} index
+         */
+        function warnDashOnlyMixinName(name, index) {
+            if (name === '.-' || name === '#-') {
+                warn('The dash-only mixin names .-() and #-() are deprecated and will be removed in Less 5.x. Rename the mixin to use a valid CSS identifier.', index, 'DEPRECATED', 'dash-only-mixin-name');
+            }
+        }
+
+        /**
+         * CSS @charset is a source-header declaration, not a general dynamic at-rule.
+         * Less 4 keeps its historical interpolation behavior for compatibility, but
+         * makes each dynamic spelling visible before Less 5 rejects it.
+         *
+         * @param {import('../tree/node.js').default} value
+         * @param {number} index - source position of the @charset token
+         */
+        function warnDynamicCharset(value, index) {
+            if (value instanceof tree.Variable ||
+                (value instanceof tree.Quoted &&
+                    (value.containsVariables() || value.value.match(value.propRegex)))) {
+                warn('Dynamic @charset interpolation is deprecated and will be removed in Less 5.x. Use a static quoted encoding declaration instead.', index, 'DEPRECATED', 'dynamic-charset');
+            }
+        }
+
         function expect(arg, msg) {
             // some older browsers return typeof 'function' for RegExp
             const result = (arg instanceof Function) ? arg.call(parsers) : parserInput.$re(arg);
@@ -4923,7 +4981,7 @@
                         }
 
                         function condition() {
-                            return [expect(parsers.condition, 'expected condition')];
+                            return [expect(() => parsers.condition(false, true), 'expected condition')];
                         }
                     },
 
@@ -5048,6 +5106,8 @@
 
                         parserInput.save();
                         if (parserInput.currentChar() === '@' && (name = parserInput.$re(/^@@?[\w-]+/))) {
+                            warnNumericVariableName(name, index);
+                            warnDashOnlyVariableName(name, index);
                             ch = parserInput.currentChar();
                             if ((ch === '(' && !parserInput.prevChar().match(/^\s/))
                                 || (ch === '[' && !parserInput.prevChar().match(/^\s/))) {
@@ -5070,6 +5130,8 @@
                         const index = parserInput.i;
 
                         if (parserInput.currentChar() === '@' && (curly = parserInput.$re(/^@\{([\w-]+)\}/))) {
+                            warnNumericVariableName(curly[1], index);
+                            warnDashOnlyVariableName(curly[1], index);
                             return new(tree.Variable)(`@${curly[1]}`, index + currentIndex, fileInfo);
                         }
                     },
@@ -5192,7 +5254,11 @@
                 variable: function () {
                     let name;
 
-                    if (parserInput.currentChar() === '@' && (name = parserInput.$re(/^(@[\w-]+)\s*:/))) { return name[1]; }
+                    if (parserInput.currentChar() === '@' && (name = parserInput.$re(/^(@[\w-]+)\s*:/))) {
+                        warnNumericVariableName(name[1], parserInput.i - name[0].length);
+                        warnDashOnlyVariableName(name[1], parserInput.i - name[0].length);
+                        return name[1];
+                    }
                 },
 
                 //
@@ -5222,6 +5288,8 @@
                         }
 
                         if (!inValue) {
+                            warnNumericVariableName(name[1], i);
+                            warnDashOnlyVariableName(name[1], i);
                             name = name[1];
                         }
 
@@ -5378,6 +5446,9 @@
                             if (inValue || parsers.end()) {
                                 parserInput.forget();
                                 const mixin = new(tree.mixin.Call)(elements, args, index + currentIndex, fileInfo, !lookups && important);
+                                for (const element of elements) {
+                                    warnDashOnlyMixinName(element.value, element._index - currentIndex);
+                                }
                                 if (lookups) {
                                     return new tree.NamespaceValue(mixin, lookups);
                                 }
@@ -5574,6 +5645,7 @@
                         let ruleset;
                         let cond;
                         let variadic = false;
+                        const index = parserInput.i;
                         if ((parserInput.currentChar() !== '.' && parserInput.currentChar() !== '#') ||
                             parserInput.peek(/^[^{]*\}/)) {
                             return;
@@ -5609,6 +5681,7 @@
 
                             if (ruleset) {
                                 parserInput.forget();
+                                warnDashOnlyMixinName(name, index);
                                 return new(tree.mixin.Definition)(name, params, ruleset, cond, variadic);
                             } else {
                                 parserInput.restore();
@@ -6657,6 +6730,9 @@
                         if (!value) {
                             error(`expected ${name} identifier`);
                         }
+                        if (nonVendorSpecificName === '@charset') {
+                            warnDynamicCharset(value, index);
+                        }
                     } else if (hasExpression) {
                         // `@namespace` may carry an interpolated `@{ns}` prefix (or a
                         // deprecated bare `@ns`). Parse that prefix directly so `@{ns}`
@@ -6861,7 +6937,7 @@
                         return condition || a;
                     }
                 },
-                condition: function (needsParens) {
+                condition: function (needsParens, allowConditionOperands) {
                     let result;
                     let logical;
                     let next;
@@ -6869,13 +6945,13 @@
                         return parserInput.$str('or');
                     }
 
-                    result = this.conditionAnd(needsParens);
+                    result = this.conditionAnd(needsParens, allowConditionOperands);
                     if (!result) {
                         return ;
                     }
                     logical = or();
                     if (logical) {
-                        next = this.condition(needsParens);
+                        next = this.condition(needsParens, allowConditionOperands);
                         if (next) {
                             result = new(tree.Condition)(logical, result, next);
                         } else {
@@ -6884,13 +6960,13 @@
                     }
                     return result;
                 },
-                conditionAnd: function (needsParens) {
+                conditionAnd: function (needsParens, allowConditionOperands) {
                     let result;
                     let logical;
                     let next;
                     const self = this;
                     function insideCondition() {
-                        const cond = self.negatedCondition(needsParens) || self.parenthesisCondition(needsParens);
+                        const cond = self.negatedCondition(needsParens, allowConditionOperands) || self.parenthesisCondition(needsParens, allowConditionOperands);
                         if (!cond && !needsParens) {
                             return self.atomicCondition(needsParens);
                         }
@@ -6904,9 +6980,12 @@
                     if (!result) {
                         return ;
                     }
+                    if (allowConditionOperands) {
+                        result = this.atomicCondition(needsParens, result, allowConditionOperands) || result;
+                    }
                     logical = and();
                     if (logical) {
-                        next = this.conditionAnd(needsParens);
+                        next = this.conditionAnd(needsParens, allowConditionOperands);
                         if (next) {
                             result = new(tree.Condition)(logical, result, next);
                         } else {
@@ -6915,9 +6994,9 @@
                     }
                     return result;
                 },
-                negatedCondition: function (needsParens) {
+                negatedCondition: function (needsParens, allowConditionOperands) {
                     if (parserInput.$str('not')) {
-                        const result = this.parenthesisCondition(needsParens);
+                        const result = this.parenthesisCondition(needsParens, allowConditionOperands);
                         if (result) {
                             result.negate = !result.negate;
                             return result;
@@ -6934,11 +7013,11 @@
                         }
                     }
                 },
-                parenthesisCondition: function (needsParens) {
+                parenthesisCondition: function (needsParens, allowConditionOperands) {
                     function tryConditionFollowedByParenthesis(me) {
                         let body;
                         parserInput.save();
-                        body = me.condition(needsParens);
+                        body = me.condition(needsParens, allowConditionOperands);
                         if (!body) {
                             parserInput.restore();
                             return ;
@@ -6975,7 +7054,7 @@
                     parserInput.forget();
                     return body;
                 },
-                atomicCondition: function (needsParens, preparsedCond) {
+                atomicCondition: function (needsParens, preparsedCond, allowConditionOperands) {
                     const entities = this.entities;
                     const index = parserInput.i;
                     let a;
@@ -6984,7 +7063,7 @@
                     let op;
 
                     const cond = (function() {
-                        return this.addition() || entities.keyword() || entities.quoted() || entities.mixinLookup();
+                        return (allowConditionOperands && this.parenthesisCondition(needsParens)) || this.addition() || entities.keyword() || entities.quoted() || entities.mixinLookup();
                     }).bind(this);
 
                     if (preparsedCond) {
@@ -7145,6 +7224,11 @@
                         }
                         for (k = 0; k < name.length; k++) {
                             s = name[k];
+                            if (s.charAt(0) === '@') {
+                                const variableName = s.slice(2, -1);
+                                warnNumericVariableName(variableName, index[k]);
+                                warnDashOnlyVariableName(variableName, index[k]);
+                            }
                             name[k] = (s.charAt(0) !== '@' && s.charAt(0) !== '$') ?
                                 new(tree.Keyword)(s) :
                                 (s.charAt(0) === '@' ?
@@ -11097,35 +11181,47 @@
          */
         matchArgs(args, context) {
             const allArgsCnt = (args && args.length) || 0;
-            let len;
-            const optionalParameters = this.optionalParameters;
-            const requiredArgsCnt = !args ? 0 : args.reduce(function (/** @type {number} */ count, /** @type {MixinArg} */ p) {
-                if (optionalParameters.indexOf(p.name) < 0) {
-                    return count + 1;
-                } else {
-                    return count;
+            const evaldArguments = new Array(this.params.length);
+            const positionalArgs = [];
+            let positionalIndex = 0;
+
+            for (let i = 0; i < allArgsCnt; i++) {
+                const arg = /** @type {MixinArg[]} */ (args)[i];
+                if (!arg.name) {
+                    positionalArgs.push(arg);
+                    continue;
                 }
-            }, 0);
 
-            if (!this.variadic) {
-                if (requiredArgsCnt < this.required) {
+                const paramIndex = this.params.findIndex((param, index) => param.name === arg.name && !evaldArguments[index]);
+                if (paramIndex < 0) {
                     return false;
                 }
-                if (allArgsCnt > this.params.length) {
-                    return false;
+                evaldArguments[paramIndex] = arg;
+            }
+
+            for (let i = 0; i < this.params.length; i++) {
+                if (evaldArguments[i]) {
+                    continue;
                 }
-            } else {
-                if (requiredArgsCnt < (this.required - 1)) {
+                if (this.params[i].variadic) {
+                    positionalIndex = positionalArgs.length;
+                    continue;
+                }
+                if (positionalIndex < positionalArgs.length) {
+                    evaldArguments[i] = positionalArgs[positionalIndex++];
+                } else if (!this.params[i].name || !this.params[i].value) {
                     return false;
                 }
             }
 
-            // check patterns
-            len = Math.min(requiredArgsCnt, this.arity);
+            if (positionalIndex < positionalArgs.length) {
+                return false;
+            }
 
-            for (let i = 0; i < len; i++) {
+            // check patterns
+            for (let i = 0; i < this.arity; i++) {
                 if (!this.params[i].name && !this.params[i].variadic) {
-                    if (/** @type {MixinArg[]} */ (args)[i].value.eval(context).toCSS(/** @type {EvalContext} */ ({})) != /** @type {Node} */ (this.params[i].value).eval(context).toCSS(/** @type {EvalContext} */ ({}))) {
+                    if (evaldArguments[i].value.eval(context).toCSS(/** @type {EvalContext} */ ({})) != /** @type {Node} */ (this.params[i].value).eval(context).toCSS(/** @type {EvalContext} */ ({}))) {
                         return false;
                     }
                 }
@@ -14292,7 +14388,7 @@
     };
 
     var name = "less";
-    var version = "4.7.0";
+    var version = "4.8.0";
     var description = "Leaner CSS";
     var homepage = "http://lesscss.org";
     var author = {