fix: resolve headless mode config path on macOS/Linux

Fixed critical bug where headless mode on macOS and Linux was reading
per-printer settings from the wrong directory, causing custom camera
and LED configurations to appear as "not configured" even though they
worked in normal UI mode.

Root cause: Module import order caused ConfigManager and
PrinterDetailsManager singletons to capture app.getPath('userData')
before app.setName('FlashForgeUI') was called, resulting in different
paths for headless vs normal mode.

Solution: Created bootstrap.ts to set app name before any module imports,
ensuring all singletons use the correct userData path across all platforms
and execution modes.

Also includes CI improvements:
- Switch from npm install to npm ci in GitHub Actions for faster builds
- Add platform-specific CI build commands
- Disable source maps and declaration files in production builds

Author: GhostTypes

.github/workflows/release.yml

@@ -41,10 +41,10 @@ jobs:
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
-      - name: Run npm install
+      - name: Run npm ci
         shell: bash
         run: |
-          npm install --verbose
+          npm ci
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
@@ -65,21 +65,11 @@ jobs:
         run: |
           # Use platform-specific build commands
           if [[ "$RUNNER_OS" == "Linux" ]]; then
-            npm run build:linux
+            npm run build:ci:linux
           elif [[ "$RUNNER_OS" == "Windows" ]]; then
-            npm run build:win
+            npm run build:ci:win
           elif [[ "$RUNNER_OS" == "macOS" ]]; then
-            npm run build:mac
-          fi
-
-      - name: List Build Artifacts
-        shell: bash
-        run: |
-          echo "Listing contents of dist directory:"
-          if [[ "$RUNNER_OS" == "Windows" ]]; then
-            dir /s dist || echo "dist directory not found or empty."
-          else
-            ls -R dist || echo "dist directory not found or empty."
+            npm run build:ci:mac
           fi
 
       - name: Upload Build Artifacts

package.json

@@ -18,6 +18,9 @@
     "build:linux": "npm run build && electron-builder --linux --publish never --config electron-builder-config.js",
     "build:win": "npm run build && electron-builder --win --publish never --config electron-builder-config.js && rimraf dist/win-unpacked",
     "build:mac": "npm run build && electron-builder --mac --publish never --config electron-builder-config.js",
+    "build:ci:linux": "npm run build && electron-builder --linux --publish never --config electron-builder-config.js",
+    "build:ci:win": "npm run build && electron-builder --win --publish never --config electron-builder-config.js && rimraf dist/win-unpacked",
+    "build:ci:mac": "npm run build && electron-builder --mac --publish never --config electron-builder-config.js",
     "lint": "eslint src/**/*.ts",
     "lint:fix": "eslint src/**/*.ts --fix",
     "linecount": "powershell -ExecutionPolicy Bypass -File scripts/count_lines.ps1",

src/bootstrap.ts

@@ -0,0 +1,42 @@
+/**
+ * @fileoverview Bootstrap module for FlashForgeUI Electron application.
+ *
+ * CRITICAL: This file must be imported FIRST in index.ts, before any other imports.
+ *
+ * Purpose:
+ * Sets the Electron app name and user model ID before any singletons are instantiated.
+ * This ensures that all services using app.getPath('userData') point to the correct
+ * directory across all platforms and execution modes (normal UI and headless).
+ *
+ * Problem it solves:
+ * - Singletons like ConfigManager and PrinterDetailsManager capture app.getPath('userData')
+ *   during construction
+ * - If app.setName() is called after these singletons are created, they will use the wrong
+ *   directory (e.g., "Electron" instead of "FlashForgeUI")
+ * - This caused headless mode on macOS/Linux to read from a different config directory
+ *   than the main UI, resulting in missing per-printer settings (custom camera/LED config)
+ *
+ * Platform-specific userData paths:
+ * - macOS: ~/Library/Application Support/FlashForgeUI/
+ * - Linux: ~/.config/FlashForgeUI/
+ * - Windows: %APPDATA%/FlashForgeUI/
+ *
+ * Without this bootstrap (default "Electron" name):
+ * - macOS: ~/Library/Application Support/Electron/
+ * - Linux: ~/.config/Electron/
+ * - Windows: %APPDATA%/Electron/
+ */
+
+import { app } from 'electron';
+
+// Set app name BEFORE any singletons are created
+// This ensures ConfigManager, PrinterDetailsManager, and all other services
+// use the correct userData directory path
+app.setName('FlashForgeUI');
+
+// Set AppUserModelId to match electron-builder appId for proper notification routing
+// This works across all platforms (Windows uses it for Action Center, macOS for notification attribution)
+app.setAppUserModelId('com.ghosttypes.flashforgeui');
+
+console.log('[Bootstrap] App name set to "FlashForgeUI"');
+console.log(`[Bootstrap] userData path: ${app.getPath('userData')}`);

src/index.ts

@@ -17,6 +17,9 @@
  * - Managing application-level concerns like power-saving and environment detection.
  */
 
+// CRITICAL: Bootstrap must be imported FIRST to set app name before any singletons are created
+import './bootstrap';
+
 import { app, BrowserWindow, dialog, powerSaveBlocker, ipcMain } from 'electron';
 import { getConfigManager } from './managers/ConfigManager';
 import { getPrinterConnectionManager } from './managers/ConnectionFlowManager';
@@ -73,13 +76,8 @@ if (!gotTheLock) {
   });
 }
 
-// Set AppUserModelId to match electron-builder appId for proper notification routing
-// This works across all platforms (Windows uses it for Action Center, macOS for notification attribution)
-app.setAppUserModelId('com.ghosttypes.flashforgeui');
-
-// Ensure app uses the correct name for userData directory
-// This must be set before any services that use app.getPath('userData') are initialized
-app.setName('FlashForgeUI');
+// Note: app.setName() and app.setAppUserModelId() are now called in bootstrap.ts
+// to ensure they execute before any singleton initialization
 
 // Initialize global reference for camera IPC handler
 global.printerBackendManager = undefined;

tsconfig.json

@@ -12,9 +12,9 @@
     "moduleResolution": "node",
     "allowSyntheticDefaultImports": true,
     "resolveJsonModule": true,
-    "declaration": true,
-    "declarationMap": true,
-    "sourceMap": true,
+    "declaration": false,
+    "declarationMap": false,
+    "sourceMap": false,
     "noImplicitAny": true,
     "noImplicitReturns": true,
     "noImplicitThis": true,