fix(cli): UnicodeEncodeError on Windows without PYTHONUTF8=1

[Wolfvin]

Bug

On Windows, running any command that outputs Unicode characters (e.g. → in trace output) without PYTHONUTF8=1 crashes with:

UnicodeEncodeError: 'charmap' codec can't encode character '→' in position 440: character maps to <undefined>

Root cause: Windows default stdout encoding is cp1252 (or similar), which cannot encode → and other Unicode symbols used in path strings like sidepanel.ts → auth/google-auth-cache.ts.

Fix

In codelens.py (or the CLI entry point), force UTF-8 stdout/stderr at startup:

import sys, io
if sys.stdout.encoding != 'utf-8':
    sys.stdout = io.TextIOWrapper(sys.stdout.buffer, encoding='utf-8', errors='replace')
    sys.stderr = io.TextIOWrapper(sys.stderr.buffer, encoding='utf-8', errors='replace')

This should be done unconditionally at the top of the entry point — do not rely on the user setting PYTHONUTF8=1. Users on Windows should not need an env var to get working output.

Definition of Done

• All commands produce correct output on Windows without PYTHONUTF8=1
• → and other Unicode chars in trace paths render correctly or are replaced with ASCII fallback (->) without crashing
• No change in behavior on Linux/macOS

Labels

type: bug-fix exec: parallel

[Wolfvin]

🔄 Taking this issue

Mengerjakan issue ini -- 2026-07-03T08:34:41Z

Catatan: tidak ada issue open dengan label exec:parallel saat ini (semua sudah closed atau direlabel). BOS Wolfvin sudah confirm hal sama di #178. Aku ambil #179 karena clean, isolated, Windows-only.

[Wolfvin]

✅ PR siap review

PR: #183 -- fix(cli): force UTF-8 stdout/stderr at startup, no PYTHONUTF8 needed (closes #179)
Branch: fix/issue-179-windows-unicode-encode
Summary: Windows users no longer need PYTHONUTF8=1 to run codelens commands. scripts/codelens.py now wraps sys.stdout/sys.stderr as UTF-8 TextIOWrapper at import time, before any user-code imports. No-op on Linux/macOS and under pytest capsys. Old stream's buffer is detach()-ed before the new wrapper is built so GC of the old wrapper doesn't close the underlying buffer.
Tests: 7 passed (new TestForceUtf8Stdio class in tests/test_codelens.py), 75 passed in full test_codelens.py file, 0 failed. Pre-existing failures in test_adr.py / test_codelensignore.py / test_confidence.py / test_formatters_phase2.py / test_universal_grammar_loader.py verified to also fail on origin/main (environment-specific: missing tree_sitter, stale fixtures, schema_version field not yet implemented) — not regressions from this PR.