Skip to content

fix: harden input validation, file safety, and error handling #227

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ankit1999 wants to merge 8 commits into
base: main
Choose a base branch
from
Open

fix: harden input validation, file safety, and error handling #227

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
9c97815
feat: add custom HuggingFace voice model support
ankit1999 Mar 1, 2026
a7e698a
fix: harden input validation, file safety, and error handling
ankit1999 Mar 2, 2026
888d5a5
fix(review): address second round of code review feedback
ankit1999 Apr 19, 2026
10446f1
fix(review): address third round of code review feedback
ankit1999 Apr 19, 2026
d983119
fix(review): address fourth round of code review feedback
ankit1999 Apr 19, 2026
4447a87
fix(review): address fifth round of code review feedback
ankit1999 Apr 19, 2026
ff9c08d
Merge origin/main into feat/custom-voice-models
ankit1999 Apr 19, 2026
ae135ae
chore: remove workflow files (will be added separately with workflow …
ankit1999 Apr 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 0 additions & 26 deletions .github/workflows/ci.yml
View file
Open in desktop

This file was deleted.

251 changes: 0 additions & 251 deletions .github/workflows/release.yml
View file
Open in desktop

This file was deleted.

10 changes: 6 additions & 4 deletions app/src/components/Generation/FloatingGenerateBox.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@ import { Form, FormControl, FormField, FormItem, FormMessage } from '@/component
import {
Select,
SelectContent,
SelectGroup,
SelectItem,
SelectLabel,
Comment on lines +11 to +13
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot Apr 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Remove unused imports.

SelectGroup and SelectLabel are imported but never used in this component.

🧹 Proposed fix to remove unused imports 
 import {
   Select,
   SelectContent,
-  SelectGroup,
   SelectItem,
-  SelectLabel,
   SelectTrigger,
   SelectValue,
 } from '@/components/ui/select';
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
SelectGroup,
SelectItem,
SelectLabel,
import {
Select,
SelectContent,
SelectItem,
SelectTrigger,
SelectValue,
} from '@/components/ui/select';
🧰 Tools
🪛 Biome (2.4.11)

[error] 11-13: Several of these imports are unused.

(lint/correctness/noUnusedImports)

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@app/src/components/Generation/FloatingGenerateBox.tsx` around lines 11 - 13,
The import list in FloatingGenerateBox.tsx includes unused symbols SelectGroup
and SelectLabel; remove these two from the named imports (the import that
currently lists SelectGroup, SelectItem, SelectLabel) so only used symbols like
SelectItem remain, then run a quick type-check/ESLint to ensure no other unused
imports remain.

SelectTrigger,
SelectValue,
} from '@/components/ui/select';
Expand Down Expand Up @@ -234,10 +236,10 @@ export function FloatingGenerateBox({
ref={containerRef}
className={cn(
'fixed right-auto',
isStoriesRoute
? // Position aligned with story list: after sidebar + padding, width 360px
'left-[calc(5rem+2rem)] w-[360px]'
: 'left-[calc(5rem+2rem)] right-8 lg:right-auto lg:w-[calc((100%-5rem-4rem)/2-1rem)]',
isStoriesRoute
? // Position aligned with story list: after sidebar + padding, width 360px
'left-[calc(5rem+2rem)] w-[360px]'
: 'left-[calc(5rem+2rem)] right-8 lg:right-auto lg:w-[calc((100%-5rem-4rem)/2-1rem)]',
)}
style={{
// On stories route: offset by track editor height when visible
Expand Down
2 changes: 2 additions & 0 deletions app/src/components/Generation/GenerationForm.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,9 @@ import { Input } from '@/components/ui/input';
import {
Select,
SelectContent,
SelectGroup,
SelectItem,
SelectLabel,
Comment on lines +18 to +20
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot Apr 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Remove unused imports.

SelectGroup and SelectLabel are imported but never used in this component.

🧹 Proposed fix to remove unused imports 
 import {
   Select,
   SelectContent,
-  SelectGroup,
   SelectItem,
-  SelectLabel,
   SelectTrigger,
   SelectValue,
 } from '@/components/ui/select';
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
SelectGroup,
SelectItem,
SelectLabel,
import {
Select,
SelectContent,
SelectItem,
SelectTrigger,
SelectValue,
} from '@/components/ui/select';
🧰 Tools
🪛 Biome (2.4.11)

[error] 18-20: Several of these imports are unused.

(lint/correctness/noUnusedImports)

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@app/src/components/Generation/GenerationForm.tsx` around lines 18 - 20, The
imports SelectGroup and SelectLabel in the GenerationForm component are unused;
remove them from the import list that currently includes SelectGroup,
SelectItem, SelectLabel so only the needed symbols (e.g., SelectItem) remain,
updating the import statement at the top of GenerationForm.tsx (the
component/function named GenerationForm) to eliminate SelectGroup and
SelectLabel.

SelectTrigger,
SelectValue,
} from '@/components/ui/select';
Expand Down
Loading