Swallowed Errors / Missing Error Handling. Use this skill whenever diffs may introduce reliability issues on all, especially in all. Actively look for: Catching exceptions without logging, re-throwing, or handling them — causing silent failures. Also includes missing try/catch around operations... and report findings with high severity expectations and actionable fixes.
reliabilityhighallallCatching exceptions without logging, re-throwing, or handling them — causing silent failures. Also includes missing try/catch around operations that can fail.
pass/return.catch(() => {}) on promisestry {
processPayment(order);
} catch (PaymentException e) {
// silently swallowed
}
order.setStatus("COMPLETED"); // runs even if payment failed
Expected finding: High — Swallowed exception. Payment failure is silently ignored, order marked as completed. Log the error and handle the failure state.
async function syncData() {
await fetch('/api/sync').catch(() => {}); // errors silently ignored
console.log('Sync complete'); // logs success even on failure
}
Expected finding: High — Error suppressed. Failed sync appears successful. Handle or propagate the error.
try {
processPayment(order);
} catch (PaymentException e) {
logger.error("Payment failed for order {}", order.getId(), e);
order.setStatus("PAYMENT_FAILED");
throw new OrderProcessingException("Payment failed", e);
}
Why it's correct: Error is logged, state is updated, and exception is propagated.