Add logging and fix packaging issues

Author: BernardJen

README.md

@@ -28,6 +28,18 @@ brew install --cask libreoffice
 brew install poppler
 ```
 
+## Troubleshooting
+
+### Logs Location
+If you encounter issues, check the application logs:
+- **macOS**: `~/Library/Logs/SlideCue/main.log`
+- **Windows**: `%USERPROFILE%\AppData\Roaming\SlideCue\logs\main.log`
+
+### Common Issues
+- **PNGs not generating**: Ensure LibreOffice and Poppler are installed on macOS
+- **Server won't start**: Check if port 3000 is available or being blocked by firewall
+- **Permission errors**: Grant Accessibility and Automation permissions to SlideCue in System Settings
+
 ## Getting Started
 
 ### Development

package-lock.json

@@ -34,18 +34,28 @@
       "out/**/*",
       "package.json"
     ],
+    "extraResources": [
+      {
+        "from": "resources/remote",
+        "to": "remote"
+      }
+    ],
     "mac": {
       "category": "public.app-category.productivity",
       "icon": "resources/icon.icns",
       "identity": null,
       "target": [
         {
           "target": "dmg",
-          "arch": ["universal"]
+          "arch": [
+            "universal"
+          ]
         },
         {
           "target": "zip",
-          "arch": ["universal"]
+          "arch": [
+            "universal"
+          ]
         }
       ]
     },
@@ -54,7 +64,9 @@
       "target": [
         {
           "target": "nsis",
-          "arch": ["x64"]
+          "arch": [
+            "x64"
+          ]
         }
       ]
     },
@@ -70,6 +82,7 @@
   },
   "dependencies": {
     "bonjour-service": "^1.3.0",
+    "electron-log": "^5.4.3",
     "electron-updater": "^6.6.2",
     "express": "^5.2.1",
     "qrcode.react": "^4.2.0",

package.json

@@ -1,4 +1,5 @@
 import { app, BrowserWindow, ipcMain, dialog, protocol, net } from 'electron';
+import log from 'electron-log';
 import path from 'path';
 import os from 'os';
 import fs from 'fs/promises';
@@ -89,13 +90,15 @@ ipcMain.handle('import-presentation', async () => {
   }
 
   presentationFile = result.filePaths[0];
+  log.info('Selected presentation:', presentationFile);
 
-  // Create temp directory for thumbnails
+  // Create directory for thumbnails in app userData
+  const appDataDir = path.join(app.getPath('userData'), 'thumbnails');
   currentThumbsDir = path.join(
-    os.tmpdir(),
-    'slidecue-thumbs',
+    appDataDir,
     Date.now().toString()
   );
+  log.info('Creating thumbnails directory:', currentThumbsDir);
   await fs.mkdir(currentThumbsDir, { recursive: true });
 
   // Send progress callback
@@ -105,9 +108,11 @@ ipcMain.handle('import-presentation', async () => {
 
   // Now that we have a file selected, start the loading screen
   sendProgress(0, 5, 'Starting...');
+  log.info('Starting import process');
 
   // Export thumbnails with progress
   sendProgress(1, 5, 'Opening presentation...');
+  log.info('Opening presentation in PowerPoint');
   await automation.openPresentation(presentationFile);
 
   sendProgress(2, 5, 'Getting slide count...');
@@ -174,7 +179,8 @@ ipcMain.handle('get-slide-info', async () => {
 
 // Cleanup temp directories (synchronous for use in quit handler)
 function cleanupTempDirs() {
-  const tempBase = os.tmpdir();
+  log.info('Cleaning up thumbnail directories');
+  const tempBase = path.join(app.getPath('userData'), 'thumbnails');
   const dirsToClean = ['slidecue-thumbs', 'slidecue-presentations'];
 
   for (const dir of dirsToClean) {

src/main/index.ts

@@ -5,6 +5,7 @@ import { join, basename } from 'path';
 import { tmpdir } from 'os';
 import { PowerPointAutomation, SlideInfo, SlideMetadata, ProgressCallback } from './types';
 import { parsePresentationData, PresentationData, getNextVisibleSlide, getSlideData } from './slideParser';
+import log from 'electron-log';
 
 const execAsync = promisify(exec);
 
@@ -123,7 +124,11 @@ end tell
     try {
       // Use LibreOffice headless conversion (most reliable)
       console.log('Using LibreOffice to convert PPTX to PDF...');
-      await execAsync(`/Applications/LibreOffice.app/Contents/MacOS/soffice --headless --convert-to pdf --outdir "${outputDir}" "${pptxPath}"`);
+      log.info('Starting LibreOffice conversion for:', pptxPath);
+      const sofficeCmd = `/Applications/LibreOffice.app/Contents/MacOS/soffice --headless --convert-to pdf --outdir "${outputDir}" "${pptxPath}"`;
+      log.info('Running soffice command:', sofficeCmd);
+      await execAsync(sofficeCmd);
+      log.info('LibreOffice conversion completed');
 
       // Rename the output file (LibreOffice uses the original filename)
       const baseName = basename(pptxPath).replace(/\.(pptx?|PPTX?)$/, '');
@@ -141,9 +146,16 @@ end tell
       // Check if PDF was created
       await access(pdfPath);
       console.log('PDF created, converting to PNG...');
+      log.info('PDF created successfully at:', pdfPath);
 
       // Convert PDF pages to PNG using pdftoppm
-      await execAsync(`pdftoppm -png -r 150 "${pdfPath}" "${outputDir}/slide"`);
+      // Use full path to pdftoppm since packaged apps don't have full PATH
+      const pdftoppmPath = await execAsync('which pdftoppm').then(r => r.stdout.trim()).catch(() => '/opt/homebrew/bin/pdftoppm');
+      log.info('Using pdftoppm at:', pdftoppmPath);
+      const pdftoppmCmd = `"${pdftoppmPath}" -png -r 150 "${pdfPath}" "${outputDir}/slide"`;
+      log.info('Running pdftoppm command:', pdftoppmCmd);
+      await execAsync(pdftoppmCmd);
+      log.info('Successfully converted PDF to PNGs');
 
       // pdftoppm creates slide-1.png, slide-2.png, etc.
       // LibreOffice only exports visible slides, so we need to rename them
@@ -174,6 +186,7 @@ end tell
       await unlink(pdfPath).catch(() => {});
 
     } catch (e) {
+      log.error('Thumbnail export failed:', e);
       console.error('Export failed:', e);
     }
 
@@ -208,16 +221,19 @@ end tell
   async startSlideshow() {
     currentSlide = 1;
     currentAnimationStep = 0;
+    log.info('Starting slideshow');
 
+    // Use keyboard shortcut which works in any language
     await runAppleScript(`
 tell application "Microsoft PowerPoint"
   activate
 end tell
