Skip to content

Fix critical bugs and upgrade compatibility for latest Qoder versions#2

Open
Copilot wants to merge 2 commits intomainfrom
copilot/check-version-support-upgrade
Open

Fix critical bugs and upgrade compatibility for latest Qoder versions#2
Copilot wants to merge 2 commits intomainfrom
copilot/check-version-support-upgrade

Conversation

Copy link
Contributor

Copilot AI commented Mar 3, 2026

Audit of the current codebase revealed several runtime-breaking bugs and missing support for latest Qoder version changes.

Bug fixes

  • NameError in perform_super_deep_cleanup: Referenced undefined safe_keywords; corrected to protected_keywords
  • AttributeError in perform_full_reset: Called perform_login_identity_cleanup(qoder_support_dir) which didn't exist. Added implementation with cookie/token/OAuth cleanup.
  • Duplicate method definitions: is_qoder_running and perform_hardware_fingerprint_reset each defined twice — second (incomplete) definitions silently overrode the first. Removed duplicates.
  • close_qoder() was a no-op: Showed success message without actually terminating the process. Now calls taskkill/pkill.
  • No Linux support in get_qoder_data_dir(): Fell through to macOS path. Now returns ~/.config/Qoder on Linux.
  • Hardcoded system.platform = 'darwin' in telemetry reset: Now detects actual OS.

Latest Qoder compatibility

  • Added telemetry keys: telemetry.firstSessionDate, telemetry.lastSessionDate
  • system.version now uses generate_system_version() instead of inline random range
  • Updated OS version pools: macOS 16 (Tahoe), Windows 11 25H2, Linux 6.x kernels
  • Relaxed PyQt5>=5.15.0 (removed <6.0.0 cap) for Python 3.12+ compatibility

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

- Fix NameError: undefined safe_keywords variable (use protected_keywords)
- Add missing perform_login_identity_cleanup method
- Remove duplicate is_qoder_running and perform_hardware_fingerprint_reset methods
- Add proper Linux path support in get_qoder_data_dir
- Update OS version generation (macOS 16, Windows 25H2, Linux 6.x)
- Add newer telemetry keys (firstSessionDate, lastSessionDate)
- Fix system.platform to use correct OS value instead of hardcoded darwin
- Fix close_qoder to actually terminate the process
- Relax PyQt5 version constraint for Python 3.12+ compatibility
- Update CHANGELOG.md with v1.1.0

Co-authored-by: VoDaiLocz <88762074+VoDaiLocz@users.noreply.github.com>
Copilot AI changed the title [WIP] Check current version support and upgrade if necessary Fix critical bugs and upgrade compatibility for latest Qoder versions Mar 3, 2026
@VoDaiLocz VoDaiLocz marked this pull request as ready for review March 3, 2026 11:01
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