Fix misc. lints, replace and remove some deps (#1668)

Author: askoufis

examples/remix/package.json

@@ -13,7 +13,6 @@
     "@remix-run/react": "^2.17.2",
     "@remix-run/serve": "^2.17.2",
     "@vanilla-extract/css": "workspace:*",
-    "isbot": "^4",
     "react": "^18.2.0",
     "react-dom": "^18.2.0"
   },

package.json

@@ -62,5 +62,5 @@
   "manypkg": {
     "workspaceProtocol": "require"
   },
-  "packageManager": "pnpm@10.10.0"
+  "packageManager": "pnpm@10.24.0"
 }

packages/css/src/conditionalRulesets.ts

@@ -22,7 +22,7 @@ export class ConditionalRuleset {
    *
    * e.g. mobile -> tablet, desktop
    */
-  precedenceLookup: Map<Query, Set<String>>;
+  precedenceLookup: Map<Query, Set<string>>;
 
   constructor() {
     this.ruleset = new Map();

packages/css/src/transformCss.ts

@@ -287,7 +287,7 @@ class Stylesheet {
         value &&
         (value.includes('"') ||
           value.includes("'") ||
-          /^([A-Za-z\-]+\([^]*|[^]*-quote|inherit|initial|none|normal|revert|unset)(\s|$)/.test(
+          /^([A-Za-z-]+\([^]*|[^]*-quote|inherit|initial|none|normal|revert|unset)(\s|$)/.test(
             value,
           ))
           ? value

packages/css/src/validateSelector.ts

@@ -4,7 +4,7 @@ import dedent from 'dedent';
 
 // https://stackoverflow.com/questions/3561493/is-there-a-regexp-escape-function-in-javascript
 function escapeRegex(string: string) {
-  return string.replace(/[-\/\\^$*+?.()|[\]{}]/g, '\\$&');
+  return string.replace(/[-/\\^$*+?.()|[\]{}]/g, '\\$&');
 }
 
 export const validateSelector = (
@@ -24,7 +24,7 @@ export const validateSelector = (
   try {
     selectorParts = parse(selector);
   } catch (err) {
-    throw new Error(`Invalid selector: ${replaceTarget()}`);
+    throw new Error(`Invalid selector: ${replaceTarget()}`, { cause: err });
   }
 
   selectorParts.forEach((tokens) => {
@@ -67,6 +67,7 @@ export const validateSelector = (
         
         If your selector is targeting something global, use the 'globalStyle' function instead, e.g. if you wanted to write ${'`& h1`'}, you should instead write 'globalStyle(${'`${parent} h1`'}, { ... })'
       `,
+        { cause: err },
       );
     }
   });

packages/css/src/vars.ts

@@ -165,7 +165,7 @@ export function createGlobalThemeContract(
         : (value as string);
 
     const varName =
-      typeof rawVarName === 'string' ? rawVarName.replace(/^\-\-/, '') : null;
+      typeof rawVarName === 'string' ? rawVarName.replace(/^--/, '') : null;
 
     if (
       typeof varName !== 'string' ||

packages/integration/src/processVanillaFile.ts

@@ -269,7 +269,9 @@ function stringifyExports(
         } catch (err) {
           console.error(err);
 
-          throw new Error('Invalid function serialization params');
+          throw new Error('Invalid function serialization params', {
+            cause: err,
+          });
         }
       }
 

packages/sprinkles/src/createSprinkles.ts

@@ -170,9 +170,6 @@ export const createSprinkles =
               }
             }
 
-            const format = (v: string | number) =>
-              typeof v === 'string' ? `"${v}"` : v;
-
             const invalidPropValue = (
               prop: string,
               value: string | number,
@@ -277,3 +274,7 @@ export const createSprinkles =
       properties: new Set(sprinklesKeys),
     });
   };
+
+function format(v: string | number) {
+  return typeof v === 'string' ? `"${v}"` : v;
+}

pnpm-lock.yaml

@@ -38,7 +38,7 @@ const getAdapter = () => {
 };
 
 function extractFilesFromCodeBlock(code) {
-  const fileMatches = code.matchAll(/(?:\/\/\s(?<fileName>[\w\.]+\.tsx?))/g);
+  const fileMatches = code.matchAll(/(?:\/\/\s(?<fileName>[\w.]+\.tsx?))/g);
 
   let lastIndex = code.length - 1;