fix: docker check usaba -not sobre output vacio en vez de LASTEXITCODE #2

Merged
hermes merged 1 commits from fix/docker-check-lastexitcode into main 2026-09-12 19:33:08 +02:00
Owner

Summary

Detectado en smoke test con PowerShell 7.4.6 (Linux + shim de cmd):

  • if (-not (cmd /c "docker info >NUL 2>&1")) { comprueba el OUTPUT del comando, no su exit code. Con >NUL el output es siempre vacío → -not "" es $true → en Windows el script entraba en la rama de error aunque Docker estuviera OK.

Fix (los 4 packs): capturar a $null y decidir por $LASTEXITCODE:

$null = cmd /c "docker info >NUL 2>&1"
if ($LASTEXITCODE -ne 0) { ... }

Smoke test

  • Parse AST: 4/4 OK (pwsh 7.4.6)
  • Preflight (Docker + compose + RAM con mock Win32): 4/4 OK
  • WordPress: .env generado con 2 contraseñas random distintas, sin placeholder
  • Flujo local completo hasta descarga de modelo: OK
  • Producción del host intacta (0 contenedores creados)
## Summary Detectado en smoke test con PowerShell 7.4.6 (Linux + shim de cmd): - `if (-not (cmd /c "docker info >NUL 2>&1")) {` comprueba el **OUTPUT** del comando, no su exit code. Con `>NUL` el output es siempre vacío → `-not ""` es `$true` → en Windows el script entraba en la rama de error **aunque Docker estuviera OK**. Fix (los 4 packs): capturar a `$null` y decidir por `$LASTEXITCODE`: ```powershell $null = cmd /c "docker info >NUL 2>&1" if ($LASTEXITCODE -ne 0) { ... } ``` ## Smoke test - Parse AST: 4/4 OK (pwsh 7.4.6) - Preflight (Docker + compose + RAM con mock Win32): 4/4 OK - WordPress: .env generado con 2 contraseñas random distintas, sin placeholder - Flujo local completo hasta descarga de modelo: OK - Producción del host intacta (0 contenedores creados)
hermes added 1 commit 2026-09-12 19:32:54 +02:00
if (-not (cmd /c ...)) comprueba el OUTPUT (siempre vacio con >NUL),
no el exit code: en Windows el script entraba en la rama de error
aunque Docker estuviera OK. Detectado en smoke test con pwsh 7.4.6.
hermes merged commit bbe0a639dd into main 2026-09-12 19:33:08 +02:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: d0a1/packs#2