Agent skill

Idiomatic Code Review

by eric-tramel in eric-tramel/moraine

Reviews a pull request for idiomatic implementation: use of the language, standard library, ecosystem conventions and the repository's own patterns.

Apache-2.0Auto-check passedDevelopment

Install Idiomatic Code Review

skills CLI
$ npx skills add eric-tramel/moraine --skill code-review-idomatic -a claude-code

Project install by default; add -g for ~/.claude/skills/.

GitHub CLI
$ gh skill install eric-tramel/moraine code-review-idomatic --agent claude-code

Project scope by default; add --scope user for a personal install. Needs GitHub CLI 2.90.0 or later (public preview).

Manual copy
$ git clone --depth 1 https://github.com/eric-tramel/moraine.git skills-src && mkdir -p .claude/skills && cp -r skills-src/plugins/moraine-dev/skills/code-review-idomatic .claude/skills/code-review-idomatic && rm -rf skills-src

Use ~/.claude/skills/ instead of .claude/skills for a personal install. The folder must contain SKILL.md.

Claude Code skills documentation · loads skills from .claude/skills/

Facts

Skill name
code-review-idomatic
GitHub stars
117
Token cost
~469 tokens
SKILL.md length
160 words
Files
2
Skills in repo
19
Repo updated
First seen
Licence
Apache-2.0

At a glance

Reviews a pull request for idiomatic implementation: use of the language, standard library, ecosystem conventions and the repository's own patterns.

  • Checking a PR for hand-rolled code where the standard library would do
  • SKILL.md covers Persona, Focus, Non-Goals and Output
  • Instructions only: no scripts, shell commands, URLs or credentials in SKILL.md
  • Reviewing Rust changes for idiomatic Result, Option and Path handling

What it does

This skill sets up a review persona, CodeReviewIdomatic, that asks whether code is written the way an experienced maintainer would expect. It prefers standard library and established crate APIs over custom ones, checks error handling, ownership, async, parsing, serialization, path handling and collection use against language norms, and flags surprising control flow, bespoke parsers, hand-rolled encoders or custom synchronization when standard tools would be clearer.

For Rust it favors clear Result and Option flow, the question mark operator, iterator and slice APIs, standard Path and PathBuf handling and existing workspace helpers. It also checks that new code follows the style of nearby modules. Purely aesthetic rewrites and overriding a deliberate local pattern are out of scope. Findings lead the output, ordered by review value, each with a priority, file and line and the standard alternative. With no findings, the review says the code fits the conventions.

When your agent uses it

  • Checking a PR for hand-rolled code where the standard library would do
  • Reviewing Rust changes for idiomatic Result, Option and Path handling
  • Verifying that new code matches the style of nearby modules

Example prompts

  • “Review this PR for idiomatic Rust and flag any custom parsers the standard library could replace.”
  • “Check whether the new path handling uses Path and PathBuf the way this repo does.”
  • “Do an idiom review of the branch and list the standard alternatives.”

What it can do on your machine

Read from SKILL.md and the folder at commit 2fb4aec. It shows what the files ask for, not the result of running them.

  • Tool permissions

    Pre-approves nothing: there is no allowed-tools line, so your agent's usual permission prompts apply.

    From allowed-tools in the SKILL.md frontmatter.

  • Runs code

    No scripts in the folder and no shell commands in SKILL.md (its code samples are markdown).

    From the folder's file list and the shell code blocks in SKILL.md.

  • Network

    No URLs in SKILL.md.

    From URLs in SKILL.md, links to its own repository left out.

  • Credentials

    Names no API keys, tokens, secrets or passwords.

    From names ending in _API_KEY, _TOKEN, _SECRET, _KEY or _PASSWORD in SKILL.md.

Context cost

Idiomatic Code Review loads about 469 tokens when it runs. Until then it costs about 76 tokens; SKILL.md has 160 words of instructions outside code blocks.

Always · name and description, kept in context so the agent knows when to use it
~76
When it runs · the whole SKILL.md, loaded when a task matches
~469

Estimates: characters ÷ 4, the usual rule of thumb; real counts depend on the model's tokenizer. Scripts and assets cost tokens only if the agent reads them.

Safety

Auto-check passed

The automated check found no risky patterns in SKILL.md.

Automated static check — not a guarantee. Review scripts before installing. It scans the text of SKILL.md for risky patterns (piping downloads into a shell, reading credential files, hidden Unicode, destructive commands); files beside SKILL.md are not scanned.

SKILL.md

The full file from eric-tramel/moraine at commit 2fb4aec, republished under its Apache-2.0 licence (© eric-tramel). 160 words, ~469 tokens.

Download SKILL.mdSave it as .claude/skills/code-review-idomatic/SKILL.md (or your agent's skills folder). This skill also uses 1 other file; get the full folder from GitHub.
name
code-review-idomatic
description
Review a PR through the CodeReviewIdomatic persona. Use when asked to check whether code is idiomatic for the implementation language, follows standard library and ecosystem conventions, avoids unnecessarily novel implementations, and matches established patterns in this repository.

CodeReviewIdomatic

Persona

Review for idiomatic implementation. Ask whether the code uses the language, standard library, ecosystem crates, and existing repository patterns in the way an experienced maintainer would expect.

Focus

  • Prefer standard library and established crate APIs over custom implementations.
  • Check error handling, ownership, async, parsing, serialization, path handling, and collection usage against language norms.
  • For Rust, favor clear Result/Option flow, ?, iterator or slice APIs where they improve clarity, standard Path/PathBuf handling, and existing workspace helpers.
  • Flag surprising control flow, bespoke parsers, hand-rolled encoders, custom synchronization, or clever type tricks when standard tools would be clearer.
  • Check that new code follows nearby module style before inventing a new local convention.

Non-Goals

Do not request idiomatic rewrites that are purely aesthetic. Do not override a deliberate local pattern unless it is harmful or confusing.

Output

Lead with findings ordered by review value. Include the standard alternative:

markdown
- [P2] Use `Path::strip_prefix` instead of string slicing paths
  `crates/example/src/source.rs:88`
  The current implementation assumes `/` separators and byte offsets. `Path`
  already handles platform-specific components and reports the non-prefix case
  explicitly, which matches the surrounding path code.

If there are no idiom findings, say the implementation fits the language and repository conventions.

© eric-tramel, Apache-2.0. Rendered from Markdown: HTML in the file is shown as text, images as links, and headings moved down two levels. Raw file

Files

SKILL.md and 1 other file in plugins/moraine-dev/skills/code-review-idomatic of eric-tramel/moraine.

  • SKILL.md
  • agents/openai.yaml

Open the folder on GitHubat commit 2fb4aec

Compare with similar skills

Idiomatic Code Review next to the 5 skills that share the most tags, products or categories with it. Stars are the repository's; “used in” counts other GitHub owners with a copy.

Idiomatic Code Review compared with similar skills
SkillStarsUsed inTokensAuto-checkLicenceRepo updated
Idiomatic Code Review this skilleric-tramel/moraine117—~469Automated safety check: PassApache-2.0
Cross-Language Coding Standardszereight/gitlab-mcp2k1 repos~1.4kAutomated safety check: PassMIT
Code Review Excellenceandrew-yangy/gru-ai155—~1.7kAutomated safety check: NotesMIT
Code Qualitywaybarrios/opencode-power-pack533—~1.8kAutomated safety check: PassMIT
WooCommerce Code Reviewwoocommerce/woocommerce11k3 repos~1.1kAutomated safety check: PassCustom licence
Skill Doli Code ReviewDolibarr/dolibarr7.7k1 repos~1.1kAutomated safety check: PassMIT

Similar skills

  • Shared reference for naming, function size, complexity and error handling rules that reviewer agents apply across TypeScript, Python, Go, Rust, Java, C# and Swift.

    2k GitHub starsUsed in 1 repo~1.4k tokens
    DevelopmentAuto-check passed
  • Code Review Excellence

    andrew-yangy/gru-ai

    Provides comprehensive code review guidance for React 19, Vue 3, Rust, TypeScript, Java, Python, and C/C++.

    155 GitHub stars~1.7k tokensUpdated 7 mo ago
    DevelopmentAuto-check: notes
  • Code Quality

    waybarrios/opencode-power-pack

    Agents should invoke this skill for code reviews, linting/formatting setup, maintainability checks, complexity concerns, warning cleanup, coding standards, or quality gates in Rust, TypeScript…

    533 GitHub stars~1.8k tokensUpdated yesterday
    DevelopmentAuto-check passed
  • WooCommerce Code Review

    woocommerce/woocommerce

    Reviews WooCommerce code changes against the project's standards, flagging backend PHP architecture, naming, documentation, data integrity and testing violations.

    11k GitHub starsUsed in 3 repos~1.1k tokens
    DevelopmentAuto-check passed
  • Skill Doli Code Review

    Dolibarr/dolibarr

    Reviews Dolibarr PHP code for compliance with coding standards and security best practices, and fixes identified issues.

    7.7k GitHub starsUsed in 1 repo~1.1k tokens
    DevelopmentAuto-check passed
  • Dignified Python Standards

    docling-project/docling

    Applies opinionated production Python conventions chosen by the project's Python version: modern type syntax, pathlib, explicit checks and interface guidance.

    68k GitHub stars~1.5k tokensUpdated today
    DevelopmentAuto-check passed

More from eric-tramel/moraine

All 19 skills in this repo
  • Moraine Release Publisher

    eric-tramel/moraine

    Cuts and publishes a Moraine release end to end: version bump PR, tag, GitHub release notes, workflow check and PyPI package verification.

    117 GitHub stars~2.5k tokensUpdated 2 days ago
    Auto-check passed
  • Moraine Release Notes

    eric-tramel/moraine

    Rewrites a moraine GitHub release body into a usage-focused format with install block, what's new, platform table, upgrade notes and one deduplicated changelog.

    117 GitHub stars~2.7k tokensUpdated 2 days ago
    Auto-check passed
  • Multi-Persona Code Review

    eric-tramel/moraine

    Coordinates a delegated review of a Moraine PR or local change by seven focused reviewer subagents, merges their findings and follows up on the fixes.

    117 GitHub stars~1.3k tokensUpdated 2 days ago
    Auto-check passed
  • Elegance-Focused Code Review

    eric-tramel/moraine

    Looks at a pull request for the smallest good design, hunting for simplifications and architectural moves that remove whole classes of problems.

    117 GitHub stars~473 tokensUpdated 2 days ago
    Auto-check passed
  • Code Review Security Review

    eric-tramel/moraine

    Review a PR through the CodeReviewSecurityReview persona. An agent skill from eric-tramel/moraine.

    117 GitHub stars~497 tokensUpdated 2 days ago
    Auto-check passed
  • Crystallize

    eric-tramel/moraine

    Turn a rough feature, bug, refactor, architecture, documentation, or operations idea into a ready-to-implement local plan file.

    117 GitHub stars~1.6k tokensUpdated 2 days ago
    Auto-check passed

Works with

Categories

Questions about Idiomatic Code Review

What does Idiomatic Code Review do?

Reviews a pull request for idiomatic implementation: use of the language, standard library, ecosystem conventions and the repository's own patterns. This skill sets up a review persona, CodeReviewIdomatic, that asks whether code is written the way an experienced maintainer would expect. It prefers standard library and established crate APIs over custom ones, checks error handling, ownership, async, parsing, serialization, path handling and collection use against language norms, and flags surprising control flow, bespoke parsers, hand-rolled encoders or custom synchronization when standard tools would be clearer.

When should I use Idiomatic Code Review?

Idiomatic Code Review fits situations like: checking a PR for hand-rolled code where the standard library would do; reviewing Rust changes for idiomatic Result, Option and Path handling; verifying that new code matches the style of nearby modules.

How do I install Idiomatic Code Review in Claude Code?

Run `npx skills add eric-tramel/moraine --skill code-review-idomatic -a claude-code`. Or copy the skill folder (plugins/moraine-dev/skills/code-review-idomatic in eric-tramel/moraine) into .claude/skills/code-review-idomatic in your project. Claude Code loads it when a task matches its description.

How do I install Idiomatic Code Review in Codex?

Run `npx skills add eric-tramel/moraine --skill code-review-idomatic -a codex`. Or copy the skill folder (plugins/moraine-dev/skills/code-review-idomatic in eric-tramel/moraine) into .agents/skills/code-review-idomatic in your project. Codex loads it when a task matches its description.

Can I use Idiomatic Code Review in Cursor, Gemini CLI or GitHub Copilot?

Cursor, Gemini CLI, GitHub Copilot and OpenCode also load SKILL.md folders. With the skills CLI, run `npx skills add eric-tramel/moraine --skill code-review-idomatic -a cursor` (or -a gemini-cli, github-copilot or opencode for the others). To copy it by hand, put the folder in .cursor/skills/code-review-idomatic, .gemini/skills/code-review-idomatic, .github/skills/code-review-idomatic and .opencode/skills/code-review-idomatic in your project.

What does Idiomatic Code Review need to run?

SKILL.md names no scripts, command-line tools or credentials: Idiomatic Code Review is instructions for the agent only.

Does Idiomatic Code Review access the network?

SKILL.md contains no URLs. Any network use would come from the scripts or tools the agent runs. This is read from the text; nothing was executed.

Is Idiomatic Code Review safe to install?

Our automated static check of SKILL.md found no risky patterns, such as piping downloads into a shell, reading credential files or hidden Unicode. It is not a guarantee. Review the folder before installing.

What licence does Idiomatic Code Review use?

Idiomatic Code Review is published under the Apache-2.0 licence (the repository's licence). It allows redistribution, so the full SKILL.md is shown on this page.

How many tokens does Idiomatic Code Review use?

About 469 tokens (SKILL.md is roughly 1.9k characters). Agents keep only the skill's name and description in context until a task matches; then they load SKILL.md in full.

What are the alternatives to Idiomatic Code Review?

Skills that share tags, products or a category with Idiomatic Code Review: Cross-Language Coding Standards (zereight/gitlab-mcp, 2k stars), Code Review Excellence (andrew-yangy/gru-ai, 155 stars), Code Quality (waybarrios/opencode-power-pack, 533 stars) and WooCommerce Code Review (woocommerce/woocommerce, 11k stars). The comparison table on this page puts their stars, adoption, token cost, safety result and licence side by side.

Who maintains Idiomatic Code Review?

eric-tramel (a GitHub user) maintains it in eric-tramel/moraine, which has 117 GitHub stars. The repository holds 19 skills in this directory. The repository was last updated on October 5, 2026.

Source: eric-tramel/moraine on GitHub. Facts on this page come from the repository at the commit we read; the author's words are quoted as theirs.