Chore release 8.6.0 into release

.cursorignore

@@ -0,0 +1,104 @@
+# Add directories or file patterns to ignore during indexing (e.g. foo/ or *.csv)
+
+# Generated files for local API server config paths
+api/.env
+api/index.json
+.opentrons_config
+
+# MyPy caches
+.mypy_cache
+
+
+build/
+!app-shell/build
+api/docs/build/
+develop-eggs/
+dist/
+downloads/
+eggs/
+.eggs/
+lib64/
+parts/
+sdist/
+var/
+*.egg-info/
+.installed.cfg
+*.egg
+
+# PyInstaller
+pyinstaller/build
+pyinstaller/dist
+
+# Unit test / coverage reports
+htmlcov/
+.tox/
+coverage
+.coverage
+.coverage.*
+.cache
+nosetests.xml
+coverage.xml
+*,cover
+.hypothesis/
+
+# Translations
+*.mo
+*.pot
+
+# Django stuff:
+*.log
+
+# Sphinx documentation
+docs/_build/
+docs/build/
+
+# mkdocs documentation
+docs/*/site/
+
+# PyBuilder
+target/
+.idea/
+.setup_litter
+
+
+*.DS_Store
+
+
+# ignore linux swap files
+*.swp
+
+# local VERSION.json file when pushing to Flex
+*new_version_file.json
+
+# include archived API docs
+!api/docs/dist/
+api/docs/dist/hardware/
+api/docs/dist/v2/
+
+
+# local IDE configs
+.vscode
+*.code-workspace
+*.sublime-project
+*.sublime-workspace
+
+# build artifacts
+api-client/dist/
+app/dist/
+app-shell/dist/
+app-shell-odd/dist/
+components/lib
+discovery-client/lib
+labware-designer/dist/
+labware-library/dist/
+opentrons-ai-client/dist/
+protocol-designer/dist/
+react-api-client/dist/
+shared-data/lib/
+step-generation/lib/
+
+# docker
+Dockerfile
+DOCKER.md
+docker-compose.yml
+.dockerignore

.eslintignore

@@ -38,6 +38,13 @@ analyses-snapshot-testing/files
 # don't format the snapshots
 analyses-snapshot-testing/tests/__snapshots__
 analyses-snapshot-testing/analysis_results
+analyses-snapshot-testing/chunks
 
 opentrons-ai-server/package
-opentrons-ai-server/api/storage/index/
+opentrons-ai-server/api/storage/
+
+# Leave Markdown formatting alone because some of it is load-bearing for MkDocs.
+# Note that this is a Prettier thing, not an ESLint thing. Prettier shares this config file.
+/docs/**/*.md
+# And ignore non-semantic formatting issues in mkdocs YAML configuration files.
+/docs/**/*.yml

.eslintrc.js

@@ -16,7 +16,14 @@ module.exports = {
     'plugin:react/jsx-runtime',
   ],
 
-  plugins: ['react', 'react-hooks', 'json', 'testing-library', 'opentrons'],
+  plugins: [
+    'react',
+    'react-hooks',
+    'json',
+    'testing-library',
+    'opentrons',
+    '@eslint-react',
+  ],
 
   rules: {
     camelcase: 'off',
@@ -120,9 +127,13 @@ module.exports = {
       },
     },
     {
-      files: ['./app/src/**/*.@(ts|tsx)'],
+      files: [
+        './app/src/**/*.@(ts|tsx)',
+        './opentrons-ai-client/src/**/*.@(ts|tsx)',
+      ],
       rules: {
         'import/no-absolute-path': 'off',
+        '@eslint-react/no-nested-component-definitions': 'error',
       },
     },
     {
@@ -183,6 +194,7 @@ module.exports = {
         'opentrons/no-imports-up-the-tree-of-life': 'warn',
         'opentrons/no-margins-in-css': 'warn',
         'opentrons/no-margins-inline': 'warn',
+        '@eslint-react/no-nested-component-definitions': 'error',
       },
     },
     // apply application structure import requirements to app
@@ -209,5 +221,24 @@ module.exports = {
         'opentrons/no-margins-inline': 'warn',
       },
     },
