Merge pull request #31 from kryptodrex/issue-fixes

Version 0.3.0 Updates and Fixes

Author: kryptodrex

electron/main.ts

@@ -1169,6 +1169,40 @@ ipcMain.handle('save-budget', async (event, filePath: string, data: string) => {
   }
 });
 
+/**
+ * Rename budget file
+ * @param oldPath - Existing file path
+ * @param newPath - Desired file path
+ */
+ipcMain.handle('rename-budget-file', async (_event, oldPath: string, newPath: string) => {
+  try {
+    if (!oldPath || !newPath) {
+      return { success: false, error: 'Both old and new file paths are required.' };
+    }
+
+    if (oldPath === newPath) {
+      return { success: true, filePath: oldPath };
+    }
+
+    // Ensure source file exists before attempting rename.
+    await fs.access(oldPath);
+
+    // Avoid clobbering an existing file at the target path.
+    try {
+      await fs.access(newPath);
+      return { success: false, error: 'A file with that name already exists.' };
+    } catch {
+      // Target does not exist - safe to continue.
+    }
+
+    await fs.rename(oldPath, newPath);
+    return { success: true, filePath: newPath };
+  } catch (error) {
+    console.error('Error renaming budget file:', error);
+    return { success: false, error: (error as Error).message };
+  }
+});
+
 /**
  * Load budget data from file
  * @param filePath - Where to load the file from

electron/preload.ts

@@ -22,6 +22,12 @@ contextBridge.exposeInMainWorld('electronAPI', {
   // Returns: { success: boolean, error?: string }
   saveBudget: (filePath: string, data: string) => 
     ipcRenderer.invoke('save-budget', filePath, data),
+
+  // Rename an existing budget file
+  // Takes: current path and new path
+  // Returns: { success: boolean, filePath?: string, error?: string }
+  renameBudgetFile: (oldPath: string, newPath: string) =>
+    ipcRenderer.invoke('rename-budget-file', oldPath, newPath),
 
   // Load budget from file
   // Takes: file path

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "paycheck-planner",
   "private": true,
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Desktop paycheck planning and budget management application.",
   "author": {
     "name": "Paycheck Planner",

package.json

@@ -31,6 +31,12 @@ const AccountsManager: React.FC<AccountsManagerProps> = ({ onClose }) => {
     updateAccount(id, updates);
   };
 
+  const handleMoveAccount = (fromIndex: number, toIndex: number) => {
+    const newAccounts = [...accounts];
+    [newAccounts[fromIndex], newAccounts[toIndex]] = [newAccounts[toIndex], newAccounts[fromIndex]];
+    updateBudgetData({ accounts: newAccounts });
+  };
+
   const handleDeleteAccount = (id: string) => {
     if (accounts.length <= 1) {
       return;
@@ -140,6 +146,7 @@ const AccountsManager: React.FC<AccountsManagerProps> = ({ onClose }) => {
           onAdd={handleAddAccount}
           onUpdate={handleUpdateAccount}
           onDelete={handleDeleteAccount}
+          onMove={handleMoveAccount}
           showToggleButton={true}
           infoMessage="These accounts are specific to this plan. Changes will only affect this plan file."
         />