clojure-review

Name: clojure-review
Rating: 5 (44733 reviews)
Author: metabase

Review Clojure and ClojureScript code changes for compliance with Metabase coding standards, style violations, and code quality issues. Use when reviewing pull requests or diffs containing Clojure/ClojureScript code.

TL;DR

Review Clojure/ClojureScript code for style, quality, and compliance with Metabase standards.

What Is It

Review code for style guide compliance
Flag style violations and quality issues
Ensure public API and test coverage meet standards
Check naming, documentation, and organization rules

Key Features

Check Metabase Clojure style guide compliance
Flag style violations only (no praise for good work)
No comments on missing parentheses (caught by linter)
Public vars must have docstrings with purpose, inputs, outputs
Use kebab-case for query params, nouns for functions
Functions >20 lines should be split
API endpoints must have tests and proper schemas
No hardcoded driver names in driver logic

How to Use

1.Scan changes using quick checklist
2.Flag style violations and missing docs
3.Ensure naming, organization, and API conventions are followed
4.Use pattern matching table to identify common issues

Guardrails & Gotchas

Do not praise trivial changes or correct style
Do not say 'looks good' or confirm correctness
Do not comment on missing parentheses
Do not use `:clj-kondo/ignore` for module patterns
Do not ignore missing author/date in TODOs

GitHub Stats

Author

@metabase

Repository

metabase/metabase

Stars

44733

Explore More Skills

move-code-quality

@davila7

Analyzes Move language packages against the official Move Book Code Quality Checklist. Use this skill when reviewing Move code, checking Move 2024 Edition compliance, or analyzing Move packages for best practices. Activates automatically when working with .move files or Move.toml manifests.

12.4k

1.1k

1w ago

clojure-review

Explore More Skills

move-code-quality

e2e-testing-patterns

testing-anti-patterns