Repository navigation
feat(process): runner acotado para instalación Codex - #58
Merged
Merged
Conversation
This was referenced Oct 10, 2026
egdev6
marked this pull request as ready for review
October 10, 2026 20:59
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 únicotype: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.
-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, nomain.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.Chain Context
feat/codex-install-trackerfeat/codex-install-02-backendRollback: 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.