fix: upgrade dependencies, ditch patches and move tinycolor2 patch (#1442)

Author: jorenbroekema

Diff for: .changeset/itchy-eagles-refuse.md

@@ -0,0 +1,5 @@
+---
+'style-dictionary': patch
+---
+
+Move the tinycolor2 patch from a patch file to a 3-liner fix in our source code, so it gets correctly published and usable by consumers.

Diff for: lib/common/formats.js

@@ -158,7 +158,11 @@ function stripMetaProps(tokens, options) {
 async function formatJS(content, ts = false) {
   return prettier.format(content, {
     parser: ts ? `typescript` : `babel`,
-    plugins: [prettierPluginBabel, prettierPluginEstree, prettierPluginTypescript],
+    plugins: [
+      prettierPluginBabel,
+      /** @type {import('prettier').Plugin} */ (prettierPluginEstree),
+      prettierPluginTypescript,
+    ],
   });
 }
 

Diff for: lib/common/transforms.js

@@ -44,9 +44,15 @@ const camelOpts = {
  * @returns {boolean}
  */
 export function isColor(token, options) {
+  const val = options.usesDtcg ? token.$value : token.value;
+  const type = options.usesDtcg ? token.$type : token.type;
   return (
-    (options.usesDtcg ? token.$type : token.type) === 'color' &&
-    Color(options.usesDtcg ? token.$value : token.value).isValid()
+    type === 'color' &&
+    Color(val).isValid() &&
+    // exclude gradients from being color transformed
+    ['linear', 'radial', 'conic']
+      .map((pre) => `${pre}-gradient`)
+      .every((pre) => !`${val}`.startsWith(pre) && !`${val}`.startsWith(`repeating-${pre}`))
   );
 }
 

Diff for: package-lock.json

@@ -130,8 +130,7 @@
   },
   "devDependencies": {
     "@astrojs/check": "^0.9.4",
-    "@astrojs/starlight": "^0.29.2",
-    "@babel/plugin-proposal-private-property-in-object": "^7.21.11",
+    "@astrojs/starlight": "^0.31.1",
     "@changesets/cli": "^2.27.1",
     "@commitlint/cli": "^18.4.3",
     "@commitlint/config-conventional": "^18.4.3",
@@ -142,16 +141,16 @@
     "@types/chai": "^4.3.9",
     "@types/tinycolor2": "^1.4.6",
     "@types/uuid": "^9.0.8",
-    "@typescript-eslint/parser": "^8.16.0",
+    "@typescript-eslint/parser": "^8.22.0",
     "@web/test-runner": "^0.18.2",
     "@web/test-runner-commands": "^0.9.0",
     "@web/test-runner-playwright": "^0.11.0",
     "acorn": "^8.11.3",
-    "astro": "^4.16.16",
+    "astro": "^5.2.3",
     "chai": "^5.0.0-alpha.2",
-    "eslint": "^9.14.0",
+    "eslint": "^9.19.0",
     "eslint-plugin-mocha": "^10.5.0",
-    "eslint-plugin-react": "^7.37.2",
+    "eslint-plugin-react": "^7.37.4",
     "estree-walker": "^3.0.3",
     "fs-extra": "^10.0.0",
     "hanbi": "^1.0.1",
@@ -169,7 +168,7 @@
     "npm-run-all": "^4.1.5",
     "sass": "^1.69.5",
     "sharp": "^0.32.5",
-    "starlight-links-validator": "^0.13.3",
+    "starlight-links-validator": "^0.14.2 ",
     "stylus": "^0.56.0",
     "typescript": "^5.7.2",
     "unist-util-visit": "^5.0.0",

Diff for: package.json

@@ -1,36 +1,26 @@
 diff --git a/node_modules/@astrojs/starlight/utils/createTranslationSystem.ts b/node_modules/@astrojs/starlight/utils/createTranslationSystem.ts
-index 540b74b..0a06296 100644
+index 540b74b..7e8e38f 100644
 --- a/node_modules/@astrojs/starlight/utils/createTranslationSystem.ts
 +++ b/node_modules/@astrojs/starlight/utils/createTranslationSystem.ts
 @@ -125,6 +125,7 @@ function buildResources<T extends Record<string, string | undefined>>(
  // strings. We let TypeScript merge them into a single union type so that plugins with a TypeScript
  // configuration preventing `UserI18nKeys` to be properly inferred can still get auto-completion
  // for built-in UI strings.
-+// @ts-expect-error not imported, only in locals.d.ts
++// @ts-expect-error StarlightApp missing
  export type I18nKeys = keyof BuiltInStrings | UserI18nKeys | keyof StarlightApp.I18n;
 
  export type I18nT = TFunction<'starlight', undefined> & {
-diff --git a/node_modules/@astrojs/starlight/utils/translations-fs.ts b/node_modules/@astrojs/starlight/utils/translations-fs.ts
-index 4bcfcda..9b783a9 100644
---- a/node_modules/@astrojs/starlight/utils/translations-fs.ts
-+++ b/node_modules/@astrojs/starlight/utils/translations-fs.ts
-@@ -1,6 +1,7 @@
- import fs from 'node:fs';
- import path from 'node:path';
- import { fileURLToPath } from 'node:url';
-+// @ts-expect-error @types/js-yaml not installed
- import yaml from 'js-yaml';
- import type { i18nSchemaOutput } from '../schemas/i18n';
- import { createTranslationSystem } from './createTranslationSystem';
 diff --git a/node_modules/@astrojs/starlight/utils/translations.ts b/node_modules/@astrojs/starlight/utils/translations.ts
-index 499ded2..87e0a1a 100644
+index 71ce6e7..6707e1e 100644
 --- a/node_modules/@astrojs/starlight/utils/translations.ts
 +++ b/node_modules/@astrojs/starlight/utils/translations.ts
-@@ -1,5 +1,7 @@
+@@ -1,6 +1,9 @@
  import { getCollection, type CollectionEntry, type DataCollectionKey } from 'astro:content';
-+// @ts-expect-error virtual imports not working in context of installed as dependency
++// @ts-expect-error virtual module not found
  import config from 'virtual:starlight/user-config';
-+// @ts-expect-error virtual imports not working in context of installed as dependency
++// @ts-expect-error virtual module not found
+ import project from 'virtual:starlight/project-context';
++// @ts-expect-error virtual module not found
  import pluginTranslations from 'virtual:starlight/plugin-translations';
  import type { i18nSchemaOutput } from '../schemas/i18n';
  import { createTranslationSystem } from './createTranslationSystem';