Fixed file extension extraction issue for filenames with multiple dots.

[NimeshaKahingala]

Refactored File Extension Extraction

• Improved file extension extraction to handle filenames with multiple dots dynamically.
• Instead of checking for fixed extensions (.yml or .json) and slicing based on fixed lengths, the new solution uses lastIndexOf('.') to reliably separate the filename and extension.
• This improves flexibility by supporting any file extension while preventing incorrect slicing for filenames with multiple dots.

Backend Refactor

• Moved the file extension splitting logic to the backend to keep the frontend logic cleaner and abstract internal implementation details.

[RobertLRead]

I think line 16 should have a semicolon appended at the end (;)

[NimeshaKahingala]

I did this modification.

[RobertLRead]

I think this would be slightly better if rewritten so that there is only one return statement, with the conditional statement setting the values.

export function getFileNameAndFileType(filename: string): { fileName: string; fileType: string } {
const lastDotIndex = filename.lastIndexOf(".");

const fileName = (lastDotIndex === -1) ? filename : filename.substring(0, lastDotIndex); // Extract name before last dot
const fileType = (lastDotIndex === -1) ? "" : filename.substring(lastDotIndex + 1); // Extract extension after last dot

return { fileName, fileType };

}

[NimeshaKahingala]

I did the following modifications.

• Improved error handling compared to the previous behavior where invalid filenames would silently return an empty file type.
• Now throws errors for missing or invalid file extensions (e.g., no . or filename ends with a .).

[RobertLRead]

I approve this, but can you please make and test the small changes I request?