Skip to content

Commit 7a0b4cf

Browse files
committed
fix: move logs to ~/Library/Logs/mole, add system idle assets cleanup
- Move log files from ~/.config/mole/ to ~/Library/Logs/mole/ per macOS convention. Fixes #569. - Add safe_sudo_find_delete for /Library/Application Support/ com.apple.idleassetsd/Customer/ screensaver videos. Closes #570. - Update tests to reflect new log file paths.
1 parent 3980971 commit 7a0b4cf

File tree

4 files changed

+14
-8
lines changed

4 files changed

+14
-8
lines changed

lib/clean/user.sh

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -427,6 +427,12 @@ clean_support_app_data() {
427427
safe_find_delete "$idle_assets_dir" "*" "$support_age_days" "f" || true
428428
fi
429429

430+
# Clean system-level idle/aerial screensaver videos (macOS re-downloads as needed).
431+
local sys_idle_assets_dir="/Library/Application Support/com.apple.idleassetsd/Customer"
432+
if sudo test -d "$sys_idle_assets_dir" 2> /dev/null; then
433+
safe_sudo_find_delete "$sys_idle_assets_dir" "*" "$support_age_days" "f" || true
434+
fi
435+
430436
# Clean old aerial wallpaper videos (can be large, safe to remove).
431437
safe_clean ~/Library/Application\ Support/com.apple.wallpaper/aerials/videos/* "Aerial wallpaper videos"
432438

lib/core/log.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,9 @@ fi
2121
# Logging Configuration
2222
# ============================================================================
2323

24-
readonly LOG_FILE="${HOME}/.config/mole/mole.log"
25-
readonly DEBUG_LOG_FILE="${HOME}/.config/mole/mole_debug_session.log"
26-
readonly OPERATIONS_LOG_FILE="${HOME}/.config/mole/operations.log"
24+
readonly LOG_FILE="${HOME}/Library/Logs/mole/mole.log"
25+
readonly DEBUG_LOG_FILE="${HOME}/Library/Logs/mole/mole_debug_session.log"
26+
readonly OPERATIONS_LOG_FILE="${HOME}/Library/Logs/mole/operations.log"
2727
readonly LOG_MAX_SIZE_DEFAULT=1048576 # 1MB
2828
readonly OPLOG_MAX_SIZE_DEFAULT=5242880 # 5MB
2929

tests/cli.bats

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ EOF
206206
[ "$status" -eq 0 ]
207207
MOLE_OUTPUT="$output"
208208

209-
DEBUG_LOG="$HOME/.config/mole/mole_debug_session.log"
209+
DEBUG_LOG="$HOME/Library/Logs/mole/mole_debug_session.log"
210210
[ -f "$DEBUG_LOG" ]
211211

212212
run grep "Mole Debug Session" "$DEBUG_LOG"
@@ -228,7 +228,7 @@ EOF
228228
run env HOME="$HOME" TERM="xterm-256color" MOLE_TEST_MODE=1 MO_DEBUG=1 "$PROJECT_ROOT/mole" clean --dry-run
229229
[ "$status" -eq 0 ]
230230

231-
DEBUG_LOG="$HOME/.config/mole/mole_debug_session.log"
231+
DEBUG_LOG="$HOME/Library/Logs/mole/mole_debug_session.log"
232232

233233
run grep "User:" "$DEBUG_LOG"
234234
[ "$status" -eq 0 ]

tests/core_common.bats

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ setup() {
5050
stdout_output="$(HOME="$HOME" bash --noprofile --norc -c "source '$PROJECT_ROOT/lib/core/common.sh'; log_info '$message'")"
5151
[[ "$stdout_output" == *"$message"* ]]
5252

53-
local log_file="$HOME/.config/mole/mole.log"
53+
local log_file="$HOME/Library/Logs/mole/mole.log"
5454
[[ -f "$log_file" ]]
5555
grep -q "INFO: $message" "$log_file"
5656
}
@@ -64,13 +64,13 @@ setup() {
6464
[[ -s "$stderr_file" ]]
6565
grep -q "$message" "$stderr_file"
6666

67-
local log_file="$HOME/.config/mole/mole.log"
67+
local log_file="$HOME/Library/Logs/mole/mole.log"
6868
[[ -f "$log_file" ]]
6969
grep -q "ERROR: $message" "$log_file"
7070
}
7171

7272
@test "rotate_log_once only checks log size once per session" {
73-
local log_file="$HOME/.config/mole/mole.log"
73+
local log_file="$HOME/Library/Logs/mole/mole.log"
7474
mkdir -p "$(dirname "$log_file")"
7575
dd if=/dev/zero of="$log_file" bs=1024 count=1100 2> /dev/null
7676

0 commit comments

Comments
 (0)