localGPT
localGPT copied to clipboard
Bug/fix windows npm utf8 issues
๐ Description
Fixes windows os related error messages on windows resulting from utf-8 issues and not finding the npm on windows system After these updates, run_system.py works in windows 11
Fixes #(issue number)
๐ฏ Type of Change
- [X ] ๐ Bug fix (non-breaking change which fixes an issue)
- [ ] โจ New feature (non-breaking change which adds functionality)
- [ ] ๐ฅ Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] ๐ Documentation update
- [ ] ๐งช Test improvements
- [ ] ๐ง Code refactoring
- [ ] ๐จ UI/UX improvements
๐งช Testing
Test Environment
- [ ] Tested with Docker deployment
- [ ] Tested with direct Python deployment
- [ ] Tested on macOS
- [ ] Tested on Linux
- [ ] Tested on Windows
Test Cases
- [ ] All existing tests pass
- [ ] New tests added for new functionality
- [ ] Manual testing completed
- [ ] System health check passes
# Commands used for testing
python system_health_check.py
python run_system.py --health
# Add any specific test commands here
๐ Checklist
Code Quality
- [ ] Code follows the project's coding standards
- [ ] Self-review of the code completed
- [X ] Code is properly commented
- [ ] Type hints added (Python)
- [ ] No console.log statements left in production code
Documentation
- [ ] Documentation updated (if applicable)
- [ ] API documentation updated (if applicable)
- [ ] README updated (if applicable)
- [ ] CONTRIBUTING.md guidelines followed
Dependencies
- [ ] No new dependencies added, or new dependencies are justified
- [ ] requirements.txt updated (if applicable)
- [ ] package.json updated (if applicable)
๐ฅ๏ธ Screenshots (if applicable)
Add screenshots to help reviewers understand the changes.
๐ Performance Impact
Describe any performance implications:
- [ ] No performance impact
- [ ] Performance improved
- [ ] Performance may be affected (explain below)
๐ Migration Notes
If this is a breaking change, describe what users need to do:
- [ ] No migration needed
- [ ] Migration steps documented below
๐ Additional Notes
Any additional information that reviewers should know.