chore: upgrade @databiosphere/findable-ui to v53.1.0 (#4843)#4844
Merged
Conversation
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Upgrades the application’s @databiosphere/findable-ui dependency to ^53.1.0 and migrates usage from the removed Markdown component to MarkdownRenderer, including the associated prop contract change (content → value).
Changes:
- Bump
@databiosphere/findable-uifrom^52.2.0to^53.1.0and removeisomorphic-dompurifyfrom dependencies. - Update the app components barrel export to re-export
MarkdownRendererinstead ofMarkdown. - Update AnVIL Catalog study detail configuration + view model builder to use
MarkdownRendererand return{ value: ... }.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Upgrades findable-ui and removes isomorphic-dompurify dependency. |
| package-lock.json | Locks updated dependency graph for findable-ui upgrade and removed dompurify-related packages. |
| app/components/index.tsx | Removes Markdown re-export and adds MarkdownRenderer re-export. |
| app/viewModelBuilders/catalog/anvil-catalog/common/viewModelBuilders.ts | Updates buildStudyDescription typing and props shape to match MarkdownRenderer. |
| site-config/anvil-catalog/dev/detail/study/overviewMainColumn.ts | Switches study description component config from Markdown to MarkdownRenderer. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
NoopDog
approved these changes
May 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Upgrades
@databiosphere/findable-uifrom52.2.0→^53.1.0and adopts the v53.0.0 breaking changes.Closes #4843.
Breaking changes from v53.0.0
Markdowncomponent andisomorphic-dompurifypeer dependency. Consumers should migrate toMarkdownRenderer(unified/remark/rehype-react, sanitised at the AST stage viarehype-sanitize).Changes in this PR
package.json—@databiosphere/findable-ui: ^53.1.0; removedisomorphic-dompurify(no longer a findable-ui peer, and no direct usages in this repo).app/components/index.tsx— replaced theMarkdownbarrel re-export withMarkdownRenderer.app/viewModelBuilders/catalog/anvil-catalog/common/viewModelBuilders.ts—buildStudyDescriptionnow returns{ value }instead of{ content }and is typed againsttypeof C.MarkdownRenderer.site-config/anvil-catalog/dev/detail/study/overviewMainColumn.ts—C.Markdown→C.MarkdownRendererin the component config.Peer-dependency audit
All other findable-ui peer dependencies (emotion, MUI v7, tanstack, ky, next, react 19, rehype/remark/unified, yup, etc.) are already satisfied.
next-authis optional in findable-ui and not used here.react-gtm-module(removed as a findable-ui peer in v52.0.0) stays in this repo'sdependenciesbecausepages/_app.tsximports it directly.Test plan
npx tsc --noEmitnpm run check-formatnpm run lintnpm run build:anvil-cmgnpm run build-dev:anvil-catalognpm run test:anvil-cmg(re-ran after intermittent firefox-only timeouts on first attempt; unrelated to this change and consistent with the known flaky firefox issue tracked in e2e: anvil-dataset "displays download file manifest" times out on firefox #4840)MarkdownRendereron anvil-catalog study detail page — confirmed renders correctlyOut of scope
The collapsible Terra info work (#4816, branch
fran/4816-collapsable-terra-info) depends on findable-ui v53 features. After this PR merges, that branch should rebase ontomain.🤖 Generated with Claude Code