fix: use unified isProduction check with app.isPackaged

Use app.isPackaged alongside NODE_ENV to determine production mode
in both logger and window setup. Also adjust devTools toggle to rely
on the unified isProduction variable for consistent behavior.

Author: Kasper24

src/electron/main/lib/logger.ts

@@ -1,8 +1,9 @@
+import { app } from "electron/main";
 import pino from "pino";
 import pinoPretty from "pino-pretty";
 import { logFilePath } from "@electron/main/lib/paths.js";
 
-const isProduction = process.env.NODE_ENV === "production";
+const isProduction = process.env.NODE_ENV === "production" || app.isPackaged;
 
 const logger = pino(
   {

src/electron/main/setup/window.ts

@@ -1,6 +1,8 @@
 import path from "path";
 import { app, BrowserWindow } from "electron";
 
+const isProduction = process.env.NODE_ENV === "production" || app.isPackaged;
+
 const createWindow = () => {
   const iconPath = path.join(import.meta.dirname, "..", "..", "assets", "icon.png");
   const mainWindow = new BrowserWindow({
@@ -12,7 +14,7 @@ const createWindow = () => {
     show: false,
     titleBarStyle: "hidden",
     webPreferences: {
-      devTools: !app.isPackaged,
+      devTools: isProduction,
       contextIsolation: true,
       sandbox: true,
       allowRunningInsecureContent: false,