Replace eslint with oxlint

- replace eslint with oxlint
- add vscode suggested extensions and settings
- replace npm test:plus w/ npm check

Author: cmoesel

.github/pull_request_template.md

@@ -9,7 +9,7 @@ For items that are not-applicable, mark "N/A" and ✔.
 - [ ] Tests are included and test edge cases
 - [ ] Tests have been run locally and pass
 - [ ] Code coverage has not gone down and all code touched or added is covered.
-- [ ] Code passes lint and prettier (hint: use `npm run test:plus` to run tests, lint, and prettier)
+- [ ] Code passes lint and prettier (hint: use `npm run check` to run tests, lint, and prettier)
 - [ ] All dependent libraries are appropriately updated or have a corresponding PR related to this change
 - [ ] `cql4browsers.js` built with `npm run build:browserify` if source changed.
 

.gitignore

@@ -3,7 +3,9 @@ lib-test
 node_modules
 .DS_Store
 .nyc_output
-.vscode
+.vscode/*
+!.vscode/extensions.json
+!.vscode/settings.sample.json
 coverage
 *.original.coffee
 yarn-error.log

.oxlintrc.json

@@ -0,0 +1,48 @@
+{
+  "$schema": "./node_modules/oxlint/configuration_schema.json",
+  "env": {
+    "builtin": true,
+    "es6": true,
+    "mocha": true,
+    "node": true
+  },
+  "ignorePatterns": [
+    "**/lib",
+    "**/lib-test",
+    "test-server/**",
+    "examples/browser/cql4browsers.js"
+  ],
+  "rules": {
+    "no-loss-of-precision": "off",
+    "no-unused-vars": [
+      "error",
+      {
+        "argsIgnorePattern": "^_"
+      }
+    ],
+    "no-var": "error",
+    "prefer-const": "error",
+    "prefer-rest-params": "error",
+    "prefer-spread": "error",
+    "curly": "error",
+    "no-console": [
+      "warn",
+      {
+        "allow": [
+          "warn",
+          "error"
+        ]
+      }
+    ]
+  },
+  "overrides": [
+    {
+      "files": [
+        "test/elm/**/*.js"
+      ],
+      "rules": {
+        "unicorn/no-empty-file": "off"
+      }
+    }
+  ]
+}

.vscode/extensions.json

@@ -0,0 +1,7 @@
+{
+  "recommendations": [
+    // recommended for lint and code formatting
+    "oxc.oxc-vscode",
+    "esbenp.prettier-vscode"
+  ]
+}

.vscode/settings.sample.json

@@ -0,0 +1,10 @@
+{
+  "[javascript][typescript][typescriptreact]": {
+    "editor.formatOnSave": true,
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
+  },
+  "[json]": {
+    "editor.formatOnSave": true,
+    "editor.defaultFormatter": "vscode.json-language-features"
+  }
+}

DEPENDENCY_NOTES.md

@@ -3,5 +3,5 @@ The following notes about the app's dependencies should be revisited and resolve
 As of 2026-07-27:
 
 - `@types/node`: Consider updating to `26.x` in the future, but for now `24.x` still has plent of time in LTS.
-- `typescript`: `7.x` uses a new native compiler without a native API. Libraries like `ts-node` and `typescript-eslint` do not yet support it.
+- `typescript`: `7.x` uses a new native compiler without a native API. Libraries like `ts-node` do not yet support it.
 - `tslog` (test-server only): `5.x` is ESM-only and replaces v4's flat logger settings such as `hideLogPositionForProduction` and `prettyLogTemplate`. Updating will require some migration.

bin/browserify.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-/* eslint-disable no-console */
+/* oxlint-disable no-console */
 
 // We cannot use browserify CLI because there is no way to pass options into presets
 

eslint.config.mjs

@@ -560,7 +560,6 @@ __exportStar(require("./ratio"), exports);
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.MAX_TIME_VALUE = exports.MIN_TIME_VALUE = exports.MAX_DATE_VALUE = exports.MIN_DATE_VALUE = exports.MAX_DATETIME_VALUE = exports.MIN_DATETIME_VALUE = exports.Date = exports.DateTime = void 0;
-/* eslint-disable @typescript-eslint/ban-ts-comment */
 const uncertainty_1 = require("./uncertainty");
 const util_1 = require("../util/util");
 const luxon_1 = require("luxon");
