Add linter for support blocks filled only with false

scripts/fix/index.ts

@@ -12,6 +12,7 @@ import fixPropertyOrder from './property-order.js';
 import fixStatementOrder from './statement-order.js';
 import fixLinks from './links.js';
 import fixStatus from './status.js';
+import fixSupportHistory from './support-history.js';
 
 const dirname = fileURLToPath(new URL('.', import.meta.url));
 
@@ -40,6 +41,7 @@ function load(...files: string[]): void {
         fixStatementOrder(file);
         fixLinks(file);
         fixStatus(file);
+        fixSupportHistory(file);
       }
 
       continue;

scripts/fix/support-history.ts

@@ -0,0 +1,44 @@
+/* This file is a part of @mdn/browser-compat-data
+ * See LICENSE file for more information. */
+
+import { readFileSync, writeFileSync, unlinkSync } from 'node:fs';
+import { hasSupportHistory } from '../../test/linter/test-support-history';
+
+/**
+ * @param {Identifier} data
+ */
+function check(data) {
+  let hasIssue = false;
+  if (data.__compat && !hasSupportHistory(data)) {
+    hasIssue = true;
+  }
+  for (const member in data) {
+    if (member === '__compat') {
+      continue;
+    }
+    if (check(data[member])) {
+      delete data[member];
+    }
+  }
+  return hasIssue;
+}
+
+function fixSupportHistory(filename: string) {
+  const actual = readFileSync(filename, 'utf-8').trim();
+  const data = JSON.parse(actual);
+
+  check(data);
+  const expected = JSON.stringify(data, null, 2);
+
+  let target = data;
+  while (Object.keys(target).length === 1) {
+    target = Object.values(target)[0];
+  }
+  if (Object.keys(target).length) {
+    writeFileSync(filename, expected + '\n', 'utf-8');
+  } else {
+    unlinkSync(filename);
+  }
+}
+
+export default fixSupportHistory;

test/linter/index.ts

@@ -15,6 +15,7 @@ import testSchema from './test-schema.js';
 import testSpecURLs from './test-spec-urls.js';
 import testStatus from './test-status.js';
 import testStyle from './test-style.js';
+import testSupportHistory from './test-support-history.js';
 import testVersions from './test-versions.js';
 
 export default new Linters([
@@ -30,5 +31,6 @@ export default new Linters([
   testSpecURLs,
   testStatus,
   testStyle,
+  testSupportHistory,
   testVersions,
 ]);

test/linter/test-support-history.ts

@@ -0,0 +1,45 @@
+import chalk from 'chalk-template';
+import { Linter, Logger } from '../utils.js';
+
+import type {
+  CompatStatement,
+  SimpleSupportStatement,
+} from '../../types/types';
+
+function stringOrArrayIncludes(
+  target: SimpleSupportStatement['notes'],
+  str: string,
+) {
+  if (!target) {
+    return false;
+  }
+  if (Array.isArray(target)) {
+    return target.some((item) => item.includes(str));
+  }
+  return target.includes(str);
+}
+
+function includesTrackingBug(statement: SimpleSupportStatement) {
+  return (
+    stringOrArrayIncludes(statement.notes, 'crbug.com') ||
+    stringOrArrayIncludes(statement.notes, 'bugzil.la') ||
+    stringOrArrayIncludes(statement.notes, 'webkit.org/b/')
+  );
+}
+
+export function hasSupportHistory(compat: CompatStatement) {
+  return Object.values(compat.support).some(
+    (c) => Array.isArray(c) || !!c.version_added || includesTrackingBug(c),
+  );
+}
+
+export default {
+  name: 'Support history',
+  description: 'Ensure that prefixes in support statements are valid',
+  scope: 'feature',
+  check(logger: Logger, { data }: { data: CompatStatement }) {
+    if (!hasSupportHistory(data)) {
+      logger.error(chalk`No support and no tracking bug`);
+    }
+  },
+} as Linter;