V3.0.1: Fix clean exit and icon loading for packaged app

Ensures the application process exits cleanly by stopping animations and explicitly quitting the QApplication on window close. Adds a get_resource_path utility for robust icon loading in both PyInstaller onefile and onedir modes, and updates documentation and version numbers to 3.0.1 to reflect these bugfixes.

Author: thefamer

DESIGN_SYSTEM.md

@@ -1,4 +1,4 @@
-# ResearchFlow V3.0.0 Design System
+# ResearchFlow V3.0.1 Design System
 
 This document serves as the Single Source of Truth (SSOT) for the visual design and user interface interactions of ResearchFlow V2.0.0.
 

README.md

@@ -9,7 +9,7 @@
 </p>
 
 <p align="center">
-  <img src="https://img.shields.io/badge/version-3.0.0-blue.svg" alt="Version">
+  <img src="https://img.shields.io/badge/version-3.0.1-blue.svg" alt="Version">
   <img src="https://img.shields.io/badge/python-3.10+-green.svg" alt="Python">
   <img src="https://img.shields.io/badge/platform-Windows-lightgrey.svg" alt="Platform">
   <img src="https://img.shields.io/badge/license-MIT-orange.svg" alt="License">
@@ -28,9 +28,13 @@ ResearchFlow is a portable, aesthetically pleasing desktop application designed
 
 ---
 
-## ✨ What's New in V3.0.0
+## ✨ What's New in V3.0.1
 
-### 📦 Portable Executable
+### 🐛 Bugfixes
+- **Clean Exit**: Fixed issue where application process remained running after closing the window.
+- **Icon Loading**: Fixed window icon not displaying in packaged .exe builds.
+
+### 📦 V3.0.0 - Portable Executable
 - **One-Click .exe**: Download and run – no Python installation required!
 - **True Portable Design**: All project data stored next to the `.exe`, perfect for USB drives.
 - **PyInstaller Optimized**: Properly handles frozen environment paths.
@@ -132,12 +136,17 @@ Create a standalone portable executable that runs without Python installed:
 
 ```bash
 pip install pyinstaller
-pyinstaller --noconsole --onefile --icon=icon.ico --name="ResearchFlow" main.py
+
+# Option 1: Single folder (recommended, faster startup)
+pyinstaller --noconsole --onedir --icon=icon.ico --add-data "icon.ico;." --name="ResearchFlow" main.py
+
+# Option 2: Single file (slower startup, but just one file)
+pyinstaller --noconsole --onefile --icon=icon.ico --add-data "icon.ico;." --name="ResearchFlow" main.py
 ```
 
-The generated `ResearchFlow.exe` will be in the `dist/` folder. 
+The generated files will be in the `dist/` folder. 
 
-> **✅ True Portable**: The `projects/` data folder is automatically created next to the `.exe` file, not in any temp or system folder. Copy the `.exe` and your `projects/` folder anywhere!
+> **✅ True Portable**: The `projects/` data folder is automatically created next to the `.exe` file, not in any temp or system folder. Copy the folder and your `projects/` anywhere!
 
 ---
 

main.py

@@ -26,7 +26,7 @@
     ProjectData, NodeData, NodeMetadata, Position, EdgeData, Snippet,
     generate_uuid
 )
-from utils import ProjectManager, extract_title_from_filename, get_app_root, ModernTheme
+from utils import ProjectManager, extract_title_from_filename, get_app_root, get_resource_path, ModernTheme
 from graphics_items import (
     BaseNodeItem, PipelineModuleItem, ReferenceNodeItem, EdgeItem,
     TempConnectionLine, Colors, SnippetItem
@@ -1073,7 +1073,7 @@ def _show_about(self) -> None:
         QMessageBox.about(
             self, "About ResearchFlow",
             "<h2>ResearchFlow</h2>"
-            "<p>Version 3.0.0</p>"
+            "<p>Version 3.0.1</p>"
             "<p>A portable research management tool for academics.</p>"
             "<p>Built with Python and PyQt6.</p>"
             "<hr>"
@@ -1129,10 +1129,24 @@ def _auto_save(self) -> None:
             self.project_manager.save_project()
 
     def closeEvent(self, event) -> None:
-        """Handle window close."""
+        """Handle window close - ensure clean shutdown."""
+        # Stop any running animations to prevent process hanging
+        if self._dock_animation:
+            self._dock_animation.stop()
+            self._dock_animation = None
+        
+        # Stop view zoom animation
+        if hasattr(self, 'view') and self.view._zoom_animation:
+            self.view._zoom_animation.stop()
+            self.view._zoom_animation = None
+        
+        # Save project if open
         if self.project_manager.is_project_open:
             self._save_project()
+        
+        # Accept close and quit application
         event.accept()
+        QApplication.instance().quit()
 
 
 # ============================================================================
@@ -1145,9 +1159,10 @@ def main():
 
     app = QApplication(sys.argv)
 
-    # Set Global Icon
-    icon_path = str(get_app_root() / "icon.ico")
-    app.setWindowIcon(QIcon(icon_path))
+    # Set Global Icon (use resource path for PyInstaller compatibility)
+    icon_path = get_resource_path("icon.ico")
+    if icon_path.exists():
+        app.setWindowIcon(QIcon(str(icon_path)))
 
     # Apply Modern Theme Stylesheet
     app.setStyleSheet(ModernTheme.get_stylesheet())

utils.py

@@ -17,7 +17,7 @@
 
 def get_app_root() -> Path:
     """
-    Get the application root directory.
+    Get the application root directory (for data storage like projects/).
     Handles both normal Python execution and PyInstaller bundled .exe.
     """
     if getattr(sys, 'frozen', False):
@@ -29,6 +29,20 @@ def get_app_root() -> Path:
         return Path(__file__).parent.resolve()
 
 
+def get_resource_path(relative_path: str) -> Path:
+    """
+    Get the path to a bundled resource file.
+    Works both in normal Python and PyInstaller frozen environment.
+    """
+    if getattr(sys, 'frozen', False):
+        # PyInstaller creates a temp folder and stores path in _MEIPASS (onefile)
+        # For onedir mode, resources are in the same folder as .exe
+        base_path = getattr(sys, '_MEIPASS', Path(sys.executable).parent)
+        return Path(base_path) / relative_path
+    else:
+        return Path(__file__).parent / relative_path
+
+
 def get_projects_dir() -> Path:
     """Get the projects directory path."""
     return get_app_root() / "projects"