Merge pull request #10 from JeffGepiga/OptimizedServices

Optimized services

Author: JeffGepiga

.github/copilot-instructions.md

@@ -1,9 +1,9 @@
 # DevBox Pro - AI Coding Agent Instructions
 
 ## Project Overview
-DevBox Pro is an Electron desktop app providing a local PHP development environment with bundled services (PHP, MySQL/MariaDB, Redis, Nginx/Apache, etc.). No Docker required.
+DevBox Pro is an Electron desktop app for local PHP and Node.js development with bundled runtimes and services. It supports multi-version PHP, Node.js, MySQL, MariaDB, Redis, PostgreSQL, MongoDB, Nginx, Apache, Mailpit, phpMyAdmin, MinIO, Memcached, and related tooling without Docker.
 
-**Tech Stack:** Electron 28 + React 18 + Vite + TailwindCSS
+**Tech Stack:** Electron 40 + React 19 + Vite 7 + TailwindCSS 4
 
 ## Architecture
 
@@ -13,16 +13,30 @@ DevBox Pro is an Electron desktop app providing a local PHP development environm
 - **Communication**: All cross-process calls go through IPC via `preload.js` → `window.devbox.*` API
 - **Shared Config** (`src/shared/serviceConfig.js`): Single source of truth for service versions and ports
 
-### Manager Pattern (src/main/services/)
-Each service domain has a dedicated manager class initialized in `main.js`:
-- `ServiceManager` - Starts/stops services (Nginx, MySQL, Redis, etc.), manages ports
-- `ProjectManager` - CRUD for projects, virtual host generation, compatibility checks
-- `DatabaseManager` - MySQL/MariaDB operations, import/export, credentials
-- `BinaryDownloadManager` - Downloads/extracts binaries with progress tracking
-- `CliManager` - External CLI tool (`dvp`) for terminal integration
+### Service Architecture (Critical)
+Service managers have been refactored into thin facade classes plus concern-specific mixins under `src/main/services/<domain>/`.
+
+Examples:
+- `ProjectManager.js` composes mixins from `src/main/services/project/`
+- `ServiceManager.js` composes mixins from `src/main/services/service/`
+- `BinaryDownloadManager.js` composes mixins from `src/main/services/binary/`
+- `DatabaseManager.js` composes mixins from `src/main/services/database/`
+- `GitManager.js` composes mixins from `src/main/services/git/`
+- `CompatibilityManager.js` composes mixins from `src/main/services/compatibility/`
+- `SupervisorManager.js` composes mixins from `src/main/services/supervisor/`
 
 Managers receive references to each other via `managers` object for cross-communication.
 
+### Manager Responsibilities
+- `ProjectManager` - Project CRUD, install flows, environment sync, vhost orchestration, compatibility checks
+- `ServiceManager` - Starts/stops web servers and bundled services, owns runtime config generation
+- `DatabaseManager` - Multi-engine database operations, import/export, credentials, metadata
+- `BinaryDownloadManager` - Download, extract, import, update, and discover binary assets
+- `CliManager` - External `dvp` command integration and PATH/project mapping
+- `GitManager` - Git executable discovery, clone/auth, SSH key workflows, progress reporting
+- `CompatibilityManager` - Bundled/remote compatibility rules and config normalization
+- `SupervisorManager` - Background worker/process management for PHP, Node.js, Python, and queues
+
 ### IPC Handler Pattern
 All frontend-to-backend calls defined in `src/main/ipc/handlers.js`:
 ```javascript
@@ -41,34 +55,60 @@ Exposed to renderer via `preload.js` as `window.devbox.projects.create(config)`.
 ### Service Versioning
 Multiple versions of services can run simultaneously on different ports:
 - Port offsets defined in `shared/serviceConfig.js` (`VERSION_PORT_OFFSETS`)
-- First web server to start claims ports 80/443; second gets 8081/8444
+- Base/default ports defined in `shared/serviceConfig.js` (`DEFAULT_PORTS`)
+- First web server to start can claim ports `80` and `443`; the alternate server is assigned fallback ports and proxied automatically
 
 ### Project Configuration
 Projects stored in `configStore.get('projects')` array with:
 - Per-project PHP version, web server choice, SSL settings
-- `.test` domain generation (e.g., `myproject.test`)
-- Virtual host configs auto-generated for Nginx/Apache
+- Per-project Node.js version and runtime command configuration for Node projects
+- `.test` domain generation (for example `myproject.test`)
+- Virtual host configs auto-generated for Nginx and Apache
+- Compatibility checks accept both saved-project and flattened UI payloads
 
 ### Binary Management
 - Binaries downloaded to `app.getPath('userData')/resources/`
 - Remote config at `config/binaries.json` defines download URLs
 - Extraction runs in worker thread (`extractWorker.js`) to avoid blocking UI
+- Installed/versioned assets are discovered via manager mixins under `src/main/services/binary/`
 
 ### Process Spawning (Windows)
-Always use `windowsHide: true` to prevent console window flashing:
+Always use `windowsHide: true` to prevent console window flashing.
+
+Preferred approach:
 ```javascript
