-
Notifications
You must be signed in to change notification settings - Fork 5
Open
Labels
enhancementNew feature or requestNew feature or requesthelp wantedExtra attention is neededExtra attention is needed
Description
Summary
BunKill is currently only tested on macOS. Linux has not been validated but is likely closer to working than Windows since many commands (du, rm, path separators) are shared. This issue tracks testing, validation, and any fixes needed for Linux.
Areas to Test
1. Terminal UI
- Raw mode (
setRawMode) behavior across different terminal emulators (gnome-terminal, konsole, alacritty, kitty, tmux, screen) - ANSI escape codes for colors, cursor movement, and screen clearing
- Keyboard input handling (arrow keys, Page Up/Down, Home/End send different escape sequences on some terminals)
2. File System Operations
getDirectorySize()atsrc/scanner.ts:157usesdu -sk— verify output format matches expectations across distrosdeleteModules()atsrc/scanner.ts:426usesrm -rf— should work but needs validation- Permission error handling — Linux permission codes may differ from macOS
3. Path Handling
- Path separators should be fine (
/is standard on Linux) - Home directory resolution (
os.homedir()) - Hidden directory detection (
.prefix)
4. Opening Directories
src/cli.ts:743usesxdg-open— verify this works across desktop environments (GNOME, KDE, etc.)
Testing Needed
- Test on Ubuntu 22.04+/24.04 (most common)
- Test on Fedora
- Test on Arch Linux
- Test inside tmux and screen
- Test with different terminal emulators
- Run
bunkill --dry-runand verify scan results - Run
bunkill --delete-allon a test directory - Test interactive mode (all keyboard shortcuts)
- Test
--full-scanmode - Test
--exclude-hiddenflag - Verify
du -skoutput parsing works across locales
Proposed Improvements
- Add fallback for
du -skif output format differs (some locales may affect number formatting) - Consider adding
LANG=Cenv when spawningduto ensure consistent output - Add CI testing on Ubuntu (GitHub Actions
ubuntu-latest) - Document minimum Linux requirements in README
Acceptance Criteria
- All features work on Ubuntu 22.04+
- Interactive UI works in common terminal emulators
- CI pipeline includes Linux testing
- README updated with Linux installation and usage instructions
- Known limitations documented (if any)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requesthelp wantedExtra attention is neededExtra attention is needed