refacotor: put apis properly

Author: abose

src-electron/main-app-ipc.js

@@ -1,18 +1,17 @@
+/**
+ * IPC handlers for electronAppAPI
+ * Preload location: contextBridge.exposeInMainWorld('electronAppAPI', { ... })
+ */
+
 const { app, ipcMain } = require('electron');
 const { spawn } = require('child_process');
 const readline = require('readline');
-const path = require('path');
-const fs = require('fs');
 const { productName } = require('./package.json');
 
 let processInstanceId = 0;
 // Map of instanceId -> { process, terminated }
 const spawnedProcesses = new Map();
 
-// In-memory key-value store shared across all windows (mirrors Tauri's put_item/get_all_items)
-// Used for multi-window storage synchronization
-const sharedStorageMap = new Map();
-
 function waitForTrue(fn, timeout) {
     return new Promise((resolve) => {
         const startTime = Date.now();
@@ -122,38 +121,6 @@ function registerAppIpcHandlers() {
     ipcMain.handle('get-app-name', () => {
         return productName;
     });
-
-    // Set zoom factor on the webview (mirrors Tauri's zoom_window)
-    ipcMain.handle('zoom-window', (event, scaleFactor) => {
-        event.sender.setZoomFactor(scaleFactor);
-    });
-
-    // In-memory storage for multi-window sync (mirrors Tauri's put_item/get_all_items)
-    ipcMain.handle('put-item', (event, key, value) => {
-        sharedStorageMap.set(key, value);
-    });
-
-    ipcMain.handle('get-all-items', () => {
-        return Object.fromEntries(sharedStorageMap);
-    });
-
-    // Get path to phnode binary
-    ipcMain.handle('get-phnode-path', () => {
-        const phNodePath = path.resolve(__dirname, 'bin', 'phnode');
-        if (!fs.existsSync(phNodePath)) {
-            throw new Error(`phnode binary does not exist: ${phNodePath}`);
-        }
-        return phNodePath;
-    });
-
-    // Get path to src-node (for development)
-    ipcMain.handle('get-src-node-path', () => {
-        const srcNodePath = path.resolve(__dirname, '..', '..', 'phoenix', 'src-node');
-        if (!fs.existsSync(srcNodePath)) {
-            throw new Error(`src-node path does not exist: ${srcNodePath}`);
-        }
-        return srcNodePath;
-    });
 }
 
 module.exports = {

src-electron/main-fs-ipc.js

@@ -1,3 +1,8 @@
+/**
+ * IPC handlers for electronFSAPI
+ * Preload location: contextBridge.exposeInMainWorld('electronFSAPI', { ... })
+ */
+
 const { ipcMain, dialog, BrowserWindow } = require('electron');
 const path = require('path');
 const fsp = require('fs/promises');

src-electron/main.js

@@ -1,9 +1,14 @@
-const { app, BrowserWindow, protocol } = require('electron');
+const { app, BrowserWindow, protocol, Menu, ipcMain } = require('electron');
 const path = require('path');
+const fs = require('fs');
 
 const { registerAppIpcHandlers, terminateAllProcesses } = require('./main-app-ipc');
 const { registerFsIpcHandlers, getAppDataDir } = require('./main-fs-ipc');
 
+// In-memory key-value store shared across all windows (mirrors Tauri's put_item/get_all_items)
+// Used for multi-window storage synchronization
+const sharedStorageMap = new Map();
+
 let mainWindow;
 
 async function createWindow() {
@@ -21,9 +26,6 @@ async function createWindow() {
     // Load the test page from the http-server
     mainWindow.loadURL('http://localhost:8000/src/');
 
-    // Open DevTools for debugging
-    mainWindow.webContents.openDevTools();
-
     mainWindow.on('closed', () => {
         mainWindow = null;
     });
@@ -39,12 +41,57 @@ async function gracefulShutdown(exitCode = 0) {
 registerAppIpcHandlers();
 registerFsIpcHandlers();
 
+/**
+ * IPC handlers for electronAPI
+ * Preload location: contextBridge.exposeInMainWorld('electronAPI', { ... })
+ */
+
+// Set zoom factor on the webview (mirrors Tauri's zoom_window)
+ipcMain.handle('zoom-window', (event, scaleFactor) => {
+    event.sender.setZoomFactor(scaleFactor);
+});
+
+// In-memory storage for multi-window sync (mirrors Tauri's put_item/get_all_items)
+ipcMain.handle('put-item', (event, key, value) => {
+    sharedStorageMap.set(key, value);
+});
+
+ipcMain.handle('get-all-items', () => {
+    return Object.fromEntries(sharedStorageMap);
+});
+
+// Toggle DevTools
+ipcMain.handle('toggle-dev-tools', (event) => {
+    event.sender.toggleDevTools();
+});
+
+// Get path to phnode binary
+ipcMain.handle('get-phnode-path', () => {
+    const phNodePath = path.resolve(__dirname, 'bin', 'phnode');
+    if (!fs.existsSync(phNodePath)) {
+        throw new Error(`phnode binary does not exist: ${phNodePath}`);
+    }
+    return phNodePath;
+});
+
+// Get path to src-node (for development)
+ipcMain.handle('get-src-node-path', () => {
+    const srcNodePath = path.resolve(__dirname, '..', '..', 'phoenix', 'src-node');
+    if (!fs.existsSync(srcNodePath)) {
+        throw new Error(`src-node path does not exist: ${srcNodePath}`);
+    }
+    return srcNodePath;
+});
+
 // Handle quit request from renderer
 app.on('quit-requested', (exitCode) => {
     gracefulShutdown(exitCode);
 });
 
 app.whenReady().then(async () => {
+    // Remove default menu bar
+    Menu.setApplicationMenu(null);
+
     // Register asset:// protocol for serving local files from appLocalData/assets/
     const appDataDir = getAppDataDir();
     const assetsDir = path.join(appDataDir, 'assets');

src-electron/preload.js

@@ -71,6 +71,9 @@ contextBridge.exposeInMainWorld('electronAPI', {
     putItem: (key, value) => ipcRenderer.invoke('put-item', key, value),
     getAllItems: () => ipcRenderer.invoke('get-all-items'),
 
+    // Toggle DevTools
+    toggleDevTools: () => ipcRenderer.invoke('toggle-dev-tools'),
+
     // Path to phnode binary (src-electron/bin/phnode)
     getPhNodePath: () => ipcRenderer.invoke('get-phnode-path'),