-delay 0.2
+delay 0.3
 tell application "System Events"
   keystroke return using {command down, shift down}
 end tell
     `);
+    
     await new Promise(resolve => setTimeout(resolve, 500));
 
     // PowerPoint may skip hidden slide 1, query actual position

src/main/pptx/macos.ts

@@ -5,6 +5,7 @@ import { networkInterfaces } from 'os';
 import { readdir } from 'fs/promises';
 import { join, dirname } from 'path';
 import { fileURLToPath } from 'url';
+import log from 'electron-log';
 
 let server: ReturnType<typeof createServer> | null = null;
 let io: Server | null = null;
@@ -15,16 +16,21 @@ let currentPort: number = 3000;
 function isPortAvailable(port: number): Promise<boolean> {
   return new Promise((resolve) => {
     const testServer = createServer();
-    testServer.once('error', () => resolve(false));
+    testServer.once('error', (err) => {
+      log.warn(`Port ${port} is not available:`, err.message);
+      resolve(false);
+    });
     testServer.once('listening', () => {
       testServer.close();
+      log.info(`Port ${port} is available`);
       resolve(true);
     });
     testServer.listen(port, '0.0.0.0');
   });
 }
 
 async function findAvailablePort(startPort: number): Promise<number> {
+  log.info(`Looking for available port starting from ${startPort}`);
   let port = startPort;
   while (!(await isPortAvailable(port))) {
     port++;
@@ -56,6 +62,10 @@ export async function startServer(
   pin: string;
   io: Server;
 }> {
+  log.info('Starting Express server...');
+  log.info('Remote UI path:', remoteUIPath);
+  log.info('Thumbnails directory:', thumbsDir);
+  
   const app = express();
   server = createServer(app);
   io = new Server(server, {
@@ -65,6 +75,9 @@ export async function startServer(
   currentPin = generatePin();
   thumbnailsDir = thumbsDir;
   currentPort = await findAvailablePort(3000);
+  
+  log.info(`Found available port: ${currentPort}`);
+  log.info(`Generated PIN: ${currentPin}`);
 
   // Serve remote UI at root
   app.use(express.static(remoteUIPath));
@@ -106,9 +119,20 @@ export async function startServer(
     }
   });
 
-  server.listen(currentPort, '0.0.0.0');
+  server.listen(currentPort, '0.0.0.0', () => {
+    log.info(`Server started successfully on port ${currentPort}`);
+  });
+  
+  server.on('error', (err: any) => {
+    log.error('Server error:', err);
+    if (err.code === 'EADDRINUSE') {
+      log.error(`Port ${currentPort} is already in use`);
+    }
+  });
 
   const localIP = getLocalIP();
+  log.info(`Server URL: http://${localIP}:${currentPort}`);
+  
   return {
     url: `http://${localIP}:${currentPort}`,
     pin: currentPin,