Secure Error Handling (harperaa) vs Security Reviewer (jeffallan)
Secure Error Handling (harperaa) and Security Reviewer (jeffallan) are both Security skills, so an agent choosing between them is matching on descriptions that overlap. Here is where they actually diverge.
Secure Error Handling
harperaa
A secure-coding pattern for the error path: return generic, environment-aware messages to users while logging the detail server-side, so a stack trace, database error, or file path never hands an attacker a map of your system. It shapes how failures are surfaced — not a scanner that finds the leaks for you, and narrower than a full security review: it does not cover auth, input validation, or the other vulnerability classes, only how errors are reported.
1 scenario in the bank answer to it
Security Reviewer
jeffallan
Performs end-to-end security examinations of code, infrastructure, dependencies and secrets and compiles a prioritized audit report that assigns severity to each finding and lists concrete remediation steps. Unlike tools that only add inline comments, check authentication patterns, or scan a branch diff, this one is aimed at producing a compliance-aware, prioritized audit document rather than a quick pre-merge or single-file review.
1 scenario in the bank answer to it
What is the difference between Secure Error Handling (harperaa) and Security Reviewer (jeffallan)?
- Secure Error Handling (harperaa)
- A secure-coding pattern for the error path: return generic, environment-aware messages to users while logging the detail server-side, so a stack trace, database error, or file path never hands an attacker a map of your system. It shapes how failures are surfaced — not a scanner that finds the leaks for you, and narrower than a full security review: it does not cover auth, input validation, or the other vulnerability classes, only how errors are reported.
- Security Reviewer (jeffallan)
- Performs end-to-end security examinations of code, infrastructure, dependencies and secrets and compiles a prioritized audit report that assigns severity to each finding and lists concrete remediation steps. Unlike tools that only add inline comments, check authentication patterns, or scan a branch diff, this one is aimed at producing a compliance-aware, prioritized audit document rather than a quick pre-merge or single-file review.
Should I use Secure Error Handling or Security Reviewer?
The clearest answer is a situation each one is unambiguously right for. Both of these are drawn from the game's question bank.
Reach for Secure Error Handling when
After last Tuesday's PostgreSQL meltdown, the company's public 500 page leaked the live connection string and internal locations to every visitor, so the CTO demanded a coding convention that shows zero implementation detail to browsers while keeping all diagnostics in a backend log.
The skill shapes the visible side of failures by framing a coding convention: it tells you to give browsers a bland, context-aware notice and to sequester every technical artifact—live connection strings, internal locations, diagnostic output—inside server logs, which exactly resolves the split the CTO asked for. semgrep-rule-creator-trailofbits is the genuinely tempting distractor because the leak feels like a source-level artifact you could catch with a static-analysis rule, but that skill merely writes reusable Semgrep detection rules for specific bug patterns and paired test cases; it never influences what a web framework displays on an HTTP response or where operational logs are directed, so it cannot close the gap between user-facing text and backend retention.
Reach for Security Reviewer when
An auditor is coming and wants one document that grades the whole system — app code, servers, third-party libraries, leaked secrets — with a severity on every issue and a fix for each, ranked worst-first.
security-reviewer-jeffallan produces the end-to-end, severity-ranked audit document spanning code, infrastructure, dependencies and secrets. security-review-owasp and security-review-waybarrios only look at a diff or branch for exploits; security-review-affaan-m is build-time guidance, not an audit. The tell is a ranked document across the whole system.
What they have in common
Both are filed under Security, the axis along which they collide. That shared ground is what makes an agent pick between them on description alone — and what makes it pick wrong.
Nearby comparisons
Reading the difference is not the same as spotting it at speed. That is the game.
Today's session