From 5fb406b450939e04fc128e1fc670e179ee74f944 Mon Sep 17 00:00:00 2001 From: Dustin Healy <54083382+dustinhealy@users.noreply.github.com> Date: Tue, 23 Jun 2026 19:37:17 -0700 Subject: [PATCH] fix(ci): update UIResourceCarousel dimensions test and sort Mention.tsx imports The carousel card container was changed to use dynamic height (defaulting to 360px) rather than a fixed minHeight. The test expected minHeight which no longer exists. Also removes the animationDelay assertion which jsdom does not verify via toHaveStyle. Mention.tsx had a pre-existing import-sort drift that the CI sort-check detected; sorted to clear the gate. --- client/src/components/Chat/Input/Mention.tsx | 2 +- .../Messages/Content/__tests__/UIResourceCarousel.test.tsx | 5 ++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/client/src/components/Chat/Input/Mention.tsx b/client/src/components/Chat/Input/Mention.tsx index c2d352a4e6..dd2d33d11b 100644 --- a/client/src/components/Chat/Input/Mention.tsx +++ b/client/src/components/Chat/Input/Mention.tsx @@ -1,8 +1,8 @@ import { memo, useState, useRef, useEffect } from 'react'; -import { useRecoilValue, useSetRecoilState } from 'recoil'; import { AutoSizer, List } from 'react-virtualized'; import { Spinner, useCombobox } from '@librechat/client'; import { EModelEndpoint } from 'librechat-data-provider'; +import { useRecoilValue, useSetRecoilState } from 'recoil'; import type { RecoilState } from 'recoil'; import type { MentionOption, ConvoGenerator } from '~/common'; import { useGetConversation, useLocalize, TranslationKeys } from '~/hooks'; diff --git a/client/src/components/Chat/Messages/Content/__tests__/UIResourceCarousel.test.tsx b/client/src/components/Chat/Messages/Content/__tests__/UIResourceCarousel.test.tsx index a593a03ba3..d26b7f85ff 100644 --- a/client/src/components/Chat/Messages/Content/__tests__/UIResourceCarousel.test.tsx +++ b/client/src/components/Chat/Messages/Content/__tests__/UIResourceCarousel.test.tsx @@ -133,12 +133,11 @@ describe('UIResourceCarousel', () => { it('applies correct dimensions to resource containers', () => { const { container } = render(); const iframes = container.querySelectorAll('iframe[data-sandbox-url]'); - iframes.forEach((iframe, index) => { + iframes.forEach((iframe) => { const card = iframe.parentElement?.parentElement; expect(card).toHaveStyle({ width: '230px', - minHeight: '360px', - animationDelay: `${index * 100}ms`, + height: '360px', }); }); });