🎨 Palette: 폼 필수 입력 필드 유효성 검사 UX 개선

[seonghobae]

💡 What: 폼에서 필수 입력 필드(파일, 크기)의 내용이 비워졌을 때 무음으로 유효성 검사 메시지를 지우는 대신, 'This field is required.'라는 에러 메시지와 함께 시각적인 에러 상태를 표시하도록 수정했습니다.
🎯 Why: 기존에는 입력 필드를 지웠을 때 에러 상태가 초기화되어, 사용자가 해당 필드가 필수 항목이라는 것을 즉각적으로 인지하기 어려웠습니다.
📸 Before/After: 필수 필드를 비웠을 때 빨간색 경고 메시지가 표시됩니다.
♿ Accessibility: aria-invalid="true"를 명시적으로 설정하여 스크린 리더 사용자에게 입력값이 유효하지 않음(필수 항목 누락)을 정확하게 전달합니다.

---

PR created automatically by Jules for task 14359548984610969297 started by @seonghobae

---

Summary by CodeRabbit

• 버그 수정
  • 업로드 시 필수 입력값이 비어 있으면 미리보기 영역에 “This field is required.” 오류 메시지가 표시됩니다.
  • 오류 상태가 빨간색으로 명확히 표시되어 누락된 입력을 쉽게 확인할 수 있습니다.
  • 브라우저 양식 검증과 스크린 리더에서 필수 입력 누락 상태를 인식할 수 있도록 접근성 표시가 개선되었습니다.
  • 파일 또는 대상 크기를 입력한 뒤에는 오류 상태가 정상적으로 갱신됩니다.

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

📝 Walkthrough

Walkthrough

업로드 UI에서 파일 또는 대상 크기가 비어 있으면 명시적 오류 메시지와 aria-invalid="true"를 표시합니다. 브라우저 검증 상태를 설정하고 관련 테스트와 실행 지침을 갱신합니다.

Changes

필수 입력 검증

Layer / File(s)	Summary
빈 입력 오류 피드백 saas_web.py, tests/test_empty_target_validation.py, .jules/palette.md	파일 또는 대상 크기가 비어 있으면 "This field is required."를 표시하고, setCustomValidity와 aria-invalid="true"를 설정합니다. 테스트는 새 검증 상태를 확인합니다. 실행 지침은 동일한 동작을 설명합니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: 🟡 Moderate · up to 3dd48

The change improves required-field feedback, but the current head may throw during Batch form initialization before validation listeners are registered, leaving required-field checks inactive; its guidance also contains conflicting empty-number behavior. Merge should wait for the initialization issue and documentation conflict to be corrected.

Possibly related PRs

• ContextualWisdomLab/codec-carver#494: 동일한 saas_web.py 검증 로직과 테스트를 수정하여 필수 입력 오류 및 aria-invalid="true"를 유지합니다.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 필수 입력 필드의 유효성 검사 UX 개선이라는 주요 변경 사항을 정확하게 요약합니다. 변경된 파일과 PR 목표에 직접 부합하며 구체적입니다.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (1 skipped: 1 …
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ux/form-validation-14359548984610969297

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

자동 정리: base 대비 실제 변경(diff)이 0건이라 이 PR을 닫습니다. 변경을 추가한 뒤 reopen하세요.

[devin-ai-integration]

Devin Review found 1 new potential issue.

⚠️ 1 issue in files not directly in the diff

⚠️ Required fields lose validation feedback

Clearing any required file or size field erases its message and aria-invalid. Users lose immediate visual and screen-reader feedback for the missing value.