Fix open logs on Windows by using real path

[sejas]

Related issues

• Fixes STU-102

Proposed Changes

• Avoid using Symlink for current log file.
• Return the real path to logFile even after rotations.
• Keep parity between OS.

Testing Instructions

• Run npm start
• Go to Import / Export tab
• Try importing this broken zip file empty.txt.zip
• Observe the alert and click on Open logs
• Observe the correct log file opens correctly
• Test on Mac and Windows

To test if it works after a few rotations I applied this diff and repeated the test instructions a couple of times:

diff --git a/src/logging.ts b/src/logging.ts
index 7cc49e66..d0827ad5 100644
--- a/src/logging.ts
+++ b/src/logging.ts
@@ -109,5 +109,6 @@ export function getLogsFilePath(): string {
 	if ( ! currentLogFile ) {
 		throw new Error( 'Logging system not initialized' );
 	}
+	logStream?.rotate( true );
 	return currentLogFile;
 }

import-site-open-logs-with-real-name.mp4

Pre-merge Checklist

• Have you checked for TypeScript, React or other console errors?

[gavande1]

I have tested this and it works as expected. LGTM 👍

[wojtekn]

Thanks for the fix @sejas.

Should we update a Q&A entry to cover for the change?

[sejas]

@wojtekn , good catch! Thanks for the suggestion. I've updated the documentation in English and Spanish.
I considered merging both instruction steps into one, but since each platform uses a different path, I preferred to keep them separate.

Before	After

[wojtekn]

@sejas looks great, thanks!