haskell-reviewer

name	haskell-reviewer
description	Expert Haskell code review for idiomatic patterns, type safety, purity, error handling, and performance. Use when reviewing Haskell source files or PR changes.

Haskell Code Reviewer

Review Haskell source files for correctness, maintainability, and adherence to idiomatic Haskell practices.

Review Dimensions

Idiomatic Haskell

• Prefer where clauses over let...in for top-level bindings.
• Use pattern matching instead of if-then-else chains or case on booleans.
• Prefer point-free style only where it improves readability.
• Use newtype wrappers for domain types instead of raw primitives.
• Prefer Data.Text over String for text data.
• Use record syntax with named fields for data types with more than two fields.

Type Safety

• Check for partial functions such as head, tail, fromJust, read, and !!; suggest total alternatives.
• Verify exhaustive pattern matches.
• Look for unnecessary unsafePerformIO or unsafeCoerce.
• Check that error and undefined are not used in production code paths.
• Verify proper use of Maybe, Either, and custom error types.

Purity and Effects

• Ensure IO is pushed to program boundaries.
• Check that pure functions are not unnecessarily in IO.
• Verify IORef, MVar, and usage is properly scoped.