Skip to content

Fix: Stabilize setup.bat (encoding issues, Python detection, and venv…#51

Merged
Surajkumar5050 merged 1 commit intoSurajkumar5050:mainfrom
Yagasaki7K:main
Feb 28, 2026
Merged

Fix: Stabilize setup.bat (encoding issues, Python detection, and venv…#51
Surajkumar5050 merged 1 commit intoSurajkumar5050:mainfrom
Yagasaki7K:main

Conversation

@Yagasaki7K
Copy link
Contributor

@Yagasaki7K Yagasaki7K commented Feb 27, 2026

🔍 Description

This PR improves the reliability and cross-environment stability of setup.bat.

It resolves execution errors such as:

'is not recognized as an internal or external command'

The root causes were identified as:

  • UTF-8 with BOM encoding conflicts in CMD
  • Fragile Python version parsing logic
  • Unicode characters breaking command parsing
  • Inconsistent virtual environment activation
  • ESC sequence capture instability for ANSI colors

The updated script maintains the original behavior and UX while making execution deterministic and safe across Windows 10/11 environments.


🛠️ Changes Made:

  • Replaced fragile Python version parsing with launcher-based detection (py -3.x)
  • Added fallback detection for default python in PATH
  • Enforced explicit activate.bat call for virtual environment activation
  • Removed Unicode symbols from title to prevent locale parsing issues
  • Stabilized ANSI escape capture for color support
  • Simplified venv recreation logic
  • Improved error handling for environment creation and dependency installation
  • Hardened startup shortcut creation flow

✅ Pre-Push Checklist:

  • I have updated pyproject.toml with any new dependencies.
  • I have run setup.bat and verified the installation is successful.
  • I have verified that start_zyron.bat launches without module errors.
  • I have updated README.md if new commands or features were added.
  • I have linked this PR to an existing Issue (if applicable).

📥 Screenshots/Gifs (If UI changes):

N/A – No UI changes. Script stability improvements only.


Maintaining Zyron stability is our priority!

Issue: #50

@Surajkumar5050
Copy link
Owner

Thank you for the concern & fix :)
checking it now in my local setup...

@Surajkumar5050
Copy link
Owner

checked , & its working so nicely . thanks brother :)

@Surajkumar5050 Surajkumar5050 merged commit 67034bc into Surajkumar5050:main Feb 28, 2026
2 checks passed
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