Merge pull request #25 from vijaythecoder/feature/macos-permissions-integration

Feature/macos permissions integration

Author: vijaythecoder

.claude/settings.local.json

@@ -0,0 +1,8 @@
+{
+  "permissions": {
+    "allow": [
+      "mcp__sequential-thinking__sequentialthinking"
+    ],
+    "deny": []
+  }
+}

.github/workflows/lint.yml

@@ -25,9 +25,14 @@ jobs:
           php-version: '8.4'
 
       - name: Install Dependencies
+        run: composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
+      
+      - name: Install Node Dependencies
         run: |
-          composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
-          npm install
+          # Workaround for npm optional dependencies bug
+          # https://github.com/npm/cli/issues/4828
+          rm -rf node_modules package-lock.json
+          npm install --no-optional
 
       - name: Run Pint
         run: vendor/bin/pint

.github/workflows/tests.yml

@@ -29,10 +29,13 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: '22'
-          cache: 'npm'
 
       - name: Install Node Dependencies
-        run: npm ci
+        run: |
+          # Workaround for npm optional dependencies bug
+          # https://github.com/npm/cli/issues/4828
+          rm -rf node_modules package-lock.json
+          npm install --no-optional
 
       - name: Install Dependencies
         run: composer install --no-interaction --prefer-dist --optimize-autoloader

.gitignore

@@ -24,3 +24,6 @@ yarn-error.log
 /.zed
 /dist
 .DS_Store
+
+# NativePHP local development
+nativephp-electron/

CLAUDE.md

@@ -232,4 +232,33 @@ This project uses MIT License with Commons Clause:
 - NativePHP allows running as desktop application
 - Concurrently runs multiple processes in development (server, queue, logs, vite)
 - Built primarily for macOS desktop usage
-- Uses Pusher for additional real-time features
+- Uses Pusher for additional real-time features
+
+## AI Team Configuration (autogenerated by team-configurator, 2025-01-30)
+**Important: YOU MUST USE subagents when available for the task.**
+
+### Detected Technology Stack
+- **Backend Framework**: Laravel 12.0 (PHP 8.2+)
+- **Frontend Framework**: Vue 3.5.13 with TypeScript and Inertia.js
+- **CSS Framework**: Tailwind CSS 4.1.1
+- **Build Tool**: Vite 6
+- **Desktop Platform**: NativePHP/Electron
+- **Test Framework**: Pest PHP
+- **Database**: SQLite (dual database setup)
+- **Real-time Features**: OpenAI Realtime API, WebSockets
+- **State Management**: Pinia
+- **Additional Tools**: Prettier, ESLint, Laravel Pint
+
+### Assigned AI Specialists
+
+| Task | Agent | Notes |
+|------|-------|-------|
+| Laravel backend development | @agent-laravel-backend-expert | Controllers, services, models, Inertia.js integration |
+| Database and Eloquent queries | @agent-laravel-eloquent-expert | ORM optimization, complex queries, migrations |
+| Vue component development | @agent-vue-component-architect | Vue 3 Composition API, component patterns |
+| State management (Pinia) | @agent-vue-state-manager | Pinia stores, state architecture |
+| Frontend styling and UI | @agent-tailwind-frontend-expert | Tailwind CSS, responsive design, Reka UI components |
+| API design and architecture | @agent-api-architect | RESTful design, API best practices |
+| Documentation updates | @agent-documentation-specialist | README, API docs, architecture guides |
+| Code review | @agent-code-reviewer | Security, quality, maintainability checks |
+| Performance optimization | @agent-performance-optimizer | Bottleneck identification, optimization |

NATIVEPHP_BUILD_FIX.md

@@ -0,0 +1,79 @@
+# NativePHP Build Fix Guide
+
+## Problem Summary
+When building the NativePHP/Electron app, frontend assets weren't being properly included, causing "Vite not found" errors on fresh installations.
+
+## Root Causes Identified
+
+1. **Hot Reload File**: The `public/hot` file was being included in builds, causing Laravel to look for Vite dev server
+2. **Missing npm install**: Build process wasn't installing npm dependencies before building
+3. **Build artifacts**: Development artifacts were interfering with production builds
+
+## Fixes Applied
+
+### 1. Updated `config/nativephp.php`
+
+- Added `public/hot` to `cleanup_exclude_files` to prevent it from being included in builds
+- Added `npm install --omit=dev` to prebuild commands before `npm run build`
+
+### 2. Created Build Preparation Script
+
+Created `build-prepare.sh` to ensure clean builds:
+- Removes development artifacts (hot file, .vite cache)
+- Installs production npm dependencies
+- Builds frontend assets
+- Verifies manifest.json exists
+- Optimizes Laravel for production
+
+## Build Process
+
+### For Development
+```bash
+# Start development environment
+composer dev
+
+# Or for NativePHP development
+composer native:dev
+```
+
+### For Production Build
+
+1. **Prepare the build** (run this before building):
+   ```bash
+   ./build-prepare.sh
+   ```
+
+2. **Build the application**:
+   ```bash
+   php artisan native:build mac arm64
+   ```
+
+3. **Test the build** on a fresh machine:
+   - Copy the built app from `dist/` directory
+   - Install and run - it should work without any Vite errors
+
+## Important Notes
+
+1. **Always run build-prepare.sh** before building for production
+2. **Never commit the `public/hot` file** to version control
+3. **Ensure `public/build/manifest.json` exists** after building
+4. The prebuild commands in `config/nativephp.php` will automatically:
+   - Install npm dependencies
+   - Build frontend assets
+   - Optimize Laravel
+
+## Troubleshooting
+
+If you still see "Vite not found" errors:
+
+1. Check if `public/hot` file exists in the built app - it shouldn't
+2. Verify `public/build/manifest.json` exists in the built app
+3. Ensure the app is running in production mode (`APP_ENV=production`)
+4. Check Laravel logs for any asset-related errors
+
+## Testing on Fresh Machine
+
+When testing on a new machine:
+1. The app should work immediately after installation
+2. No need to run `npm install` or `npm run dev`
+3. All assets should be pre-built and included