🐛 BUG: Missing return statements after sending responses leads to request processing continuation

[GaneshPatil7517]

Summary
Several controllers are missing return statements after sending error responses, causing the code to continue executing even after the response is sent. This can lead to unexpected behavior and potential server crashes.

Affected Files

• src/controllers/editProject.ts - Line 10
• src/controllers/forkProject.ts - Line 8
• src/controllers/pullRequest.ts - Line 7

Current Behaviour

// [editProject.ts](http://_vscodecontentref_/0)
if (!commitMessage) {
    res.status(300).json({ message: "Commit message is required" });
}
await updateProjectDataFile(repoName, projectData, commitMessage); // Still executes!

Expected Behaviour

if (!commitMessage) {
    res.status(300).json({ message: "Commit message is required" });
    return; // Should stop execution
}

Impact
Potential duplicate responses
Server-side errors
Unpredictable API behaviour

Suggested Fix
Add return statements after all early response sends.

[Sana9058]

@GaneshPatil7517 Hi, I would like to work on this issue if it's still open. Please let me know if I can proceed.

[GaneshPatil7517]

hey @Sana9058 already working on it, my suggestion understand the code base and raise the issue after that get approval from maintainers then Raise the PR Thank You.......

[Sana9058]

@GaneshPatil7517 Thank you for the clarification. I was not aware that it was already being worked on. Please let me know if I can help with any other issue.