Problem
In src/managers/summary-manager.js, catch blocks around file reads (e.g., readFile, readWeekDailySummaries, readMonthWeeklySummaries) catch all errors and return []. This silently hides permission errors, disk-full errors, and other I/O failures — making them look like "no data found."
Expected behavior
Only suppress ENOENT (file not found). Rethrow all other errors so they surface to the caller.
Suggested fix
} catch (err) {
if (err.code === 'ENOENT') return [];
throw err;
}Found by CodeRabbit review on wiggitywhitney/spinybacked-orbweaver-eval#7.
Problem
In
src/managers/summary-manager.js, catch blocks around file reads (e.g.,readFile,readWeekDailySummaries,readMonthWeeklySummaries) catch all errors and return[]. This silently hides permission errors, disk-full errors, and other I/O failures — making them look like "no data found."Expected behavior
Only suppress
ENOENT(file not found). Rethrow all other errors so they surface to the caller.Suggested fix
Found by CodeRabbit review on wiggitywhitney/spinybacked-orbweaver-eval#7.