fix(entity-editor): prevent server crash when form submitted after page refresh

[Prateekiiitg56]

Description

Fixes server crash that occurs when users refresh the entity-editor page after filling the form and then submit.

Problem

When a user:

1. Fills out the entity-editor form
2. Refreshes the browser page
3. Submits the form

The server would crash because the Redux store resets to initial state on refresh, but HTML inputs visually retain their values. Upon submission, the form sends empty/malformed Redux state instead of the actual form input values.

Changes

• Server-side defensive programming: Added null/undefined checks in sanitizeBody() and processSingleEntity() to handle missing data gracefully
• Early validation: Check for required fields (aliases, name, sortName) before processing
• Error handling: Wrapped transformation logic in try-catch with informative error messages
• Client-side validation: Added pre-submission checks to catch incomplete state on the client
• User guidance: Clear error messages instructing users to refill form after refresh

Testing

• ✅ Code compiles successfully with no errors
• ✅ Prevents crashes from malformed data
• ✅ Provides user-friendly error messages
• Manual testing recommended: Fill form → Refresh page → Submit → Verify error message instead of crash

[Copilot]

Pull request overview

This PR fixes a server crash that occurs when users refresh the entity-editor page after filling the form and then submit. The root cause is that Redux store resets to initial state on page refresh while HTML inputs visually retain their values, resulting in empty/malformed data being submitted to the server.

Changes:

• Added defensive null/undefined checks in server-side functions (sanitizeBody() and processSingleEntity()) to handle missing or malformed data
• Implemented early validation with clear error messages for required fields (aliases, name, sortName) on both server and client
• Wrapped transformNewForm() call in try-catch to gracefully handle transformation errors

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File	Description
src/server/routes/entity/entity.tsx	Added defensive checks in sanitizeBody() for aliases array and disambiguation, plus early validation in processSingleEntity() for required fields
src/server/helpers/entityRouteUtils.tsx	Added validation checks for req.body, nameSection, and submissionSection before processing, with try-catch around transformNewForm()
src/client/entity-editor/submission-section/actions.ts	Added client-side pre-submission validation to check for required state fields (nameSection, aliasEditor) before submitting

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.