Skip to content

fix(install): harden shell paths and symlink backups - #40

Merged
barbatdev merged 1 commit into
mainfrom
fix/shell-path-hardening
Jul 15, 2026
Merged

barbatdev merged 1 commit into
mainfrom
fix/shell-path-hardening

Conversation

@barbatdev

Copy link
Copy Markdown
Owner

Resultado

Endurece la integración shell y el instalador de cortex-dotfiles a partir del QA posterior al PR #38. El entrypoint propio queda fijado a la ruta administrada por dotfiles y el instalador preserva destinos existentes antes de crear cada symlink.

Cambios

  • Fija el entrypoint de dotfiles en ~/.config/cortex-dotfiles/shell/cortex-dotfiles.zsh.
  • Corrige el uso de la variable local path, que rompía PATH por el comportamiento especial de Zsh.
  • Hace idempotentes las inserciones en PATH al cargar el fragmento más de una vez.
  • Crea y normaliza local/env.zsh con permisos 0600.
  • Respalda archivos, directorios y symlinks mediante mv sobre el destino exacto.
  • Rechaza colisiones de backup y destinos existentes que no hayan sido respaldados.
  • Verifica que el symlink final apunte al origen esperado.
  • Elimina la entrada duplicada de herdr en el instalador.
  • Alinea el README con el ownership observable de este repo y referencia cortex#1259 para el contrato del core.

Verificación

  • ./scripts/check-install-symlinks.sh
  • ./scripts/check-cortex-shell-integration.sh
  • ./scripts/check-navigation-shortcuts.sh
  • ./scripts/check-agent-state.sh
  • ./install.sh --check

Scope

Este PR cubre únicamente cortex-dotfiles. No modifica el producto Cortex ni redefine el ownership de sus variables core.

Derivado del comentario de QA en PR #38.

Revisión

Revisá primero scripts/install-helpers.sh y sus regresiones en scripts/check-install-symlinks.sh; después, la carga shell en zsh/zshrc y zsh/cortex-dotfiles.zsh.

@barbatdev
barbatdev merged commit 678d897 into main Jul 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant