frontend: update the UI registry past 3.4.1 - #2659
Merged
Merged
Conversation
The vendored copies under `redpanda-ui/` had fallen behind the deployed registry: 137 of 166 files differed, none of them cosmetically. Pulled by writing the published bytes for all 92 installed items, then asserting byte-parity for every target. No item carries a `cssVars`, `css` or `tailwind` payload and no target falls outside the vendored directory, so the direct write is equivalent to a CLI pull. `bun run lint` leaves the result at parity. Two items arrive with it. `sanitized-html` is new in the closure, required by `code-block`, `code-tabs` and `code-block-dynamic`, all of which are installed. `auto-form/renderers/renderer-types.ts` is a new file in the auto-form item. Dependencies follow the registry where console was behind it: `@base-ui/react` to ^1.8.0, `@tanstack/react-table` to ^9.2.3 and `zod` to ^4.6.1. `lottie-web` is promoted to a direct dependency because `loading-animation.tsx` imports it and it was only reaching us through `lottie-react`. `dompurify`, which `sanitized-html` needs, was already pinned at the range it asks for. `Link` now requires `href`, so the two react-markdown anchor renderers render a destinationless anchor as text instead of a dead link. `style/theme.css` changes are rewrapped shadow values and one declaration reorder. All 207 custom properties are unchanged, so the token contract with cloud-ui is untouched and this needs no paired change there. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Contributor
✅ Clean — no registry drift, off-token colours, or ad-hoc classesApp:
Generated by lookout audit-changes. |
r-vasquez
approved these changes
Sep 22, 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
The vendored registry under
redpanda-ui/had fallen behind the deployed registry.137of166files differed, and none of it was cosmetic: upstream landed theimport type Reactpass, refactoreduseIsMobileontouseMediaQuery, and reworked combobox, multi-select and motion-highlight substantially.Pulled by fetching the per-item payloads for all
92installed items and writing the published bytes, then asserting byte-parity for every target. No item carries acssVars,cssortailwindpayload, and no target falls outside the vendored directory, so a direct write is equivalent to a CLI pull and is auditable in a wayshadcn add --overwriteis not.bun run lintleaves the tree at parity, checked against a pre-lint snapshot.What comes with it
sanitized-htmlis new in the dependency closure.code-block,code-tabsandcode-block-dynamicall require it now and all three are installed, so without it they break.auto-form/renderers/renderer-types.tsis a new file in the auto-form item.highlighted-input.tsx,listbox.tsx,stat/index.tsx, and thesecret-selector-scopes.test.tsguard test we keep because the registry ships that file untested.Dependencies
Console was behind the registry on three ranges, so they move to what it asks for:
@base-ui/react^1.7.0^1.8.0@tanstack/react-table^9.1.2^9.2.3zod^4.4.3^4.6.1lottie-webbecomes a direct dependency at^5.13.0.loading-animation.tsximports it directly and it was only reaching us throughlottie-react, so it was a phantom dependency that happened to resolve.dompurify, whichsanitized-htmlneeds, was already pinned at exactly the range it asks for. Both lockfiles are regenerated with the pinned bun (1.4.0).Ranges where console is ahead of the registry are left alone:
@bufbuild/protobufand@tanstack/react-router.Callsite changes
Linknow requireshref. The two react-markdown anchor renderers passedstring | undefined, since markdown can produce an anchor with no destination, so they render that case as text rather than a dead link.Theming
style/theme.csslooks noisy but is semantically identical: rewrapped--shadow-*values and onefont-weight/line-heightreorder. All207custom properties are unchanged in both name and value, so the token contract with cloud-ui is untouched and this does not need a paired change there.Testing
type:check,lint,build, and both test suites at1063unit and1602integration, all matching master's baseline exactly, so the pull is test-neutral. The only type errors the upgrade produced were the twoLinkcallsites above.Reviewers: the registry files are vendored upstream content, so the diff worth reading is
package.json, the two callsites, and the fact that byte-parity is asserted rather than eyeballed.🤖 Generated with Claude Code