feat: Split out deprecation warning for age=="89+"

Author: effigies

src/issues/list.ts

@@ -133,6 +133,10 @@ export const bidsIssues: IssueDefinitionRecord = {
     reason:
       'A column required in a TSV file has been redefined in a sidecar file. This redefinition is being ignored.',
   },
+  TSV_PSEUDO_AGE_DEPRECATED: {
+    severity: 'warning',
+    reason: 'Use of the value "89+" in column "age" is deprecated. Use 89 for all ages 89 and over.',
+  },
   INVALID_GZIP: {
     severity: 'error',
     reason: 'The gzip file could not be decompressed. It may be corrupt or misnamed.',

src/schema/tables.ts

@@ -257,9 +257,23 @@ export function evalColumns(
       rule: schemaPath,
     }
 
+    const ageCheck = name == 'age'
+    let ageWarningIssued = false
+
     const column = context.columns[name] as string[]
     for (const [index, value] of column.entries()) {
       if (!checkValue(value, spec)) {
+        if (ageCheck && value === '89+') {
+          if (!ageWarningIssued) {
+            ageWarningIssued = true
+            context.dataset.issues.add({
+              code: 'TSV_PSEUDO_AGE_DEPRECATED',
+              location: context.path,
+              line: index + 2,
+            })
+          }
+          continue
+        }
         const issueMessage = `'${value}'` +
           (value.match(/^\s*(NA|na|nan|NaN|n\/a)\s*$/) ? ", did you mean 'n/a'?" : '')
         context.dataset.issues.add({