+    {
+      files: ['**/*.tsx'],
+      excludedFiles: ['**/*.stories.tsx'],
+      rules: {
+        // TODO: Switch this rule to 'error' once the CSS modules migration is complete.
+        'react/forbid-dom-props': [
+          'warn',
+          {
+            forbid: [
+              {
+                propName: 'style',
+                message:
+                  'Inline styles are not allowed. Use CSS modules instead.',
+              },
+            ],
+          },
+        ],
+      },
+    },
   ],
 }

.github/CODEOWNERS

@@ -1,19 +1,16 @@
-# catchall: someone in software should be notified for any change in this repo
-* @Opentrons/software
+# Add teams here to be notified automatically when a PR changes certain file paths,
+# but be mindful of the rate of false positives -- we want to avoid notification spam.
 
-# language expertise verticals - if a PR changes things in multiple projects let's
-# notify those language communities
-*.js @Opentrons/js
-*.py @Opentrons/py
-*.d.ts @Opentrons/js
-/webpack-config @Opentrons/js
+/components @Opentrons/components-devs
 
-# subprojects by language - some subprojects are shared by teams but united by a
-# language community (including makefiles and config) so mark them appropriately
-/app @Opentrons/js
-/api-client @Opentrons/js
-/react-api-client @Opentrons/js
-/app-shell @Opentrons/js
-/components @Opentrons/js @Opentrons/components-devs
-/api @Opentrons/py
+# Changes to nonspecific files in shared-data are probably probably changes to the
+# actual data (e.g. deck definitions) and thus affect everyone, so notify everyone.
+# This needs to come first so it can be overridden by more specific rules.
+/shared-data @Opentrons/js @Opentrons/py
+# If a file is definitely JS or Python, notify just that language's community.
+# If this happens to all the files in the PR, it will override the "notify everyone" rule above.
 /shared-data/js @Opentrons/js
+/shared-data/**/*.ts @Opentrons/js
+/shared-data/python @Opentrons/py
+/shared-data/python_tests @Opentrons/py
+/shared-data/**/*.py @Opentrons/py

.github/actions/js/setup/action.yml

@@ -17,6 +17,13 @@ runs:
         node-version: '22.11.0'
         cache: 'yarn'
 
+    - name: 'Cache Cypress binary'
+      id: cache-cypress
+      uses: actions/cache@v4
+      with:
+        path: ~/.cache/Cypress
+        key: cypress-${{ runner.os }}-${{ hashFiles('**/yarn.lock') }}
+
     - name: 'Install udev for USB-detection'
       if: runner.os == 'Linux'
       shell: bash

.github/workflows/analyses-snapshot-lint.yaml

@@ -23,26 +23,18 @@ jobs:
     steps:
       - name: Checkout opentrons repo
         uses: 'actions/checkout@v4'
-
-      - name: Setup Python
-        uses: 'actions/setup-python@v5'
+      - name: Setup UV
+        uses: astral-sh/setup-uv@v6
         with:
-          python-version: '3.13.0'
-          cache: 'pipenv'
-          cache-dependency-path: analyses-snapshot-testing/Pipfile.lock
-      - name: Setup
-        id: install
-        working-directory: ./analyses-snapshot-testing
-        run: make setup
-      - name: black-check
-        if: always() && steps.install.outcome == 'success' || steps.install.outcome == 'skipped'
+          python-version: "3.10"
+          enable-cache: true
+          cache-dependency-glob: "./analyses-snapshot-testing/uv.lock"
+      - name: Sync the project
         working-directory: ./analyses-snapshot-testing
-        run: make black-check
-      - name: ruff
-        if: always() && steps.install.outcome == 'success' || steps.install.outcome == 'skipped'
+        run: make ci-setup
+      - name: ruff-check
         working-directory: ./analyses-snapshot-testing
         run: make ruff-check
-      - name: mypy
-        if: always() && steps.install.outcome == 'success' || steps.install.outcome == 'skipped'
+      - name: unit tests
         working-directory: ./analyses-snapshot-testing
-        run: make mypy
+        run: make unit-test