feat: add eslint-plugin-jsx-a11y for accessibility linting (ASTD-198) (#431)

* feat: add eslint-plugin-jsx-a11y for accessibility linting (ASTD-198)

- Add eslint-plugin-jsx-a11y to pnpm catalog
- Add per-package a11y ESLint configs (eslint.config.a11y.js) for studio and common
- Add lint:a11y npm script to both packages with warning baselines
  - studio: 73 warnings
  - common: 18 warnings
- All jsx-a11y rules run at warn level initially

Signed-off-by: mschwab <mschwab@nvidia.com>

* refactor(studio): consolidate a11y eslint config to web root

The per-package eslint.config.a11y.js files in common and studio were
byte-identical. Move to a single web/eslint.config.a11y.js and point both
lint:a11y scripts at it (../../eslint.config.a11y.js), keeping per-package
--max-warnings. Move eslint-plugin-jsx-a11y and globals to the root
package.json since the shared config resolves them from there.

Resolves CodeRabbit finding about missing globals dependency in
web/packages/common/package.json by eliminating the duplicate config.

Signed-off-by: mschwab <mschwab@nvidia.com>

---------

Signed-off-by: mschwab <mschwab@nvidia.com>

Author: marcusds

web/eslint.config.a11y.js

@@ -0,0 +1,29 @@
+// SPDX-FileCopyrightText: Copyright (c) 2025-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+// SPDX-License-Identifier: Apache-2.0
+
+import tsEslintParser from '@typescript-eslint/parser';
+import jsxA11yPlugin from 'eslint-plugin-jsx-a11y';
+import globals from 'globals';
+
+export default [
+  {
+    files: ['**/*.{ts,tsx}'],
+    languageOptions: {
+      ecmaVersion: 2022,
+      sourceType: 'module',
+      parser: tsEslintParser,
+      globals: {
+        ...globals.browser,
+        ...globals.node,
+      },
+    },
+    plugins: {
+      'jsx-a11y': jsxA11yPlugin,
+    },
+    rules: {
+      ...Object.fromEntries(
+        Object.keys(jsxA11yPlugin.rules).map((rule) => [`jsx-a11y/${rule}`, 'warn'])
+      ),
+    },
+  },
+];

web/package.json

@@ -39,6 +39,7 @@
     "concurrently": "^10.0.3",
     "eslint": "10.2.1",
     "eslint-plugin-import-x": "^4.16.2",
+    "eslint-plugin-jsx-a11y": "catalog:",
     "eslint-plugin-no-only-tests": "^3.3.0",
     "eslint-plugin-react-hooks": "^5.2.0",
     "eslint-plugin-react-refresh": "^0.4.20",

web/packages/common/package.json

@@ -10,6 +10,7 @@
     "format:fix": "prettier --write './**/*.{js,jsx,ts,tsx}'",
     "lint": "eslint . --report-unused-disable-directives --max-warnings 0",
     "lint:fix": "eslint . --fix --report-unused-disable-directives --max-warnings 0",
+    "lint:a11y": "eslint . --config ../../eslint.config.a11y.js --no-config-lookup --no-inline-config --max-warnings 18",
     "test": "vitest --run",
     "test:ci": "vitest run --coverage",
     "test:watch": "vitest watch",

web/packages/studio/package.json

@@ -17,6 +17,7 @@
     "dev:preview": "pnpm build:dev && vite preview",
     "lint": "eslint . --report-unused-disable-directives --max-warnings 0",
     "lint:fix": "eslint . --fix --report-unused-disable-directives --max-warnings 0",
+    "lint:a11y": "eslint . --config ../../eslint.config.a11y.js --no-config-lookup --no-inline-config --max-warnings 73",
     "format": "prettier --check './**/*.{js,jsx,ts,tsx}'",
     "format:fix": "prettier --write './**/*.{js,jsx,ts,tsx}'",
     "preview": "vite preview",
@@ -111,7 +112,6 @@
     "@vitest/coverage-v8": "catalog:",
     "@vitest/ui": "catalog:",
     "blob-polyfill": "^9.0.20240710",
-    "globals": "^15.11.0",
     "happy-dom": "catalog:",
     "js-yaml": "^4.2.0",
     "jsdom": "catalog:",