`e965d9f`

respect HISTFILE env var at startup, disable in tests

Shell: check HISTFILE env var before loading history file. Previously
always loaded ~/.fortsh_history regardless of HISTFILE setting.

Tests: set HISTFILE=/dev/null in PTY environment to prevent history
file from polluting per-test history. Fixes history test 5 (up arrow
at boundary recalled commands from previous sessions).

Also restore clear_buffer in non-marker sync path to fix history
tests 8, 11, 12.

Authored by

espadonne 1 month ago

SHA: e965d9feff4efbf01eb57bccbfc4d9050dbbe12c
Parents: 58c634e
Tree: 4464160

2 changed files

Status	File	+	-
M	`src/fortsh.f90`	11	2
M	`tests/interactive/fortsh_pty.py`	1	0

src/fortsh.f90modified

      ! Set HISTCONTROL for history management
      call set_histcontrol(shell%histcontrol)
 -    ! Load command history from file
 -    if (len_trim(shell%histfile) > 0) then
 +    ! Check HISTFILE env var override before loading
 +    block
 +      character(len=:), allocatable :: histfile_env
 +      histfile_env = get_environment_var('HISTFILE')
 +      if (len(histfile_env) > 0) then
 +        shell%histfile = histfile_env
 +      end if
 +    end block
++
 +    ! Load command history from file (skip if /dev/null)
 +    if (len_trim(shell%histfile) > 0 .and. trim(shell%histfile) /= '/dev/null') then
        call load_history_from_file(trim(shell%histfile), shell%histsize)
      end if
    end if

tests/interactive/fortsh_pty.pymodified

          # Note: Completion is NOT disabled here - tests can use Tab completion
          env["FORTSH_MINIMAL_ECHO"] = "1"
          env["FORTSH_TEST_MODE"] = "1"
 +        env["HISTFILE"] = "/dev/null"  # Don't load/save history from file
          # Use /dev/null for clean testing unless specified
          if rc_file is not None: