- Abort with a clear log_fatal when the up-front sudo -v fails, instead
of an unexplained errexit later on.
- Move the shelly bootstrap ahead of any gum use, and install gum from
pacman in its own non-fatal transaction: a repo set without gum or a
stale sync db must not take base-devel/git down with it. No -Sy (that
would be a partial upgrade); the gum safety net retries via shelly,
and gum is dropped from the shelly prerequisite list.
- run_step: spinner via gum spin when gum exists, plain otherwise. The
step's output is captured and replayed on failure through stdout, so
makepkg/git errors land in $LOG rather than only on the terminal.
- try_step: one helper for the run / "try it manually" hint /
record_failure blocks (7 copies). The hint is built from the command
that ran, so it cannot drift. makepkg now uses -D (pacman >= 7.0) so
the build step is a single command.
- Sudo keep-alive loop stops its own sleep from a TERM trap, replacing
the racy pkill -P in cleanup; its stderr stays out of the log.
Verified: bash -n + shellcheck clean; try_step/run_step and the loop's
shutdown exercised with stub commands. Not run end to end on Arch.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Route failures through log_error/log_fatal (gum log --level error/fatal
with kitchen timestamps and colour), falling back to plain echo before
the gum bootstrap. log_fatal guards against gum's fatal level exiting 1
under errexit. record_failure uses log_error; $REPORT keeps plain
colour-free lines for the gum pager.
- Fatal level on the four aborting errors (gum install failures, gum
required, fish-shell refusal), error level on the non-fatal headlines
and invalid menu option; the "run it manually" command blocks stay as
plain echo.
- Snapshot the package database (pacman, else rpm) before the first
install step and report the net delta in the INSTALL SUMMARY. apt-only
systems report nothing, by request.
- Ask for sudo once up front and refresh the timestamp in the background
(250s, sudo -n true only) so shelly/makepkg stay un-sudo'd and the user
is never re-prompted mid-run; the loop is killed by a cleanup EXIT trap
that also removes WORK_DIR.
- Replace the numbered text menu with gum choose, wrap sub-script runs in
gum spin, and add sudo -v checkpoints before each privileged menu step.
- Add an INSTALL SUMMARY block (log/report paths, failure count).
- Move the fish-shell refusal after the gum bootstrap so it can always
use gum confirm; drop the ERR trap and hand-rolled select_options now
that errexit + gum cover those paths.
- timestamped .zshrc backups (was: .zshrc.bak overwritten every run, so
only the latest pre-run state survived)
- only remove ~/.oh-my-zsh when it actually contains oh-my-zsh.sh; warn
and leave anything else alone (was: unguarded rm -rf)
- add --needed to the pacman prerequisite install
- drop the empty ~/.fzf/shell/key-bindings.zsh placeholder (nothing
sourced it; fzf integration comes from the oh-my-zsh plugin)
- LICENSE spelling in banners (zsh.sh, installbeta.sh)
- print a 'log out and back in' notice after usermod -aG docker (both
fresh-install and already-installed branches; previously missing)
- remove the unexplained sleep 10 in the already-installed docker branch
- remove 'echo "+ sleep 10" && sleep 10' fake-command lines in failure
branches (record_failure already reports the failure)
- affinity keeps its 10s read-the-instructions delay but announces it
honestly instead of echoing a fake '+ 10 sleep' command
- set -Eeuo pipefail at the top (was only enabled after the menu, so
package installs and shelly bootstrap ran unchecked); preamble audited
for guarded vs unguarded commands ([ -t 1 ] && ... became an if, clear
is now 'clear || true')
- error_handler now reports exit status, line number and the failing
command, and the ERR trap is installed after strict mode so preamble
failures abort instead of printing a message and continuing
- exit 1 when any step was recorded as failed (was: always exit 0)
- single STAMP shared by LOG and REPORT; both created via mktemp
(predictable /tmp log names + tee -a was the same symlink class)
- local script/config fallbacks resolved via BASH_SOURCE (SCRIPT_DIR),
not the caller's CWD; falls back to remote fetch when piped
- $? captured via rc=$? inside each failure branch (idempotent under
reordering) for all record_failure call sites
- ${1:-} for the flag check, ${SHELL:-} in the fish detection,
tty-guarded fish warning prompt
- mktemp -d workspace (trap-cleaned) for the shelly AUR clone, shelly toml
downloads, docker installer and affinity script - no more predictable
/tmp/shelly_install, /tmp/shelly-standard.toml, /tmp/shelly-aur.toml
- download get.docker.sh and AffinityLinuxInstaller.py with curl -fsSL to a
file instead of piping to sh/python3 (the python pipe was missing -f, so an
HTTP 404 body was executed as Python)
- plain numbered menu fallback when gum is unavailable (Fedora has neither
shelly nor apt, so gum was never installed and 'gum choose' failed with a
misleading 'User cancelled installation.')
- $? capture via rc=$? inside the failure branch (safe under reordering)
alvinlolloandClaude Opus 5.5
alvinlollo