[P1][Windows] Background/index worker falha com WinError 6 ao herdar stdin inválido

[wesleysimplicio]

Problema observado

No Windows com Python 3.14, os caminhos de worker do simplicio-mapper falham antes de iniciar o processo filho quando o processo pai tem stdin capturado/fechado, como ocorre em pytest e em alguns hosts Codex/PowerShell.

Reprodução passo a passo

1. Usar checkout do mapper em Windows.
2. Executar:

   python -m pytest tests/python/test_cli.py::CliTest::test_background_index_reports_pid_and_log -q

3. Observar falha antes de criar o worker:
   OSError: [WinError 6] Identificador inválido
4. O traceback aponta para subprocess.Popen -> _get_handles -> _winapi.DuplicateHandle.
5. Os mesmos sintomas aparecem em scan --sync e scan assíncrono, impedindo o teste do worker com timeout.
6. O erro não é específico do repositório mapeado: testes que chamam git init também podem reproduzi-lo sob o mesmo host de teste, mas o mapper agrava o impacto porque seus caminhos de background dependem de Popen.

Impacto

• O mapper não inicia deep pass/background worker em Windows sob pytest/hosts sem stdin herdável.
• O Loop não consegue obter mapper receipt nem avançar do gate de survey.
• A falha ocorre antes de lock/timeout/worker state, portanto fica sem receipt de domínio.
• A regressão bloqueia a validação de [P0][Index Lock Regression] Recuperar lock órfão e impedir deep pass preso em deep_running #201/[bug] bounded scan times out on simplicio-code with no progress or fallback #230 e o drain de feat: piloto 1 do autoresearch — otimizar as heurísticas do encoder TOON por loop evolutivo (eval = round-trip gate + tokens medidos nos artefatos reais) #151 no Agent.

Plano de correção

1. Tornar o worker não-interativo, passando stdin explícito subprocess.DEVNULL.
2. Manter stdout/stderr no log estruturado e não herdar handles do host.
3. Adicionar teste Windows/pytest para background, sync e async.
4. Verificar que timeout/kill ainda recupera lock e escreve receipt terminal.
5. Validar pacote instalado além do checkout in-tree.

Critérios de aceite

• index --background --json inicia em Windows com stdin fechado/capturado.
• scan --sync --json inicia o worker e retorna envelope completo ou timeout estruturado.
• scan --json assíncrono inicia e reaping não deixa handles inválidos.
• Testes passam sob pytest com stdin capturado e em execução direta.
• Nenhum stdin do usuário é consumido pelo worker.
• O erro de criação do processo retorna reason code e não deixa lock órfão.
• Smoke com wheel/CLI instalado passa no Windows.

[wesleysimplicio]

Implementação em andamento — timeout bounded + Windows worker hardening

A branch local agora contém uma correção focada, ainda não publicada:

• scan --sync inicia o index em subprocesso controlável em vez de executar deep pass indefinidamente dentro da CLI.
• --timeout N encerra o worker e descendentes, retorna exit code 1 e grava phase=timeout, failure_reason=scan_timeout, PID, elapsed bound e exit code.
• O lock é recuperado somente após validar que o owner morreu; lock vivo não é removido.
• status --json converte timeout em estado terminal failed e preserva reason code/next action.
• O worker recebe stdin=subprocess.DEVNULL, eliminando WinError 6 DuplicateHandle em hosts Windows com stdin inválido.

Evidência local

• 16 testes focados passaram: sync complete/lock guard, background, async, timeout terminal e lock recovery.
• python -m py_compile dos módulos alterados passou.
• git diff --check passou.
• Repro real contra simplicio-agent com scan --json --sync --timeout 8: exit 1 em ~8s, phase=timeout, failure_reason=scan_timeout, lock recuperado e status terminal.
• Suíte combinada: 81 passed, 3 falhas pré-existentes de git init/WinError 6 sob o host Windows; o worker mapper específico foi corrigido e os testes de background passaram.

A correção ainda precisa de PR/CI e validação em wheel instalado antes de fechar qualquer issue.

[wesleysimplicio]

Fresh regression-suite evidence — 2026-07-17

Runtime execution of the focused mapper suite after the worktree-scope change:

python -m pytest tests/python/test_mapper_parse.py tests/python/test_cli.py tests/python/test_cli_coverage.py tests/python/test_lock_recovery.py -q
95 passed, 3 failed in 21.32s

The three failures are unrelated to the new scope assertion and reproduce the existing Windows handle problem:

• MapperArtifactsTest::test_build_artifacts_emits_deterministic_degraded_shape
• MapperArtifactsTest::test_git_status_marks_untracked_files_inside_new_dirs
• CliTest::test_index_refreshes_when_dirty_file_changes_again

All fail at test setup on subprocess.run(["git", "init"], capture_output=True) with OSError: [WinError 6] Identificador inválido from subprocess._winapi.DuplicateHandle. The worker launch path itself uses stdin=subprocess.DEVNULL and the new regression test passes. This remains an environment/test-harness finding that must be isolated or fixed before the full DoD can be claimed.