Closed kielbasa-elp closed 3 weeks ago
[!CAUTION]
Review failed
The pull request is closed.
The changes in this pull request encompass various components across the web-remix
application, primarily focusing on refactoring to standard functional component definitions that accept ref
as a prop instead of using React.forwardRef
. This transition enhances type safety and simplifies the code structure. Additionally, several components and hooks have been updated to utilize a new context access method, replacing React.useContext
with React.use
. Other modifications include improvements in error handling, response structures in API handlers, and updates to type declarations to allow for null
values.
File Path | Change Summary |
---|---|
apps/web-remix/app/components/audioRecorder/useAudioVisualize.tsx | Updated canvas parameter type to RefObject<HTMLCanvasElement | null> . Added error handling in visualizeAudio function. |
apps/web-remix/app/components/chat/ChatMarkdown.tsx | Updated children prop handling with type casting. Added error handling in the Code function. |
apps/web-remix/app/components/chat/ChatMentionList.tsx | Changed from forwardRef to functional component with ref in props. |
apps/web-remix/app/components/chat/voice/Voicechat.tsx | Updated audioRef and canvasRef types to allow null . Modified onBlockOutput function signature. |
apps/web-remix/app/components/datepicker/Datepicker.tsx | Updated DatepickerClient function signature to destructure children from props. |
apps/web-remix/app/components/dropdown/Dropdown.tsx | Replaced DropdownContext.Provider with DropdownContext . |
apps/web-remix/app/components/dropdown/DropdownContext.ts | Changed context access from useContext to use . |
apps/web-remix/app/components/form/fields/asyncSelect.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/checkbox.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/creatableAsyncSelect.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/editor.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/field.context.tsx | Updated context usage and simplified provider structure. |
apps/web-remix/app/components/form/fields/field.label.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/file.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/number.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/quantity.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/radio.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/select.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/text.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/fields/toggle.field.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/inputs/file.input.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/form/inputs/number.input.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/iconButton/index.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/modal/confirm.tsx | Added internal close function for modal lifecycle management. |
apps/web-remix/app/components/pages/knowledgeBase/collectionGraph/activeNodeProvider.tsx | Changed context access from useContext to use . |
apps/web-remix/app/components/pages/knowledgeBase/collectionGraph/components/NodePreviewSidebar.tsx | Changed context access from useContext to use . |
apps/web-remix/app/components/pages/knowledgeBase/collectionGraph/details/components/NodePreview.tsx | Removed SearchSchema import and updated searchParams type. |
apps/web-remix/app/components/pages/knowledgeBase/collectionOverview/CollectionCostsTable.tsx | Added null check for value in description column rendering. |
apps/web-remix/app/components/pages/knowledgeBase/components/ListActionProvider.tsx | Changed context access from ListActionContext.Provider to ListActionContext . |
apps/web-remix/app/components/pages/pipelines/EditBlockForm.tsx | Changed context provider usage and refined form submission logic. |
apps/web-remix/app/components/pages/pipelines/NodeDropdown/NodeDropdown.tsx | Changed from forwardRef to functional component. |
apps/web-remix/app/components/pages/pipelines/Nodes/CommentNodes/CommentEditor.tsx | Changed context access from WysiwygContext.Provider to WysiwygContext . |
apps/web-remix/app/components/pages/pipelines/RunPipelineProvider.tsx | Changed context access from useContext to use . |
apps/web-remix/app/components/pages/pipelines/new/page.tsx | Added formRef prop to ValidatedForm for programmatic submission. |
apps/web-remix/app/components/pages/pipelines/useCopyPasteNode.tsx | Updated wrapper property type to allow null . |
apps/web-remix/app/components/pages/pipelines/useDraggableNodes.tsx | Updated wrapper property type to allow null . |
apps/web-remix/app/components/pages/pipelines/usePipelineRun.tsx | Initialized buildel and run refs with null . |
apps/web-remix/app/components/pages/pipelines/usePipelineRunLogs.tsx | Initialized buildel and run refs with null . |
apps/web-remix/app/components/sidebar/NavSidebar.tsx | Changed context access from useContext to use . |
apps/web-remix/app/components/sidebar/siebar.tsx | Changed context access from useContext to use . |
apps/web-remix/app/components/table/table.components.tsx | Changed ExternalLinkCell from forwardRef to functional component. |
apps/web-remix/app/components/tabs/Tab.tsx | Changed TabButton from forwardRef to functional component. |
apps/web-remix/app/components/tabs/TabGroup.tsx | Changed context access from TabsContext.Provider to TabsContext . |
apps/web-remix/app/components/ui/avatar.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/breadcrumb.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/button.tsx | Changed Button from forwardRef to functional component. |
apps/web-remix/app/components/ui/card.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/carousel.tsx | Changed Carousel from forwardRef to functional component. |
apps/web-remix/app/components/ui/chart.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/checkbox.tsx | Changed Checkbox from forwardRef to functional component. |
apps/web-remix/app/components/ui/dialog.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/drawer.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/dropdown-menu.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/input.tsx | Changed Input from forwardRef to functional component. |
apps/web-remix/app/components/ui/label.tsx | Changed Label from forwardRef to functional component. |
apps/web-remix/app/components/ui/menubar.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/radio-group.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/scroll-area.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/select.tsx | Changed components from forwardRef to functional components. |
apps/web-remix/app/components/ui/switch.tsx | Changed Switch from forwardRef to functional component. |
apps/web-remix/app/components/ui/textarea.tsx | Changed Textarea from forwardRef to functional component. |
apps/web-remix/app/components/ui/tooltip.tsx | Changed TooltipContent from forwardRef to functional component. |
apps/web-remix/app/hooks/useLongPress.tsx | Initialized timeout and target refs with null . |
apps/web-remix/app/tests/handlers/pipelines.handlers.ts | Enhanced PipelineHandlers class with improved response structures. |
apps/web-remix/app/tests/handlers/secret.handlers.ts | Added hidden_value property to createHandler response. |
apps/web-remix/app/tests/setup.tests.ts | Updated default user initialization in getSessionCookie . |
apps/web-remix/app/utils/nonce-provider.ts | Incorrectly changed context access from useContext to use . |
apps/web-remix/package.json | Updated @types/react and @types/react-dom to scoped package references. |
In the code where rabbits play,
Components change in a merry way.
FromforwardRef
to props so neat,
Making our functions light and sweet!
With types refined and errors tamed,
Our app is ready, all acclaimed! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
Release Notes
New Features
Bug Fixes
CollectionCostsTable
.Refactor
React.forwardRef
to standard functional components for simpler prop handling.Chores
package.json
.