fix: Refactor beforeUpload to avoid multiple identical returns

- Consolidated multiple return false statements into single return
- Added comment explaining why false is always returned (manual upload)
- Resolves SonarQube code smell about functions with duplicate returns

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: athul-rs

frontend/src/components/agency/file-upload/FileUpload.jsx

@@ -21,10 +21,10 @@ const FileUpload = ({
   const beforeUpload = (file) => {
     if (fileList.length >= WORKFLOW_PAGE_MAX_FILES) {
       message.error(WORKFLOW_VALIDATION_MESSAGES.MAX_FILES_EXCEEDED);
-      return false;
+    } else {
+      setFileList([...fileList, file]);
     }
-    setFileList([...fileList, file]);
-    return false;
+    return false; // Always prevent automatic upload (manual upload on submit)
   };
 
   const submitFile = () => {