Skip to content

feat(codex): backend de instalación consentida y detección - #59

Merged
egdev6 merged 5 commits into
mainfrom
feat/codex-install-02-backend
Oct 10, 2026
Merged

egdev6 merged 5 commits into
mainfrom
feat/codex-install-02-backend

Conversation

@egdev6

@egdev6 egdev6 commented Oct 10, 2026

Copy link
Copy Markdown
Owner

Related issue / Issue relacionada

Refs #56. El tracker #57 cierra la issue al integrar toda la funcionalidad y validar Windows.

ODD: N/A, petición directa autorizada. Issue status:approved; PR con único type:feature.

Scope / Alcance

Unidad2/4: backend de instalación asistida y detección actualizada, sobre el runner de #58.

  • Estado IPC de solo lectura y comando de instalación sin argumentos, exclusivo de Ajustes; guardia de concurrencia antes del diálogo nativo anclado a la ventana.
  • Node/npm existentes>=22 de arquitectura compatible, paquete/registro fijos; Node ejecuta el script npm emparejado con verbo install, sin shell.
  • Prefijo/caché propios, npmrc vacío y entorno limitado; scripts npm ignorados. Verificar binario nativo y versión antes de promover stage; no sobrescribir prefix existente ni borrar uno corrupto automáticamente.
  • Detección mediante PATH del proceso/registro fresco y ubicaciones gestionadas, sin probes en sondeo pasivo.
  • Instalar no guarda configuración ni inicia OAuth. Registro de comandos/estado y status async para no bloquear el hilo UI.

3files,1746adiciones+12eliminaciones=1758líneas. Es la mejor unidad coherente de backend sin separar pruebas de su política; sigue sobre400. No se comprime código ni elimina comentarios/tests, ni se solicita una etiqueta size:exception que el destino no usa.

Excluye nueva UI, documentación final, instalación Node/elevación/cambios globales npm/PATH y release. El backend aún no tiene botón nuevo en esta unidad.

Coordination / Coordinación

ODD N/A; sin cambios de asignación.

Contributor / Contribuidor: @egdev6

Propietario autorizó cadena con tracker y realizará Windows. Depende de #58; base inmediata feat/codex-install-01-process, no main. Los fuentes Rust completos coinciden byte a byte con el conjunto revisado anteriormente.

Acceptance and validation / Aceptación y validación

  • cargo test --offline --locked:94passed/0failed/0ignored, incluyendo26setup y6runner; cargo build --offline --locked:PASS, sin warnings observados en esta unidad.
  • Fixture real UNIX con Node existente y JS inerte (ruta con espacios) observa argv de npm_command; no ejecuta npm real/Codex/Node --version/-p process.arch ni descarga herramientas. Fakes estrictos protegen programa Node emparejado, script y verbo install; flags fijos y controles negativos cubiertos.
  • Evidencia RED anterior: producción vieja17passed/9failed (.js como ejecutable y verbo faltante), seguida de GREEN94; no se atribuye Windows a estos tests.
  • DiffcheckPASS, alcance3files1758líneas; Rust exactamente idéntico al árbol de fuente aprobado. ASSESSmedium/runtime-large exige autoverificación realizada, no verificador separado. No se hereda automáticamente aprobación nativa del conjunto a esta snapshot parcial.
  • Revisión previa del conjunto resolvió4findings sobre las mismas2causas raíz con178líneas dentro plan180 y fue aprobada/reconocida; evidencia contextual, no CI parcial.
  • Incidente previo informado: un verificador ejecutó node--version una vez sin autorización. No se repite el probe en esta unidad.
  • No ejecutado: Windows compilación/runtime, instalación npm/Codex real, OAuth, credenciales/firma/DOM. Dueño validará Windows antes de integrar.
  • Bloqueos: revisión de unidad, UI/docs pendientes y Windows. No afirmar CIverde sin checks observados ni merge-ready.

Chain Context

Campo Valor
Tracker #57 borrador/no-merge
Posición 2de4
Base feat/codex-install-01-process
Depende de #58
Sigue feat/codex-install-03-ui
Presupuesto 1758/400 orientativo
Inicio Runner y dependencias de #58
Fin Backend consentido/detección registrado, sin botón nuevo
main
└── #57 tracker
    └── #58 01-process
        └── 📍 esta PR:02-backend
            └──03-ui
                └──04-docs

Rollback por unidad requiere retirar dependientes. Los límites de Job Object post-spawn/descendientes escapados/readers desconectados y captura por ejecución permanecen documentados. No merge del tracker sin cadena completa y Windows. No release ni operaciones de secretos/firma.

@egdev6 egdev6 added the type:feature New functionality label Oct 10, 2026
docs(codex): guías bilingües y cierre de cadena de instalación
@egdev6
egdev6 deleted the branch main October 10, 2026 20:59
@egdev6 egdev6 closed this Oct 10, 2026
feat(settings): controles persistentes de instalación Codex
@egdev6 egdev6 reopened this Oct 10, 2026
@egdev6
egdev6 changed the base branch from feat/codex-install-01-process to main October 10, 2026 21:13
@egdev6
egdev6 marked this pull request as ready for review October 10, 2026 21:16
@egdev6
egdev6 merged commit fe52b9d into main Oct 10, 2026
@egdev6
egdev6 deleted the feat/codex-install-02-backend branch October 10, 2026 21:16
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