Skip to content

feat(process): runner acotado para instalación Codex - #58

Merged
egdev6 merged 1 commit into
feat/codex-install-trackerfrom
feat/codex-install-01-process
Oct 10, 2026
Merged

egdev6 merged 1 commit into
feat/codex-install-trackerfrom
feat/codex-install-01-process

Conversation

@egdev6

@egdev6 egdev6 commented Oct 10, 2026

Copy link
Copy Markdown
Owner

Related issue / Issue relacionada

Refs #56. El cierre de la funcionalidad completa queda en el tracker #57 (Closes #56), después de integrar todas las unidades y validar Windows.

ODD: no aplica; trabajo directo autorizado por el propietario. Issue con status:approved; esta PR lleva único type:feature.

Scope / Alcance

Unidad 1/4: runner de subprocess reutilizable, dependencias de plataforma ya bloqueadas y declaración del módulo. Sin instalador, IPC nuevo ni interfaz todavía.

  • Captura máxima64KiB por stream, drenaje del exceso, plazo y reaping limitados; sin joins incondicionales.
  • Contención de árbol propio: process groups Unix y Job Objects Windows mediante bindings tipados; no nightly ni llamadas no documentadas.
    -6fixtures reales genéricos de procesos: salida, errores, límites, timeout, entorno y pipes heredados. No npm/Codex/probes Node reales.

461 líneas (4 archivos). Una separación coherente mantuvo pruebas con runner; excede400 por61, sin comprimir ni eliminar comentarios/tests. El repositorio no exige etiqueta de excepción. Los archivos copiados coinciden con la fuente final revisada; la integración actual solo agrega la declaración de módulo.

Coordination / Coordinación

ODD: N/A; no cambio de asignaciones ni toma de roadmap.

Contributor / Contribuidor: @egdev6

Acuerdo: propietario autorizó cadena con tracker/commit/push/PR borrador; Windows lo valida él. Base inmediata feat/codex-install-tracker, no main.

Acceptance and validation / Aceptación y validación

  • cargo test --offline --locked:66passed/0failed/0ignored, incluidos6fixtures reales Linux.
  • cargo build --offline --locked:PASS.14warnings dead_code esperados en esta unidad: el módulo aún no tiene consumidor de producción; la unidad backend lo conecta. No se afirma build sin warnings.
  • git diff --cached --check:PASS; alcance4files/461adiciones. ASSESS medium/runtime-large: autoverificación requerida, realizada; no verificador separado requerido. Revisión nativa parcial no ejecutada ni heredada automáticamente del conjunto.
  • No ejecutado: Windows compile/runtime/Job Objects. Validación a cargo del propietario. No CLI real, OAuth, firma, DOM ni credenciales.
  • Límites: asignación del Job posterior al spawn no atómica; pueden escapar descendientes/persistir readers desconectados. Límite por ejecución, no garantía global de recursos.
  • Bloqueos: revisión de esta unidad y Windows; el tracker sigue no-merge. No CI verde afirmado sin checks observados.

Chain Context

Campo Valor
Cadena Codex instalación asistida
Tracker #57, borrador/no-merge
Posición 1de4
Base feat/codex-install-tracker
Depende de Marcador tracker #57
Seguimiento feat/codex-install-02-backend
Presupuesto 461/400 orientativo
Inicio Main20ef + marcador documental
Fin Runner probado y dependencias; sin instalador conectado
main
└── #57 tracker (borrador/no-merge)
    └── 📍 esta PR:01-process
        └──02-backend
            └──03-ui
                └──04-docs

Rollback: quitar esta unidad y sus dependientes; conserva el módulo completo con sus tests. No fusionar tracker antes de revisar/integrar la cadena y recibir validación Windows. Release/firma/secretos requieren permiso aparte.

@egdev6 egdev6 added the type:feature New functionality label Oct 10, 2026
@egdev6
egdev6 marked this pull request as ready for review October 10, 2026 20:59
@egdev6
egdev6 merged commit 1b8edd6 into feat/codex-install-tracker Oct 10, 2026
@egdev6
egdev6 deleted the feat/codex-install-01-process branch October 10, 2026 20:59
@egdev6
egdev6 restored the feat/codex-install-01-process branch October 10, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:feature New functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant