Add linter to make sure statuses have consistent inheritance

api/CanvasRenderingContext2D.json

@@ -1167,7 +1167,7 @@
             }
           },
           "status": {
-            "experimental": true,
+            "experimental": false,
             "standard_track": false,
             "deprecated": true
           }
@@ -1220,9 +1220,9 @@
               }
             },
             "status": {
-              "experimental": true,
+              "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         }

api/UIEvent.json

@@ -140,7 +140,7 @@
           "status": {
             "experimental": false,
             "standard_track": false,
-            "deprecated": true
+            "deprecated": false
           }
         },
         "implemented_on_event": {

api/WorkerGlobalScope.json

@@ -91,7 +91,7 @@
           "status": {
             "experimental": false,
             "standard_track": false,
-            "deprecated": true
+            "deprecated": false
           }
         },
         "close_moved": {

html/elements/script.json

@@ -551,7 +551,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": true
+              "deprecated": false
             }
           },
           "module": {

svg/elements/cursor.json

@@ -91,7 +91,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         },
@@ -138,7 +138,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         },
@@ -233,7 +233,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         }

svg/elements/missing-glyph.json

@@ -91,7 +91,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         },
@@ -138,7 +138,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         },
@@ -185,7 +185,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         },
@@ -232,7 +232,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         },
@@ -279,7 +279,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         }

svg/elements/vkern.json

@@ -91,7 +91,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         },
@@ -138,7 +138,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         },
@@ -185,7 +185,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         },
@@ -232,7 +232,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         },
@@ -279,7 +279,7 @@
             "status": {
               "experimental": false,
               "standard_track": true,
-              "deprecated": false
+              "deprecated": true
             }
           }
         }

test/lint.js

@@ -14,6 +14,7 @@ const {
   testVersions,
   testConsistency,
   testDescriptions,
+  testDeprecated,
 } = require('./linter/index.js');
 const { IS_CI } = require('./utils.js');
 const testCompareFeatures = require('./test-compare-features');
@@ -63,7 +64,8 @@ function load(...files) {
           hasConsistencyErrors = false,
           hasRealValueErrors = false,
           hasPrefixErrors = false,
-          hasDescriptionsErrors = false;
+          hasDescriptionsErrors = false,
+          hasDeprecatedErrors = false;
         const relativeFilePath = path.relative(process.cwd(), file);
 
         const spinner = ora({
@@ -103,6 +105,7 @@ function load(...files) {
             hasRealValueErrors = testRealValues(file);
             hasPrefixErrors = testPrefix(file);
             hasDescriptionsErrors = testDescriptions(file);
+            hasDeprecatedErrors = testDeprecated(file);
           }
         } catch (e) {
           hasSyntaxErrors = true;
@@ -120,6 +123,7 @@ function load(...files) {
           hasRealValueErrors,
           hasPrefixErrors,
           hasDescriptionsErrors,
+          hasDeprecatedErrors,
         ].some(x => !!x);
 
         if (fileHasErrors) {

test/linter/index.js

@@ -8,6 +8,7 @@ const testStyle = require('./test-style.js');
 const testVersions = require('./test-versions.js');
 const testConsistency = require('./test-consistency.js');
 const testDescriptions = require('./test-descriptions.js');
+const testDeprecated = require('./test-deprecated.js');
 
 module.exports = {
   testBrowsers,
@@ -19,4 +20,5 @@ module.exports = {
   testVersions,
   testConsistency,
   testDescriptions,
+  testDeprecated,
 };

test/linter/test-deprecated.js

@@ -0,0 +1,48 @@
+const chalk = require('chalk');
+const { Logger } = require('./utils.js');
+
+/**
+ * @typedef {import('../../types').Identifier} Identifier
+ */
+
+/**
+ * @param {Identifier} data
+ * @param {Logger} logger
+ * @param {boolean} isParentDeprecated,
+ * @param {string[]} path
+ */
+function checkStatus(data, logger, isParentDeprecated, path = []) {
+  const status = data.__compat && data.__compat.status;
+  const deprecated = status && status.deprecated;
+  if (!deprecated && isParentDeprecated) {
+    logger.error(
+      chalk`{red → Unexpected non-deprecated status while the parent is deprecated, in ${path.join(
+        '.',
+      )}}`,
+    );
+  }
+  for (const member in data) {
+    if (member === '__compat') {
+      continue;
+    }
+    checkStatus(data[member], logger, deprecated, [...path, member]);
+  }
+}
+
+/**
+ * @param {string} filename
+ * @returns {boolean} If the file contains errors
+ */
+function testDeprecated(filename) {
+  /** @type {Identifier} */
+  const data = require(filename);
+
+  const logger = new Logger('Deprecated');
+
+  checkStatus(data, logger, false);
+
+  logger.emit();
+  return logger.hasErrors();
+}
+
+module.exports = testDeprecated;