Show error notification if selected file is inaccessible (#843)

* send error message using ipc

* remove redundant log

* linting and copilot

* change to return success and message technique

* delete unused import

* null safety

* use error message as message

* linting

Author: Turnlings

gcs/electron/main.ts

@@ -192,13 +192,19 @@ ipcMain.handle("window:select-file-in-explorer", async (_event, filters) => {
     try {
       const stats = fs.statSync(filePath)
       return {
+        success: true,
         path: filePath,
         name: path.basename(filePath),
         size: stats.size,
       }
     } catch (err) {
-      // File is inaccessible or deleted
-      return null
+      return {
+        success: false,
+        message:
+          err instanceof Error
+            ? err.message
+            : "File is inaccessible or deleted",
+      }
     }
   }
   return null

gcs/src/components/fla/SelectFlightLog.jsx

@@ -38,8 +38,10 @@ export default function SelectFlightLog({ getLogSummary }) {
       "window:select-file-in-explorer",
       [{ name: "Flight Logs", extensions: ["log", "ftlog"] }],
     )
-    if (result) {
+    if (result?.success) {
       handleFile(result)
+    } else if (result) {
+      showErrorNotification(result.message)
     }
   }
 

gcs/src/missions.jsx

@@ -239,13 +239,15 @@ export default function Missions() {
       "window:select-file-in-explorer",
       [{ name: "Waypoint files", extensions: ["waypoints", "txt"] }],
     )
-    if (result) {
+    if (result?.success) {
       dispatch(
         emitImportMissionFromFile({
           type: activeTabRef.current,
           file_path: result.path,
         }),
       )
+    } else if (result) {
+      showErrorNotification(result.message)
     }
   }