-spawn(command, args, { windowsHide: true, shell: process.platform === 'win32' });
+spawn(executablePath, args, { windowsHide: true, shell: false });
 ```
 
+Rules:
+- Prefer direct executable paths plus argument arrays
+- Avoid `shell: true` with argument arrays; Node now emits `DEP0190` for that pattern
+- Only use shell execution when you truly need a shell builtin or wrapper, and then keep it explicit
+- Quote filesystem paths inside generated config files because install paths commonly contain spaces on Windows
+
+### Generated Configs
+- Runtime web server and service configs under the user data path are derived artifacts
+- Do not hand-edit generated Nginx, Apache, or database config files; update the generator code instead
+
+### Tests Mirror Source
+- Keep integration coverage in the main manager tests under `tests/main/services/*Manager.test.js`
+- Add focused unit tests under subfolders that mirror the mixin layout, such as `tests/main/services/project/` or `tests/main/services/git/`
+
 ## Development Workflow
 
 ```bash
 npm run dev          # Start Electron + Vite dev server concurrently
+npm run build:renderer
 npm run build:win    # Build Windows installer
 npm run build:mac    # Build macOS installer
+npm run build:all    # Build Windows and macOS packages
+npm test             # Run all Vitest suites
+npm run test:main
+npm run test:renderer
+npm run test:e2e
 ```
 
-Renderer dev server runs at `http://localhost:3000`. Main process hot-reloads on save.
+Renderer dev server runs at `http://localhost:3000`.
 
 ## File Locations
 
@@ -79,6 +119,8 @@ Renderer dev server runs at `http://localhost:3000`. Main process hot-reloads on
 | Preload API | `src/main/preload.js` |
 | React pages | `src/renderer/src/pages/` |
 | Binary URLs | `config/binaries.json` |
+| Compatibility rules | `config/compatibility.json` |
+| Service mixins | `src/main/services/<domain>/` |
 | App config | `~/.devbox-pro/` (runtime) |
 
 ## Common Patterns
@@ -91,7 +133,14 @@ Renderer dev server runs at `http://localhost:3000`. Main process hot-reloads on
 ### Adding a New Service Version
 1. Update `SERVICE_VERSIONS` in `src/shared/serviceConfig.js`
 2. Add download URLs to `config/binaries.json`
-3. Define port offset in `VERSION_PORT_OFFSETS` if needed
+3. Define port offset in `VERSION_PORT_OFFSETS` if the service is versioned and daemonized
+4. Update renderer selectors or service metadata if the service is user-facing
+
+### Extending a Manager
+1. Prefer adding a new mixin under the relevant `src/main/services/<domain>/` folder
+2. Keep the facade thin: constructor state plus `Object.assign(...)`
+3. Use `this.methodName()` for cross-mixin calls instead of cross-importing mixins
+4. Mirror new mixins with focused tests under `tests/main/services/<domain>/`
 
 ### Platform-Specific Code
 ```javascript
@@ -103,3 +152,5 @@ const exe = process.platform === 'win32' ? 'php.exe' : 'php';
 - Test service start/stop with multiple versions running
 - Verify port conflicts handled gracefully (alternate ports assigned)
 - Test on both Windows and macOS for path separator issues (`path.join` always)
+- Prefer targeted Vitest slices for the service or mixin you changed before running broader suites
+- When changing generated configs or runtime spawning, cover Windows path and quoting behavior explicitly

README.MD

@@ -1,24 +1,24 @@
 <p align="center">
-  <img src="https://img.shields.io/badge/Electron-28.0.0-47848F?style=for-the-badge&logo=electron&logoColor=white" alt="Electron">
-  <img src="https://img.shields.io/badge/React-18.2.0-61DAFB?style=for-the-badge&logo=react&logoColor=black" alt="React">
-  <img src="https://img.shields.io/badge/Vite-5.0.10-646CFF?style=for-the-badge&logo=vite&logoColor=white" alt="Vite">
-  <img src="https://img.shields.io/badge/TailwindCSS-3.4.0-38B2AC?style=for-the-badge&logo=tailwind-css&logoColor=white" alt="TailwindCSS">
+  <img src="https://img.shields.io/badge/Electron-40.0.0-47848F?style=for-the-badge&logo=electron&logoColor=white" alt="Electron">
+  <img src="https://img.shields.io/badge/React-19.0.0-61DAFB?style=for-the-badge&logo=react&logoColor=black" alt="React">
+  <img src="https://img.shields.io/badge/Vite-7.0.0-646CFF?style=for-the-badge&logo=vite&logoColor=white" alt="Vite">
+  <img src="https://img.shields.io/badge/TailwindCSS-4.0.0-38B2AC?style=for-the-badge&logo=tailwind-css&logoColor=white" alt="TailwindCSS">
 </p>
 
 # 🚀 DevBox Pro
 
-**Run legacy and modern PHP & Node.js apps side by side — no conflicts, no headaches**
+**Run legacy and modern PHP and Node.js apps side by side with isolated runtimes, bundled services, and zero Docker setup.**
 
-Maintaining an old PHP 7.4 project while building a new Laravel 11 app on PHP 8.5? Running a Next.js frontend alongside a PHP backend? DevBox Pro makes it effortless. Unlike traditional setups, you can **run multiple PHP and Node.js versions simultaneously** — giving you the freedom to work on legacy applications alongside cutting-edge projects without switching environments or breaking dependencies.
+Maintaining an old PHP 7.4 project while building a new Laravel 11 app on PHP 8.5? Running a Next.js frontend alongside a PHP backend? DevBox Pro is built for exactly that workflow. It lets you run multiple PHP, Node.js, database, and web server versions at the same time, with per-project configuration, automatic local domains, compatibility warnings, and bundled tooling.
 
-DevBox Pro bundles everything you need: PHP 7.4 through 8.5, Node.js 16 through 24, MySQL, MariaDB, Redis, Nginx, Apache, and more — all in one standalone desktop app. No Docker, no complex configurations, just instant multi-version development.
+DevBox Pro is an Electron desktop app with a React renderer and a Node.js main process. It manages PHP, Node.js, MySQL, MariaDB, Redis, Nginx, Apache, PostgreSQL, MongoDB, MinIO, Memcached, Mailpit, phpMyAdmin, Composer, Git, and more from one local interface.
 
 ---
 
 ## ✨ Features
 
 ### 🐘 Multi-PHP Version Support
-- **PHP 7.4, 8.0, 8.1, 8.2, 8.3, 8.4, 8.5** - Run any version side by side
+- **PHP 7.4, 8.0, 8.1, 8.2, 8.3, 8.4, 8.5** - Run multiple versions side by side
 - Per-project PHP version selection with compatibility validation
 - Easy extension management per version with php.ini editor
 - Pre-configured for optimal performance
@@ -32,7 +32,7 @@ DevBox Pro bundles everything you need: PHP 7.4 through 8.5, Node.js 16 through
 ### 🌐 Web Server Options
 - **Nginx 1.24, 1.26, 1.28** - High-performance, low memory footprint (recommended)
 - **Apache 2.4** - Full .htaccess support, mod_rewrite included
-- Multi-version support - run different versions per project
+- Front-door ownership and automatic proxying between Nginx and Apache
 - Automatic virtual host creation
 - HTTP & HTTPS support for every project
 
@@ -42,15 +42,21 @@ DevBox Pro bundles everything you need: PHP 7.4 through 8.5, Node.js 16 through
 | **MySQL** | 5.7, 8.0, 8.4 | Powerful relational database |
 | **MariaDB** | 10.6, 10.11, 11.4 | MySQL-compatible, open source |
 | **Redis** | 6.2, 7.2, 7.4 | In-memory cache & session storage |
+| **PostgreSQL** | 14, 15, 16, 17 | Advanced relational database |
+| **MongoDB** | 6.0, 7.0, 8.0 | Document database |
+| **Memcached** | 1.6 | Distributed memory caching |
+| **MinIO** | Latest | S3-compatible object storage |
 | **Mailpit** | Latest | Email testing with SMTP server |
 | **phpMyAdmin** | Latest | Web-based database management |
 | **Node.js** | 16, 18, 20, 22, 24 (LTS) | JavaScript runtime for projects & tooling |
+| **Python** | 3.10, 3.11, 3.12, 3.13 | Runtime for project tooling and workers |
+| **SQLite** | 3 | Embedded database runtime |
 | **Composer** | Latest | PHP dependency manager |
 
 ### 💾 Database Management
-- **Multi-version support** - Run MySQL 8.0 and MariaDB 11.4 simultaneously
+- **Multi-engine support** - MySQL, MariaDB, PostgreSQL, and MongoDB workflows
 - **Per-version data isolation** - Each version has its own data directory
-- **Database import/export** with .gz compression support
+- **Database import/export** with `.sql`, `.gz`, PostgreSQL, and MongoDB flows
 - **Progress tracking** for import/export operations
 - **phpMyAdmin integration** for web-based management
 - **Automatic database creation** per project
@@ -73,6 +79,7 @@ DevBox Pro bundles everything you need: PHP 7.4 through 8.5, Node.js 16 through
 - **Multi-version management** - Keep multiple versions installed
 - **One-click updates** - Easy version upgrades
 - **Import custom binaries** - Use your own compiled versions
+- **Remote definitions** - Binary metadata and compatibility rules can be updated without shipping a new app build
 
 ### ☁️ Cloud Configuration Updates
 - **Remote binary definitions** - New versions available without app update
@@ -82,7 +89,7 @@ DevBox Pro bundles everything you need: PHP 7.4 through 8.5, Node.js 16 through
 - **Version tracking** - Only download when updates are available
 
 ### 💻 Terminal Commands
-- **Direct command access** - Use `php`, `npm`, `node`, `composer` directly from any terminal
+- **Direct command access** - Use `php`, `npm`, `node`, `composer`, `mysql`, `mysqldump`, and related tools directly from any terminal
 - **Automatic version detection** - Detects your project and uses the correct PHP/Node.js version
 - **Works everywhere** - VS Code, Windows Terminal, PowerShell, or any terminal emulator
 - **No prefix needed** - Just run `php artisan migrate` instead of complex paths
@@ -166,7 +173,7 @@ mysqldump -u root mydb > backup.sql
 ### Prerequisites
 
 - Node.js 24+
-- npm or yarn
+- npm
 - Git
 
 ### Setup
@@ -179,19 +186,23 @@ cd DevBoxPro
 # Install dependencies
 npm install
 
-# Start development server
+# Start the Electron app with the Vite renderer dev server
 npm run dev
 
-# Build for production
-npm run build
+# Build the renderer bundle only
+npm run build:renderer
 
-# Create distribution package
-npm run dist
+# Build platform packages
+npm run build:win
+npm run build:mac
+npm run build:all
 ```
 
+The renderer dev server runs at `http://localhost:3000`.
+
 ### Testing
 
-DevBox Pro includes a comprehensive test suite covering Unit, Integration, and End-to-End (E2E) scenarios.
+DevBox Pro includes unit, integration, and end-to-end coverage.
 
 ```bash
 # Run all tests
@@ -207,7 +218,7 @@ npm run test:renderer
 npm run test:e2e
 ```
 
-The E2E tests are built using Playwright and cover the full project lifecycle, database workflows, binary downloads, SSL configuration, and settings persistence.
+The test suite covers the main-process managers, the renderer, and Playwright end-to-end scenarios for project creation, binaries, databases, SSL, and settings persistence.
 
 ### Project Structure
 
@@ -216,19 +227,22 @@ DevBoxPro/
 ├── src/
 │   ├── main/                    # Electron main process
 │   │   ├── services/
-│   │   │   ├── BinaryDownloadManager.js  # Binary downloads and archive extraction
-│   │   │   ├── CliManager.js            # External terminal integration
-│   │   │   ├── CompatibilityManager.js  # Version compatibility rules and updates
-│   │   │   ├── DatabaseManager.js       # Database operations (CRUD, import/export)
-│   │   │   ├── GitManager.js            # Git clone and SSH key workflows
-│   │   │   ├── LogManager.js            # Log management
-│   │   │   ├── PhpManager.js            # PHP version management
-│   │   │   ├── ProjectManager.js        # Project lifecycle and vhost generation
-│   │   │   ├── ServiceManager.js        # Web servers, databases, and background services
-│   │   │   ├── SslManager.js            # SSL certificates
-│   │   │   ├── SupervisorManager.js     # Process supervisor
-│   │   │   ├── UpdateManager.js         # App updates
-│   │   │   ├── extractWorker.js         # Worker-thread archive extraction
+│   │   │   ├── BinaryDownloadManager.js  # Thin facade for binary downloads and extraction
+│   │   │   ├── CliManager.js             # Thin facade for terminal integration
+│   │   │   ├── CompatibilityManager.js   # Thin facade for compatibility rules and updates
+│   │   │   ├── DatabaseManager.js        # Thin facade for multi-engine DB operations
+│   │   │   ├── GitManager.js             # Thin facade for Git clone and SSH workflows
+│   │   │   ├── ProjectManager.js         # Thin facade for project lifecycle and vhosts
+│   │   │   ├── ServiceManager.js         # Thin facade for service lifecycle and config generation
+│   │   │   ├── SupervisorManager.js      # Thin facade for worker and process supervision
+│   │   │   ├── binary/                   # Binary manager mixins
+│   │   │   ├── cli/                      # CLI manager mixins
+│   │   │   ├── compatibility/            # Compatibility manager mixins
+│   │   │   ├── database/                 # Database manager mixins
+│   │   │   ├── git/                      # Git manager mixins
+│   │   │   ├── project/                  # Project manager mixins
+│   │   │   ├── service/                  # Service manager mixins
+│   │   │   ├── supervisor/               # Supervisor manager mixins
 │   │   │   └── ...
 │   │   ├── ipc/
 │   │   │   └── handlers.js      # IPC communication
@@ -282,15 +296,22 @@ DevBoxPro/
 |---------|-------------|
 | PHP Projects | 8000+ (auto-assigned) |
 | MySQL | 3306 |
-| MariaDB | 3306 (or 3307 if MySQL is running) |
+| MariaDB | 3310 |
 | Redis | 6379 |
+| PostgreSQL | 5432 |
+| MongoDB | 27017 |
+| Memcached | 11211 |
+| MinIO API | 9000 |
+| MinIO Console | 9001 |
 | Mailpit SMTP | 1025 |
 | Mailpit Web | 8025 |
 | phpMyAdmin | 8080 |
 | Nginx HTTP | 80 |
 | Nginx HTTPS | 443 |
-| Apache HTTP | 80 (or 8081 if Nginx is on 80) |
-| Apache HTTPS | 443 |
+| Apache HTTP | 8081 when Apache is not the front-door server |
+| Apache HTTPS | 8444 when Apache is not the front-door server |
+
+Web server ownership is dynamic. The first active web server can claim ports `80` and `443`, while the other is assigned alternate ports and proxied automatically.
 
 ### Environment Variables
 
@@ -325,10 +346,12 @@ MAIL_PORT=1025
 
 ## 🛠️ Technologies
 
-- **Electron** - Cross-platform desktop framework
-- **React** - UI library
-- **Vite** - Fast build tool
-- **TailwindCSS** - Utility-first CSS
+- **Electron 40** - Cross-platform desktop framework
+- **React 19** - UI library
+- **Vite 7** - Build and dev server
+- **TailwindCSS 4** - Styling pipeline
+- **Vitest** - Unit and integration testing
+- **Playwright** - End-to-end testing
 - **electron-builder** - Distribution packaging
 
 ---