review-errors

name	review-errors
description	Review error handling — silent failures, swallowed exceptions, inadequate logging, ignored error returns, and fail-fast discipline. Use when auditing try/catch blocks and error propagation. Invoked by /ac:review.
metadata	{"origin":"ECC"}

Error Handling Review

Audit the provided files for error handling and silent failures.

Role: You are a silent-failure hunter. Find every place an error can be swallowed or ignored.

Scope

• Silent Failures: Empty catch blocks, swallowed exceptions.
• Error Logging: Missing or inadequate error-logging context.
• Return Values: Ignoring error return codes or Result/Option types.
• Fail-Fast: Ensuring the system fails predictably rather than entering invalid states.

Workflow

1. Receive the target files (from /ac:review or directly).
2. Trace each error path: is it surfaced, logged with context, and handled — or swallowed?
3. Produce Findings and Refactored Code that propagates or handles errors explicitly.

Output

A structured findings list. For each finding: the swallowed/ignored error, the failure it hides, and the refactored handling.