@medplum/eslint-config 4.5.1 → 5.0.0

package/{index.cjs → index.mjs}

@@ -1,15 +1,37 @@
-module.exports = {
-  parser: '@typescript-eslint/parser',
-  plugins: ['@typescript-eslint', 'header', 'react-refresh', 'no-only-tests'],
+// SPDX-FileCopyrightText: Copyright Orangebot, Inc. and Medplum contributors
+// SPDX-License-Identifier: Apache-2.0
+
+import eslint from '@eslint/js';
+import headerPlugin from 'eslint-plugin-header';
+import importPlugin from 'eslint-plugin-import';
+import jsdoc from 'eslint-plugin-jsdoc';
+import noOnlyTestsPlugin from 'eslint-plugin-no-only-tests';
+import reactHooks from 'eslint-plugin-react-hooks';
+import reactRefresh from 'eslint-plugin-react-refresh';
+import tseslint from 'typescript-eslint';
+
+// Workaround for eslint-plugin-header ESLint 9 compatibility issue.
+// See: https://github.com/Stuk/eslint-plugin-header/issues/57#issuecomment-2378485611
+headerPlugin.rules.header.meta.schema = false;
+
+/**
+ * Core config applies to all source files.
+ * TypeScript-specific rules are in the tsConfig below.
+ * @type {import("@eslint/config-helpers").ConfigWithExtends}
+ */
+export const coreConfig = {
   extends: [
-    'eslint:recommended',
-    'plugin:import/recommended',
-    'plugin:@typescript-eslint/recommended',
-    'plugin:@typescript-eslint/strict',
-    'plugin:jsdoc/recommended-typescript-error',
-    'plugin:react-hooks/recommended',
+    eslint.configs.recommended,
+    importPlugin.flatConfigs.recommended,
+    jsdoc.configs['flat/recommended-typescript-error'],
   ],
-  reportUnusedDisableDirectives: true,
+  languageOptions: {
+    ecmaVersion: 'latest',
+    sourceType: 'module',
+  },
+  plugins: {
+    header: headerPlugin,
+  },
   rules: {
     // ESLint - Possible Problems
     'array-callback-return': 'error',
@@ -66,6 +88,64 @@ module.exports = {
     'prefer-spread': 'error',
     radix: 'error',
 
+    'no-unused-vars': [
+      'error',
+      {
+        argsIgnorePattern: '^_',
+        caughtErrorsIgnorePattern: '^_',
+      },
+    ],
+
+    // JSDoc
+    'jsdoc/check-tag-names': [
+      'error',
+      {
+        definedTags: ['category', 'experimental', 'ts-ignore'],
+      },
+    ],
+    'jsdoc/require-hyphen-before-param-description': ['error', 'always'],
+    'jsdoc/require-jsdoc': 'off',
+    'jsdoc/tag-lines': 'off',
+    'jsdoc/require-yields-type': 'off',
+    'jsdoc/require-throws-type': 'off',
+
+    // Header
+    'header/header': [
+      'error',
+      'line',
+      [
+        ' SPDX-FileCopyrightText: Copyright Orangebot, Inc. and Medplum contributors',
+        ' SPDX-License-Identifier: Apache-2.0',
+      ],
+    ],
+  },
+};
+
+/**
+ * TypeScript config applies to all TypeScript source files.
+ * These are TypeScript-specific rules that don't apply to JavaScript files.
+ * This improves accuracy and performance.
+ * @type {import("@eslint/config-helpers").ConfigWithExtends}
+ */
+export const tsConfig = {
+  files: ['**/*.ts', '**/*.tsx'],
+  extends: [
+    tseslint.configs.recommended,
+    {
+      languageOptions: {
+        parserOptions: {
+          projectService: true,
+        },
+      },
+    },
+    tseslint.configs.strict,
+    reactHooks.configs.flat.recommended,
+    reactRefresh.configs.recommended,
+  ],
+  plugins: {
+    'no-only-tests': noOnlyTestsPlugin,
+  },
+  rules: {
     // TypeScript+ESLint
     '@typescript-eslint/array-type': 'error',
     '@typescript-eslint/consistent-generic-constructors': 'error',
@@ -127,6 +207,13 @@ module.exports = {
       },
     ],
 
+    // New strict `@typescript-eslint` rules
+    // Need to drop all use of `any` first
+    '@typescript-eslint/no-unsafe-assignment': 'off',
+    '@typescript-eslint/no-unsafe-member-access': 'off',
+    '@typescript-eslint/no-unsafe-return': 'off',
+    '@typescript-eslint/no-unsafe-argument': 'off',
+
     // imports rules for isolatedModules and verbatimModuleSyntax
     'no-duplicate-imports': 'off', // Disable base rule
     'import/consistent-type-specifier-style': ['error', 'prefer-top-level'],
@@ -163,48 +250,40 @@ module.exports = {
     '@typescript-eslint/no-dynamic-delete': 'off',
     '@typescript-eslint/no-empty-object-type': 'off',
 
-    // JSDoc
-    'jsdoc/check-tag-names': [
-      'error',
-      {
-        definedTags: ['category', 'experimental', 'ts-ignore'],
-      },
-    ],
-    'jsdoc/require-hyphen-before-param-description': ['error', 'always'],
-    'jsdoc/require-jsdoc': 'off',
-    'jsdoc/tag-lines': 'off',
-    'jsdoc/require-yields-type': 'off',
-    'jsdoc/require-throws-type': 'off',
-
-    // Header
-    'header/header': [
-      'error',
-      'line',
-      [
-        ' SPDX-FileCopyrightText: Copyright Orangebot, Inc. and Medplum contributors',
-        ' SPDX-License-Identifier: Apache-2.0',
-      ],
-    ],
-
     // No Only Tests
     'no-only-tests/no-only-tests': ['error', { fix: true }],
   },
-  ignorePatterns: [
-    'coverage',
-    '**/node_modules/',
-    '**/dist/',
-    'packages/docs/build/',
-    'packages/docs/markdown-to-mdx.cjs',
-    'packages/docs/docusaurus.config.js',
-    'packages/docs/sidebars.js',
-    'packages/eslint-config/index.cjs',
-    'packages/expo-medplum-polyfills/build',
-    'babel.config.js',
-    'babel.config.cjs',
-    'jest.sequencer.js',
-    'package-lock.json',
-    'postcss.config.cjs',
-    'rollup.config.mjs',
-    'webpack.config.js',
-  ],
 };
+
+/** @type {import("@eslint/config-helpers").ConfigWithExtendsArray} */
+export const medplumEslintConfig = [
+  {
+    ignores: [
+      '.turbo',
+      'coverage',
+      'dist',
+      'node_modules',
+      'packages/docs/build/',
+      'packages/docs/markdown-to-mdx.cjs',
+      'packages/docs/docusaurus.config.js',
+      'packages/docs/sidebars.js',
+      'packages/eslint-config/index.cjs',
+      'packages/expo-medplum-polyfills/build',
+      'packages/generator/output/',
+      'packages/react/.storybook/',
+      'package-lock.json',
+      '**/.turbo',
+      '**/coverage/',
+      '**/dist/',
+      '**/node_modules/',
+      '**/babel.config.*',
+      '**/jest.sequencer.*',
+      '**/postcss.config.*',
+      '**/rollup.config.*',
+      '**/webpack.config.*',
+      '**/*.png.ts',
+    ],
+  },
+  coreConfig,
+  tsConfig,
+];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@medplum/eslint-config",
-  "version": "4.5.1",
+  "version": "5.0.0",
   "description": "Shared ESLint configuration for Medplum projects",
   "keywords": [
     "eslint",
@@ -17,30 +17,29 @@
   },
   "license": "Apache-2.0",
   "author": "Medplum <hello@medplum.com>",
-  "main": "index.cjs",
+  "type": "module",
+  "main": "index.mjs",
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "8.46.0",
-    "@typescript-eslint/parser": "8.46.0",
-    "eslint": "8.57.1",
-    "eslint-plugin-jsdoc": "60.8.3",
+    "eslint": "9.38.0",
     "eslint-plugin-header": "3.1.1",
     "eslint-plugin-import": "2.32.0",
+    "eslint-plugin-jsdoc": "61.1.11",
     "eslint-plugin-no-only-tests": "3.3.0",
-    "eslint-plugin-react-hooks": "7.0.0",
-    "eslint-plugin-react-refresh": "0.4.23"
+    "eslint-plugin-react-hooks": "7.0.1",
+    "eslint-plugin-react-refresh": "0.4.24",
+    "typescript-eslint": "8.46.2"
   },
   "peerDependencies": {
-    "@typescript-eslint/eslint-plugin": "^8",
-    "@typescript-eslint/parser": "^8",
-    "eslint": "^8",
-    "eslint-plugin-jsdoc": "^50",
+    "eslint": "^9",
     "eslint-plugin-header": "^3",
     "eslint-plugin-import": "^2",
-    "eslint-plugin-no-only-tests": "^3.3.0",
-    "eslint-plugin-react-hooks": "^4",
-    "eslint-plugin-react-refresh": "^0"
+    "eslint-plugin-jsdoc": "^61",
+    "eslint-plugin-no-only-tests": "^3",
+    "eslint-plugin-react-hooks": "^7",
+    "eslint-plugin-react-refresh": "^0",
+    "typescript-eslint": "^8"
   },
   "engines": {
-    "node": "^20.19.0 || >=22.12.0"
+    "node": "^22.18.0 || >=24.2.0"
   }
 }