@@ -1245,7 +1244,7 @@ class DateTime extends AbstractDate {
         // consider seconds and milliseconds as a single combined precision with decimal semantics
         // this means that if milliseconds are not specified, then we treat it as though their
         // millisecond value is "0" so that no Uncertainty will be produced
-        /* eslint-disable @typescript-eslint/no-this-alias */
+        /* oxlint-disable @typescript/no-this-alias */
         let aDateTime = this;
         let bDateTime = other;
         if (this.second !== null &&
@@ -2501,7 +2500,6 @@ class ThreeValuedLogic {
             return null;
         }
         else {
-            // eslint-disable-next-line @typescript-eslint/ban-ts-comment
             // @ts-ignore
             return val.reduce((a, b) => (!a ^ !b) === 1);
         }
@@ -3942,7 +3940,6 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.build = build;
-/* eslint-disable @typescript-eslint/ban-ts-comment */
 const E = __importStar(require("./expressions"));
 const util_1 = require("../util/util");
 function build(json) {
@@ -4389,6 +4386,7 @@ exports.If = If;
 class CaseItem {
     constructor(json) {
         this.when = (0, builder_1.build)(json.when);
+        // oxlint-disable-next-line unicorn/no-thenable -- CaseItem mirrors ELM's `then` field.
         this.then = (0, builder_1.build)(json.then);
     }
 }
@@ -4467,7 +4465,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.DurationBetween = exports.DifferenceBetween = exports.TimezoneOffsetFrom = exports.TimeFrom = exports.DateFrom = exports.DateTimeComponentFrom = exports.TimeOfDay = exports.Now = exports.Today = exports.Time = exports.Date = exports.DateTime = void 0;
 exports.doAfter = doAfter;
 exports.doBefore = doBefore;
-/* eslint-disable @typescript-eslint/ban-ts-comment */
 const expression_1 = require("./expression");
 const builder_1 = require("./builder");
 const literal_1 = require("./literal");
@@ -6559,7 +6556,6 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.Precision = exports.SameOrBefore = exports.SameOrAfter = exports.SameAs = exports.Before = exports.After = exports.Length = exports.ProperContains = exports.ProperIn = exports.ProperIncludedIn = exports.ProperIncludes = exports.IncludedIn = exports.Includes = exports.Contains = exports.In = exports.Indexer = exports.Intersect = exports.Except = exports.Union = exports.NotEqual = exports.Equivalent = exports.Equal = void 0;
-/* eslint-disable @typescript-eslint/ban-ts-comment */
 const expression_1 = require("./expression");
 const logic_1 = require("../datatypes/logic");
 const datetime_1 = require("../datatypes/datetime");
@@ -7121,7 +7117,6 @@ class ByDirection extends expression_1.Expression {
         this.low_order = this.direction === 'asc' || this.direction === 'ascending' ? -1 : 1;
         this.high_order = this.low_order * -1;
     }
-    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-ignore
     async exec(ctx, a, b) {
         if (a === b) {
@@ -7152,7 +7147,6 @@ class ByExpression extends expression_1.Expression {
         this.low_order = this.direction === 'asc' || this.direction === 'ascending' ? -1 : 1;
         this.high_order = this.low_order * -1;
     }
-    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-ignore
     async exec(ctx, a, b) {
         let sctx = ctx.childContext(a);
@@ -8593,7 +8587,7 @@ function guessSpecifierType(val) {
     else if (Array.isArray(val)) {
         // Get unique types from the array (by converting to string and putting in a Set)
         const typesAsStrings = Array.from(new Set(val.map(v => JSON.stringify(guessSpecifierType(v)))));
-        const types = typesAsStrings.map(ts => (/^{/.test(ts) ? JSON.parse(ts) : ts));
+        const types = typesAsStrings.map(ts => (ts.startsWith('{') ? JSON.parse(ts) : ts));
         return {
             type: elmTypes_1.ELM_LIST_TYPE_SPECIFIER,
             elementType: types.length == 1 ? types[0] : { type: elmTypes_1.ELM_CHOICE_TYPE_SPECIFIER, choice: types }
@@ -9181,7 +9175,7 @@ class ConsoleMessageListener {
         this.logSourceOnTrace = logSourceOnTrace;
     }
     onMessage(source, code, severity, message) {
-        // eslint-disable-next-line no-console
+        // oxlint-disable-next-line no-console
         const print = severity === 'Error' ? console.error : console.log;
         let content = `${severity}: [${code}] ${message}`;
         if (severity === 'Trace' && this.logSourceOnTrace) {

examples/browser/cql4browsers.js

@@ -1,6 +1,5 @@
-/* eslint-disable
-    no-console,
-*/
+/* oxlint-disable no-console */
+
 const cql = require('../../lib/cql');
 const measure = require('./age.json');