minishell/docs/builtins_fixes.md

Correcciones en Builtins

Este documento resume los fallos detectados en los builtins actuales y la solucion aplicada en codigo.

1) Infraestructura de builtins (include/builtins.h, src/builtins/builtins.c)

Fallo

• Uso inconsistente de tipos (u_int8_t/unsigned char frente a uint8_t).
• set_builtins no comprobaba fallos de registro por builtin (duplicado de clave o insercion en hashmap), pudiendo dejar estado parcial.
• is_builtin podia dereferenciar punteros nulos.

Por que fallaba

• u_int8_t no es el tipo estandar C99 y depende de plataforma/headers.
• Si falla una insercion, la tabla quedaba inicializada parcialmente sin rollback.
• En errores de inicializacion, consultar is_builtin podia romper.

Solucion

• Unificacion de firmas a uint8_t.
• Nuevo helper register_builtin() con validacion tras ft_hashmap_put.
• Si falla cualquier alta: limpieza de minishell->builtins y retorno de error.
• Guardas nulas en is_builtin.

2) cd (src/builtins/cd/cd.c)

Fallo

• cd con demasiados argumentos devolvia 2 (bash devuelve 1).
• cd sin HOME acababa llamando a chdir(NULL).
• El manejo de error usaba comprobaciones invertidas (access) y codigos incorrectos.
• No se actualizaban PWD y OLDPWD tras chdir exitoso.
• cd - (usar OLDPWD) no estaba soportado.

Por que fallaba

• Codigos de salida incompatibles con el comportamiento esperado del shell.
• HOME no definido no se controlaba antes del chdir.
• La logica de access estaba al reves y mezclaba condiciones.
• Variables de entorno del directorio quedaban desincronizadas.
• Faltaba resolver el caso especial de - hacia OLDPWD.

Solucion

• Refactor en resolve_cd_path() para validar argumentos y HOME.
• Retorno EXIT_FAILURE en too many arguments y HOME not set.
• Error de chdir simplificado a perror("minishell: cd") + retorno 1.
• Actualizacion de OLDPWD y PWD mediante getcwd(NULL, 0) + set_env().
• Soporte de cd -: usa OLDPWD, valida OLDPWD not set e imprime el nuevo directorio tras el cambio.

3) exit (src/builtins/exit/exit.c)

Fallo

• Habia un printf de debug en ejecucion real.
• exit <no_numerico> devolvia 2 pero no cerraba el shell.
• exit n m devolvia 2; en bash es 1 y no sale del shell.
• Validacion numerica basada en ft_strisnum sin control de overflow.
• Se mostraba exit incluso en contexto no interactivo.

Por que fallaba

• Debug residual contamina salida.
• Semantica de exit incompleta respecto a bash.
• Valores fuera de rango podian tratarse como validos por conversion directa.
• Mensaje exit debe mostrarse solo en shell interactivo.

Solucion

• Eliminado debug print.
• Nuevo flujo resolve_exit_status():
  • Sin argumentos: usa msh->exit_status.
  • Argumento no numerico o fuera de long: mensaje numeric argument required, msh->exit = true, estado 2.
  • Demasiados argumentos: mensaje de error y estado 1, sin salir.
• Parser numerico propio (get_uint8_from_num + has_overflow) con soporte de signo y control de overflow.
• ft_eputendl("exit") solo si isatty(STDIN_FILENO).

4) pwd (src/builtins/pwd/pwd.c)

Fallo

• Si getcwd fallaba, el builtin devolvia EXIT_SUCCESS.
• Uso de buffer fijo (PATH_MAX) menos robusto para rutas largas.

Por que fallaba

• El shell reportaba exito aunque no pudiera obtener el directorio.
• Un buffer fijo puede truncar o fallar en escenarios de rutas profundas.

Solucion

• Cambio a getcwd(NULL, 0) con memoria dinamica.
• Si falla, perror("minishell: pwd") y retorno EXIT_FAILURE.
• free() del buffer dinamico tras imprimir.

5) echo (src/builtins/echo/echo.c, src/builtins/echo/echo_def.c)

Cambio aplicado

• Ajuste de tipos de retorno auxiliares a uint8_t para mantener consistencia con builtins.h.

Nota

• No se detectaron fallos funcionales criticos adicionales en la logica actual de echo durante esta revision.

6) Builtins faltantes: env, export, unset

Fallo

• Los builtins env, export y unset no estaban implementados ni registrados en set_builtins.

Por que fallaba

• Comandos basicos de shell no existian en la tabla de builtins.
• Cualquier prueba/flujo que dependiera de gestion de variables exportadas fallaba (listar, crear y eliminar variables de entorno).

Solucion

• Nuevos builtins implementados:
  • src/builtins/env/env.c
  • src/builtins/export/export.c
  • src/builtins/unset/unset.c
• Registro en src/builtins/builtins.c (tabla ampliada de 4 a 7 entradas).
• Nuevos prototipos en include/builtins.h.
• Soporte de borrado real de entorno mediante unset_env:
  • Declaracion en include/core.h
  • Implementacion en src/variables/environment_unset.c

7) Comportamiento aplicado en los nuevos builtins

env

• Si recibe argumentos, devuelve error (minishell: env: too many arguments) y estado 1.
• Sin argumentos, lista KEY=VALUE de las variables de entorno actuales.

export

• export NAME=VALUE: crea/actualiza la variable.
• export NAME: crea/actualiza con valor vacio (NAME=) para que aparezca en env.
• Identificadores invalidos (1A=2, etc.) devuelven error not a valid identifier y estado 1.
• Sin argumentos, reutiliza el listado de env.

unset

• Elimina variables validas del entorno en memoria.
• Identificadores invalidos devuelven error not a valid identifier y estado 1.
• Si el identificador es valido y no existe, no falla (comportamiento shell).

Validacion realizada

• norminette ejecutado sobre los archivos modificados: OK.
• Build completa no ejecutable en este entorno por falta de acceso de red al clonado de librerias (github.com), por lo que no se pudo validar runtime del binario en esta sesion.