Skip to content

Commit ac31906

Browse files
authored
Merge pull request #10 from fccview/BUG-2
Fix scripts not working if description is empty
2 parents 032c63c + b267fb9 commit ac31906

2 files changed

Lines changed: 25 additions & 8 deletions

File tree

app/_components/modals/ScriptModal.tsx

Lines changed: 23 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,10 +39,21 @@ export function ScriptModal({
3939
}: ScriptModalProps) {
4040
const handleSubmit = async (e: React.FormEvent) => {
4141
e.preventDefault();
42+
43+
if (!form.name.trim()) {
44+
showToast("error", "Validation Error", "Script name is required");
45+
return;
46+
}
47+
48+
if (!form.content.trim()) {
49+
showToast("error", "Validation Error", "Script content is required");
50+
return;
51+
}
52+
4253
const formData = new FormData();
43-
formData.append("name", form.name);
44-
formData.append("description", form.description);
45-
formData.append("content", form.content);
54+
formData.append("name", form.name.trim());
55+
formData.append("description", form.description.trim());
56+
formData.append("content", form.content.trim());
4657

4758
Object.entries(additionalFormData).forEach(([key, value]) => {
4859
formData.append(key, value);
@@ -66,18 +77,24 @@ export function ScriptModal({
6677
<div className="grid grid-cols-1 md:grid-cols-2 gap-4">
6778
<div>
6879
<label className="block text-sm font-medium text-foreground mb-2">
69-
Script Name
80+
Script Name <span className="text-red-500">*</span>
7081
</label>
7182
<Input
7283
value={form.name}
7384
onChange={(e) => onFormChange({ name: e.target.value })}
7485
placeholder="My Script"
7586
required
87+
className={
88+
!form.name.trim()
89+
? "border-red-300 focus:border-red-500 focus:ring-red-500"
90+
: ""
91+
}
7692
/>
7793
</div>
7894
<div>
7995
<label className="block text-sm font-medium text-foreground mb-2">
80-
Description
96+
Description{" "}
97+
<span className="text-muted-foreground text-xs">(optional)</span>
8198
</label>
8299
<Input
83100
value={form.description}
@@ -102,7 +119,7 @@ export function ScriptModal({
102119
<div className="flex items-center gap-2 mb-4 flex-shrink-0">
103120
<FileText className="h-4 w-4 text-primary" />
104121
<h3 className="text-sm font-medium text-foreground">
105-
Script Content
122+
Script Content <span className="text-red-500">*</span>
106123
</h3>
107124
</div>
108125
<div className="flex-1 min-h-0">

app/_utils/scriptScanner.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,13 +46,13 @@ async function scanScriptsDirectory(dirPath: string): Promise<Script[]> {
4646
const content = await fs.readFile(filePath, "utf-8");
4747
const metadata = parseMetadata(content);
4848

49-
if (metadata.id && metadata.title && metadata.description) {
49+
if (metadata.id && metadata.title) {
5050
const stats = await fs.stat(filePath);
5151

5252
scripts.push({
5353
id: metadata.id,
5454
name: metadata.title,
55-
description: metadata.description,
55+
description: metadata.description || "",
5656
filename: file,
5757
createdAt: stats.birthtime.toISOString(),
5858
});

0 commit comments

Comments
 (0)