Uh oh!
There was an error while loading. Please reload this page.
Restructure: Move the UI related components inside the UI Folder - #164
Merged
Conversation
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThis PR reorganizes the component module structure by establishing a centralized ChangesUI Module Reorganization
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 10
🧹 Nitpick comments (2)
app/components/text-to-speech/EvaluationsTab.tsx (1)
14-14: ⚡ Quick winConsider using the guideline-recommended import path for useToast.
The coding guidelines specify importing
useToast()from@/app/components/Toastor@/app/hooks/useToast. This file now imports from@/app/components/ui, which is not mentioned in the guidelines.For consistency with the established guidelines, consider either:
- Importing from
@/app/hooks/useToastinstead, or- Updating the coding guidelines to reflect
@/app/components/uias the new canonical sourceAs per coding guidelines: "Consume toast notifications from any client component using the useToast() hook imported from
@/app/components/Toastor@/app/hooks/useToast."🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@app/components/text-to-speech/EvaluationsTab.tsx` at line 14, The import in EvaluationsTab.tsx currently pulls useToast from "`@/app/components/ui`"; change the import to follow guidelines by importing useToast from either "`@/app/hooks/useToast`" or "`@/app/components/Toast`" (i.e., update the import statement that references useToast) so the hook is consumed consistently, or if "`@/app/components/ui`" is intentionally preferred, update the coding guidelines to list that path as an allowed canonical source.app/components/text-to-speech/DatasetsTab.tsx (1)
12-12: ⚡ Quick winConsider using the guideline-recommended import path for useToast.
The coding guidelines specify importing
useToast()from@/app/components/Toastor@/app/hooks/useToast. This file now imports from@/app/components/ui, which is not mentioned in the guidelines.For consistency with the established guidelines, consider either:
- Importing from
@/app/hooks/useToastinstead, or- Updating the coding guidelines to reflect
@/app/components/uias the new canonical sourceAs per coding guidelines: "Consume toast notifications from any client component using the useToast() hook imported from
@/app/components/Toastor@/app/hooks/useToast."🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@app/components/text-to-speech/DatasetsTab.tsx` at line 12, The import in DatasetsTab.tsx pulls useToast from "`@/app/components/ui`" which deviates from the guideline; change the import to use the canonical hook path (import useToast from "`@/app/hooks/useToast`" or "`@/app/components/Toast`") so DatasetsTab uses the approved source, or if the new path is intentional, update the coding guidelines to include "`@/app/components/ui`"; locate the import statement for useToast in DatasetsTab and replace the module specifier accordingly.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@app/components/evaluations/DatasetsTab.tsx`:
- Line 8: The import in DatasetsTab (currently importing Button, Modal, useToast
from "`@/app/components/ui`") must use the approved toast source; update the
import so Button and Modal remain from "`@/app/components/ui`" (or their correct
UI module) but import useToast from "`@/app/components/Toast`" or
"`@/app/hooks/useToast`" instead, ensuring the DatasetsTab component continues to
call useToast() from the approved hook location.
In `@app/components/settings/onboarding/AddProjectModal.tsx`:
- Line 4: The import for useToast in AddProjectModal.tsx should come from the
approved hook/module instead of the UI barrel; update the import statement that
currently brings useToast from "`@/app/components/ui`" to import useToast from
"`@/app/hooks/useToast`" (or "`@/app/components/Toast`") so that the Modal and
Button/Field primitives remain from "`@/app/components/ui`" while useToast is
consumed from the dedicated hook/module.
In `@app/components/settings/onboarding/AddUserModal.tsx`:
- Line 4: The current barrel import pulls useToast from "`@/app/components/ui`";
update the imports so that Button, Field, Modal continue to be imported from
"`@/app/components/ui`" but import useToast separately from the allowed module
(either "`@/app/hooks/useToast`" or "`@/app/components/Toast`") to follow project
conventions; modify the import statements in AddUserModal.tsx (references:
Button, Field, Modal, useToast) to split useToast into its own import from the
approved path.
In `@app/components/settings/onboarding/EditProjectModal.tsx`:
- Line 4: Update the import for useToast in EditProjectModal.tsx so it follows
the project standard: replace the current import from the UI barrel ("useToast"
from "`@/app/components/ui`") with the hook from "`@/app/hooks/useToast`" (or
"`@/app/components/Toast`") wherever useToast is referenced in this component
(e.g., inside the EditProjectModal component/function), ensuring no other
imports are affected.
In `@app/components/settings/onboarding/OnboardingCredentials.tsx`:
- Line 5: Import path for the useToast hook is incorrect; replace the current
import of useToast with the allowed path: update the import of useToast used in
the OnboardingCredentials component to import from "`@/app/hooks/useToast`" (or
"`@/app/components/Toast`") instead of "`@/app/components/ui`", and ensure any
references to useToast() inside OnboardingCredentials remain unchanged so the
hook is consumed from the correct module.
In `@app/components/settings/onboarding/OnboardingForm.tsx`:
- Line 4: The import for useToast is coming from the UI barrel; update the
import so Button and Field remain imported from "`@/app/components/ui`" but import
useToast from the hooks/toast module (for example "`@/app/hooks/useToast`" or
"`@/app/components/Toast`"); locate the import line that currently reads "import {
Button, Field, useToast } from '`@/app/components/ui`'" in OnboardingForm.tsx and
split it into two imports so useToast is consumed from the dedicated hook module
while keeping Button and Field from the UI barrel.
In `@app/components/settings/onboarding/OnboardingSuccess.tsx`:
- Line 4: The component is importing useToast from "`@/app/components/ui`"—update
the import so useToast is instead imported from the mandated location (either
"`@/app/components/Toast`" or "`@/app/hooks/useToast`") in the OnboardingSuccess
component; locate the import statement that currently reads "import { Button,
useToast } from '`@/app/components/ui`'" and remove useToast from that import,
then add a separate import for useToast from the correct module and keep Button
imported from "`@/app/components/ui`" unchanged.
In `@app/components/settings/onboarding/UserList.tsx`:
- Line 16: Replace the current combined import that pulls useToast from
"`@/app/components/ui`" in UserList.tsx so that useToast is imported from the
approved path (either "`@/app/components/Toast`" or "`@/app/hooks/useToast`") and
keep Button imported from "`@/app/components/ui`"; update the import statement to
remove useToast from the ui import and import the useToast symbol from the
repo-approved module instead.
In `@app/components/speech-to-text/DatasetsTab.tsx`:
- Line 12: The import of useToast in DatasetsTab.tsx should be switched to the
approved source; update the import statement that currently reads `import {
useToast } from "`@/app/components/ui`";` to import useToast from either
`@/app/components/Toast` or `@/app/hooks/useToast` (choose the one used
elsewhere in the project) so the DatasetsTab component uses the standardized
hook; locate the symbol useToast in the DatasetsTab component and replace the
import path accordingly.
In `@app/components/speech-to-text/EvaluationsTab.tsx`:
- Line 34: The toast type reference in EvaluationsTab.tsx is pointing to the
wrong module path; update the annotation for the toast variable (currently typed
as ReturnType<typeof import("`@/app/components/ui/Toast`").useToast>) to use the
approved module path—e.g. ReturnType<typeof
import("`@/app/components/Toast`").useToast> or ReturnType<typeof
import("`@/app/hooks/useToast`").useToast>—and ensure the actual useToast import
in the component matches that same module (adjust the import statement for
useToast if necessary) so the type and runtime import are aligned.
---
Nitpick comments:
In `@app/components/text-to-speech/DatasetsTab.tsx`:
- Line 12: The import in DatasetsTab.tsx pulls useToast from
"`@/app/components/ui`" which deviates from the guideline; change the import to
use the canonical hook path (import useToast from "`@/app/hooks/useToast`" or
"`@/app/components/Toast`") so DatasetsTab uses the approved source, or if the new
path is intentional, update the coding guidelines to include
"`@/app/components/ui`"; locate the import statement for useToast in DatasetsTab
and replace the module specifier accordingly.
In `@app/components/text-to-speech/EvaluationsTab.tsx`:
- Line 14: The import in EvaluationsTab.tsx currently pulls useToast from
"`@/app/components/ui`"; change the import to follow guidelines by importing
useToast from either "`@/app/hooks/useToast`" or "`@/app/components/Toast`" (i.e.,
update the import statement that references useToast) so the hook is consumed
consistently, or if "`@/app/components/ui`" is intentionally preferred, update the
coding guidelines to list that path as an allowed canonical source.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 8f125f5f-34dd-4896-ae4b-bfa881653f69
📒 Files selected for processing (111)
app/(main)/chat/page.tsxapp/(main)/configurations/page.tsxapp/(main)/configurations/prompt-editor/page.tsxapp/(main)/datasets/page.tsxapp/(main)/document/page.tsxapp/(main)/evaluations/[id]/page.tsxapp/(main)/evaluations/page.tsxapp/(main)/guardrails/page.tsxapp/(main)/keystore/page.tsxapp/(main)/knowledge-base/page.tsxapp/(main)/settings/credentials/page.tsxapp/(main)/settings/onboarding/page.tsxapp/(main)/speech-to-text/page.tsxapp/(main)/text-to-speech/page.tsxapp/components/ComingSoon.tsxapp/components/ConfigCard.tsxapp/components/ConfigSelector.tsxapp/components/GatePopover.tsxapp/components/PageHeader.tsxapp/components/auth/FeatureGateModal.tsxapp/components/auth/LoginModal.tsxapp/components/auth/TokenVerifyPage.tsxapp/components/datasets/DatasetListing.tsxapp/components/datasets/DeleteDatasetModal.tsxapp/components/datasets/UploadDatasetModal.tsxapp/components/document/DeleteDocumentModal.tsxapp/components/document/DocumentListing.tsxapp/components/document/DocumentPreview.tsxapp/components/document/UploadDocumentModal.tsxapp/components/evaluations/CreateDatasetForm.tsxapp/components/evaluations/DatasetCard.tsxapp/components/evaluations/DatasetsTab.tsxapp/components/evaluations/DeleteDatasetModal.tsxapp/components/evaluations/DetailedResultsTable.tsxapp/components/evaluations/EvalRunCard.tsxapp/components/evaluations/EvalRunsList.tsxapp/components/evaluations/EvaluationsTab.tsxapp/components/evaluations/GroupedResultsTable.tsxapp/components/evaluations/MetricsOverview.tsxapp/components/evaluations/RunEvaluationForm.tsxapp/components/evaluations/ViewDatasetModal.tsxapp/components/guardrails/BanListField.tsxapp/components/guardrails/BanListModal.tsxapp/components/guardrails/SavedConfigsList.tsxapp/components/guardrails/SchemaField.tsxapp/components/guardrails/ValidatorConfigPanel.tsxapp/components/index.tsapp/components/keystore/AddKeyModal.tsxapp/components/keystore/KeysCard.tsxapp/components/knowledge-base/CollectionDetail.tsxapp/components/knowledge-base/CollectionsList.tsxapp/components/knowledge-base/CreateCollectionForm.tsxapp/components/knowledge-base/DeleteCollectionModal.tsxapp/components/knowledge-base/DocumentChip.tsxapp/components/knowledge-base/DocumentPickerModal.tsxapp/components/knowledge-base/DocumentPreviewModal.tsxapp/components/prompt-editor/ConfigEditorPane.tsxapp/components/prompt-editor/ConfigNameSection.tsxapp/components/prompt-editor/DiffView.tsxapp/components/prompt-editor/GuardrailsSection.tsxapp/components/prompt-editor/HistorySidebar.tsxapp/components/prompt-editor/SaveConfigModal.tsxapp/components/prompt-editor/ToolsSection.tsxapp/components/providers/Providers.tsxapp/components/settings/credentials/CredentialForm.tsxapp/components/settings/credentials/CredentialFormPanel.tsxapp/components/settings/onboarding/AddProjectModal.tsxapp/components/settings/onboarding/AddUserModal.tsxapp/components/settings/onboarding/EditProjectModal.tsxapp/components/settings/onboarding/OnboardingCredentials.tsxapp/components/settings/onboarding/OnboardingForm.tsxapp/components/settings/onboarding/OnboardingSuccess.tsxapp/components/settings/onboarding/OrganizationList.tsxapp/components/settings/onboarding/ProjectList.tsxapp/components/settings/onboarding/UserList.tsxapp/components/speech-to-text/AudioFileItem.tsxapp/components/speech-to-text/CreateSTTDatasetForm.tsxapp/components/speech-to-text/DatasetsTab.tsxapp/components/speech-to-text/EvaluationsTab.tsxapp/components/speech-to-text/RunSTTEvaluationForm.tsxapp/components/speech-to-text/STTDatasetCard.tsxapp/components/speech-to-text/STTRunCard.tsxapp/components/speech-to-text/STTViewDatasetModal.tsxapp/components/text-to-speech/CreateTTSDatasetForm.tsxapp/components/text-to-speech/DatasetsTab.tsxapp/components/text-to-speech/EvaluationsTab.tsxapp/components/text-to-speech/RunTTSEvaluationForm.tsxapp/components/text-to-speech/TTSDatasetCard.tsxapp/components/text-to-speech/TTSRunCard.tsxapp/components/text-to-speech/TTSViewDatasetModal.tsxapp/components/ui/Button.tsxapp/components/ui/CodeBlock.tsxapp/components/ui/ConfigModal.tsxapp/components/ui/CopyableCodeBlock.tsxapp/components/ui/ErrorModal.tsxapp/components/ui/Field.tsxapp/components/ui/FileExtBadge.tsxapp/components/ui/InfoTooltip.tsxapp/components/ui/Loader.tsxapp/components/ui/Modal.tsxapp/components/ui/MultiSelect.tsxapp/components/ui/Select.tsxapp/components/ui/TabNavigation.tsxapp/components/ui/Tag.tsxapp/components/ui/Toast.tsxapp/components/ui/index.tsapp/hooks/useCollections.tsapp/hooks/useConfigPersistence.tsapp/hooks/useSttData.tsapp/hooks/useToast.tsapp/lib/constants.ts
💤 Files with no reviewable changes (1)
- app/components/index.ts
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
vprashrex
approved these changes
May 18, 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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Changes
app/components. Now I have moved all the UI related components inside theapp/components/uiso that we know which is the ui related components.Summary by CodeRabbit