BREAKING: Dedupe tokens: color, font-size, line-height, duration (#28)

* dedupe font-sizes

* dedupe line heights

* deduplicate colors, line-heights, font-sizes and durations

* BREAKING: target node v22+, rewrite some stuff

* rm more tsconfig

* upgrade GH actions to v5 for Node 24

* fix CI node versions

---------

Co-authored-by: Bart Veneman <[email protected]>

Author: bartveneman

.github/workflows/release.yml

@@ -11,21 +11,25 @@ jobs:
   test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v4
-      - run: npm install --ignore-scripts --no-audit --no-fund
+      - uses: actions/checkout@v5
+      - uses: actions/setup-node@v5
+        with:
+          cache: "npm"
+          node-version: 22
+      - run: npm ci --ignore-scripts --no-audit --no-fund
       - run: npm test
 
   publish-npm:
     needs: test
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v4
+      - uses: actions/checkout@v5
+      - uses: actions/setup-node@v5
         with:
-          cache: 'npm'
+          node-version: 22
+          cache: "npm"
           registry-url: https://registry.npmjs.org/
-      - run: npm install --ignore-scripts --no-audit --no-fund
+      - run: npm ci --ignore-scripts --no-audit --no-fund
       - run: npm run build
       - run: npm publish --public
         env:

.github/workflows/test.yml

@@ -14,23 +14,25 @@ jobs:
     name: Unit tests
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
       - name: Use Node.js
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           cache: "npm"
+          node-version: 22
       - run: npm install --ignore-scripts --no-audit --no-fund
       - run: npm test
 
   check:
     name: Check types
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
       - name: Use Node.js
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           cache: "npm"
+          node-version: 22
       - run: npm install --ignore-scripts --no-audit --no-fund
       - run: npm run check
 
@@ -39,11 +41,12 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
       - name: Use Node.js
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           cache: "npm"
+          node-version: 22
       - run: npm install --ignore-scripts --no-audit --no-fund
       - name: Build package
         run: npm run build

package.json

@@ -26,7 +26,7 @@
     "default": "./dist/css-design-tokens.js"
   },
   "engines": {
-    "node": ">=18"
+    "node": ">=22"
   },
   "scripts": {
     "test": "vitest run",

src/destructure-line-height.test.ts

@@ -22,6 +22,7 @@ test('percentage', () => {
 
 test('number', () => {
 	expect.soft(destructure_line_height('1')).toEqual(1)
+	expect.soft(destructure_line_height('1.0')).toEqual(1)
 	expect.soft(destructure_line_height('1.1')).toEqual(1.1)
 	expect.soft(destructure_line_height('1e2')).toEqual(100)
 })
@@ -36,6 +37,15 @@ test('length', () => {
 	expect.soft(destructure_line_height('1e2em')).toEqual({ value: 100, unit: 'em' })
 })
 
+test('zero', () => {
+	expect.soft(destructure_line_height('0%')).toEqual(0)
+	expect.soft(destructure_line_height('0.0%')).toEqual(0)
+	expect.soft(destructure_line_height('0px')).toEqual(0)
+	expect.soft(destructure_line_height('0.0px')).toEqual(0)
+	expect.soft(destructure_line_height('0')).toEqual(0)
+	expect.soft(destructure_line_height('0.0')).toEqual(0)
+})
+
 test('unprocessable values', () => {
 	expect.soft(destructure_line_height('var(--my-line-height)')).toEqual(null)
 	expect.soft(destructure_line_height('var(--my-line-height, 1.2)')).toEqual(null)

src/destructure-line-height.ts

@@ -21,8 +21,12 @@ export function destructure_line_height(value: string): Length | number | null {
 
 	switch (maybe_dimension.type) {
 		case 'Dimension': {
+			let value = Number(maybe_dimension.value)
+			if (value === 0) {
+				return 0
+			}
 			return {
-				value: Number(maybe_dimension.value),
+				value,
 				unit: maybe_dimension.unit
 			}
 		}