Remove workaround for csstree

w3c/webref#1647 has been fixed.

Author: asamuzaK

lib/parsers.js

@@ -7,6 +7,7 @@ const {
 const { next: syntaxes } = require("@csstools/css-syntax-patches-for-csstree");
 const csstree = require("css-tree");
 const { LRUCache } = require("lru-cache");
+const propertyDefinitions = require("./generated/propertyDefinitions");
 const { asciiLowercase } = require("./utils/strings");
 
 // CSS global keywords
@@ -155,19 +156,13 @@ const parseCSS = (val, opt) => cssTree.parse(prepareValue(val), opt);
  * @returns {boolean} True if the value is valid, false otherwise.
  */
 const isValidPropertyValue = (prop, val) => {
+  if (!propertyDefinitions.has(prop)) {
+    return false;
+  }
   val = prepareValue(val);
   if (val === "") {
     return true;
   }
-  // cssTree.lexer does not support deprecated system colors
-  // @see https://github.com/w3c/webref/issues/1519#issuecomment-3120290261
-  // @see https://github.com/w3c/webref/issues/1647
-  if (SYS_COLORS.has(asciiLowercase(val))) {
-    if (/^(?:-webkit-)?(?:[a-z][a-z\d]*-)*color$/i.test(prop)) {
-      return true;
-    }
-    return false;
-  }
   const cacheKey = `isValidPropertyValue_${prop}_${val}`;
   const cachedValue = lruCache.get(cacheKey);
   if (typeof cachedValue === "boolean") {
@@ -243,6 +238,9 @@ const resolveCalc = (val, opt = { format: "specifiedValue" }) => {
  * @returns {string|Array<object>|undefined} The parsed value.
  */
 const parsePropertyValue = (prop, val, opt = {}) => {
+  if (!propertyDefinitions.has(prop)) {
+    return;
+  }
   const { caseSensitive } = opt;
   val = prepareValue(val);
   if (val === "" || hasVarFunc(val)) {
@@ -272,17 +270,6 @@ const parsePropertyValue = (prop, val, opt = {}) => {
         name: lowerCasedValue
       }
     ];
-  } else if (SYS_COLORS.has(lowerCasedValue)) {
-    if (/^(?:(?:-webkit-)?(?:[a-z][a-z\d]*-)*color|border)$/i.test(prop)) {
-      parsedValue = [
-        {
-          type: AST_TYPES.IDENTIFIER,
-          name: lowerCasedValue
-        }
-      ];
-    } else {
-      parsedValue = false;
-    }
   } else {
     try {
       const ast = parseCSS(val, {

package-lock.json

@@ -17,13 +17,13 @@
   "main": "./lib/CSSStyleDeclaration.js",
   "dependencies": {
     "@asamuzakjp/css-color": "^4.1.1",
-    "@csstools/css-syntax-patches-for-csstree": "^1.0.21",
+    "@csstools/css-syntax-patches-for-csstree": "^1.0.24",
     "css-tree": "^3.1.0",
     "lru-cache": "^11.2.4"
   },
   "devDependencies": {
     "@domenic/eslint-config": "^4.0.1",
-    "@webref/css": "^8.1.1",
+    "@webref/css": "^8.1.3",
     "eslint": "^9.39.1",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-prettier": "^5.5.4",

package.json

@@ -1465,11 +1465,11 @@ describe("isValidPropertyValue", () => {
     assert.strictEqual(output, true);
   });
 
-  it("should return true", () => {
+  it("should return false", () => {
     const input = "canvas";
     const output = parsers.isValidPropertyValue("-webkit-border-after-color", input);
 
-    assert.strictEqual(output, true);
+    assert.strictEqual(output, false);
   });
 
   it("should return false", () => {