Merge branch 'master' into qctx

* master:
  Bump astral-sh/setup-uv from 6 to 7 (#4589)
  fix markdown in FAQ.md (#4584)
  fix typo in FAQ.md (#4585)
  Migrate from `react-test-renderer` to `@testing-library/react` (#4540)
  Convert @material-ui/icons to namespace imports (#4581)
  Fix `PackageRevision` GQL cache (#4583)

Author: nl0

.github/workflows/deploy-lambdas.yaml

@@ -30,7 +30,7 @@ jobs:
       contents: read
     steps:
       - uses: actions/checkout@v5
-      - uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v7
       - name: Export requirements.txt from uv.lock
         if: ${{ hashFiles(format('lambdas/{0}/uv.lock', matrix.path)) != '' }}
         run: |

.github/workflows/py-ci.yml

@@ -16,7 +16,7 @@ jobs:
         working-directory: api/python
     steps:
       - uses: actions/checkout@v5
-      - uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v7
       - name: Lint
         run: uv run poe lint
 
@@ -27,15 +27,15 @@ jobs:
         working-directory: api/python
     steps:
       - uses: actions/checkout@v5
-      - uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v7
       - name: Check formatting (quilt3)
         run: uv run poe fmt --check
 
   lint-rest:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v5
-      - uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v7
       - name: Run Ruff linter on entire codebase
         run: uvx ruff@0.12.11 check . --exclude catalog/ --exclude api/python/
 
@@ -44,7 +44,7 @@ jobs:
   #   runs-on: ubuntu-latest
   #   steps:
   #     - uses: actions/checkout@v5
-  #     - uses: astral-sh/setup-uv@v6
+  #     - uses: astral-sh/setup-uv@v7
   #     - name: Check formatting (rest of codebase)
   #       run: uvx ruff@0.12.11 format --check --exclude catalog/ --exclude api/python/
 
@@ -57,7 +57,7 @@ jobs:
       QUILT_DISABLE_USAGE_METRICS: true
     steps:
       - uses: actions/checkout@v5
-      - uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v7
       - name: Check generated docs are up-to-date
         run: uv run poe gendocs-check
 
@@ -70,7 +70,7 @@ jobs:
         working-directory: api/python
     steps:
       - uses: actions/checkout@v5
-      - uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v7
       - name: Test codeblocks
         run: uv run poe testdocs
 
@@ -87,7 +87,7 @@ jobs:
         working-directory: api/python
     steps:
       - uses: actions/checkout@v5
-      - uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v7
       - name: Run Pytest
         run: uv run --python ${{ matrix.python-version }} poe test-cov --cov-report xml
       - uses: codecov/codecov-action@v5
@@ -123,7 +123,7 @@ jobs:
         working-directory: api/python
     steps:
       - uses: actions/checkout@v5
-      - uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v7
       - name: verify git tag vs. version
         run: uv run poe verify-release-tag "${{ github.ref_name }}"
       - name: build
@@ -162,7 +162,7 @@ jobs:
       - uses: actions/setup-python@v6
         with:
           python-version-file: lambdas/${{ matrix.path }}/.python-version
-      - uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v7
       - name: Export requirements.txt from uv.lock
         if: ${{ hashFiles(format('lambdas/{0}/uv.lock', matrix.path)) != '' }}
         run: |

.github/workflows/test-quilt3-admin-codegen.yaml

@@ -24,6 +24,6 @@ jobs:
         working-directory: ./api/python
     steps:
       - uses: actions/checkout@v5
-      - uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v7
       - name: Check GraphQL client is up-to-date
         run: uv run poe gql-check

catalog/CHANGELOG.md

@@ -18,6 +18,8 @@ where verb is one of
 
 ## Changes
 
+- [Changed] Migrate unit-tests from `react-test-renderer` to `@testing-library/react` ([#4540](https://github.com/quiltdata/quilt/pull/4540))
+- [Changed] Request required fields for `PackageRevision` to proper cache ([#4583](https://github.com/quiltdata/quilt/pull/4583))
 - [Changed] Qurator: make tool messages less prominent ([#4572](https://github.com/quiltdata/quilt/pull/4572))
 - [Fixed] Qurator: Limit total search results contents context to 100k characters to avoid context window overflow ([#4573](https://github.com/quiltdata/quilt/pull/4573))
 - [Fixed] Empty directory upload on Bucket tab ([#4552](https://github.com/quiltdata/quilt/pull/4552))

catalog/app/components/Assistant/UI/Chat/DevTools.tsx

@@ -1,11 +1,7 @@
 import * as Eff from 'effect'
 import * as React from 'react'
 import * as M from '@material-ui/core'
-import {
-  Clear as ClearIcon,
-  Delete as DeleteIcon,
-  GetApp as GetAppIcon,
-} from '@material-ui/icons'
+import * as Icons from '@material-ui/icons'
 
 import JsonDisplay from 'components/JsonDisplay'
 
@@ -52,7 +48,7 @@ function ModelIdOverride({ value, setValue }: ModelIdOverrideProps) {
                 size="small"
               >
                 <M.Tooltip arrow title="Clear model ID override">
-                  <ClearIcon />
+                  <Icons.Clear />
                 </M.Tooltip>
               </M.IconButton>
             </M.InputAdornment>
@@ -123,15 +119,15 @@ function RecordingControls({ enabled, log, enable, clear }: RecordingControlsPro
             onClick={handleDownload}
             size="small"
             variant="outlined"
-            startIcon={<GetAppIcon />}
+            startIcon={<Icons.GetApp />}
           >
             Download Log
           </M.Button>
           <M.Button
             onClick={clear}
             size="small"
             variant="outlined"
-            startIcon={<DeleteIcon />}
+            startIcon={<Icons.Delete />}
           >
             Clear Log
           </M.Button>

catalog/app/components/BreadCrumbs/BreadCrumbs.spec.tsx

@@ -1,61 +1,55 @@
 import * as React from 'react'
-import renderer from 'react-test-renderer'
+import { render } from '@testing-library/react'
 import { MemoryRouter } from 'react-router-dom'
 
 import * as BreadCrumbs from './'
 
 describe('components/BreadCrumbs', () => {
   describe('Segment', () => {
     it('should render EMPTY', () => {
-      const tree = renderer.create(<BreadCrumbs.Segment />).toJSON()
-      expect(tree).toMatchSnapshot()
+      const { container } = render(<BreadCrumbs.Segment />)
+      expect(container).toMatchSnapshot()
     })
     it('should render label', () => {
-      const tree = renderer.create(<BreadCrumbs.Segment label="Lorem ipsum" />).toJSON()
-      expect(tree).toMatchSnapshot()
+      const { container } = render(<BreadCrumbs.Segment label="Lorem ipsum" />)
+      expect(container).toMatchSnapshot()
     })
     it('should render link', () => {
-      const tree = renderer
-        .create(
-          <MemoryRouter>
-            <BreadCrumbs.Segment label="A" to="/a" />
-          </MemoryRouter>,
-        )
-        .toJSON()
-      expect(tree).toMatchSnapshot()
+      const { container } = render(
+        <MemoryRouter>
+          <BreadCrumbs.Segment label="A" to="/a" />
+        </MemoryRouter>,
+      )
+      expect(container).toMatchSnapshot()
     })
   })
   describe('render', () => {
     it('basic breadcrumbs', () => {
       const crumbs = BreadCrumbs.getCrumbs('aa a/bb-b/c/d_d', (x) => x, 'ROOT')
-      const tree = renderer
-        .create(<MemoryRouter>{BreadCrumbs.render(crumbs)}</MemoryRouter>)
-        .toJSON()
-      expect(tree).toMatchSnapshot()
+      const { container } = render(
+        <MemoryRouter>{BreadCrumbs.render(crumbs)}</MemoryRouter>,
+      )
+      expect(container).toMatchSnapshot()
     })
     it('and transform links', () => {
       const crumbs = BreadCrumbs.getCrumbs('a/b', (x) => x, 'ROOT')
       const getLinkProps = ({ to }: { to?: string }) => ({
         href: `https://quiltdata.com/${to || ''}`,
       })
-      const tree = renderer
-        .create(
-          <MemoryRouter>{BreadCrumbs.render(crumbs, { getLinkProps })}</MemoryRouter>,
-        )
-        .toJSON()
-      expect(tree).toMatchSnapshot()
+      const { container } = render(
+        <MemoryRouter>{BreadCrumbs.render(crumbs, { getLinkProps })}</MemoryRouter>,
+      )
+      expect(container).toMatchSnapshot()
     })
     it('and make links for every crumb', () => {
       const crumbs = BreadCrumbs.getCrumbs('a/b', (x) => x, 'ROOT', { tailLink: true })
       const getLinkProps = ({ to }: { to?: string }) => ({
         href: `https://quiltdata.com/${to || ''}`,
       })
-      const tree = renderer
-        .create(
-          <MemoryRouter>{BreadCrumbs.render(crumbs, { getLinkProps })}</MemoryRouter>,
-        )
-        .toJSON()
-      expect(tree).toMatchSnapshot()
+      const { container } = render(
+        <MemoryRouter>{BreadCrumbs.render(crumbs, { getLinkProps })}</MemoryRouter>,
+      )
+      expect(container).toMatchSnapshot()
     })
     it('and end with separator', () => {
       const crumbs = BreadCrumbs.getCrumbs('a/b', (x) => x, 'ROOT', {
@@ -65,19 +59,17 @@ describe('components/BreadCrumbs', () => {
       const getLinkProps = ({ to }: { to?: string }) => ({
         href: `https://quiltdata.com/${to || ''}`,
       })
-      const tree = renderer
-        .create(
-          <MemoryRouter>{BreadCrumbs.render(crumbs, { getLinkProps })}</MemoryRouter>,
-        )
-        .toJSON()
-      expect(tree).toMatchSnapshot()
+      const { container } = render(
+        <MemoryRouter>{BreadCrumbs.render(crumbs, { getLinkProps })}</MemoryRouter>,
+      )
+      expect(container).toMatchSnapshot()
     })
     it('without root label', () => {
       const crumbs = BreadCrumbs.getCrumbs('a/b/c', (x) => x)
-      const tree = renderer
-        .create(<MemoryRouter>{BreadCrumbs.render(crumbs)}</MemoryRouter>)
-        .toJSON()
-      expect(tree).toMatchSnapshot()
+      const { container } = render(
+        <MemoryRouter>{BreadCrumbs.render(crumbs)}</MemoryRouter>,
+      )
+      expect(container).toMatchSnapshot()
     })
   })
   it('copyWithoutSpaces', () => {