Skip to content

A lot of fixes for the app#22

Merged
f1shyondrugs merged 17 commits intoPointerIDE:mainfrom
memuguenstig-lab:feature/settings-fix-build-scripts
Mar 27, 2026
Merged

A lot of fixes for the app#22
f1shyondrugs merged 17 commits intoPointerIDE:mainfrom
memuguenstig-lab:feature/settings-fix-build-scripts

Conversation

@memuguenstig-lab
Copy link

@memuguenstig-lab memuguenstig-lab commented Mar 26, 2026

Summary

This PR fixes the critical settings loading error that occurred when loading settings files and adds comprehensive cross-platform build scripts with automated prerequisite checking.

Fixed Issues

  • ✅ Settings loading error (SyntaxError: Unexpected token '<')
  • ✅ API endpoint accessibility in development mode
  • ✅ Error response format (now returns JSON instead of HTML)
  • ✅ UnicodeDecodeError when loading .env file
  • ✅ Yarn install peer dependency warnings (dmg-builder, electron-builder-squirrel-windows)
  • ✅Added buttons to close the app (why wasnt this in already xd)
  • A lot more stuff isnt in this summary, il maybe add it later

Changes Made

  • backend/backend.py: Added global HTTPException handler returning JSON responses + UTF-8 encoding for load_dotenv()
  • src/services/FileSystemService.ts: Enhanced error handling with content-type validation
  • vite.config.ts: Added API endpoint proxy configuration
  • package.json: Added missing electron-builder peer dependencies
  • App/README.md: Updated with build scripts documentation
  • .env: Fixed encoding to proper UTF-8
  • New files: build.bat, build.ps1, build.sh, BUILD_SCRIPTS_README.md, PULL_REQUEST.md

New Features

  • ✅ Cross-platform build scripts (Windows batch, PowerShell, bash)
  • ✅ Automatic prerequisite checking
  • ✅ Port conflict detection
  • ✅ Enhanced error handling and logging
  • ✅ Debug mode support

Testing

  • ✅ Settings load successfully
  • ✅ API endpoints properly proxied
  • ✅ Error messages are helpful
  • ✅ Build scripts work on all platforms
  • ✅ App starts without UnicodeDecodeError
  • ✅ Yarn install runs without peer dependency warnings

Files Changed

  • backend/backend.py - Exception handler + env loading fix
  • src/services/FileSystemService.ts - Error handling
  • vite.config.ts - API proxy configuration
  • package.json - Added missing dependencies
  • .env - Fixed encoding
  • README.md - Documentation updates
  • 6 new files (build scripts + documentation)

tmpAdmin added 17 commits March 26, 2026 09:51
- Fixed backend HTTPException handling to return JSON instead of HTML
- Enhanced FileSystemService error handling with content-type validation
- Added Vite proxy configuration for API endpoints
- Created automated build scripts (batch, PowerShell, bash) with prerequisite checking
- Fixed: UnicodeDecodeError when loading .env file with wrong encoding
- Added: Explicit UTF-8 encoding for load_dotenv() with fallback
- Fixed: .env file encoding to proper UTF-8
- Enhanced: Environment variable loading robustness
- Added: dmg-builder and electron-builder-squirrel-windows as explicit dev dependencies
- Fixed: Yarn install warnings about missing peer dependencies
- Enhanced: Build dependency resolution for electron-builder
@memuguenstig-lab memuguenstig-lab changed the title Fix: Settings loading error and add build automation scripts A lot of fixes for the app Mar 26, 2026
@f1shyondrugs
Copy link
Collaborator

changes look good, i think - i guess it runs 😭

@f1shyondrugs f1shyondrugs merged commit a236a87 into PointerIDE:main Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants