@jesscss/less-parser 2.0.0-alpha.7 → 2.0.0-alpha.8

package/package.json

@@ -4,7 +4,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "2.0.0-alpha.7",
+  "version": "2.0.0-alpha.8",
   "description": "Jess LESS parser",
   "main": "lib/index.cjs",
   "types": "lib/index.d.ts",
@@ -42,10 +42,10 @@
   "dependencies": {
     "chevrotain": "^12.0.0",
     "known-css-properties": "~0.37.0",
-    "@jesscss/css-parser": "2.0.0-alpha.7"
+    "@jesscss/css-parser": "2.0.0-alpha.8"
   },
   "peerDependencies": {
-    "@jesscss/core": "2.0.0-alpha.7"
+    "@jesscss/core": "2.0.0-alpha.8"
   },
   "peerDependenciesMeta": {
     "@jesscss/core": {
@@ -55,10 +55,10 @@
   "devDependencies": {
     "@types/node": "^18.19.31",
     "less": "^4.6.3",
-    "parseman": "0.25.0",
+    "parseman": "link:/Users/matthew/git/oss/parser-thing",
     "tsx": "~4.21.0",
-    "@jesscss/shared": "2.0.0-alpha.1",
-    "@jesscss/core": "2.0.0-alpha.7"
+    "@jesscss/core": "2.0.0-alpha.8",
+    "@jesscss/shared": "2.0.0-alpha.1"
   },
   "author": "Matthew Dean <matthew-dean@users.noreply.github.com>",
   "license": "MIT",

package/src/builders.ts

@@ -245,6 +245,13 @@ export class LessGrammar extends CssParser {
       }
     }
     const valItems = items.slice(colonIdx + 1, end);
+    // A tolerated trailing comma (`@x: a, b, c,;`) is not a value item — Less 4.x
+    // drops it, so a comma-list value keeps N items, not N + 1 empty. (Plain
+    // declarations go through the CSS builder, which already filters empty
+    // segments; the Less var path assembles valItems directly, so strip it here.)
+    if (valItems.length > 0 && valItems[valItems.length - 1]!.comp === ',') {
+      valItems.pop();
+    }
     if (valItems.length) {
       const vText = this._source.slice(valItems[0]!.span.start, valItems[valItems.length - 1]!.span.end);
       if (/(?:^|[\s,])\.-?[_a-zA-Z]/.test(vText)) {

package/src/grammar.ts

@@ -449,10 +449,11 @@ export const lessGrammar = compose([cssGrammar, rules({ trivia: rw }, (g: any) =
   const declaration = choice(g.VarDeclaration, g.CustomDeclaration, g.Declaration);
 
   // ── Values (Less: + Reference, NamedColor, EscapedValue) ────────────────────
-  // A comma must be followed by a value — a trailing comma (`a, b,`) is a parse
-  // error in Less v5 (stricter than Less 4.x, which tolerated it). The dangling
-  // comma is left unconsumed and surfaces as one syntax error via the net.
-  const valueList = sequence(g.valueSequence, many(sequence(literal(','), g.valueSequence)));
+  // A comma-separated value list, tolerating a single trailing comma (`a, b,`) as
+  // Less 4.x does — its `value` parser breaks out of the comma loop when no
+  // expression follows, silently dropping the dangling comma (the value builder's
+  // empty-segment filter mirrors this, so the trailing comma adds no list item).
+  const valueList = sequence(g.valueSequence, many(sequence(literal(','), g.valueSequence)), optional(literal(',')));
   // A space-separated value sequence: each item is a full top-level EXPRESSION
   // (topSum), so arithmetic folds into the grammar (`1 + 2` → one Operation) while
   // non-operator items stay a list (`1px 2px 3px`). topSum collapses to the bare