diff --git a/docs/function-classification-schema.md b/docs/function-classification-schema.md index 57a0039..5bf75a1 100644 --- a/docs/function-classification-schema.md +++ b/docs/function-classification-schema.md @@ -72,3 +72,10 @@ schema's own rollout caught several false positives this way: a piped `read` misread as an interactive prompt, a documented `--yes` flag missed as an escape hatch, and cleanup of a function's own temp output flagged as `destructive` despite the explicit exclusion above. + +`rm` specifically has its own internal flag check (any flag other than +`-r`/`-R`/`--recursive` falls back to `command rm` *inside the shadow +itself*, before it ever touches trash) — a caller writing plain `rm -f` +or `rm -rf` is not bypassing anything itself, the shadow is. Only tag +`bypasses-shadow(rm)` when the caller explicitly writes `command rm` or +`builtin rm`; a bare `rm -f`/`rm -rf` call gets no shadow tag at all. diff --git a/functions/_scrollback_prune_junk.fish b/functions/_scrollback_prune_junk.fish index 4d8b165..022f35b 100644 --- a/functions/_scrollback_prune_junk.fish +++ b/functions/_scrollback_prune_junk.fish @@ -2,7 +2,7 @@ # SPDX-License-Identifier: AGPL-3.0-or-later # CLASSIFICATION -# bypasses-shadow(cat) +# bypasses-shadow(cat,rm), destructive # # SYNOPSIS # _scrollback_prune_junk [dir] @@ -29,7 +29,7 @@ function _scrollback_prune_junk --description 'Remove empty, trivial, and Kitty # Remove any completely empty log file regardless of source for f in $dir/*.log $dir/*.txt test -f $f || continue - not test -s $f; and rm $f + not test -s $f; and command rm -f $f end # Remove any log with only a single meaningful line (e.g. [exited], a lone prompt, or a trivial error) @@ -37,7 +37,7 @@ function _scrollback_prune_junk --description 'Remove empty, trivial, and Kitty test -f $f || continue set -l line_count (command cat $f | sed 's/\x1b\[[0-9;:]*[a-zA-Z]//g' | grep -cv '^\s*$') if test $line_count -le 1 - rm $f + command rm -f $f end end @@ -45,7 +45,7 @@ function _scrollback_prune_junk --description 'Remove empty, trivial, and Kitty for f in $dir/scrollback_*.log $dir/scrollback_*.txt test -f $f || continue if command cat $f | sed 's/\x1b\[[0-9;:]*[a-zA-Z]//g' | grep -q 'Enter the new title for this tab below' - rm $f + command rm -f $f end end end diff --git a/functions/config-settings.fish b/functions/config-settings.fish index e873f09..0478ba6 100644 --- a/functions/config-settings.fish +++ b/functions/config-settings.fish @@ -8,6 +8,9 @@ # __fish_palette, __config_settings_state, __config_settings_apply, # __config_settings_set_value, python3 # +# CLASSIFICATION +# bypasses-shadow(rm) +# # SYNOPSIS # config-settings [-h | --help] # diff --git a/functions/dng2avif.fish b/functions/dng2avif.fish index 4185b8f..c435ca5 100644 --- a/functions/dng2avif.fish +++ b/functions/dng2avif.fish @@ -119,7 +119,7 @@ function dng2avif --description 'Convert DNG raw to 10-bit HDR AVIF' end # Final Cleanup - test -f "$temp_pnm"; and rm "$temp_pnm" + test -f "$temp_pnm"; and rm -f "$temp_pnm" set -l size (stat -c '%s' "$output" | numfmt --to=iec) echo (set_color yellow)"Complete: $output ($size)"(set_color normal) diff --git a/functions/edit.fish b/functions/edit.fish index b312b83..155c208 100644 --- a/functions/edit.fish +++ b/functions/edit.fish @@ -7,6 +7,9 @@ # COMPONENT # aliases/dev-tools # +# CLASSIFICATION +# bypasses-shadow(rm) +# # SYNOPSIS # edit [-V|-t] [-e EDITOR] [-c] [-x TEXT] [-n] [-v|-s] [FILE...] # diff --git a/functions/fc.fish b/functions/fc.fish index a65e230..0825ba9 100644 --- a/functions/fc.fish +++ b/functions/fc.fish @@ -4,6 +4,9 @@ # CATEGORY # 03-editors-and-viewers # +# CLASSIFICATION +# bypasses-shadow(rm) +# # SYNOPSIS # fc [command_prefix] # @@ -50,15 +53,15 @@ function fc --description 'Edit and execute the last command (Bash-style fc)' # Final check if user cleared the file in the editor if test -s $tmpfile set -l command (cat $tmpfile) - rm $tmpfile + command rm -f $tmpfile commandline -r "$command" commandline -f execute else - rm $tmpfile + command rm -f $tmpfile echo "fc: Aborted (empty file)" end else - rm $tmpfile + command rm -f $tmpfile echo "fc: Could not retrieve history" end end diff --git a/functions/key-crypt.fish b/functions/key-crypt.fish index 946b876..5bd9dae 100644 --- a/functions/key-crypt.fish +++ b/functions/key-crypt.fish @@ -7,6 +7,9 @@ # DEPENDENCIES # gpg, tar # +# CLASSIFICATION +# bypasses-shadow(rm), destructive +# # SYNOPSIS # key-crypt [options] [output] # key-crypt -i -o [options] diff --git a/functions/scrub.fish b/functions/scrub.fish index 7444c81..d91c36d 100644 --- a/functions/scrub.fish +++ b/functions/scrub.fish @@ -5,7 +5,7 @@ # 01-file-and-directory # # CLASSIFICATION -# bypasses-shadow(rm), destructive +# uses-shadow(rm), bypasses-shadow(rm), destructive # # SYNOPSIS # scrub [-a] [-d] [-h]