fix: calculate node.length correctly for nested var() (#70)

Author: bartveneman

src/parse-value.test.ts

@@ -130,6 +130,24 @@ describe('Value Node Types', () => {
 				expect(value?.line).toBe(3)
 				expect(value?.column).toBe(10)
 			})
+
+			it('should have correct offset and length for var()', () => {
+				const value = getValue('div { color: var(--item); }')
+				expect(value?.start).toBe(13)
+				expect(value?.length).toBe(11)
+				expect(value?.end).toBe(24)
+				expect(value?.line).toBe(1)
+				expect(value?.column).toBe(14)
+			})
+
+			it('should have correct offset and length for var() with fallback', () => {
+				const value = getValue('div { color: var(--item1, var(--item2)); }')
+				expect(value?.start).toBe(13)
+				expect(value?.length).toBe(26)
+				expect(value?.end).toBe(39)
+				expect(value?.line).toBe(1)
+				expect(value?.column).toBe(14)
+			})
 		})
 
 		describe('OPERATOR', () => {
@@ -513,6 +531,18 @@ describe('Value Node Types', () => {
 				expect(func?.has_children).toBe(true)
 			})
 
+			it('should provide node.value for var() function with fallback', () => {
+				const root = parse('body { color: var(--primary-color, 1); }')
+				const decl = root.first_child?.first_child?.next_sibling?.first_child
+				const func = decl?.values[0]
+
+				expect(func?.type).toBe(FUNCTION)
+				expect(func?.name).toBe('var')
+				expect(func?.text).toBe('var(--primary-color, 1)')
+				expect(func?.value).toBe('--primary-color, 1')
+				expect(func?.has_children).toBe(true)
+			})
+
 			it('should parse transform value', () => {
 				const root = parse('body { transform: translateX(10px) rotate(45deg); }')
 				const decl = root.first_child?.first_child?.next_sibling?.first_child

src/parse-value.ts

@@ -247,10 +247,10 @@ export class ValueParser {
 			if (token_type === TOKEN_EOF) break
 			if (this.lexer.token_start >= this.value_end) break
 
-			// Track parentheses depth
-			if (token_type === TOKEN_LEFT_PAREN || token_type === TOKEN_FUNCTION) {
-				paren_depth++
-			} else if (token_type === TOKEN_RIGHT_PAREN) {
+			// Check for closing paren
+			// Note: We don't track paren_depth for TOKEN_LEFT_PAREN or TOKEN_FUNCTION here
+			// because parse_value_node() will recursively handle them
+			if (token_type === TOKEN_RIGHT_PAREN) {
 				paren_depth--
 				if (paren_depth === 0) {
 					content_end = this.lexer.token_start // Position of ')'